Page MenuHomeDevCentral
Differential D817

Provide motd command
ClosedPublic
Actions

Authored by dereckson on Jan 10 2017, 22:43.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 07:36
Unknown Object (File)
Tue, Dec 17, 03:46
Unknown Object (File)
Fri, Dec 13, 10:14
Unknown Object (File)
Fri, Dec 13, 10:14
Unknown Object (File)
Fri, Dec 13, 10:14
Unknown Object (File)
Fri, Dec 13, 10:14
Unknown Object (File)
Fri, Dec 13, 09:58
Unknown Object (File)
Fri, Dec 13, 09:33
View All Files
Subscribers
None

Details

Reviewers
dereckson
Sandlayth
Commits
rOPSdc64d1015d91: Provide motd command
Summary

A popular way to connect to a remote shell server is to reattach
to the tmux session. For example, ssh -t eglide.org tmux attach.

In such scenario, motd isn't printed at login time.
To allow to read MOTD in such sc, we provide a convenient command.

Test Plan

motd

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson created this revision.Jan 10 2017, 22:43
Harbormaster completed remote builds in B1254: Diff 2072.Jan 10 2017, 22:43
Sandlayth accepted this revision.Jan 12 2017, 09:20
This revision is now accepted and ready to land.Jan 12 2017, 09:20
dereckson updated this revision to Diff 2098.Jan 13 2017, 08:37

Rebased against master

dereckson added a comment.Jan 13 2017, 08:42

Should be added to top.tls too.

dereckson planned changes to this revision.Jan 13 2017, 08:43
Ysul
$ arc patch D817
$ salt eglide state.apply roles/core/motd/
eglide:
    Data failed to compile:
----------
    The function "mode 755" in state "/usr/local/bin/motd" in SLS "roles/core/motd/" has whitespace, a function with whitespace is not supported, perhaps this is an argument that is missing a ":"
----------
    Too many functions declared in state 'file' in SLS 'roles/core/motd/'
roles/core/motd/init.sls
26

mode:

dereckson marked an inline comment as done.Jan 13 2017, 08:44
In D817#12510, @dereckson wrote:

Should be added to top.tls too.

Actually, we're adding a new function in the already existing motd unit but it's already in top.tls.

dereckson updated this revision to Diff 2109.Jan 13 2017, 12:14

Fixed mode typo

This revision is now accepted and ready to land.Jan 13 2017, 12:14
Harbormaster completed remote builds in B1278: Diff 2109.Jan 13 2017, 12:14
dereckson accepted this revision.Jan 13 2017, 12:14
Closed by commit rOPSdc64d1015d91: Provide motd command (authored by dereckson, committed by dereckson). · Explain WhyJan 13 2017, 12:15
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
roles/
core/
motd/
files/
6 lines
5 lines

Diff 2110

View Options

roles/core/motd/files/motd

Loading...
View Options

roles/core/motd/init.sls

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator