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 status badges/links to app UIs #3210

Closed
3 tasks done
jonrkarr opened this issue Oct 12, 2021 · 6 comments · Fixed by #3237
Closed
3 tasks done

Add status badges/links to app UIs #3210

jonrkarr opened this issue Oct 12, 2021 · 6 comments · Fixed by #3237
Assignees

Comments

@jonrkarr
Copy link
Member

jonrkarr commented Oct 12, 2021

  • Pull status from new /health endpoints
  • Bottom of all pages
  • Main menus of apps
@jonrkarr
Copy link
Member Author

@bilalshaikh42 can we get status badges from Upptime?

@bilalshaikh42
Copy link
Member

bilalshaikh42 commented Oct 12, 2021

Sure. We can get badges for essentially any url. Though I'm not sure what an uptime badge for the readme or docs would convey. Those are statically hosted by GitHub, so should always be up.

I think I'm missing some context here

@jonrkarr
Copy link
Member Author

jonrkarr commented Oct 14, 2021

Ideally, I think we'd make the static UI apps (simulators, dispatch, platform) clearly display a warning message if the status is down. The upptime app already stores JSON documents with the status of each service. We could modify the upptime GitHub action to merge that information into 1 JSON document and then read that into the UIs. Due to CORS, I think we need to save that to GitHub pages (i.e., subdirectory of the status repo served by GitHub pages). Then I think the apps could read the status from a URL like https://biosimulations.github.io/status/upptime-summary.json.

@bilalshaikh42 thoughts?

@bilalshaikh42
Copy link
Member

Oh, I see what you mean now. I thought you meant creating buttons for the webpages, not including the info in the webpages.

The website status.biosimulations.org uses the following file to determine the current state. We could do the same

https://raw.githubusercontent.com/biosimulations/status-monitor/master/history/summary.json

@bilalshaikh42
Copy link
Member

We could modify the upptime GitHub action to merge that information into 1 JSON document and then read that into the UIs. Due to CORS, I think we need to save that to GitHub pages (i.e., subdirectory of the status repo served by GitHub pages). Then I think the apps could read the status from a URL like https://biosimulations.github.io/status/upptime-summary.json.

There is already the summary json linked above. The status front end seems to be able to make a xhr request to it without cors issues, so hopefully should work on the front end as well

@jonrkarr
Copy link
Member Author

Addressed by new status notifications at bottom of apps

biosimulations-daemon added a commit that referenced this issue Nov 4, 2021
# [6.0.0](v5.9.0...v6.0.0) (2021-11-04)

### Bug Fixes

* **api:** build fix for new axios types ([e7ea984](e7ea984))
* **dispatch-service:** add a temporary check for mistructured logs ([a67bfa1](a67bfa1)), closes [#3482](#3482) [#3482](#3482)
* **dispatch-service:** remove ssl skip when downloading archive ([b4bd2c0](b4bd2c0)), closes [#3092](#3092)
* **dispatch,api,dispatch-service:** fixed data model for exceptions in simulation run logs ([191f1d3](191f1d3))
* fix imports ([a076005](a076005))
* **simulators:** fixed text overflow of simulator test results ([7a8b11a](7a8b11a))

### Code Refactoring

* **api:** cleaned up simulation run files ([0213fb5](0213fb5))

### Features

* added endpoints for getting summaries of projects ([2e9a54e](2e9a54e))
* **api:** added database model and validation for logs ([6a3a344](6a3a344))
* **api:** added endpoints for getting individual SED elements; closes [#3439](#3439) ([8aa64fc](8aa64fc))
* **combine-api:** add dynamic module for combine api-client ([e7c2448](e7c2448)), closes [#3180](#3180)
* **datamodel:** add validation for image digests ([6106e54](6106e54))
* **dispatch-service:** enhanced tracking of processing results ([b4f01e3](b4f01e3))
* **dispatch:** updated publication form, finished switching to Endpoints ([a5bef72](a5bef72))
* **exceptions:** improve error handling ([4a3e8c7](4a3e8c7))
* **hsds:** handle transient hsds query failures ([f4a19f5](f4a19f5)), closes [#3413](#3413)
* **ontology:** added parent/child relationships to ontology terms ([4107f1d](4107f1d))
* **simulators-api:** add validation for api models ([ce2c5bb](ce2c5bb))
* **simulators,dispatch,platform:** added status bar to bottom of apps; closes [#3210](#3210) ([3630c23](3630c23))

### Reverts

* **dispatch-service:** revert using new config service to provide basepath ([e479d2e](e479d2e))

### BREAKING CHANGES

* **api:** moves simulation run file information from 'Simulation Files' collection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants