Uploaded image for project: 'Red Hat Internal Developer Platform'
  1. Red Hat Internal Developer Platform
  2. RHIDP-3931

Added alert on the deletion icon during bulk imports

Prepare for Y ReleasePrepare for Z ReleaseRemove QuarterXMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Normal Normal
    • 1.4
    • 1.3
    • Bulk Import Plugin
    • None
    • 2
    • False
    • Hide

      None

      Show
      None
    • False
    • RHIDP-4171 - Improve user experience on the bulk import pages
    • Hide
      Before this update, repositories were added to the {product-short} from various sources, such as `app-config` files or GitHub discovery. The Bulk Import plugin only tracked repositories accessible using the configured GitHub integrations. When both plugins were enabled, repositories discovered by GitHub Discovery appeared on Bulk Import pages. However, deleting these repositories from Bulk Import Jobs had no effect, as entities from discovery or `app-config.yaml` file remained in the {product-short} catalog.

      With this update, an alert on the deletion icon notifies the user to modify the source (either the `catalog-info` within the repository or the `app-config.yaml` file if the file originates from there) to remove the catalog entity.
      Show
      Before this update, repositories were added to the {product-short} from various sources, such as `app-config` files or GitHub discovery. The Bulk Import plugin only tracked repositories accessible using the configured GitHub integrations. When both plugins were enabled, repositories discovered by GitHub Discovery appeared on Bulk Import pages. However, deleting these repositories from Bulk Import Jobs had no effect, as entities from discovery or `app-config.yaml` file remained in the {product-short} catalog. With this update, an alert on the deletion icon notifies the user to modify the source (either the `catalog-info` within the repository or the `app-config.yaml` file if the file originates from there) to remove the catalog entity.
    • Bug Fix
    • Done
    • RHDH Core Team 3263, RHDH Plugins 3265, RHDH Plugins 3266

      Description of problem:

      Entities of repos under a configured org in `backstage-plugin-catalog-backend-module-github-org-dynamic` plugin are not deleted from the catalog when the imported repo is deleted from bulk imports.

      Screen Recording 2024-09-11 at 3.45.22 PM.mov

      Prerequisites (if any, like setup, operators/versions):

      Steps to Reproduce

      1. Enable bulk-import plugin.
      2. Enable /dynamic-plugins/dist/backstage-plugin-catalog-backend-module-github-org-dynamic plugin and configure an org.
      3. Navigate to bulk import from sidebar and click on add repo
      4. Select one repo from org configured in `catalog-backend-module-github-org` plugin and one from another repo that isn't configured in above plugin.
      5. Click on create pull request and merge the PRs of both repos.
      6. Wait for entities of both repo to appear in catalog.
      7. Navigate back to bulk import and delete both repos from added repositories table.
      8. Navigate to catalog and verify the entities are deleted for both the repos.

      Actual results: Entities of repos under a configured org in catalog-backend-module-github-org plugin are not deleted from the catalog when the repo is deleted from bulk imports

      Expected results: entities of both repos should be deleted.

      Reproducibility (Always/Intermittent/Only Once): Always

      Build Details: 1.3/latest

      Additional info (Such as Logs, Screenshots, etc):

      Screen Recording 2024-09-11 at 3.45.22 PM.mov

        1. Catalog-unregister-github-provider.mp4
          1.45 MB
        2. Github-discovery.mp4
          1.14 MB
        3. Github-org.mp4
          2.49 MB
        4. Screen Recording 2024-09-11 at 3.45.22 PM.mov
          16.07 MB

              rh-ee-dzemanov Dominika Zemanovicova
              skhileri Subhash Khileri
              RHIDP - Plugins
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: