-
Notifications
You must be signed in to change notification settings - Fork 15
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
Improvements v2 #23
Comments
Actual env files that are required by every service |
Anvansh to pick up enhancements. |
Marking the enhancements for which I have created a PR here. Still unable to link issues and PRs to the original checklist.
|
@tushar5526 to review these PRs and merging. |
Previous PRs merged by @tushar5526 @RyanWalker277 to raise 1 PR today. |
Raised the PR yesterday, synced dev with main and also completed work on some existing PRs and got them merged. Currently working on #58, since I am not that familiar with ODK, it might take a while to figure out the exact cause of the issue, will talk to @pankajjangid05 to figure out exact issue. Had a talk with yesterday with @tushar5526 and @chinmoy12c regarding the same. |
There are 3 containers related to ODK, namely The the aggregate-db container works fine. I can log in with the same creds as given in |
Had a talk with @tushar5526, the issue is with the aggregate-server, admin creds are not picked up from .env. Looking into it. Will send a patch soon. |
PR raised on odk (samagra-comms/odk#6) for #58. |
@tushar5526 to verify PR. |
Improvements are almost closed. @RyanWalker277 to present macro wins with these changes. |
Fix:
Enhancements
db
. Service owners can clarify here.github
token that has read access to packages in install script after cloning uci-apis Ask for github token that has read access to packages in install script after cloning uci-apis #47Good to have
VAULT_
variable name - Fix variable name utils#34docker-compose-swarm.yml
anddb-compose
from Dockerswarm servers for users who want to deploy it as a swarm. stateless and stateful services are already segregated.The text was updated successfully, but these errors were encountered: