OpenTreeOfLife / ot-ansible

BSD 2-Clause "Simplified" License
0 stars 0 forks source link

Fix webapp configs #32

Closed jimallman closed 1 year ago

jimallman commented 1 year ago

Cleanup and consolidation of Ansible configuration of our web-apps

NB - This is working on devtree, along with a companionfix-webapp-configs PR in opentree.

jimallman commented 1 year ago

This could probably use a final review for our always-on-production endpoints. If necessary, we can use both dev (default) and production versions of a method, by defining two different endpoints in our config, like so (and using both names as appropriate in the web-app code):

# a typical endpoint, on both default/dev and production
getContextForNames_url = {default_apis}/v3/tnrs/infer_context
PRODUCTION_getContextForNames_url = {production_apis}/v3/tnrs/infer_context

# and this is a cached call for both default/dev and production
getSynthesisSourceList_url = {CACHED_default_apis}/v3/tree_of_life/about
PRODUCTION_getSynthesisSourceList_url = {CACHED_production_apis}/v3/tree_of_life/about
snacktavish commented 1 year ago

This branch was merged into development - can we delete this branch "fix-webapp-configs" and work from development?

snacktavish commented 1 year ago

I merged most of these into development, and used that to deploy production phylesystem-api! Can we move the webhook deploy commit to a new branch off of development?

jimallman commented 1 year ago

Can we move the webhook deploy commit to a new branch off of development?

Agreed. Moving these changes to a new branch, and closing this one.