Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jul 10, 2025

This is needed in order to avoid libxml and xmlsec using different version of libxml2 (binary lxml embeds its own libxml2, while xmlsec uses system one). See https://bugs.launchpad.net/lxml/+bug/2110068

Fixes: #52539


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools area:production-image Production image improvements and fixes area:providers backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch provider:amazon AWS/Amazon - related issues labels Jul 10, 2025
This is needed in order to avoid libxml and xmlsec using different
version of libxml2 (binary lxml embeds its own libxml2, while xmlsec
uses system one). See https://bugs.launchpad.net/lxml/+bug/2110068

Fixes: apache#52539
@potiuk potiuk force-pushed the fix-lxml-xmlsec-mismatch branch from 9c87a09 to fc27492 Compare July 10, 2025 13:38
@potiuk potiuk added all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs full tests needed We need to run full set of tests for this PR to merge labels Jul 10, 2025
@potiuk potiuk closed this Jul 10, 2025
@potiuk potiuk reopened this Jul 10, 2025
@potiuk
Copy link
Member Author

potiuk commented Jul 10, 2025

Let's see :)

Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

Interesting, lets see

@potiuk potiuk merged commit 7f5c8db into apache:main Jul 10, 2025
157 of 175 checks passed
@potiuk potiuk deleted the fix-lxml-xmlsec-mismatch branch July 10, 2025 14:08
@github-actions
Copy link

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

Status Branch Result
v3-0-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 7f5c8db v3-0-test

This should apply the commit to the v3-0-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

potiuk added a commit to potiuk/airflow that referenced this pull request Jul 10, 2025
…che#53137)

This is needed in order to avoid libxml and xmlsec using different
version of libxml2 (binary lxml embeds its own libxml2, while xmlsec
uses system one). See https://bugs.launchpad.net/lxml/+bug/2110068

Fixes: apache#52539
(cherry picked from commit 7f5c8db)
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
…che#53137)

This is needed in order to avoid libxml and xmlsec using different
version of libxml2 (binary lxml embeds its own libxml2, while xmlsec
uses system one). See https://bugs.launchpad.net/lxml/+bug/2110068

Fixes: apache#52539
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:dev-tools area:production-image Production image improvements and fixes area:providers backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch full tests needed We need to run full set of tests for this PR to merge provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lxml is held by apache-airflow-providers-amazon from upgrade

4 participants