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

Add image name meta label to DO service discovery #8497

Merged
merged 2 commits into from
Feb 16, 2021
Merged

Add image name meta label to DO service discovery #8497

merged 2 commits into from
Feb 16, 2021

Conversation

Amygos
Copy link
Contributor

@Amygos Amygos commented Feb 16, 2021

The label __meta_digitalocean_image expose the slug of the image and
the slug is only present in the public images.
To refer a user-generated image (snapshot or custom) we can use
the image's display name.

See: https://developers.digitalocean.com/documentation/v2/#images

Signed-off-by: Matteo Valentini matteo.valentini@nethesis.it

The label `__meta_digitalocean_image` expose the `slug` of the image and
the `slug` is only present in the public images.
To refer a user-generated image (`snapshot` or `custom`) we can use
the image's display name.

See: https://developers.digitalocean.com/documentation/v2/#images

Signed-off-by: Matteo Valentini <matteo.valentini@nethesis.it>
Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

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

Thanks, I propose some adjustments to the documentation.

docs/configuration/configuration.md Outdated Show resolved Hide resolved
@roidelapluie
Copy link
Member

Sorry to ask that, but can you fix the DCO?

Co-authored-by: Julien Pivotto <roidelapluie@gmail.com>
Signed-off-by: Matteo Valentini <matteo.valentini@nethesis.it>
@roidelapluie roidelapluie merged commit 2ac8029 into prometheus:master Feb 16, 2021
@roidelapluie
Copy link
Member

Thanks!!

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