Ironic introspection fails because unexpected keyword "insecure"

Bug #1656692 reported by Sagi (Sergey) Shnaidman
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
High
Dougal Matthews
tripleo
Fix Released
Critical
Dougal Matthews

Bug Description

Baremetal introspection fails with error:

Introspection completed with errors:
Failed to run action [action_ex_id=68aa6b01-e16b-45a5-aa3b-e31419635c1b, action_cls='<class 'mistral.actions.action_factory.BaremetalIntrospectionAction'>', attributes='{u'client_method_name': u'introspect'}', params='{u'uuid': u'all', u'new_ipmi_username': None, u'new_ipmi_password': None}']
 BaremetalIntrospectionAction.introspect failed: <type 'exceptions.TypeError'>: __init__() got an unexpected keyword argument 'insecure'

http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/25f1837/console.html#_2017-01-15_07_02_32_683044

The breaking commit: https://github.com/openstack/mistral/commit/1eff558e438564af4c3325c558a38e0462302e8f

The ironic introspection ClientV1 doesn't have a keyword "insecure":
https://github.com/openstack/python-ironic-inspector-client/blob/master/ironic_inspector_client/common/http.py#L97-L108

which is required by new code in mistral:
https://github.com/openstack/mistral/blob/1eff558e438564af4c3325c558a38e0462302e8f/mistral/actions/openstack/actions.py#L445

Changed in tripleo:
status: New → Confirmed
status: Confirmed → Triaged
importance: Undecided → Critical
tags: added: alert promotion-blocker
Changed in tripleo:
milestone: none → ocata-3
Dougal Matthews (d0ugal)
Changed in mistral:
importance: Undecided → High
status: New → Confirmed
assignee: nobody → Dougal Matthews (d0ugal)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (master)

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

Changed in mistral:
status: Confirmed → In Progress
Dougal Matthews (d0ugal)
Changed in tripleo:
assignee: nobody → Dougal Matthews (d0ugal)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (master)

Reviewed: https://review.openstack.org/420547
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=76f20ee437c595ec8c3abaade99d1a6f3adc0f91
Submitter: Jenkins
Branch: master

commit 76f20ee437c595ec8c3abaade99d1a6f3adc0f91
Author: Dougal Matthews <email address hidden>
Date: Mon Jan 16 07:50:23 2017 +0000

    Remove insecure flag from the Baremetal Introspection client

    The baremetal inspector client does not take the "insecure" argument. It
    was incorrectly added in 600dd4765409068266f2bb2d858e48cf4e0ec245.

    This is currently breaking TripleO.

    Closes-Bug: #1656692
    Change-Id: Ic1f55e9a243302d4d2715654b4d9d4abe5000d2a

Changed in mistral:
status: In Progress → Fix Released
Revision history for this message
Emilien Macchi (emilienm) wrote :

I'm closing the bug on tripleo side, we've got a promotion. Nice work Dougal!

Changed in tripleo:
status: Triaged → Fix Released
tags: removed: alert promotion-blocker
tags: added: promotion-blocker
Revision history for this message
Ben Nemec (bnemec) wrote :

I'm reopening until this is proven fixed. The nonha jobs are currently still all failing on this.

Changed in tripleo:
status: Fix Released → Triaged
tags: added: alert
Revision history for this message
Ben Nemec (bnemec) wrote :

I should have noted that we just had a promotion, so we do believe it's fixed, but I have yet to see a job actually pass. When we do, this can be closed again.

Revision history for this message
Ben Nemec (bnemec) wrote :

nonha jobs are passing again

Changed in tripleo:
status: Triaged → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 4.0.0.0b3

This issue was fixed in the openstack/mistral 4.0.0.0b3 development milestone.

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.