-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
ROX-20058: Use GCP secret manager + Helm for infra deployments #1015
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
15eb95b
wip: secrets + Helm charts
tommartensen 0846847
fix and stuff
tommartensen 2196ee7
Merge branch 'master' into tm/helm-charts
tommartensen a1a3b2a
progress
tommartensen 50caa4e
Merge branch 'master' into tm/helm-charts
tommartensen 70e5c12
add secret management script
tommartensen 2788b18
Merge branch 'tm/helm-charts' of github.com:stackrox/infra into tm/he…
tommartensen b33dcf8
finish stuff for development
tommartensen ea2ac57
fix stupid typo
tommartensen cd4eb66
fix cap
tommartensen f3cd6cc
install argo
tommartensen ba40223
fix bats e2e test target
tommartensen 750ed49
update documentation
tommartensen 90540ce
update
tommartensen 22000e6
fix deployment workflow
tommartensen 526e1e5
fix deployment workflow
tommartensen b05e6dd
sequentialize PRs
tommartensen 58cb41d
send it
tommartensen cb6a8a4
add name
tommartensen 5081f5d
fix infractl pull
tommartensen 67ab57e
use correct capabilities lookup
tommartensen ce8e555
use testmode instead of capabilities
tommartensen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
name: Deploy infra | ||
run-name: >- | ||
${{ | ||
format('Deploy infra version {0} to {1}', | ||
|
@@ -53,14 +54,14 @@ jobs: | |
gcloud container clusters get-credentials infra-${{ inputs.environment }} \ | ||
--project stackrox-infra \ | ||
--region us-west2 | ||
make install-${{ inputs.environment }} | ||
ENVIRONMENT=${{ inputs.environment }} make install-argo helm-deploy | ||
|
||
- name: Notify infra channel about new version | ||
env: | ||
SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }} | ||
uses: slackapi/slack-github-action@v1.23.0 | ||
with: | ||
channel-id: C01H4DC33K3 #acs-infra | ||
channel-id: CVANK5K5W #acs-infra | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yay! |
||
payload: >- | ||
{ | ||
"blocks": [ | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay!