Skip to content

chore(ci): fix doc pipeline#873

Merged
traefiker merged 2 commits intotraefik:v1.4from
darkweaver87:fix/docs-pyyaml
Mar 23, 2026
Merged

chore(ci): fix doc pipeline#873
traefiker merged 2 commits intotraefik:v1.4from
darkweaver87:fix/docs-pyyaml

Conversation

@darkweaver87
Copy link
Copy Markdown
Collaborator

@darkweaver87 darkweaver87 commented Mar 17, 2026

What does this PR do?

Fixes the documentation build pipeline on v1.4 that fails with AttributeError: cython_sources during pip install.

Changes

  • Migrate Semaphore CI from cache to artifact commands
  • Bump PyYAML from 6.0.1 to 6.0.2 (fixes cython_sources build error)

Context

The structor-based publish workflow (on master) builds documentation for every versioned branch using each branch's own docs.Dockerfile. This creates a circular dependency: master can't publish because v1.4's Dockerfile is broken, and v1.4 can't be fixed without master passing.

This PR fixes the v1.4 side. The corresponding master fix is in #875.

@darkweaver87 darkweaver87 force-pushed the fix/docs-pyyaml branch 4 times, most recently from 99f9b5f to 0ab378b Compare March 17, 2026 15:38
@darkweaver87 darkweaver87 changed the base branch from master to v1.4 March 17, 2026 16:04
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 17, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Member

@mloiseleur mloiseleur left a comment

Choose a reason for hiding this comment

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

I see the CI update but not the fix on failed dep. Is this expected ?

@darkweaver87
Copy link
Copy Markdown
Collaborator Author

@mloiseleur yes, the failure is on master see #874

@darkweaver87 darkweaver87 requested a review from mloiseleur March 18, 2026 09:41
Fixes cython_sources AttributeError during pip install.
@darkweaver87 darkweaver87 added this to the v1.4 milestone Mar 23, 2026
@traefiker traefiker merged commit f75cb29 into traefik:v1.4 Mar 23, 2026
4 checks passed
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.

4 participants