On 06/29/2015 11:16 PM, Matt Fischer wrote:
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
mailto:m...@mattfischer.com wrote:
I can take a look at these tonight. Maybe also Clayton can
I merged the https://review.openstack.org/#/c/197302/ as we had agreed that
once we had enough +1's on the EL7 review we would merge it.
The Puppet4 spec tests had already been approved.
--
David Moreau Simard
Cloud Engineering | Operations
On 2015-06-30 07:06 PM, Andrew Woodward wrote:
On
On Tue, Jun 30, 2015 at 6:28 AM Emilien Macchi emil...@redhat.com wrote:
On 06/29/2015 11:16 PM, Matt Fischer wrote:
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
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
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
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