[rdo-dev] Reviews needed (Swift)

[Alfredo Moralejo Alonso]

On Mon, Mar 22, 2021 at 10:04 PM Pete Zaitcev <zaitcev at redhat.com> wrote:

> Dear Alfredo:
>
> Thanks for the reviews, especially the tricky one, the 32374 about
> the distributed expirer. Thanks to you, Yatin was able to land them.
>

Note that this change has broken both tripleo and
puppet-openstack-integration promotion jobs.

See the comments in
https://review.rdoproject.org/r/c/openstack/swift-distgit/+/32374

Should we revert and try to implement changes in a backwards compatible way?



>
> Although I'm on core for Swift upstream, I'm not a maintainer for RDO.
> I keep these RPMs out of habit, because I was the maintainer in Fedora,
> before they were evicted.
>
> I'd love to find some volunteers, and perhaps trade some services.
> Maybe become a maintainer for something like Manila?
>
> -- Pete
>
> On Mon, 22 Mar 2021 11:36:05 +0100
> Alfredo Moralejo Alonso <amoralej at redhat.com> wrote:
>
> > Hi Pete,
> >
> > Thanks for the heads up, I've reviewed them.
> >
> > Remember that you can add more swift maintainers to the package so that
> > there are more people able to review and merge reviews. You can see an
> > example in https://review.rdoproject.org/r/c/rdoinfo/+/28148 .
> >
> > Best regards,
> >
> > Alfredo
> >
> >
> > On Sat, Mar 20, 2021 at 12:47 AM Pete Zaitcev <zaitcev at redhat.com>
> wrote:
> >
> > > Hello:
> > >
> > > I have a few reviews outstanding, some of which are for real bugs
> > > and stuck since October. Is there something that can be done?
> > >
> > > https://review.rdoproject.org/r/c/openstack/swift-distgit/+/29930
> > > bz#1884167
> > > <
> https://review.rdoproject.org/r/c/openstack/swift-distgit/+/29930bz#1884167
> >
> > > stuck since October 12, 2020
> > >
> > > https://review.rdoproject.org/r/c/openstack/swift-distgit/+/29952
> > > bz#1882278
> > > <
> https://review.rdoproject.org/r/c/openstack/swift-distgit/+/29952bz#1882278
> >
> > > stuck since October 7, 2020
> > >
> > > https://review.rdoproject.org/r/c/openstack/swift-distgit/+/32374
> > > fresh, has +2 (thanks, Javier!), needs someone to land it
> > >
> > > https://review.rdoproject.org/r/c/openstack/swift-distgit/+/32499
> > > fresh but needs attention
> > >
> > >
> > >
> https://review.rdoproject.org/r/c/openstack/smergedmergedwift-distgit/+/32542
> > > <https://review.rdoproject.org/r/c/openstack/swift-distgit/+/32542>
> > > fresh but needs attention
> > >
> > > Thanks in advance,
> > > -- Pete
> > >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rdoproject.org/pipermail/dev/attachments/20210324/b5359457/attachment.html>