Skip to content

Conversation

@davidh44
Copy link
Contributor

@davidh44 davidh44 commented May 8, 2025

Motivation and Context

Add migration-test Maven profile and excludev2-migration-tests in regular builds. This can reduce 20+ mins and speed up the build. A separate CB will be created to run those tests in parallel.

Previous changes for endpoint tests:
#3553 & #4860

Modifications

Testing

Tested locally:

mvn install -pl :v2-migration-tests -> tests skipped

mvn install -pl :v2-migration-tests -P migration-test -> tests ran

Codebuild jobs passed

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
  • My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

  • I confirm that this pull request can be released under the Apache 2 license

@davidh44 davidh44 requested a review from a team as a code owner May 8, 2025 17:19
@davidh44 davidh44 added the changelog-not-required Indicate changelog entry is not required for a specific PR label May 8, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 8, 2025

@davidh44 davidh44 added this pull request to the merge queue May 9, 2025
Merged via the queue into master with commit 7965c88 May 9, 2025
21 of 22 checks passed
@davidh44 davidh44 deleted the hdavidh/migration-tests-separate-build branch May 13, 2025 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-not-required Indicate changelog entry is not required for a specific PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants