Project

General

Profile

Bug #12365

app/services/fact_parser.rb does not find virtual interfaces with alphanum aliases

Added by Julien Pivotto almost 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Importers
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

We have in our lab virtual interfaces with the login of the developers (e.g eth0:roidelapluie). They are recognized as physical interface and overwrite etho.

Expected in the foreman:
eth0 phy
eth0_roidelapluie virt
eth0_julien virt

got:
eth0_julien phy

Associated revisions

Revision 86701faf (diff)
Added by Julien Pivotto almost 4 years ago

Fixes #12365 - Recognize virtual interfaces with alphanum aliases as such

We have in our lab virtual interfaces with the login of the developers
(e.g eth0:roidelapluie). They are recognized as physical interface and
overwrite eth0.

Expected in the foreman:
eth0 phy
eth0_roidelapluie virt
eth0_julien virt

got:
eth0_julien phy

This commit fixes that by changing the regex that checks for virtual
interfaces.

Revision aabadd90 (diff)
Added by Julien Pivotto over 3 years ago

Fixes #12365 - Recognize virtual interfaces with alphanum aliases as such

We have in our lab virtual interfaces with the login of the developers
(e.g eth0:roidelapluie). They are recognized as physical interface and
overwrite eth0.

Expected in the foreman:
eth0 phy
eth0_roidelapluie virt
eth0_julien virt

got:
eth0_julien phy

This commit fixes that by changing the regex that checks for virtual
interfaces.

(cherry picked from commit 86701faf31a16925a6c689d1bcd6630c655dada7)

History

#1 Updated by Dominic Cleal almost 4 years ago

  • Subject changed from app/services/fact_parser.rb does not find virtual interfaces with alphanum aliases to app/services/fact_parser.rb does not find virtual interfaces with alphanum aliases
  • Category set to Importers

#2 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Julien Pivotto
  • Pull request https://github.com/theforeman/foreman/pull/2883 added

#3 Updated by Anonymous almost 4 years ago

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

#4 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 104

Also available in: Atom PDF