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

CRAYSAT-1330: Specify image name directly in CFS session when possible #253

Merged
merged 2 commits into from
Aug 9, 2024

Conversation

haasken-hpe
Copy link
Contributor

@haasken-hpe haasken-hpe commented Aug 8, 2024

Summary and Scope

  • CRAYSAT-1330: Specify image name directly in CFS session when possible

    When possible, specify the desired resulting image name directly when
    creating the CFS session instead of requiring a costly and failure-prone
    image rename operation after the CFS configuration has been completed.

    This greatly improves the reliability and performance of sat bootprep
    for image customization.

    Test Description:
    Not tested yet.

  • CRAYSAT-1330: Update to version 2.0.0 of csm-api-client

    Version 2.0.0 of csm-api-client introduces the ability to specify the
    name of images when images are customized in CFS.

Issues and Related PRs

Testing

Tested on:

  • rocket

Test description:

Created a build of the sat container image and tested as follows:

Created a bootprep file that builds and customizes an image from the CSM barebones image. Also tested the same thing but with it building the image from recipe first.

Verified that the CFS session specified the destination image name, and that SAT didn't have to rename the image. Verified that the image existed afterwards.

Risks and Mitigations

This has the same risks as mentioned in Cray-HPE/python-csm-api-client#30

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable
  • Remove commit that uses unstable builds of csm-api-client when stable build is published

@haasken-hpe haasken-hpe changed the title Craysat 1330 cfs rename parameter CRAYSAT-1330: Specify image name directly in CFS session when possible Aug 8, 2024
@shivaprasad-metimath
Copy link
Contributor

License check seems to be failing, need to update the year in the files

@haasken-hpe haasken-hpe force-pushed the CRAYSAT-1330-cfs-rename-parameter branch from 7c4df3c to 520f8ee Compare August 9, 2024 16:25
@haasken-hpe
Copy link
Contributor Author

License check seems to be failing, need to update the year in the files

It just failed on the Dockerfile, and I'm going to remove the commit that changes that file anyway now that I have a stable 2.0.0 build of csm-api-client.

@haasken-hpe
Copy link
Contributor Author

I do need a changelog update though, oops!

Version 2.0.0 of `csm-api-client` introduces the ability to specify the
name of images when images are customized in CFS.
When possible, specify the desired resulting image name directly when
creating the CFS session instead of requiring a costly and failure-prone
image rename operation after the CFS configuration has been completed.

This greatly improves the reliability and performance of `sat bootprep`
for image customization.

Test Description:
Not tested yet.
@haasken-hpe haasken-hpe force-pushed the CRAYSAT-1330-cfs-rename-parameter branch from 520f8ee to ec2a1fd Compare August 9, 2024 16:35
@haasken-hpe haasken-hpe merged commit 7088e90 into main Aug 9, 2024
3 checks passed
@haasken-hpe haasken-hpe deleted the CRAYSAT-1330-cfs-rename-parameter branch August 9, 2024 20:54
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.

3 participants