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

Create Thumbnail Doesn't Work (Demo) #5743

Closed
1 task
joncameron opened this issue Mar 22, 2024 · 5 comments
Closed
1 task

Create Thumbnail Doesn't Work (Demo) #5743

joncameron opened this issue Mar 22, 2024 · 5 comments
Labels

Comments

@joncameron
Copy link
Contributor

joncameron commented Mar 22, 2024

Description

On Terraform-based Avalon instances like the demo.avalonmediasystem.org, thumbnail images are unable to be set. Using the modal to select a new poster/thumbnail seems to hit a 500 error on the POST to /still for a masterfile.

To reproduce

Done Looks Like

  • Poster/thumbnail image update is successful when selecting the "Update Poster Image" button from the "Create Thumbnail" modal
@elynema
Copy link
Contributor

elynema commented Apr 12, 2024

nginx caching on thumbnails and poster images, but nothing to expire cache when these are updated in the application. This is impacting terraform sites. After 30 minutes, when the cache expires, the new thumbnails should show up.

@cjcolvar
Copy link
Member

The caching layer is setup in avalon-docker on the aws_min branch which is only used by avalon-terraform.

Here's the config:
https://github.com/avalonmediasystem/avalon-docker/blob/aws_min/cache.conf
and the nginx docs for these directives:
nginx.org/en/docs/http/ngx_http_proxy_module.html

We should figure out how to expire the cache for thumbnail/posters or maybe consider disabling the cache until we can figure out a better solution.

@cjcolvar
Copy link
Member

On the demo server I commented out the caching of poster and thumbnail images and restarted the cache container. Updating a thumbnail/poster took effect immediately instead of having to wait 30 minutes. If this feels like a long term solution then we could make the change in avalon-docker. If not, I can explore different cache expiration approaches.

@joncameron
Copy link
Contributor Author

@cjcolvar What do you think would be best? I'm fine with commenting out lines for the Docker install if that resolves the issue in that enviroment. I can create a new issue either way.

@cjcolvar
Copy link
Member

cjcolvar commented Jun 5, 2024

I'll make this change in avalon-docker and we can revisit caching another time. This change will be on the aws_min branch so it only affects the Terraform environment.

Completed here: avalonmediasystem/avalon-docker@5b34058

@cjcolvar cjcolvar closed this as completed Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants