Bug 830309 - Add ssh_url to application REST object so that clients don't need to make it up
Summary: Add ssh_url to application REST object so that clients don't need to make it up
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Master
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Abhishek Gupta
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-08 20:00 UTC by Clayton Coleman
Modified: 2015-05-15 00:51 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-25 18:27:32 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Clayton Coleman 2012-06-08 20:00:38 UTC
Two stories this sprint need the SSH URL.  We can currently get it by parsing the git_url on application and then returinng userinfo, host, and port, but this is potentially broken in the future if those URLs change meaning (if we switch to git protocol urls).

Add "ssh_url" to application.  Fairly important for US2339 and US2276, please prioritize accordingly.

Comment 1 Clayton Coleman 2012-06-08 20:07:44 UTC
To clarify - ssh://<userid>@<host>[:<port>] is what i was looking for - the client can be responsible for transcribing the URI into command line format if necessary.

Comment 2 Abhishek Gupta 2012-06-16 01:58:30 UTC
Fixed and waiting for merge
https://github.com/openshift/crankcase/pull/136

Comment 3 Xiaoli Tian 2012-06-19 11:11:42 UTC
Verified it on devenv_1857,ssh_url is listed 
curl -k -H "Accept: application/json" --user "xtian+b101:123456"  https://localhost/broker/rest/domains/domainbill004/applications/phpapp0/

...............
{
    "domain_id": "domainbill004",
    "git_url": "ssh://f21b3c7c0da74022815e8fa6dd3783e7.rhcloud.com/~/git/phpapp0.git/",
    "uuid": "f21b3c7c0da74022815e8fa6dd3783e7",
    "scale_min": 1,
    "ssh_url": "ssh://f21b3c7c0da74022815e8fa6dd3783e7.rhcloud.com",
    "health_check_path": "health_check.php"
  },


Note You need to log in before you can comment on or make changes to this bug.