provide mac addresses of interfaces on admin node in settings.yaml

Bug #1304521 reported by Dima Shulyak
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Matthew Mosesohn

Bug Description

Currently dhcp checker relies on master node interfaces info availability to determine that dhcp response comes from dhcp server installed on master node.

After nailgun will be containerized it will be broken.

Tags: nailgun
Dima Shulyak (dshulyak)
Changed in fuel:
milestone: none → 5.0
Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

Does dhcp checker check the source and destination mac address?

Revision history for this message
Dima Shulyak (dshulyak) wrote :

it verifies that source mac address in dhcp response matches one from masters

Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

we are using dhcp relay, so nothing will change from the slave node point of view. do you mean you need to have this information in the container running mcollective agent on the node ?

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Vladimir, how can nailgun application know the correct MAC address to validate Fuel's dhcp server (dnsmasq) other than by knowing what the base host's mac address is on its admin interface? If the app runs inside Docker, it won't be able to see the mac address. It needs to be done by adding it to astute.yaml.

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/90124

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Related fix proposed to fuel-web (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/90328

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/90124
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=2688913905c9798873eb16140b278957302094bc
Submitter: Jenkins
Branch: master

commit 2688913905c9798873eb16140b278957302094bc
Author: Matthew Mosesohn <email address hidden>
Date: Thu Apr 24 19:49:36 2014 +0400

    add mac address to nailgun settings

    Change-Id: Idb892d27a942a565726c785f34702b804e99e4e2
    Closes-Bug: #1304521

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/90119
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=5c261760b625df1dfe253aae3b8fad675a7be0d3
Submitter: Jenkins
Branch: master

commit 5c261760b625df1dfe253aae3b8fad675a7be0d3
Author: Matthew Mosesohn <email address hidden>
Date: Thu Apr 24 19:42:01 2014 +0400

    Add mac address to astute.yaml

    Change-Id: Ic0710b99d0a4598ef8ac8d576b153a2a99c83497
    Closes-Bug: #1304521

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Related fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/90328
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=43fdef4393e64a7c23df844a11ad3f8fc820283b
Submitter: Jenkins
Branch: master

commit 43fdef4393e64a7c23df844a11ad3f8fc820283b
Author: Dima Shulyak <email address hidden>
Date: Fri Apr 25 13:00:03 2014 +0300

    Obtain master mac addres from config

    Nailgun will be containerized therefore master mac can not be
    obtained in run time from host, and needs to be configured
    by fuelmenu and puppet on deploy

    Change-Id: Ice00270d9874ed7ff8383186f3d4811f2ec091b5
    Related-Bug: #1304521

Alexey (aterekhin)
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.