Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@khilman
Copy link
Contributor

@khilman khilman commented Apr 27, 2021

Signed-off-by: Kevin Hilman khilman@baylibre.com

Signed-off-by: Kevin Hilman <khilman@baylibre.com>
@khilman
Copy link
Contributor Author

khilman commented Apr 27, 2021

Currently set to run daily on the jenkins k8s runner nodes.

Can be run manually as the buildslave user via ~/bin/cleanup.sh

@khilman khilman requested a review from a team April 27, 2021 18:02
Copy link
Collaborator

@gctucker gctucker left a comment

Choose a reason for hiding this comment

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

Looks good. Have you tried this on any of the actual Jenkins nodes yet or just locally on some other host?

group: buildslave
mode: 0755

- name: Daily cron for k8s garbage collection
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can this be easily turned into a systemd timer instead of cron? Or is Ansible going to create a systemd timer anyway with ansible.builtin.cron?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I assumed it would create a systemd timer, but in fact it creats a cron entry for the buildslave user. I'll need to read ansible docs to see if there's an easy way.

@gctucker
Copy link
Collaborator

Looks like something to bring up in the next weekly meeting as we've now got a weekly cleanup job run during production updates and other moving parts around how we manage Kubernetes clusters.

@khilman
Copy link
Contributor Author

khilman commented Nov 28, 2022

FWIW, I'm pretty sure this is already running as a cron job on the jenkins worker(s).

@gctucker
Copy link
Collaborator

OK so maybe we should close this now and reopen a task if any garbage collection still appears to be needed?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants