Botond Dénes

Results 853 comments of Botond Dénes

v7: * Add more comments to the validator. * Add more comments to the unit test. * Split unit test into separate file (`test_sstable_validation.py`).

v7.1: * Fixed typo in test name.

v8: * conftest.py: bump scope of `scylla_path` to `session`.

> CI state **`FAILURE`** - https://jenkins.scylladb.com/job/scylla-master/job/scylla-ci/1043/ Known CI issue: https://github.com/scylladb/scylladb/issues/13211. Re-kicking.

@tchaikov did we finish migrating the tests to the native tools? If so, this issue can be closed.

> But I think @denesb suggested in his PR that some of these packages are needed not because of static linking. Turns out `-ludev` is only needed to link statically...

> To ensure that I did not break anything, I run the following tests locally. > > Run in `tests/nodetool`: > > * 1: `pytest --nodetool=scylla .` > > *...

@pwrobelse when pushing a new version of a PR, please also add a comment with a short changelog, describing what has changed in the new version, compared with the previous...

@lkshminarayanan when pushing a new version, please produce a changelog, describing what changed compared to the previous version. This helps reviewer(s) focus on just the changed areas, speeding up the...