2024-06-16T14:25:18.990Z | <Leonid Usov> A [suspicious error](https://jenkins.ceph.com/job/ceph-pull-requests/136911/consoleFull#-757550002a811ea2-3e7b-466b-84b4-d13df7e35809) from the test make check when building reef backports
```/home/jenkins-build/build/workspace/ceph-pull-requests/src/seastar/src/core/reactor_backend.cc:1215:15: error: no member named 'features' in 'io_uring'
if (~ring.features & required_features) {
~~~~ ^
/home/jenkins-build/build/workspace/ceph-pull-requests/src/seastar/src/core/reactor_backend.cc:1216:147: error: no member named 'features' in 'io_uring'
maybe_throw(std::runtime_error(fmt::format("missing required io_ring features, required 0x{:x} available 0x{:x}", required_features, ring.features)));```
Found this issue <https://tracker.ceph.com/issues/64587>, should it be backported to reef? |
2024-06-16T16:12:34.817Z | <Adam Kraitman> Hey, I have re-enabled tags on <http://tracker.ceph.com|tracker.ceph.com> we had that on the old redmine version but I had to install the new plugin version to make it work on the new redmine version: https://files.slack.com/files-pri/T1HG3J90S-F0790HR7KHN/download/image.png |
2024-06-16T18:05:08.023Z | <Leonid Usov> i noticed, they look nice on the right side |
2024-06-16T19:01:59.667Z | <Ronen Friedman> Hi all. I have now spent a full day trying to get a PR to pass 'make check' in Jenkins.
The repeated failure is with run-tox-mgr, which I know has happened before.
Can/should this test be bypassed in CI? It is a major waste of time... |
2024-06-16T20:46:23.845Z | <Leonid Usov> if you made changes to python code, then tox error may be yours to fix |
2024-06-16T20:46:30.534Z | <Leonid Usov> do you have a link to the failed run? |
2024-06-16T20:54:31.281Z | <Leonid Usov> Is it this one?
```cephadm/configchecks.py:94:6: E204 whitespace after decorator '@'
cephadm/module.py:3017:10: E204 whitespace after decorator '@'
cephadm/tests/test_cephadm.py:982:6: E204 whitespace after decorator '@'
3 E204 whitespace after decorator '@'``` |
2024-06-16T20:54:48.376Z | <Leonid Usov> I have seen it earlier today on a different PR |
2024-06-16T20:55:12.531Z | <Leonid Usov> Is it [this one](https://jenkins.ceph.com/job/ceph-pull-requests-arm64/58023/consoleFull#-910469020e840cee4-f4a4-4183-81dd-42855615f2c1)?
```cephadm/configchecks.py:94:6: E204 whitespace after decorator '@'
cephadm/module.py:3017:10: E204 whitespace after decorator '@'
cephadm/tests/test_cephadm.py:982:6: E204 whitespace after decorator '@'
3 E204 whitespace after decorator '@'``` |
2024-06-16T20:55:52.148Z | <Leonid Usov> PR <https://github.com/ceph/ceph/pull/57865> ? |
2024-06-16T20:56:56.859Z | <Leonid Usov> I see the history. Same error. Are you fully rebased over the latest main?
```flake8: commands[0] /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr> flake8 --config=tox.ini alerts balancer cephadm cli_api crash devicehealth diskprediction_local hello iostat localpool nfs orchestrator prometheus rbd_support rgw selftest smb
cephadm/configchecks.py:94:6: E204 whitespace after decorator '@'
cephadm/module.py:3017:10: E204 whitespace after decorator '@'
cephadm/tests/test_cephadm.py:982:6: E204 whitespace after decorator '@'
3 E204 whitespace after decorator '@'
flake8: exit 1 (2.56 seconds) /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr> flake8 --config=tox.ini alerts balancer cephadm cli_api crash devicehealth diskprediction_local hello iostat localpool nfs orchestrator prometheus rbd_support rgw selftest smb pid=1112066
flake8: FAIL ✖ in 7.8 seconds``` |
2024-06-16T21:01:31.961Z | <Leonid Usov> hmm.. OK, this may be the builder issue. Apparently, the code it’s barking at is quite old, and it indeed has a space after the `@` in decorators.
Could it be that flake8 got updated and now has a new rule?: https://files.slack.com/files-pri/T1HG3J90S-F07897BUH2R/download/image.png |
2024-06-16T21:01:31.964Z | <Leonid Usov> https://files.slack.com/files-pri/T1HG3J90S-F078C13RH35/download/image.png |