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

Add a Salesforce Functions builder variant #392

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Sep 25, 2023

We want to move Salesforce Functions buildpacks to their own builder image variant to more cleanly separate the two use cases (particularly given that Salesforce Functions is now deprecated), and prevent a number of issues (such as builds on the functions build system being able to pass detection even when the match against a non-functions buildpack).

For more details, see:
https://salesforce.quip.com/MoDFAX3MEf8b

The first stage of this is to create the new builder image variant.

This new variant is only being published to the internal private registry, since it's only being used by Kodon (the SF CLI no longer supports using containers to run functions) so doesn't need to be available on Docker Hub.

Once it's released, we can then switch Kodon to using the new tag, deploy Kodon to all functions spaces, and then finally remove the functions buildpacks from heroku/builder:22.

We'll also need to adjust the CNB release automation so that it updates the new image variant too:
https://github.com/heroku/languages-github-actions/blob/8f1d3319996df5ec9eb3b4c9c7c0d5de78fa9dba/.github/workflows/_buildpacks-release.yml#L315

GUS-W-14179967.

@edmorley edmorley self-assigned this Sep 25, 2023
@edmorley edmorley force-pushed the edmorley/salesforce-functions-variant branch 2 times, most recently from 93dcbd3 to c57e7c6 Compare September 25, 2023 19:55
@edmorley edmorley marked this pull request as ready for review September 25, 2023 20:04
@edmorley edmorley requested a review from a team as a code owner September 25, 2023 20:04
@edmorley
Copy link
Member Author

Note to self: When this is ready to be merged, also adjust the required status checks in the branch protection settings.

@edmorley edmorley force-pushed the edmorley/salesforce-functions-variant branch from c57e7c6 to 5cb210f Compare September 25, 2023 20:12
@edmorley edmorley changed the title Add a Salesforce Functions image variant Add a Salesforce Functions builder variant Sep 25, 2023
@edmorley edmorley force-pushed the edmorley/salesforce-functions-variant branch 2 times, most recently from fc6611f to d672692 Compare September 25, 2023 20:47
We want to move Salesforce Functions buildpacks to their own builder
image variant to more cleanly separate the two use cases (particularly
given that Salesforce Functions is now deprecated), and prevent a number
of issues (such as builds on the functions build system being able to
pass detection even when the match against a non-functions buildpack).

For more details, see:
https://salesforce.quip.com/MoDFAX3MEf8b

The first stage of this is to create the new builder image variant.

This new variant is only being published to the internal private
registry, since it's only being used by Kodon (the SF CLI no longer
supports using containers to run functions).

Once it's released, we can then switch Kodon to using the new tag,
deploy Kodon to all functions instances, and then finally remove the
functions buildpacks from `heroku/builder:22`.

We'll also need to adjust the CNB release automation so that it
updates the new image variant too:
https://github.com/heroku/languages-github-actions/blob/8f1d3319996df5ec9eb3b4c9c7c0d5de78fa9dba/.github/workflows/_buildpacks-release.yml#L315

GUS-W-14179967.
@edmorley edmorley force-pushed the edmorley/salesforce-functions-variant branch from d672692 to 385deaf Compare September 26, 2023 09:32
@edmorley edmorley enabled auto-merge (squash) September 26, 2023 09:34
@edmorley
Copy link
Member Author

Branch protection list of required CI checks updated.

@edmorley edmorley merged commit 0d868f7 into main Sep 26, 2023
39 checks passed
@edmorley edmorley deleted the edmorley/salesforce-functions-variant branch September 26, 2023 09:38
edmorley added a commit to heroku/languages-github-actions that referenced this pull request Sep 26, 2023
edmorley added a commit that referenced this pull request Oct 2, 2023
A new functions only builder image variant was added in #392 (see that
PR for why), and now the functions Kodon instances have been switched
to use this new image in:
heroku/kodon#408

As such, the functions buildpacks in `heroku/builder:22` are now unused
and can be removed.

In addition, the functions examples have been moved under the functions
builder's directory, since its now the only builder using them.

GUS-W-14194543.
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.

2 participants