Uploaded image for project: 'Red Hat Advanced Cluster Management'
  1. Red Hat Advanced Cluster Management
  2. ACM-14931

[Tech debt] Refactor import controller

XMLWordPrintable

    • [Tech debt] Refactor import controller
    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • False
    • Green
    • Done
    • 0% To Do, 0% In Progress, 100% Done
    • Important

      OCP/Telco Definition of Done
      https://docs.google.com/document/d/1TP2Av7zHXz4_fmeX4q9HB0m9cqSZ4F6Jd4AiVoaF_2s/edit#heading=h.gaa58bzbvwde
      Epic Template descriptions and documentation.
      https://docs.google.com/document/d/14CUCEg6hQ_jpsFzJtWo29GfFVWmun2Uivrxq3_Fkgdg/edit
      ACM-wide Product Requirements (Top-level Epics)
      https://docs.google.com/document/d/1uIp6nS2QZ766UFuZBaC9USs8dW_I5wVdtYF9sUObYKg/edit

      *<--- Cut-n-Paste the entire contents of this description into your new
      Epic --->*

      Epic Goal

      Refactor import controller

      • Refine the logic of import and detach;
      • Reuse the garbage collection logic available in the upstream repo.

      Why is this important?

      the cluster import/detach is the key function in RHACM and CME. and the related codes in import-controller are more complicated and not easy to maintain, and the related issues occurred and hard to address and fix. so to refactor these process can make it more easier to maintain, and improve the product qualities. 

      Scenarios

      cluster import :  

      1. import cluster manually
      2. auto import cluster.
      3. provision cluster.
      4. create hosted cluster.

      cluster detach :

      1. delete cluster and all related resources are cleaned up.
      2. delete cluster and all related resources are cleaned up by force after cluster is unavailable. 
      3. delete hosted cluster and all related resources are cleaned up.

      Acceptance Criteria

      all cluster import and detach cases can pass. 

      Dependencies (internal and external)

      1.  

      Previous Work (Optional):

      1. ...

      Open questions:

      Done Checklist

      • CI - CI is running, tests are automated and merged.
      • Release Enablement <link to Feature Enablement Presentation>
      • DEV - Upstream code and tests merged: <link to meaningful PR or GitHub
        Issue>
      • DEV - Upstream documentation merged: <link to meaningful PR or GitHub
        Issue>
      • DEV - Downstream build attached to advisory: <link to errata>
      • QE - Test plans in Polarion: <link or reference to Polarion>
      • QE - Automated tests merged: <link or reference to automated tests>
      • DOC - Doc issue opened with a completed template. Separate doc issue
        opened for any deprecation, removal, or any current known
        issue/troubleshooting removal from the doc, if applicable.

              zyin@redhat.com Zhiwei Yin
              leyan@redhat.com Le Yang
              Hui Chen Hui Chen
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: