Skip to content
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.

Sign up for GitHub

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

Jump to bottom

octopus: qa: check mounts attribute in ctx #45633

Merged
merged 1 commit into from May 31, 2022
Merged

octopus: qa: check mounts attribute in ctx #45633

merged 1 commit into from May 31, 2022

Conversation

joscollin
Copy link
Member

@joscollin joscollin commented Mar 25, 2022 •
edited

check mounts attribute in ctx before unmounting in bark()

Fixes: https://tracker.ceph.com/issues/49898
(cherry picked from commit b4b932f)

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@github-actions github-actions bot added this to the octopus milestone Mar 25, 2022
@joscollin joscollin added cephfs Ceph File System tests labels Mar 25, 2022
@joscollin
Copy link
Member Author

jenkins retest this please

@joscollin
Copy link
Member Author

jenkins test make check

@joscollin joscollin requested review from a team and vshankar March 25, 2022 08:54
@joscollin
Copy link
Member Author

jenkins test docs

@joscollin
Copy link
Member Author

jenkins retest this please

@joscollin
Copy link
Member Author

jenkins retest this please

neesingh-rh
neesingh-rh approved these changes Apr 28, 2022
View reviewed changes
Copy link
Contributor

@neesingh-rh neesingh-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,good to have a check!!

@joscollin
qa: check mounts attribute in ctx
546ce3a 
check mounts attribute in ctx before unmounting in bark()

Fixes: https://tracker.ceph.com/issues/49898
Signed-off-by: Jos Collin <jcollin@redhat.com>
(cherry picked from commit b4b932f)
@ljflores
Copy link
Contributor

Rados suite results:

http://pulpito.front.sepia.ceph.com/?branch=wip-yuri3-testing-2022-05-12-1609-octopus

Failures, unrelated:
1. https://tracker.ceph.com/issues/52316
2. https://tracker.ceph.com/issues/51076
3. https://tracker.ceph.com/issues/55723 --> new Tracker; this failure is persistent but unrelated to this batch.
4. https://tracker.ceph.com/issues/50222
5. https://tracker.ceph.com/issues/55774 --> new Tracker; this failure is persistent but unrelated to this batch.
6. https://tracker.ceph.com/issues/55776 --> opened a new one for this; failed twice but then passed. Appears unrelated to the PRs in this batch.

Details:
1. qa/tasks/mon_thrash.py: _do_thrash AssertionError len(s['quorum']) == len(mons) - Ceph - RADOS
2. "wait_for_recovery: failed before timeout expired" during thrashosd test with EC backend. - Ceph - RADOS
3. octopus: dashboard failures due to timed-out or failed connections - Ceph - Mgr - Dashboard
4. osd: 5.2s0 deep-scrub : stat mismatch - Ceph - RADOS
5. octopus: prometheus, and selftest failures due to timed-out or failed connections - Ceph - RADOS
6. octopus: map exx had wrong cluster addr - Ceph - RADOS

Untracked & unrelated:
6832783 - Unable to locate package libcephfs1 - same job passed in a rerun. Looks like an unrelated infrastructure issue.

@yuriw yuriw merged commit 630a85c into ceph:octopus May 31, 2022
@joscollin joscollin deleted the wip-55054-octopus branch June 1, 2022 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rishabh-d-dave rishabh-d-dave rishabh-d-dave approved these changes

@kotreshhr kotreshhr kotreshhr approved these changes

@neesingh-rh neesingh-rh neesingh-rh approved these changes

@nmshelke nmshelke nmshelke approved these changes

@vshankar vshankar Awaiting requested review from vshankar

Assignees
No one assigned
Labels
cephfs Ceph File System needs-qa tests wip-yuri3-testing
Projects
None yet
Milestone
octopus
7 participants
@joscollin @ljflores @rishabh-d-dave @kotreshhr @neesingh-rh @nmshelke @yuriw