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

feat(kubecost): Bump chart to 1.108.1 #1520

Merged
merged 10 commits into from
Nov 11, 2024
Merged

Conversation

SandhyaRavi2403
Copy link
Contributor

@SandhyaRavi2403 SandhyaRavi2403 commented Oct 28, 2024

What type of PR is this?
Feat

What this PR does/ why we need it:
Bump kubecost to 1.108.1

Which issue(s) this PR fixes:
https://jira.nutanix.com/browse/NCN-103597

Manually Tested

`sandhya@nkp-sandhya:~/charts$ helm install --repo https://mesosphere.github.io/charts/stable kubecost kubecost

WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /home/sandhya/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /home/sandhya/.kube/config
NAME: kubecost
LAST DEPLOYED: Mon Nov 11 07:58:29 2024
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None`
`sandhya@nkp-sandhya:~/charts$ helm upgrade kubecost ./stable/kubecost
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /home/sandhya/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /home/sandhya/.kube/config
Release "kubecost" has been upgraded. Happy Helming!
NAME: kubecost
LAST DEPLOYED: Mon Nov 11 07:59:49 2024
NAMESPACE: default
STATUS: deployed
REVISION: 2`
sandhya@nkp-sandhya:~/charts$ kubectl get pods
NAME                                                READY   STATUS    RESTARTS   AGE
kubecost-cost-analyzer-675dd76cfb-hnqhw             2/2     Running   0          3m28s
kubecost-grafana-6b64fcdcc4-5nqcc                   3/3     Running   0          3m28s
kubecost-prometheus-alertmanager-5b86d54cdb-j9gpw   1/1     Running   0          3m25s
kubecost-prometheus-server-589b8485b6-7qj6g         3/3     Running   0          3m26s

@SandhyaRavi2403 SandhyaRavi2403 self-assigned this Oct 28, 2024
@SandhyaRavi2403 SandhyaRavi2403 requested review from a team as code owners October 28, 2024 07:51
@SandhyaRavi2403 SandhyaRavi2403 added the ok-to-test Signals mergebot that CI checks are ready to be kicked off label Oct 28, 2024
@SandhyaRavi2403 SandhyaRavi2403 force-pushed the sandy/cve-fix-kubecost branch 2 times, most recently from 8c9b53f to 9e45fce Compare October 28, 2024 11:20
@SandhyaRavi2403 SandhyaRavi2403 added ready ready and removed ready ready labels Oct 28, 2024
@SandhyaRavi2403 SandhyaRavi2403 force-pushed the sandy/cve-fix-kubecost branch 3 times, most recently from 8b747d2 to e68077c Compare November 4, 2024 04:44
Copy link
Contributor

@mhrabovcin mhrabovcin left a comment

Choose a reason for hiding this comment

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

I've manually tested the upgrade. CI failures are caused by the use of existing values of the chart testing library - helm/chart-testing#525

@mhrabovcin mhrabovcin merged commit 722286b into master Nov 11, 2024
5 of 6 checks passed
@mhrabovcin mhrabovcin deleted the sandy/cve-fix-kubecost branch November 11, 2024 09:22
github-actions bot pushed a commit that referenced this pull request Nov 11, 2024
…chart to 1.108.1 * feat(kubecost): update kubectl to 1.30.5 * feat(kubecost): update kubecost * feat(kubecost): update kubecost * feat(kubecost): update chart values * feat(kubecost): update chart values * Update stable/kubecost/values.yaml Co-authored-by: Martin Hrabovcin <[email protected]> * Update stable/kubecost/values.yaml Co-authored-by: Martin Hrabovcin <[email protected]> * feat(kubecost): revert changes * feat(kubecost): revert changes --------- Co-authored-by: Martin Hrabovcin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Signals mergebot that CI checks are ready to be kicked off
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants