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

fix(kps): Remove duplicate apiserver / unused keepalived target (2.4) #1491

Merged
merged 3 commits into from
Aug 17, 2023

Conversation

github-actions[bot]
Copy link
Contributor

Backport

This will backport the following commits from main to release-2.4:

Questions ?

Please refer to the Backport tool documentation

…ig (#1483)

* fix(kps): Remove duplicate apiserver scrapeconfig

It looks like the KPS chart already defines a servicemonitor which
scrapes the apiserver metrics. Our additional scrapeconfig definition is
resulting in scraping the same apiserver targets and generating
duplicate metrics. We can remove our scrape config and rely on the chart
created servicemonitor.

* fix(kps): Remove unnecessary keepalived scrape target

(cherry picked from commit 90b170c)
@gracedo gracedo added ready-for-review ok-to-test Signals mergebot that CI checks are ready to be kicked off backport-pr This is a backport PR labels Aug 16, 2023
@gracedo gracedo changed the title [release-2.4] fix(kps): Remove duplicate apiserver and unused keepalived scrapeconfig (#1483) fix(kps): Remove duplicate apiserver and unused keepalived scrapeconfig (2.4) Aug 16, 2023
@gracedo gracedo changed the title fix(kps): Remove duplicate apiserver and unused keepalived scrapeconfig (2.4) fix(kps): Remove duplicate apiserver / unused keepalived scrapeconfig (2.4) Aug 16, 2023
* ci(pre-commit): Ignore commit title length in gitlint config, validate in PR title instead

* fix(gha): Remove invalid pull_request_target action for pr linter
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Aug 16, 2023
@gracedo gracedo changed the title fix(kps): Remove duplicate apiserver / unused keepalived scrapeconfig (2.4) fix(kps): Remove duplicate apiserver / unused keepalived target (2.4) Aug 16, 2023
@gracedo gracedo enabled auto-merge (squash) August 16, 2023 23:19
@gracedo gracedo disabled auto-merge August 17, 2023 16:07
@gracedo gracedo merged commit 92c75cb into release-2.4 Aug 17, 2023
8 checks passed
@gracedo gracedo deleted the backport/release-2.4/pr-1483 branch August 17, 2023 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-pr This is a backport PR ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants