4.4.3 Unshaded Release - #1152
Open
sfc-gh-alhuang wants to merge 4 commits into
Open
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1152 +/- ##
==========================================
- Coverage 59.54% 55.83% -3.71%
==========================================
Files 214 214
Lines 14616 14616
Branches 1606 1606
==========================================
- Hits 8703 8161 -542
- Misses 5257 5858 +601
+ Partials 656 597 -59 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Maven Central started returning 429 during the 4.4.3 unshaded release when resolving com.google.cloud:first-party-dependencies:3.18.0 (a transitive parent POM of com.google.cloud.opentelemetry:exporter-metrics). The existing <repository id="central"> override in the generated settings.xml does not intercept transitive parent-POM lookups, so they fell back to repo.maven.apache.org. Replace the override with a <mirror mirrorOf="*,!ossrh,!snapshot"> block so every outbound request (including transitive resolution) is rewritten to the internal Artifactory at development-maven-virtual. Apply the same block to snapshot_deploy.sh, which previously had no Artifactory config at all. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Switch the Artifactory mirror URL in deploy.sh and snapshot_deploy.sh from artifactory.int.snowflakecomputing.com (internal-CA cert) to artifactory.ci1.us-west-2.aws-dev.app.snowflake.com (publicly-trusted cert). Both URLs front the same Artifactory backend, but the latter is what the current onboarding wiki recommends and avoids cert-trust issues on runners that don't carry the Snowflake internal CA. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Apply the same Artifactory mirror fix to unshaded_deploy.sh and unshaded_snapshot_deploy.sh that was previously applied to deploy.sh and snapshot_deploy.sh. These are the scripts the MavenPushSnowpipeJavaSDKUnshaded Jenkins job actually runs, so the earlier fix had no effect on the unshaded release. Also fixes a pre-existing XML typo: the profile-based settings block used <n> instead of <name>, producing "[WARNING] Unrecognised tag: 'n'" at build time. Switching to <mirror> removes the entire profile block. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.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.
Change will not be merged. This is just for the release of unshaded version.
This PR creates the unshaded release for version 4.4.3
Changes: