Bug 999189

Summary: If gear upgrade extension version does not match supplied version, it should not be loaded.
Product: OpenShift Online Reporter: Paul Morie <pmorie>
Component: ContainersAssignee: Paul Morie <pmorie>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: low Docs Contact:
Priority: medium    
Version: 2.xCC: bmeng, chunchen, jhonce, xtian
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-08-29 12:54:10 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 Paul Morie 2013-08-20 21:52:56 UTC
Gear upgrade extension should be a no-op if it's for the wrong version (current behavior is to raise an exception).

Comment 1 Paul Morie 2013-08-21 16:51:59 UTC
PR submitted to master.

Comment 2 openshift-github-bot 2013-08-21 18:01:46 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/f7e50e2165cfe1e74ff5386179de3a6c1cbad73b
Fix bug 999189: gear upgrade extension is a no-op if release version doesn't match supplied version

Comment 4 Meng Bo 2013-08-22 10:32:22 UTC
Tested on devenv-stage_448, the patch should work fine.

# oo-admin-upgrade upgrade-gear --version 3.0.32 --app-name appa --login bmeng --upgrade-gear=5215e6c499055c0995000002

       "log": [
            "Version mismatch between supplied release version (3.0.32) and extension version (2.0.32", 

But there missing a ")" in the end of the message.

Assign it back to fix the typo.

Comment 5 Paul Morie 2013-08-22 14:42:38 UTC
PRs submitted to master and stage for this.

Comment 7 Meng Bo 2013-08-23 10:00:38 UTC
"log":["Version mismatch between supplied release version (2.0.33) and extension version (2.0.32)"

The typo fixed on devnev-stage_452.