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

Test controller against consul:1.9.0-beta1 #645

Closed
wants to merge 1 commit into from
Closed

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Oct 13, 2020

Companion to hashicorp/consul-k8s#362. The only real change in this PR is to uncomment t.Skip for namespace tests.

@lkysow lkysow force-pushed the test-1.9.0-beta1 branch 3 times, most recently from 96a3481 to fed5496 Compare October 15, 2020 00:34
@lkysow lkysow marked this pull request as ready for review October 30, 2020 21:12
@lkysow lkysow requested review from a team, ndhanushkodi and thisisnotashwin and removed request for a team October 30, 2020 21:13
@@ -199,6 +199,7 @@ jobs:
-kubeconfig="$HOME/.kube/$OC_PRIMARY_NAME" \
-secondary-kubeconfig="$HOME/.kube/$OC_SECONDARY_NAME" \
-debug-directory="$TEST_RESULTS/debug" \
-run TestController \
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 remove when merging

// todo: remove when 1.9.0 is released.
"global.image": "hashicorpdev/consul",
"global.image": "consul:1.9.0-beta1",
"global.imageK8S": "ashwinvenkatesh/consul-k8s@sha256:2225cf62ea3c068e870ad0134dac50e4753cbcf7a1503d3d6d2d897e9fc78aa1",
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 remove when merging

"global.image": "hashicorp/consul-enterprise:1.8.4-ent",
// todo: remove when Helm chart updates to 1.9.0
"global.image": "hashicorp/consul-enterprise:1.9.0-ent-beta1",
"global.imageK8S": "ashwinvenkatesh/consul-k8s@sha256:2225cf62ea3c068e870ad0134dac50e4753cbcf7a1503d3d6d2d897e9fc78aa1",
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 remove when merging

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.

Woohoo!! Love not having to skip tests!! 🦊

@lkysow
Copy link
Member Author

lkysow commented Nov 9, 2020

How many PRs for this one change did I open up?!? 🤯 #676 was merged so closing this.

@lkysow lkysow closed this Nov 9, 2020
@lkysow lkysow deleted the test-1.9.0-beta1 branch November 9, 2020 22:40
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.

2 participants