Skip to content
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

chore: insert OwnerReferences param to pipelines #452

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

ksimon1
Copy link
Member

@ksimon1 ksimon1 commented Jul 9, 2024

What this PR does / why we need it:
chore: insert OwnerReferences param to pipelines

sometimes the pipeline can fail and cleanup tasks are not triggered. By inserting the ownerReferences to created objects, when the pipelineRun fails, they can still be deleted by deleting the pipelineRun

Release note:

chore: insert OwnerReferences param to pipelines

@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Jul 9, 2024
@kubevirt-bot kubevirt-bot requested a review from lyarwood July 9, 2024 16:21
@openshift-ci openshift-ci bot requested a review from jcanocan July 9, 2024 16:21
@kubevirt-bot kubevirt-bot requested a review from opokornyy July 9, 2024 16:21
@openshift-ci openshift-ci bot requested a review from lyarwood July 9, 2024 16:21
Copy link

openshift-ci bot commented Jul 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ksimon1

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

The pull request process is described 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 9, 2024
Copy link
Contributor

@jcanocan jcanocan left a comment

Choose a reason for hiding this comment

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

Nice job @ksimon1! Just a couple of suggestions, feel free to ignore them or part of them if you don't fully agree.

release/pipelines/windows-efi-installer/README.md Outdated Show resolved Hide resolved
templates-pipelines/windows-customize/README.md Outdated Show resolved Hide resolved
sometimes the pipeline can fail and cleanup tasks are
not triggered. By inserting the ownerReferences to created
objects, when the pipelineRun fails, they can still be deleted
by deleting the pipelineRun

Signed-off-by: Karel Simon <[email protected]>
@ksimon1 ksimon1 force-pushed the pipelines-references branch from e0e1645 to 85c00a5 Compare July 10, 2024 11:45
@jcanocan
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 10, 2024
@ksimon1
Copy link
Member Author

ksimon1 commented Jul 10, 2024

/retest

@ksimon1 ksimon1 merged commit f3b74f5 into kubevirt:main Jul 10, 2024
10 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. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants