On Fri, Aug 22, 2014 at 01:05:13PM +0530, Pranith Kumar Karampuri wrote: > > 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. The patches are still available in the review request under the "refs/changes/CD/ABCD/P" ABCD is the 4-digit change#, and P is the number of the patch. You can fetch the change, merge it and push to the repo again. I'm just not sure if the commit-id will change that way... I might be able to try/test that later today. HTH, Niels > > Pranith > > > > > > > >On Fri, Aug 22, 2014 at 12:01 AM, Justin Clift <justin@xxxxxxxxxxx > ><mailto: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 <mailto: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 <mailto: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 > > <mailto:pkarampu@xxxxxxxxxx> > > > > > To: "Vijay Bellur" < vbellur@xxxxxxxxxx > > <mailto:vbellur@xxxxxxxxxx> > > > > > Cc: "Gluster Devel" < gluster-devel@xxxxxxxxxxx > > <mailto: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 <mailto:Gluster-devel@xxxxxxxxxxx> > > > > http://supercolony.gluster.org/mailman/listinfo/gluster-devel > > > > _______________________________________________ > > > > Gluster-devel mailing list > > > > Gluster-devel@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> > > > > http://supercolony.gluster.org/mailman/listinfo/gluster-devel > > > > > > > > > > > > _______________________________________________ > > > > Gluster-devel mailing list > > > > Gluster-devel@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> > > > > http://supercolony.gluster.org/mailman/listinfo/gluster-devel > > > > > > > > > > _______________________________________________ > > > Gluster-devel mailing list > > > Gluster-devel@xxxxxxxxxxx <mailto: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 > > <http://twitter.com/realjustinclift> > > > > _______________________________________________ > > Gluster-devel mailing list > > Gluster-devel@xxxxxxxxxxx <mailto: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
