Skip to content

fix: migrate Orchestrator plugins to ref:// references (RHIDP-16970) - #325

Closed
Fortune-Ndlovu wants to merge 1 commit into
redhat-developer:mainfrom
Fortune-Ndlovu:fix/RHIDP-16970-orchestrator-ref-prefix
Closed

Fortune-Ndlovu wants to merge 1 commit into
redhat-developer:mainfrom
Fortune-Ndlovu:fix/RHIDP-16970-orchestrator-ref-prefix

Conversation

@Fortune-Ndlovu

@Fortune-Ndlovu Fortune-Ndlovu commented Sep 16, 2026

Copy link
Copy Markdown
Member

Summary

  • Migrate Orchestrator dynamic plugin configuration from registry-specific oci://...:{{inherit}} URLs to stable ref:// plugin names
  • Update orchestrator README example to match

Related

Use stable ref:// plugin names instead of registry-specific oci://
URLs with {{inherit}} in orchestrator local setup and docs.

RHIDP-16970

Signed-off-by: Fortune Ndlovu <fndlovu@redhat.com>
@github-actions

Copy link
Copy Markdown

Warning

This PR targets main. This repository uses a main/dev branching model.

  • All development work (features, docs, dependency updates) should target dev.
  • Bug fixes for a supported release should target release-x.y.
  • Only maintainer-managed merges and cherry-picks should go into main.

If this is intentional (e.g., a backport or maintainer merge), you can ignore this message.
Otherwise, please retarget your PR to dev.

See the branching strategy for details.

@sonarqubecloud

Copy link
Copy Markdown

@rhdh-qodo-merge

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Migrate Orchestrator plugins to stable ref:// references

🐞 Bug fix ⚙️ Configuration changes 📝 Documentation 🕐 Less than 10 minutes

Grey Divider

AI Description

• Replace registry-specific Orchestrator plugin URLs with stable ref:// package references.
• Align setup documentation with the corrected dynamic plugin configuration.
Diagram

graph TD
  Docs["README Guidance"] --> Config["Plugin Config"] --> Installer["Plugin Installer"] --> Resolver["Reference Resolver"] --> Frontend["Frontend Plugins"]
  Resolver --> Backend["Backend Plugins"]
Loading
High-Level Assessment

The PR uses the intended stable reference abstraction and keeps runtime configuration and documentation synchronized. Retaining direct OCI URLs was considered but would preserve registry coupling and {{inherit}} tag dependence, so the current approach is preferable.

Files changed (2) +8 / -8

Bug fix (1) +4 / -4
dynamic-plugins.yamlConfigure Orchestrator plugins through ref:// identifiers +4/-4

Configure Orchestrator plugins through ref:// identifiers

• Migrates the Orchestrator frontend, backend, scaffolder module, and form-widget packages from explicit registry URLs to stable references. Existing enablement and Data Index configuration remain unchanged.

orchestrator/configs/dynamic-plugins/dynamic-plugins.yaml

Documentation (1) +4 / -4
README.mdDocument stable Orchestrator plugin references +4/-4

Document stable Orchestrator plugin references

• Replaces all four registry-specific OCI package examples with stable 'ref://' plugin names. The documented setup now matches the runtime configuration.

orchestrator/README.md

@rhdh-qodo-merge

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can group findings by type and pick your Finding display, from Minimal to Full

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@rhdh-qodo-merge rhdh-qodo-merge Bot added documentation Improvements or additions to documentation Bug fix labels Sep 16, 2026
@rhdh-qodo-merge

Copy link
Copy Markdown
Contributor

Important

The /generate_labels command by Qodo is sunsetting on the 1st of October 2026 and will no longer be available. We recommend switching to the latest Qodo review capabilities. Learn more

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR should target the dev branch instead: #325 (comment)

@Fortune-Ndlovu

Copy link
Copy Markdown
Member Author

Closing this PR — the ref:// orchestrator migration for rhdh-local is already merged on dev in #304.

This PR targeted main, which uses the pinned rhdh:1.10.3 image. That version does not resolve ref:// references during plugin install, which is why CI failed with:

InstallException: No integrity hash provided for Package ref://red-hat-developer-hub-backstage-plugin-orchestrator

Per the branching model, development work targets dev (which uses :next images with ref:// support). Changes flow to main at GA via release branches.

RHIDP-16970 rhdh-local scope is covered by #304 on dev.

Signed-off-by: Fortune Ndlovu fndlovu@redhat.com

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

Labels

Bug fix documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants