Hello! Yesterday, Corry, Kevin, Konstantinos and I had our combined review queue session, and here are the notes:
- tomcat - https://bugs.launchpad.net/charms/+source/tomcat/+bug/1538715 - Community request to have a webapp-container subordinate relation. This keeps subs from having to relate using juju-info, and it offers additional data (like webapp install path) that wouldn’t be available over juju-info. - +1 and pushed. Updated tomcat charms for precise/trusty that offer this relation are now available in the store. - spagobi - https://bugs.launchpad.net/charms/+bug/1534497 - We cannot proceed with this charm at the moment because of the following reasons - there are no tests validating the operational aspects of the charm - charm proof test is failing since some required items are missing (eg copyright file) - We deployed the charm using the instructions in the README but we couldn't reach to a web interface. We spotted a number of errors in the logs and we asked the charm author to review them. - xl c/c++ - https://bugs.launchpad.net/charms/+bug/1489829 - Recent code changes look great, but we are unable to test this charm at the moment while we wait for our ppc64le hardware to become available. - Minecraft charm - https://bugs.launchpad.net/charms/+bug/1523896 - This is the second round of reviewing this charm. The author addressed the comments we had. - all tests pass - the layer that builds the charm is provided here: https://github.com/mattyw/minecraft-charm - +1 with moving forward with this charm - postgresql (fix max connections) - https://code.launchpad.net/~stub/charms/trusty/postgresql/fix-max-connections/+merge/282974 - Had some issues with tests due to tester running juju 1.26 alpha3 - downgraded and all tests passed, +1 approved. - ibm-db2 - https://bugs.launchpad.net/charms/+bug/1477057 - All review items have been addressed and the charm has been promulgated. have a great weekend! Andrew
-- Juju mailing list Juju@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju