Skip to content
This repository has been archived by the owner on May 5, 2023. It is now read-only.

Remove the datasource default value for the prometheus target #11

Closed
wants to merge 1 commit into from
Closed

Remove the datasource default value for the prometheus target #11

wants to merge 1 commit into from

Conversation

andrein
Copy link

@andrein andrein commented Mar 9, 2021

This is inspired by #9, but instead of removing the datasource parameter
completely, it only removes the default value.
This is a good compromise, since if you're using the mixed datasource,
you're probably going to want to customize the datasource on the target
level anyway, so a default value doesn't make much sense.

This is inspired by #9, but instead of removing the datasource parameter
completely, it only removes the default value.
This is a good compromise, since if you're using the mixed datasource,
you're probably going to want to customize the datasource on the target
level anyway, so a default value doesn't make much sense.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@andrein andrein closed this by deleting the head repository Nov 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants