Bug 953563 - Host Creation - all valid mac address notations should be usable.
Summary: Host Creation - all valid mac address notations should be usable.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: Nightly
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: Unspecified
Assignee: Tomas Strachota
QA Contact: Corey Welton
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-04-18 13:30 UTC by Corey Welton
Modified: 2019-09-25 20:45 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-04-24 17:10:15 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 3170 0 None None None 2016-04-22 15:13:06 UTC
Red Hat Bugzilla 971012 0 unspecified CLOSED New Host: Foreman should strip extraneous whitespace/return friendly error if MAC addr entered with preceding spaces 2021-02-22 00:41:40 UTC

Internal Links: 971012

Description Corey Welton 2013-04-18 13:30:19 UTC
Description of problem:
Currently when creating a host, mac addresses can only be entered using colon-separated notation.  We should be more flexible than this.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.  Begin creation of host in foreman portion of build
2.  Attempt to enter a mac address, "50-81-AB-92-B1-20"
3.  fill in the rest of your data; submit.
  
Actual results:

User is told that the submitted format is invalid.  If user enters it colon-separated (i.e., 50:81:AB:92:B1:20) it is accepted.

Expected results:

All valid formats are accepted.  Note that I don't really care which way it is saved.... user can enter it dash-separated and it can be saved in a colon separated format.  But we should allow it.

Additional info:

See also - http://en.wikipedia.org/wiki/MAC_address#Notational_conventions

Comment 1 Ohad Levy 2013-04-21 18:24:00 UTC
actually, just fyi, foreman also accepts macs without seperation (e.g. aabbccddeeff)

Comment 2 Bryan Kearney 2013-05-28 19:45:18 UTC
Based on Comment 1, moving this to ON_QA.

Comment 3 Corey Welton 2013-06-05 13:01:53 UTC
Not fixed, or at least not in current puddle.

entering "50-81-AB-92-B1-20" as a MAC address returns:

can't be blank and is invalid

Comment 4 Corey Welton 2013-06-05 13:02:35 UTC
* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.8.9-1.el6_4.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.8.9-1.el6_4.noarch
* candlepin-tomcat6-0.8.9-1.el6_4.noarch
* elasticsearch-0.19.9-8.el6sat.noarch
* foreman-1.1.10002-44.noarch
* foreman-ec2-1.1.10002-44.noarch
* foreman-installer-puppet-concat-0-2.d776701.git.0.21ef926.el6sat.noarch
* foreman-installer-puppet-dhcp-0-5.3a4a13c.el6sat.noarch
* foreman-installer-puppet-dns-0-7.fcae203.el6sat.noarch
* foreman-installer-puppet-foreman-0-6.568c5c4.el6sat.noarch
* foreman-installer-puppet-foreman_proxy-0-8.bd1e35d.el6sat.noarch
* foreman-installer-puppet-puppet-0-3.ab46748.el6sat.noarch
* foreman-installer-puppet-tftp-0-5.ea6c5e5.el6sat.noarch
* foreman-installer-puppet-xinetd-0-50a267b8.git.0.44aca6a.el6sat.noarch
* foreman-libvirt-1.1.10002-44.noarch
* foreman-postgresql-1.1.10002-44.noarch
* foreman-proxy-1.1.10002-1.el6sat.noarch
* foreman-proxy-installer-1.0.1-8.f5ae2cd.el6sat.noarch
* katello-1.4.2-8.el6sat.noarch
* katello-all-1.4.2-8.el6sat.noarch
* katello-candlepin-cert-key-pair-1.0-1.noarch
* katello-certs-tools-1.4.2-2.el6sat.noarch
* katello-cli-1.4.2-6.el6sat.noarch
* katello-cli-common-1.4.2-6.el6sat.noarch
* katello-common-1.4.2-8.el6sat.noarch
* katello-configure-1.4.3-12.el6sat.noarch
* katello-configure-foreman-1.4.3-12.el6sat.noarch
* katello-foreman-all-1.4.2-8.el6sat.noarch
* katello-glue-candlepin-1.4.2-8.el6sat.noarch
* katello-glue-elasticsearch-1.4.2-8.el6sat.noarch
* katello-glue-pulp-1.4.2-8.el6sat.noarch
* katello-qpid-broker-key-pair-1.0-1.noarch
* katello-qpid-client-key-pair-1.0-1.noarch
* katello-selinux-1.4.3-3.el6sat.noarch
* openldap-2.4.23-32.el6_4.1.x86_64
* openldap-devel-2.4.23-32.el6_4.1.x86_64
* pulp-rpm-plugins-2.1.1-1.el6sat.noarch
* pulp-selinux-2.1.1-1.el6sat.noarch
* pulp-server-2.1.1-1.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-ldap_fluff-0.1.7-3.el6sat.noarch
* ruby193-rubygem-net-ldap-0.2.2-7.el6_4.noarch
* signo-0.0.15-1.el6sat.noarch
* signo-katello-0.0.15-1.el6sat.noarch

Comment 5 Dominic Cleal 2013-10-10 12:55:00 UTC
Merged upstream, please cherry pick:
https://github.com/theforeman/foreman/commit/d42d1f5

Comment 8 Corey Welton 2013-10-21 15:03:39 UTC
QA Verified 

Satellite-6.0.2-RHEL-6-20131016.0

Comment 11 Bryan Kearney 2014-04-24 17:10:15 UTC
This was verified and delivered with MDP2. Closing it out.

Comment 12 Bryan Kearney 2014-04-24 17:11:34 UTC
This was delivered and verified with MDP2. Closing the bug.


Note You need to log in before you can comment on or make changes to this bug.