[Helm Chart] Update pre-release doc URLs to point to staged documentation (#50569)#54493
[Helm Chart] Update pre-release doc URLs to point to staged documentation (#50569)#54493prdai wants to merge 1 commit into
Conversation
potiuk
left a comment
There was a problem hiding this comment.
I don't think it's a good idea - how do you distinguish whether it's pre-release or not? We cannot really commit the code for pre-release / staging docs because we would have to somehow remember to bring them back to PROD - that's very, very brittle.
How it should be done - the links should be still "prod" - but they should be dynamically replaced with the staged build when .RC version of the chart is being prepared as release - so it should be part of the "breeze release-management" commands and code that is run to prepare the release, not committed to the repo
|
Hi @Programmer-RD-AI, Let me know what you think! |
|
hi @HsiuChuanHsu please feel free to take over, i have been unable to look into this since the initial pr, thanks :) i will close the pr so that you can start? |
|
Thanks! |
This PR updates the documentation links in
Chart.yamlandvalues.yamlfor Helm Chart pre-release candidates.Previously, these links pointed to the production docs at https://airflow.apache.org, which is inconsistent with other pre-release artifacts (e.g., PyPI packages) that reference staged docs.
Changes in this PR:
related: #50569