[LU-7699] Overhaul lustre's versioning

• Type: Technical task
• Resolution: Done
• Priority: Critical
• Fix Version/s: Lustre 2.9.0
• Assignee: Christopher Morrone (Inactive)
• Reporter: Christopher Morrone (Inactive)

Description

There are numerous oddities about Lustre's system for applying version numbers in the configuration and packaging system. At this point, it is looking strongly like an overhaul is in order.

Issue Links

• is blocking: LU-7642 Allow lustre source build without git working directory
• is blocking: LU-7643 Remove kernel version string from Lustre release field
• is blocking: LU-7645 Stop controlling the RPM Release field from Lustre's build system
• is related to: LU-7976 sles builds severely impacted by recent naming changes
• is related to: LU-474 fix release build issues

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/19488/
Subject: LU-7699 build: Convert version underscores to dashes for dpkg
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f2d28899a2c3b5a427f4322623ba138887fe6adc

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/18111/
Subject: LU-7699 build: Convert lustre_ver.h.in into a static .h file
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6e84dbce05959e854d886772841a5d6d0690eb65

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/18108/
Subject: LU-7699 build: Eliminate lustre_build_version.h
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9c710162e5acebd860f1d3f0e1bf204ac1ba98c1

[Christopher Morrone (Inactive)]

Already mentioned this to Chris, but this may have broken SUSE builds on the Lustre Buildbot.

I don't think so. I think that this is a problem in the buildbot configuration. I will open a github ticket shortly.

[Giuseppe Di Natale (Inactive)]

Already mentioned this to Chris, but this may have broken SUSE builds on the Lustre Buildbot. The error I'm seeing on builds is "error: line 86: Empty tag: Release:". Example log is here.

[James A Simmons]

Yeah more simplification. Debian platforms are back in business. Thanks Chris.

[Gerrit Updater]

Christopher J. Morrone ([e-mail]) uploaded a new patch: http://review.whamcloud.com/19488
Subject: LU-7699 build: Convert version underscores to dashes for dpkg
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 368150054a379f1da81dce8b2e5016016f126655

[Christopher Morrone (Inactive)]

Thanks for the legwork on that, James!

We don't even need sed any more; I'll just use tr.

[James A Simmons]

I tracked down what needs to be changed for make debs. We have:

lversion=$$(sed -ne 's/^#define VERSION "(.*)"$$/\1/p' config.h);

The sed command needs to be modified to change '_' to '-'. I'm no sed master so Chris what needs to be done to make that so.

[Christopher Morrone (Inactive)]

Also I found when you have a top level .git directory if you remove the LUSTRE-VERSION-FILE it will not regenerate and it causes make rpms to fail.

Yeah, that is true. For now, you will just need to rerun autogen.sh to make any version changes. This could be automated in the future, but we pretty much need to fix autoreconf (LU-7700) to do it sanely.

[James A Simmons]

I discovered the is needed on rpms systems for the version field the hard way today. For dpkgs both Name and Version fields can not have '_'. I need to rethink this. I did see many sites stating you should avoid '-' at all cost for rpms Also I found when you have a top level .git directory if you remove the LUSTRE-VERSION-FILE it will not regenerate and it causes make rpms to fail.