[LU-109] Tree cleanup for 2.1 Created: 03/Mar/11 Updated: 04/Mar/11 Resolved: 04/Mar/11 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.1.0 |
| Fix Version/s: | Lustre 2.1.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Christopher Morrone | Assignee: | Oleg Drokin |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 10482 |
| Description |
|
I wanted to bring attention to two simple tree cleanup patches that I put in gerrit before the change to requiring LU-?? in the first comment line: The first patch simply removes the "nodelustre" script, which was apparently used many years ago in the days of the LLNL MCR machine. It uses lconf and lustre xml config files which are entirely out of use. http://review.whamcloud.com/291 The second one removes some backwards compatibility code in the lustre.spec file to handle the modules "alias lustre llite". I think that is so old that we no longer need it. It also removes a reference to the same thing in an error message which confused one of our sysadmins. http://review.whamcloud.com/264 |
| Comments |
| Comment by Christopher Morrone [ 04/Mar/11 ] |
|
Oleg, having a conversation seems to be hard with gerrit, so I'll reply to your comment in http://review.whamcloud.com/264 here: Yes, I noticed that too and submitted patch http://review.whamcloud.com/263. But Andreas correctly pointed out that Brian Murrell had already removed those lines in http://review.whamcloud.com/260. |
| Comment by Peter Jones [ 04/Mar/11 ] |
|
Oleg is working on this |
| Comment by Oleg Drokin [ 04/Mar/11 ] |
|
landed both patches |