Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Harmonize the staging and prod services for PyPI files hosting #96

Merged
merged 7 commits into from
Sep 12, 2022

Conversation

ewdurbin
Copy link
Member

This primarily refactors the two terraform configurations so that they're not in the same file 😂

The only actual change is renaming the friendly name of our custom VCL in the files service.

Note: The "PyPI Staging File Hosting" service in Fastly currently takes no traffic! Next step is to migrate it to :)

Towards #95

@ewdurbin
Copy link
Member Author

ewdurbin commented Aug 30, 2022

This is ✨ almost ✨ ready, the only thing I'm concerned about is if the production and test services need different aws/gcs credentials. to be, addressed :)

Copy link
Member Author

@ewdurbin ewdurbin left a comment

Choose a reason for hiding this comment

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

Verified that the credentials used for Fastly to read S3 and GCS objects are the same between test-files.pythonhosted.org and files.pythonhosted.org. Also that the credentials for writing linehaul output are also shared, and that the pipelines for processing access logs are independent (aka we won't write test.pypi.org data to pypi.org's dataset).

@ewdurbin
Copy link
Member Author

In order to deploy this we'll have to deactivate the hostname on the old service before deploying.

We'll throw up a status on https://status.python.org to note that test.pypi.org downloads may be temporarily impacted.

@ewdurbin ewdurbin merged commit 9851336 into main Sep 12, 2022
@ewdurbin ewdurbin deleted the terraform_test_pypi_org branch September 12, 2022 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant