Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-07-07 Thread Xavier Hernandez
On 07/07/17 11:25, Pranith Kumar Karampuri wrote: On Fri, Jul 7, 2017 at 2:46 PM, Xavier Hernandez > wrote: On 07/07/17 10:12, Pranith Kumar Karampuri wrote: On Fri, Jul 7, 2017 at 1:13 PM, Xavier Hernandez

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-07-07 Thread Xavier Hernandez
Hi Pranith, On 05/07/17 12:28, Pranith Kumar Karampuri wrote: On Tue, Jul 4, 2017 at 2:26 PM, Xavier Hernandez > wrote: Hi Pranith, On 03/07/17 08:33, Pranith Kumar Karampuri wrote: Xavi, Now that the change

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-07-05 Thread Pranith Kumar Karampuri
On Tue, Jul 4, 2017 at 2:26 PM, Xavier Hernandez wrote: > Hi Pranith, > > On 03/07/17 08:33, Pranith Kumar Karampuri wrote: > >> Xavi, >> Now that the change has been reverted, we can resume this >> discussion and decide on the exact format that considers, tier, dht,

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-07-04 Thread Xavier Hernandez
Hi Pranith, On 03/07/17 08:33, Pranith Kumar Karampuri wrote: Xavi, Now that the change has been reverted, we can resume this discussion and decide on the exact format that considers, tier, dht, afr, ec. People working geo-rep/dht/afr/ec had an internal discussion and we all agreed that

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-07-03 Thread Pranith Kumar Karampuri
Xavi, Now that the change has been reverted, we can resume this discussion and decide on the exact format that considers, tier, dht, afr, ec. People working geo-rep/dht/afr/ec had an internal discussion and we all agreed that this proposal would be a good way forward. I think once we agree

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-21 Thread Karthik Subrahmanya
On Wed, Jun 21, 2017 at 1:56 PM, Xavier Hernandez wrote: > That's ok. I'm currently unable to write a patch for this on ec. Sunil is working on this patch. ~Karthik > If no one can do it, I can try to do it in 6 - 7 hours... > > Xavi > > > On Wednesday, June 21, 2017

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-21 Thread Xavier Hernandez
That's ok. I'm currently unable to write a patch for this on ec. If no one can do it, I can try to do it in 6 - 7 hours... Xavi On Wednesday, June 21, 2017 09:48 CEST, Pranith Kumar Karampuri wrote:    On Wed, Jun 21, 2017 at 1:00 PM, Xavier Hernandez

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-21 Thread Pranith Kumar Karampuri
On Wed, Jun 21, 2017 at 1:00 PM, Xavier Hernandez wrote: > I'm ok with reverting node-uuid content to the previous format and create > a new xattr for the new format. Currently, only rebalance will use it. > > Only thing to consider is what can happen if we have a half

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-21 Thread Xavier Hernandez
I'm ok with reverting node-uuid content to the previous format and create a new xattr for the new format. Currently, only rebalance will use it. Only thing to consider is what can happen if we have a half upgraded cluster where some clients have this change and some not. Can rebalance work in

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Nithya Balachandran
On 21 June 2017 at 10:26, Pranith Kumar Karampuri wrote: > > > On Wed, Jun 21, 2017 at 10:07 AM, Nithya Balachandran > wrote: > >> >> On 20 June 2017 at 20:38, Aravinda wrote: >> >>> On 06/20/2017 06:02 PM, Pranith Kumar Karampuri

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Pranith Kumar Karampuri
On Wed, Jun 21, 2017 at 10:07 AM, Nithya Balachandran wrote: > > On 20 June 2017 at 20:38, Aravinda wrote: > >> On 06/20/2017 06:02 PM, Pranith Kumar Karampuri wrote: >> >> Xavi, Aravinda and I had a discussion on #gluster-dev and we agreed to go >>

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Nithya Balachandran
On 20 June 2017 at 20:38, Aravinda wrote: > On 06/20/2017 06:02 PM, Pranith Kumar Karampuri wrote: > > Xavi, Aravinda and I had a discussion on #gluster-dev and we agreed to go > with the format Aravinda suggested for now and in future we wanted some > more changes for dht

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Aravinda
On 06/20/2017 06:02 PM, Pranith Kumar Karampuri wrote: Xavi, Aravinda and I had a discussion on #gluster-dev and we agreed to go with the format Aravinda suggested for now and in future we wanted some more changes for dht to detect which subvolume went down came back up, at that time we will

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Pranith Kumar Karampuri
Xavi, Aravinda and I had a discussion on #gluster-dev and we agreed to go with the format Aravinda suggested for now and in future we wanted some more changes for dht to detect which subvolume went down came back up, at that time we will revisit the solution suggested by Xavi. Susanth is doing

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Xavier Hernandez
Hi Aravinda, On 20/06/17 12:42, Aravinda wrote: I think following format can be easily adopted by all components UUIDs of a subvolume are seperated by space and subvolumes are separated by comma For example, node1 and node2 are replica with U1 and U2 UUIDs respectively and node3 and node4 are

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Sunil Kumar Heggodu Gopala Acharya
EC also sends all zeros if the node is down. Regards, Sunil kumar Acharya Senior Software Engineer Red Hat T: +91-8067935170 TRIED. TESTED. TRUSTED. On Tue,

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Karthik Subrahmanya
On Tue, Jun 20, 2017 at 4:12 PM, Aravinda wrote: > I think following format can be easily adopted by all components > > UUIDs of a subvolume are seperated by space and subvolumes are separated > by comma > > For example, node1 and node2 are replica with U1 and U2 UUIDs

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Aravinda
I think following format can be easily adopted by all components UUIDs of a subvolume are seperated by space and subvolumes are separated by comma For example, node1 and node2 are replica with U1 and U2 UUIDs respectively and node3 and node4 are replica with U3 and U4 UUIDs respectively

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Aravinda
Hi Xavi, On 06/20/2017 02:51 PM, Xavier Hernandez wrote: Hi Aravinda, On 20/06/17 11:05, Pranith Kumar Karampuri wrote: Adding more people to get a consensus about this. On Tue, Jun 20, 2017 at 1:49 PM, Aravinda > wrote: regards

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Xavier Hernandez
Hi Aravinda, On 20/06/17 11:05, Pranith Kumar Karampuri wrote: Adding more people to get a consensus about this. On Tue, Jun 20, 2017 at 1:49 PM, Aravinda > wrote: regards Aravinda VK On 06/20/2017 01:26 PM, Xavier Hernandez

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Pranith Kumar Karampuri
Adding more people to get a consensus about this. On Tue, Jun 20, 2017 at 1:49 PM, Aravinda wrote: > > regards > Aravinda VK > > > On 06/20/2017 01:26 PM, Xavier Hernandez wrote: > >> Hi Pranith, >> >> adding gluster-devel, Kotresh and Aravinda, >> >> On 20/06/17 09:45,

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Aravinda
regards Aravinda VK On 06/20/2017 01:26 PM, Xavier Hernandez wrote: Hi Pranith, adding gluster-devel, Kotresh and Aravinda, On 20/06/17 09:45, Pranith Kumar Karampuri wrote: On Tue, Jun 20, 2017 at 1:12 PM, Xavier Hernandez > wrote:

Re: [Gluster-devel] geo-rep regression because of node-uuid change

2017-06-20 Thread Xavier Hernandez
Hi Pranith, adding gluster-devel, Kotresh and Aravinda, On 20/06/17 09:45, Pranith Kumar Karampuri wrote: On Tue, Jun 20, 2017 at 1:12 PM, Xavier Hernandez > wrote: On 20/06/17 09:31, Pranith Kumar Karampuri wrote: The way