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

fix nginx docker error #2486

Merged
merged 3 commits into from
Dec 5, 2024
Merged

fix nginx docker error #2486

merged 3 commits into from
Dec 5, 2024

Conversation

cp-at-mit
Copy link
Contributor

What are the relevant tickets?

NA

Description (What does it do?)

Fixes an error that Anna and I began seeing today. This error is likely related to the upgrade of nginx docker image being used, which was recently updated in #2461 and released in #2480.

This issue was previously seen when updating the nginx docker image here: https://mitodl.slack.com/archives/C02649X2P1V/p1723640539007599

It was also seen in a different project here: https://mitodl.slack.com/archives/C03K5HYGPT9/p1719507075299629?thread_ts=1719497201.536049&cid=C03K5HYGPT9

Scre

Screenshot 2024-12-04 at 2 28 41 PM enshots (if appropriate):

How can this be tested?

Docker containers should build and run as expected.

@cp-at-mit cp-at-mit requested a review from annagav December 4, 2024 19:32
@cp-at-mit cp-at-mit marked this pull request as ready for review December 4, 2024 20:31
Copy link
Contributor

@annagav annagav left a comment

Choose a reason for hiding this comment

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

👍

@cp-at-mit cp-at-mit merged commit 8e549c5 into main Dec 5, 2024
7 checks passed
@cp-at-mit cp-at-mit deleted the fix-nginx-docker-error branch December 5, 2024 14:22
@odlbot odlbot mentioned this pull request Dec 5, 2024
3 tasks
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.

2 participants