chore: Remove PSP support #609
Conversation
|
retest-all |
|
/retest-all |
1 similar comment
|
/retest-all |
|
/retest-nic_operator_helm |
|
/retest-nic_operator_kind /retest-copy-right /retest-blackduck_scan /retest-image_scan |
almaslennikov
force-pushed
the
drop-psp
branch
from
a0cdb5c to
1d7150d
Compare
|
/retest-nic_operator_helm |
| @@ -608,15 +584,4 @@ spec: | |||
| provider: | |||
| name: NVIDIA | |||
| url: https://github.com/Mellanox/network-operator/ | |||
| relatedImages: | |||
There was a problem hiding this comment.
related images shouldn't be deleted
There was a problem hiding this comment.
They are actually automatically deleted by DEFAULT_CHANNEL=v23.7.0 CHANNELS=v23.7.0 VERSION=23.7.0 make bundle command
There was a problem hiding this comment.
Seems that the operator-sdk is now ignoring the relatedImages from the config/manifests/bases
It used to work in previous version...
There was a problem hiding this comment.
it stopped working in operator-sdk v1.19.1
e0ne
added
the
on hold
|
@almaslennikov can you rebase this one? |
almaslennikov
force-pushed
the
drop-psp
branch
from
1d7150d to
67e6481
Compare
Pod Security Policy was deprecated for a long time, and in k8s 1.25 it was completely dropped. To align with the newest versions, dropping the PSP from our code Signed-off-by: amaslennikov <[email protected]>
almaslennikov
force-pushed
the
drop-psp
branch
from
67e6481 to
1755e2d
Compare
Done |
Thanks! Can we remove the "hold" label? |
adrianchiris
removed
the
on hold
|
Merging per 2 LGTMs once CI pass |
|
/retest-nic_operator_helm |
1 similar comment
|
/retest-nic_operator_helm |
Pod Security Policy was deprecated for a long time, and in k8s 1.25 it was completely dropped.
To align with the newest versions, dropping the PSP from our code