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

Feature/atomic database creation 2 #19

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

hkernbach
Copy link
Member

@hkernbach hkernbach commented Aug 23, 2019

added test to check supervision cleanup job in case of coordinator crash and coordinator reboot

IMPORTANT:: This change needs to be available in the arangodb-instance-manager module. Otherwise, the test will not even start.

For development purpose, you can apply that diff just inside the node_modules folder.

commit 7522d400b2393f60ca138d2d8df621b0d6953da3 (HEAD -> master)
Author: hkernbach <>
Date:   Fri Aug 23 14:32:47 2019 +0200

    bug-fix/added-fishbowl-collection

diff --git a/src/InstanceManager.ts b/src/InstanceManager.ts
index f50b8dc..fbadfac 100644
--- a/src/InstanceManager.ts
+++ b/src/InstanceManager.ts
@@ -638,6 +638,7 @@ export default class InstanceManager {
       "_appbundles",
       "_apps",
       "_aqlfunctions",
+      "_fishbowl",
       "_frontend",
       "_graphs",
       "_analyzers",

@hkernbach hkernbach requested a review from kvahed August 23, 2019 12:48
@hkernbach
Copy link
Member Author

Execute with: yarn test -- --grep "Database"

@hkernbach
Copy link
Member Author

PR: arangodb-helper/arangodb-instance-manager#5
(not merged yet)

@hkernbach
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants