This repository was archived by the owner on Jun 29, 2022. It is now read-only.
Packet worker/controller: improve wait-for-dns output#735
Merged
Conversation
invidian
suggested changes
Jul 27, 2020
Member
invidian
left a comment
There was a problem hiding this comment.
Thanks for the PR @marga-kinvolk. Please run make update-assets and commit generated code together with your changes, to make the CI pass.
I also pointed one suggestion in the comment.
assets/lokomotive-kubernetes/packet/flatcar-linux/kubernetes/workers/cl/worker.yaml.tmpl
Outdated
Show resolved
Hide resolved
59801f7 to
07e743b
Compare
ipochi
suggested changes
Jul 27, 2020
Member
ipochi
left a comment
There was a problem hiding this comment.
Thanks for the PR @marga-kinvolk
+1 for make update-assets , even running make should suffice. Generated files is pkg/assets/generated_assets.go
dig returns 0 even if the domain passed doesn't exist. So the script as is was printing `Failed with exit code 0:` which was not a useful error message. Instead, modify these scripts to print the actual error (that the nameservers weren't found) when the output is empty.
07e743b to
de44146
Compare
Contributor
Author
|
Sorry, I missed the callout to |
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
dig returns 0 even if the domain passed doesn't exist. So the script as is was printing
Failed with exit code 0:when there was a nameserver configuration error, which was not a useful error message:Instead, modify these scripts to print the actual error (that the nameservers weren't found) when the output is empty.