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

Deployed NPDA loses connection to the database after a few days #207

Closed
mbarton opened this issue Jul 22, 2024 · 1 comment · Fixed by #242
Closed

Deployed NPDA loses connection to the database after a few days #207

mbarton opened this issue Jul 22, 2024 · 1 comment · Fixed by #242
Assignees
Labels
bug Something isn't working high high priority (max 3 items)

Comments

@mbarton
Copy link
Member

mbarton commented Jul 22, 2024

We need to port across this fix from E12: rcpch/rcpch-audit-engine#1004

It's not obvious how to to run this in Azure Container Apps. There are scheduled jobs which could write the token to the same Azure file share we store our config. But you configure them totally separately from the apps so the update job would have it's own managed identity and the main app would end up using a token from a different managed identity. They could perhaps share a user managed identity?

We could alternatively update the token on a cron within the docker container or a thread within the app.

@mbarton mbarton added bug Something isn't working high high priority (max 3 items) labels Jul 22, 2024
@mbarton
Copy link
Member Author

mbarton commented Jul 29, 2024

I think I’m leaning towards a thread to solve this as then it doesn’t matter how we deploy the container

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high high priority (max 3 items)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant