Fuel for OpenStack

Deploy additional nodes failed with enable verbose

Bug #1582727 reported by Nikita Karpin
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Stanislaw Bogatkin
Fuel for OpenStack 10.0
Mitaka
Fix Released
High
Stanislaw Bogatkin
Fuel for OpenStack 9.0
Newton
Fix Committed
High
Stanislaw Bogatkin
Fuel for OpenStack 10.0

Bug Description

Detailed bug description:
 after change of environment configuration (for example addition of 2 ironic nodes), 'deploy changes' action failed just after deployment start

Steps to reproduce:

1. Deploy cluster with the following configuration:

1x Controller
1x Compute
1x Ironic.

Settings:
Compute - QEMU.
Network - Neutron with VLAN segmentation, DVR enabled.
Storage Backends - Cinder LVM over iSCSI for volumes.
Puppet debug logging - enabled
OpenStack debug logging - enabled

2. Add 1 Ironic and 1 Controller+Ironic node.
3. Deploy changes.
Expected results:
deployment pass
Actual result:
deploy failed. nailgun fails to serialize task graph
Reproducibility:
 n/a
Workaround:
 none
Impact:
 cannot add ironic nodes
Description of the environment:
 Operation system: ubuntu
 Versions of components: 9.0 iso 366
Additional information:
 task "ironic-conductor" failed to evaluate yaql condition. it fails with "KeyError: u'verbose'" exception, and it seems $.verbose doesn't work

    "condition": {
        "yaql_exp": "$.ironic.enabled and changedAny($.network_scheme, $.ironic, $.network_metadata.vips, $.get('database_vip'), $.get('neutron_endpoint'), $.get('glance_api_servers'), $.get('amqp_hosts'), $.debug, $.verbose, $.get('use_syslog'), $.get('syslog_log_facility_ironic'), $.rabbit, $.storage, $.get('kombu_compression'))\n"
    }

Vladimir Sharshov (vsharshov)
summary: - deploy changes fails
+ Deploy additional nodes failed with enable verbose
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

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

Reviewed: https://review.openstack.org/317558
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=bf8f1962a34e724505e3dbe2975db9a359259ebb
Submitter: Jenkins
Branch: master

commit bf8f1962a34e724505e3dbe2975db9a359259ebb
Author: Stanislaw Bogatkin <email address hidden>
Date: Tue May 17 18:08:14 2016 +0300

    Delete 'verbose' parameter from ironic-conductor

    Parameter 'verbose' is deprecated and will be removed from
    ironic ASAP, so we don't need to use it at all.

    Change-Id: Ib7fe815ca07b48ff64fd83cd93c08b444d341166
    Closes-Bug: #1582727

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

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/317986

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

Change abandoned by Stanislaw Bogatkin (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/317986
Reason: Abandoned in favor of I59ada13003f5239352df4394788b6e5b8789fd53

Revision history for this message
Michael Polenchuk (mpolenchuk) wrote :

Actually it was fixed by https://review.openstack.org/317359

Revision history for this message
Stanislaw Bogatkin (sbogatkin) wrote :

Btw, it shouldn't be in that patch at all, cause commit message and bug reference was about absolutely another problem. Please, do not make fixes which doesn't written in bug description - it leads us to such problems.

Revision history for this message
Michael Polenchuk (mpolenchuk) wrote :

The issue also pointed in:
https://bugs.launchpad.net/fuel/+bug/1578724/comments/9

Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/317424
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=e10ba1e8781201d7711e903f64bc129dc936d1cc
Submitter: Jenkins
Branch: stable/mitaka

commit e10ba1e8781201d7711e903f64bc129dc936d1cc
Author: Michael Polenchuk <email address hidden>
Date: Tue May 17 15:03:36 2016 +0300

    Get "verbose" key safely for ironic conductor

    * use get("verbose") func in task yaql expr
    * deprecate this option since it has been done
      in oslo.log already

    Change-Id: I59ada13003f5239352df4394788b6e5b8789fd53
    Related-Bug: #1582727

tags: added: in-stable-mitaka
ElenaRossokhina (esolomina)
tags: added: on-verification
Revision history for this message
ElenaRossokhina (esolomina) wrote :

Verified, mos iso 422
[root@nailgun ~]# shotgun2 short-report
cat /etc/fuel_build_id:
 422
cat /etc/fuel_build_number:
 422
cat /etc/fuel_release:
 9.0
cat /etc/fuel_openstack_version:
 mitaka-9.0
rpm -qa | egrep 'fuel|astute|network-checker|nailgun|packetary|shotgun':
 fuel-release-9.0.0-1.mos6347.noarch
 fuel-bootstrap-cli-9.0.0-1.mos284.noarch
 fuel-migrate-9.0.0-1.mos8405.noarch
 rubygem-astute-9.0.0-1.mos746.noarch
 fuel-provisioning-scripts-9.0.0-1.mos8709.noarch
 network-checker-9.0.0-1.mos72.x86_64
 fuel-mirror-9.0.0-1.mos137.noarch
 fuel-openstack-metadata-9.0.0-1.mos8709.noarch
 fuel-notify-9.0.0-1.mos8405.noarch
 nailgun-mcagents-9.0.0-1.mos746.noarch
 python-fuelclient-9.0.0-1.mos317.noarch
 fuelmenu-9.0.0-1.mos270.noarch
 fuel-9.0.0-1.mos6347.noarch
 fuel-utils-9.0.0-1.mos8405.noarch
 fuel-setup-9.0.0-1.mos6347.noarch
 fuel-library9.0-9.0.0-1.mos8405.noarch
 shotgun-9.0.0-1.mos90.noarch
 fuel-agent-9.0.0-1.mos284.noarch
 fuel-ui-9.0.0-1.mos2706.noarch
 fuel-ostf-9.0.0-1.mos934.noarch
 fuel-misc-9.0.0-1.mos8405.noarch
 python-packetary-9.0.0-1.mos137.noarch
 fuel-nailgun-9.0.0-1.mos8709.noarch

tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (stable/newton)

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/421367

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/421415
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=15ba6adbea125c9f3cbcfade2eb198989a060e1d
Submitter: Jenkins
Branch: master

commit 15ba6adbea125c9f3cbcfade2eb198989a060e1d
Author: Michael Polenchuk <email address hidden>
Date: Tue May 17 15:03:36 2016 +0300

    [sync/mitaka] Get "verbose" key safely for ironic conductor

    * use get("verbose") func in task yaql expr
    * deprecate this option since it has been done
      in oslo.log already

    Change-Id: I59ada13003f5239352df4394788b6e5b8789fd53
    Related-Bug: #1582727

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

Change abandoned by Dmitry Ilyin (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/421367

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

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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