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

Support for 'package uninstall' #68

Closed
kokosing opened this issue Oct 6, 2015 · 2 comments
Closed

Support for 'package uninstall' #68

kokosing opened this issue Oct 6, 2015 · 2 comments

Comments

@kokosing
Copy link
Contributor

kokosing commented Oct 6, 2015

As there is 'package install', but no support 'package uninstall' or 'update', So there is no way to update a package.

@cawallin
Copy link
Member

'package uninstall' should be quite similar to 'package install'. It needs to uninstall a given RPM package on all of the nodes of the cluster.

A few notes:

  • presto-admin is based on Fabric (http://docs.fabfile.org/en/1.10/), and most of the annotations are from Fabric.
  • The @runs_once annotation requires some trickery to make the unit tests work correctly; check out other unit tests that use that annotation to see the way to get around it
  • In addition to unit tests, we write product tests for presto-admin. These run on a docker cluster that the tests create locally, and can be run via make test-all (optionally, you can specify the suite -- aka tests.product.test_package_install -- to run only the relevant tests). They are located in tests/product.

@kokosing
Copy link
Contributor Author

kokosing commented Jul 7, 2016

Fixed by 502de65

@kokosing kokosing closed this as completed Jul 7, 2016
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

No branches or pull requests

2 participants