Uploaded image for project: 'OpenShift Cloud'
  1. OpenShift Cloud
  2. OCPCLOUD-2719

[AWS] Should the ignition version be extracted rather than fixed

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • None
    • 3
    • False
    • None
    • False
    • OCPSTRAT-683 - Migrate MAPI to Cluster API for AWS -Phase 1
    • CLOUD Sprint 262, CLOUD Sprint 262

      Background

      Currently, in the MAPI to CAPI conversion we hard code the ignition version. However, this will mean it may not match the user data secret we are passing through.

      The version isn't currently used in the code anywhere, because we use the "passthrough" version of the logic, but it may be used in the future.

      We should investigate bringing the version out of the user data and supporting the correct version from there.

      Alternatively, we could see about making the ignition version optional in certain circumstances within CAPA (when using this passthrough mode)

      Steps

      • Review the available ignition versions in CAPA
      • Determine if the version is being used for anything in CAPA. (We could make it non required if it is not required for the UncompressedUserData mode).
      • Determine appropriate versions that may have been used in OCP before
      • Investigate extracting the ignition version from the user data secret and adding it directly to the spec

      Stakeholders

      • Cluster Infra

      Definition of Done

      • We either don't care about the ignition version, or it's plumbed to the correct version
      • Might want to propose a patch to make ignition version optional for the UncompressedUserData mode
      • Docs
      • <Add docs requirements for this card>
      • Testing
      • <Explain testing that will be added>

            Unassigned Unassigned
            joelspeed Joel Speed
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: