You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, salt manages four environment variables: SENTRY_DSN (for Sentry reporting), FASTLY_SERVICE_ID and FASTLY_TOKEN (for purging the CDN), and PYTHON_DOCS_ENABLE_ANALYTICS=1, to add the analytics scripts.
We could add PYTHON_DOCS_ENABLE_ANALYTICS=1 to build_docs.py itself, but the other variables should be kept secret. Currently, we need to manually export them into a shell when doing rebuilds for everything to work as expected.
Activity
AA-Turner commentedon Apr 11, 2025
Currently, salt manages four environment variables:
SENTRY_DSN(for Sentry reporting),FASTLY_SERVICE_IDandFASTLY_TOKEN(for purging the CDN), andPYTHON_DOCS_ENABLE_ANALYTICS=1, to add the analytics scripts.We could add
PYTHON_DOCS_ENABLE_ANALYTICS=1tobuild_docs.pyitself, but the other variables should be kept secret. Currently, we need to manually export them into a shell when doing rebuilds for everything to work as expected.A
move environment vars to a configuration file
ewdurbin commentedon Apr 11, 2025
#269 (nice) is an option. if it's acceptable I can update psf-salt and I think we're done here.
StanFromIreland commentedon Apr 4, 2026
I think this is complete, I didn't have to set anything when running manual rebuilds, and Ee's psf-salt PR is merged. Please reopen if I'm incorrect.