Bug #16935
updating a NIC's "Device identifier" if it doesn't already have one fails if the "Device identifier" includes a . or :
Description
Description of problem:
updating a NIC's "Device identifier" if it doesn't already have one fails if the "Device identifier" includes a . or :
Version-Release number of selected component (if applicable):
nightly
How reproducible:
100%
Steps to Reproduce:
1. Create a new ( dont set Device identifier on the NIC)
2. edit host and try to set Device identifier to eth1:210 for example
3.
Actual results:
Error: Can't add or remove : from identifier
Expected results:
to be able to set it.
Additional info
Related issues
Associated revisions
Fixes #16935 - Allow : & . device ID if there isnt one set
(cherry picked from commit 16cbc5c31d1bc12b10ebf2d9bb40e5b0e33d9e06)
History
#1
Updated by The Foreman Bot over 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/3942 added
#2
Updated by Sean O'Keeffe over 5 years ago
I forgot to add downstream bug link: https://bugzilla.redhat.com/show_bug.cgi?id=1384980
#3
Updated by Dominic Cleal over 5 years ago
- Related to Bug #10566: Unique interface identifier requirement prevents host from being edited if duplicate identifier exists added
#4
Updated by Dominic Cleal over 5 years ago
Linked bug explicitly prevented interfaces from changing their type, by adding an identifier to a physical interface to change it to a virtual interface.
#5
Updated by Marek Hulán over 5 years ago
- Category changed from Orchestration to Network
#6
Updated by Daniel Lobato Garcia over 5 years ago
- Bugzilla link set to 1384980
#7
Updated by Marek Hulán over 5 years ago
- Legacy Backlogs Release (now unused) set to 189
#8
Updated by Sean O'Keeffe over 5 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 16cbc5c31d1bc12b10ebf2d9bb40e5b0e33d9e06.
#9
Updated by Dominic Cleal over 5 years ago
- Related to Bug #17494: Test fails because of the stray `end` in the test file added
#10
Updated by Marek Hulán over 5 years ago
Did you also find out why the tests in this PR succeeded?
#11
Updated by Marek Hulán over 5 years ago
- Related to Bug #13161: bond0.vlan breaks fact upload added
Fixes #16935 - Allow : & . device ID if there isnt one set