search

juju / juju-bundlelib

A Python library for working with Juju bundles.
GNU Lesser General Public License v3.0
1 stars 14 forks source link

Invalid placement syntax not caught in proof #50

Open AdamIsrael opened 8 years ago

AdamIsrael commented 8 years ago

I'm reviewing a bundle that passes charm proof but fails to push to the charm store when the lxd placement directive omits a machine id or unit name.

$ charm push . cs:~midonet-charmers/bundle/openstack-midonet-liberty
ERROR cannot post archive: bundle verification failed: ["invalid placement syntax \"lxc:\"","invalid placement syntax \"lxc:\"","invalid placement syntax \"lxc:\""]

From bundle.yaml:

  mysql:
    charm: "cs:trusty/mysql-18"
    num_units: 1
    options:
      "max-connections": 500
    annotations:
      "gui-x": "531"
      "gui-y": "237.2949192431123"
    to:
      - "lxc:"

I've verified that charm push succeeds if the lxc: placement directive is removed.

cherylj commented 8 years ago

There was a bug in 2.0-beta9 where juju didn't recognize the 'lxc' placement directive, but it was fixed in beta10. Are you still running into the problem? (this was fixed under https://bugs.launchpad.net/juju-core/+bug/1593838)