roles/ironic-conductor noop test broken

Bug #1615078 reported by Alex Schultz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Critical
Alex Schultz

Bug Description

https://review.openstack.org/#/c/355868/ introduced an issue with the noop tests where the roles/ironic-conductor tests are failing with:

FAILED roles/ironic-conductor ubuntu neut_tun.ironic-ironic
  passed should compile into a catalogue without dependency cycles
  passed should ensure that ironic-fa-deploy is installed
  passed should declare ironic class correctly
  failed should declare ironic::conductor class correctly (undefined local variable or method `api_url' for #<RSpec::ExampleGroups::RolesIronicConductorPp:0x007fe586053548>)
  passed should configure the database connection string
  passed ironic config should have propper config options
  passed should create /var/lib/ironic/tftpboot/pxelinux.0
  passed should not contain Service[nova-compute] with ensure => "runnin

The api_url in the test should be in quotes. For some reason the voting noop tests were not run in fuel CI when fuel: recheck was done so this was missed.

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/358038

Changed in fuel:
assignee: nobody → Alex Schultz (alex-schultz)
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/358038
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=dcb730a10602435c838503caef2cf50d24a10a35
Submitter: Jenkins
Branch: master

commit dcb730a10602435c838503caef2cf50d24a10a35
Author: Alex Schultz <email address hidden>
Date: Fri Aug 19 12:24:37 2016 -0600

    Fix ironic-conductor noop tests

    This change fixes the noop test for the api_url verification in the
    ironic-conductor test.

    Change-Id: I0b40a4a611000feeafca31aba5273d6015fd16b0
    Closes-Bug: #1615078

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0rc1

This issue was fixed in the openstack/fuel-library 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0

This issue was fixed in the openstack/fuel-library 10.0.0 release.

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.