Re: [openstack-dev] [TripleO] Change 247669: Ceph Puppet: implement per-pool parameters:

[Giulio Fidente]

On 04/22/2016 09:51 AM, Shinobu Kinjo wrote:

Hi TripleO Team,

If you could take care of ${subject}, it would be nice.

[1] https://review.openstack.org/#/c/247669


ack, I liked revision 6 more than 7 but it might not work as 
interpolation always returns a string and we need a dict there


I'll post an update as soon as I get it to work
--
Giulio Fidente
GPG KEY: 08D733BA | IRC: gfidente

__
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

Re: [openstack-dev] [TripleO] Change 247669: Ceph Puppet: implement per-pool parameters:

[Shinobu Kinjo]

Thanks, Steve.

Cheers,
S

- Original Message -
From: "Steven Hardy" <sha...@redhat.com>
To: "OpenStack Development Mailing List (not for usage questions)" 
<openstack-dev@lists.openstack.org>
Cc: ski...@redhat.com
Sent: Sunday, April 24, 2016 11:14:58 PM
Subject: Re: [openstack-dev] [TripleO] Change 247669: Ceph Puppet: implement 
per-pool parameters:

On Fri, Apr 22, 2016 at 08:00:02AM -0400, Emilien Macchi wrote:
> On Fri, Apr 22, 2016 at 3:51 AM, Shinobu Kinjo <shinobu...@gmail.com> wrote:
> > Hi TripleO Team,
> >
> > If you could take care of ${subject}, it would be nice.
> >
> > [1] https://review.openstack.org/#/c/247669
> >
> 
> Well, the patch is in merge conflict, and has negative review from Cyril.
> I would suggest the committer, Felipe Alfaro Solana, to rebase it, and
> address the review (or at least shows some activity).

Agreed it looks like the patch needs an update.

I added the bug to our newton-1 milestone and increased the priority so we
can track landing the fix and prioritize reviews when the patch is updated.

Steve

__
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

Re: [openstack-dev] [TripleO] Change 247669: Ceph Puppet: implement per-pool parameters:

[Steven Hardy]

On Fri, Apr 22, 2016 at 08:00:02AM -0400, Emilien Macchi wrote:
> On Fri, Apr 22, 2016 at 3:51 AM, Shinobu Kinjo  wrote:
> > Hi TripleO Team,
> >
> > If you could take care of ${subject}, it would be nice.
> >
> > [1] https://review.openstack.org/#/c/247669
> >
> 
> Well, the patch is in merge conflict, and has negative review from Cyril.
> I would suggest the committer, Felipe Alfaro Solana, to rebase it, and
> address the review (or at least shows some activity).

Agreed it looks like the patch needs an update.

I added the bug to our newton-1 milestone and increased the priority so we
can track landing the fix and prioritize reviews when the patch is updated.

Steve

__
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

Re: [openstack-dev] [TripleO] Change 247669: Ceph Puppet: implement per-pool parameters:

[Emilien Macchi]

On Fri, Apr 22, 2016 at 3:51 AM, Shinobu Kinjo  wrote:
> Hi TripleO Team,
>
> If you could take care of ${subject}, it would be nice.
>
> [1] https://review.openstack.org/#/c/247669
>

Well, the patch is in merge conflict, and has negative review from Cyril.
I would suggest the committer, Felipe Alfaro Solana, to rebase it, and
address the review (or at least shows some activity).
Also, there is a weekly meeting:
https://wiki.openstack.org/wiki/Meetings/TripleO
Where anyone is welcome to join and ask for reviews.

Thanks,
-- 
Emilien Macchi

__
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-dev] [TripleO] Change 247669: Ceph Puppet: implement per-pool parameters:

[Shinobu Kinjo]

Hi TripleO Team,

If you could take care of ${subject}, it would be nice.

[1] https://review.openstack.org/#/c/247669

Cheers,
S

__
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