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.

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: ignore evicted client warnings #38422

Merged
merged 2 commits into from Feb 5, 2021
Merged

octopus: qa: ignore evicted client warnings #38422

merged 2 commits into from Feb 5, 2021

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm commented Dec 3, 2020 •
edited

backport tracker: https://tracker.ceph.com/issues/48285

backport of #37528
parent tracker: https://tracker.ceph.com/issues/47689

this backport was staged using ceph-backport.sh version 15.1.1.389
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

@batrick @smithfarm
mgr: log reason for respawn
1e56f9d 
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit b835b07)
@smithfarm smithfarm added this to the octopus milestone Dec 3, 2020
@batrick @smithfarm
qa: ignore evicted client warnings for nautilus upgrades
f0438ee 
Nautilus monitors do not note the client sessions in the mgrmap. So when
we upgrade the warnings are unavoidable.

Fixes: https://tracker.ceph.com/issues/47689
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 2163c60)

Conflicts:
	qa/suites/upgrade/nautilus-x-singleton/1-install/nautilus.yaml
	qa/suites/upgrade/nautilus-x/parallel/1-ceph-install/nautilus.yaml
	qa/suites/upgrade/nautilus-x/stress-split/1-ceph-install/nautilus.yaml
- be5f525 is not being backported
@smithfarm smithfarm removed the request for review from sebastian-philipp December 3, 2020 11:51
@smithfarm
Copy link
Contributor Author

@batrick AFAICT 2ee9365 has not been backported to octopus, so I used log-whitelist instead of log-ignorelist. But I see that you have log-ignorelist here:

qa/suites/fs/basic_functional/tasks/cap-flush.yaml
3:    log-ignorelist:

(This is the only instance of log-ignorelist in octopus qa/suites tree...)

@batrick
Copy link
Member

batrick commented Dec 4, 2020

@batrick AFAICT 2ee9365 has not been backported to octopus, so I used log-whitelist instead of log-ignorelist. But I see that you have log-ignorelist here:

qa/suites/fs/basic_functional/tasks/cap-flush.yaml
3:    log-ignorelist:

(This is the only instance of log-ignorelist in octopus qa/suites tree...)

I think that's correct.

@batrick batrick removed their request for review December 7, 2020 21:15
@yuriw
Copy link
Contributor

yuriw commented Jan 19, 2021

wip-yuri5-testing-2021-01-19-0810-octopus

@yuriw yuriw added the core label Feb 4, 2021
@yuriw yuriw merged commit 8190273 into ceph:octopus Feb 5, 2021
@smithfarm smithfarm deleted the wip-48285-octopus branch February 6, 2021 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@neha-ojha neha-ojha neha-ojha approved these changes

@tchaikov tchaikov Awaiting requested review from tchaikov

@batrick batrick Awaiting requested review from batrick

Assignees
No one assigned
Labels
cephfs Ceph File System core needs-qa octopus-batch-1 tests wip-yuri5-testing
Projects
None yet
Milestone
octopus
4 participants
@smithfarm @batrick @yuriw @neha-ojha