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

documentation for using scraper with Resource Discovery is missing a critical setting #1495

Closed
cscoon opened this issue Feb 8, 2021 · 1 comment · Fixed by #1746
Closed
Assignees
Labels
documentation All issues related to documentation on how to use it, deploy it, etc enhancement Enhancements for current features

Comments

@cscoon
Copy link

cscoon commented Feb 8, 2021

Report

I attempted to deploy the scraper and point it to my Resource Discovery deployment several times using the documentation, but it consistently failed. This was the relevant config section I passed to the scraper helm chart:

resourceDiscovery:
host: promitor-agent-resource-discovery.monitoring.svc.cluster.local
port: 8889

Each time I deployed, the scraper pod went into CrashLoopBackOff status and wrote an error message in the log. I've attached it to this issue.

After a lot of digging through the Promitor and Promitor chart repos, I found the issue - if the enabled: true property isn't included, it defaults to false, and Promitor scraper fails validation.

The problem is that not only does the documentation not mention that this property defaults to false, but the documentation I found doesn't even mention the existence of the 'resourceDiscovery.enabled' property at all, leading to several failed deployments.

The documentation should be updated to include this critical property.

Here are the documentation locations I found that show examples of 'resourceDiscovery:' configuration, but make no mention of the 'enabled' property. Please note that there may be more locations that I didn't run across:

https://promitor.io/configuration/v2.x/runtime/scraper.html
https://promitor.io/configuration/v2.x/runtime/scraper.html

Expected Behavior

I expected the code samples in the documentation to include required fields for the scraper to run when it's configured to use Resource Discovery.

Actual Behavior

The code samples did not include the critical 'enabled' field, resulting in the pod going into CrashLoopBackOff status.

Steps to Reproduce the Problem

  1. Deploy Resource Discovery.
  2. Deploy the scraper, pointing to Resource Discovery, but without the 'enabled' property.
  3. Scraper fails with attached error

Component

Scraper

Version

2.0.0

Configuration

Configuration:

# Add your scraping configuration here

Logs

example

Platform

No response

Contact Details

No response

scraper error

@cscoon cscoon added the bug Something isn't working label Feb 8, 2021
@tomkerkhove tomkerkhove added documentation All issues related to documentation on how to use it, deploy it, etc enhancement Enhancements for current features and removed bug Something isn't working labels Feb 8, 2021
@tomkerkhove
Copy link
Owner

Nice find, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation All issues related to documentation on how to use it, deploy it, etc enhancement Enhancements for current features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants