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 REST API versioning support (second try) #3408

Merged
merged 9 commits into from
Oct 29, 2018

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Oct 26, 2018

Reverts #3404

Steps to test:

  • check that both /api/buildinfo and /api/v1/buildinfo reply
  • check that standalone datastore setup still works (will update guide after this PR is merged)

Changes to standalone datastore setup

add these three lines to the end of your application.conf (no changes needed in routes file anymore)

play.modules.disabled += "com.scalableminds.webknossos.datastore.DataStoreModule"
datastore.enabled = false
play.http.router = "noDS.Routes"

Changes necessary in kube

  • play.http.router should be changed to "noDS.noTS.Routes" for mhlab, as they don’t have a local datastore nor tracingstore. see Migrations.md

@fm3 fm3 self-assigned this Oct 26, 2018
@fm3 fm3 requested a review from jfrohnhofen October 26, 2018 09:36
@fm3 fm3 merged commit a1e85f1 into master Oct 29, 2018
@fm3 fm3 deleted the revert-3404-revert-3385-rest-api-versioning branch October 29, 2018 09:17
jfrohnhofen added a commit that referenced this pull request Nov 14, 2018
* master:
  remove -v (verbose) from backend-tests in CI (#3425)
  Add recommended configuration to TaskTypes (backend only) (#3415)
  Even if initialData is disabled, insert local stores (#3421)
  Add REST API docs (#3376)
  clearer message when annotation couldnt be found while logged out (#3413)
  prepare release 18.11.0 (#3414)
  Handle Cross-Datastore Duplicates, introducing scratch datastores (#3394)
  restrict user visibility (#3389)
  Add REST API versioning support (second try) (#3408)
  Enable hybrid tracings by default (#3399)
jfrohnhofen added a commit that referenced this pull request Nov 14, 2018
* isosurface:
  expose voxel dimensions as parameter, proper scale, #3313
  remove -v (verbose) from backend-tests in CI (#3425)
  Add recommended configuration to TaskTypes (backend only) (#3415)
  Even if initialData is disabled, insert local stores (#3421)
  Add REST API docs (#3376)
  clearer message when annotation couldnt be found while logged out (#3413)
  prepare release 18.11.0 (#3414)
  Handle Cross-Datastore Duplicates, introducing scratch datastores (#3394)
  restrict user visibility (#3389)
  Add REST API versioning support (second try) (#3408)
  Enable hybrid tracings by default (#3399)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants