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

prometheus metrics for brozzler, plus yt-dlp #287

Closed
wants to merge 15 commits into from

Conversation

galgeek
Copy link
Contributor

@galgeek galgeek commented Sep 11, 2024

No description provided.

@galgeek galgeek force-pushed the new_metrics branch 2 times, most recently from b3e08b9 to 362b6fd Compare September 13, 2024 01:02
@galgeek galgeek self-assigned this Sep 13, 2024
@galgeek galgeek force-pushed the new_metrics branch 2 times, most recently from 776840c to d83262f Compare September 13, 2024 16:17
Copy link

@wilsoniya wilsoniya left a comment

Choose a reason for hiding this comment

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

Thanks for using http-sd-registry!

brozzler/cli.py Show resolved Hide resolved
brozzler/cli.py Show resolved Hide resolved
default=None,
help="Prometheus scrape target registry URL",
)
arg_parser.add_argument(

Choose a reason for hiding this comment

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

Consider adding:

    choices=["qa", "prod"],

brozzler/cli.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
brozzler/metrics.py Show resolved Hide resolved
@galgeek
Copy link
Contributor Author

galgeek commented Sep 19, 2024

Note: I'm focusing now on updating #289, which replaces this, and plan to close this PR soon.

@galgeek galgeek removed their assignment Sep 19, 2024
@galgeek
Copy link
Contributor Author

galgeek commented Sep 24, 2024

closing in favor of #289

@galgeek galgeek closed this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants