Landscape-client should allow for several different juju-units on the same machine
Bug #1325599 reported by
Chris Glass
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
High
|
Chris Glass |
Bug Description
Right now, we only consider a single json file for juju information.
It is possible that several juju units are deployed on the same machine (rare, but possible).
We need the landscape client to consider a set of juju-info files instead of just one, so that the landscape-client charm can output one json file per unit installe don the machine.
Related branches
lp:~tribaal/landscape-client/multiple-juju-unit-files
- Alberto Donato (community): Approve
- Free Ekanayaka (community): Approve
-
Diff: 26 lines (+9/-2)1 file modifiedlandscape/message_schemas.py (+9/-2)
lp:~tribaal/landscape-client/revert-weird-merge-error
- Björn Tillenius (community): Approve
- Alberto Donato (community): Approve
-
Diff: 566 lines (+248/-84)10 files modifiedlandscape/broker/registration.py (+5/-5)
landscape/broker/tests/helpers.py (+4/-1)
landscape/broker/tests/test_registration.py (+56/-6)
landscape/deployment.py (+6/-1)
landscape/lib/juju.py (+37/-17)
landscape/lib/tests/test_juju.py (+80/-22)
landscape/message_schemas.py (+8/-3)
landscape/monitor/jujuinfo.py (+17/-6)
landscape/monitor/tests/test_jujuinfo.py (+30/-18)
landscape/tests/test_deployment.py (+5/-5)
lp:~adam-collard/charms/trusty/landscape-client/landscape-client-multiple-juju-files
- Данило Шеган (community): Approve
- Geoff Teale (community): Approve
- Adam Collard: Abstain
-
Diff: 241 lines (+81/-20)3 files modifiedhooks/common.py (+25/-3)
hooks/hooks.py (+28/-11)
hooks/test_hooks.py (+28/-6)
tags: | removed: kanban |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Branch was reverted because we made an backward- incompatible changes in message_schemas.py and that prevented all tarmac builds/merges for landscape-server code from landing as well as general landscape-test runs. Because we dropped an existing schema message type JUJU_INFO server could no longer import it. https:/ /ci.lscape. net/job/ tarmac- lander/ 1139/consoleFul l