Skip to content

Alter Dockerfile to run the api and pre-create indices for es#10

Open
teferi wants to merge 1 commit intowriter_apifrom
docker
Open

Alter Dockerfile to run the api and pre-create indices for es#10
teferi wants to merge 1 commit intowriter_apifrom
docker

Conversation

@teferi
Copy link
Collaborator

@teferi teferi commented Dec 16, 2016

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling c1b1043 on docker into 926a517 on storage.

runbook/main.sh Outdated
#!/bin/bash

# ensure indices for runbooks exist
runbook-init-storage
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should split Runbooks to 2 kind of APIs:

  • Readonly API
  • Write only API
  • Runbook Runner API (worker)

Runbook Runner & Write API may have to run on site (because of security/network reasons)
Upgrades & indexes checks should be done inside write API part

If we split in this way we also have smooth workflow for upgrades:

  • Turn all Write API
  • Run index upgrade (Run books & Read API should not be affected)
  • Turn all Runners API
  • Perform upgrades for all runs data (Write API & Read API are not affected), however users can't execute runbooks for specific site
  • Rolling upgrades of all read APIs

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 72.115% when pulling cc3efa0 on docker into 68a426b on writer_api.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 72.115% when pulling 53ea7dd on docker into 156f310 on writer_api.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 66.071% when pulling f7f5bdd on docker into 323ff03 on writer_api.

@teferi teferi force-pushed the writer_api branch 2 times, most recently from ded2b48 to 9d9e3a9 Compare December 26, 2016 15:07
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 66.192% when pulling a807f1d on docker into 9d9e3a9 on writer_api.

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.

3 participants