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

Azure DNS zone pagination loop bug in getBaseDomainID function

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • 4.21.0
    • HyperShift / ARO
    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Description of problem:

      The `getBaseDomainID` function in `cmd/infra/azure/create.go` was using an `if` statement instead of a `for` loop to iterate through DNS zone pages, causing it to only process the first page of results instead of all available pages.
      

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

      OpenShift 4.21 

      How reproducible:

      Always reproducible when there are multiple pages of DNS zones in Azure
      

      Steps to Reproduce:

      1. Set up Azure environment with multiple DNS zones (requiring pagination)
      2. Run HyperShift Azure infrastructure creation commands
      3. The function will only check the first page of DNS zones
      4. If the target DNS zone is on subsequent pages, the function will fail to find it
      

      Actual results:

      The function only processes the first page of DNS zones from the Azure DNS API, potentially missing the target DNS zone if it appears on later pages.
      

      Expected results:

      The function should iterate through all pages of DNS zones to find the target zone, ensuring no zones are skipped during the search.
      

              rh-ee-supatil Suraj Patil
              jparrill@redhat.com Juan Manuel Parrilla Madrid
              None
              None
              He Liu He Liu
              None
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated: