edeploy plugin puts too much data in Ironic extra column

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

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) on 2015-06-02
Changed in ironic-discoverd:
status: New → Triaged
importance: Undecided → Critical
milestone: none → 2.0.0

+1 for swift!

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) on 2015-06-08
Changed in ironic-inspector:
assignee: nobody → John Trowbridge (trown)
John Trowbridge (trown) on 2015-06-08
Changed in ironic-inspector:
status: Triaged → In Progress
John Trowbridge (trown) wrote :
Changed in ironic-inspector:
status: In Progress → Fix Committed
Dmitry Tantsur (divius) on 2015-07-09
Changed in ironic-inspector:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers