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.

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-17123, ROX-19217: Move GKEs to a RH project #963

Merged
merged 12 commits into from
Sep 21, 2023

Conversation

gavin-stackrox
Copy link
Contributor

@gavin-stackrox gavin-stackrox commented Aug 28, 2023

This PR switch GKE based flavors to a RH project. It:

  • updates hardcoded project name refs
  • surfaces the admin password for demo, qa-demo
  • removes Auth0 for demo, qa-demo
  • switches to a new domain for demos (demos.rox.systems) so that the complete provision could be tested on this PR without affecting production demos

See also: https://github.com/stackrox/automation-flavors/pull/151, the image links will be updated to a release tag once that merges.

Testing

Created one of each flavor using the PR infra instance. Verified access and in the case of demos that they 'look right'.

$ bin/infractl -k -e localhost:8443 list
gj-09-12-4 
  Flavor:      qa-demo
  Owner:       [email protected]
  Description: 
  Status:      READY
  Created:     26m ago
  Lifespan:    2d remaining
gj-09-12-2 
  Flavor:      demo
  Owner:       [email protected]
  Description: 
  Status:      READY
  Created:     1h0m ago
  Lifespan:    2d remaining
gj-09-11-1 
  Flavor:      gke-default
  Owner:       [email protected]
  Description: 
  Status:      READY
  Created:     18h ago
  Lifespan:    4d remaining

Note: CI is failing on this PR due to all the noise from test cluster creation. Once these clusters have expired I will destroy the PR infra instance and retest.

@gavin-stackrox gavin-stackrox requested a review from a team as a code owner August 28, 2023 23:39
@ghost
Copy link

ghost commented Aug 28, 2023

A single node development cluster (infra-pr-963) was allocated in production infra for this PR.

CI will attempt to deploy us.gcr.io/stackrox-infra/infra-server:0.7.11-27-g7e77d4088b to it.

🔌 You can connect to this cluster with:

gcloud container clusters get-credentials infra-pr-963 --zone us-central1-a --project acs-team-temp-dev

🛠️ And pull infractl from the deployed dev infra-server with:

nohup kubectl -n infra port-forward svc/infra-server-service 8443:8443 &
make pull-infractl-from-dev-server

🚲 You can then use the dev infra instance e.g.:

bin/infractl -k -e localhost:8443 whoami

⚠️ Any clusters that you start using your dev infra instance should have a lifespan shorter then the development cluster instance. Otherwise they will not be destroyed when the dev infra instance ceases to exist when the development cluster is deleted. ⚠️

Further Development

☕ If you make changes, you can commit and push and CI will take care of updating the development cluster.

🚀 If you only modify configuration (chart/infra-server/configuration) or templates (chart/infra-server/{static,templates}), you can get a faster update with:

make install-local

Logs

Logs for the development infra depending on your @redhat.com authuser:

Or:

kubectl -n infra logs -l app=infra-server --tail=1 -f

@gavin-stackrox gavin-stackrox force-pushed the gavin/ROX-17123,ROX-19217/move-GKEs-to-RH branch from a461c66 to 271153b Compare September 1, 2023 01:45
serviceAccountKeySecret:
name: google-credentials-demo
key: google-credentials.json
key: read-certs-google-credentials.json
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️ for more specific service account credential names!

Copy link
Contributor

@BradLugo BradLugo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@tommartensen tommartensen left a comment

Choose a reason for hiding this comment

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

LGTM.
FYI, I created https://issues.redhat.com/browse/ROX-19672, not sure if that was on your radar.

@gavin-stackrox gavin-stackrox force-pushed the gavin/ROX-17123,ROX-19217/move-GKEs-to-RH branch from 9458f5e to 7e77d40 Compare September 21, 2023 15:21
@gavin-stackrox gavin-stackrox enabled auto-merge (squash) September 21, 2023 15:21
@gavin-stackrox gavin-stackrox merged commit 24d14df into master Sep 21, 2023
6 of 7 checks passed
@gavin-stackrox gavin-stackrox deleted the gavin/ROX-17123,ROX-19217/move-GKEs-to-RH branch September 21, 2023 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants