-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[CI] Change the required checks to match refactored Pulsar CI #14944
Conversation
Since the required checks have already been changed, it's not possible to revert the changes in |
c2fc9e4
to
4fb17ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/pulsarbot rerun-failure-checks |
- Python client is required for Functions and Schema tests and tests were needed to move to a different category - Sql integration tests ("CI - System - Sql") are disabled until apache#14951 has been resolved
4fb17ac
to
daa105c
Compare
Removing the workaround must happen in a separate PR after |
- Python client is required for Functions and Schema tests and tests were needed to move to a different category - Sql integration tests ("CI - System - Sql") are disabled until apache#14951 has been resolved
Motivation
revisit [CI] Change required checks to match checks in refactored Pulsar CI workflow #14939 changes for [CI] Refactor GitHub Actions based Pulsar CI #14819
Python client is required for Functions and Schema tests and
tests were needed to move to a different category
Sql integration tests ("CI - System - Sql") are disabled until
Pulsar SQL won't start with Java 11.0.14.1 #14951 has been resolved
Additional context:
Modifications
rename
CI - Integration - Function
toCI - System - Function
andCI - Integration - Schema
toCI - System - Schema
remove Sql integration tests ("CI - System - Sql") from required checks