-
Notifications
You must be signed in to change notification settings - Fork 95
Update README.md #1134
Update README.md #1134
Conversation
Adding uninstallation instruction before installing managed plugin
README.md
Outdated
@@ -49,10 +49,19 @@ esxcli software vib install -v /tmp/<vib_name>.vib | |||
Make sure you provide the **absolute path** to the `.vib` file or the install will fail. | |||
|
|||
### On Docker Host (VM) | |||
#### Managed Plugin | |||
Please make sure to uninstall older releases of DEB/RPM if installed using following commands. |
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.
Remove "if installed"
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.
Done!
README.md
Outdated
#### Managed Plugin | ||
Please make sure to uninstall older releases of DEB/RPM if installed using following commands. | ||
``` | ||
-sudo dpkg -P docker-volume-vsphere # Ubuntu or deb based distros |
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.
Why "-P" and not "-r"? As per documentation "-P" removes conf files as well. Managed plugin uses same file.
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.
I see, make sense.
README.md
Outdated
Please make sure to uninstall older releases of DEB/RPM if installed using following commands. | ||
``` | ||
-sudo dpkg -P docker-volume-vsphere # Ubuntu or deb based distros | ||
-sudo rpm -e docker-volume-vsphere # Photon or rpm based distros |
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.
Remove "-" prefix from commands here and other places.
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.
done!
Addressing Prashant's comment.
Adding uninstallation instruction before installing managed plugin