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

Plugins support #21

Merged
merged 7 commits into from
Nov 29, 2019
Merged

Plugins support #21

merged 7 commits into from
Nov 29, 2019

Conversation

blackandred
Copy link
Contributor

No description provided.

@blackandred blackandred self-assigned this Nov 27, 2019
@blackandred blackandred changed the title WIP: Plugins support Plugins support Nov 29, 2019
@blackandred blackandred merged commit c0275f7 into master Nov 29, 2019
@@ -1,9 +1,11 @@
#!/bin/bash

set -e
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This causes the script to finish early when python /checkdb.py finishes with a non-0 exit code, meaning new installs fail.

I can't figure out whether that's expected, and whether there's missing documentation for bootstrapping, because the PR has no description, documentation, or visible testing. Is there a public ticket? Or is this an error?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will have a look on this one, thank you.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right in this point. We actually miss a team and a code review. One person cannot be a developer, self code reviewer and a tester ;)

I introduced the RC versioning, so the testing should be possible easier now, but we miss people 😉

Copy link
Contributor Author

@blackandred blackandred Dec 28, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I resolved the issue in:
b35d22d

Also now to test the clear environment we could do:

make clear start

That would delete all volumes to make sure the application will run from scratch.

The second way of testing should be to stop the environment and make start without clearing anything to see if it gets up second time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm releasing a 1.0.1 as a hotfix release. Thank you for the report.

blackandred added a commit that referenced this pull request Dec 28, 2019
blackandred added a commit that referenced this pull request Dec 28, 2019
@blackandred blackandred deleted the plugins-support branch December 28, 2019 10:20
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.

2 participants