Fix main CI: JFrog mirror bypass and Windows pwsh parse error#1428
Merged
vikrantpuppala merged 4 commits intomainfrom Apr 28, 2026
Merged
Fix main CI: JFrog mirror bypass and Windows pwsh parse error#1428vikrantpuppala merged 4 commits intomainfrom
vikrantpuppala merged 4 commits intomainfrom
Conversation
Three CI workflows on main were failing: 1. runIntegrationTests.yml and bugCatcher.yml were caching ~/.m2, which restored a stale settings.xml on top of the JFrog-configured one written by the preceding Configure maven step. Maven then bypassed the mirror and tried to hit Maven Central directly, which the protected runner cannot reach. Narrow the cache path to ~/.m2/repository so settings.xml is preserved. 2. loggingTesting.yml's Get JFrog OIDC token and Configure maven steps used bash syntax (if [ ... ], heredoc) but had no shell directive. On Windows runners the default shell is pwsh, which fails to parse the bash if statement. Pin both steps to shell: bash. Co-authored-by: Isaac Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>
samikshya-db
approved these changes
Apr 28, 2026
Collaborator
Author
|
Recreating with rebased commit and bugCatcher.yml change dropped — see new PR. |
bugCatcher only runs Mondays 00:00 UTC and is not blocking main CI. Dropping the cache-path fix here keeps this PR focused on the two workflows that are actually broken (runIntegrationTests, loggingTesting). The same fix can be applied to bugCatcher in a follow-up if desired. Co-authored-by: Isaac Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>
2 tasks
bugCatcher hits the same stale-cache bug as runIntegrationTests — its ~/.m2 cache restore overwrites the JFrog-configured settings.xml, making Maven bypass the mirror. Same one-line fix. Co-authored-by: Isaac Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
Three CI workflows on
mainhave been failing. This PR fixes the root causes.1.
Integration Tests Workflow - Main Branch(failing 30+ runs in a row since Apr 8)path: ~/.m2with a long-lived restore-key${{ runner.os }}-m2. It restores a stale cache from before the JFrog OIDC migration, whose~/.m2/settings.xml(the github-server one written byactions/setup-java) overwrites the JFrog mirror config that the preceding "Configure maven" step just wrote.repo.maven.apache.orgdirectly, which the protected runner cannot reach:Could not transfer artifact ... from/to central (https://repo.maven.apache.org/maven2): Remote host terminated the handshake.pathto~/.m2/repositorysosettings.xmlis left alone (matches the pattern used bywarmMavenCache.yml).2.
Weekly bug catcher— same root cause as #1Same fix in
bugCatcher.yml.3.
Test JDBC Logging(Windows jobs failing since Apr 27)if [ -z "$X" ],set -euo pipefail) but has noshell:directive. On Windows runners the default shell ispwsh, which fails parsing the bashifwithMissing '(' after 'if'. The "Configure maven" step has the same problem (bash heredoc).shell: bash. Linux jobs were unaffected because their default shell is already bash.Test plan
Integration Tests Workflow - Main Branchpasses on the next push tomainTest JDBC LoggingWindows jobs pass on the next push tomainWeekly bug catcherrun (Mondays 00:00 UTC) is greenNO_CHANGELOG=true
OVERRIDE_FREEZE=true
This pull request and its description were written by Isaac.