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

Added in Customer Google Tag to values and environment variables #1373

Merged
merged 1 commit into from
Apr 15, 2022

Conversation

teevans
Copy link
Contributor

@teevans teevans commented Apr 15, 2022

What does this PR change?

This add the GoogleCustomerTag value to the Values file in the helm chart under kubecostFrontend, and adds the GoogleCustomerTag as an environment variable to be returned by /getConfigs

Does this PR rely on any other PRs?

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

This allows users to use their own Google Global Tag to track the utilization of their Kubecost frontend!

Links to Issues or ZD tickets this PR addresses or fixes

How was this PR tested?

Manually by deploying to my GKE cluster.

Have you made an update to documentation?

No - Will do prior to closing after we find out if I did this right.

@teevans teevans merged commit b824702 into develop Apr 15, 2022
@@ -198,6 +198,8 @@ kubecostFrontend:
#limits:
# cpu: "100m"
# memory: "256Mi"
# customerGoogleTag allows you to embed your Google Global Site Tag to track usage of Kubecost.
Copy link
Contributor

Choose a reason for hiding this comment

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

@teevans sorry for catching this late, but what if we moved this under the reporting block? that's where all related analytics flags are.

https://github.com/kubecost/cost-analyzer-helm-chart/blob/master/cost-analyzer/values.yaml#L609

@chipzoller chipzoller deleted the thomas/gtagconfig branch October 4, 2023 23:20
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.

4 participants