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

Update to latest serviceresolver crd #619

Merged
merged 1 commit into from
Sep 29, 2020

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Sep 24, 2020

@@ -140,7 +140,8 @@ jobs:
-kubeconfig="$primary_kubeconfig" \
-secondary-kubeconfig="$secondary_kubeconfig" \
-debug-directory="$TEST_RESULTS/debug" \
-consul-k8s-image=hashicorpdev/consul-k8s:crd-controller-base-latest
-consul-k8s-image=hashicorpdev/consul-k8s:crd-controller-base-latest \
-consul-image=hashicorpdev/consul
Copy link
Member Author

Choose a reason for hiding this comment

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

This tracks consul master

Copy link

@thisisnotashwin thisisnotashwin left a comment

Choose a reason for hiding this comment

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

💥

@lkysow lkysow force-pushed the crd-controller-resolver-latest branch from 347b3b3 to 66873f7 Compare September 24, 2020 22:54
@@ -36,6 +36,7 @@ func TestConnectInject(t *testing.T) {
"global.tls.enabled": strconv.FormatBool(c.secure),
"global.tls.enableAutoEncrypt": strconv.FormatBool(c.autoEncrypt),
"global.acls.manageSystemACLs": strconv.FormatBool(c.secure),
"global.image": "hashicorpdev/consul",
Copy link
Member Author

Choose a reason for hiding this comment

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

Will track master which has latest ServiceResolver merged.

@lkysow lkysow requested a review from ishustava September 25, 2020 18:14
@lkysow lkysow force-pushed the crd-controller-resolver-latest branch from 66873f7 to 34c1cdf Compare September 29, 2020 16:50
@lkysow lkysow merged commit 52fc5d5 into crd-controller-base Sep 29, 2020
@lkysow lkysow deleted the crd-controller-resolver-latest branch September 29, 2020 16:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants