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 |
| Andreas Hasenack (ahasenack) wrote | #1 |
| tags: | removed: needs-testing |
| Changed in landscape-client: | |
| status: | Fix Committed → Fix Released |
| affects: | Ubuntu Jaunty → landscape-client (Ubuntu Jaunty) |
| Martin Pitt (pitti) wrote Please test proposed package | #2 |
| Changed in landscape-client (Ubuntu Karmic): | |
| status: | New → Fix Committed |
| tags: | added: verification-needed |
| Changed in landscape-client (Ubuntu Jaunty): | |
| status: | New → Fix Committed |
| Martin Pitt (pitti) wrote | #3 |
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in landscape-client (Ubuntu Jaunty): | |
| status: | Fix Committed → Fix Released |
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in landscape-client (Ubuntu Karmic): | |
| status: | Fix Committed → Fix Released |
| 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.