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

Customizable Consul sync tag (fixes #42) #39

Closed
wants to merge 1 commit into from

Conversation

r2bit
Copy link
Contributor

@r2bit r2bit commented Nov 29, 2018

Customizable Consul catalog sync tag makes it possible to share a single Consul cluster with more than one K8S cluster. With the hardcoded tag ("k8s"), all catalog-sync processes would delete each other's services from Consul.
Controlled via "-consul-k8s-tag" flag.

@Ginja Ginja mentioned this pull request Dec 7, 2018
@r2bit r2bit force-pushed the configurable-sync-tag branch from b33e67a to 6f32331 Compare December 10, 2018 17:58
@r2bit r2bit changed the title Customizable Consul sync tag Customizable Consul sync tag (fixes #42) Dec 10, 2018
@tvi
Copy link

tvi commented Dec 26, 2018

@mitchellh would it be possible to get review on this? and potentially get it released? Thanks

@jacksontj
Copy link

👍 from me, this is a requirement before we could use this.

@jacksontj
Copy link

cc @adilyse as you seem to be the most recently active :)

Copy link
Contributor

@adilyse adilyse 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 the great contribution @r2bit ! It doesn't need any code changes, but I'm going to rebase it first since I've just changed some things in these same files.

@adilyse
Copy link
Contributor

adilyse commented Jan 8, 2019

Merged the rebased version with attribution in #51. Thanks again for a great contribution!

@adilyse adilyse closed this Jan 8, 2019
@r2bit r2bit deleted the configurable-sync-tag branch January 15, 2019 14:13
ndhanushkodi pushed a commit to ndhanushkodi/consul-k8s that referenced this pull request Jul 9, 2021
Fix connect inject configuration name
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.

4 participants