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

feat: add the build tag to the member (serf_lan/wan) labels #228

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ylmrx
Copy link

@ylmrx ylmrx commented May 25, 2022

We need those here, they could prove useful to everyone ! :)

Take care <3

@ylmrx ylmrx force-pushed the feat/add_build_label_to_members branch from c92c31c to 08354df Compare May 25, 2022 16:43
ylmrx added 2 commits May 25, 2022 18:57
Signed-off-by: ylamouroux <ylamouroux@veepee.com>
Signed-off-by: ylamouroux <ylamouroux@veepee.com>
@ylmrx ylmrx force-pushed the feat/add_build_label_to_members branch from ba7bc36 to 413d3b8 Compare May 25, 2022 16:57
@ylmrx
Copy link
Author

ylmrx commented May 25, 2022

@simonpasquier Hello Simon, we'd like to propose this tiny bit of code, I'm failing to reach for the variable (ie. consul version) from the test.go code though, as I'm super not used to this CI :)

@SuperQ
Copy link
Member

SuperQ commented Mar 29, 2023

This should be a separate info metric, not attached to the status metric.

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.

None yet

2 participants