Skip to content

make bundle and VERSION update - after rebase#1901

Closed
mpryc wants to merge 1 commit into
openshift:oadp-1.5from
mpryc:oadp-1.5-non-admin-manifests
Closed

make bundle and VERSION update - after rebase#1901
mpryc wants to merge 1 commit into
openshift:oadp-1.5from
mpryc:oadp-1.5-non-admin-manifests

Conversation

@mpryc
Copy link
Copy Markdown
Contributor

@mpryc mpryc commented Aug 11, 2025

Updated VERSION in the Makefile to 1.5.1
Ran: make bundle

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Aug 11, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mpryc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 11, 2025
- velero.io
resources:
- '*'
- backuprepositories
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The '*' which is one line above should cover the removals.

Comment thread config/rbac/role.yaml
- velero.io
resources:
- '*'
- backuprepositories
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The '*' above should cover all the removals.

Comment thread config/rbac/role.yaml
- apiGroups:
- ""
resources:
- persistentvolumerclaims
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

persistentvolumerclaims - I think this could have been mistake

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

persistentvolumes is something more problematic as it's not covered by other rbac privs here and I am not sure if we should give that somewhere.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those are not present in the oadp-dev branch of oadp-operator directory.

@mpryc mpryc force-pushed the oadp-1.5-non-admin-manifests branch from f5c8449 to 532a3a3 Compare August 11, 2025 19:43
@mpryc
Copy link
Copy Markdown
Contributor Author

mpryc commented Aug 12, 2025

/test unit-test

@mpryc mpryc force-pushed the oadp-1.5-non-admin-manifests branch from 532a3a3 to f61deca Compare August 12, 2025 09:50
@mpryc
Copy link
Copy Markdown
Contributor Author

mpryc commented Aug 12, 2025

The error in the unit-tests comes from the make bundle-isupdated target, but I am little confused why this happens. I've tried to reproduce the issue on my local computer with the same exact source code gh pr checkout 1901 and the make bundle-isupdated does not give me this error.

Updated VERSION in the Makefile to 1.5.1

Ran:  make bundle

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc mpryc force-pushed the oadp-1.5-non-admin-manifests branch from f61deca to 377195e Compare August 12, 2025 09:55
@mpryc mpryc changed the title make update-non-admin-manifests - after rebase make bundle and VERSION update - after rebase Aug 12, 2025
@mpryc
Copy link
Copy Markdown
Contributor Author

mpryc commented Aug 12, 2025

Note: non-admin-backup has nothing to do with the bundle manifest change.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Aug 12, 2025

@mpryc: 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/unit-test 377195e link true /test unit-test

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@mpryc
Copy link
Copy Markdown
Contributor Author

mpryc commented Aug 12, 2025

Fixed in #1903

@mpryc mpryc closed this Aug 12, 2025
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant