Skip to content

Deploy only the openstack-cloud HPI during CD - #484

Merged
krisstern merged 2 commits into
masterfrom
chore/fix-deployment-issues
Sep 10, 2026
Merged

Deploy only the openstack-cloud HPI during CD#484
krisstern merged 2 commits into
masterfrom
chore/fix-deployment-issues

Conversation

@krisstern

@krisstern krisstern commented Sep 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Skip Artifactory deploy for the aggregator (openstack-cloud-parent) and ATH (ui-tests) modules.
  • CD still uploads org.jenkins-ci.plugins:openstack-cloud only, which matches the plugin path RPU already authorizes.
  • This removes the parent/plugin name mismatch and the ui-tests 403. maven-deploy-plugin 3.x deploys all remaining artifacts from the last reactor module, so skipping those extra GAVs is required.

Do not re-run CD on 1488.va_d8f3ccf1a_02 — that version already exists in releases (parent POM + plugin HPI). This commit produces a new incrementals version.

krisstern and others added 2 commits September 11, 2026 06:35
Only the openstack-cloud HPI should be uploaded to Artifactory so CD
stops 403ing on openstack-cloud-parent and ui-tests.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved blocking issues were identified.

Pull request overview

Updates Maven deployment configuration so CD publishes only the openstack-cloud HPI.

Changes:

  • Skips deployment for the root aggregator.
  • Skips deployment for ui-tests.
  • Enables deployment for the plugin module.
File summaries
File Description
ui-tests/pom.xml Skips ATH deployment.
pom.xml Skips aggregator deployment.
plugin/pom.xml Enables plugin HPI deployment.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@krisstern
krisstern merged commit 5116ebf into master Sep 10, 2026
15 of 17 checks passed
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.

2 participants