Skip to content

feat: restore os provider on core library#64764

Merged
potiuk merged 3 commits intoapache:mainfrom
Owen-CH-Leung:feat/os_provider_change_on_core
Apr 10, 2026
Merged

feat: restore os provider on core library#64764
potiuk merged 3 commits intoapache:mainfrom
Owen-CH-Leung:feat/os_provider_change_on_core

Conversation

@Owen-CH-Leung
Copy link
Copy Markdown
Contributor

@Owen-CH-Leung Owen-CH-Leung commented Apr 6, 2026

This PR is a follow-up on #64364.

It introduces the changes required for bringing opensearch provider back for remote task log for Airflow 3. Also it enables the feature to automatically ship task log to opensearch.

@boring-cyborg boring-cyborg bot added area:ConfigTemplates area:dev-tools area:logging backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch labels Apr 6, 2026
@Owen-CH-Leung Owen-CH-Leung force-pushed the feat/os_provider_change_on_core branch from fe76853 to 8632a35 Compare April 7, 2026 04:55
@eladkal
Copy link
Copy Markdown
Contributor

eladkal commented Apr 7, 2026

Probably need also to set min version 1.9.0 in

"apache-airflow-providers-opensearch>=1.5.0"

@Owen-CH-Leung Owen-CH-Leung force-pushed the feat/os_provider_change_on_core branch from 8632a35 to 39ad380 Compare April 8, 2026 13:44
@Owen-CH-Leung Owen-CH-Leung force-pushed the feat/os_provider_change_on_core branch from 4775249 to 06e47ee Compare April 8, 2026 14:05
@eladkal
Copy link
Copy Markdown
Contributor

eladkal commented Apr 8, 2026

We need also to add it to

# Temporary override for providers that are not yet included in constraints or when they need
# minimum versions for compatibility with Airflow 3
MIN_VERSION_OVERRIDE: dict[str, Version] = {
"amazon": parse_version("2.1.3"),
"fab": parse_version("2.2.0"),
"openlineage": parse_version("2.3.0"),
"git": parse_version("0.0.2"),
"common.messaging": parse_version("2.0.0"),
"elasticsearch": parse_version("6.5.0"),
}

@Owen-CH-Leung
Copy link
Copy Markdown
Contributor Author

We need also to add it to

# Temporary override for providers that are not yet included in constraints or when they need
# minimum versions for compatibility with Airflow 3
MIN_VERSION_OVERRIDE: dict[str, Version] = {
"amazon": parse_version("2.1.3"),
"fab": parse_version("2.2.0"),
"openlineage": parse_version("2.3.0"),
"git": parse_version("0.0.2"),
"common.messaging": parse_version("2.0.0"),
"elasticsearch": parse_version("6.5.0"),
}

Thanks! I've made the changes

@Owen-CH-Leung Owen-CH-Leung force-pushed the feat/os_provider_change_on_core branch from 2919d25 to b1b39ac Compare April 9, 2026 23:26
@eladkal eladkal marked this pull request as ready for review April 10, 2026 05:49
Copy link
Copy Markdown
Contributor

@eladkal eladkal left a comment

Choose a reason for hiding this comment

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

LGTM

@potiuk potiuk merged commit 9465591 into apache:main Apr 10, 2026
573 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 10, 2026
* add changes for os provider

* bump pyproject.toml provider lib version

* Fix failing CI
(cherry picked from commit 9465591)

Co-authored-by: Owen Leung <owen.leung2@gmail.com>
@github-actions
Copy link
Copy Markdown

Backport successfully created: v3-2-test

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-2-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Apr 10, 2026
* add changes for os provider

* bump pyproject.toml provider lib version

* Fix failing CI
(cherry picked from commit 9465591)

Co-authored-by: Owen Leung <owen.leung2@gmail.com>
eladkal pushed a commit that referenced this pull request Apr 13, 2026
* add changes for os provider

* bump pyproject.toml provider lib version

* Fix failing CI
(cherry picked from commit 9465591)

Co-authored-by: Owen Leung <owen.leung2@gmail.com>
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