Eucalytpus Monitoring Plugin isn't installable
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
High
|
Kapil Thangavelu | ||
landscape-client (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Jaunty |
Fix Released
|
Undecided
|
Unassigned | ||
Karmic |
Fix Released
|
Undecided
|
Unassigned | ||
Lucid |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
the eucalpytus plugin was intended to be installed as a manager plugin, via landscape-config --include-
def get_plugins(self):
return [namedClass(
for plugin_name in self.config.
which requires that the module name be the same as the class name. the eucalpytus plugin has a module name 'eucalyptus' and a class name 'EucalyptusClou
incidentally the plugin also contains dead code
plugin_name = 'eucalytpus-
which looks like it was trying to address registration.
additionally landscape-client will fail to start if an invalid plugin is configured without any log messages or user visible feedback even with log-level turned to debug.
Related branches
- Free Ekanayaka (community): Approve
- Jamu Kakar (community): Approve
-
Diff: 204 lines (+42/-27)4 files modifiedlandscape/manager/deployment.py (+11/-8)
landscape/manager/eucalyptus.py (+2/-2)
landscape/manager/tests/test_deployment.py (+15/-3)
landscape/manager/tests/test_eucalyptus.py (+14/-14)
Changed in landscape-client: | |
status: | New → In Progress |
assignee: | nobody → Kapil Thangavelu (hazmat) |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
milestone: | 1.4.5 → 1.5.0 |
tags: | added: testing |
tags: | removed: testing |
tags: | added: needs-testing |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
affects: | Ubuntu Jaunty → landscape-client (Ubuntu Jaunty) |
Changed in landscape-client (Ubuntu Lucid): | |
status: | New → Fix Released |
Changed in landscape-client (Ubuntu): | |
status: | New → Fix Released |
tags: | removed: verification-needed |
Confirmed fixed in staging packages.