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 Ficolo builder configuration #28

Closed
wants to merge 4 commits into from
Closed

Add Ficolo builder configuration #28

wants to merge 4 commits into from

Conversation

mkaapu
Copy link
Contributor

@mkaapu mkaapu commented Nov 28, 2023

Add common configuration to install NixOS for all Ficolo build servers replacing current Ubuntu servers

HW configuration is based on configuration generated by the nixos-generate-config tool on one of the target machines.
Preconfigured IP address is provided to Ficolo builders by an external DHCP server based on the MAC address of the server.
Disk configuration (disk-config.nix) formats 4 separate disks. Initial disk setup should be identical on all Ficolo build servers.

Configuration alias to install NixOS to the current build3 Ubuntu machine with the IP address 172.18.20.104 added.
The 'mkaapu' user account configuration added.

All existing users will be added to installed Ficolo build machines.

Signed-off-by: Marko Kaapu <marko.kaapu@unikie.com>
Signed-off-by: Marko Kaapu <marko.kaapu@unikie.com>
Copy link
Collaborator

@henrirosten henrirosten left a comment

Choose a reason for hiding this comment

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

Please add a description to this PR, shortly explaining what was changed and why this change is needed.

The new host configuration is also missing secrets.yaml meaning, for instance, that the host ssh key is not pre-defined in the configuration. Is this intentional?

hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
hosts/ficolobuild/default.nix Outdated Show resolved Hide resolved
- open-ssh will be configured using the service-openss module
- remove hw configuration variables with empty lists
- no networkmanager enabled
- no hostname set

Signed-off-by: Marko Kaapu <marko.kaapu@unikie.com>
Signed-off-by: Marko Kaapu <marko.kaapu@unikie.com>
@mkaapu
Copy link
Contributor Author

mkaapu commented Nov 29, 2023

Please add a description to this PR, shortly explaining what was changed and why this change is needed.

Description added.

The new host configuration is also missing secrets.yaml meaning, for instance, that the host ssh key is not pre-defined in the configuration. Is this intentional?

I leave secrets.yaml undefined now as this is supposed to be a general installation configuration for all Ficolo builders and I do not yet know the specific use cases or builds which will be running on these build servers. So, host keys etc. secrets need be set up manually after the installation / server. This is however something which could be automated with some extra implementation and modules (new future feature maybe?).

@mkaapu
Copy link
Contributor Author

mkaapu commented Nov 30, 2023

Closing this and creating a new pull request (#32) where main branch is merged to the new development branch with conflict resolution.

@mkaapu mkaapu deleted the ficolo-builder branch November 30, 2023 14:22
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.

2 participants