Skip to content
This repository has been archived by the owner on Mar 13, 2021. It is now read-only.

Add a builder for applications #52

Merged
merged 3 commits into from
May 20, 2019

Conversation

scothis
Copy link
Member

@scothis scothis commented May 16, 2019

Applications are built using the Cloud Foundery CNB builder,
cloudfoundry/cnb:bionic.

This is the same structure as the function build template, without function specific args and an application builder image. It is used to build Application resources in riff-system.

Applications are built using the Cloud Foundery CNB builder,
cloudfoundry/cnb:bionic.
apiVersion: build.knative.dev/v1alpha1
kind: ClusterBuildTemplate
metadata:
name: cf-cnb

Choose a reason for hiding this comment

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

I think we should we call this riff-application-cnb and then rename the current riff-cnb to riff-function-cnb.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll rename cf-cnb-* to riff-app-cnb*. I don't want to change riff-cnb-* since that would be a breaking change with the current riff cli.

Copy link
Member Author

Choose a reason for hiding this comment

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

bumped the version and renamed the clusterbuildtemplates to riff-application and riff-function

as gs://projectriff/riff-buildtemplate/riff-app-cnb-clusterbuildtemplate.yaml
@scothis scothis force-pushed the application-builder branch from 6849ead to c9364e9 Compare May 20, 2019 21:26
Rename build templates to:
- riff-application-clusterbuildtemplate
- riff-function-clusterbuildtemplate
@scothis scothis merged commit 3bc61aa into projectriff:master May 20, 2019
@scothis scothis deleted the application-builder branch May 20, 2019 21:47
@nebhale nebhale added semver:minor A change requiring a minor version bump type:enhancement A general enhancement labels Oct 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
semver:minor A change requiring a minor version bump type:enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants