https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-16242 Lustre <p>RHEL 9 and its derivatives are no longer distributed with python 2. Two scripts specify python2 as their interpreter:<br/> contrib/scripts/gerrit_checkpatch.py<br/> lustre/scripts/zfsobj2fid<br/> I did attempt to create the distribution with tests and i/o kit disabled, but the lustre-client-dkms RPM still required /usr/bin/python2.<br/> I modified the interpreter to python3 for those scripts and tested. The RPM will no longer requires /usr/bin/python2 and can then be installed on rocky9. </p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> #!/usr/bin/env python2 </pre> </div></div> <p>becomes:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> #!/usr/bin/env python3 </pre> </div></div> <p>When I looked for previous issues related to this, it appeared that 'python' was changed to 'python2' for debian builds. I didn't test these scripts with python3.</p> LU-16242

Self-built lustre-client-dkms requires /usr/bin/python2

Question/Request Minor Resolved Duplicate WC Triage Jesse Stroik Mon, 17 Oct 2022 16:18:31 +0000 Tue, 1 Aug 2023 00:21:06 +0000 Tue, 1 Aug 2023 00:21:06 +0000 0 4 <p>RHEL 9 and its derivatives are no longer distributed with python 2. Two scripts specify python2 as their interpreter:</p> <p>contrib/scripts/gerrit_checkpatch.py</p> <p>lustre/scripts/zfsobj2fid</p> <p>I did attempt to create the distribution with tests and i/o kit disabled, but the lustre-client-dkms RPM still required /usr/bin/python2.</p> <p>I modified the interpreter to python3 for those scripts and tested. The RPM will no longer requires /usr/bin/python2 and can then be installed on rocky9. </p> <p>#!/usr/bin/env python2</p> <p>becomes:</p> <p>#!/usr/bin/env python3</p> <p>When I looked for previous issues related to this, it appeared that 'python' was changed to 'python2' for debian builds. I didn't test these scripts with python3.</p> <p>Hi Jesse, could you please push a patch for changing these tools to use python3. Have you actually tried running them after changing the python version? I'm not much of a python person myself, but know that they may need minor updates to work with the new version. </p> <p>Yes please push the patches. I would be great to drop python2.</p> <p>I have <b>not</b> tested these scripts.</p> <p>pylint suggests minor changes are needed lustre/scripts/zfsobj2fid.  It looks like someone might have already updated this script and added pout.decode() call that I would have missed. It's in the <a href="https://github.com/LLNL/lustre-tools-llnl/commit/184e95f601d6e4d84b1fcb335ff26bf363ebf948" class="external-link" target="_blank" rel="nofollow noopener">LLNL lustre-tools repo</a></p> <p>The other script, contrib/scripts/gerrit_checkpatch.py, looks more involved. I'm not a python programmer so I don't think I'd be a good choice to cobble together changes to that <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/wink.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>This was fixed on master via patch <a href="https://review.whamcloud.com/50241" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/50241</a> "<tt><a href="https://jira.whamcloud.com/browse/LU-16626" title="Don&#39;t use or check for python2 for lustre-zfs-dkms rpm" class="issue-link" data-issue-key="LU-16626"><del>LU-16626</del></a> build: remove python2 dependencies</tt>".</p> Related LU-16626 LU-15874 Development Epic/Theme build Rank 1|i03307: Rank (Obsolete) 9223372036854775807