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

Add services.hercules-ci-agent #586

Merged
merged 4 commits into from Jan 3, 2023

Conversation

roberth
Copy link
Contributor

@roberth roberth commented Dec 22, 2022

Adds hercules-ci-agent.
Installation instructions are provided on hercules-ci.com / docs.hercules-ci.com and will be updated after merge.
A typical configuration consists of services.hercules-ci-agent.enable = true and two or three files in /var/lib/hercules-ci-agent.

Migration note for existing users

For users migrating from old services and/or the current flake darwinModule (which will be updated to match this PR):

  • The nix-darwin module uses new user id and group id numbers, to match the upstream nix-darwin module.
    The main benefit of the upstream change is that the agent user will not appear as a normal (human) user on the system.

    To migrate, run:

    # if you have deployed an older version of the nix-darwin PR before,
    # the following delete commands will prevents this error:
    #     creating group _hercules-ci-agent...
    #     <main> attribute status: eDSRecordAlreadyExists
    #     <dscl_cmd> DS Error: -14135 (eDSRecordAlreadyExists)
    sudo dscl . -delete '/Groups/_hercules-ci-agent'
    sudo dscl . -delete '/Users/_hercules-ci-agent'
    
    # update the flake inputs / expressions / channels
    # and deploy the new version, e.g:
    sudo darwin-rebuild switch
    
    sudo chown -R _hercules-ci-agent:_hercules-ci-agent /var/lib/hercules-ci-agent

Source files originate from the hercules-ci-agent repository and
I will make sure to keep them in sync, bidirectionally.

The module is split into two files to make maintenance of the
common parts with NixOS easier.
This problem is multiple releases in the past now. It has served
its purpose.
@roberth roberth marked this pull request as ready for review December 23, 2022 11:07
@roberth roberth mentioned this pull request Dec 23, 2022
@domenkozar domenkozar merged commit 3db1d87 into LnL7:master Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants