Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve RHEL container files #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tiran
Copy link

@tiran tiran commented Jun 19, 2024

  • Update to Python 3.10.14, which is currently the latest security release of Python 3.10 series.
  • Use crypto policy DEFAULT:SHA1 instead of LEGACY to work around Habana GPG key with SHA-1 signature. It's less insecure than LEGACY.
  • Enable GPG checks for CentOS repositories.
  • Only install 64bit Intel oneMKL packages. The 32bit packages are not needed.
  • Use $(DOCKER) variable in Makefile for PyTorch container, so make DOCKER=podman works correctly.
  • Explicitly set PYTHON_VERSION, so the package installer uses a fully qualified Python binary (e.g. python3.10) for installation.
  • Use a .pth file instead of PYTHONPATH env var

@tiran tiran force-pushed the rhel-improvements branch from 0c350f0 to b524d8b Compare June 19, 2024 13:43
- Update to Python 3.10.14, which is currently the latest security
  release of Python 3.10 series.
- Use crypto policy `DEFAULT:SHA1` instead of `LEGACY` to work around
  Habana GPG key with SHA-1 signature. It's less insecure than `LEGACY`.
- Enable GPG checks for CentOS repositories.
- Only install 64bit Intel oneMKL packages. The 32bit packages are not
  needed.
- Use `$(DOCKER)` variable in Makefile for PyTorch container, so
  `make DOCKER=podman` works correctly.
- Explicitly set `PYTHON_VERSION`, so the package installer uses a
  fully qualified Python binary (e.g. `python3.10`) for installation.
- Use a `.pth` file instead of `PYTHONPATH` env var

Signed-off-by: Christian Heimes <[email protected]>
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.

1 participant