Bug 1020365
Summary: | [deploy] Providing invalid Git ref to `rhc deploy` corrupts app-deployments and breaks application | ||||||
---|---|---|---|---|---|---|---|
Product: | OpenShift Online | Reporter: | Dan Mace <dmace> | ||||
Component: | Containers | Assignee: | Dan Mace <dmace> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | libra bugs <libra-bugs> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | 2.x | CC: | bmeng, jkeck, 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: | 2014-02-26 19:08:02 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: | |||||||
Bug Depends On: | 1019646 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Dan Mace
2013-10-17 14:12:40 UTC
Created attachment 813462 [details]
Check git ref before running deploy
A straightforward approach is to check that the given git ref is a valid one before deploying. This patch does the job.
However, this is apparently insufficient. For some reason the error message is not getting on the extended SSH data channel. As a result, 'rhc' command still thinks that the deploy command succeeded:
$ bx bin/rhc deploy foobar -a rack
Deployment of git ref 'foobar' in progress for application rack ...
Git ref foobar is invalid
Success
https://github.com/openshift/origin-server/pull/3916 NOTE: Please test this issue using ssh commands directly against the gear, e.g. `ssh user@host "gear deploy bogusref"` since a current rhc bug will report success even when the gear command reports failure: https://bugzilla.redhat.com/show_bug.cgi?id=1020473 *** Bug 1020862 has been marked as a duplicate of this bug. *** Adding Bug 1019646 as dependency, due to SSH handling of 'rhc'. $ ssh 52650b4c2b21cea5e0000190.rhcloud.com "gear deploy aaaassss" Git ref aaaassss is invalid Checked on devenv_3921, issue has been fixed. |