Skip to content

feat: uptake etcd v3 snapshot facilities for embedded etcd#3907

Open
cbalan wants to merge 1 commit into
loft-sh:mainfrom
cbalan:engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster
Open

feat: uptake etcd v3 snapshot facilities for embedded etcd#3907
cbalan wants to merge 1 commit into
loft-sh:mainfrom
cbalan:engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster

Conversation

@cbalan
Copy link
Copy Markdown
Contributor

@cbalan cbalan commented Apr 29, 2026

What issue type does this pull request address? (keep at least one, remove the others)
/kind feature

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
Part of #ENGCP-655

Please provide a short message that should be published in the vcluster release notes
Introduce native etcd v3 snapshot/restore for embedded etcd via etcdutl, while preserving the existing key-value tar-stream approach for other backing stores (SQLite, deployed/external etcd).

What else do we need to know?

E2E Tests

Default Test Execution

The mandatory PR suite runs automatically. Only specify additional test suites below if needed.

Adding New Test Suites

When adding a new ginkgo test suite:

  • Add labels to the test suite
  • Update label-filter section below to execute the new test suite
  • Verify test suite runs in CI/CD pipeline

Additional test suites

Additional test suite(s) that will be executed before the mandatory PR suite:

snapshots

@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch 3 times, most recently from c07ab8b to b160493 Compare April 29, 2026 22:19
@cbalan
Copy link
Copy Markdown
Contributor Author

cbalan commented Apr 30, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch from 4976cec to 7b06eac Compare May 4, 2026 09:49
@cbalan cbalan marked this pull request as ready for review May 5, 2026 13:19
@cbalan cbalan requested review from a team as code owners May 5, 2026 13:19
Copy link
Copy Markdown
Contributor

@mfranczy mfranczy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As agreed at the meeting, please review if we can safely modify the bbolt db offline, in the meantime I will take a closer look into the peers problem.

@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch from 519d2e9 to 276f25c Compare May 6, 2026 16:47
Comment thread pkg/snapshot/client.go
Comment thread pkg/snapshot/restoreclient.go Outdated
Comment thread pkg/snapshot/restoreclient.go Outdated
Comment thread pkg/snapshot/restoreclient.go
@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch from a597930 to 77af521 Compare May 7, 2026 15:08
Comment thread pkg/snapshot/restoreclient.go
@mfranczy
Copy link
Copy Markdown
Contributor

mfranczy commented May 7, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 08c46e2a17

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread pkg/snapshot/restoreclient.go
@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch 6 times, most recently from d6f16fb to 608eb22 Compare May 12, 2026 12:17
@mfranczy
Copy link
Copy Markdown
Contributor

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 608eb225fd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread pkg/snapshot/restoreclient.go
Comment thread pkg/snapshot/restoreclient.go Outdated
@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch 3 times, most recently from 3218542 to b70942c Compare May 12, 2026 19:38
Comment thread pkg/snapshot/client.go Outdated
Comment thread pkg/snapshot/client.go Outdated
Comment thread pkg/snapshot/client.go Outdated
Comment thread pkg/snapshot/client.go Outdated
Comment thread pkg/snapshot/restoreclient.go Outdated
@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch from 0314be6 to 0eed315 Compare May 15, 2026 10:07
@cbalan cbalan force-pushed the engcp-655-uptake-etcd-v3-snapshot-restore-facilities-for-vcluster branch from 0eed315 to b227700 Compare May 15, 2026 12:27
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.

3 participants