This is an archive of the discontinued Mercurial Phabricator instance.

Differential D1431

sshpeer: add a configurable hint for the ssh error message
ClosedPublic

Authored by zuza on Nov 15 2017, 7:43 PM.

Details

Reviewers
durham
mitrandir
durin42
Group Reviewers
hg-reviewers
Commits
rHG8b1c887d52e7: sshpeer: add a configurable hint for the ssh error message
Summary

Adding a possibility to configure error hint to be shown in the case of problems with SSH. Example of such hint can be "Please see http://company/internalwiki/ssh.html".

Test Plan
  • Ran hg pull with broken link and verified the output has no hint by default:
pulling from ssh://brokenrepository.com//repo
remote: ssh: Could not resolve hostname brokenrepository.com: Name or service not known
abort: no suitable response from remote hg!
pulling from ssh://brokenrepository.com//repo
remote: ssh: Could not resolve hostname brokenrepository.com: Name or service not known
abort: no suitable response from remote hg!
(Please see http://company/internalwiki/ssh.html)

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

zuza created this revision.Nov 15 2017, 7:43 PM
zuza edited the summary of this revision. (Show Details)Nov 15 2017, 7:51 PM
zuza edited the test plan for this revision. (Show Details)
zuza added reviewers: durham, mitrandir.
zuza updated this revision to Diff 3540.Nov 15 2017, 7:58 PM
zuza edited the test plan for this revision. (Show Details)Nov 15 2017, 8:48 PM
zuza updated this revision to Diff 3541.
zuza updated this revision to Diff 3542.Nov 15 2017, 8:52 PM
durham requested changes to this revision.Nov 15 2017, 9:05 PM
durham added inline comments.
mercurial/sshpeer.py
210

The standard Mercurial way to format this would be something like:

msg = self.ui.config("ssh", "errormessage",
                     _("no suitable response from remote hg"))
self._abort(error.RepoError(msg))
tests/test-ssh.t
314

These lines seem unrelated to your change and therefore probably shouldn't have changed. Usually this is caused by your editor trying to remove trailing whitespace from a file, then run-tests -i adding the missing new line back in. Can you undo these new line changes? 'hg uncommit tests/test-ssh.t && hg commit -i' could make it pretty straightforward.

447

Same for this type of change

This revision now requires changes to proceed.Nov 15 2017, 9:05 PM
durham added a comment.Nov 15 2017, 9:05 PM

I'd also update the commit summary with an example of what a better message might be. Like "there was an ssh error, please see http://company/internalwiki/ssh.html"

zuza updated this revision to Diff 3548.Nov 15 2017, 9:18 PM
zuza marked 3 inline comments as done.Nov 15 2017, 9:19 PM
yuja added a subscriber: yuja.Nov 16 2017, 8:07 AM
In D1431#23709, @durham wrote:

I'd also update the commit summary with an example of what a better message might be. Like "there was an ssh error, please see http://company/internalwiki/ssh.html"

So the purpose of this change is quite similar to ui.supportcontact, right?

My two cents. I think it's probably better to make an additional hint configurable,
instead of replacing the original message.

mercurial/sshpeer.py
207–210

new config needs to be registered to the configitems module.

zuza edited the summary of this revision. (Show Details)Nov 16 2017, 1:58 PM
zuza edited the test plan for this revision. (Show Details)
zuza retitled this revision from sshpeer: making the ssh error message configurable to sshpeer: add a configurable hint for the ssh error message.
zuza updated this revision to Diff 3572.
zuza marked an inline comment as done.Nov 16 2017, 1:59 PM
durin42 accepted this revision.Nov 17 2017, 4:03 PM
durin42 requested changes to this revision.Nov 17 2017, 5:06 PM

D'oh: this also needs to be documented, probably in hg help config. That's from mercurial/help/config.txt - hopefully it'll be fairly straightforward to add?

This revision now requires changes to proceed.Nov 17 2017, 5:06 PM
zuza updated this revision to Diff 3631.Nov 20 2017, 4:44 AM
durin42 accepted this revision.Nov 20 2017, 6:29 PM
Closed by commit rHG8b1c887d52e7: sshpeer: add a configurable hint for the ssh error message (authored by zuza). · Explain WhyNov 20 2017, 6:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/sshpeer.py (6 lines)
Mtests/test-ssh.t (26 lines)

Diff 3541

mercurial/sshpeer.py

sub = util.popen4(cmd, bufsize=0) sub = util.popen4(cmd, bufsize=0)
self._pipeo, self._pipei, self._pipee, self._subprocess = sub self._pipeo, self._pipei, self._pipee, self._subprocess = sub
self._pipei = util.bufferedinputpipe(self._pipei) self._pipei = util.bufferedinputpipe(self._pipei)
self._pipei = doublepipe(self.ui, self._pipei, self._pipee) self._pipei = doublepipe(self.ui, self._pipei, self._pipee)
self._pipeo = doublepipe(self.ui, self._pipeo, self._pipee) self._pipeo = doublepipe(self.ui, self._pipeo, self._pipee)
def badresponse(): def badresponse():
self._abort(error.RepoError(_('no suitable response from ' self._abort(error.RepoError(self.ui.config(
'remote hg'))) "ssh",
"errormessage",
_("no suitable response from remote hg"))))
durhamUnsubmitted
Done

The standard Mercurial way to format this would be something like:

msg = self.ui.config("ssh", "errormessage",
                     _("no suitable response from remote hg"))
self._abort(error.RepoError(msg))
durham: The standard Mercurial way to format this would be something like: ``` msg = self.ui.config…
yujaUnsubmitted
Done

new config needs to be registered to the configitems module.

yuja: new config needs to be registered to the configitems module.
try: try:
# skip any noise generated by remote shell # skip any noise generated by remote shell
self._callstream("hello") self._callstream("hello")
r = self._callstream("between", pairs=("%s-%s" % ("0"*40, "0"*40))) r = self._callstream("between", pairs=("%s-%s" % ("0"*40, "0"*40)))
except IOError: except IOError:
badresponse() badresponse()

tests/test-ssh.t

$ hg ci --close-branch -mc1 $ hg ci --close-branch -mc1
$ hg up -q default $ hg up -q default
configure for serving configure for serving
$ cat <<EOF > .hg/hgrc $ cat <<EOF > .hg/hgrc
> [server] > [server]
> uncompressed = True > uncompressed = True
>
> [hooks] > [hooks]
> changegroup = sh -c "printenv.py changegroup-in-remote 0 ../dummylog" > changegroup = sh -c "printenv.py changegroup-in-remote 0 ../dummylog"
> EOF > EOF
$ cd .. $ cd ..
repo not found error repo not found error
$ hg clone -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/nonexistent local $ hg clone -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/nonexistent local
changeset: 3:a28a9d1a809c changeset: 3:a28a9d1a809c
tag: tip tag: tip
parent: 0:1160648e36ce parent: 0:1160648e36ce
user: test user: test
date: Thu Jan 01 00:00:00 1970 +0000 date: Thu Jan 01 00:00:00 1970 +0000
summary: add summary: add
find incoming on the remote side find incoming on the remote side
$ hg incoming -R ../remote -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/local $ hg incoming -R ../remote -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/local
comparing with ssh://user@dummy/local comparing with ssh://user@dummy/local
searching for changes searching for changes
changeset: 3:a28a9d1a809c changeset: 3:a28a9d1a809c
tag: tip tag: tip
parent: 0:1160648e36ce parent: 0:1160648e36ce
user: test user: test
date: Thu Jan 01 00:00:00 1970 +0000 date: Thu Jan 01 00:00:00 1970 +0000
summary: add summary: add
find incoming on the remote side (using absolute path) find incoming on the remote side (using absolute path)
$ hg incoming -R ../remote -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" "ssh://user@dummy/`pwd`" $ hg incoming -R ../remote -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" "ssh://user@dummy/`pwd`"
comparing with ssh://user@dummy/$TESTTMP/local comparing with ssh://user@dummy/$TESTTMP/local
searching for changes searching for changes
changeset: 3:a28a9d1a809c changeset: 3:a28a9d1a809c
tag: tip tag: tip
parent: 0:1160648e36ce parent: 0:1160648e36ce
user: test user: test
date: Thu Jan 01 00:00:00 1970 +0000 date: Thu Jan 01 00:00:00 1970 +0000
summary: add summary: add
push push
$ hg push $ hg push
pushing to ssh://user@dummy/remote pushing to ssh://user@dummy/remote
searching for changes searching for changes
remote: adding changesets remote: adding changesets
remote: adding manifests remote: adding manifests
remote: adding file changes remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files remote: added 1 changesets with 1 changes to 1 files
$ cd ../remote $ cd ../remote
check remote tip check remote tip
$ hg tip $ hg tip
changeset: 3:a28a9d1a809c changeset: 3:a28a9d1a809c
tag: tip tag: tip
parent: 0:1160648e36ce parent: 0:1160648e36ce
user: test user: test
date: Thu Jan 01 00:00:00 1970 +0000 date: Thu Jan 01 00:00:00 1970 +0000
summary: add summary: add
$ hg verify $ hg verify
checking changesets checking changesets
checking manifests checking manifests
crosschecking files in changesets and manifests crosschecking files in changesets and manifests
checking files checking files
2 files, 4 changesets, 3 total revisions 2 files, 4 changesets, 3 total revisions
$ hg cat -r tip foo $ hg cat -r tip foo
bleah bleah
changeset: 4:6c0482d977a3 changeset: 4:6c0482d977a3
parent: 0:1160648e36ce parent: 0:1160648e36ce
user: test user: test
date: Thu Jan 01 00:00:00 1970 +0000 date: Thu Jan 01 00:00:00 1970 +0000
summary: z summary: z
durhamUnsubmitted
Done

These lines seem unrelated to your change and therefore probably shouldn't have changed. Usually this is caused by your editor trying to remove trailing whitespace from a file, then run-tests -i adding the missing new line back in. Can you undo these new line changes? 'hg uncommit tests/test-ssh.t && hg commit -i' could make it pretty straightforward.

durham: These lines seem unrelated to your change and therefore probably shouldn't have changed.
clone bookmarks clone bookmarks
$ hg -R ../remote bookmark test $ hg -R ../remote bookmark test
$ hg -R ../remote bookmarks $ hg -R ../remote bookmarks
* test 4:6c0482d977a3 * test 4:6c0482d977a3
$ hg clone -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/remote local-bookmarks $ hg clone -e "\"$PYTHON\" \"$TESTDIR/dummyssh\"" ssh://user@dummy/remote local-bookmarks
requesting all changes requesting all changes
adding changesets adding changesets
stderr from remote commands should be printed before stdout from local code (issue4336) stderr from remote commands should be printed before stdout from local code (issue4336)
$ hg clone remote stderr-ordering $ hg clone remote stderr-ordering
updating to branch default updating to branch default
3 files updated, 0 files merged, 0 files removed, 0 files unresolved 3 files updated, 0 files merged, 0 files removed, 0 files unresolved
$ cd stderr-ordering $ cd stderr-ordering
$ cat >> localwrite.py << EOF $ cat >> localwrite.py << EOF
> from mercurial import exchange, extensions > from mercurial import exchange, extensions
>
durhamUnsubmitted
Done

Same for this type of change

durham: Same for this type of change
> def wrappedpush(orig, repo, *args, **kwargs): > def wrappedpush(orig, repo, *args, **kwargs):
> res = orig(repo, *args, **kwargs) > res = orig(repo, *args, **kwargs)
> repo.ui.write('local stdout\n') > repo.ui.write('local stdout\n')
> return res > return res
>
> def extsetup(ui): > def extsetup(ui):
> extensions.wrapfunction(exchange, 'push', wrappedpush) > extensions.wrapfunction(exchange, 'push', wrappedpush)
> EOF > EOF
$ cat >> .hg/hgrc << EOF $ cat >> .hg/hgrc << EOF
> [paths] > [paths]
> default-push = ssh://user@dummy/remote > default-push = ssh://user@dummy/remote
> [ui] > [ui]
> crash = ${TESTDIR}/crashgetbundler.py > crash = ${TESTDIR}/crashgetbundler.py
> EOF > EOF
$ hg --config ui.ssh="\"$PYTHON\" $TESTDIR/dummyssh" pull $ hg --config ui.ssh="\"$PYTHON\" $TESTDIR/dummyssh" pull
pulling from ssh://user@dummy/remote pulling from ssh://user@dummy/remote
searching for changes searching for changes
remote: abort: this is an exercise remote: abort: this is an exercise
abort: pull failed on remote abort: pull failed on remote
[255] [255]
abort with default error message when there is a ssh problem when pulling
$ hg pull ssh://brokenrepository
pulling from ssh://brokenrepository/
remote: ssh: Could not resolve hostname brokenrepository: Name or service not known
abort: no suitable response from remote hg!
[255]
abort with configured error message when there is a ssh problem when pulling
$ hg pull ssh://brokenrepository --config ssh.errormessage="SSH error"
pulling from ssh://brokenrepository/
remote: ssh: Could not resolve hostname brokenrepository: Name or service not known
abort: SSH error!
[255]