Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dig.cname function #61992

Merged
merged 4 commits into from
May 5, 2022
Merged

Conversation

nicholasmhughes
Copy link
Collaborator

What does this PR do?

This PR adds a function to the dig execution module which supports returning CNAME records for a given host.

What issues does this PR fix or reference?

Fixes: #61991

New Behavior

New function to support CNAME record queries via dig.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@nicholasmhughes nicholasmhughes requested a review from a team as a code owner April 27, 2022 14:42
@nicholasmhughes nicholasmhughes requested review from dwoz and removed request for a team April 27, 2022 14:42
@nicholasmhughes
Copy link
Collaborator Author

re-run pr-fedora-34-x86_64-py3-pytest

@nicholasmhughes
Copy link
Collaborator Author

re-run pr-ubuntu-2004-amd64-py3-pycryptodome-pytest

salt/modules/dig.py Show resolved Hide resolved
salt/modules/dig.py Show resolved Hide resolved
@Ch3LL Ch3LL added the Phosphorus v3005.0 Release code name and version label Apr 28, 2022
@nicholasmhughes
Copy link
Collaborator Author

re-run pr-alma-8-x86_64-py3-pytest

@nicholasmhughes
Copy link
Collaborator Author

re-run pr-centos-7-x86_64-py3-pytest

@nicholasmhughes
Copy link
Collaborator Author

re-run pr-centosstream-8-x86_64-py3-pytest

@nicholasmhughes
Copy link
Collaborator Author

re-run pr-debian-9-amd64-py3-pytest

@Ch3LL Ch3LL merged commit 0bb5940 into saltstack:master May 5, 2022
@nicholasmhughes nicholasmhughes deleted the add-dig-cname branch May 5, 2022 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Phosphorus v3005.0 Release code name and version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] add CNAME support to the dig module
2 participants