Defining distros = 'all' in a module for documentation results in a module skip as 'all' doesn't match distro 'x'
Bug #1715690 reported by
Chad Smith
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| cloud-init |
Fix Released
|
Medium
|
Chad Smith | ||
Bug Description
2017-09-07 06:03:25,838 - stages.py[INFO]: Skipping modules ['runcmd'] because they are not verified on distro 'ubuntu'. To run anyway, add them to 'unverified_
Seen after introduction of distros attribute on cc_runcmd.py in revision cc9762a2d737ead
Also validated that Skipping a module is only a log, the module is still run:
# later in the same log
Running module runcmd (<module 'cloudinit.
Related branches
~chad.smith/cloud-init:config-modules-allow-distros-all
- Ryan Harper: Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 256 lines (+131/-34)4 files modifiedcloudinit/config/cc_runcmd.py (+2/-1)
cloudinit/distros/__init__.py (+4/-0)
cloudinit/stages.py (+20/-13)
tests/unittests/test_runs/test_simple_run.py (+105/-20)
| Changed in cloud-init: | |
| importance: | Undecided → High |
| assignee: | nobody → Chad Smith (chad.smith) |
| Changed in cloud-init: | |
| status: | New → Confirmed |
| description: | updated |
| Changed in cloud-init: | |
| importance: | High → Medium |
| Changed in cloud-init: | |
| status: | Confirmed → In Progress |
| Changed in cloud-init: | |
| status: | In Progress → Fix Committed |
| milestone: | none → 0.7.10 |
Revision history for this message
| Scott Moser (smoser) wrote Fixed in Cloud-init 17.1 | #1 |
| Changed in cloud-init: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| James Falcon (falcojr) wrote | #2 |
To post a comment you must log in.
This bug is believed to be fixed in cloud-init in 17.1. If this is still a problem for you, please make a comment and set the state back to New
Thank you.