Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Add datacenter flag to controller deployment #626

Merged
merged 2 commits into from
Oct 1, 2020

Conversation

thisisnotashwin
Copy link

This is a companion PR to hashicorp/consul-k8s#343 which passes the datacenter flag to the controller.

@thisisnotashwin thisisnotashwin requested review from lkysow, a team and kschoche and removed request for a team September 30, 2020 19:54
Comment on lines 143 to 144
-consul-k8s-image=ashwinvenkatesh/consul-k8s:meta@sha256:531a8865c4c585b53962bef8ff7e7df375741b16ee4f012d53f348deb30ba5f9 \
-consul-image=ashwinvenkatesh/consul:meta@sha256:ba215ab440990e19e87f5fa0952f981dcbf2605175b97899dfec259ee1abeca5
Copy link
Author

Choose a reason for hiding this comment

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

These will be reset before the code gets merged

Copy link
Contributor

@kschoche kschoche left a comment

Choose a reason for hiding this comment

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

💥 🧨 Looks great!

@thisisnotashwin thisisnotashwin force-pushed the add-metadata-to-crds branch 3 times, most recently from 51d0fb5 to 9dd843e Compare October 1, 2020 13:56
@lkysow
Copy link
Member

lkysow commented Oct 1, 2020

@ashwin-venkatesh we need this merged I think for the tests to pass now cuz we've made that flag mandatory.

@thisisnotashwin thisisnotashwin merged commit cc03ef6 into crd-controller-base Oct 1, 2020
@thisisnotashwin thisisnotashwin deleted the add-metadata-to-crds branch October 1, 2020 22:52
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.

3 participants