Skip to content

Conversation

@saschagrunert
Copy link
Member

@saschagrunert saschagrunert commented Aug 26, 2024

What type of PR is this?

/kind feature

What this PR does / why we need it:

This should allow using different SELinux contexts if multiple pods use a single image.

Inherited from 61d4a59#diff-1e0877112035840d3e2f073c693665f8daffa241a440e54a866ad7382b510ea2R1240

Which issue(s) this PR fixes:

Follow-up on #8317

Refers to:

Special notes for your reviewer:

/hold

This needs to be well tested.

Does this PR introduce a user-facing change?

None

@saschagrunert saschagrunert requested a review from mrunalp as a code owner August 26, 2024 08:59
@openshift-ci openshift-ci bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/feature Categorizes issue or PR as related to a new feature. labels Aug 26, 2024
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 26, 2024
@codecov
Copy link

codecov bot commented Aug 26, 2024

Codecov Report

Attention: Patch coverage is 10.00000% with 54 lines in your changes missing coverage. Please review.

Project coverage is 49.25%. Comparing base (44b7644) to head (9ad5c5a).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8521      +/-   ##
==========================================
- Coverage   49.35%   49.25%   -0.10%     
==========================================
  Files         153      153              
  Lines       17206    17254      +48     
==========================================
+ Hits         8492     8499       +7     
- Misses       7648     7687      +39     
- Partials     1066     1068       +2     

@saschagrunert
Copy link
Member Author

/retest

@saschagrunert saschagrunert force-pushed the image-volumes-additional-bind-mount branch 5 times, most recently from b05101a to 68f3aa3 Compare August 27, 2024 09:38
@saschagrunert
Copy link
Member Author

/retest

2 similar comments
@saschagrunert
Copy link
Member Author

/retest

@saschagrunert
Copy link
Member Author

/retest

@saschagrunert saschagrunert force-pushed the image-volumes-additional-bind-mount branch 5 times, most recently from 8092ef2 to fcdef6b Compare August 27, 2024 13:07
@saschagrunert
Copy link
Member Author

/retest
/unhold

@cri-o/cri-o-maintainers PTAL

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 27, 2024
@saschagrunert
Copy link
Member Author

/retest

@saschagrunert saschagrunert force-pushed the image-volumes-additional-bind-mount branch from fcdef6b to d791d0d Compare August 28, 2024 06:58
@saschagrunert
Copy link
Member Author

/retest

@saschagrunert
Copy link
Member Author

@cri-o/cri-o-maintainers PTAL, CI is green

This should allow using different SELinux contexts if multiple pods
use a single image.

Signed-off-by: Sascha Grunert <[email protected]>
@saschagrunert saschagrunert force-pushed the image-volumes-additional-bind-mount branch from d791d0d to 9ad5c5a Compare August 28, 2024 15:53
@kwilczynski
Copy link
Contributor

/approve
/lgtm

@kwilczynski
Copy link
Contributor

/retest

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 28, 2024
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 28, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kwilczynski, saschagrunert

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

@saschagrunert
Copy link
Member Author

/retest

1 similar comment
@saschagrunert
Copy link
Member Author

/retest

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. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants