Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

chore: change vault-env references to secret-init + add OpenBao injector #174

Merged
merged 12 commits into from
Mar 20, 2024
Merged

chore: change vault-env references to secret-init + add OpenBao injector #174

merged 12 commits into from
Mar 20, 2024

Conversation

csatib02
Copy link
Member

@csatib02 csatib02 commented Mar 1, 2024 •
edited
Loading

Overview

  • This PR changes Vault-env references to secret-init references.
  • One secrets-webhook dependency needs to be added after secrets-webhook is added to the BV stack.
  • This PR also introduces OpenBao injector

@csatib02 csatib02 added kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/feature Categorizes issue or PR as related to a new feature. labels Mar 1, 2024
@csatib02 csatib02 requested review from ramizpolic and akijakya March 1, 2024 11:09
@csatib02 csatib02 self-assigned this Mar 1, 2024
@csatib02 csatib02 requested a review from a team as a code owner March 1, 2024 11:09
@github-actions github-actions bot added the size/L Denotes a PR that changes 500-999 lines label Mar 1, 2024
@csatib02 csatib02 removed the request for review from a team March 1, 2024 11:11
@csatib02 csatib02 mentioned this pull request Mar 1, 2024
Merged
akijakya
akijakya approved these changes Mar 13, 2024
View reviewed changes
Copy link
Member

@akijakya akijakya left a comment

Choose a reason for hiding this comment

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

LGTM!

README.md Outdated Show resolved Hide resolved
@csatib02 csatib02 force-pushed the feat/add-secretes_webhook-import-add-bao-injector branch 4 times, most recently from 3af4e1f to b446f1c Compare March 20, 2024 09:53
@github-actions github-actions bot added size/XL Denotes a PR that changes 1000+ lines size/L Denotes a PR that changes 500-999 lines and removed size/L Denotes a PR that changes 500-999 lines size/XL Denotes a PR that changes 1000+ lines labels Mar 20, 2024
@csatib02 csatib02 force-pushed the feat/add-secretes_webhook-import-add-bao-injector branch from 91b657a to 88d1347 Compare March 20, 2024 10:16
@github-actions github-actions bot added size/XL Denotes a PR that changes 1000+ lines and removed size/L Denotes a PR that changes 500-999 lines labels Mar 20, 2024
csatib02 added 9 commits March 20, 2024 11:21
@csatib02
chore: change vault-env references to secret-init
aa909f1 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
feat(bao): add bao injector
b1a79a2 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
chore
2c6b144 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
Add missing dependency
a31ce86 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
fix linters, move injectors to pkg
af77452 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
chore
6de1461 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
fix lint
2dcc1bd 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
chore
b719ff0 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
add bao services to ci
c7c5ae0 
Signed-off-by: Bence Csati <[email protected]>
csatib02 added 2 commits March 20, 2024 11:28
@csatib02
remove failing tests temporarily
bc6763b 
Signed-off-by: Bence Csati <[email protected]>
@csatib02
fix lint
f86f6a1 
Signed-off-by: Bence Csati <[email protected]>
@csatib02 csatib02 force-pushed the feat/add-secretes_webhook-import-add-bao-injector branch from 88d1347 to f86f6a1 Compare March 20, 2024 10:29
@github-actions github-actions bot added size/L Denotes a PR that changes 500-999 lines and removed size/XL Denotes a PR that changes 1000+ lines labels Mar 20, 2024
@csatib02
fix: readme
935ea62 
Signed-off-by: Bence Csati <[email protected]>
@csatib02 csatib02 merged commit ffd0ade into main Mar 20, 2024
12 checks passed
@csatib02 csatib02 deleted the feat/add-secretes_webhook-import-add-bao-injector branch March 20, 2024 10:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@akijakya akijakya akijakya approved these changes

@ramizpolic ramizpolic Awaiting requested review from ramizpolic

Assignees

@csatib02 csatib02

Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 500-999 lines
Projects
Project backlog
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@csatib02 @akijakya