Closed razo7 closed 8 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: razo7
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test 4.14-openshift-e2e /test 4.15-openshift-e2e
Following our discussion this PR should be the implementation of the Second option (also pasting here the relevant part).
- At the moment there is no validation for fence agent name so this allow a security breach for the user to execute unwanted commands.
- There are several options on how to address this:
- First (simple) option: is a hard coded list of fence agents that will be used for validation
- Second (advanced) option: retrieving the fence agents name during runtime and use that data for validation.
- The main pro for using the Second option that it'll support different setups of fence agents (mainly keeping in mind upstream / downstream differences ) where the con is that it is more complicated.
[Edit] Following our conversation I see that the fence_agent file is being populated from /usr/sbin folder during containter build phase here. I would still prefer an in memory list of the fence agents over a file, but I understand that the file implementation is simpler so it's not important enough to be a blocker IMO.
/test 4.14-openshift-e2e /test 4.15-openshift-e2e
/test 4.14-openshift-e2e
/test 4.14-openshift-e2e /test 4.15-openshift-e2e
/lgtm
/retest
/retest
/hold We might want to ~to~ do this check in Webhook
@razo7: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/4.12-openshift-e2e | 4b815edeec4d4ab9596d163c03431bcd3bb41e42 | link | true | /test 4.12-openshift-e2e |
Full PR test history. Your PR dashboard.
PR needs rebase.
close in favor of #119
Verify and restrict far CRs to specific (and available) agents that exist in the container/pod.
ECOPROJECT-1753