Bug #16935

updating a NIC's "Device identifier" if it doesn't already have one fails if the "Device identifier" includes a . or :

Added by Sean O'Keeffe 5 months ago. Updated 4 months ago.

Status:Closed
Priority:Normal
Assigned To:Sean O'Keeffe
Category:Network
Target version:-
Difficulty: Bugzilla link:1384980
Found in release: Pull request:https://github.com/theforeman/foreman/pull/3942
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

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

Related to Foreman - Bug #10566: Unique interface identifier requirement prevents host fro... Closed 05/20/2015
Related to Foreman - Bug #17494: Test fails because of the stray `end` in the test file Closed 11/27/2016
Related to Foreman - Bug #13161: bond0.vlan breaks fact upload Feedback 01/12/2016

Associated revisions

Revision 16cbc5c3
Added by Sean O'Keeffe 4 months ago

Fixes #16935 - Allow : & . device ID if there isnt one set

History

#1 Updated by The Foreman Bot 5 months 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 5 months ago

I forgot to add downstream bug link: https://bugzilla.redhat.com/show_bug.cgi?id=1384980

#3 Updated by Dominic Cleal 5 months ago

  • Related to Bug #10566: Unique interface identifier requirement prevents host from being edited if duplicate identifier exists added

#4 Updated by Dominic Cleal 5 months 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 5 months ago

  • Category changed from Orchestration to Network

#6 Updated by Daniel Lobato Garcia 5 months ago

  • Bugzilla link set to 1384980

#7 Updated by Marek Hulán 4 months ago

  • Release set to 1.14.0

#8 Updated by Sean O'Keeffe 4 months ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#9 Updated by Dominic Cleal 4 months ago

  • Related to Bug #17494: Test fails because of the stray `end` in the test file added

#10 Updated by Marek Hulán 4 months ago

Did you also find out why the tests in this PR succeeded?

#11 Updated by Marek Hulán about 1 month ago

  • Related to Bug #13161: bond0.vlan breaks fact upload added

Also available in: Atom PDF