-
Notifications
You must be signed in to change notification settings - Fork 28
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
Strategy for shipping new monarch-ui #91
Comments
It looks like a nice weekend project |
I believe we have now completed Phase 4 of #91
|
@cmungall @monicacecilia @mellybelly @iimpulse @kshefchek |
This is to allow us to maintain a small number of rest endpoints supported by our legacy api - simsearch, compare, score, {id}.json. We have not communicated with our users in any way that we will be removing support, so that's the first step. I agree we should strip away the front end to avoid confusion, and make it purely a rest API with those functions. |
My sense is the sooner you do that the better. just suggest migrating to biolink and provide a timetable of one to two weeks and to contact you if a problem.
Nathan
… On Jul 10, 2019, at 9:53 AM, Kent Shefchek ***@***.***> wrote:
This is to allow us to maintain a small number of rest endpoints supported by our legacy api - simsearch, compare, score, {id}.json. We have not communicated with our users in any way that we will be removing support, so that's the first step. I agree we should strip away the front end to avoid confusion, and make it purely a rest API with those functions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I think if it purely switching URLs to biolink it would be good and well, but the endpoints have different arguments and json responses, I think some calls like {id}.json now require going to two different endpoints for the same data - I just see it potentially taking users longer to change their code. And it takes little resourcing from us to run a tiny nodejs app. @monicacecilia do we have anyone tasked with emailing our users? |
Hi @kshefchek. You and I chatted about communicating with our users a little while ago; see also exchanged messages in this ticket (https://github.com/monarch-initiative/operations/issues/111). First, we need a list of users and contact information. |
@nathandunn @tomc @kshefchek and @DoctorBud had an opportunity to meet in Eugene yesterday and we came up with a plan that would allow us to move the monarch-ui to production at https://monarchinitiative.org, while satisfying constraints like
https://monarchinitiative.org/<id>.json
for legacy API usershttps://monarchinitiative.org/<type>/<id>.json
for legacy API usershttps://monarchinitiative.org/score
for legacy API usershttps://monarchinitiative.org/??owlsim??
for legacy API users (@kshefchek what is the other legacy API endpoint folks might be using?)I've refined the plan we discussed so that it is more symmetrical and beta vs production distinctions are clearer. In addition, this strategy doesn't require any complex DNS-swap like I was proposing yesterday.
Phase 1a - Create legacy.monarchinitiative.org
HAProxy
to direct traffic forlegacy.m.org
to the current production VM containingm.org
.legacy.m.org
(is this true @kshefchek @Kennric?)Phase 1b - Create legacybeta.monarchinitiative.org
HAProxy
to direct traffic forlegacybeta.m.org
to the current beta VM containingbeta.m.org
.legacybeta.m.org
(is this true @kshefchek @Kennric?)Phase 1c - Create preview.monarchinitiative.org
HAProxy
to direct traffic forpreview.m.org
to the current production VM containingm.org
.preview.m.org
(is this true @kshefchek @Kennric?)Phase 2 - Update monarch-app-beta VM with latest legacy monarch-app
Phase 3 - Verify that legacy.m.org and legacybeta.m.org work
nginx
conf on beta/prod will need to be adjusted so that those VMs can serve both monarch-app and monarch-ui.Phase 4 - On beta, use
nginx
to route legacy paths to legacybeta.m.orgnginx
so that a request asking for https://beta.monarchinitiative.org/MONDO:0007947.json, a legacy API, will NOT end up running the new monarch-ui, but will instead be transparently routed to https://legacybeta.monarchinitiative.org/MONDO:0007947.json/MONDO:0007947.json
,/disease/MONDO:0007947.json
,/score
, and??owlsim??
.Phase 5 - Install monarch-ui on production as preview.m.org
Take a breath
Phase 5 - Ribbon-cutting Ceremony ... 'Make it So!'
The text was updated successfully, but these errors were encountered: