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

Reintroduce [[stacks]] table to workaround release issues. #210

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

joshwlewis
Copy link
Member

@joshwlewis joshwlewis commented Feb 28, 2024

The 3.0.0 release is currently blocked: https://github.com/heroku/procfile-cnb/actions/runs/8025023370/job/21924849039. It's failing with ERROR: saving image: no compatible stacks among provided buildpacks.

The issue is that pack buildpack package requires [[stacks]] to be present in buildpack.toml, which is unexpected (and maybe unintentional), since [[stacks]] were deprecated in Buildpack API 0.10.

We could wait for an upstream fix (like buildpacks/pack#2081 🤞🏻 ), or we could workaround the issue by adding the [[stacks]] table back in.

Functionally, having no [[stacks]] table and having [[stacks]] id = "*" seem to be the same. And while the [[stacks]] table is deprecated, there isn't any messaging saying so in the pack output. So, the only downside that I see to adding [[stacks]] back in is that we'll probably want to remove it again later.

@joshwlewis joshwlewis requested a review from a team as a code owner February 28, 2024 00:20
@edmorley
Copy link
Member

edmorley commented Feb 28, 2024

I've triggered Dependabot manually via https://github.com/heroku/procfile-cnb/network/updates to pick up the libcnb 0.19.0 release needed for [[stacks]] compat, which is now merged to main.

@joshwlewis
Copy link
Member Author

I've triggered Dependabot manually.

Thanks. I went looking for a button, but didn't find it and opened #212 manually. I'll close that one.

@edmorley
Copy link
Member

It's kinda tucked away: Main repo view -> Insights tab -> Dependency graph on side menu -> Dependabot tab.

@joshwlewis joshwlewis enabled auto-merge (squash) February 28, 2024 00:34
Copy link
Member

@edmorley edmorley left a comment

Choose a reason for hiding this comment

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

The release date will need updating in the changelog before re-release:
https://github.com/heroku/procfile-cnb/blob/306b4f2b83146fd90f455a30ce8a6b1fbfbbe3b2/CHANGELOG.md?plain=1#L10

I'm happy for that change to be made in this PR, or as a separate PR - whatever you prefer :-)

@joshwlewis joshwlewis merged commit e2ec608 into main Feb 28, 2024
4 checks passed
@joshwlewis joshwlewis deleted the add-stacks-table branch February 28, 2024 00:36
@edmorley
Copy link
Member

Ah auto-merge beat me to it. I'll open a PR for the date :-)

@edmorley
Copy link
Member

Opened #213.

edmorley added a commit that referenced this pull request Feb 28, 2024
Since the original release was delayed due to the packaging step
failing (see #210).

(I've used 28th not 27th for the date, since we/the automation use
UTC dates in the changelog.)
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.

2 participants