Skip to content

Conversation

yiannistri
Copy link
Collaborator

@yiannistri yiannistri commented Feb 10, 2022

  • Created a bucket (currently unencrypted) to store state
  • Added a simple gke-cluster module to allow us to easily provision new clusters.
  • Added definitions for gitlab-01, dex-01 and demo-02 clusters under a new domain.

@yiannistri yiannistri added the exclude from release notes Use this label to exclude a PR from the release notes label Feb 10, 2022
@yiannistri yiannistri force-pushed the add-terraform branch 8 times, most recently from 503f1fd to 49edbf3 Compare February 12, 2022 19:09
Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

All looks good to my eye! 👍

@@ -438,7 +438,7 @@ func RunInProcessGateway(ctx context.Context, addr string, setters ...Option) er
// Secure `/v1` and `/gitops/api` API routes
grpcHttpHandler = auth.WithAPIAuth(grpcHttpHandler, srv)
gitopsBrokerHandler = auth.WithAPIAuth(gitopsBrokerHandler, srv)
staticAssets = auth.WithAPIAuth(staticAssets, srv)
staticAssets = auth.WithWebAuth(staticAssets, srv)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah I didn't notice there was a diff here! 👍

@yiannistri yiannistri merged commit 526104f into main Feb 14, 2022
@yiannistri yiannistri deleted the add-terraform branch February 14, 2022 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude from release notes Use this label to exclude a PR from the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants