Service deploy with a maas-tags constraint fails
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pyjuju |
Fix Released
|
Critical
|
Kapil Thangavelu | ||
0.6 |
Fix Released
|
Critical
|
Martin Packman |
Bug Description
Using juju 0.6.0.1+
adminccs@
2013-02-08 14:03:24,803 INFO Searching for charm cs:precise/
2013-02-08 14:03:25,651 INFO Connecting to environment...
2013-02-08 14:03:26,446 INFO Connected to environment.
2013-02-08 14:03:26,464 INFO Using cached charm version of nova-compute
2013-02-08 14:03:27,092 ERROR descriptor 'issuperset' requires a 'set' object but received a 'str'
The problem seems to be at the machine assignment stage, commenting the constraint test in the assign_to_machine function is my current workaround :
--- service.py.orig 2013-02-08 14:07:34.509537377 -0500
+++ /usr/lib/
@@ -974,10 +974,10 @@
"""
- if not machine_
- log.warning(
- "Unit %s assigned to machine %s with incompatible constraints",
- self.unit_name, machine_state.id)
+ #if not machine_
+ # log.warning(
+ # "Unit %s assigned to machine %s with incompatible constraints",
+ # self.unit_name, machine_state.id)
def assign_
Related branches
- Juju Engineering: Pending requested
-
Diff: 133 lines (+34/-9)5 files modifiedjuju/lib/serializer.py (+3/-1)
juju/machine/tests/test_constraints.py (+1/-0)
juju/providers/maas/provider.py (+2/-2)
juju/providers/maas/tests/test_provider.py (+28/-4)
juju/unit/lifecycle.py (+0/-2)
- Juju Engineering: Pending requested
-
Diff: 144 lines (+55/-11)3 files modifiedjuju/providers/maas/maas.py (+16/-6)
juju/providers/maas/tests/test_launch.py (+23/-4)
juju/providers/maas/tests/test_maas.py (+16/-1)
- Martin Packman (community): Approve
-
Diff: 220 lines (+85/-17)5 files modifiedjuju/providers/maas/maas.py (+16/-6)
juju/providers/maas/provider.py (+2/-2)
juju/providers/maas/tests/test_launch.py (+23/-4)
juju/providers/maas/tests/test_maas.py (+16/-1)
juju/providers/maas/tests/test_provider.py (+28/-4)
Changed in juju: | |
importance: | Undecided → Critical |
status: | New → In Progress |
Changed in juju: | |
milestone: | 0.7 → 0.6.1 |
status: | In Progress → Fix Committed |
Changed in juju: | |
milestone: | 0.6.1 → none |
milestone: | none → 0.6.1 |
Changed in juju: | |
milestone: | 0.6.1 → 0.7 |
Changed in juju: | |
status: | Fix Committed → Fix Released |
I also found that and managed to fix it. It was a typo, for details see: bazaar. launchpad. net/~novaki/ juju/maas- tags-fix/ revision/ 616?start_ revid=616
http://
Created a merge request: /code.launchpad .net/~juju/ juju/trunk/ +merge/ 148640
https:/