Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make a macOS user capable of being the service's user-account #40

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

petemounce
Copy link
Contributor

@petemounce petemounce commented Sep 2, 2020

Changes

Now, running the role on macOS will create a user suitable for being the service's account.

That user has various sensible defaults set up

  • randomly generated 32-character password
  • not an admin unless requested
  • macOS behaviour like Siri etc registered as seen (so 1st login isn't bothered with those things, if a login ever happens)

This is consistent with the behaviour

  • on Debian (where the .deb package creates the user)
  • on Windows (where this role creates the user, optionally as an admin)

Verification

  • Local run of the script to create an admin and a standard user:

Screenshot 2020-09-02 at 13 14 48

  • Local anka bake starting from a bare-OS image that does not have the buildkite-agent created within it:

  • verify that the buildkite-agent binary can start up and run when the user is created via this method. Screenshot of connected-to-bk agent page:

  • confirm all of the above via a physical macOS:

@petemounce petemounce self-assigned this Sep 2, 2020
@improbable-prow-robot improbable-prow-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 2, 2020
@improbable-prow-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@improbable-prow-robot improbable-prow-robot added the size/L Denotes a PR that changes 150-299 lines, ignoring generated files. label Sep 2, 2020
@improbable-prow-robot improbable-prow-robot added size/XL Denotes a PR that changes 300-599 lines, ignoring generated files. and removed size/L Denotes a PR that changes 150-299 lines, ignoring generated files. labels Sep 2, 2020
@@ -1,15 +1,15 @@
---
- name: make password for user
set_fact:
user_password: "{{ lookup('password', '/tmp/bk-agent-password length=32 chars=ascii_letters,digits,punctuation') }}"
buildkite_agent_user_password: "{{ lookup('password', '/tmp/bk-agent-password length=32 chars=ascii_letters,digits,punctuation') }}"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was renamed to namespace the fact to the role.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XL Denotes a PR that changes 300-599 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants