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

CASMPET-7376: Update kiali-operator to v2.5.0 #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bmcdonald3
Copy link

@bmcdonald3 bmcdonald3 commented Feb 7, 2025

Summary and Scope

Add kiali-operator image v2.5.0 for upgrade in CSM 1.7.

Issues and Related PRs

Testing

  • Virtual Shasta

Test description:

Running the following script:

#!/bin/bash

kubectl get pods -n istio-system | grep kiali

kubectl port-forward svc/kiali -n istio-system 20001:20001 &
PORT_FORWARD_PID=$!

sleep 5

# Fetch Kiali metrics
echo "Fetching Kiali metrics..."
curl -s http://localhost:20001/kiali/api/namespaces | head -c 50
echo ''

kill $PORT_FORWARD_PID

Resulted in the following output:

# ./test-kiali.sh
kiali-6fc596d955-779b6                                 1/1     Running   0          20h
Forwarding from 127.0.0.1:20001 -> 20001
Forwarding from [::1]:20001 -> 20001
Fetching Kiali metrics...
Handling connection for 20001
[{"name":"argo","cluster":"Kubernetes","isAmbient"

Tested before and after upgrade:

# kubectl describe pod -n operators cray-kiali-kiali-operator-bd69d5879-w6482 | grep Image
    Image:         artifactory.algol60.net/csm-docker/stable/quay.io/kiali/kiali-operator:v2.5.0

Risks and Mitigations

Low risk.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable

Sorry, something went wrong.

Copy link

@leliasen-hpe leliasen-hpe left a comment

Choose a reason for hiding this comment

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

Does the kiali image in cray-kiali/values.yaml line 55 need to be changed?

@bmcdonald3
Copy link
Author

Well, this upgrade is only for the kiali-operator as that was what was listed in https://rndwiki-pro.its.hpecorp.net/display/CSMTemp/PET+Maintenance, but I am seeing now that is just the upstream chart listed. I will update the Kiali image as well! (will need to add that to the image PR)

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.

None yet

2 participants