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

Add scripts/setup_covers1.sh #4064

Closed
wants to merge 3 commits into from
Closed

Add scripts/setup_covers1.sh #4064

wants to merge 3 commits into from

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Nov 10, 2020

Related to #4053, #4054, #4055

Similar to #3938 but for covers, instead of for web

Allow us to convert ol-covers1 with a fresh Ubuntu install into an Open Library covers node running Python 3.8 on Docker.

  • Does covers use memcached? A: YES.
  • Does covers need access to olsystem?
  • If so, could/should it be a targeted subset of olsystem?
  • Does covers need access to infogami?
  • Enable Sentry via Add Sentry to Coverstore #4055
  • scripts/setup_covers1.sh and scripts/setup_web1.sh are extremely similar:
    • Should we have just one script: SERVICE=covers scripts/setup_ol_server.sh
    • Can/should we rsync /opt/olsystem from ol-home?
    • Can/should we rsync /opt/openlibrary from ol-home?
    • Can/should we rsync /opt/openlibrary/vendor/infogami from ol-home?
  • Do we want to comment out sudo docker-compose logs -f --tail=100 $SERVICE ?

Technical

Testing

Screenshot

Stakeholders

@cclauss cclauss added Module: Cover Service Cover Store (book covers service) Theme: Upgrade to Python 3 labels Nov 10, 2020
@cclauss cclauss added this to the Active Sprint milestone Nov 10, 2020
@cclauss
Copy link
Contributor Author

cclauss commented Nov 11, 2020

Closing in favor of #4072

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: Cover Service Cover Store (book covers service) python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants