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

Remove mika and mkaapu users #338

Merged
merged 1 commit into from
Jan 6, 2025
Merged

Remove mika and mkaapu users #338

merged 1 commit into from
Jan 6, 2025

Conversation

vjuntunen
Copy link
Contributor

Remove mika and mkaapu users from ghaf-infra user
and from testagents

@henrirosten
Copy link
Collaborator

henrirosten commented Jan 3, 2025

You also need to remove all other references to those users, and their keys in ssh-keys.yaml:

❯ grep -r mika . -n
./terraform/binary-cache.tf:34:      for user in toset(["bmg", "flokli", "hrosten", "jrautiola", "vjuntunen", "mkaapu", "karim", "cazfi", "mika"]) : {
./terraform/jenkins-controller.tf:36:      for user in toset(["bmg", "flokli", "hrosten", "jrautiola", "mkaapu", "mika", "karim", "cazfi", "vjuntunen", "ktu", "alextserepov"]) : {
./ssh-keys.yaml:16:mika:
./ssh-keys.yaml:17: - ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIKoroafXsrWkoLewJ7EYcHodqlYILV2T8xtRo6RL99vz mika@nixos
./hosts/ghaf-proxy/configuration.nix:28:      user-mika
./hosts/builders/ficolo.nix:24:      user-mika
./hosts/builders/hetzarm/configuration.nix:28:      user-mika
./hosts/himalia/configuration.nix:24:      user-mika
❯ grep -r mkaapu . -n
./terraform/binary-cache.tf:34:      for user in toset(["bmg", "flokli", "hrosten", "jrautiola", "vjuntunen", "mkaapu", "karim", "cazfi", "mika"]) : {
./terraform/jenkins-controller.tf:36:      for user in toset(["bmg", "flokli", "hrosten", "jrautiola", "mkaapu", "mika", "karim", "cazfi", "vjuntunen", "ktu", "alextserepov"]) : {
./ssh-keys.yaml:13:mkaapu:
./ssh-keys.yaml:14: - ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJYpWmLovw4yIfjhWZ75PgtVfk9ZUcYHRLORcDltoufa mkaapu@nixzure
./hosts/ghaf-proxy/configuration.nix:33:      user-mkaapu
./hosts/binarycache/configuration.nix:31:      user-mkaapu
./hosts/ghaf-log/configuration.nix:29:      user-mkaapu
./hosts/builders/ficolo.nix:22:      user-mkaapu
./hosts/builders/hetzarm/configuration.nix:26:      user-mkaapu

@vjuntunen
Copy link
Contributor Author

Oh! You're right. That's why I opened this Pr :)

Remove mika and mkaapu users from ghaf-infra user
and from testagents

Signed-off-by: Ville-Pekka Juntunen <[email protected]>
@vjuntunen vjuntunen merged commit 58ed984 into tiiuae:main Jan 6, 2025
6 checks passed
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.

3 participants