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 <m...@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" <xar...@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-requ...@lists.openstack.org?subject:unsubscribe >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> >>
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev