service set (and deploy) uses wrong YAML config syntax
Bug #1167465 reported by
Roger Peppe
This bug report is a duplicate of:
Bug #1121907: File given to deploy --config is incompatible with pyjuju.
Edit
Remove
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Confirmed
|
High
|
William Reade |
Bug Description
The python version expects a map[string]
where the top level map holds service names.
The Go version expects map[string]
Personally I think the python behaviour is bogus, but we should be compatible
with it.
Related branches
lp:~fwereade/juju-core/config-5-state-service-config-yaml
- Juju Engineering: Pending requested
-
Diff: 340 lines (+105/-53)7 files modifiedcmd/juju/config_test.go (+6/-7)
juju/conn_test.go (+9/-3)
state/apiserver/client_test.go (+5/-7)
state/apiserver/perm_test.go (+2/-2)
state/service.go (+40/-27)
state/service_test.go (+19/-5)
state/statecmd/deploy_test.go (+24/-2)
Changed in juju-core: | |
importance: | Undecided → High |
status: | New → Confirmed |
Changed in juju-core: | |
assignee: | nobody → William Reade (fwereade) |
To post a comment you must log in.