Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(security-apps): Update Vault Chart from 0.24.1 to 0.26.1 #1141

Merged
merged 2 commits into from
Nov 7, 2023
Merged

feat(security-apps): Update Vault Chart from 0.24.1 to 0.26.1 #1141

merged 2 commits into from
Nov 7, 2023

Conversation

maechtu
Copy link
Contributor

@maechtu maechtu commented Nov 2, 2023 •
edited
Loading

Description

Changes:
Update vault helm-chart to 0.26.1 most important changes:

  • Default vault version updated to 1.15.1
  • Default vault-k8s version updated to 1.3.1
  • Default vault-csi-provider version updated to 1.4.1

Issues

Checklist

  • This PR contains a description of the changes I'm making
  • I updated the version in Chart.yaml
  • I updated the changelog with an artifacthub.io/changes annotation in Chart.yaml, check the example in the documentation.
  • I updated applicable README.md files using pre-commit run
  • I documented any high-level concepts I'm introducing in docs/
  • CI is currently green and this is ready for review
  • I am ready to test changes after they are applied and released

@maechtu maechtu self-assigned this Nov 2, 2023
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 2, 2023
@maechtu maechtu force-pushed the feat/security-apps/update-vault-0.26.1 branch 2 times, most recently from 61c33c4 to 5034834 Compare November 2, 2023 14:41
@github-actions github-actions bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 2, 2023
@maechtu maechtu marked this pull request as ready for review November 2, 2023 14:44
@maechtu maechtu requested a review from a team as a code owner November 2, 2023 14:44
@maechtu maechtu force-pushed the feat/security-apps/update-vault-0.26.1 branch from 5034834 to b859720 Compare November 7, 2023 08:15
@maechtu maechtu changed the title feat(security-apps): Update Vault from 0.24.1 to 0.26.1 feat(security-apps): Update Vault Chart from 0.24.1 to 0.26.1 Nov 7, 2023
@maechtu maechtu merged commit e0561d0 into main Nov 7, 2023
3 checks passed
@maechtu maechtu deleted the feat/security-apps/update-vault-0.26.1 branch November 7, 2023 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hairmare hairmare hairmare approved these changes

@inisitijitty inisitijitty Awaiting requested review from inisitijitty inisitijitty is a code owner automatically assigned from adfinis/helm-charts

@vmaillot vmaillot Awaiting requested review from vmaillot vmaillot is a code owner automatically assigned from adfinis/helm-charts

Assignees

@maechtu maechtu

Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maechtu @hairmare