Skip to content
This repository has been archived by the owner. It is now read-only.

motdgen depends on locksmithd, blocking user logins #1774

Closed
llamahunter opened this issue Jan 25, 2017 · 2 comments
Closed

motdgen depends on locksmithd, blocking user logins #1774

llamahunter opened this issue Jan 25, 2017 · 2 comments

Comments

@llamahunter
Copy link

@llamahunter llamahunter commented Jan 25, 2017

Issue Report

Bug

CoreOS Version

NAME=CoreOS
ID=coreos
VERSION=1185.5.0
VERSION_ID=1185.5.0
BUILD_ID=2016-12-07-0937
PRETTY_NAME="CoreOS 1185.5.0 (MoreOS)"
ANSI_COLOR="1;32"
HOME_URL="https://coreos.com/"
BUG_REPORT_URL="https://github.com/coreos/bugs/issues"

Environment

AWS and Openstack

Expected Behavior

User is able to log in to node even if locksmithd has not run (i.e. it is being held off so that it doesn't release its cluster reboot lock until the current node comes back up in a known good state).

Actual Behavior

User is unable to log in because locksmithd is before motdgen which is before a whole bunch of stuff necessary for users to be able to log in.

Reproduction Steps

  1. Configure a systemd unit to hold off the start of locksmithd for an indefinite period
  2. Boot up the node
  3. Try to log in via ssh

Other Information

Feature Request

Environment

See above

Desired Feature

motdgen.service is not listed as depending on locksmithd.service

Other Information

Alternatively, some way to let locksmithd come up normally, but provide a hook to not release its software update reboot lock until given some all clear signal.

@crawford
Copy link
Member

@crawford crawford commented Feb 24, 2017

Now that Ignition exists, we have a deterministic way of deciding the reboot strategy before the message of the day is generated. It's probably time we strip all of that timer logic from motdgen.

@crawford
Copy link
Member

@crawford crawford commented Mar 22, 2017

@crawford crawford closed this Mar 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.