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

pacific: ceph-volume: fix drive-group issue that expects the batch_args to be a string #51209

Merged
merged 1 commit into from May 2, 2023

Conversation

guits
Copy link
Contributor

@guits guits commented Apr 25, 2023

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


backport of #49586
parent tracker: https://tracker.ceph.com/issues/59203

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

The drive-group expects the batch_args to be a string,
however in the current version it is passed as a list
of one element, thus calling the first item of the list solves the issue.

Fixes: https://tracker.ceph.com/issues/59203

Signed-off-by: Mohan Sharma <mohan7427@gmail.com>
(cherry picked from commit 7602a99)
@guits guits requested a review from a team as a code owner April 25, 2023 08:10
@guits guits added this to the pacific milestone Apr 25, 2023
@adk3798
Copy link
Contributor

adk3798 commented May 1, 2023

https://pulpito.ceph.com/adking-2023-04-25_18:48:20-orch:cephadm-wip-adk4-testing-2023-04-25-1313-pacific-distro-default-smithi/

reruns of failed jobs: https://pulpito.ceph.com/adking-2023-05-01_11:51:02-orch:cephadm-wip-adk4-testing-2023-04-25-1313-pacific-distro-default-smithi/

After reruns, 4 failures:

  • 2 tests still hit the remote connection issue that caused most of the 24 failures in the first run and happens before the test itself actually starts.
  • Other 2 failures were on the dashboard test_e2e test that has been known t fail in pacific runs

Overall, nothing to block merging

@guits guits merged commit 8bebdee into ceph:pacific May 2, 2023
8 checks passed
@guits guits deleted the wip-59256-pacific branch May 2, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants