Replace ping command and output with dig command and output for DNS resolution #3617
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
|
The PR preview for bac1b7b is available at theforeman-foreman-documentation-preview-pr-3617.surge.sh The following output files are affected by this PR: |
| ---- | ||
|
|
||
| . To avoid discrepancies with static and transient host names, set all the host names on the system by entering the following command: | ||
| . To avoid discrepancies with static and transient host names, set all the host names on the system with the following command: |
There was a problem hiding this comment.
This might work even better:
Suggested change
| . To avoid discrepancies with static and transient host names, set all the host names on the system with the following command: | |
| . To avoid discrepancies with static and transient host names, set all the host names on the system: |
aneta-petrova
added
style review done
|
@Levi-Leah In the cherry-pick list, you selected Satellite versions only. I don't think there is a need to do that, is there? If not, then please check all versions, even the upstream-only ones. |
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.
What changes are you introducing?
Replaced the ping command and its related output with the dig command, as dig is more appropriate for demonstrating DNS resolution.
Why are you introducing these changes? (Explanation, links to references, issues, etc.)
Since the goal is to demonstrate DNS resolution, the dig command is a more accurate and reliable tool for this purpose.
Bug SAT-28901 (public)
Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)
Alternatively, ping command can be replaced with nslookup command.
Checklists
Please cherry-pick my commits into: