Ignore _merged_spec.yaml when computing build cache key #18105
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
The build has some cache misses when run twice without any change (on a fresh clone). The cost here is 10s of wall clock time:
The problem is that the file
modules/openapi-generator-maven-plugin/src/test/resources/default/_merged_spec.yaml
generated by themaven-plugin
module tests is located in thesrc/test/resources
folder which is used as goal input bycompiler:testCompile
andsurefire:test
from themaven-plugin
module.This means that the cache key is different, therefore the cache misses.
Steps to reproduce
Clone the project in a new location
Fix
The best solution would be to generate the
_merged_spec.yaml
in thetarget
directory, however I couldn't find a way to configure this.The approach taken is to ignore the file while computing the cache key with help of RuntimeNormalization.
PR checklist
master
(upcoming 7.1.0 minor release - breaking changes with fallbacks),8.0.x
(breaking changes without fallbacks)