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

[hack] update with new sail stuff #7333

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

jmazzitelli
Copy link
Collaborator

@jmazzitelli jmazzitelli commented May 3, 2024

Some things have changed with Sail operator. This updates the hack scripts to work with it.

To test:

  1. Start an OpenShift cluster (e.g. hack/crc-openshift.sh start)
  2. Use oc to log into the cluster as a kubeadmin user
  3. Install the operators for Sail, Kiali, Tempo, etc.: hack/istio/sail/install-ossm-release.sh install-operators
  4. Install a Istio, Kiali, Prometheus, etc. etc.: hack/istio/sail/install-ossm-release.sh install-istio
  5. See that everything eventually installs successfully: hack/istio/sail/install-ossm-release.sh status
  6. Bring up the Kiali UI in a browser: hack/istio/sail/install-ossm-release.sh kiali-ui
  7. See that you can get to the UI and things look OK

When you are done, you can delete everything except the operators with: hack/istio/sail/install-ossm-release.sh delete-istio

After Istio and all other components are removed, you can then delete the operators which takes you back to a clean slate: hack/istio/sail/install-ossm-release.sh delete-operators

@jmazzitelli jmazzitelli self-assigned this May 3, 2024
@jmazzitelli
Copy link
Collaborator Author

Note the one change, replacing quay.io/official-images/busybox:1.28 with quay.io/jitesoft/alpine:latest was because the original one is no longer accessible, and all we really need is some small image with the ability to run the small sh command.

@jmazzitelli jmazzitelli marked this pull request as ready for review May 7, 2024 13:27
@jshaughn jshaughn self-requested a review May 9, 2024 19:15
@jshaughn jshaughn added the test: n/a PR does not need test additions or updates label May 9, 2024
Copy link
Collaborator

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

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

Ran through the test steps, works like a chaahm!

@jmazzitelli jmazzitelli merged commit 21ddd2f into kiali:master May 9, 2024
9 checks passed
@jmazzitelli jmazzitelli deleted the hack-sail branch May 9, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test: n/a PR does not need test additions or updates
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants