Skip to content

[ST] Add systemtest with user specific configuraton for ssl.* fields for custom type of listener #11500

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

Merged
merged 5 commits into from
Jun 6, 2025

Conversation

Frawless
Copy link
Member

@Frawless Frawless commented Jun 3, 2025

Type of change

  • Enhancement / new feature

Description

This PR adds system test for #11052

It also removes some classes for certs generation that can be used from test-frame library now.

Checklist

  • Write tests
  • Make sure all tests pass

@Frawless Frawless added this to the 0.47.0 milestone Jun 3, 2025
@Frawless
Copy link
Member Author

Frawless commented Jun 3, 2025

/packit test --labels regression --env TESTS=ListenersST,CustomCaST

2 similar comments
@Frawless
Copy link
Member Author

Frawless commented Jun 3, 2025

/packit test --labels regression --env TESTS=ListenersST,CustomCaST

@Frawless
Copy link
Member Author

Frawless commented Jun 3, 2025

/packit test --labels regression --env TESTS=ListenersST,CustomCaST

@Frawless
Copy link
Member Author

Frawless commented Jun 3, 2025

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Frawless Frawless marked this pull request as ready for review June 4, 2025 07:00
@Frawless Frawless requested review from see-quick and scholzj June 4, 2025 07:00
Copy link
Member

@scholzj scholzj left a comment

Choose a reason for hiding this comment

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

The test looks good, thanks 👍

Copy link
Member

@see-quick see-quick left a comment

Choose a reason for hiding this comment

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

Looks good, just few nits.

Copy link
Member

@see-quick see-quick left a comment

Choose a reason for hiding this comment

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

LGTM just few nits, which should be applied. Good job 👍 .

Frawless added 5 commits June 6, 2025 12:03
Signed-off-by: Jakub Stejskal <[email protected]>
Signed-off-by: Jakub Stejskal <[email protected]>
@Frawless Frawless force-pushed the st-custom-listener-ssl-config branch from 212b9ab to 468d809 Compare June 6, 2025 10:03
@Frawless Frawless merged commit 4b54540 into strimzi:main Jun 6, 2025
13 checks passed
@Frawless Frawless deleted the st-custom-listener-ssl-config branch June 6, 2025 12:52
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