knative / operator

Combined operator for Knative.
Apache License 2.0
181 stars 98 forks source link

Move to Security-Guard 0.6 #1604

Closed davidhadas closed 9 months ago

davidhadas commented 9 months ago

Moved to Security-Guard 0.6

knative-prow[bot] commented 9 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: davidhadas

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: - ~~[OWNERS](https://github.com/knative/operator/blob/main/OWNERS)~~ [davidhadas] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
knative-prow[bot] commented 9 months ago

@davidhadas: 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
unit-tests_operator_main 037a15331e501d4870dfdde779c3a8259f51cbc4 link true /test unit-tests

Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
houshengbo commented 9 months ago

run ./hack/update-codegen.sh, to update the manifests for security guard.

davidhadas commented 9 months ago

./hack/update-codegen.sh

hi @houshengbo, I already did - what may be the reason that ./hack/update-codegen.sh does not pick up 0.6?

davidhadas commented 9 months ago

/hold

davidhadas commented 9 months ago

/unhold 0.6.1 is released

davidhadas commented 9 months ago

1611 seem to resolve this.

Closing..