Skip to content

Conversation

@samhotep
Copy link
Member

@samhotep samhotep commented Mar 14, 2025

Done

  • Added a cli command that deprecates assets with missing data, and replaces this data with a placeholder. This is done to prevent 404 and 503 errors when an asset's data goes missing.

QA

  • Check out this feature branch
  • Start services (if not already running) using docker compose up -d
  • Run the site using the command dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8017
  • Upload some images.
  • Stop the swift service, and start a fresh one without any volumes using
docker ps | grep swift # to get the container id
docker stop <swift-container-id>
docker run --rm -d -p 8080:8080 openstackswift/saio:latest
  • Try to access the images already uploaded. There should be 404 errors.
  • Run the following command:
dotrun exec flask database deprecate-stale-assets
  • Try to access these assets, or search for them. They should be deprecated.
  • Run through the following QA steps

Issue / Card

Fixes WD-17954

@samhotep samhotep changed the base branch from main to staging-assets-new March 14, 2025 09:30
@samhotep samhotep requested a review from akbarkz March 14, 2025 13:07
@akbarkz
Copy link
Contributor

akbarkz commented Mar 17, 2025

@samhotep We are mostly trying to solve the issue with missing assets on production. How will it be done then in terms of executing the CLI command you introduced with this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants