Skip to content

Conversation

@janboll
Copy link
Collaborator

@janboll janboll commented Jan 15, 2026

AROSLSRE-367

E2E tests create a new assignment per E2E test run. These are not cleaned up. Hence add some automation that removes stale role assignments.

Special notes for your reviewer

Going to run this after E2E runs, similar to deprovision step. Will require a new key in the secret with the Kusto Infra subscription name. That will be exported as "CUSTOMER_SUBSCRIPTION" to reuse the TestContext framework for authentication. It's a bit hacky, but most obvious and cheapest to implement.

@openshift-ci openshift-ci bot requested review from bennerv and geoberle January 15, 2026 13:48
@janboll janboll force-pushed the cleanup-kusto-permissions branch from a19110c to fbdfa8c Compare January 15, 2026 15:03
E2E tests create a new assignment per E2E test run. These are not
cleaned up. Hence add some automation that removes stale role
assignments.
Copy link
Collaborator

@raelga raelga left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Jan 15, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: janboll, raelga

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-merge-bot openshift-merge-bot bot merged commit e35d92d into main Jan 15, 2026
24 checks passed
@openshift-merge-bot openshift-merge-bot bot deleted the cleanup-kusto-permissions branch January 15, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants