[MERGED] osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread Harald Welte 
Harald Welte has submitted this change and it was merged.

Change subject: ansible: docker: cleanup old docker images
..


ansible: docker: cleanup old docker images

Fixes: OS#3144
Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
---
M ansible/roles/docker/tasks/main.yml
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved; Verified



diff --git a/ansible/roles/docker/tasks/main.yml 
b/ansible/roles/docker/tasks/main.yml
index 1603175..0a58a2c 100644
--- a/ansible/roles/docker/tasks/main.yml
+++ b/ansible/roles/docker/tasks/main.yml
@@ -26,3 +26,11 @@
 name: "{{ jenkins_user }}"
 groups: docker
 append: yes
+
+- name: cleanup old docker images
+  cron:
+name: cleanup-docker-images
+minute: 0
+hour: '*/3'
+user: "{{ jenkins_user }}"
+job: "test -x /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh 
&& /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh >/dev/null"

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 3
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte

osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread Harald Welte 

Patch Set 3: Code-Review+2 Verified+1

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 3
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte 
Gerrit-HasComments: No

[PATCH] osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread lynxis lazus 
Hello Harald Welte,

I'd like you to reexamine a change.  Please visit

https://gerrit.osmocom.org/7716

to look at the new patch set (#3).

ansible: docker: cleanup old docker images

Fixes: OS#3144
Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
---
M ansible/roles/docker/tasks/main.yml
1 file changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/16/7716/3

diff --git a/ansible/roles/docker/tasks/main.yml 
b/ansible/roles/docker/tasks/main.yml
index 1603175..0a58a2c 100644
--- a/ansible/roles/docker/tasks/main.yml
+++ b/ansible/roles/docker/tasks/main.yml
@@ -26,3 +26,11 @@
 name: "{{ jenkins_user }}"
 groups: docker
 append: yes
+
+- name: cleanup old docker images
+  cron:
+name: cleanup-docker-images
+minute: 0
+hour: '*/3'
+user: "{{ jenkins_user }}"
+job: "test -x /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh 
&& /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh >/dev/null"

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 3
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte

[PATCH] osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread lynxis lazus 
Hello Harald Welte,

I'd like you to reexamine a change.  Please visit

https://gerrit.osmocom.org/7716

to look at the new patch set (#2).

ansible: docker: cleanup old docker images

Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
---
M ansible/roles/docker/tasks/main.yml
1 file changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/16/7716/2

diff --git a/ansible/roles/docker/tasks/main.yml 
b/ansible/roles/docker/tasks/main.yml
index 1603175..0a58a2c 100644
--- a/ansible/roles/docker/tasks/main.yml
+++ b/ansible/roles/docker/tasks/main.yml
@@ -26,3 +26,11 @@
 name: "{{ jenkins_user }}"
 groups: docker
 append: yes
+
+- name: cleanup old docker images
+  cron:
+name: cleanup-docker-images
+minute: 0
+hour: '*/3'
+user: "{{ jenkins_user }}"
+job: "test -x /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh 
&& /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh >/dev/null"

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 2
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte

osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread Harald Welte 

Patch Set 1: Code-Review+1

this is only once per night.  We've seen this is insufficient on heavy activity 
on build nodes in the past.  I think I wrote it somewhere already (redmine 
ticket?) that we need this more frequently, like every 6 or 12 hours

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 1
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte 
Gerrit-HasComments: No

osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread Harald Welte 

Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 1
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte 
Gerrit-HasComments: No

[PATCH] osmo-ci[master]: ansible: docker: cleanup old docker images

2018-04-10 Thread lynxis lazus 

Review at  https://gerrit.osmocom.org/7716

ansible: docker: cleanup old docker images

Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
---
M ansible/roles/docker/tasks/main.yml
1 file changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/16/7716/1

diff --git a/ansible/roles/docker/tasks/main.yml 
b/ansible/roles/docker/tasks/main.yml
index 1603175..ea1b24c 100644
--- a/ansible/roles/docker/tasks/main.yml
+++ b/ansible/roles/docker/tasks/main.yml
@@ -26,3 +26,11 @@
 name: "{{ jenkins_user }}"
 groups: docker
 append: yes
+
+- name: cleanup old docker images
+  cron:
+name: cleanup-docker-images
+minute: 0
+hour: 2
+user: "{{ jenkins_user }}"
+job: "test -x /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh 
&& /home/{{ jenkins_user }}/osmo-ci/scripts/docker-cleanup.sh >/dev/null"

-- 
To view, visit https://gerrit.osmocom.org/7716
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I77fe38775b3f6055691bbf654de4019916b75fd7
Gerrit-PatchSet: 1
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus