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.

Sign up for GitHub

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

Jump to bottom

CRAYSAT-1767: Set cray-sat version in csm-config #2962

Merged
merged 5 commits into from
Oct 26, 2023
Merged

CRAYSAT-1767: Set cray-sat version in csm-config #2962

merged 5 commits into from
Oct 26, 2023

Commits on Oct 25, 2023

  1. CRAYSAT-1767: Set cray-sat version in csm-config 

    Add an initContainer definition to the values defined for the
`csm-config` Helm chart. This initContainer sets the cray-sat image tag
to match the version included in the CSM build, so that they are always
consistent. The Ansible content in csm-config sets this cray-sat
container image tag in the file `/opt/cray/etc/sat/version`, so that the
sat-podman wrapper script knows which version of the image to run.

Add logic to `release.sh` that inspects the container image tags
specified in `docker/index.yaml` and injects those tags for the cray-sat
container image and the Alpine Linux container image into the csm-config
Helm chart values in the Loftsman manifest.

Modify `lib/setup-nexus.sh` to set the `cray-sat` container image
version and tag the latest one as `csm-latest`. We are phasing out
reliance on this `csm-latest` tag, but it is still the default used
by the sat-podman wrapper script if the file `/opt/cray/etc/sat/version`
does not exist. Modify `lib/setup-nexus.sh` to also write the tag of the
`cray-sat` image to `/opt/cray/etc/sat/version` as well. This is only
executed on `ncn-m001`, but that is sufficient for bootstrapping. When
the CSM layer of the CFS configuration is applied, this file will be set
on all master and worker management nodes.

Test Description:
Only tested one piece at a time so far.
    @haasken-hpe
    haasken-hpe committed Oct 25, 2023
    Configuration menu
    Copy the full SHA
    aff761a View commit details
    Browse the repository at this point in the history

  2. CRAYSAT-1767: Ensure /opt/cray/etc/sat dir exists in setup-nexus.sh 

    Update `lib/setup-nexus.sh` to ensure the directory `/opt/cray/etc/sat`
exists before writing to the `version` file in that directory. On an
upgrade, this directory should already exist because it will have been
created by management node personalization performed by CFS. However, on
a fresh install, this script is executed on the PIT, and this directory
will not exist. In fact, in that case, `sat` is not even installed on
the PIT, so writing to this file won't affect anything, but it also
won't hurt anything.
    @haasken-hpe
    haasken-hpe committed Oct 25, 2023
    Configuration menu
    Copy the full SHA
    2c2ff32 View commit details
    Browse the repository at this point in the history

  3. CRAYSAT-1767: Use alpine:3 rolling tag 

    Use the rolling tag of `alpine:3` instead of `alpine:3.18` since we do
not depend on any particular functionality specific to a version of the
`alpine` container in its usage in the `csm-config` Helm chart's
initContainer.
    @haasken-hpe
    haasken-hpe committed Oct 25, 2023
    Configuration menu
    Copy the full SHA
    df70c27 View commit details
    Browse the repository at this point in the history

  4. CRAYSAT-1767: Update to latest release of csm-config 

    Update the `sysmgmt.yaml` Loftsman manifest to specify the latest 1.17.0
version of the `csm-config` Helm chart. This is the newly published
release which includes the `csm.ncn.sat` role which is being modified by
the `initContainer` which is being used here.
    @haasken-hpe
    haasken-hpe committed Oct 25, 2023
    Configuration menu
    Copy the full SHA
    75c0f2f View commit details
    Browse the repository at this point in the history

  5. CRAYSAT-1767: Download cray-sat image from csm-docker 

    The cray-sat image now publishes to csm-docker, and this is where the
newer 3.0.0 version of cray-sat-podman expects it to be uploaded to
Nexus on the system. Make the following changes:

* Download cray-sat from csm-docker. This also has the side effect of
  uploading to the same path in Nexus upon installation
* Change code in `lib/setup-nexus.sh` to tag the image as `csm-latest`
  in its new location in Nexus
* Modify the `yq` command in `release.sh` that reads the `cray-sat`
  container image version from `docker/index.yaml`
* Remove harmless but now unnecessary condition on `sat-docker` path in
  `build/images/inspect.sh`

Test Description:
Tested the `yq` command independently to ensure it could still get the
version of `cray-sat`.
    @haasken-hpe
    haasken-hpe committed Oct 25, 2023
    Configuration menu
    Copy the full SHA
    446d10b View commit details
    Browse the repository at this point in the history