[RFE] Stop storing configdrive in the database
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Wishlist
|
Julia Kreger |
Bug Description
In the Ironic database the 'instance_info' field as stored as a TEXT field. With MySQL this has a limit of 64KiB. The configdrive is stored in the instance_info field and so can not be larger than 64KiB. We have reports from multiple users who would like to be able to use configdrives larger than 64KIB.
Proposal is to switch to using the MEDIUMTEXT field type for instance_info for MySQL databases, which would allow up to 16MiB for the instance_info.
Also propose to have a configuration setting for the maximum size for the configdrive to by default be set to 64KiB. Database entry would still be MEDIUMTTEXT. Any configdrive that exceeds the config setting would be rejected. This would maintain the current size limitation and operators would not be surprised by their database growing unexpectedly. Operators could then change the config setting to increase the maximum size of the configdrive.
Update from dtantsur: since bug #1575935 is fixed, we no longer need to store the configdrive in the database at all.
Changed in ironic: | |
importance: | Undecided → Medium |
summary: |
- nodes are not powered-on due to short nodes.instance_info + node deployment fails when user passes a long cloud-init script |
summary: |
- node deployment fails when user passes a long cloud-init script + RFE: Increase size of data base entry for instance_info to allow + configdrives larger than 64KB |
tags: | added: rfe |
description: | updated |
summary: |
RFE: Increase size of data base entry for instance_info to allow - configdrives larger than 64KB + configdrives larger than 64KiB |
description: | updated |
Changed in ironic: | |
importance: | Medium → Wishlist |
summary: |
- RFE: Increase size of data base entry for instance_info to allow - configdrives larger than 64KiB + [RFE] Stop storing configdrive in the database |
Changed in ironic: | |
status: | Invalid → Confirmed |
assignee: | John L. Villalovos (happycamp) → nobody |
description: | updated |
Changed in ironic: | |
assignee: | nobody → Dmitry Tantsur (divius) |
status: | Confirmed → In Progress |
Changed in ironic: | |
assignee: | Dmitry Tantsur (divius) → Julia Kreger (juliaashleykreger) |
I'm going to follow Mistral's solution to the same text-is- too-short- for-me issue: /github. com/openstack/ mistral/ blob/master/ mistral/ db/sqlalchemy/ types.py# L86
https:/
TBD: medium vs long text type