Skip to content

[8.0] Platform detection on recent EL9 broken#8185

Merged
chrisburr merged 1 commit into
DIRACGrid:rel-v8r0from
sfayer:fix_platformdet
May 5, 2025
Merged

[8.0] Platform detection on recent EL9 broken#8185
chrisburr merged 1 commit into
DIRACGrid:rel-v8r0from
sfayer:fix_platformdet

Conversation

@sfayer
Copy link
Copy Markdown
Member

@sfayer sfayer commented May 2, 2025

Hi,

A recent update to glibc on EL9 platforms has resulted in dirac-platform returning an incorrect result:

$ rpm -q glibc
glibc-2.34-125.el9_5.8.x86_64
$ dirac-platform
Linux_x86_64_glibc-2.35
# This should clearly be 2.34! It only went wrong after glibc was recently updated.
# Python itself gets it correct:
$ python -c "import platform; print(platform.libc_ver())"
('glibc', '2.34')

It appears DIRAC contains a copy of the platform.libc_ver() function from the python standard library due to a really old bug: This copy has now aged and doesn't work correctly any more. As a fix, I propose going back to the standard version and dropping the copy entirely (as the bug was fixed over a decade ago!).

Regards,
Simon

BEGINRELEASENOTES
*Core
FIX: Platform detection on recent EL9 broken
ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label May 2, 2025
@chrisburr chrisburr merged commit 0b13bd2 into DIRACGrid:rel-v8r0 May 5, 2025
34 of 43 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels May 5, 2025
@DIRACGridBot
Copy link
Copy Markdown

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/14831410664

Failed:

  • integration
    cherry-pick 0b13bd2 into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-0b13bd25a-integration
    git cherry-pick -x -m 1 0b13bd25a
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #8185 Platform detection on recent EL9 broken' --author='Simon Fayer <sf105@ic.ac.uk>'
    git push -u origin cherry-pick-2-0b13bd25a-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] Platform detection on recent EL9 broken' \
         --body 'Sweep #8185 `Platform detection on recent EL9 broken` to `integration`.
    
    Adding original author @sfayer as watcher.
    
    BEGINRELEASENOTES
    *Core
    FIX: Platform detection on recent EL9 broken
    ENDRELEASENOTES
    Closes #8186'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants