Skip to content

tentacle: qa/multisite: switch to boto3#67318

Merged
batrick merged 5 commits intoceph:tentaclefrom
adamemerson:wip-74526-tentacle
Mar 3, 2026
Merged

tentacle: qa/multisite: switch to boto3#67318
batrick merged 5 commits intoceph:tentaclefrom
adamemerson:wip-74526-tentacle

Conversation

@adamemerson
Copy link
Contributor

Original tracker: https://tracker.ceph.com/issues/74526
Backport tracker: https://tracker.ceph.com/issues/74886

Show available Jenkins commands

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

@github-actions github-actions bot added this to the tentacle milestone Feb 11, 2026
@adamemerson adamemerson requested a review from smanjara February 11, 2026 21:47
@adamemerson
Copy link
Contributor Author

jenkins test docs

@batrick
Copy link
Member

batrick commented Feb 12, 2026

jenkins test make check

@batrick
Copy link
Member

batrick commented Feb 12, 2026

jenkins test docs

@bluikko
Copy link
Contributor

bluikko commented Feb 12, 2026

The docs job should succeed if you can rebase onto fresh tentacle that has commit 5f53ebc.

@batrick
Copy link
Member

batrick commented Feb 13, 2026

retest this please

Shilpa Jagannath added 5 commits February 13, 2026 15:33
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit c84efd9)
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 47ad27c)
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 3438add)

Conflicts:
	src/test/rgw/rgw_multi/tests.py
 - Whitespace and other drift
 - Tentacle doesn't sync object locks
 - `test_suspended_delete_marker_incremental_sync` is a new test but
   not a new behavior.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 90eb061)
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
…tools.py.

tools.py should eventually be removed once all other dependent tests move to boto3

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit 3d08847)

Conflicts:
	src/test/rgw/rgw_multi/tests.py
 - Assert changes
 - We still don't replicate object locks in tentacle
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
@batrick
Copy link
Member

batrick commented Feb 13, 2026

trivial rebase to kick test

@batrick
Copy link
Member

batrick commented Feb 19, 2026

@cbodley @adamemerson how about this one?

@cbodley
Copy link
Contributor

cbodley commented Feb 24, 2026

backport probably needs #67083. we're seeing lots of errors like "An error occurred (AccessDenied) when calling the CreateBucket operation" on main because the boto3 changes merged before that fix

@adamemerson
Copy link
Contributor Author

@batrick @yuriw This should be good to merge.

@cbodley
Copy link
Contributor

cbodley commented Mar 3, 2026

backport probably needs #67083. we're seeing lots of errors like "An error occurred (AccessDenied) when calling the CreateBucket operation" on main because the boto3 changes merged before that fix

did you not see lots of these failures in the rgw/multisite suite?

also note that this pr on main caused a run-tox-qa regression fixed in #67486

@smanjara
Copy link
Contributor

smanjara commented Mar 3, 2026

backport probably needs #67083. we're seeing lots of errors like "An error occurred (AccessDenied) when calling the CreateBucket operation" on main because the boto3 changes merged before that fix

did you not see lots of these failures in the rgw/multisite suite?

the original PR #66358 that introduced CreateBucket issue hasn't been merged into tentacle yet. so the fix in #67083 may not be necessary.

@batrick
Copy link
Member

batrick commented Mar 3, 2026

@adamemerson can you link to the test results you were reviewing?

@batrick batrick merged commit 70f8415 into ceph:tentacle Mar 3, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants