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

[FLINK-32563] add additionalExcludes property to add exclusions to surefire tests #26

Merged

Conversation

echauchot
Copy link

Related to what was discussed here, this PR adds an optional property to add exclusions to surefire tests (among other things for skipping archunit tests).

This also increments the minor version of org.apache.flink:flink-connector-parent and requires releasing flink-connector-parent to use in ci_utils branch.

As discussed, I have tested the integration with ci_utils there using:

  • a flink-connector-shared-utils-test clone repo and a io.github.echauchot.flink:flink-connector-parent custom artifact to be able to directly commit and install the artifact in the CI
  • a custom ci script that does the cloning and mvn install in the ci.yml github action script for testing

R: @zentol

@zentol zentol merged commit 808184a into apache:parent_pom Nov 15, 2023
2 checks passed
@echauchot
Copy link
Author

echauchot commented Nov 16, 2023

Thanks for merging @zentol. For just this pom, how is the release done ? Does it still require RC + vote or just a deploy ?

@echauchot echauchot deleted the FLINK-32563-additionalExcludes-parent_pom branch January 10, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants