Skip to content

rgw: remove boto in test bucket notification#66745

Closed
thuvh wants to merge 1 commit intoceph:mainfrom
thuvh:fix_boto_v2
Closed

rgw: remove boto in test bucket notification#66745
thuvh wants to merge 1 commit intoceph:mainfrom
thuvh:fix_boto_v2

Conversation

@thuvh
Copy link
Contributor

@thuvh thuvh commented Dec 26, 2025

Fixes: 73663

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands

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

@thuvh thuvh requested a review from a team as a code owner December 26, 2025 11:06
@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

14 similar comments
@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@ceph-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@thuvh thuvh marked this pull request as draft December 26, 2025 11:06
@thuvh thuvh force-pushed the fix_boto_v2 branch 2 times, most recently from a9bb6c8 to a99c2b3 Compare December 26, 2025 18:47
Signed-off-by: Hoai-Thu Vuong <thuvh87@gmail.com>
@cbodley
Copy link
Contributor

cbodley commented Jan 8, 2026

looks like a good start. i assume you'll still need to change a lot of the sdk calls from boto2 to boto3 ones?

@thuvh
Copy link
Contributor Author

thuvh commented Jan 8, 2026

looks like a good start. i assume you'll still need to change a lot of the sdk calls from boto2 to boto3 ones?

Of course, find out a ton of changes when testing

@yuvalif
Copy link
Contributor

yuvalif commented Jan 18, 2026

@thuvh any update on this PR?

@thuvh
Copy link
Contributor Author

thuvh commented Jan 23, 2026

@yuvalif not yet, but I have a question and dm you in slack

@cbodley
Copy link
Contributor

cbodley commented Feb 12, 2026

@yuvalif not yet, but I have a question and dm you in slack

did you get the answers you needed there? if there are outstanding questions, maybe we can discuss here?

@cbodley cbodley requested a review from smanjara February 12, 2026 17:21
@yuvalif
Copy link
Contributor

yuvalif commented Feb 18, 2026

since there was no activity on this PR, I created a new one: #67387
@thuvh would appriciate your review

@thuvh thuvh closed this Feb 20, 2026
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.

4 participants