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

Hotfix: Vacant properties pmtiles missing #709

Conversation

CodeWritingCow
Copy link
Collaborator

Description

This PR fixes a production issue with the map failing to load because vacant_properties_tiles.pmtiles is missing from Google Cloud. As a temporary fix, the PR updates the map to load vacant_properties_tiles_staging.pmtiles until the missing file is restored.

Screenshots

User reported error this morning;

Screen Shot 2024-06-25 at 10 43 14 AM

Live website stuck on spinning loader while getting 404 for missing pmtiles (prod version)
Screen Shot 2024-06-25 at 11 41 15 AM

Website in local env loads correctly when getting pmtiles (staging version)
Screen Shot 2024-06-25 at 11 41 50 AM

Load map file's staging version as temporary fix
for issue of production version file missing
from Google Cloud.
Copy link

vercel bot commented Jun 25, 2024

@CodeWritingCow is attempting to deploy a commit to the Clean and Green Philly Team on Vercel.

A member of the Team first needs to authorize it.

@CodeWritingCow
Copy link
Collaborator Author

Closing this PR as @brandonfcohen1 has fixed the problem on Google Cloud.

Currently, when we run the Python pipeline, it uploads the tiles to Google Cloud as vacant_properties_tiles_staging.pmtiles. We have to rename it to vacant_properties_tiles.pmtiles in the cloud bucket. For more information, see https://github.com/CodeForPhilly/clean-and-green-philly/blob/main/docs/SETUP/BACK_END.md#google-cloud-gcp

@CodeWritingCow CodeWritingCow deleted the codewritingcow/hotfix-vacant_properties_tiles-missing branch June 25, 2024 17:42
@CodeWritingCow CodeWritingCow added the bug Something isn't working label Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant