Skip to content

issue #7547: harden CI SNAPSHOT upload of marketplace plugin zips#7634

Draft
mattcasters wants to merge 1 commit into
apache:mainfrom
mattcasters:issue-7547-2
Draft

issue #7547: harden CI SNAPSHOT upload of marketplace plugin zips#7634
mattcasters wants to merge 1 commit into
apache:mainfrom
mattcasters:issue-7547-2

Conversation

@mattcasters

Copy link
Copy Markdown
Contributor

Summary

Hardens the existing main-branch Deploy path so marketplace-installable plugin SNAPSHOT zips are reliably published to ASF Nexus.

  • Run ./tools/verify-ci-snapshot-zips.sh before wagon:upload so the job fails if any optional-plugins.yaml artifact is missing a zip in local-snapshots-dir.
  • Pin wagon-maven-plugin to 2.0.2 and fix wagon excludes for Maven layout (**/hop-core/**/*.zip, etc.).
  • Document the CI publish flow and ASF install smoke checks in the marketplace README.

This does not invent a new publisher: Test & Build still deploys to local-snapshots-dir, and Deploy still wagon-uploads to repository.apache.org snapshots with marketplace plugin zips included.

Test plan

  • Diff review of Jenkinsfile, pom.xml deploy-snapshots profile, verify script, README
  • After merge: confirm main Deploy runs verify + wagon, then check a new unique zip under e.g. …/hop-tech-parquet/2.19.0-SNAPSHOT/ and …/hop-engines-spark/2.19.0-SNAPSHOT/
  • Optional: lean client ./hop marketplace install hop-tech-parquet against ASF primary

Related: #7547, follow-up to #7577

… wagon upload

Harden the main-branch Deploy path so optional plugin zips (spark, beam,
parquet, etc.) are present in local-snapshots-dir before wagon:upload,
pin wagon-maven-plugin, and match excludes to Maven layout under fromDir.
@mattcasters
mattcasters marked this pull request as draft July 26, 2026 21:38
@mattcasters

Copy link
Copy Markdown
Contributor Author

Not sure it's needed. It seems to work fine so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant