Ah, I don't have +2 on that repo, but the lgtm so your original plan is fine. On Mon, Jun 29, 2015 at 5:59 PM, Matt Fischer <matt at mattfischer.com> wrote: > I can take a look at these tonight. Maybe also Clayton can review them? > Neither of us were involved in the patches to my knowledge. > On Jun 29, 2015 5:09 PM, "Andrew Woodward" <xarses at gmail.com> wrote: > >> Hi >> >> Recent changes in the puppet modules infra left stackforge/puppet-ceph CI >> broken. We've resolved the issues in [1][2] However we are short on >> non-involved core-reviewers. >> >> I propose that we leave the patchs open through Wednesday and use lazy >> consensus and merge it if we don't receive any negative feedback. >> >> [1] https://review.openstack.org/#/c/179645/ >> [2] https://review.openstack.org/#/c/195959/ >> -- >> >> -- >> >> Andrew Woodward >> >> Mirantis >> >> Fuel Community Ambassador >> >> Ceph Community >> >> __________________________________________________________________________ >> OpenStack Development Mailing List (not for usage questions) >> Unsubscribe: >> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> >> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150629/c62e2c12/attachment.html>