New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
qa/upgrade: conditionally disable update_features tests #41514
Conversation
will test them through teuthology |
fc7cf53
to
44267c0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
src/test/librbd/test_notify.py
Outdated
print("rebuild object map") | ||
image.rebuild_object_map() | ||
assert(not image.is_exclusive_lock_owner()) | ||
assert((image.flags() & RBD_FLAG_OBJECT_MAP_INVALID) == 0) | ||
|
||
if 'RBD_DISABLE_UPDATE_FEATURES' in os.environ: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually it should be not in os.environ
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trociny you are right, apologies, silly of me, updated.
44267c0
to
3ce5703
Compare
with the recent support for async rbd operations from pacific+ when an older client(non async support) goes on upgrade, and simultaneously interacts with a newer client which expects the requests to be async, experiences hang; considering the return code for request completion to be acknowledgement for async request, which then keeps waiting for another acknowledgement of request completion. this if happens should be a rare only when lockowner is an old client and should be deferred if compatibility issues arises. see also: 5412304(octopus) Signed-off-by: Deepika <dupadhya@redhat.com>
3ce5703
to
9c0b239
Compare
with the recent support for async rbd operations from pacific+ when an
older client(non async support) goes on upgrade, and simultaneously
interacts with a newer client which expects the requests to be async,
experiences hang; considering the return code for request completion to
be acknowledgement for async request, which then keeps waiting for
another acknowledgement of request completion.
see also: 5412304(octopus)
Signed-off-by: Deepika dupadhya@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox