Skip to content
This repository has been archived by the owner on Jul 9, 2022. It is now read-only.

Use buildpack in docs #312

Closed
wants to merge 1 commit into from

Conversation

ericbottard
Copy link
Contributor

Fixes #308

@@ -69,7 +69,7 @@ deployment to fail

[subs=attributes]
```
cf push dataflow-server -m 2G -k 2G --no-start -p spring-cloud-dataflow-server-cloudfoundry-{project-version}.jar
cf push dataflow-server -b java_buildpack_offline -m 2G -k 2G --no-start -p spring-cloud-dataflow-server-cloudfoundry-{project-version}.jar
Copy link
Contributor

Choose a reason for hiding this comment

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

cf buildpacks doesn't list java_buildpack_offline as a known buildpack. May be it is just for the CF environment I target to. Also, not sure which buildpacks get added by default for any CF environment. May be we can use the one (java_buildpack ?) that comes from default?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was one of my concern, that names are not consistent.
Tested on our GCP env.

Not sure there is one "that comes from default" actually... Maybe this is not worth the trouble after all. WDYT @markpollack ?

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe -b java_buildpack is the default value in a PCF install. Let's align with that default value for the docs.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, merging the PR with the change to use java_buildpack

@ilayaperumalg ilayaperumalg self-assigned this Apr 24, 2017
@ilayaperumalg
Copy link
Contributor

ilayaperumalg commented Apr 28, 2017

Rebased and merged as 6ddea7e

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants