| 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: | |||||
| 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. |