Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(14112)

Issue 37930043: SSH keys 1: abort if no keys

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by matthew.scott
Modified:
10 years, 4 months ago
Reviewers:
mp+197959, gary.poster
Visibility:
Public.

Description

SSH keys 1: abort if no keys Juju needs ssh keys available to run. This ensures via ssh-add that there are keys available to the agent; if not, it exits with instructions on how to generate a key. To test: make check To QA: 1. mv ~/.ssh ~/.ssh.old 2. .venv/bin/python juju-quickstart # should produce error 3. mv ~/.ssh.old ~/.ssh 4. ssh-agent 5. ssh-add 6. .venv/bin/python juju-quickstart # should bootstrap https://code.launchpad.net/~makyo/juju-quickstart/ssh-1-check-keys/+merge/197959 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 4

Patch Set 2 : SSH keys 1: abort if no keys #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -0 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M quickstart/app.py View 1 1 chunk +15 lines, -0 lines 0 comments Download
M quickstart/manage.py View 1 chunk +2 lines, -0 lines 0 comments Download
M quickstart/tests/test_app.py View 1 1 chunk +25 lines, -0 lines 0 comments Download
M quickstart/tests/test_manage.py View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 4
matthew.scott
Please take a look.
10 years, 4 months ago (2013-12-05 20:35:40 UTC) #1
matthew.scott
Notes below https://codereview.appspot.com/37930043/diff/1/quickstart/app.py File quickstart/app.py (right): https://codereview.appspot.com/37930043/diff/1/quickstart/app.py#newcode83 quickstart/app.py:83: retcode = utils.call('/usr/bin/ssh-add', '-l')[0] retcode will be ...
10 years, 4 months ago (2013-12-05 20:38:19 UTC) #2
gary.poster
LGTM and QA OK. Thank you! https://codereview.appspot.com/37930043/diff/1/quickstart/app.py File quickstart/app.py (right): https://codereview.appspot.com/37930043/diff/1/quickstart/app.py#newcode83 quickstart/app.py:83: retcode = utils.call('/usr/bin/ssh-add', ...
10 years, 4 months ago (2013-12-06 00:07:55 UTC) #3
matthew.scott
10 years, 4 months ago (2013-12-06 16:20:36 UTC) #4
*** Submitted:

SSH keys 1: abort if no keys

Juju needs ssh keys available to run.  This ensures via ssh-add that there are
keys available to the agent; if not, it exits with instructions on how to
generate a key.

To test: make check

To QA:
1. mv ~/.ssh ~/.ssh.old
2. .venv/bin/python juju-quickstart # should produce error
3. mv ~/.ssh.old ~/.ssh
4. ssh-agent
5. ssh-add
6. .venv/bin/python juju-quickstart # should bootstrap

R=gary.poster
CC=
https://codereview.appspot.com/37930043
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b