Skip to content

fix: resolve metadata validation failures#1915

Merged
dzemanov merged 11 commits intoredhat-developer:mainfrom
polasudo:fix/metadata-validation-main
Feb 5, 2026
Merged

fix: resolve metadata validation failures#1915
dzemanov merged 11 commits intoredhat-developer:mainfrom
polasudo:fix/metadata-validation-main

Conversation

@polasudo
Copy link
Member

@polasudo polasudo commented Feb 3, 2026

https://issues.redhat.com/browse/RHDHBUGS-2574

Description of problem:

Version metadata for plugin, packages and dynamicArtifact is mismatched.

Mismatches can be viewed here: https://github.com/redhat-developer/rhdh-plugin-export-overlays/actions/runs/21521471288/job/62013576902

version: 0.6.2
backstage:
role: backend-plugin
supportedVersions: 1.39.1
Copy link

Choose a reason for hiding this comment

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

I am not sure if it's part of this PR, however I want to share that I noticed. Some of .yaml use old supportedVersions version. For example, for the adoption-insights version was updated to 1.45.2 - ref.
Should the version also be updated in the PR?

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, yes on a given rhdh release we would want this to match the backstage version. @polasudo I wonder this field could probably be set automatically couldn't it?

Copy link
Member

Choose a reason for hiding this comment

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

This would be good to update. However, only for plugins that are on that Backstage version. I think that if some plugins are on older Backstage version, they would need to be manually validated that they work.

Copy link
Member Author

Choose a reason for hiding this comment

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

@gashcrumb best thing would be to update it but the I am not exactly sure why they are not updated, while other are updated, maybe the problem is that they are not build for higher versions of backstage ?

Copy link
Member

Choose a reason for hiding this comment

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

Some are not updated because it was forgotten. Some are not updated because they are on older backstage version then 1.45. On the other hand, looking at catalog for 1.8, looks like all of the plugins there had supportedVersions set to 1.42, even if they were on different version. E.g roadie jira https://github.com/redhat-developer/rhdh/blob/release-1.8/catalog-entities/marketplace/packages/roadiehq-backstage-plugin-jira.yaml#L21 is on 1.40.2: https://github.com/RoadieHQ/roadie-backstage-plugins/blob/main/plugins/frontend/backstage-plugin-jira/CHANGELOG.md#2131, but supportedVersions are set to 1.42: https://github.com/redhat-developer/rhdh/blob/release-1.8/catalog-entities/marketplace/packages/roadiehq-backstage-plugin-jira.yaml#L24

Copy link
Member

Choose a reason for hiding this comment

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

I have updated supportedVersions for plugins that are on that version.
Let me know if we want to update all of them.
Remaining ones should be:

  Roadie Backstage Plugins (8 plugins)
  1. roadiehq-backstage-plugin-argo-cd-backend.yaml
  2. roadiehq-scaffolder-backend-module-http-request.yaml
  3. roadiehq-scaffolder-backend-module-utils.yaml
  4. roadiehq-scaffolder-backend-argocd.yaml
  5. roadiehq-backstage-plugin-security-insights.yaml
  6. roadiehq-backstage-plugin-jira.yaml
  7. roadiehq-backstage-plugin-github-pull-requests.yaml
  8. roadiehq-backstage-plugin-github-insights.yaml
  9. roadiehq-backstage-plugin-datadog.yaml
  Red Hat Resource Optimization (2 plugins)
  1. redhat-resource-optimization.yaml
  2. redhat-resource-optimization-backend.yaml
  AWS Plugins (2 plugins)
  1. aws-amazon-ecs-plugin-for-backstage.yaml
  2. aws-amazon-ecs-plugin-for-backstage-backend.yaml
  PagerDuty Plugins (2 plugins)
  1. pagerduty-backstage-plugin.yaml
  2. pagerduty-backstage-plugin-backend.yaml
  Immobiliarelabs Plugins (2 plugins)
  1. immobiliarelabs-backstage-plugin-gitlab.yaml
  2. immobiliarelabs-backstage-plugin-gitlab-backend.yaml
  Parfuemerie Plugins (1 plugin)
  1. parfdg-scaffolder-backend-mod-azure-repositories.yaml

Copy link
Member

Choose a reason for hiding this comment

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

@josephca didn't we delete a bunch of that from the overlays repo?

kadel
kadel previously requested changes Feb 3, 2026
@openshift-ci openshift-ci bot removed the lgtm label Feb 3, 2026
@polasudo polasudo force-pushed the fix/metadata-validation-main branch from f4038de to 8b9cf01 Compare February 3, 2026 16:32
- Fix annotation format: url https:// → url:https://
- Fix version mismatches: rbac (1.47.3), argocd (2.4.3), dynamic-home-page (1.10.4)
- Fix OCI tag mismatches: dynatrace-dql (bs_1.45.3__), apic (bs_1.45.3__)
- Remove test plugin metadata files (*-test.yaml)
- Update supportedVersions where applicable

https://issues.redhat.com/browse/RHDHBUGS-2574

Mismatches can be viewed here: https://github.com/redhat-developer/rhdh-plugin-export-overlays/actions/runs/21521471288/job/62013576902
@polasudo polasudo force-pushed the fix/metadata-validation-main branch from 8b9cf01 to f8d0ea1 Compare February 4, 2026 08:12
Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

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

Thank you so much @polasudo!
/lgtm
/approve

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

⚠️
Test workflow skipped: PR doesn't touch exactly one workspace.

dzemanov and others added 6 commits February 4, 2026 11:35
Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
- redhat-resource-optimization: 1.2.1 → 2.0.1
- redhat-resource-optimization-backend: 1.2.1 → 2.0.2

Versions match pinned commit 0ef0e1adb07534b55fded4967ea046b0fa467b2d
Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
@nickboldt
Copy link
Member

See also related PR #1929

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 5, 2026

Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Feb 5, 2026
@dzemanov dzemanov requested a review from kadel February 5, 2026 08:31
@polasudo
Copy link
Member Author

polasudo commented Feb 5, 2026

/publish

@polasudo
Copy link
Member Author

polasudo commented Feb 5, 2026

/lgtm
/approve

@openshift-ci
Copy link

openshift-ci bot commented Feb 5, 2026

@polasudo: you cannot LGTM your own PR.

Details

In response to this:

/lgtm
/approve

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci
Copy link

openshift-ci bot commented Feb 5, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dzemanov, gashcrumb, polasudo

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

PR action (/publish) cancelled: PR doesn't touch only 1 workspace.

@polasudo polasudo dismissed kadel’s stale review February 5, 2026 08:31

issues has been fixed

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

⚠️
Test workflow skipped: PR doesn't touch exactly one workspace.

@dzemanov dzemanov merged commit 5202f45 into redhat-developer:main Feb 5, 2026
3 of 4 checks passed
@dzemanov
Copy link
Member

dzemanov commented Feb 5, 2026

/cherry-pick release-1.9

@openshift-cherrypick-robot

@dzemanov: #1915 failed to apply on top of branch "release-1.9":

Applying: fix: resolve metadata validation failures
Using index info to reconstruct a base tree...
A	workspaces/apiconnect/metadata/apic-backstage-plugin.yaml
A	workspaces/dynatrace-dql/metadata/dynatrace-backstage-plugin-dql-backend.yaml
A	workspaces/dynatrace-dql/metadata/dynatrace-backstage-plugin-dql.yaml
Falling back to patching base and 3-way merge...
Auto-merging workspaces/dynatrace/metadata/backstage-community-plugin-dynatrace.yaml
CONFLICT (content): Merge conflict in workspaces/dynatrace/metadata/backstage-community-plugin-dynatrace.yaml
CONFLICT (modify/delete): workspaces/dynatrace-dql/metadata/dynatrace-backstage-plugin-dql-backend.yaml deleted in HEAD and modified in fix: resolve metadata validation failures. Version fix: resolve metadata validation failures of workspaces/dynatrace-dql/metadata/dynatrace-backstage-plugin-dql-backend.yaml left in tree.
CONFLICT (modify/delete): workspaces/apiconnect/metadata/apic-backstage-plugin.yaml deleted in HEAD and modified in fix: resolve metadata validation failures. Version fix: resolve metadata validation failures of workspaces/apiconnect/metadata/apic-backstage-plugin.yaml left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 fix: resolve metadata validation failures

Details

In response to this:

/cherry-pick release-1.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

polasudo added a commit that referenced this pull request Feb 5, 2026
* fix: resolve metadata validation failures

- Fix annotation format: url https:// → url:https://
- Fix version mismatches: rbac (1.47.3), argocd (2.4.3), dynamic-home-page (1.10.4)
- Fix OCI tag mismatches: dynatrace-dql (bs_1.45.3__), apic (bs_1.45.3__)
- Remove test plugin metadata files (*-test.yaml)
- Update supportedVersions where applicable

https://issues.redhat.com/browse/RHDHBUGS-2574

Mismatches can be viewed here: https://github.com/redhat-developer/rhdh-plugin-export-overlays/actions/runs/21521471288/job/62013576902

* fix: correct packageName for redhat-resource-optimization

Update packageName to match actual source package.json:
- @red-hat-developer-hub/backstage-plugin-optimizations → @red-hat-developer-hub/plugin-redhat-resource-optimization
- @red-hat-developer-hub/backstage-plugin-optimizations-backend → @red-hat-developer-hub/plugin-redhat-resource-optimization-backend

* Update rbac

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

* fix: update redhat-resource-optimization versions to match source

- redhat-resource-optimization: 1.2.1 → 2.0.1
- redhat-resource-optimization-backend: 1.2.1 → 2.0.2

Versions match pinned commit 0ef0e1adb07534b55fded4967ea046b0fa467b2d

* Update supportedVersions for plugins on that Backstage version

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

* Fix source-location format

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

* Fix external locations

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

* Fix wrong tab

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

* Downgrade resource optimization version

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>

---------

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
Co-authored-by: Dominika Zemanovicova <dzemanov@redhat.com>
@dzemanov
Copy link
Member

dzemanov commented Feb 5, 2026

Note: cherry-pick failed because some workspaces were removed for release-1.9 branch in this commit: 6d81ca9, workspaces removed:

  • acs
  • apiconnect
  • aws-ecs
  • dynatrace-dql
  • mta
  • openshift-image-registry
  • redhat-resource-optimization
  • scaffolder-backend-module-azure-repositories
  • tech-insights

polasudo added a commit that referenced this pull request Feb 5, 2026
fix: resolve metadata validation failures (#1915)

* fix: resolve metadata validation failures

- Fix annotation format: url https:// → url:https://
- Fix version mismatches: rbac (1.47.3), argocd (2.4.3), dynamic-home-page (1.10.4)
- Fix OCI tag mismatches: dynatrace-dql (bs_1.45.3__), apic (bs_1.45.3__)
- Remove test plugin metadata files (*-test.yaml)
- Update supportedVersions where applicable

https://issues.redhat.com/browse/RHDHBUGS-2574

Mismatches can be viewed here: https://github.com/redhat-developer/rhdh-plugin-export-overlays/actions/runs/21521471288/job/62013576902

* fix: correct packageName for redhat-resource-optimization

Update packageName to match actual source package.json:
- @red-hat-developer-hub/backstage-plugin-optimizations → @red-hat-developer-hub/plugin-redhat-resource-optimization
- @red-hat-developer-hub/backstage-plugin-optimizations-backend → @red-hat-developer-hub/plugin-redhat-resource-optimization-backend

* Update rbac



* fix: update redhat-resource-optimization versions to match source

- redhat-resource-optimization: 1.2.1 → 2.0.1
- redhat-resource-optimization-backend: 1.2.1 → 2.0.2

Versions match pinned commit 0ef0e1adb07534b55fded4967ea046b0fa467b2d

* Update supportedVersions for plugins on that Backstage version



* Fix source-location format



* Fix external locations



* Fix wrong tab



* Downgrade resource optimization version



---------

Signed-off-by: Dominika Zemanovicova <dzemanov@redhat.com>
Co-authored-by: Dominika Zemanovicova <dzemanov@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants