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

Added support in nmcli for ipv4.dns-options #6902

Merged
merged 9 commits into from
Jul 13, 2023
Merged

Added support in nmcli for ipv4.dns-options #6902

merged 9 commits into from
Jul 13, 2023

Conversation

yan12125
Copy link
Contributor

@yan12125 yan12125 commented Jul 9, 2023

SUMMARY

The nmcli command has an important option "ipv4.dns-options", This PR adds the support for this option in the nmcli module

Closes #4308

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

nmcli

ADDITIONAL INFORMATION

This a rebased and improved version of #4308.


@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module plugins plugin (any type) tests tests unit tests/unit labels Jul 9, 2023
Copy link
Collaborator

@russoz russoz left a comment

Choose a reason for hiding this comment

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

LGTM

@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-7 labels Jul 9, 2023
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks! Will merge by the end of this week if nobody objects.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Jul 13, 2023
@felixfontein felixfontein merged commit cc8e2d6 into ansible-collections:main Jul 13, 2023
@patchback
Copy link

patchback bot commented Jul 13, 2023

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/cc8e2d676abda7d7dbef2a1411b85ed8a992fbc6/pr-6902

Backported as #6926

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jul 13, 2023
* Added support for ipv4.dns_options in nmcli module

* added support for dns6-options

* Added version added

Co-authored-by: Felix Fontein <[email protected]>

* added version_added: 4.6.0 for dns6 options

* added changelog fragment

* Rename 4308-added-support-in-nmcli-for-ipvx-dns-options to 4308-added-support-in-nmcli-for-ipvx-dns-options.yml

* Update changelogs/fragments/4308-added-support-in-nmcli-for-ipvx-dns-options.yml

Co-authored-by: Felix Fontein <[email protected]>

* Fix and add tests

* Update PR number and version_added

---------

Co-authored-by: Matteo Caruso <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit cc8e2d6)
@felixfontein
Copy link
Collaborator

@yan12125 thanks for your contribution!
@russoz thanks for reviewing!

felixfontein pushed a commit that referenced this pull request Jul 13, 2023
…4.dns-options (#6926)

Added support in nmcli for ipv4.dns-options (#6902)

* Added support for ipv4.dns_options in nmcli module

* added support for dns6-options

* Added version added

Co-authored-by: Felix Fontein <[email protected]>

* added version_added: 4.6.0 for dns6 options

* added changelog fragment

* Rename 4308-added-support-in-nmcli-for-ipvx-dns-options to 4308-added-support-in-nmcli-for-ipvx-dns-options.yml

* Update changelogs/fragments/4308-added-support-in-nmcli-for-ipvx-dns-options.yml

Co-authored-by: Felix Fontein <[email protected]>

* Fix and add tests

* Update PR number and version_added

---------

Co-authored-by: Matteo Caruso <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit cc8e2d6)

Co-authored-by: Chih-Hsuan Yen <[email protected]>
@yan12125 yan12125 deleted the pr-4308 branch July 13, 2023 21:24
valeriopoggi pushed a commit to valeriopoggi/community.general that referenced this pull request Jul 17, 2023
* Added support for ipv4.dns_options in nmcli module

* added support for dns6-options

* Added version added

Co-authored-by: Felix Fontein <[email protected]>

* added version_added: 4.6.0 for dns6 options

* added changelog fragment

* Rename 4308-added-support-in-nmcli-for-ipvx-dns-options to 4308-added-support-in-nmcli-for-ipvx-dns-options.yml

* Update changelogs/fragments/4308-added-support-in-nmcli-for-ipvx-dns-options.yml

Co-authored-by: Felix Fontein <[email protected]>

* Fix and add tests

* Update PR number and version_added

---------

Co-authored-by: Matteo Caruso <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request module module plugins plugin (any type) tests tests unit tests/unit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants