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

chore(security-apps): update vault to v0.19 #574

Merged
merged 1 commit into from
Feb 21, 2022
Merged

chore(security-apps): update vault to v0.19 #574

merged 1 commit into from
Feb 21, 2022

Conversation

tongpu
Copy link
Member

@tongpu tongpu commented Feb 21, 2022 •
edited
Loading

Description

Update Vault Helm chart in security-apps to v0.19

Changes

Updates

  • Vault image defaults to 1.9.2
  • Vault K8s image defaults to 0.14.2

Features

  • Added configurable podDisruptionBudget for injector GH-653
  • Make terminationGracePeriodSeconds configurable for server GH-659
  • Added configurable update strategy for injector GH-661
  • csi: ability to set priorityClassName for CSI daemonset pods GH-670

Improvements

  • Set the namespace on the OpenShift Route GH-679
  • Add volumes and env vars to helm hook test pod GH-673
  • Make TLS configurable for OpenShift routes GH-686

Issues

Fixes #564

Checklist

  • I updated the version in Chart.yaml
  • I updated applicable README.md files using pre-commit run
  • I documented any high-level concepts I'm introducing in docs/
  • If I updated a dependency tool, or app, this PR contains a short summary of the changes I'm pulling
  • CI is currently green and this is ready for review
  • I am ready to test changes after they are applied and released

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 21, 2022
@tongpu tongpu marked this pull request as ready for review February 21, 2022 12:34
@tongpu tongpu requested a review from a team as a code owner February 21, 2022 12:34
@tongpu tongpu requested review from inisitijitty and isantospardo and removed request for a team February 21, 2022 12:34
@tongpu tongpu enabled auto-merge (squash) February 21, 2022 12:35
isantospardo
isantospardo approved these changes Feb 21, 2022
View reviewed changes
Copy link
Contributor

@isantospardo isantospardo left a comment

Choose a reason for hiding this comment

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

LGTM

@tongpu tongpu merged commit aecfbb1 into master Feb 21, 2022
@tongpu tongpu deleted the chore/update_vault branch February 21, 2022 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@isantospardo isantospardo isantospardo approved these changes

@inisitijitty inisitijitty Awaiting requested review from inisitijitty inisitijitty was automatically assigned from adfinis/helm-charts

Assignees
No one assigned
Labels
size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Version 0.19.0 of vault is available!
2 participants
@tongpu @isantospardo