Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

feat: expose version+revision in producer and exm #898

Merged
merged 2 commits into from
Oct 22, 2020

Conversation

obdulia-losantos
Copy link
Collaborator

@obdulia-losantos obdulia-losantos commented Oct 22, 2020

To identify easily the deployed containers.

To keep the same endpoints as in the rest of services, added in producer:

  • /health returns 200 always
  • /check-app returns 200 and the current version and revision
  • /check-app/aether-kernel the same behavior as /kernelcheck but added because it's the same url that odk, kernel-ui and gather use to check kernel connection.

shawnsarwar
shawnsarwar previously approved these changes Oct 22, 2020
@obdulia-losantos obdulia-losantos merged commit ba88878 into develop Oct 22, 2020
@obdulia-losantos obdulia-losantos deleted the feat-version branch October 22, 2020 13:16
shawnsarwar pushed a commit that referenced this pull request Oct 22, 2020
* fix: we were accidentally discarding the errors section and setting is_extracted -> true when all entities failed to be created. (#896)

* feat: expose version+revision in producer and exm (#898)

* feat: expose version+revision in producer and exm

* fix: missing quote

Co-authored-by: Obdulia Losantos <obdulia.losantos@ehealthnigeria.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants