Skip to content

Fix OpenAI provider get_provider_info#63404

Merged
jason810496 merged 3 commits into
apache:mainfrom
jason810496:fix/ci/openai-provider-get-provider-info
Mar 12, 2026
Merged

Fix OpenAI provider get_provider_info#63404
jason810496 merged 3 commits into
apache:mainfrom
jason810496:fix/ci/openai-provider-get-provider-info

Conversation

@jason810496

@jason810496 jason810496 commented Mar 12, 2026

Copy link
Copy Markdown
Member

Why

The /opt/airflow/scripts/in_container/run_provider_yaml_files_check.py is still failing. We need to edit the airflow-core/src/airflow/provider.yaml.schema.json and providers/openai/src/airflow/providers/openai/get_provider_info.py to get them pass.

  During handling of the above exception, another exception occurred:

  Traceback (most recent call last):
    File "/opt/airflow/scripts/in_container/run_provider_yaml_files_check.py", line 771, in <module>
      all_parsed_yaml_files: dict[str, dict] = _load_package_data(paths)
    File "/opt/airflow/scripts/in_container/run_provider_yaml_files_check.py", line 163, in _load_package_data
      raise RuntimeError(msg)
  RuntimeError: Unable to parse: /opt/airflow/providers/openai/provider.yaml. Original error ValidationError: ['ai', 'openai', 'machine-learning', 'generative-ai', 'llm', 'text-generation', 'embeddings', 'ai-inference'] is too long

  Failed validating 'maxItems' in schema['properties']['integrations']['items']['properties']['tags']:
      {'description': "List of tags describing the integration. While we're "
                      'using RST, only one tag is supported per integration.',
       'type': 'array',
       'items': {'type': 'string',
                 'enum': ['ai',
                          'alibaba',
                          'apache',
                          'aws',
                          'azure',
                          'dbt',
                          'gcp',
                          'gmp',
                          'google',
                          'kafka',
                          'protocol',
                          'service',
                          'software',
                          'yandex']},
       'minItems': 1,
       'maxItems': 1}

  On instance['integrations'][0]['tags']:
      ['ai',
       'openai',
       'machine-learning',
       'generative-ai',
       'llm',
       'text-generation',
       'embeddings',
       'ai-inference']
  Error 1 returned

@amoghrajesh amoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

Tests need fixing though

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the previous fail tests (static-check, doc builds) are now passed, so I merging now.

@jason810496 jason810496 merged commit 1b4fafb into apache:main Mar 12, 2026
127 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport failed to create: v3-1-test. View the failure log Run details

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 1b4fafb v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

If you don't have cherry-picker installed, see the installation guide.

PascalEgn pushed a commit to PascalEgn/airflow that referenced this pull request Mar 12, 2026
* Fix OpenAI provider get_provider_info

* Fix index.rst

* Fix newline static check for index.rst
Pyasma pushed a commit to Pyasma/airflow that referenced this pull request Mar 13, 2026
* Fix OpenAI provider get_provider_info

* Fix index.rst

* Fix newline static check for index.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants