Skip to content
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.

k8s-cloudwatch adapter no longer an implementor of custom.metrics API? #26

Closed
ghostsquad opened this issue Mar 24, 2020 · 1 comment
Closed
Labels
question Further information is requested

Comments

@ghostsquad
Copy link

I noticed that in the readme, it shows some output from a deployment:

apiservice.apiregistration.k8s.io/v1beta1.custom.metrics.k8s.io created
apiservice.apiregistration.k8s.io/v1beta1.external.metrics.k8s.io created
clusterrole.rbac.authorization.k8s.io/k8s-cloudwatch-adapter:custom-metrics-reader created
clusterrole.rbac.authorization.k8s.io/k8s-cloudwatch-adapter:external-metrics-reader created
clusterrole.rbac.authorization.k8s.io/k8s-cloudwatch-adapter-resource-reader created
clusterrolebinding.rbac.authorization.k8s.io/k8s-cloudwatch-adapter:custom-metrics-reader created
clusterrolebinding.rbac.authorization.k8s.io/k8s-cloudwatch-adapter:external-metrics-reader created

but the manifests here no longer deploy the APIService for custom.metrics:

https://github.com/awslabs/k8s-cloudwatch-adapter/blob/master/deploy/adapter.yaml

At one point in the past, maybe it did. Is it safe to assume that the README can be updated, and additionally, if one deployed an old version of the k8s-cloudwatch-adapter and has since updated it, that this APIService should be deleted?

kubectl delete apiservice v1beta1.custom.metrics.k8s.io
@chankh
Copy link
Contributor

chankh commented Mar 26, 2020

you are correct, custom.metrics is no longer being used in the adapter, so it is save to delete the apiservice if you had that created from an older version. I will update the readme.

chankh added a commit that referenced this issue Mar 26, 2020
@ghostsquad ghostsquad changed the title k8s-cloudwatch adapter no longer an implementor or custom.metrics API? k8s-cloudwatch adapter no longer an implementor of custom.metrics API? Mar 26, 2020
@chankh chankh added the question Further information is requested label May 15, 2020
@chankh chankh closed this as completed May 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants