-
Notifications
You must be signed in to change notification settings - Fork 104
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
Support Amazon Linux 2023 #2692
Merged
Merged
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
d7046e8
[POC] Add support for Amazon Linux 2023
enrico-usai 4787aa1
[AL2023] Add support for Amazon Linux 2023 - ARMPL
118a1be
[AL2023] Installing NVIDIA driver and kernel-modules-extra which is r…
d205aca
[AL2023] Installing gdrcopy
b19b8bb
[AL2023] Installing stunnel
bad0831
[AL2023] Disable SElinux requires a reboot
2780d79
[AL2023] ModuleFile path is /etc/environment-modules/modulespath for …
bb5b9e1
[AL2023] We do not install DCV with Al2023 as its not supported.
293bac8
[AL2023] Installing only available packages
9cb52e2
[AL2023] Add support for Amazon Linux 2023 - EFA
b49713a
[AL2023] Add support for Amazon Linux 2023 - EFS
ba5038c
[AL2023] Add support for Amazon Linux 2023 - Ephemeral drive
0092c16
[AL2023]Add support for Amazon Linux 2023 - Lustre Client
e67ab39
[AL2023] Removing `cfnconfig_mixed` test from environment_install as …
6ab95f3
[AL2023]Add support for Amazon Linux 2023 - Mysql Client ( used Rhel9)
f8cd9ad
[AL2023]Add support for Amazon Linux 2023 - Munge
2ef8cc2
[AL2023]Add support for Amazon Linux 2023 - Slurm
4686d14
[AL2023]Add support for Amazon Linux 2023 - CloudWatchAgent
0517b0d
[AL2023]Add Comment for installing full featured packages
be7ce03
[AL2]removing duplicate functions in AL2022
7ab5ed8
[AL2023]CookStyle change
e62711e
[AL2023] Adding Github Actions for running Kitchen tests for Al2023
ee0f2b2
[AL2023] Unit tests for Nvidia Driver
f25ad05
[AL2023] Installing the full versions of packages as some of them con…
b346ad5
[AL2023] yum-utils is not supported in AL2023
8065007
[AL2023] Supporting AL2023 for Disable Sudo Access
6a6c076
[AL2023] Adding packages which are now supported with differnt names
3b21c22
[AL2023] Supporting Cluster User for Al2023
419a273
[AL2023]Adding Amazon in test metadata
10faecd
[AL2023] Install dmidecode
c424659
[AL2023]Update metadata for AL2023
960ff9f
[AL2023] Adding Guards for NVIDIA
51930ac
[AL2023] Support Cuda Installation by using root directory for instal…
6351b86
[AL2023] Adding installation of rsyslog and enabling the service
9136993
[AL2023] Supporting system authentication for AD
07a8dd2
[AL2023] Remove check to assert service running
d42a555
[AL2023] Avoid openssh recipe on Amazon Linux 2023 + docker
hanwen-cluster 89d077c
[AL2023] mock directory when running tests on docker
hanwen-cluster b58c61a
[AL2023] remove prehook
hanwen-cluster 33fd1d0
Remove support information in metadata
hanwen-cluster 3f84028
[AL2023] Refactor rsyslog installation
hanwen-cluster 9286e4c
[AL2023] Remove network interfaces configuration
hanwen-cluster 1357fd7
[AL2023] Remove outdated packages
hanwen-cluster 5552b26
Merge branch 'develop' into al2023
hanwen-pcluste File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/cuda-install
is meant to be a temporary directory, so we should create it under/tmp
.That said it's fine to use recursive deletion of that folder since it's a temp directory created by us and not meant ot be used by the user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/tmp
on some OSes has size limit. That's why we had to change the directoryThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then please add a comment on top of it to explain why we are not using tmp dir.
However, we should be in control of the tmp dir size and adjust it to our needs.
May you please track in the backlog the possibility to control the /tmp size as a separate partition (that is also a storage best practice)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, added a comment