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

Support Nomad canary deployment #9216

Merged
merged 5 commits into from Aug 1, 2022
Merged

Conversation

rtribotte
Copy link
Member

@rtribotte rtribotte commented Jul 28, 2022

What does this PR do?

This PR adds support for Nomad canary deployment.

Motivation

When Nomad orchestrator is a provider (of service registration) for Traefik,
one might have the need to distinguish within Traefik between a Canary instance of a service, or a production one.
For example if one does not want them to be part of the same load-balancer.

Therefore, this option, which is meant to be provided as one of the values of the canary_tags field in the Nomad service stanza, allows Traefik to identify that the associated instance is a canary one.

Fixes #8987
Supersedes #9012

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Co-authored-by: Kevin Pollet pollet.kevin@gmail.com
Co-authored-by: Mathieu Lonjaret mathieu.lonjaret@gmail.com

@apollo13
Copy link
Contributor

I have tested it and it works great 👍

Copy link
Member

@kevinpollet kevinpollet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM but I don't agree with oneliners if

@traefiker traefiker merged commit 2a2ea75 into traefik:master Aug 1, 2022
v2 automation moved this from To review to Done Aug 1, 2022
@rtribotte rtribotte deleted the canary-tag branch August 3, 2022 08:36
@AmirL

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v2
Done
Development

Successfully merging this pull request may close these issues.

Need a way to define or alter the traefik service name based on consul tags
7 participants