Skip to content

Conversation

@jason810496
Copy link
Member

Why

Fix the CI error on the v2-10-test branch after bumping the Breeze version.
Related Slack discussion: https://apache-airflow.slack.com/archives/C015SLQF059/p1742884602236699

What

  • Commit the latest uv.lock in dev/breeze.

@jason810496
Copy link
Member Author

Documents still need a fix, will look into it tomorrow.

@jason810496
Copy link
Member Author

Hi @potiuk,

I need some help with the docs issue. I haven’t seen any changes related to the docs configuration applied to the main branch but not ported to v2-10-test. Since the error logs show the same content as the main branch, this seems to be a configuration-related issue, right?

@potiuk
Copy link
Member

potiuk commented Mar 31, 2025

Let's merge it first as is. and Open a new PR with doc fixes.

@potiuk potiuk merged commit abf9d22 into apache:v2-10-test Mar 31, 2025
58 of 88 checks passed
@potiuk
Copy link
Member

potiuk commented Mar 31, 2025

If you look at the top (first errors) - seems that there are some missing/wrong links defined. the "helm-chart" refers to manage-dags-files from helm-chart package - it's likely the main version does not have the link any more - and likely celery__workr_precheck is missing in v2-10-test config. We need to just find where those labels are used and replace them with the right labels or remove.

The thing is that in v2-10-test we don not build "helm_chart" - we are using main inventory to check if links are working.

------------------------------ Error   2 --------------------
 WARNING: undefined label: 'config:celery__worker_precheck'

File path: /opt/airflow/docs/apache-airflow/<unknown>
------------------------------ Error   3 --------------------
 WARNING: unknown document: 'helm-chart:manage-dags-files'

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