Re: [Nfs-ganesha-devel] Ready for Matt's c++ patches and then tag V2.4.0!

2016-09-20 Thread Frank Filz
Two more patches. One that probably conflicts with the c++ changes

Sent from my iPhone

> On Sep 20, 2016, at 6:01 PM, Matt W. Benjamin  wrote:
> 
> I rebased yesterday--is there more to do?
> 
> Matt
> 
> - "Frank Filz"  wrote:
> 
>> Matt, Could you rebase and submit your c++ compile patches?
>> 
>> Assuming nothing crashes and burns, I will merge those and tag V2.4.0
>> by
>> tomorrow afternoon.
>> 
>> On the concall, we discussed keeping things a bit in flux and not
>> formally
>> opening V2.5 until after Bakeathon.
>> 
>> Bill has some RDMA patches he wants tested at Bakeathon. We could
>> include
>> those (though he could also just bring a branch running those to
>> BAT...) in
>> next. I suspect there will also be some performance fixes and other
>> bug
>> fixes over the next few weeks.
>> 
>> We could tag V2.4.1 right after Bakeathon with anything up until that
>> point.
>> 
>> Thanks all,
>> 
>> Frank
>> 
>> 
>> ---
>> This email has been checked for viruses by Avast antivirus software.
>> https://www.avast.com/antivirus
>> 
>> 
>> --
>> ___
>> Nfs-ganesha-devel mailing list
>> Nfs-ganesha-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
> 
> -- 
> Matt Benjamin
> CohortFS, LLC.
> 315 West Huron Street, Suite 140A
> Ann Arbor, Michigan 48103
> 
> http://cohortfs.com
> 
> tel.  734-761-4689 
> fax.  734-769-8938 
> cel.  734-216-5309 


--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Ready for Matt's c++ patches and then tag V2.4.0!

2016-09-20 Thread Matt W. Benjamin
Is RDMA feature complete for NFS3?  That would be interesting.

Matt

- "Frank Filz"  wrote:

> Matt, Could you rebase and submit your c++ compile patches?
> 
> Assuming nothing crashes and burns, I will merge those and tag V2.4.0
> by
> tomorrow afternoon.
> 
> On the concall, we discussed keeping things a bit in flux and not
> formally
> opening V2.5 until after Bakeathon.
> 
> Bill has some RDMA patches he wants tested at Bakeathon. We could
> include
> those (though he could also just bring a branch running those to
> BAT...) in
> next. I suspect there will also be some performance fixes and other
> bug
> fixes over the next few weeks.
> 
> We could tag V2.4.1 right after Bakeathon with anything up until that
> point.
> 
> Thanks all,
> 
> Frank
> 
> 
> ---
> This email has been checked for viruses by Avast antivirus software.
> https://www.avast.com/antivirus
> 
> 
> --
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

-- 
Matt Benjamin
CohortFS, LLC.
315 West Huron Street, Suite 140A
Ann Arbor, Michigan 48103

http://cohortfs.com

tel.  734-761-4689 
fax.  734-769-8938 
cel.  734-216-5309 

--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] Ready for Matt's c++ patches and then tag V2.4.0!

2016-09-20 Thread Matt W. Benjamin
I rebased yesterday--is there more to do?

Matt

- "Frank Filz"  wrote:

> Matt, Could you rebase and submit your c++ compile patches?
> 
> Assuming nothing crashes and burns, I will merge those and tag V2.4.0
> by
> tomorrow afternoon.
> 
> On the concall, we discussed keeping things a bit in flux and not
> formally
> opening V2.5 until after Bakeathon.
> 
> Bill has some RDMA patches he wants tested at Bakeathon. We could
> include
> those (though he could also just bring a branch running those to
> BAT...) in
> next. I suspect there will also be some performance fixes and other
> bug
> fixes over the next few weeks.
> 
> We could tag V2.4.1 right after Bakeathon with anything up until that
> point.
> 
> Thanks all,
> 
> Frank
> 
> 
> ---
> This email has been checked for viruses by Avast antivirus software.
> https://www.avast.com/antivirus
> 
> 
> --
> ___
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

-- 
Matt Benjamin
CohortFS, LLC.
315 West Huron Street, Suite 140A
Ann Arbor, Michigan 48103

http://cohortfs.com

tel.  734-761-4689 
fax.  734-769-8938 
cel.  734-216-5309 

--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


[Nfs-ganesha-devel] Ready for Matt's c++ patches and then tag V2.4.0!

2016-09-20 Thread Frank Filz
Matt, Could you rebase and submit your c++ compile patches?

Assuming nothing crashes and burns, I will merge those and tag V2.4.0 by
tomorrow afternoon.

On the concall, we discussed keeping things a bit in flux and not formally
opening V2.5 until after Bakeathon.

Bill has some RDMA patches he wants tested at Bakeathon. We could include
those (though he could also just bring a branch running those to BAT...) in
next. I suspect there will also be some performance fixes and other bug
fixes over the next few weeks.

We could tag V2.4.1 right after Bakeathon with anything up until that point.

Thanks all,

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


[Nfs-ganesha-devel] Announce Push of V2.4-rc6

2016-09-20 Thread Frank Filz
Branch next

Tag:V2.4-rc6

NOTE: This merge includes an ntirpc pullup, please update your submodule.

Release Highlights

* Several FSAL_GLUSTER fixes

* Handle ntohl of exportid in file handle differently so FSAL_PROXY
unafected

* ntirpc pullup - Coverity fixes and other

* Fix attribute pass back in FSAL_PROXY

* Fix gid regression in FSAL_RGW

Signed-off-by: Frank S. Filz 

Contents:

7b76e77 Frank S. Filz V2.4-rc6
fa90477 Matt Benjamin rgw: fix gid regression in fsal_rgw_setattr2
c5eade6 Patrice LUCAS Fix file handle bytes switching in xdr_nfs_fh4
XDR_ENCODE
3bede69 Patrice LUCAS FIX attributes pass back in pxy_lookup_path
68f76bf Matt Benjamin ntirpc pullup
bdcd2d7 Jiffin Tony Thottan FSAL_GLUSTER : Avoid double free for
glfsal_module
ef109a2 Daniel Gryniewicz FSAL_RGW - Add back a close() op
fffcfbf Soumya Koduri FSAL_GLUSTER: Use chk_verifier_stat to verifier
comparision in open2()
b5019ad Soumya Koduri FSAL_GLUSTER: Clear ATTR_RDATTR_ERR mask bit in case
of successful read
89ca7f7 Soumya Koduri FSAL_GLUSTER: avoid fetching attributes in setattr2
cbeaa47 Soumya Koduri FSAL_GLUSTER: Avoid redundant fsync operation


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: rgw: fix gid regression in fsal_rgw_setattr2

2016-09-20 Thread GerritHub
>From Matt Benjamin :

Matt Benjamin has uploaded a new change for review.

  https://review.gerrithub.io/295195

Change subject: rgw: fix gid regression in fsal_rgw_setattr2
..

rgw: fix gid regression in fsal_rgw_setattr2

This apparently broke with support_ex--but is a typo.

Matt

Change-Id: Idb8878a1fb928714f3b5dcf086c2a4c1fc81f126
Signed-off-by: Matt Benjamin 
---
M src/FSAL/FSAL_RGW/handle.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/95/295195/1
-- 
To view, visit https://review.gerrithub.io/295195
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idb8878a1fb928714f3b5dcf086c2a4c1fc81f126
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Matt Benjamin 

--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] pre-merge for V2.4-rc6

2016-09-20 Thread Frank Filz
> I have posted a pre-merge for V2.4-rc6, it contains the patches that have
so
> far been code review +2, verif +1 as found here:
> 
> https://review.gerrithub.io/#/q/status:open+label:Code-
> Review%253D2+project:
> ffilz/nfs-ganesha+branch:next
> 
> I have posted to my ffilz githib next branch:
> 
> https://github.com/ffilz/nfs-ganesha/commits/next
> 
> The primary intent is for Matt to do a rebase of his C++ changes, but if
> anyone else wants to throw mud at it and see what sticks, we have a day or
> two for additional patches before we have to close down 2.4.0.
> 
> NOTE: This branch is subject to rebasing or other git history mangling
things,
> that's why I'm NOT pushing it to nfs-ganesha/next...

I have added Patrice's two patches to the above.

Marc, would you please test FSAL_GPFS pNFS against this branch.

Thanks

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: FIX attributes pass back in pxy_lookup_path

2016-09-20 Thread GerritHub
>From Patrice LUCAS :

Patrice LUCAS has uploaded a new change for review.

  https://review.gerrithub.io/295148

Change subject: FIX attributes pass back in pxy_lookup_path
..

FIX attributes pass back in pxy_lookup_path

In pxy_lookup_path, we test pnext to pass back attributes
only for the last entry when calling pxy_lookup_impl. The test
was inverted and led to entries without attrs when mdcache was
upon FSAL_PROXY. FSAL_PROXY tests revealed first mount error
when attrs of root entry were empty.

Change-Id: I2232e951e4ce04a95fe20077a47d07b6e7fe6064
Signed-off-by: Patrice LUCAS 
---
M src/FSAL/FSAL_PROXY/handle.c
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/48/295148/1
-- 
To view, visit https://review.gerrithub.io/295148
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2232e951e4ce04a95fe20077a47d07b6e7fe6064
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Patrice LUCAS 

--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] ntirpc-1.4.0 GA sooner, rather than later.

2016-09-20 Thread Matt Benjamin
they are the same now because I moved the tag;  I can try to restore it so 
there is a difference, but it would be unfortunate

- Original Message -
> From: "Kaleb S. KEITHLEY" 
> To: "Matt Benjamin" , 
> nfs-ganesha-devel@lists.sourceforge.net
> Sent: Tuesday, September 20, 2016 10:21:37 AM
> Subject: Re: ntirpc-1.4.0 GA sooner, rather than later.
> 
> On 09/20/2016 10:06 AM, Matt Benjamin wrote:
> > Sorry.  I had moved the 1.4.0 tag, and then decided I'd better create a new
> > one.
> >
> 
> Well, _release_ tags should never be moved.
> 
> But is there supposed to be a difference between 1.4.0 and 1.4.1,
> because there isn't, as far as I can tell. At least a `diff -r` of the
> two tar files (untarred) doesn't show any diff.
> 
> 
> > Matt
> >
> > - Original Message -
> >> From: "Kaleb S. KEITHLEY" 
> >> To: "Matt Benjamin" 
> >> Sent: Tuesday, September 20, 2016 7:53:12 AM
> >> Subject: Re: ntirpc-1.4.0 GA sooner, rather than later.
> >>
> >> On 09/19/2016 05:43 PM, Matt Benjamin wrote:
> >>> had to push a 1.4.1
> >>>
> >>
> >> Why? There's no diff between the two trees?
> >>
> >> Curiosity is killing the cat. ;-)
> >>
> >> --
> >>
> >> Kaleb
> >>
> >>
> >>
> >
> 
> 

-- 
Matt Benjamin
Red Hat, Inc.
315 West Huron Street, Suite 140A
Ann Arbor, Michigan 48103

http://www.redhat.com/en/technologies/storage

tel.  734-707-0660
fax.  734-769-8938
cel.  734-216-5309

--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


Re: [Nfs-ganesha-devel] ntirpc-1.4.0 GA sooner, rather than later.

2016-09-20 Thread Kaleb S. KEITHLEY
On 09/20/2016 10:06 AM, Matt Benjamin wrote:
> Sorry.  I had moved the 1.4.0 tag, and then decided I'd better create a new 
> one.
>

Well, _release_ tags should never be moved.

But is there supposed to be a difference between 1.4.0 and 1.4.1, 
because there isn't, as far as I can tell. At least a `diff -r` of the 
two tar files (untarred) doesn't show any diff.


> Matt
>
> - Original Message -
>> From: "Kaleb S. KEITHLEY" 
>> To: "Matt Benjamin" 
>> Sent: Tuesday, September 20, 2016 7:53:12 AM
>> Subject: Re: ntirpc-1.4.0 GA sooner, rather than later.
>>
>> On 09/19/2016 05:43 PM, Matt Benjamin wrote:
>>> had to push a 1.4.1
>>>
>>
>> Why? There's no diff between the two trees?
>>
>> Curiosity is killing the cat. ;-)
>>
>> --
>>
>> Kaleb
>>
>>
>>
>


--
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel