Skip to content
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

[ST] Add CC API Users test #10473

Merged
merged 3 commits into from
Aug 21, 2024
Merged

[ST] Add CC API Users test #10473

merged 3 commits into from
Aug 21, 2024

Conversation

im-konge
Copy link
Member

@im-konge im-konge commented Aug 20, 2024

Type of change

  • New test

Description

This PR adds test for the new feature - CC API users. It creates API user with USER permissions (that has GET permissions to almost all endpoints) and verifies that we can check the /state endpoint without need to disable security.

Checklist

  • Write tests
  • Make sure all tests pass

Signed-off-by: Lukas Kral <[email protected]>
@im-konge im-konge added this to the 0.44.0 milestone Aug 20, 2024
@im-konge im-konge requested review from kyguy and a team August 20, 2024 09:38
@im-konge im-konge self-assigned this Aug 20, 2024
Signed-off-by: Lukas Kral <[email protected]>
Copy link
Contributor

@henryZrncik henryZrncik left a comment

Choose a reason for hiding this comment

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

Good addition great Username! Would it be possible to add some simple doc for the testCruiseControlAPIUsers test ?

Signed-off-by: Lukas Kral <[email protected]>
@im-konge
Copy link
Member Author

Would it be possible to add some simple doc for the testCruiseControlAPIUsers test ?

Good point, totally forgot to add it 😄 thanks!

@im-konge
Copy link
Member Author

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@im-konge im-konge merged commit 5ea2119 into strimzi:main Aug 21, 2024
21 checks passed
@im-konge im-konge deleted the cc-api-users-test branch August 21, 2024 08:23
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.

4 participants