Ironic Inspector

edeploy plugin puts too much data in Ironic extra column

Bug #1461252 reported by John Trowbridge
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic Inspector
Fix Released
Critical
John Trowbridge
Ironic Inspector 2.0.0

Bug Description

The edeploy plugin stores all of the facts it collects in the extra column of the Ironic db. When using real hardware, edeploy collects a large amount of facts and can lead to overflowing that column:

"InternalServerError: Remote error: DBError (DataError) (1406, "Data too long for column 'extra' at row 1") 'UPDATE nodes SET updated_at=%s, properties=%s, extra=%s WHERE nodes.id = %s'"

This causes the inspection ramdisk to fail and drop to an emergency shell.

We should store this data in Swift instead.

Dmitry Tantsur (divius)
Changed in ironic-discoverd:
status: New → Triaged
importance: Undecided → Critical
milestone: none → 2.0.0
Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

+1 for swift!

Revision history for this message
Dmitry Tantsur (divius) wrote :

I would also suggest renaming the plugin. It has nothing to do with edeploy now actually. What about "extra_hardware"?

John Trowbridge (trown)
Changed in ironic-inspector:
assignee: nobody → John Trowbridge (trown)
John Trowbridge (trown)
Changed in ironic-inspector:
status: Triaged → In Progress
Revision history for this message
John Trowbridge (trown) wrote :

Fix has been committed: https://review.openstack.org/#/c/189427/

Changed in ironic-inspector:
status: In Progress → Fix Committed
Dmitry Tantsur (divius)
Changed in ironic-inspector:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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