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
shellhub-agent: Add service and package expressions #104419
Conversation
c3c1c74
to
6bb01ef
Compare
@SuperSandro2000 I succeed in clean it. Please do another review on this when you have time. Thanks for all the comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not that familiar with modules so someone else should take a look at it.
6bb01ef
to
13f71b1
Compare
Is there anything else pending for this PR? |
Result of |
Result of 1 package blacklisted:
1 package built:
|
13f71b1
to
33b1f9f
Compare
33b1f9f
to
1b2389d
Compare
Hello all, I rebased this on top of master to resolve the conflict. Could someone take a look at this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would break out the package and service into 2 different ocmmits:
shellhub-agent: init at 0.4.2
nixos/shellhub-agent: init
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
ShellHub is a modern SSH server for remotely accessing Linux devices via command line (using any SSH client) or web-based user interface, designed as an alternative to _sshd_. Think ShellHub as centralized SSH for the the edge and cloud computing. More information about the project in: - https://shellhub.io - https://github.com/shellhub-io/shellhub Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
1b2389d
to
bb38f75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
cc @Mic92 or @infinisil for final look-over of module
bb38f75
to
f14d381
Compare
This provides the service to configure the ShellHub Agent. Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
f14d381
to
06edbab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's great, thanks :)
Motivation for this change
We aim to ease the use of ShellHub and NixOS is one of the distros we'd like to support. It is working very well on our machines.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)