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

[systemtest] Fix routes tests in ListenersST and add route tag #8138

Merged
merged 3 commits into from
Feb 23, 2023

Conversation

im-konge
Copy link
Member

Type of change

  • Bugfix
  • Enhancement

Description

This PR fixes failing tests for routes in ListenersST, which are failing after #7526.
Also, as part of this, I'm adding tag for route tests.

Checklist

  • Make sure all tests pass

Signed-off-by: Lukas Kral <[email protected]>
@im-konge im-konge added this to the 0.34.0 milestone Feb 21, 2023
@im-konge im-konge self-assigned this Feb 21, 2023
@im-konge im-konge changed the title [systemtest] Fix Routes tests in ListenersST and add route tag [systemtest] Fix routes tests in ListenersST and add route tag Feb 21, 2023
@im-konge
Copy link
Member Author

@strimzi-ci run tests test_only groups=route

@strimzi-ci
Copy link

▶️ Build started - check Jenkins for more info. ▶️

@strimzi-ci
Copy link

Build FAILURE - check Jenkins for more info. ❗

@im-konge
Copy link
Member Author

@strimzi-ci run tests test_only groups=route

@strimzi-ci
Copy link

▶️ Build started - check Jenkins for more info. ▶️

@strimzi-ci
Copy link

Build FAILURE - check Jenkins for more info. ❗

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

@strimzi-ci run tests test_only groups=route

@strimzi-ci
Copy link

▶️ Build started - check Jenkins for more info. ▶️

@strimzi-ci
Copy link

Build SUCCESS - check Jenkins for more info. ✅

@im-konge im-konge merged commit 303d2a1 into strimzi:main Feb 23, 2023
@im-konge im-konge deleted the fix-route-tests branch February 23, 2023 08:18
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