Uploaded image for project: 'Apache Airflow'
  1. Apache Airflow
  2. AIRFLOW-5410

AirflowConfigParser no longer find conf entries with capitalized letters

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.10.5
    • Fix Version/s: None
    • Component/s: configuration
    • Labels:
      None

      Description

      The following commit changed the behavior of configuration parsing globally (although the commit message only says "kubernetes"):
      AIRFLOW-4316 Support setting kubernetes_environment_variables config section from env var (#5668)

       

      This used to work:

      [mysection]
      Guess = "test"
      assert conf.get("mysection", "Guess")  == "test"

      It stopped working since 1.10.5. It'll break because the key "Guess" is not found. To get around the issue, "Guess" needs to be changed to all lower case
      "guess". This change can cause surprises because it is not obvious from the changelog this commit changed conf parsing everywhere.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              yuqian90 Qian Yu
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: