Use bzr+ssh in branch upload URLs

Bug #133599 reported by Jonathan Lange
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Tim Penhey

Bug Description

Currently we support bzr+ssh but don't advertise it on branch web pages.

Once we get Bazaar 0.91 on Launchpad (and thus get improved streaming code which solves performance problems with tarballs), we should make a song and dance about this.

The minimum song and dance is changing the upload URL displayed on branch pages to use bzr+ssh instead of sftp.

Tags: lp-code
Jonathan Lange (jml)
Changed in launchpad-bazaar:
importance: Undecided → Medium
Revision history for this message
David Allouche (ddaa) wrote :

How should we help Ubuntu users (dapper, edgy, feisty) whose packaged bzr does not support bzr+ssh?

Revision history for this message
Jonathan Lange (jml) wrote :

Provide both URLs, perhaps?

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I think the placement of the upload URL could be improved at the same time. Because it is not in the same place as the download URL, I had trouble finding it.

So, maybe something like ...
------------
== For Bazaar 0.11 and later ==

Download URL: bzr+ssh://bazaar.launchpad.net/~foo/bar/hum
     Example: bzr pull http://bazaar.launchpad.net/~foo/bar/hum

  Upload URL: bzr+ssh://<email address hidden>/~foo/bar/hum
     Example: bzr push --remember bzr+ssh://<email address hidden>/~foo/bar/hum

== For all versions of Bazaar (slower) ==

Download URL: http://bazaar.launchpad.net/~foo/bar/hum
     Example: bzr pull http://bazaar.launchpad.net/~foo/bar/hum

  Upload URL: sftp://<email address hidden>/~foo/bar/hum
     Example: bzr push --remember sftp://<email address hidden>/~foo/bar/hum
------------

Adjust the commands as desired. Use <tr class="lesser"> for the examples.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I was vaguely under the impression that we didn't want to advertise these URLs until we're using bzr 0.91. Which isn't going to happen in 1.1.9.

Revision history for this message
Tim Penhey (thumper) wrote :

OK to advertise pushing.

And given that the default pull / branch location is going to be http, I think we're ok putting this in.

Changed in launchpad-bazaar:
assignee: nobody → thumper
status: New → Confirmed
Revision history for this message
David Allouche (ddaa) wrote :

This should probably not be done until the smart server can stream tarball, to avoid overloading the server.

Tim Penhey (thumper)
Changed in launchpad-bazaar:
status: Confirmed → In Progress
Revision history for this message
Tim Penhey (thumper) wrote :

Fixed in RF 4866.

Changed in launchpad-bazaar:
status: In Progress → Fix Committed
Tim Penhey (thumper)
Changed in launchpad-bazaar:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.