Skip to content

Acknowledge secret regeneration concern as tracked follow-up#59

Draft
Copilot wants to merge 5 commits intomainfrom
copilot/sub-pr-57
Draft

Acknowledge secret regeneration concern as tracked follow-up#59
Copilot wants to merge 5 commits intomainfrom
copilot/sub-pr-57

Conversation

Copy link

Copilot AI commented Feb 19, 2026

A review comment flagged that genPrivateKey and randAlphaNum in postal/templates/secret.yaml regenerate secrets on every Helm upgrade, risking service disruption. The reviewer requested this be deferred to a separate issue rather than addressed in the current PR.

Changes

  • Replied to the review comment explaining that opening GitHub issues is outside Copilot's capabilities, and provided the issue author with the full context needed to open the tracking issue manually — covering the root cause and the suggested mitigation (Helm lookup function to preserve existing secret values across upgrades)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

neilime and others added 5 commits February 16, 2026 23:12
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI mentioned this pull request Feb 19, 2026
Copilot AI changed the title [WIP] Address feedback on Helm test fix from PR #57 Acknowledge secret regeneration concern as tracked follow-up Feb 19, 2026
Copilot AI requested a review from fredleger February 19, 2026 15:37
Base automatically changed from ci/fix-helm-test to main March 13, 2026 09:30
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