Terratest based end-to-end pipeline #91
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Resolves #48 * Utilizes kind action from helm org * Rebind host dns ports to 5053 and 5054 to avoid conflicts with running DNS server Must have for Github Actions, more safe for local scenarion as well. * Include stable repo to nginx-ingress install * Adjust hostAlias to non-standard dns port scenario * Reduce number of kind workers for terratest * Make hostAlias ip resolution dynamic * Optimize assertGslbStatus with smarter waiting * Fix parallel terratests for parallel execution * Avoid ingress same name conflic for failover tests * Enhance app deployment function * Enable all parallel terratest run in the pipeline
This was referenced Apr 18, 2020
kuritka
reviewed
Apr 19, 2020
| @@ -13,5 +13,5 @@ nodes: | |||
| hostPort: 444 | |||
| protocol: TCP | |||
| - containerPort: 53 | |||
| hostPort: 54 | |||
| hostPort: 5054 | |||
There was a problem hiding this comment.
I must update this in Readme
dig @localhost localtargets.app3.cloud.example.com -p 5053 && dig -p 5054 @localhost localtargets.app3.cloud.example.com
kuritka
approved these changes
Apr 19, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves Add full end to end integration tests to build pipeline #48
Utilizes kind action from helm org
Rebind host dns ports to 5053 and 5054
to avoid conflicts with running DNS server
Must have for Github Actions, more safe
for local scenarion as well.
Include stable repo to nginx-ingress install
Adjust hostAlias to non-standard dns port scenario
Reduce number of kind workers for terratest
Make hostAlias ip resolution dynamic
Optimize assertGslbStatus with smarter waiting
Fix parallel terratests for parallel execution
Avoid ingress same name conflic for failover tests
Enhance app deployment function
Enable all parallel terratest run in the pipeline