Skip to content

monitoring: enable ServiceMonitor by default - #12

Merged
BacLuc merged 1 commit into
masterfrom
enable-servicemonitor-by-default
Jul 27, 2026
Merged

BacLuc merged 1 commit into
masterfrom
enable-servicemonitor-by-default

Conversation

@BacLuc

@BacLuc BacLuc commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.

@BacLuc BacLuc added enhancement New feature or request bump:minor labels Jul 24, 2026
@github-actions

github-actions Bot commented Jul 24, 2026 •

Copy link
Copy Markdown

🚀 This PR has been released as v2.2.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@martini-source
martini-source requested a review from a team July 24, 2026 09:05
@martini-source

Copy link
Copy Markdown

looks good to me; requested review from Aldebaran since they maintain it and I'm not sure if there is a reason they haven't set is as default.

@BacLuc
BacLuc requested a review from bastjan July 24, 2026 09:11

@simu simu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, assuming that you've tested this change on a real cluster.

@BacLuc

BacLuc commented Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

Tested it on the lab. Targets are up:

image

and we have metrics:
image

@BacLuc
BacLuc merged commit 9a45a2c into master Jul 27, 2026
10 checks passed
@BacLuc
BacLuc deleted the enable-servicemonitor-by-default branch July 27, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants