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

Remove cluster-monitoring #85512

Merged
merged 1 commit into from Dec 18, 2019

Conversation

@serathius
Copy link
Contributor

serathius commented Nov 21, 2019

Heapster is deprecated and no longer supported

/kind cleanup

Remove cluster-monitoring addon
@eparis

This comment has been minimized.

Copy link
Member

eparis commented Nov 22, 2019

/lgtm

@eparis

This comment has been minimized.

Copy link
Member

eparis commented Nov 22, 2019

/approve

@serathius

This comment has been minimized.

Copy link
Contributor Author

serathius commented Nov 24, 2019

/hold
/cc @loburm PTAL

@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Nov 24, 2019

@serathius: GitHub didn't allow me to request PR reviews from the following users: PTAL.

Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/hold
/cc @loburm PTAL

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@loburm

This comment has been minimized.

Copy link
Contributor

loburm commented Nov 26, 2019

/lgtm

@serathius

This comment has been minimized.

Copy link
Contributor Author

serathius commented Nov 26, 2019

/unhold

Heapster is deprecated and no longer supported
@serathius serathius force-pushed the serathius:remove-cluster-monitoring branch from 6f8f1ef to 31fb04f Dec 9, 2019
@k8s-ci-robot k8s-ci-robot removed the lgtm label Dec 9, 2019
@serathius

This comment has been minimized.

Copy link
Contributor Author

serathius commented Dec 10, 2019

/assign @cblecker

@cblecker

This comment has been minimized.

Copy link
Member

cblecker commented Dec 17, 2019

/priority backlog
/kind deprecation
/lgtm
/approve

@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Dec 17, 2019

@cblecker: The label(s) kind/deprecation cannot be applied, because the repository doesn't have them

In response to this:

/priority backlog
/kind deprecation
/lgtm
/approve

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Dec 17, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cblecker, eparis, serathius

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cblecker

This comment has been minimized.

Copy link
Member

cblecker commented Dec 18, 2019

/kind deprecation

@fejta-bot

This comment has been minimized.

Copy link

fejta-bot commented Dec 18, 2019

/retest
This bot automatically retries jobs that failed/flaked on approved PRs (send feedback to fejta).

Review the full test history for this PR.

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

@k8s-ci-robot k8s-ci-robot merged commit 127c47c into kubernetes:master Dec 18, 2019
15 of 16 checks passed
15 of 16 checks passed
tide Not mergeable. Retesting: pull-kubernetes-kubemark-e2e-gce-big
Details
cla/linuxfoundation serathius authorized
Details
pull-kubernetes-bazel-build Job succeeded.
Details
pull-kubernetes-bazel-test Job succeeded.
Details
pull-kubernetes-conformance-kind-ipv6 Job succeeded.
Details
pull-kubernetes-dependencies Job succeeded.
Details
pull-kubernetes-e2e-gce Job succeeded.
Details
pull-kubernetes-e2e-gce-100-performance Job succeeded.
Details
pull-kubernetes-e2e-gce-device-plugin-gpu Job succeeded.
Details
pull-kubernetes-e2e-kind Job succeeded.
Details
pull-kubernetes-integration Job succeeded.
Details
pull-kubernetes-kubemark-e2e-gce-big Job succeeded.
Details
pull-kubernetes-node-e2e Job succeeded.
Details
pull-kubernetes-node-e2e-containerd Job succeeded.
Details
pull-kubernetes-typecheck Job succeeded.
Details
pull-kubernetes-verify Job succeeded.
Details
@k8s-ci-robot k8s-ci-robot added this to the v1.18 milestone Dec 18, 2019
tom1299 added a commit to tom1299/website that referenced this pull request Dec 18, 2019
Cluster monitoring user heapster has been removed as part of the merge request kubernetes/kubernetes#85512. Therefore the section about cluster monitoring as well as the link to the now missing page was removed.
kwmonroe added a commit to charmed-kubernetes/cdk-addons that referenced this pull request Jan 9, 2020
Upstream removed ./cluster/addons/cluster-monitoring in 1.18:

kubernetes/kubernetes#85512

Get rid of the related bits in our cdk-addons snap.
kwmonroe added a commit to charmed-kubernetes/cdk-addons that referenced this pull request Jan 13, 2020
* lp 1858651: remove cluster-monitoring addons

Upstream removed ./cluster/addons/cluster-monitoring in 1.18:

kubernetes/kubernetes#85512

Get rid of the related bits in our cdk-addons snap.

* remove unused dash_ig_context

* remove monitorstorage for 1.18+
tom1299 added a commit to tom1299/website that referenced this pull request Jan 23, 2020
Cluster monitoring user heapster has been removed as part of the merge request kubernetes/kubernetes#85512. Therefore the section about cluster monitoring as well as the link to the now missing page was removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.