Bug 991387
Summary: | Should not show succeed when erase an existing cartridge in /usr/libexec/openshift/cartridges via oo-admin-cartridge | ||
---|---|---|---|
Product: | OpenShift Online | Reporter: | Yan Du <yadu> |
Component: | Containers | Assignee: | Jhon Honce <jhonce> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | libra bugs <libra-bugs> |
Severity: | low | Docs Contact: | |
Priority: | medium | ||
Version: | 2.x | CC: | mmasters, pmorie, xtian |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-01-30 00:48:17 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Yan Du
2013-08-02 10:16:12 UTC
Erase will only remove versions of cartridges not included in /usr/libexec/openshift/cartridges If that is not your case please reopen. test on devenv-stage_437 1. cartridge can be erased if the cartridge version is not included in /usr/libexec/openshift/cartridges 2. should not show "succeeded" when trying to erase the cartridge which is still included in /usr/libexec/openshift/cartridges, a friendly message show up will be perfect. Thanks. *** Bug 998026 has been marked as a duplicate of this bug. *** oo-admin-cartridge removes/adds files from the cartridge repository in /var/lib/openshift/.cartridge_repository. If you attempt to remove versions of cartridges not installed in the repository error messages will be generated. Every restart of mcollective causes the cartridge repository to be reinstalled. test on devenv_3677 issue still can be reproduced. steps: 1. modified the version in vim /usr/libexec/openshift/cartridges/zend/metadata/manifest.yml (eg: change "Cartridge-Version: 0.0.3" to "Cartridge-Version: 0.0.1") 2. oo-admin-cartridge -a install -s /usr/libexec/openshift/cartridges/zend 3. oo-admin-cartridge -l (redhat, zend, 5.6, 0.0.1) (redhat, zend, 5.6, 0.0.3) 4. even the version is included in /usr/libexec/openshift/cartridges/zend/metadata/manifest.yml, try to erase it. [root@ip-10-178-12-200 ~]# oo-admin-cartridge --action erase --name zend --version 5.6 --cartridge_version 0.0.1 succeeded 5. use oo-admin-cartridge -l to check, found the zend 5.6 version 0.0.1 is still existed (redhat, zend, 5.6, 0.0.1) (redhat, zend, 5.6, 0.0.3) if the cartridge can't be removed when it is included, then in step4, should not show succeeded, maybe should show something else, like "included cartrdige can't be removed." showing succeeded is a little confusing, since the cartridge can't be removed (result in step5). Thanks As long as the command "oo-admin-cartridge" goes into the "install" process, it will return a "succeeded". [root@ip-10-185-58-142 cgroup]# oo-admin-cartridge -a install -s /usr/libexec/openshift/cartridges/abc install failed for /usr/libexec/openshift/cartridges/abc: Illegal path to cartridge source: '/usr/libexec/openshift/cartridges/abc' succeeded This could be confusing. Commit pushed to master at https://github.com/openshift/origin-server https://github.com/openshift/origin-server/commit/d4e4f322a1a0185df57f434c20e02d16554b8545 Fix bug 991387: Don't allow erase of carts in the cart_base_path Test on devenv_4003, when install/erase an invalid cartridge, will not show succeed message again. [root@ip-10-179-31-109 ~]# oo-admin-cartridge -a install -s /usr/libexec/openshift/cartridges/abc install failed for /usr/libexec/openshift/cartridges/abc: Illegal path to cartridge source: '/usr/libexec/openshift/cartridges/abc' installation failed [root@ip-10-179-31-109 ~]# oo-admin-cartridge --action erase --name zend --version 5.6 --cartridge_version 0.0.1 requested cartridge does not exist: (key not found: (zend, 5.6, 0.0.1)) [root@ip-10-179-31-109 ~]# oo-admin-cartridge --action erase --name zend --version 5.6 --cartridge_version 0.0.6 Couldn't erase cartridge: (Cannot erase cartridge installed in CARTRIDGE_BASE_PATH) |