Skip to content

Conversation

@DXCanas
Copy link
Member

@DXCanas DXCanas commented Dec 4, 2025

Summary

Cleaning up infra-related files that no longer appear to be in use. Tried to bundle mass deletions by subject so we can go back for history.

Of Historical Note:

  • Probers
  • PR Deployments

References

#5520
https://github.com/learningequality/infrastructure/issues/853

Reviewer guidance

It might be clearer to go commit-by-commit rather than seeing the full file changelog.

Really, I'm asking for y'all to make sure there are no breaking changes for the local dev workflow (I will deal with consequences to infra-side)

@DXCanas DXCanas requested review from bjester and rtibbles December 4, 2025 23:51
@DXCanas DXCanas force-pushed the studio-infra-file-cleanup branch from cddc677 to aae5494 Compare December 4, 2025 23:52
Comment on lines 11 to 13
# Really seems like it _should_ be here, as it's referenced by `nginx.conf`.
# But it's hasn't been for years.
# COPY ${SRC_DIR}/mime.types /etc/nginx/mime.types
Copy link
Member Author

Choose a reason for hiding this comment

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

Odd thing I found 🤷

…ndard.

And deleting defunct cloudbuild prod yaml
Making a separate commit for bookmarking purposes. Seems we used to deploy to a separate/distinct dev cluster, and create new postgres instances for every single pr.
Standardizing on image-specific docker locations.
Will delete after both have been updated.
@DXCanas DXCanas force-pushed the studio-infra-file-cleanup branch 3 times, most recently from 0de1409 to 0c39da8 Compare December 5, 2025 22:09
@DXCanas
Copy link
Member Author

DXCanas commented Dec 5, 2025

It doesn't look it, but I did make sure to register file movements, renames, etc with git. In GH, the file history looks incomplete. But if you blame by line, things should look the way we expect.

@DXCanas DXCanas marked this pull request as ready for review December 5, 2025 22:34
@DXCanas DXCanas changed the base branch from unstable to hotfixes December 5, 2025 22:37
@DXCanas DXCanas force-pushed the studio-infra-file-cleanup branch from 645e5a5 to 1394aba Compare December 5, 2025 22:41
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.

2 participants