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

Commander may overwrite instance while upgrading another #3910

Closed
fchavant opened this issue Jul 2, 2019 · 1 comment
Closed

Commander may overwrite instance while upgrading another #3910

fchavant opened this issue Jul 2, 2019 · 1 comment

Comments

@fchavant
Copy link
Contributor

fchavant commented Jul 2, 2019

Expected behavior

Upgrading an instance name primary does not affect the backup instance.

Actual behavior

~/.lisk/instances/backup/ gets removed by core:upgrade primary

Steps to reproduce

lisk core:install primary
lisk core:install backup
lisk core:upgrade primary

Which version(s) does this affect? (Environment, OS, etc...)

2.2.0

@ManuGowda
Copy link
Contributor

This issue is no longer valid, as the backups logic was moved to the root of .lisk so the new structure will be

.lisk/instances
.lisk/backups

Here is the logs from testing

❯ ls -l ~/.lisk/instances       
total 8
drwxr-xr-x  30 manu  staff  960 Aug 16 10:35 backups
-rw-r--r--   1 manu  staff  139 Aug 16 10:35 error.log
drwxr-xr-x  30 manu  staff  960 Aug 16 10:33 primary
                                                                                                                  
lisk-sdk/commander git/release/2.3.0  
❯ ls -l ~/.lisk/         
total 8
drwxr-xr-x   4 manu  staff  128 Aug 16 10:35 backups
-rw-r--r--   1 manu  staff  117 Aug 14 16:34 config.json
drwx------   3 manu  staff   96 Apr 12 18:00 configstore
drwxr-xr-x   5 manu  staff  160 Aug 16 10:35 instances
drwxr-xr-x   2 manu  staff   64 Apr 24 15:31 network
drwxr-xr-x  12 manu  staff  384 Aug  8 15:37 pm2
drwxr-xr-x   4 manu  staff  128 Feb 25 11:56 temp
                                                                                                                  
lisk-sdk/commander git/release/2.3.0  
❯ ls -l ~/.lisk/backups      
total 0
drwxr-xr-x  30 manu  staff  960 Aug 16 10:35 backups
drwxr-xr-x  30 manu  staff  960 Aug 16 10:33 primary

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

No branches or pull requests

2 participants