Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

MTL-2422 -- SLES 15.6 -- CSM 1.6 #708

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

MTL-2422 -- SLES 15.6 -- CSM 1.6 #708

wants to merge 19 commits into from

Conversation

jpdavis-prof
Copy link
Contributor

Summary and Scope

https://jira-pro.it.hpe.com:8443/browse/MTL-2422

Issue Type

  • RFE Pull Request

Prerequisites

  • I have included documentation in my PR (or it is not required)
  • I have tested this by building an image using this branch HEAD, or this is does not pertain to the node-images pipeline.
  • I tested this on internal system or this pertains to the node-images pipeline (if yes and not build related, please include results or a description of the test)

Risks and Mitigations

jpdavis-prof and others added 19 commits July 26, 2024 13:59
downgrade libicu75 to libicu73
downgrade metal-ipxe
After Kernel 6.0.1 `nf_conntrack_helper` was removed.

To replace it, an iptables prerouting rule can be implemented to help TFTP/UDP packets properly return through a NAT.

This allows TFTP to work from Kubernetes in SLES-15-SP6.
In some environments such as vShasta, we may not have an `ipam` key in the `ds.meta-data` dictionary. Assuming the key exists causes the template engine to fail. This file is templated by our `net-init.sh` script, the failure causes that script to exit early.
Since not all of `net-init.sh` has been confirmed to work when `ipam` isn't defined, and due to `iptables_config`'s criticality, run `iptables_config` earlier at a point we know works without `ipam` set.
Adds a `ExecCondition` that checks whether our `/etc/iptables/metal.conf` file exists before trying to start. This will more clearly state why the service failed instead of leaving the `ExecStart` command output to the user to interpret.

Alpha-sorts the properties.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants