Merge lp:~rpodolyaka/cirros/config-drive into lp:cirros
Proposed by
Roman Podoliaka
on 2016-07-04
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 | 2016-07-04 | Pending | |
Review via email:
|
Description of the change
To post a comment you must log in.
Unmerged revisions
- 370. By Roman Podoliaka on 2016-07-04
-
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