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

Bump to 2.1.1 #56

Merged
merged 280 commits into from
Sep 14, 2021
Merged

Bump to 2.1.1 #56

merged 280 commits into from
Sep 14, 2021

Conversation

jan--f
Copy link

@jan--f jan--f commented Aug 19, 2021

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

kevinschumacher and others added 30 commits December 8, 2020 12:40
* [stable/kube-state-metrics] Add service type

* Incorporate code review feedback

* Fix the ordering of type, nodePort and loadBalancerIP
…2124)

kube-state-metrics has a collector for each kind of Kubernetes resource
it collects metrics from. Each of those collectors directly influences
the RBAC requirements for the Pods. Additionally due to the nature of
how kube-state-metrics is versioned and released it is not easy to
determine which collectors are using "alpha" APIs, which are not
necessarily enabled in a cluster, but kube-state-metrics by default has
all collectors enabled.
* Convert registry to k8s.gcr.io

* fix hardcoded version in readme
…#3231)

* [stable/kube-state-metrics] allow for nodeSelector to be configured

* increment version
… (#3595)

* [stable/kube-state-metrics] allow for namespace flag to be configured

* add newline to file
* kube-state-metrics: Update tag to 1.3.1

* kube-state-metrics: Fix invalid Chart.yaml authors

Had the author's full name rather than GitHub name. This
broke the CI checks.
* modify helpers to apply popular rule to decide fullname

* bump up chart version up to 0.7.2

* replace author name to github-account
* review readme

* Update Chart.yaml
Signed-off-by: David J. M. Karlsen <david@davidkarlsen.com>
Signed-off-by: Bart Verwilst <bart@verwilst.be>
…tycontexts) (#9010)

* Enable RBAC by default (needed for securitycontexts)

Signed-off-by: Bart Verwilst <bart@verwilst.be>

* s/True/true

Signed-off-by: Bart Verwilst <bart@verwilst.be>
Signed-off-by: Chris O'Brien <chrisob91@gmail.com>
* Fix indentation of priorityClassName

Signed-off-by: Chris O'Brien <chrisob91@gmail.com>

* Bump chart version

Signed-off-by: Chris O'Brien <chrisob91@gmail.com>
Signed-off-by: Louise Champ <louise@livewyer.com>
Signed-off-by: Derek Heldt-Werle <derek.heldt-werle@viasat.com>
Signed-off-by: Jesse Suen <jessesuen@gmail.com>
…#11316)

Signed-off-by: tariqibrahim <tariq.ibrahim@microsoft.com>
Signed-off-by: tariqibrahim <tariq.ibrahim@microsoft.com>
…11405)

* [stable/kube-state-metrics] Add imagePullSecrets in serviceAccount

Signed-off-by: Tanguy Buchier <tanguy.buchier@gmail.com>

* [stable/kube-state-metrics] Add imagePullSecrets in serviceAccount

Signed-off-by: Tanguy Buchier <tanguy.buchier@gmail.com>

* [stable/kube-state-metrics] Add imagePullSecrets in serviceAccount

Signed-off-by: Tanguy Buchier <tanguy.buchier@gmail.com>
* Added replicas to kube-state-metrics

Signed-off-by: Andreas Antonsson <andreas.antonsson@collectorbank.se>

* Bumped chart version and added replicas to readme.md Signed of by Andreas Antonsson tehho1337@gmail.com

Signed-off-by: Andreas Antonsson <andreas.antonsson@collectorbank.se>
…#12145)

* [stable/kube-state-metrics] add optional prometheus service monitor

Signed-off-by: Ryan Belgrave <ryan.belgrave@optum.com>

* bump chart version to 1.0.0

Signed-off-by: Ryan Belgrave <ryan.belgrave@optum.com>
…(#12207)

Signed-off-by: Guillermo Guirao Aguilar <contact@guillermoguiraoaguilar.com>
fpetkovski and others added 8 commits July 21, 2021 13:43
Co-authored-by: Manuel Rüger <manuel@rueg.eu>
Signed-off-by: Manuel Rüger <manuel@rueg.eu>
* [CHANGE] Replace multiListWatcher with independent listWatchers per namespace kubernetes#1499
* [CHANGE] go.mod: Update and minimize dependencies kubernetes#1529
* [BUGFIX] Use BuilderInterface instead of internal/store.Builder in metricshandler kubernetes#1537
* [BUGFIX] Add WithAllowLabels to public BuilderInterface kubernetes#1514
Signed-off-by: Damien Grisonnet <dgrisonn@redhat.com>
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 19, 2021
@jan--f
Copy link
Author

jan--f commented Aug 23, 2021

@simonpasquier Seems like the revert of 3e37e8f causes the test failure. Doe we need to carry that after all or does we need a fix for the test?

@simonpasquier
Copy link

Ah I think we got a typo: kube-state-metrics runs with --metric-labels-allowlist=pods=[*],node=[*] but the latter part should nodes=[*]

@jan--f
Copy link
Author

jan--f commented Aug 24, 2021

/retest

@jan--f
Copy link
Author

jan--f commented Aug 24, 2021

/hold
Postponed till 4.10

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 24, 2021
@jan--f
Copy link
Author

jan--f commented Sep 6, 2021

/unhold

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 6, 2021
@simonpasquier
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 14, 2021
@openshift-ci
Copy link

openshift-ci bot commented Sep 14, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jan--f, simonpasquier

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:
  • OWNERS [jan--f,simonpasquier]

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

@openshift-merge-robot openshift-merge-robot merged commit fad488c into openshift:master Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.