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

qa/suites/rados/cephadm/upgrade: fix iscsi upgrade test #40034

Merged
merged 2 commits into from
Mar 11, 2021
Merged

qa/suites/rados/cephadm/upgrade: fix iscsi upgrade test #40034

merged 2 commits into from
Mar 11, 2021

Conversation

liewegas
Copy link
Member

@liewegas liewegas commented Mar 11, 2021 •
edited
Loading

  • start upgrade from 15.2.9 when testing iscsi (many earlier versions either don't support iscsi or are buggy... but this one works)
  • fix the mgr service spec to have the correct count

@liewegas liewegas requested a review from a team as a code owner March 11, 2021 13:38
@liewegas liewegas force-pushed the cephadm-fix-iscsi-upgrade branch from f4783ce to de3a1a7 Compare March 11, 2021 14:50
@github-actions github-actions bot added the tests label Mar 11, 2021
@liewegas liewegas force-pushed the cephadm-fix-iscsi-upgrade branch 2 times, most recently from 17ace2e to fdec98b Compare March 11, 2021 18:41
@liewegas liewegas force-pushed the cephadm-fix-iscsi-upgrade branch 2 times, most recently from 0a53d0a to ac8af15 Compare March 11, 2021 21:11
@liewegas
Copy link
Member Author

liewegas commented Mar 11, 2021 •
edited
Loading

https://pulpito.ceph.com/sage-2021-03-11_21:14:03-rados:cephadm:upgrade-master-distro-basic-smithi/

@liewegas liewegas force-pushed the cephadm-fix-iscsi-upgrade branch from ac8af15 to 6ccc3d8 Compare March 11, 2021 21:49
@liewegas liewegas merged commit 1d590a0 into ceph:master Mar 11, 2021
@liewegas liewegas mentioned this pull request Mar 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebastian-philipp sebastian-philipp sebastian-philipp approved these changes

Assignees
No one assigned
Labels
cephadm core needs-qa tests wip-sage-testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@liewegas @sebastian-philipp @tchaikov