Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-33661

aws: rename `preserverBootstrapIgnition` install-config option

XMLWordPrintable

    • No
    • False
    • Hide

      None

      Show
      None
    • Hide
      The `preserveBootstrapIgnition` parameter for  {aws-short}  in the `install-config.yaml` file has been deprecated. You can use the `bestEffortDeleteIgnition` parameter instead.
      (link:https://issues.redhat.com/browse/OCPBUGS-33661[*OCPBUGS-33661*])
      Show
      The `preserveBootstrapIgnition` parameter for  {aws-short}  in the `install-config.yaml` file has been deprecated. You can use the `bestEffortDeleteIgnition` parameter instead. (link: https://issues.redhat.com/browse/OCPBUGS-33661 [* OCPBUGS-33661 *])
    • Deprecated Functionality
    • Done

      Description of problem:

      `preserveBootstrapIgnition` was named after the implementation details in terraform for how to make deleting S3 objects optional. The motivation behind the change was that some customers run installs in subscriptions where policies do not allow deleting s3 objects. They didn't want the install to fail because of that.

With the move from terraform to capi/capa, this is now implemented differently: capa always tries to delete the s3 objects but will ignore any permission errors if `preserveBootstrapIgnition` is set.

We should rename this option so it's clear that the objects will be deleted if there are enough permissions. My suggestion is to name something similar to what's named in CAPA: `allowBestEffortDeleteIgnition`.

Ideally we should deprecate `preserveBootstrapIgnition` in 4.16 and remove it in 4.17.

      Version-Release number of selected component (if applicable):

          4.14+ but I don't think we want to change this for terraform-based installs

      How reproducible:

          always

      Steps to Reproduce:

          1.
    2.
    3.

      Actual results:

      Expected results:

      Additional info:

          https://github.com/openshift/installer/pull/7288

              rdossant Rafael Fonseca dos Santos
              rdossant Rafael Fonseca dos Santos
              Yunfei Jiang Yunfei Jiang
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: