Skip to content

Commit

Permalink
Merge pull request #126 from kubecost/mmd/fix-use-proxy
Browse files Browse the repository at this point in the history
Fix use-proxy, bad port -> string conversion
  • Loading branch information
michaelmdresser authored Jul 11, 2022
2 parents b2fa864 + fff659e commit 9c33aa5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/query/allocation.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ func QueryAllocation(p AllocationParameters) ([]map[string]kubecost.Allocation,
return nil, fmt.Errorf("failed to create clientset for proxied query: %s", err)
}

bytes, err = clientset.CoreV1().Services(p.KubecostNamespace).ProxyGet("", p.ServiceName, string(p.ServicePort), p.AllocationPath, p.QueryParams).DoRaw(p.Ctx)
bytes, err = clientset.CoreV1().Services(p.KubecostNamespace).ProxyGet("", p.ServiceName, fmt.Sprint(p.ServicePort), p.AllocationPath, p.QueryParams).DoRaw(p.Ctx)
if err != nil {
return nil, fmt.Errorf("failed to proxy get kubecost. err: %s; data: %s", err, bytes)
}
Expand Down

0 comments on commit 9c33aa5

Please sign in to comment.