Merge lp:~rpodolyaka/cirros/config-drive into lp:cirros
Proposed by
Roman Podoliaka
Status: | Needs review |
---|---|
Proposed branch: | lp:~rpodolyaka/cirros/config-drive |
Merge into: | lp:cirros |
Diff against target: |
36 lines (+14/-5) 1 file modified
src/lib/cirros/ds/configdrive (+14/-5) |
To merge this branch: | bzr merge lp:~rpodolyaka/cirros/config-drive |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
cirros developers | Pending | ||
Review via email: mp+299044@code.launchpad.net |
Description of the change
To post a comment you must log in.
Unmerged revisions
- 370. By Roman Podoliaka
-
config drive: also check uppercase VFAT labels
Some dosfstools versions unconditionally produce uppercase FS label,
which are currently ignored by Cirros init system (but work with
cloud-init), thus, such config drives will not be found on VM boot.while the config drive spec says that a lowercase label "config-2"
should be used, it does not go into details whether case-sensitive
or case-insesitive string comparison is expected.We could easily make this work for CentOS / RHEL users, if we simply
checked for both "config-2" and "CONFIG-2" before giving up.Closes-Bug: #1598783