Skip to content

remove unnecessary stuff for EB#3040

Merged
emmachughes merged 1 commit intomasterfrom
remove-eb-stuff
Mar 13, 2025
Merged

remove unnecessary stuff for EB#3040
emmachughes merged 1 commit intomasterfrom
remove-eb-stuff

Conversation

@emmachughes
Copy link
Contributor

  1. Volumes are not persistent on EB. S3 is used for storage.
  2. The startup script was copied from the production example, and only sets up CA for first time use. Since it stores the first-time-setup-completed file in the non-persistent volume, this would result in CA being added over and over every time new instances were created.
  3. nginx does not support HTTP/3, so listening on 443/udp is unnecessary.

@github-actions github-actions bot added Component: Content Author All issues that affect content author (sourcecode/apis/contentauthor) Component: Hub The core of Edlib - handling search, filtering, my content and shared content labels Mar 13, 2025
@Kamelon7
Copy link
Contributor

Will potential future deploys require CA to be added manually after first deploy?

@emmachughes
Copy link
Contributor Author

Yes. There are a number of ways this could be solved, but I think we should cross that bridge when we get there. For now, let's just remove the script to prevent CA being added over and over.

@emmachughes emmachughes merged commit 466a31e into master Mar 13, 2025
11 checks passed
@emmachughes emmachughes deleted the remove-eb-stuff branch March 13, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Content Author All issues that affect content author (sourcecode/apis/contentauthor) Component: Hub The core of Edlib - handling search, filtering, my content and shared content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants