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

cloud,backupccl: add object-locked test variants #115616

Merged

Conversation

stevendanna
Copy link
Collaborator

@stevendanna stevendanna commented Dec 5, 2023

This adds a test variant that runs backup/restore against an object-locked bucket. We assume that a second AWS bucket exists with the -locked prefix to run these tests.

This test would have discovered a recent, near-miss:

--- FAIL: TestCloudBackupRestoreS3/object-locked-bucket (2.34s)
--- PASS: TestCloudBackupRestoreS3/regular-bucket (19.41s)

Epic: none

Fixes #79771

Release note: None

This adds a test variant that runs backup/restore against an
object-locked bucket. We assume that a second AWS bucket exists with
the -locked prefix to run these tests.

This test would have discovered a recently, near-miss:

    --- FAIL: TestCloudBackupRestoreS3/object-locked-bucket (2.34s)
    --- PASS: TestCloudBackupRestoreS3/regular-bucket (19.41s)

Epic: none

Release note: None
@stevendanna stevendanna requested a review from a team as a code owner December 5, 2023 16:53
@stevendanna stevendanna requested review from adityamaru and removed request for a team December 5, 2023 16:53
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@stevendanna
Copy link
Collaborator Author

The ignore-whitespace diff is easier to read on this one:

https://github.com/cockroachdb/cockroach/pull/115616/files?diff=unified&w=1

@stevendanna
Copy link
Collaborator Author

@adityamaru
Copy link
Contributor

@stevendanna should we rebase and merge now that the patch has been reverted on master #115713

@adityamaru
Copy link
Contributor

bors r=adityamaru

@craig
Copy link
Contributor

craig bot commented Dec 7, 2023

Build succeeded:

@craig craig bot merged commit af1fda5 into cockroachdb:master Dec 7, 2023
11 checks passed
@adityamaru
Copy link
Contributor

blathers backport 23.2 23.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

backupccl: Switch backup roachtests to use object locked buckets
3 participants