-
Notifications
You must be signed in to change notification settings - Fork 200
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
bump pipelines to 0.47, pac to 0.19, and fix breaking changes #1481
Conversation
/cc @vdemeester |
The following is the coverage report on the affected files.
|
829722b
to
71e062a
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: piyush-garg 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 |
The following is the coverage report on the affected files.
|
Signed-off-by: Jeeva Kandasamy <jkandasa@redhat.com>
71e062a
to
636bdbb
Compare
/test all |
The following is the coverage report on the affected files.
|
/lgtm |
Changes
bump PAC dependency to 0.19.0 (from v0.17.2)
bump pipelines dependency to 0.47.0 (from 0.45.0) and addresses breaking changes
trusted-resources-verification-no-match-policy
and removesresource-verification-mode
under tekton pipelinesRemoved
verification-mode
Actually we have to deprecate
resource-verification-mode
in favor oftrusted-resources-verification-no-match-policy
, But I removed it completely. why?resource-verification-mode
andenable-provenance-in-status
fields for Pipelines #1194resource-verification-mode
resource-verification-mode
, but implemented asverification-mode
referverification-mode
field never used in pipelines, hence I removed the field completely.Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
make test lint
before submitting a PRSee the contribution guide for more details.
Release Notes