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

acceptance-tests: disable DNS by default in tests to avoid false positives #1058

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

ishustava
Copy link
Contributor

Changes proposed in this PR:

How I've tested this PR:
🐢

How I expect reviewers to test this PR:

  • code review

Checklist:

  • Bats tests added
  • CHANGELOG entry added (HashiCorp engineers only, community PRs should not add a changelog entry)

@ishustava ishustava force-pushed the acceptance-tests-disable-dns branch from 7609ce1 to 76119e4 Compare July 27, 2021 19:45
@ishustava ishustava requested review from a team, ndhanushkodi and thisisnotashwin and removed request for a team July 27, 2021 20:23
@@ -55,7 +55,7 @@ commands:
type: string
consul-k8s-image:
type: string
default: "docker.mirror.hashicorp.services/hashicorpdev/consul-k8s:latest"
default: "hashicorpdev/consul-k8s:2dfffed"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

undo when hashicorp/consul-k8s#576 is merged

Copy link
Contributor

@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.

heh!! that was a clever refactor!! LGTM

@ishustava ishustava merged commit 1eaabb7 into master Jul 30, 2021
@ishustava ishustava deleted the acceptance-tests-disable-dns branch July 30, 2021 00:55
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