Update linux-common role to support Infiniband netdevs #230
Conversation
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
JohnGarbutt
reviewed
Oct 8, 2024
* Allow image builds to run on other clouds * Update zenith tag * Add support for specifying a volume type
Co-authored-by: mkjpryor <[email protected]>
…3) (#227) Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
* Update to use linux-ansible-init role from Galaxy and remove in-repo * Update requirements.yml add external ansible-init to requirements * Update main.yml update ansible-init dependency for linux-webconsole * Update main.yml * Update main.yml * Update main.yml * Update requirements.yml * Use ansible-init from image_utils collection * bump azimuth-image-utils release * verbose linux-rstudio.pkr.hcl * verbose linux-rstudio.pkr.hcl * image-utils 0.2.0 * Update linux-rstudio.pkr.hcl --------- Co-authored-by: bertie <[email protected]>
* Move common tasks into a shared linux-common role * Update Trivy action version * Remove isolate command to prevent ansible-init from running
Co-authored-by: mkjpryor <[email protected]>
#244) Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
Co-authored-by: mkjpryor <[email protected]>
tomclark0
changed the title
|
Changing the renderer without changing the activator led to the network interface bouncing after network-online.target is reached. |
mkjpryor
requested changes
Oct 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Introduces a linux-common role, used by all linux based images for changes from default source images.
This role is initially used to provide InfiniBand support using inbox drivers from the relevant distro.
It may later be serve other uses cases where additional packages are required as a baseline, e.g for diagnostic uses.
The default cloud-init network config renderer for Ubuntu (netplan) does not support InfiniBand type devices. To mitigate this, the renderer priority list is changed to match EL9 distros.
The required package for InfiniBand inbox drivers in Ubuntu is kernel-module-extra-$(uname -r). This is a per-kernel package, i.e any kernel update requires the explicit installation of the new version.
To support kernel updates within an image, the linux-generic metapackage is used to provide tracking across kernel versions.
EL9 based distros require only additional packages and module loading.