Skip to content

chore(e2e): introduce e2e tests for RBAC Auditor log #2759

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 13 commits into from
Closed

chore(e2e): introduce e2e tests for RBAC Auditor log #2759

wants to merge 13 commits into from

Conversation

dzemanov
Copy link
Member

Description

Introduces tests for Auditor service in RBAC plugin.

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@openshift-ci openshift-ci bot requested review from josephca and psrna April 14, 2025 07:49
@dzemanov dzemanov changed the title chore(e2e): Introduce e2e tests for RBAC Auditor log chore(e2e): introduce e2e tests for RBAC Auditor log Apr 14, 2025
@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov dzemanov mentioned this pull request Apr 14, 2025
Open
5 tasks
@PatAKnight
Copy link
Member

/retest

1 similar comment
@PatAKnight
Copy link
Member

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov dzemanov force-pushed the RHIDP-6618-e2e-auditor branch from e1a02e6 to bcfcd49 Compare April 15, 2025 14:21
@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov
Copy link
Member Author

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov
Copy link
Member Author

/retest

4 similar comments
@dzemanov
Copy link
Member Author

/retest

@dzemanov
Copy link
Member Author

/retest

@dzemanov
Copy link
Member Author

/retest

@dzemanov
Copy link
Member Author

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov
Copy link
Member Author

/retest

dzemanov added 7 commits April 29, 2025 11:03
@dzemanov
Introduce rbac auditor tests
34db7d2 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Test
1028c0c 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Catch the logs
87cb03b 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Test pod logs
3d4e6c6 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Enable change of namespace
964100e 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Update pod selector
e42e006 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Filter the logs more precisely
5a6a24d 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Switch to json logs
bf14943 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov dzemanov force-pushed the RHIDP-6618-e2e-auditor branch from 5da0996 to bf14943 Compare April 29, 2025 09:59
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Apr 29, 2025

[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 josephca for approval. For more information see the Code Review Process.

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

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

@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

dzemanov added 2 commits April 29, 2025 13:21
@dzemanov
Fix policy-read by-query
c9a40d3 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Enable more precise filtering
35c104f 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

dzemanov added 2 commits April 29, 2025 15:13
@dzemanov
Make permission-evaluation more concrete
fb613c4 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@dzemanov
Remove testing lines
d7c5cee 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov
Copy link
Member Author

/retest

@dzemanov
Increase tail number
333a1bc 
Signed-off-by: Dominika Zemanovicova <[email protected]>
@github-actions GitHub Actions
Copy link
Contributor

The image is available at:

@dzemanov
Copy link
Member Author

dzemanov commented May 2, 2025

Closing in favor of #2827

@dzemanov dzemanov closed this May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@josephca josephca Awaiting requested review from josephca

@psrna psrna Awaiting requested review from psrna

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dzemanov @PatAKnight @openshift-merge-robot