-
Notifications
You must be signed in to change notification settings - Fork 356
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
[RFC0028] Enable using protected CNBs #3855
[RFC0028] Enable using protected CNBs #3855
Conversation
000999b
to
035baf5
Compare
035baf5
to
9ab08c6
Compare
a45ce11
to
4e3382a
Compare
a3d1a1d
to
33625e7
Compare
Co-authored-by: Johannes Dillmann <[email protected]> Co-authored-by: Ralf Pannemans <[email protected]> Co-authored-by: Nicolas Bender <[email protected]>
Co-authored-by: Pavel Busko <[email protected]> Co-authored-by: Nicolas Bender <[email protected]>
33625e7
to
22696f4
Compare
{ | ||
name: 'some-name', | ||
relationships: { space: { data: { guid: space.guid } } }, | ||
lifecycle: { type: 'cnb', data: { buildpacks: ['http:https://buildpack.com'], credentials: { registry: { user: 'password' } } } } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lifecycle: { type: 'cnb', data: { buildpacks: ['http:https://buildpack.com'], credentials: { registry: { user: 'password' } } } } | |
lifecycle: { type: 'cnb', data: { buildpacks: ['http:https://example.com'], credentials: { registry: { user: 'password' } } } } |
lifecycle_data = response_body['lifecycle']['data'] | ||
|
||
expect(response).to have_http_status :created | ||
expect(lifecycle_data).to eq({ 'buildpacks' => ['http:https://buildpack.com'], 'stack' => 'default-stack-name', 'credentials' => '***' }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expect(lifecycle_data).to eq({ 'buildpacks' => ['http:https://buildpack.com'], 'stack' => 'default-stack-name', 'credentials' => '***' }) | |
expect(lifecycle_data).to eq({ 'buildpacks' => ['http:https://example.com'], 'stack' => 'default-stack-name', 'credentials' => '***' }) |
@@ -1,5 +1,5 @@ | |||
require 'spec_helper' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't that file be located at spec/unit/lib/cloud_controller/diego/buildpack/lifecycle_data_spec.rb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point, the file has been moved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, just found one minor thing.
I'm a bit unsure if we should have two lifecycle_data.rb
classes but also don't have a better idea at the moment
Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:
Building on #3778, this PR introduces the ability to use the
cnb
lifecycle with password protected Cloud Native Buildpacks.An explanation of the use cases your change solves
Links to any other associated PRs
I have reviewed the contributing guide
I have viewed, signed, and submitted the Contributor License Agreement
I have made this pull request to the
main
branchI have run all the unit tests using
bundle exec rake
I have run CF Acceptance Tests