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

add cost model image tag as env var #3710

Merged
merged 2 commits into from
Oct 15, 2024
Merged

Conversation

mittal-ishaan
Copy link
Contributor

@mittal-ishaan mittal-ishaan commented Oct 11, 2024

What does this PR change?

Adds cost model image tag as env var in the container

Does this PR rely on any other PRs?

NA

How does this PR impact users? (This is the kind of thing that goes in release notes!)

Kubecost version in diagnostics page will not show cost-model image tag

Links to Issues or tickets this PR addresses or fixes

https://kubecost.atlassian.net/browse/ENG-2359

What risks are associated with merging this PR? What is required to fully test this PR?

NA

How was this PR tested?

Tried all the scenarios that set the cost model image and checked the corresponding helm template.

Have you made an update to documentation? If so, please provide the corresponding PR.

NA

@mittal-ishaan mittal-ishaan marked this pull request as ready for review October 14, 2024 15:09
Copy link
Collaborator

@jessegoodier jessegoodier left a comment

Choose a reason for hiding this comment

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

love this!

@mittal-ishaan mittal-ishaan merged commit 49ec3fe into develop Oct 15, 2024
20 checks passed
@mittal-ishaan mittal-ishaan deleted the cost-model-image-tag branch October 15, 2024 13:23
@thomasvn thomasvn added the v2.5 label Oct 16, 2024
@cpanixperi
Copy link

Hi, We are facing issue on our production deployment. With this error:

Failed to pull image "gcr.io/kubecost1/kubecost-network-costs:v16.5": rpc error: code = NotFound desc = failed to pull and unpack image "gcr.io/kubecost1/kubecost-network-costs:v16.5": failed to resolve reference "gcr.io/kubecost1/kubecost-network-costs:v16.5": gcr.io/kubecost1/kubecost-network-costs:v16.5: not found

Did this change break something ?

@mittal-ishaan
Copy link
Contributor Author

No, this change should not be affecting network cost, But could you share the helm values you are using and I can check if we missing something

@cpanixperi
Copy link

We are using v1.101.2 of the helm chart. From last 24hrs it is failing to fetch the image "gcr.io/kubecost1/kubecost-network-costs:v16.5"

@mittal-ishaan
Copy link
Contributor Author

the correct image to this image from what I understand is v0.16.5 i.e. the image should be gcr.io/kubecost1/kubecost-network-costs:v0.16.5. Have you overridden any helm values around it that is causing the image to be gcr.io/kubecost1/kubecost-network-costs:v16.5 and not the one I am suggesting?

@cpanixperi
Copy link

No , we haven't done any changes to the chart in last 1 and half years. This image gcr.io/kubecost1/kubecost-network-costs getting pulled by the chart internally. We are not mentioning the tag v16.5 any where in our code.

We believe something changed in kubecost side in last 24 hrs which is breaking this deployment in our clusters .

This shows there was some change 20hrs ago.
https://console.cloud.google.com/gcr/images/kubecost1/GLOBAL/kubecost-network-costs?pli=1

@chipzoller
Copy link
Collaborator

@cpanixperi, the issue here is not related to this PR. Please open a support request a support.kubecost.com.

@cpanixperi
Copy link

Thanks @chipzoller . Looks like the change is reverted now. There should have been some communication before making such breaking changes.

@jessegoodier
Copy link
Collaborator

The 1.102 release is immutable.

Agree with Chip, the issue is certainly not related to this PR and I'm not aware of any other issues with gcr.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants