Skip to content

Commit b339f56

Browse files
committed
Improve fast workflow performance and logging
- Exclude root module from test discovery to prevent full builds - Add -q flag to Maven commands to reduce log verbosity - Remove Free Disk Space step that was causing hangs - Optimize workflow for true fast feedback (10-20 min vs 1.5 hrs) Signed-off-by: [email protected]
1 parent 0b96f4c commit b339f56

File tree

3 files changed

+5
-11
lines changed

3 files changed

+5
-11
lines changed

.github/scripts/test_discovery.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ def _discover_affected_modules(self, changed_files: List[str]) -> List[str]:
112112

113113
for file_path in changed_files:
114114
module = self._find_module_for_file(file_path)
115-
if module:
115+
if module and module != ".": # Exclude root module to prevent full builds
116116
modules.add(module)
117117

118118
return sorted(list(modules))

.github/workflows/main-push-fast.yml

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,6 @@ jobs:
3030
- uses: actions/checkout@v4
3131
with: { fetch-depth: 2 } # Need HEAD and HEAD~1 for single commit diff
3232

33-
- name: Free Disk Space
34-
uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1
35-
with:
36-
large-packages: false
37-
docker-images: false
38-
3933
- uses: actions/setup-java@v4
4034
with:
4135
java-version: '17'
@@ -114,10 +108,10 @@ jobs:
114108
echo "INFO: No affected modules detected - running quick verification build"
115109
echo "This could mean no Java/build files were changed, or only docs were modified"
116110
echo "Running a minimal compile check to ensure the build isn't broken"
117-
./mvnw -B -T 1C compile -DfailIfNoTests=false
111+
./mvnw -B -q -T 1C compile -DfailIfNoTests=false
118112
else
119113
echo "INFO: Running tests for affected modules: $MODS"
120-
./mvnw -B -T 1C -Pci-fast-integration-tests -DfailIfNoTests=false -pl "$MODS" -amd verify
114+
./mvnw -B -q -T 1C -Pci-fast-integration-tests -DfailIfNoTests=false -pl "$MODS" -amd verify
121115
fi
122116
123117
- name: Deploy to Artifactory (affected modules only)
@@ -128,4 +122,4 @@ jobs:
128122
run: |
129123
MODS="${{ steps.mods.outputs.modules }}"
130124
echo "INFO: Deploying affected modules to Artifactory: $MODS"
131-
./mvnw -B -s settings.xml -DfailIfNoTests=false -pl "$MODS" -amd deploy
125+
./mvnw -B -q -s settings.xml -DfailIfNoTests=false -pl "$MODS" -amd deploy

.github/workflows/maintenance-fast.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,4 +91,4 @@ jobs:
9191
echo "Check the 'Compute impacted modules' step output for debugging info"
9292
exit 1
9393
fi
94-
./mvnw -B -T 1C -Pintegration-tests -DfailIfNoTests=false -pl "$MODS" -amd verify
94+
./mvnw -B -q -T 1C -Pintegration-tests -DfailIfNoTests=false -pl "$MODS" -amd verify

0 commit comments

Comments
 (0)