Skip to content

Fix detection of RHEL 6 ComputeNode#255

Merged
nir0s merged 4 commits into
python-distro:masterfrom
adamjstewart:fixes/rhel
Mar 30, 2020
Merged

Fix detection of RHEL 6 ComputeNode#255
nir0s merged 4 commits into
python-distro:masterfrom
adamjstewart:fixes/rhel

Conversation

@adamjstewart
Copy link
Copy Markdown
Contributor

@adamjstewart adamjstewart commented Feb 27, 2020

This PR fixes detection of RHEL 6 ComputeNode.

This bug was originally reported by @CreRecombinase in spack/spack#15251 and fixed in Spack's vendored copy of distro in spack/spack#15253

Let me know if you want me to add unit tests for this. spack/spack#15251 has the output of lsb_release -a for the problematic nodes.

@adamjstewart adamjstewart changed the title Fix detection of redhat enterprise compute node Fix detection of RHEL 6 ComputeNode Feb 27, 2020
@adamjstewart
Copy link
Copy Markdown
Contributor Author

Ping @nir0s, any idea why the CI tests failed? I think they just need to be restarted.

@adamjstewart
Copy link
Copy Markdown
Contributor Author

Ping ping @nir0s

@adamjstewart
Copy link
Copy Markdown
Contributor Author

Ping ping ping @nir0s

@nir0s nir0s merged commit 4d76ae3 into python-distro:master Mar 30, 2020
@adamjstewart adamjstewart deleted the fixes/rhel branch March 30, 2020 21:06
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Apr 6, 2020
## 1.5.0 (2020.3.30)

BACKWARD COMPATIBILITY:
* Keep output as native string so we can compatible with python2 interface [[#248](python-distro/distro#248)]

ENHANCEMENTS:

BUG FIXES:
* Fix detection of RHEL 6 ComputeNode [[#255](python-distro/distro#255)]
* Fix Oracle 4/5 lsb_release id and names [[#250](python-distro/distro#250)]
* Ignore /etc/plesk-release file while parsing distribution [[#246](python-distro/distro#246)]

TESTS:
* Test on MidnightBSD [[#254](python-distro/distro#254)]

RELEASE:
* Remove build testing on Python 3.3 and add pypy 3.5 and Python 3.7 and 3.8 [[#244](python-distro/distro#244)]
* Fix Windows build due to Appveyor not supporting builds on Python 2.7 anymore [[#257](python-distro/distro#257)]

Additionally, The Python2.6 branch was fixed to support distro version 1.4.0. It is now passing all tests. Thanks [abadger](https://github.com/abadger)!
msk pushed a commit to msk/pkgsrc that referenced this pull request May 11, 2026
## 1.5.0 (2020.3.30)

BACKWARD COMPATIBILITY:
* Keep output as native string so we can compatible with python2 interface [[#248](python-distro/distro#248)]

ENHANCEMENTS:

BUG FIXES:
* Fix detection of RHEL 6 ComputeNode [[#255](python-distro/distro#255)]
* Fix Oracle 4/5 lsb_release id and names [[#250](python-distro/distro#250)]
* Ignore /etc/plesk-release file while parsing distribution [[#246](python-distro/distro#246)]

TESTS:
* Test on MidnightBSD [[#254](python-distro/distro#254)]

RELEASE:
* Remove build testing on Python 3.3 and add pypy 3.5 and Python 3.7 and 3.8 [[#244](python-distro/distro#244)]
* Fix Windows build due to Appveyor not supporting builds on Python 2.7 anymore [[#257](python-distro/distro#257)]

Additionally, The Python2.6 branch was fixed to support distro version 1.4.0. It is now passing all tests. Thanks [abadger](https://github.com/abadger)!
jperkin pushed a commit to TritonDataCenter/pkgsrc that referenced this pull request May 14, 2026
## 1.5.0 (2020.3.30)

BACKWARD COMPATIBILITY:
* Keep output as native string so we can compatible with python2 interface [[#248](python-distro/distro#248)]

ENHANCEMENTS:

BUG FIXES:
* Fix detection of RHEL 6 ComputeNode [[#255](python-distro/distro#255)]
* Fix Oracle 4/5 lsb_release id and names [[#250](python-distro/distro#250)]
* Ignore /etc/plesk-release file while parsing distribution [[#246](python-distro/distro#246)]

TESTS:
* Test on MidnightBSD [[#254](python-distro/distro#254)]

RELEASE:
* Remove build testing on Python 3.3 and add pypy 3.5 and Python 3.7 and 3.8 [[#244](python-distro/distro#244)]
* Fix Windows build due to Appveyor not supporting builds on Python 2.7 anymore [[#257](python-distro/distro#257)]

Additionally, The Python2.6 branch was fixed to support distro version 1.4.0. It is now passing all tests. Thanks [abadger](https://github.com/abadger)!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants