Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Signed-off-by: Avinal Kumar <avinal@redhat.com>
9e1f0da to
278dd59
Compare
|
I am concerned a little bit on exposing volumes generically to Build users. A Build user should not know the file system structure of any of the step's container images. Shipwright usually is opinionated/specific on capabilities. Trusted certificates would imo be a good fit for that and would therefore be an API extension. Interesting questions would then be:
|
|
Potentially replaced by #281 |
Changes
Propose extending build volumes for Build and BuildRuns
Refer #258
Submitter Checklist
See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
For pull requests that don't need to be mentioned at release time, use the
/release-note-noneProw command to add therelease-note-nonelabel to the PR. You can also write the string "NONE" as a release note in your PR description: