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

[PR #5291/ec9e10d6 backport][stable-5] nmcli: fix error when setting previously unset mac address #5318

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Sep 28, 2022

This is a backport of PR #5291 as merged into main (ec9e10d).

SUMMARY

This adds an if to avoid errors when setting a mac address in an existing connections that didn't have it and does the same thing for two other parameters.

Fixes #5290

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

nmcli

ADDITIONAL INFORMATION

N/A

* fix #5290

* add changelog fragment

* remove unnecessary braces

* Update changelogs/fragments/5291-fix-nmcli-error-when-setting-unset-mac-address.yaml

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

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit ec9e10d)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug module module net_tools new_contributor Help guide this first time contributor plugins plugin (any type) labels Sep 28, 2022
@felixfontein felixfontein merged commit d25b6e7 into stable-5 Sep 28, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-5/ec9e10d6d1dc83cb47bf30fb7fd14af65eccc6a9/pr-5291 branch September 28, 2022 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module net_tools new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants