Juju is forcing *ALL* charms in a repo to be perfect to be able to deploy *any*
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pyjuju |
Fix Released
|
Critical
|
Kapil Thangavelu | ||
juju (Ubuntu) |
Fix Released
|
High
|
Clint Byrum | ||
Precise |
Fix Released
|
High
|
Clint Byrum |
Bug Description
I have a number of half-done charms and not really working charms in my repository.
However, I can't deploy the working charms because juju is parsing *ALL* of the charms and completely dying because of one bad charm. This is completely bogus and needs to be reverted or handled far more gracefully, as I expect quite a few charm developers will be very confused by behavior like this:
$ juju -v deploy --config /tmp/myconfig.yaml local:statusnet
2012-03-11 10:26:35,231 DEBUG Initializing juju deploy runtime
2012-03-11 10:26:35,231 INFO Searching for charm
Traceback (most recent call last):
File "/usr/lib/
d = defer.maybeDefe
File "/usr/lib/
result = f(*args, **kw)
File "/usr/lib/
num_
File "/usr/lib/
return _inlineCallback
--- <exception caught here> ---
File "/usr/lib/
result = g.send(result)
File "/usr/lib/
charm = yield repo.find(
File "/usr/lib/
for charm in self._collectio
File "/usr/lib/
yield get_charm_
File "/usr/lib/
return CharmDirectory(
File "/usr/lib/
self.
File "/usr/lib/
self.
File "/usr/lib/
self.
File "/usr/lib/
d[name] = self._validate_
File "/usr/lib/
"Invalid value for %s: %r" % (name, value))
juju.charm.
2012-03-11 10:26:35,340 ERROR Traceback (most recent call last):
File "/usr/lib/
d = defer.maybeDefe
File "/usr/lib/
result = f(*args, **kw)
File "/usr/lib/
num_
File "/usr/lib/
return _inlineCallback
--- <exception caught here> ---
File "/usr/lib/
result = g.send(result)
File "/usr/lib/
charm = yield repo.find(
File "/usr/lib/
for charm in self._collectio
File "/usr/lib/
yield get_charm_
File "/usr/lib/
return CharmDirectory(
File "/usr/lib/
self.
File "/usr/lib/
self.
File "/usr/lib/
self.
File "/usr/lib/
d[name] = self._validate_
File "/usr/lib/
"Invalid value for %s: %r" % (name, value))
juju.charm.
Invalid value for use_https: False
2012-03-11 10:26:35,341 ERROR Invalid value for use_https: False
Related branches
- Juju Engineering: Pending requested
-
Diff: 242 lines (+74/-42)4 files modifiedjuju/charm/errors.py (+22/-13)
juju/charm/repository.py (+9/-4)
juju/charm/tests/test_errors.py (+1/-1)
juju/charm/tests/test_repository.py (+42/-24)
Changed in juju: | |
status: | Triaged → In Progress |
assignee: | nobody → Kapil Thangavelu (hazmat) |
Changed in juju: | |
milestone: | florence → honolulu |
Changed in juju: | |
milestone: | honolulu → florence |
Changed in juju: | |
status: | In Progress → Fix Released |
Changed in juju (Ubuntu): | |
status: | New → Triaged |
importance: | Undecided → High |
Changed in juju (Ubuntu Precise): | |
milestone: | none → ubuntu-12.04 |
assignee: | nobody → Clint Byrum (clint-fewbar) |
tags: | added: rls-mgr-p-tracking |
Changed in juju (Ubuntu Precise): | |
status: | Triaged → Fix Committed |
This was in fact fixed quite a while ago during the precise dev cycle.