Skip to content

Conversation

@hasanawad94
Copy link
Contributor

Explination of the Executor field which was added in shipwright-io/build#1956. Executor field defines the resource responsible for executing a build.

Changes

Introduces documentation and motivation for the executor field of a buildrun.

Relevant to shipwright-io/build#1956

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.

Release Notes

NONE

@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 13, 2025
@hasanawad94 hasanawad94 force-pushed the ship-0040-buildrun-executor-field branch from 969bedc to f7040ce Compare September 14, 2025 08:45
@hasanawad94 hasanawad94 changed the title feat: SHIP-0040 BuildRun Executor field feat: SHIP-0043 BuildRun Executor field Sep 14, 2025
@hasanawad94 hasanawad94 force-pushed the ship-0040-buildrun-executor-field branch from f7040ce to 39d5f19 Compare September 14, 2025 11:42
Copy link
Member

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

/approve

This captures the current state of BuildRun status. Added a comment for future reference with respect to supporting non-Tekton executors, which is out of scope for the current proposal.

@openshift-ci
Copy link

openshift-ci bot commented Nov 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan

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

The pull request process is described here

Details 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 24, 2025
Explination of the Executor field which was added in shipwright-io/build#1956.
Executor field defines the resource responsible for executing a build.

Signed-off-by: Hasan Awad <hasan.m.awad94@gmail.com>

Co-authored-by: Adam Kaplan <adam.kaplan@redhat.com>
@hasanawad94 hasanawad94 force-pushed the ship-0040-buildrun-executor-field branch from 315b859 to dff5fff Compare December 1, 2025 09:03
Copy link
Member

Choose a reason for hiding this comment

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

Note - this may get renumbered to SHIP-0044 after merge.

Copy link
Member

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

/lgtm

Approving by lazy consensus.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 12, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 2587381 into shipwright-io:main Dec 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants