Uploaded image for project: 'OpenShift Installer'
  1. OpenShift Installer
  2. CORS-2697

Hybrid SRE: Allow setting ManagedBy on Cluster Resource Group

XMLWordPrintable

    • Icon: Epic Epic
    • Resolution: Duplicate
    • Icon: Major Major
    • None
    • None
    • None
    • Allow setting ManagedBy on Cluster Resource Group
    • True
    • Show
      Terraform regression here https://github.com/hashicorp/terraform-provider-azurerm/issues/21785
    • False
    • Red
    • In Progress
    • OCPSTRAT-699 - Support setting `ManagedBy` on Cluster Resource Group
    • OCPSTRAT-699Support setting `ManagedBy` on Cluster Resource Group
    • 33% To Do, 0% In Progress, 67% Done
    • Hide

      Aug-9

      Status: Red

      PR merged but there was a regression introduced by terraform provider azurerm where installation was failing on Azure Gov Cloud. Until terraform fixes this, we need to put this change in the back burner since the changes I made to support managed by is in the latest versions of terraform.

      More information here:

      https://github.com/hashicorp/terraform-provider-azurerm/issues/21785

      Jul-26

      PR up and needs to merge. Still green.

      June 28:

      Status green. Terraform PR merged and we can bring it to the installer.

      Show
      Aug-9 Status: Red PR merged but there was a regression introduced by terraform provider azurerm where installation was failing on Azure Gov Cloud. Until terraform fixes this, we need to put this change in the back burner since the changes I made to support managed by is in the latest versions of terraform. More information here: https://github.com/hashicorp/terraform-provider-azurerm/issues/21785 Jul-26 PR up and needs to merge. Still green. June 28: Status green. Terraform PR merged and we can bring it to the installer.

      OCP/Telco Definition of Done
      Epic Template descriptions and documentation.

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

      Feature Overview

      • Allow for the setting of the field 'managedBy' on the azure resource group.

      Epic Goal

      • Users are able to set the 'managedBy' field of the azure resource group during installation.

      Why is this important?

      • ARO needs this field set for their automation tool in the background.

      Background

      • ARO needs this field set for their automation tool in the background. Doc for more details.

      Out of scope

      1. None.

      Acceptance Criteria

      • Install config allows users to set a new field with the value for the 'managedBy' field.
      • Once above field is set, the installation creates a new resource group with the 'managedBy' field set.

      Dependencies (internal and external)

      1. Terraform provider azurerm which is used for installation in Azure does not have functionality to set this field and needs to be updated accordingly to facilitate this feature.

      Customer Considerations

      1. None

      Documentation Considerations

      1. Doc for new field in the install config needs to be added.

      Interoperability Considerations

      1. None

      Open questions::

      1. None

      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 - Downstream documentation merged: <link to meaningful PR>

            rna-afk Aditya Narayanaswamy
            beth.white Beth White
            Jinyun Ma Jinyun Ma
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: