Skip to content

Secrets-vault migration fixes for nodectl v0.5.0#166

Merged
Keshoid merged 10 commits into
release/nodectl/v0.5.0from
feature/secrets-vault-migration-fixes
May 21, 2026
Merged

Secrets-vault migration fixes for nodectl v0.5.0#166
Keshoid merged 10 commits into
release/nodectl/v0.5.0from
feature/secrets-vault-migration-fixes

Conversation

@Keshoid
Copy link
Copy Markdown
Contributor

@Keshoid Keshoid commented May 21, 2026

Summary

Bundles upstream secrets-vault migration fixes with the corresponding nodectl key migrate adjustments for the v0.5.0 release.

  • secrets-vault: fix HashiCorp list-key bug; allow migrating non-extractable secrets from file to HashiCorp by going through the storage layer (load_for_migrate + store(..., override_extractable)).
  • nodectl key migrate: port the new migration logic — restrict source to file:// and destination to hashicorp://, propagate the original extractable flag, remove the dead NOT_EXTRACTABLE skip branch.
  • Docs: declare the file→HashiCorp restriction and non-extractable-key support in the helm runbook and the nodectl README.
  • e2e: run_singlehost_nodectl.py now reads VAULT_URL/VAULT_URL_NODE_X from the environment so the harness can target Hashicorp.

Copilot AI review requested due to automatic review settings May 21, 2026 07:21
@Keshoid Keshoid requested a review from ITBear May 21, 2026 07:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Keshoid Keshoid merged commit d891b5e into release/nodectl/v0.5.0 May 21, 2026
5 checks passed
@Keshoid Keshoid deleted the feature/secrets-vault-migration-fixes branch May 21, 2026 08:16
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