From 0a18cc27c4286a8452d96dd019d3018394443918 Mon Sep 17 00:00:00 2001 From: "A.Unger" Date: Tue, 23 Feb 2021 13:40:35 +0100 Subject: [PATCH] add readme --- .../unreleased/fix-accounts-service-initialization.md | 9 +++++++++ 1 file changed, 9 insertions(+) create mode 100644 changelog/unreleased/fix-accounts-service-initialization.md diff --git a/changelog/unreleased/fix-accounts-service-initialization.md b/changelog/unreleased/fix-accounts-service-initialization.md new file mode 100644 index 00000000000..1bcb5a070f3 --- /dev/null +++ b/changelog/unreleased/fix-accounts-service-initialization.md @@ -0,0 +1,9 @@ +Bugfix: Fix accounts initialization + +Originally the accounts service relies on both the `settings` and `storage-metadata` to be up and running at the moment it starts. This is an antipattern as it will cause the entire service to panic if the dependants are not present. + +We inverted this dependency and moved the default initialization data (i.e: creating roles, permissions, settings bundles) and instead of notifying the settings service that the account has to provide with such options, the settings is instead initialized with the options the accounts rely on. Essentially saving bandwith as there is no longer a gRPC call to the settings service. + +For the `storage-metadata` a retry mechanism was added that retries by default 20 times to fetch the `com.owncloud.storage.metadata` from the service registry every `500` miliseconds. If this retry expires the accounts panics, as its dependency on the `storage-metadata` service cannot be resolved. + +https://github.com/owncloud/ocis/pull/1696a