Skip to content

Add Plus metrics receiver log messages#1078

Merged
dhurley merged 5 commits intomainfrom
update-plus-metrics-receiver
Jun 5, 2025
Merged

Add Plus metrics receiver log messages#1078
dhurley merged 5 commits intomainfrom
update-plus-metrics-receiver

Conversation

@NutsaB
Copy link
Contributor

@NutsaB NutsaB commented May 14, 2025

Proposed changes

  • Logging a warning if Plus API is not configured and OSS scraper is used for scraping metrics
  • Warning If both Plus API and stub status endpoints are not configured and metrics are disabled

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • I have run make install-tools and have attached any dependency changes to this pull request
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • If applicable, I have updated any relevant documentation (README.md)
  • If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

@NutsaB NutsaB self-assigned this May 14, 2025
@NutsaB NutsaB requested a review from a team as a code owner May 14, 2025 14:54
@github-actions github-actions bot added the chore Pull requests for routine tasks label May 14, 2025
@dhurley dhurley added the v3.x Issues and Pull Requests related to the major version v3 label May 21, 2025
@dhurley dhurley marked this pull request as draft May 21, 2025 15:27
@dhurley dhurley marked this pull request as ready for review June 4, 2025 09:48
@dhurley dhurley merged commit 30cd8db into main Jun 5, 2025
18 checks passed
@dhurley dhurley deleted the update-plus-metrics-receiver branch June 5, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Pull requests for routine tasks v3.x Issues and Pull Requests related to the major version v3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants