Project

General

Profile

Bug #16520

associate VM - MAC compare is case senstive

Added by Dan Foster over 6 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Low
Category:
Compute resources
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Something that just caught me out, when associating a Xen VM with a foreman Host. The MAC comparison is case sensitive. I ended up with with the MAC in XenCenter recorded as "02:00:89:DE:AB:43" but the MAC recorded in Foreman as "02:00:89:de:ab:43", this caused the error 'No Host found to associate this vm with". Changing the MAC to be lower case is XenCentre solved the issue. It would be better if this comparison was case insensitive.

Associated revisions

Revision 428e62ad (diff)
Added by Michael Moll over 6 years ago

refs #16520 - use normalize_mac for association

Revision 04327448 (diff)
Added by Dominik Hlavac Duran almost 4 years ago

Fixes #16520 - Associating VM-mac is case sensitive

History

#1 Updated by Anonymous over 6 years ago

  • Project changed from Xen to Foreman
  • Category set to Compute resources

associate_by() is in Foreman core

#2 Updated by Anonymous over 6 years ago

#3 Updated by Adam Winberg about 5 years ago

Just got hit with this with Vsphere, where MAC address has been set manually in vcenter using uppercase, and Foreman uses lowercase. Thus I cant associate the VM with the host in Foreman ("No host found to associate this VM with").

This is on RHEL7, Foreman 1.16.0.

#4 Updated by Ondřej Pražák about 5 years ago

  • Bugzilla link set to 1524622

#5 Updated by Dominik Hlavac Duran about 4 years ago

  • Assignee set to Dominik Hlavac Duran

#6 Updated by The Foreman Bot about 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6535 added

#7 Updated by Tomer Brisker almost 4 years ago

  • Fixed in Releases 1.22.0 added

#8 Updated by Dominik Hlavac Duran almost 4 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF