-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - follow up of PR #657 #723
Conversation
Codecov Report
@@ Coverage Diff @@
## master #723 +/- ##
=======================================
Coverage 85.32% 85.32%
=======================================
Files 64 64
Lines 3993 3993
=======================================
Hits 3407 3407
Misses 399 399
Partials 187 187 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@chombium please resolve the conflicts. |
This commit, intruduces changes to the current behavior of the "make cleandocker" command. From now on, an extra parameter must be given in order to delete the persistent volumes, which up until now was done by default. The new command which represents the old behavior (deleting all artifacts created from the docker composition) is "make pv=true cleandocker". Signed-off-by: Jovan Kostovski <chombium@gmail.com>
61957bb
to
0540450
Compare
@drasko done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Merged! Thanks @chombium ! |
This commit, intruduces changes to the current behavior of the "make cleandocker" command. From now on, an extra parameter must be given in order to delete the persistent volumes, which up until now was done by default. The new command which represents the old behavior (deleting all artifacts created from the docker composition) is "make pv=true cleandocker". Signed-off-by: Jovan Kostovski <chombium@gmail.com>
This commit, intruduces changes to the current behavior of the "make cleandocker" command. From now on, an extra parameter must be given in order to delete the persistent volumes, which up until now was done by default. The new command which represents the old behavior (deleting all artifacts created from the docker composition) is "make pv=true cleandocker". Signed-off-by: Jovan Kostovski <chombium@gmail.com>
This commit, intruduces changes to the current behavior of the "make cleandocker" command. From now on, an extra parameter must be given in order to delete the persistent volumes, which up until now was done by default. The new command which represents the old behavior (deleting all artifacts created from the docker composition) is "make pv=true cleandocker". Signed-off-by: Jovan Kostovski <chombium@gmail.com>
This PR is a followup of #657. After the @drasko's comment I remembered myself how many times we had complains that some user lost his/hers data because they were updating their Mainflux installation or recreated some of the
*-db
containers.This PR, introduces changes to the current behavior of the
make cleandocker
command.From now on, an extra parameter must be given in order to delete the persistent volumes, which up until now was done by default. The new command which represents the old behavior (deleting all artifacts created from the docker composition including the persistent volumes) is
make pv=true cleandocker
.I understand that this might be a slight problem for the @mainflux/contributors as we'll have to use a new form of the command, but I agree with @drasko's comment that is better to be on the safe side and don't delete the data, the persistent volumes by default, although we, the @mainflux/contributors might have to run the command twice if we forget the
pv=true
parameter. A command alias would to the trick ;)