Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(468)

Issue 912003: Make ssh compatible with dev mode and remove testing ssh items that don't work (Closed)

Created:
10 years, 9 months ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
gauravsh
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

Make ssh compatible with dev mode and remove testing ssh items that don't work

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -23 lines) Patch
M src/platform/init/openssh-server.conf View 1 chunk +9 lines, -3 lines 2 comments Download
D src/scripts/mod_for_test_scripts/601autoStartSshd View 1 chunk +0 lines, -12 lines 0 comments Download
D src/scripts/mod_for_test_scripts/sshd.conf View 1 chunk +0 lines, -8 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
sosa
10 years, 9 months ago (2010-03-12 21:22:02 UTC) #1
gauravsh
LGTM http://codereview.chromium.org/912003/diff/1/2 File src/platform/init/openssh-server.conf (right): http://codereview.chromium.org/912003/diff/1/2#newcode7 src/platform/init/openssh-server.conf:7: start on stopped dump-boot-stats Any specific reason for ...
10 years, 9 months ago (2010-03-12 21:59:44 UTC) #2
sosa
http://codereview.chromium.org/912003/diff/1/2 File src/platform/init/openssh-server.conf (right): http://codereview.chromium.org/912003/diff/1/2#newcode7 src/platform/init/openssh-server.conf:7: start on stopped dump-boot-stats Ssh server is a development ...
10 years, 9 months ago (2010-03-12 22:01:32 UTC) #3
gauravsh
On Fri, Mar 12, 2010 at 2:01 PM, <sosa@chromium.org> wrote: > > http://codereview.chromium.org/912003/diff/1/2 > File ...
10 years, 9 months ago (2010-03-12 22:15:58 UTC) #4
ericli
10 years, 9 months ago (2010-03-12 23:15:55 UTC) #5
How this will affect an official build? no ssh there any more?

On Fri, Mar 12, 2010 at 2:15 PM, Gaurav Shah <gauravsh@chromium.org> wrote:

> On Fri, Mar 12, 2010 at 2:01 PM,  <sosa@chromium.org> wrote:
> >
> > http://codereview.chromium.org/912003/diff/1/2
> > File src/platform/init/openssh-server.conf (right):
> >
> > http://codereview.chromium.org/912003/diff/1/2#newcode7
> > src/platform/init/openssh-server.conf:7: start on stopped
> > dump-boot-stats
> > Ssh server is a development mode specific upstart job and shouldn't get
> > in the way of the core system starting up (affects perf metrics etc).
> > Since it's not a core script it should be delayed.
>
> Ahh yes, makes sense.
>
> >
> > On 2010/03/12 21:59:45, gauravsh wrote:
> >>
> >> Any specific reason for starting this late? (I am assuming
> >
> > dump-boot-stats is
> >>
> >> one of the last things that's run)
> >
> > http://codereview.chromium.org/912003
> >
>
>
>
> --
> -g
>



-- 
Eric Li
李咏竹
Google Kirkland

Powered by Google App Engine
This is Rietveld 408576698