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

feat: allow TanzuNet slug to be set in the Kilnfile stemcell criteria #413

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

crhntr
Copy link
Member

@crhntr crhntr commented Jul 6, 2023

For a while we have maintained a switch statement that mapped stemcell slugs like "ubuntu-jammy/1.2" to product slugs like "stemcells-ubuntu-jammy". This was annoying but not that big of an issue because it we do it every year or so. Instead of continuing this pattern, lets add a field to the Kilnfile so the CLI does not need to change when a new stemcell is released.

This change helps me with my buildpack work because, I want to be able to compile releases against stemcells and I want the particular stemcell to be specified in the tile source. I don't want to maintain the same switch statement in both Kiln and in the buildpack. I intend to import these functions into the buildpack.

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@crhntr
Copy link
Member Author

crhntr commented Jul 7, 2023

@jaristiz I vaguely remember talking with you about this change a long time ago.

Copy link
Contributor

@dtimm dtimm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@crhntr crhntr force-pushed the feat-set-stemcell-slug-in-kilnfile branch from baa37f6 to ea194d8 Compare July 7, 2023 17:08
@crhntr crhntr merged commit b8835d7 into main Jul 7, 2023
3 checks passed
@crhntr crhntr deleted the feat-set-stemcell-slug-in-kilnfile branch July 7, 2023 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants