fuelmenu incorrectly selects astute.yaml for update

Bug #1323369 reported by Dmitry Pyzhov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Matthew Mosesohn
5.0.x
Fix Committed
High
Matthew Mosesohn

Bug Description

I've run fuelmenu on deployed master node and changed dns settings. Changes were saved /etc/astute.yaml.
After that I've run 'dockerctl restart cobbler' and it was update with broken settings from /etc/fuel/astute.yaml:

# cat /etc/astute.yaml
HOSTNAME: fuel
DNS_DOMAIN: domain.tld
DNS_SEARCH: domain.tld
DNS_UPSTREAM: 172.18.16.11
NTP1: 0.pool.ntp.org
NTP2: 1.pool.ntp.org
NTP3: 2.pool.ntp.org
ADMIN_NETWORK:
  dhcp_pool_start: 10.20.0.128
  netmask: 255.255.255.0
  static_pool_start: 10.20.0.3
  mac: 08:00:27:38:fd:41
  dhcp_pool_end: 10.20.0.254
  interface: eth0
  static_pool_end: 10.20.0.127
  ipaddress: 10.20.0.2
TEST_DNS: www.google.com

# cat /etc/fuel/astute.yaml
HOSTNAME: fuel
DNS_DOMAIN: domain.tld
DNS_SEARCH: domain.tld
DNS_UPSTREAM: 8.8.8.8
NTP1: 0.pool.ntp.org
NTP2: 1.pool.ntp.org
NTP3: 2.pool.ntp.org
ADMIN_NETWORK:
  dhcp_pool_start: 10.20.0.128
  netmask: 255.255.255.0
  static_pool_start: 10.20.0.3
  mac: 08:00:27:38:fd:41
  dhcp_pool_end: 10.20.0.254
  interface: eth0
  static_pool_end: 10.20.0.127
  ipaddress: 10.20.0.2

# dockerctl shell cobbler
[root@9b6a7b0dfcd2 ~]# cat /etc/dnsmasq.upstream
domain
search domain.tld
nameserver 8.8.8.8

Another strange point is empty 'domain' field in dnsmasq.upstream. Maybe there should be another bug report for it.

I've copy /etc/astute.yaml to /etc/fuel/astute.yaml and run 'puppet apply /etc/puppet/modules/nailgun/examples/cobbler-only.pp' in cobbler container. After this I have semi-ready file:

# puppet apply /etc/puppet/modules/nailgun/examples/cobbler-only.pp
ls: cannot access /dev/sda?: No such file or directory
ls: cannot access /dev/sda?: No such file or directory
Warning: Could not retrieve fact fqdn
Warning: Host is missing hostname and/or domain: 9b6a7b0dfcd2
Warning: Config file /etc/puppet/hiera.yaml not found, using Hiera defaults
Notice: Compiled catalog for 9b6a7b0dfcd2 in environment production in 1.43 seconds
ls: cannot access /dev/sda?: No such file or directory
ls: cannot access /dev/sda?: No such file or directory
Notice: /Stage[main]/Cobbler::Server/Service[httpd]/enable: enable changed 'false' to 'true'
Notice: /Stage[main]/Cobbler::Server/Service[cobblerd]/enable: enable changed 'false' to 'true'
Notice: /Stage[main]/Cobbler::Server/Exec[cobbler_sync]: Triggered 'refresh' from 1 events
Notice: /Stage[main]/Cobbler::Server/Service[dnsmasq]/ensure: ensure changed 'running' to 'stopped'
Notice: /Stage[main]/Cobbler::Server/Service[dnsmasq]: Triggered 'refresh' from 2 events
Notice: /Stage[main]/Cobbler::Server/Service[xinetd]/enable: enable changed 'false' to 'true'
Notice: /Stage[main]/Cobbler::Server/Service[xinetd]: Triggered 'refresh' from 1 events
Notice: /Stage[main]/Nailgun::Cobbler/Exec[cobbler_system_edit_default]/returns: executed successfully
Notice: /Stage[main]/Nailgun::Cobbler/Exec[nailgun_cobbler_sync]: Triggered 'refresh' from 1 events
Notice: Finished catalog run in 9.32 seconds
[root@9b6a7b0dfcd2 ~]# cat /etc/dnsmasq.upstream
domain
search domain.tld
nameserver 172.18.16.11

Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

Also I had to restart nailgun receiverd in order to download diagnostic snapshot

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

The correct solution is to drop /etc/astute.yaml completely from master node and tweak fuelmenu to generate it in /etc/fuel/astute.yaml.

Changed in fuel:
milestone: none → 5.1
status: New → Confirmed
Changed in fuel:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/98743
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=178ec641d88fe14b78e5903af0175d194954dbe8
Submitter: Jenkins
Branch: master

commit 178ec641d88fe14b78e5903af0175d194954dbe8
Author: Matthew Mosesohn <email address hidden>
Date: Mon Jun 9 15:11:01 2014 +0400

    Fuelmenu cleanup for settings and interfaces

    Excludes docker* and veth* interfaces from
    being configurable. Naily facts are removed
    from fuelmenu completely.
    Update astute.yaml path to /etc/fuel

    Change-Id: Ib854c4fd17d1c62ca92f565553a6ee8ead2dd765
    Closes-Bug: #1323369
    Closes-Bug: #1327009

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/102906

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/5.0)

Reviewed: https://review.openstack.org/102906
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=63a852bc402c079083a8cd0896c44254a1adcdbc
Submitter: Jenkins
Branch: stable/5.0

commit 63a852bc402c079083a8cd0896c44254a1adcdbc
Author: Matthew Mosesohn <email address hidden>
Date: Mon Jun 9 15:11:01 2014 +0400

    Fuelmenu cleanup for settings and interfaces

    Excludes docker* and veth* interfaces from
    being configurable. Naily facts are removed
    from fuelmenu completely.
    Update astute.yaml path to /etc/fuel

    Change-Id: Ib854c4fd17d1c62ca92f565553a6ee8ead2dd765
    Closes-Bug: #1323369
    Closes-Bug: #1327009
    (cherry picked from commit 178ec641d88fe14b78e5903af0175d194954dbe8)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (stable/5.0)

Related fix proposed to branch: stable/5.0
Review: https://review.openstack.org/103634

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-main (stable/5.0)

Change abandoned by Dmitry Burmistrov (<email address hidden>) on branch: stable/5.0
Review: https://review.openstack.org/103634

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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