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

CASMTRIAGE-7252: Increase DEFAULT_IMS_IMAGE_SIZE from 30 to 60 #143

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

mharding-hpe
Copy link
Contributor

@mharding-hpe mharding-hpe commented Sep 24, 2024

CASMTRIAGE-7252 was opened because a CFS image customization session failed because the default IMS image size was too small. In an ideal world, this wouldn't happen because:

  • CFS would try to get an idea of how big the image is, and specify an appropriate build_env_size to the IMS job that it creates.
    or
  • If no build_env_size was specified, IMS would try to determine a reasonable one based on the other parameters.

However, both of those are obviously more complicated things to implement. So this PR takes the safer route of just increasing our default IMS image size to a value that does not hit the problem in question.

@mharding-hpe mharding-hpe requested a review from a team as a code owner September 24, 2024 13:47
@mharding-hpe mharding-hpe force-pushed the casmtriage-7252 branch 2 times, most recently from 425b08d to fbbf0a7 Compare September 24, 2024 14:15
@mharding-hpe mharding-hpe merged commit 8d2a3ac into develop Sep 24, 2024
2 checks passed
@mharding-hpe mharding-hpe deleted the casmtriage-7252 branch September 24, 2024 14:18
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.

1 participant