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

The agent-registration should support return different short-term tokens every time.

XMLWordPrintable

    • 2
    • False
    • None
    • False
    • Hide

      Provide the required acceptance criteria using this template.
      * ...
      Show
      Provide the required acceptance criteria using this template. * ...
    • SF Train-17
    • No

      Value Statement

      Different tokens can reduce the risk of a shared token be compromised.

      Currently, after installed the MCE, the OCP(tested with 4.15) will create the token secret for the agent-registration-bootstrap serviceaccount.

       

      This means the agent-registraiton will also return the same token of agent-registration-bootstrap-token based on the code: https://github.com/stolostron/managedcluster-import-controller/blob/2198e50b13b71d5429f5bd921bddf209fca33bfb/pkg/bootstrap/boostrapkubeconfig.go#L115 

      But we also want to provide a short-term token and the token should be different each time.

      And we want the caller to assign the expiration of the token.

      Definition of Done for Engineering Story Owner (Checklist)

      • ...

      Development Complete

      • The code is complete.
      • Functionality is working.
      • Any required downstream Docker file changes are made.

      Tests Automated

      • [ ] Unit/function tests have been automated and incorporated into the
        build.
      • [ ] 100% automated unit/function test coverage for new or changed APIs.

      Secure Design

      • [ ] Security has been assessed and incorporated into your threat model.

      Multidisciplinary Teams Readiness

      Support Readiness

      • [ ] The must-gather script has been updated.

              zxue@redhat.com ZHAO XUE
              zxue@redhat.com ZHAO XUE
              Song Lai Song Lai
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: