Conversation
|
S3SnapshotRestoreTestSuite TearDownSuite TestSuites above were modified. TestSuites below use modified code from this PR. |
navsalehi
requested review from
Copilot,
markusewalker and
susesgartner
and removed request for
Copilot
markusewalker
left a comment
markusewalker
left a comment
There was a problem hiding this comment.
Left comments that apply to both K3s and RKE2. Once addressed, let's test again to ensure all is good. Overall, nice job!
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
from
aab9bdd to
34ad7e7
Compare
|
S3SnapshotRestoreTestSuite SetupSuite TestSuites above were modified. TestSuites below use modified code from this PR. |
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
from
34ad7e7 to
01bc916
Compare
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
from
01bc916 to
52722ec
Compare
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
2 times, most recently
from
c98b76c to
5a90414
Compare
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
2 times, most recently
from
5a13ff2 to
7c49c9b
Compare
|
S3SnapshotRestoreTestSuite SetupSuite TestSuites above were modified. TestSuites below use modified code from this PR. |
navsalehi
requested review from
Josh-Diamond
and removed request for
susesgartner
|
S3SnapshotRestoreTestSuite SetupSuite TestSuites above were modified. TestSuites below use modified code from this PR. |
navsalehi
force-pushed
the
fix_s3_snapshot_isolation
branch
from
1a024dd to
99f6e81
Compare
|
S3SnapshotRestoreTestSuite SetupSuite TestSuites above were modified. TestSuites below use modified code from this PR. |
5 participants
This PR adds S3 bucket handling to the snapshot restore tests and introduces reusable helpers (CreateS3Bucket and DeleteS3Bucket) under actions/etcdsnapshot to avoid duplication.
Both K3S and RKE2 tests now create a bucket dynamically, configure ETCD to use S3, validate snapshot/restore, and clean up the bucket during teardown.