Re: patches missing in git even after gerrit says they are merged

[Pranith Kumar Karampuri <pkarampu@xxxxxxxxxx>]

On 08/22/2014 12:58 PM, Anand Avati wrote:

Whole of /review seems to be owned by gerrit already. Can someone re-post/remerge the patches? That would be the simplest. Are the patches available in the github mirror?

I will resubmit. I don't see it on github either.

Pranith

On Fri, Aug 22, 2014 at 12:01 AM, Justin Clift <justin@xxxxxxxxxxx> wrote:

We *might* need to chown all of /review/ directory down first too, as
that still has somewhat mixed permissions.  I'm not personally sure
and don't really want to touch it until Vijay/Avati have had a go.

As a data point though, there's a tarball of the /review/ directory
in /opt/justin_tmp/ that was taking just after the initial chmod
yesterday.  In case Vijay/Avati need to revert back to that.  No
idea what state these missing merged patches were in at that time
though.

/me leaves it to Vijay/Avati to figure out. :)

+ Justin

On 22/08/2014, at 7:30 AM, Humble Devassy Chirammal wrote:
> I believe Vijay/Avati can do that.
>
> On Fri, Aug 22, 2014 at 11:57 AM, Krishnan Parthasarathi <kparthas@xxxxxxxxxx> wrote:
> Humble,
>
> Who will be mergeing these patches into the repo?
>
> ~KP
>
> ----- Original Message -----
> > Below patches were in status of 'submitted , merge pending' when gerrit went
> > wrong yesterday.
> >
> > http://review.gluster.org/#/c/8459/
> > http://review.gluster.org/#/c/8466/
> > http://review.gluster.org/#/c/8507/
> >
> > Possibly we have to merge those in repo again.
> >
> > --Humble
> >
> >
> > On Fri, Aug 22, 2014 at 10:12 AM, Sachin Pandit < spandit@xxxxxxxxxx > wrote:
> >
> >
> > Hi,
> >
> > In gerrit, the status of http://review.gluster.org/#/c/8507/ is merged.
> > But I dont see the changes in git as Pranith mentioned.
> >
> > ~ Sachin.
> >
> > ----- Original Message -----
> > From: "Pranith Kumar Karampuri" < pkarampu@xxxxxxxxxx >
> > To: "Vijay Bellur" < vbellur@xxxxxxxxxx >
> > Cc: "Gluster Devel" < gluster-devel@xxxxxxxxxxx >
> > Sent: Thursday, August 21, 2014 10:05:42 PM
> > Subject: patches missing in git even after gerrit says they
> > are merged
> >
> > hi Vijay,
> > http://review.gluster.org/#/c/8466/ is one such candidate. CCed
> > gluster-devel so that others can also verify that their patches are in.
> >
> > Pranith
> > _______________________________________________
> > Gluster-devel mailing list
> > Gluster-devel@xxxxxxxxxxx
> > http://supercolony.gluster.org/mailman/listinfo/gluster-devel
> > _______________________________________________
> > Gluster-devel mailing list
> > Gluster-devel@xxxxxxxxxxx
> > http://supercolony.gluster.org/mailman/listinfo/gluster-devel
> >
> >
> > _______________________________________________
> > Gluster-devel mailing list
> > Gluster-devel@xxxxxxxxxxx
> > http://supercolony.gluster.org/mailman/listinfo/gluster-devel
> >
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxxx
> http://supercolony.gluster.org/mailman/listinfo/gluster-devel

--
GlusterFS - http://www.gluster.org

An open source, distributed file system scaling to several
petabytes, and handling thousands of clients.

My personal twitter: twitter.com/realjustinclift

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel