[lustre-devel] Request for review

[lustre-devel] Request for review

[Patrick Farrell]

Developer types,


I've got an old patch out for LU-8703 (https://jira.hpdd.intel.com/browse/LU-8307), in need of review:
https://review.whamcloud.com/#/c/20888/


It's a pretty simple fix and still applicable now, but it needs reviewers.  So I'm looking for volunteers.  Details are in the ticket.

Any takers?

Thanks,

- Patrick

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lustre.org/pipermail/lustre-devel-lustre.org/attachments/20170504/7674d08e/attachment.htm>

[lustre-devel] Request for review

[Dilger, Andreas]

On May 4, 2017, at 20:24, Patrick Farrell <paf@cray.com> wrote:
> 
> Developer types,
> 
> I've got an old patch out for LU-8703 (https://jira.hpdd.intel.com/browse/LU-8307), in need of review:
> https://review.whamcloud.com/#/c/20888/
> 
> It's a pretty simple fix and still applicable now, but it needs reviewers.  So I'm looking for volunteers.  Details are in the ticket.
> 
> Any takers?

I just found this in my inbox.  Good to see that several people stepped up to do the inspection.

Now, only a few hundred other patches that need review... :-)  I definitely think that in case of patch review that the squeaky wheel gets the attention.  Posting to lustre-devel seems like a good way to get some wider attention onto patches?

Cheers, Andreas
--
Andreas Dilger
Lustre Principal Architect
Intel Corporation

[lustre-devel] Request for review

[Simmons, James A.]

>I just found this in my inbox.  Good to see that several people stepped up to do the inspection.
>
>Now, only a few hundred other patches that need review... :-)  I definitely think that in case of patch review that the squeaky wheel gets the attention.  Posting to lustre-devel seems like a good way to get some >wider attention onto patches?

If this is the case I really like to see 

https://review.whamcloud.com/#/c/26966

reviewed. One more patch after that and the LU-6401 work will be completed.
Once done only one patch to finish off the libcfs header cleanup. So close!!!!