Skip to content

Conversation

@christoph-jerolimov
Copy link
Member

Description

Fixes some smaller catalog metadata issues:

  • Different author corrections
  • Highlights: Add some highlights of the plugins features
  • Description: Provides the set of Software Template Actions => Provides a set of Software Template Actions
  • Merge Signals Frontend and Backend plugin? => No, because they have different support levels
  • Merge Kubernetes and Kubernetes Backend? => No, because they have different support levels
  • Dynatrace Certified is also displayed as custom plugin
  • Extensions plugin should not use Marketplace category => Changed to Customize

Missing packages:

  • Ansible Plugins
  • Argo CD Software Template Actions
  • Azure Repository Software Template Actions
  • Bulk import misses backend
  • Catalog Backend Module for Microsoft Graph
  • Container Image Registry for Quay
  • Dynatrace For Managed Backend?
  • Jira Backend? => Does not exist
  • LDAP Catalog Integration
  • Tekton => Kubernetes Backend
  • Topology => Kubernetes Backend

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@christoph-jerolimov
Copy link
Member Author

@christoph-jerolimov
Copy link
Member Author

/cherry-pick release-1.6

@openshift-cherrypick-robot
Copy link
Contributor

@christoph-jerolimov: once the present PR merges, I will cherry-pick it on top of release-1.6 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-1.6

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.

Copy link
Member

@logonoff logonoff left a comment

Choose a reason for hiding this comment

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

/lgtm

@christoph-jerolimov
Copy link
Member Author

/cc @logonoff

@openshift-ci openshift-ci bot added the lgtm label Apr 22, 2025
@openshift-ci openshift-ci bot requested a review from logonoff April 22, 2025 21:01
@github-actions
Copy link
Contributor

@christoph-jerolimov
Copy link
Member Author

/retest

@ciiay
Copy link
Member

ciiay commented Apr 23, 2025

/test e2e-tests

Copy link
Member

@ciiay ciiay left a comment

Choose a reason for hiding this comment

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

Looks good in general, just left a few typo comments.

@ciiay
Copy link
Member

ciiay commented Apr 23, 2025

/test e2e-tests

@christoph-jerolimov christoph-jerolimov force-pushed the update-extensions-catalog-metadata branch from f1d1026 to 266fba1 Compare April 23, 2025 08:43
@openshift-ci openshift-ci bot removed the lgtm label Apr 23, 2025
@github-actions
Copy link
Contributor

@ciiay
Copy link
Member

ciiay commented Apr 23, 2025

/test e2e-tests

Copy link
Member

@ciiay ciiay left a comment

Choose a reason for hiding this comment

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

/lgtm

Thank you!

@ciiay
Copy link
Member

ciiay commented Apr 23, 2025

/test e2e-tests

1 similar comment
@ciiay
Copy link
Member

ciiay commented Apr 23, 2025

/test e2e-tests

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@github-actions
Copy link
Contributor

Copy link
Member

@ciiay ciiay left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Apr 24, 2025
@christoph-jerolimov
Copy link
Member Author

/approve

@openshift-ci
Copy link

openshift-ci bot commented Apr 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: christoph-jerolimov, ciiay, logonoff

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [christoph-jerolimov]

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

@christoph-jerolimov
Copy link
Member Author

@christoph-jerolimov christoph-jerolimov merged commit 7e2e728 into redhat-developer:main Apr 24, 2025
11 of 13 checks passed
@christoph-jerolimov christoph-jerolimov deleted the update-extensions-catalog-metadata branch April 24, 2025 21:27
@openshift-cherrypick-robot
Copy link
Contributor

@christoph-jerolimov: new pull request created: #2806

Details

In response to this:

/cherry-pick release-1.6

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.

christoph-jerolimov added a commit to christoph-jerolimov/rhdh that referenced this pull request Apr 25, 2025
* fix(extensions): fix rhdh capitalization

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix link in ansible plugin description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): mark ansible plugin and dynatrace as catalog-pre-installed (not custom plugin)

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for signal plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for acr plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of microsoft graph catalog module

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): rename scaffold to software-templates

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix bitbucket abstract/short description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align ansible title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align techdocs title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align categories

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of pingidentity plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix highlights

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): address code review comment

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

---------

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
openshift-merge-bot bot pushed a commit that referenced this pull request Apr 25, 2025
…nd update catalog metadata and icons (#2796)

* fix(extensions): update plugin to show a no-icon-icon when the icon is missing in the plugin metadata

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add or update some catalog icons (#2788)

* fix(extensions): add backstage icon for notification, signal, and catalog/scaffolder-integration plugins

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add backstage icon to scaffolder action plugins

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add servicenow icon

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add microsoft graph api icon

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add segment icon

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add pingidentity icon

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add backstage icon for ldap plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add rhdh icon for ocm and regex plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): add right icon for the orchestrator plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): disable gray placeholder icon in boilerplate

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

---------

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): update some catalog metadata (#2793)

* fix(extensions): fix rhdh capitalization

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix link in ansible plugin description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): mark ansible plugin and dynatrace as catalog-pre-installed (not custom plugin)

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for signal plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for acr plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of microsoft graph catalog module

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): rename scaffold to software-templates

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix bitbucket abstract/short description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align ansible title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align techdocs title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align categories

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of pingidentity plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix highlights

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): address code review comment

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

---------

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): update some catalog icons (again) (#2805)

---------

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
chadcrum pushed a commit to chadcrum/rhdh that referenced this pull request Aug 3, 2025
* fix(extensions): fix rhdh capitalization

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix link in ansible plugin description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): mark ansible plugin and dynatrace as catalog-pre-installed (not custom plugin)

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for signal plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author for acr plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of microsoft graph catalog module

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): rename scaffold to software-templates

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix bitbucket abstract/short description

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align ansible title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align techdocs title

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): align categories

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): correct plugins to packages references

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix author of pingidentity plugin

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): fix highlights

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

* fix(extensions): address code review comment

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>

---------

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants