Skip to content

Conversation

zied-jt
Copy link

@zied-jt zied-jt commented Jul 3, 2025

Context

This PR take commits from #399 and add tests for each added new key in value.yaml file.

Commits from @sadath-12

  • support: backupID in helm chart
  • fix: correct backupID field
  • re-order and add other supported target recoveries
  • add missing target params

Commits for added tests:

  • feat(cluster): Test restore with backup id
  • feat(cluster): Test restore with backup name
  • feat(cluster): Test restore with xid
  • feat(cluster): Test restore with lsn
  • feat(cluster): Test restore with time
  • feat(cluster): Test restore with exclusive
  • feat(cluster): Test restore with immediate

Tests

Test are executed with chainsaw, example:

cd charts/cluster/test/
chainsaw test --test-file chainsaw-test-xid.yaml

Related PRs

Fixes

@zied-jt zied-jt requested a review from itay-grudev as a code owner July 3, 2025 15:30
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. chart( cluster ) Related to the cluster chart size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Jul 3, 2025
@zied-jt zied-jt force-pushed the backup-sadath-12-zied-jt branch from 96bed12 to f51a315 Compare July 10, 2025 16:29
sadath-12 and others added 16 commits July 14, 2025 02:51
Signed-off-by: Zied ABID <[email protected]>
@zied-jt zied-jt force-pushed the backup-sadath-12-zied-jt branch from f51a315 to 8342dff Compare July 14, 2025 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants