[Review Queue] kubernetes-*, flannel, ibm-mobilefirst-server, Ibm-websphere-liberty and more

[Konstantinos Tsakalozos]

Hi,

Kevin, Pete and I got in our time on the queue this week. Here is what we
have:


   -

   kubernetes-master, kubernetes-worker, kubeapi-load-balancer, flannel
   -

      Checked up on the new builds for these reviews. They still do not
      pass the queue due to lack of tests
      -

   ibm-mobilefirst-server
   -

      https://review.jujucharms.com/reviews/39?revision=253
      -

      Disapproved due to a couple of issues noted in the review.
      -

   Ibm-websphere-liberty
   -

      https://review.jujucharms.com/reviews/26?revision=151
      -

      This included both design (rev0, +1/closed) and code (rev5,
      reopened). These are separate review tasks, so we’ve asked the author to
      open a new review for the code.
      -

      A quick glance at rev5 showed a few problems that should be fixed
      before requesting a new review:
      -

         https://code.launchpad.net/~kwmonroe/ibmlayers/layer-ibm-websphere
         -liberty/+merge/320077
         -

   Ibm-wxs-catalog
   -

      https://review.jujucharms.com/reviews/41
      -

      Left a comment requesting that the updated code be pushed to the ~
      ibmcharmers namespace in the store, so that the review queue can see
      it.
      -

   Ibm-dsm-enterprise
   -

      https://review.jujucharms.com/reviews/57
      -

      Marked as “needs fixing”, per the comments on the review.
      -

   Ibm-was-nd
   -

      https://review.jujucharms.com/reviews/40?revision=245
      -

      This review it blocked because the terms to be accepted and the charm
      revision under review are not published yet.
      -

      We will have to wait for the authors to address the above before we
      continue with this charm


You can find us at IRC #juju on freenode for realtime discussion.

Thanks,
Konstantinos
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju/attachments/20170317/520b0084/attachment.html>