Skip to content

SHIP-0042: Extending Build Volumes #265

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

avinal
Copy link

@avinal avinal commented May 8, 2025

Changes

Propose extending build volumes for Build and BuildRuns
Refer #258

Submitter Checklist

  • Includes tests if functionality changed/was added
  • Includes docs if changes are user-facing
  • Set a kind label on this PR
  • Release notes block has been filled in, or marked NONE

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-none Prow command to add the release-note-none label to the PR. You can also write the string "NONE" as a release note in your PR description:

@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 8, 2025
@openshift-ci openshift-ci bot requested review from HeavyWombat and qu1queee May 8, 2025 05:17
Copy link

openshift-ci bot commented May 8, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign saschaschwarze0 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@adambkaplan
Copy link
Member

@avinal can you please re-number the SHIP to 0042? We have #263 as "reserved" for SHIP-0040

Signed-off-by: Avinal Kumar <avinal@redhat.com>
@avinal avinal force-pushed the avinal/extending-volume branch from 9e1f0da to 278dd59 Compare May 22, 2025 08:20
@avinal avinal changed the title SHIP-0040: Extending Build Volumes SHIP-0042: Extending Build Volumes May 22, 2025
@qu1queee qu1queee requested a review from SaschaSchwarze0 May 26, 2025 06:41
@SaschaSchwarze0
Copy link
Member

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:

  • How would the API extension look like = do we make generic additional certs somewhere under .spec or specific places under .spec.source and .spec.output ? I tend to use the generic place because then those could also be made available to the strategy steps and not just our system steps.
  • For our own steps, we will be able to mount them and consume them.
  • We should think about how this works for strategy steps. Can we mount it just somewhere and all distros will magically pick it up? Or do we need to add some new system parameters that point to the directory with the custom certs in there (like we have other system parameters: https://shipwright.io/docs/build/buildstrategies/#system-parameters).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants