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 commented on Apr 11, 2025
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.
A
move environment vars to a configuration file
ewdurbin commented on 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 commented on 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.