[LU-3008] Tracking ticket for updating gnilnd Created: 21/Mar/13 Updated: 23/Apr/13 Resolved: 23/Apr/13 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.4.0 |
| Fix Version/s: | Lustre 2.4.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | James Shimek | Assignee: | Keith Mannthey (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | lnet, patch | ||
| Rank (Obsolete): | 7321 |
| Description |
|
Cray is pushing an update to the upstream gnilnd sources based on current development work. This ticket is for tracking that push. |
| Comments |
| Comment by James Shimek [ 22/Mar/13 ] |
|
Link to Patch: http://review.whamcloud.com/#change,5815 |
| Comment by James A Simmons [ 02/Apr/13 ] |
|
Peter could you have people look at the patch for this ticket. It would be nice to land this before the week end. Thanks |
| Comment by Peter Jones [ 02/Apr/13 ] |
|
James Efforts are really focused on things on the critical path for 2.4 this week so I really cannot guarantee that someone will get to this. However, Cray have clearly signed off on this work or else they would not have upstreamed it and they are the ones responsible for this code so I think that it should be ok for you to still use it if you choose Peter |
| Comment by Andreas Dilger [ 11/Apr/13 ] |
|
At a minimum, this ticket should contain a description of the defects and fixes included in the patch, though it would be better to include that in the commit comment. "fixes for the past year" doesn't give anyone much information. |
| Comment by James Shimek [ 13/Apr/13 ] |
|
I'll update this tracking ticket with a list of the patches that are included in the push. If needed I can update the Gerrit commit with the same information. |
| Comment by Andreas Dilger [ 14/Apr/13 ] |
|
Thanks James. I would recommend resubmitting your patch with an updated commit comment (it can be tens or hundreds of lines long if you want), and cleaning up the whitespace. Patch turnaround time is about 12h or less most times. |
| Comment by James A Simmons [ 23/Apr/13 ] |
|
Now that this patch is landed I suggest we can close this ticket. |
| Comment by Peter Jones [ 23/Apr/13 ] |
|
ok. Landed for 2.4 |