Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[metricbeat] bump kube-state-metrics to version 4.7.0 #1524

Merged
merged 4 commits into from
Mar 1, 2022
Merged

[metricbeat] bump kube-state-metrics to version 4.7.0 #1524

merged 4 commits into from
Mar 1, 2022

Conversation

beatkind
Copy link
Contributor

@beatkind beatkind commented Dec 27, 2021

I have bumped the kube-state-metrics version referenced in the metricbeat chart. Otherwise, metricbeat does not work for k8s higher than 1.21.X.

  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Fix #1576
Replace #1577

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Dec 27, 2021

💚 CLA has been signed

@beatkind
Copy link
Contributor Author

❌ Author of the following commits did not sign a Contributor Agreement: 92137db

Please, read and sign the above mentioned agreement if you want to contribute to this project

I just signed them

@peterbosalliandercom
Copy link

We are upgrading to Openshift 4.9 and this is mandatory because of 1.22 kubernetes. Can somebody please review this PR?

@Sir-Jacques
Copy link

We're upgrading to kubernetes 1.22 soon, reviewing/approving this change would be greatly appreciated! :-)

Copy link

@peterbosalliandercom peterbosalliandercom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@peterbosalliandercom
Copy link

x Author of the following commits did not sign a Contributor Agreement: 92137db
Please, read and sign the above mentioned agreement if you want to contribute to this project

I just signed them

Looks like it is not signed correctly? Did you sign all commits?

Signed-off-by: Jan Ole Rundshagen <jan@devmoon.de>
@beatkind
Copy link
Contributor Author

beatkind commented Jan 5, 2022

x Author of the following commits did not sign a Contributor Agreement: 92137db
Please, read and sign the above mentioned agreement if you want to contribute to this project

I just signed them

Looks like it is not signed correctly? Did you sign all commits?

Sorry, I was stupid on that one.

@beatkind
Copy link
Contributor Author

beatkind commented Jan 5, 2022

@peterbosalliandercom can you re-review it please?

@lucagouty
Copy link

hello @peterbosalliandercom @beatkind, any news on this one? sorry to bump it but 1.21 end of life is getting very close

pruissen
pruissen previously approved these changes Feb 3, 2022
SimonWillemsen
SimonWillemsen previously approved these changes Feb 3, 2022
@jmlrt
Copy link
Member

jmlrt commented Mar 1, 2022

ok to test

@jmlrt jmlrt dismissed stale reviews from SimonWillemsen and pruissen via 27387ad March 1, 2022 10:37
@jmlrt jmlrt changed the title feat: bumping kube-state-metrics to version 4.2.0 - #1055 [metricbeat] bump kube-state-metrics to version 4.7.0 Mar 1, 2022
@jmlrt jmlrt mentioned this pull request Mar 1, 2022
4 tasks
@jmlrt
Copy link
Member

jmlrt commented Mar 1, 2022

ok to test

@jmlrt jmlrt merged commit 8003360 into elastic:main Mar 1, 2022
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Mar 1, 2022
* feat: bumping kube-state-metrics to version 4.2.0 - elastic#1055

Signed-off-by: Jan Ole Rundshagen <jan@devmoon.de>

* update doc and tests to use prometheus-community helm repo

* bump kube-state-metric version

Co-authored-by: jmlrt <8582351+jmlrt@users.noreply.github.com>
jmlrt added a commit that referenced this pull request Mar 1, 2022
* feat: bumping kube-state-metrics to version 4.2.0 - #1055

Signed-off-by: Jan Ole Rundshagen <jan@devmoon.de>

* update doc and tests to use prometheus-community helm repo

* bump kube-state-metric version

Co-authored-by: jmlrt <8582351+jmlrt@users.noreply.github.com>

Co-authored-by: Jan Rundshagen <beatkind@users.noreply.github.com>
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt added v7.17.1 and removed v7.17.0 labels Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Mar 8, 2022
* 7.17.1 as default version.

| PR                                                        | Author                                       | Title                                                                   |
|-----------------------------------------------------------|----------------------------------------------|-------------------------------------------------------------------------|
| [elastic#1604](elastic#1604) | [@jmlrt](https://github.com/jmlrt)           | [meta] update docker images                                             |
| [elastic#1603](elastic#1603) | [@jmlrt](https://github.com/jmlrt)           | [metricbeat] add missing rolebinding and cluster role rules             |
| [elastic#1602](elastic#1602) | [@jmlrt](https://github.com/jmlrt)           | [filebeat] add missing rolebinding and cluster role rules               |
| [elastic#1593](elastic#1593) | [@jmlrt](https://github.com/jmlrt)           | [meta] add support for k8s 1.22                                         |
| [elastic#1582](elastic#1582) | [@jmlrt](https://github.com/jmlrt)           | [kibana] fix extra values default values                                |
| [elastic#1581](elastic#1581) | [@jmlrt](https://github.com/jmlrt)           | [logstash] fix ServiceAccount inconsistencies                           |
| [elastic#1580](elastic#1580) | [@jmlrt](https://github.com/jmlrt)           | [elasticsearch] fix ServiceAccount inconsistencies                      |
| [elastic#1570](elastic#1570) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add externalTrafficPolicy support                            |
| [elastic#1569](elastic#1569) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add flexible ingress                                         |
| [elastic#1563](elastic#1563) | [@jmlrt](https://github.com/jmlrt)           | [meta] bump Helm version to 3.8.0                                       |
| [elastic#1538](elastic#1538) | [@chetanv-oi](https://github.com/chetanv-oi) | [elasticsearch] move the yaml separator inside the condition            |
| [elastic#1530](elastic#1530) | [@jmlrt](https://github.com/jmlrt)           | [kibana] use bash for readiness script                                  |
| [elastic#1527](elastic#1527) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] add pod labels                                             |
| [elastic#1524](elastic#1524) | [@beatkind](https://github.com/beatkind)     | [metricbeat] bump kube-state-metrics to version 4.7.0                   |
| [elastic#1521](elastic#1521) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] fix podLabels                                              |
| [elastic#1494](elastic#1494) | [@ebuildy](https://github.com/ebuildy)       | [elasticsearch] add keystore container securityContext                  |
| [elastic#1450](elastic#1450) | [@dmarcs](https://github.com/dmarcs)         | [logstash] allow array values for extra                                 |
| [elastic#1422](elastic#1422) | [@framsouza](https://github.com/framsouza)   | [metricbeat] adding custerRole permissions for leader election          |
| [elastic#1420](elastic#1420) | [@framsouza](https://github.com/framsouza)   | [elasticsearch] [logstash] add support to PodDisruptionBudget policy/v1 |
| [elastic#1417](elastic#1417) | [@framsouza](https://github.com/framsouza)   | [kibana] add annotations at deployment level                            |
jmlrt added a commit that referenced this pull request Mar 8, 2022
* 7.17.1 release changelog

* 7.17.1 as default version.

| PR                                                        | Author                                       | Title                                                                   |
|-----------------------------------------------------------|----------------------------------------------|-------------------------------------------------------------------------|
| [#1604](#1604) | [@jmlrt](https://github.com/jmlrt)           | [meta] update docker images                                             |
| [#1603](#1603) | [@jmlrt](https://github.com/jmlrt)           | [metricbeat] add missing rolebinding and cluster role rules             |
| [#1602](#1602) | [@jmlrt](https://github.com/jmlrt)           | [filebeat] add missing rolebinding and cluster role rules               |
| [#1593](#1593) | [@jmlrt](https://github.com/jmlrt)           | [meta] add support for k8s 1.22                                         |
| [#1582](#1582) | [@jmlrt](https://github.com/jmlrt)           | [kibana] fix extra values default values                                |
| [#1581](#1581) | [@jmlrt](https://github.com/jmlrt)           | [logstash] fix ServiceAccount inconsistencies                           |
| [#1580](#1580) | [@jmlrt](https://github.com/jmlrt)           | [elasticsearch] fix ServiceAccount inconsistencies                      |
| [#1570](#1570) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add externalTrafficPolicy support                            |
| [#1569](#1569) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add flexible ingress                                         |
| [#1563](#1563) | [@jmlrt](https://github.com/jmlrt)           | [meta] bump Helm version to 3.8.0                                       |
| [#1538](#1538) | [@chetanv-oi](https://github.com/chetanv-oi) | [elasticsearch] move the yaml separator inside the condition            |
| [#1530](#1530) | [@jmlrt](https://github.com/jmlrt)           | [kibana] use bash for readiness script                                  |
| [#1527](#1527) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] add pod labels                                             |
| [#1524](#1524) | [@beatkind](https://github.com/beatkind)     | [metricbeat] bump kube-state-metrics to version 4.7.0                   |
| [#1521](#1521) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] fix podLabels                                              |
| [#1494](#1494) | [@ebuildy](https://github.com/ebuildy)       | [elasticsearch] add keystore container securityContext                  |
| [#1450](#1450) | [@dmarcs](https://github.com/dmarcs)         | [logstash] allow array values for extra                                 |
| [#1422](#1422) | [@framsouza](https://github.com/framsouza)   | [metricbeat] adding custerRole permissions for leader election          |
| [#1420](#1420) | [@framsouza](https://github.com/framsouza)   | [elasticsearch] [logstash] add support to PodDisruptionBudget policy/v1 |
| [#1417](#1417) | [@framsouza](https://github.com/framsouza)   | [kibana] add annotations at deployment level                            |

* add breaking changes + update all toc instructions

* fix breaking change error
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Mar 9, 2022
This commit fix the Metricbeat upgrade test by bumping the source
version to 7.17.1. This is required because
elastic#1524 introduced a new
kube-state-metrics version that prevent upgrade from previous versions.
jmlrt added a commit that referenced this pull request Mar 17, 2022
This commit fix the Metricbeat upgrade test by bumping the source
version to 7.17.1. This is required because
#1524 introduced a new
kube-state-metrics version that prevent upgrade from previous versions.
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Mar 17, 2022
This commit fix the Metricbeat upgrade test by bumping the source
version to 7.17.1. This is required because
elastic#1524 introduced a new
kube-state-metrics version that prevent upgrade from previous versions.
jmlrt added a commit that referenced this pull request Mar 17, 2022
This commit fix the Metricbeat upgrade test by bumping the source
version to 7.17.1. This is required because
#1524 introduced a new
kube-state-metrics version that prevent upgrade from previous versions.
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
galina-tochilkin pushed a commit to mtp-devops/3d-party-helm that referenced this pull request Dec 20, 2022
* 7.17.1 release changelog

* 7.17.1 as default version.

| PR                                                        | Author                                       | Title                                                                   |
|-----------------------------------------------------------|----------------------------------------------|-------------------------------------------------------------------------|
| [#1604](elastic/helm-charts#1604) | [@jmlrt](https://github.com/jmlrt)           | [meta] update docker images                                             |
| [#1603](elastic/helm-charts#1603) | [@jmlrt](https://github.com/jmlrt)           | [metricbeat] add missing rolebinding and cluster role rules             |
| [#1602](elastic/helm-charts#1602) | [@jmlrt](https://github.com/jmlrt)           | [filebeat] add missing rolebinding and cluster role rules               |
| [#1593](elastic/helm-charts#1593) | [@jmlrt](https://github.com/jmlrt)           | [meta] add support for k8s 1.22                                         |
| [#1582](elastic/helm-charts#1582) | [@jmlrt](https://github.com/jmlrt)           | [kibana] fix extra values default values                                |
| [#1581](elastic/helm-charts#1581) | [@jmlrt](https://github.com/jmlrt)           | [logstash] fix ServiceAccount inconsistencies                           |
| [#1580](elastic/helm-charts#1580) | [@jmlrt](https://github.com/jmlrt)           | [elasticsearch] fix ServiceAccount inconsistencies                      |
| [#1570](elastic/helm-charts#1570) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add externalTrafficPolicy support                            |
| [#1569](elastic/helm-charts#1569) | [@jmlrt](https://github.com/jmlrt)           | [logstash] add flexible ingress                                         |
| [#1563](elastic/helm-charts#1563) | [@jmlrt](https://github.com/jmlrt)           | [meta] bump Helm version to 3.8.0                                       |
| [#1538](elastic/helm-charts#1538) | [@chetanv-oi](https://github.com/chetanv-oi) | [elasticsearch] move the yaml separator inside the condition            |
| [#1530](elastic/helm-charts#1530) | [@jmlrt](https://github.com/jmlrt)           | [kibana] use bash for readiness script                                  |
| [#1527](elastic/helm-charts#1527) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] add pod labels                                             |
| [#1524](elastic/helm-charts#1524) | [@beatkind](https://github.com/beatkind)     | [metricbeat] bump kube-state-metrics to version 4.7.0                   |
| [#1521](elastic/helm-charts#1521) | [@ebuildy](https://github.com/ebuildy)       | [apm-server] fix podLabels                                              |
| [#1494](elastic/helm-charts#1494) | [@ebuildy](https://github.com/ebuildy)       | [elasticsearch] add keystore container securityContext                  |
| [#1450](elastic/helm-charts#1450) | [@dmarcs](https://github.com/dmarcs)         | [logstash] allow array values for extra                                 |
| [#1422](elastic/helm-charts#1422) | [@framsouza](https://github.com/framsouza)   | [metricbeat] adding custerRole permissions for leader election          |
| [#1420](elastic/helm-charts#1420) | [@framsouza](https://github.com/framsouza)   | [elasticsearch] [logstash] add support to PodDisruptionBudget policy/v1 |
| [#1417](elastic/helm-charts#1417) | [@framsouza](https://github.com/framsouza)   | [kibana] add annotations at deployment level                            |

* add breaking changes + update all toc instructions

* fix breaking change error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[metricbeat] kube-state-metrics incompatible with k8s v1.22
8 participants