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

fix(uid): set default container uid (backport #11) #14

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Nov 6, 2024

Fixes #10
Related to cryostatio/cryostat-operator#969

To test using the Cryostat Operator on OpenShift:

  1. export OPENSHIFT_OAUTH_PROXY_IMG=quay.io/andrewazores/openshift-oauth-proxy:uid-1
  2. Build and deploy Operator in default namespace
  3. Create Cryostat CR in default namespace with that as the target namespace: TARGET_NAMESPACES=$(oc project -q) make create_cryostat_cr
  4. Ensure that the Cryostat Deployment is created and Pod becomes Ready
  5. Ensure that the auth-proxy container within the Pod is running the image above (built from this PR)
  6. Ensure that the auth-proxy container is running normally and Cryostat can be accessed by logging in through the proxy as usual

Or to test using Cryostat Helm on OpenShift:

  1. Switch to default project (oc project default)
  2. Deploy with existing upstream images to reproduce the bug: $ helm install cryostat --set authentication.openshift.enabled=true --set core.route.enabled=true ./charts/cryostat/
  3. helm uninstall cryostat
  4. Deploy with this PR's auth-proxy rebuild: $ helm install cryostat --set openshiftOauthProxy.image.repository=quay.io/andrewazores/openshift-oauth-proxy --set openshiftOauthProxy.image.tag=uid-1 --set authentication.openshift.enabled=true --set core.route.enabled=true ./charts/cryostat/
  5. Check OpenShift console to ensure that the Pod becomes ready, the container is running the expected image, etc.
  6. Go to Cryostat Route and ensure that it's possible to log in and access Cryostat through the proxy
    This is an automatic backport of pull request fix(uid): set default container uid #11 done by Mergify.

@andrewazores andrewazores merged commit eb9419e into cryostat-v3.0 Nov 6, 2024
2 checks passed
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.

1 participant