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

fix network costs pod #791

Merged
merged 1 commit into from
Feb 24, 2021
Merged

Conversation

AjayTripathy
Copy link
Contributor

@AjayTripathy AjayTripathy commented Feb 24, 2021

Tested by setting networkCosts.enabled: true, with

  1. networkCosts.additionalLabels: {}

  2. additionalLabels.foo: bar

  3. pods install as expected

NAME                                                READY   STATUS      RESTARTS   AGE
cost-analyzer-checks-1614183000-x4vxk               0/1     Completed   0          25m
cost-analyzer-checks-1614183600-tdrn6               0/1     Completed   0          15m
cost-analyzer-checks-1614184200-pd5rv               0/1     Completed   0          5m47s
kubecost-cost-analyzer-5fd9b487d5-m8fs6             3/3     Running     0          9m17s
kubecost-grafana-6df5cc66b6-6t4sg                   3/3     Running     0          11h
kubecost-kube-state-metrics-57d4dfc748-klkmk        1/1     Running     0          11h
kubecost-network-costs-942d6                        1/1     Running     0          8m57s
kubecost-network-costs-kcpk8                        1/1     Running     0          9m13s
kubecost-network-costs-n29j5                        1/1     Running     0          9m15s
kubecost-network-costs-n2bp4                        1/1     Running     0          8m54s
kubecost-network-costs-pf8fq                        1/1     Running     0          9m3s
kubecost-network-costs-xc62p 
  1. pods have expected labels:
 kubectl get pods -n kubecost kubecost-network-costs-942d6  -o yaml
apiVersion: v1
kind: Pod
metadata:
  creationTimestamp: "2021-02-24T16:26:51Z"
  generateName: kubecost-network-costs-
  labels:
    app: kubecost-network-costs
    controller-revision-hash: 6d875795c7
    foo: bar

@dwbrown2
Copy link
Contributor

can you share how this was tested?

@AjayTripathy AjayTripathy merged commit fd11c9f into develop Feb 24, 2021
@chipzoller chipzoller deleted the AjayTripathy-fix-networkcosts branch October 4, 2023 23:05
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.

3 participants