Skip to content

[v1.18] Add NamespaceManagement/status to operator Role YAML, and add e2e tests for namespaceManagement (#948)#961

Merged
jgwest merged 1 commit into
redhat-developer:v1.18from
jgwest:cherry-pick-948-sept-2025
Sep 9, 2025
Merged

[v1.18] Add NamespaceManagement/status to operator Role YAML, and add e2e tests for namespaceManagement (#948)#961
jgwest merged 1 commit into
redhat-developer:v1.18from
jgwest:cherry-pick-948-sept-2025

Conversation

@jgwest

@jgwest jgwest commented Sep 9, 2025

Copy link
Copy Markdown
Member

What type of PR is this?

/kind bug

What does this PR do / why we need it:

  • Cherry pick of https://github.com/redhat-developer/gitops-operator/pull/948/files
  • Only substantive change is updating kubebuilder tag in controllers/gitopsservice_controller.go, which causes operator RBAC to be updated
  • Updates ClusterRole used by operator to include NamespaceManagement/status, which was missing.
  • Also includes E2E tests

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

* Add e2e tests for namespaceManagement

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>

* Add namespacemanagement/status to RBAC, and tweak test contents

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>

* Fix merge conflicts

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>

* Add remaining namespaceManagement tests

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>

* Add additional Consistently checks

Signed-off-by: Jonathan West <jonwest@redhat.com>

* Add a new test and address review comments

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>

---------

Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
Signed-off-by: Jonathan West <jonwest@redhat.com>
Co-authored-by: Jonathan West <jonwest@redhat.com>
@openshift-ci openshift-ci Bot added the kind/bug Something isn't working label Sep 9, 2025
@openshift-ci

openshift-ci Bot commented Sep 9, 2025

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign svghadi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@jgwest jgwest changed the title Cherrypick to 1.18: Add e2e tests for namespaceManagement (#948) [v1.18] Add e2e tests for namespaceManagement (#948) Sep 9, 2025
@jgwest jgwest changed the title [v1.18] Add e2e tests for namespaceManagement (#948) [v1.18] Add NamespaceManagement/status to operator Role, and add e2e tests for namespaceManagement (#948) Sep 9, 2025
@jgwest jgwest changed the title [v1.18] Add NamespaceManagement/status to operator Role, and add e2e tests for namespaceManagement (#948) [v1.18] Add NamespaceManagement/status to operator Role YAML, and add e2e tests for namespaceManagement (#948) Sep 9, 2025
@Rizwana777

Copy link
Copy Markdown
Collaborator

LGTM, thank you @jgwest

@jgwest jgwest merged commit 7e1761f into redhat-developer:v1.18 Sep 9, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants