new bundle: midonet

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/3512/

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-lxc/3462/

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/3530/

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-lxc/3480/

Hi Antoni,

Thanks for your patience on this. I ran into a small issue with the bundle and had to track down the exact cause.

Running `charm proof` on the bundle returns this error:

E: machines spec identifiers must be digits

The error is coming from the 'undefined' section under 'machines'. Can you remove that stanza? That's the only barrier to getting the bundle added to the store.

Thanks!

On Tue, Jun 14, 2016 at 10:13 PM, Adam Israel <email address hidden>
wrote:

> Hi Antoni,
>
> Thanks for your patience on this. I ran into a small issue with the
> bundle and had to track down the exact cause.
>
> Running `charm proof` on the bundle returns this error:
>
> E: machines spec identifiers must be digits
>
> The error is coming from the 'undefined' section under 'machines'. Can
> you remove that stanza?

Done
http://bazaar.launchpad.net/~celebdor/charms/bundles/openstack-midonet-liberty/bundle/revision/2

> That's the only barrier to getting the bundle
> added to the store.
>
> Thanks!
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1552445
>
> Title:
> new bundle: midonet
>
> Status in Juju Charms Collection:
> New
>
> Bug description:
> This is a submission for making
>
> http://bazaar.launchpad.net/~celebdor/charms/bundles/openstack-
> midonet-liberty/bundle
>
> an official bundle in the store.
>
> It provides a 4 node maas openstack deployment with Networking
> provided by the MidoNet SDN.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/charms/+bug/1552445/+subscriptions
>

Hi Antoni,

Thanks for fixing that! I found one other problem while uploading to the store. The "to" directive needs to be either a machine or unit, and there were a couple cases where the placement was to "lxd:". `charm proof` should have caught that, and I've filed a bug[1] to address that. In the meantime, I patched the charm, uploaded it to the store[2], and opened a merge request[3] under your branch to fix the issue.

[1] https://github.com/juju/juju-bundlelib/issues/50
[2] https://jujucharms.com/openstack-midonet-liberty/bundle/0
[3] https://code.launchpad.net/~aisrael/charms/bundles/openstack-midonet-liberty/fix-lxd/+merge/297483

Please let us know if you have any troubles or outstanding issues that still need to be addressed.

This item has failed automated testing! Results available here http://juju-ci.vapour.ws/job/charm-bundle-test-aws/4714/