Friday, 2018-04-06

*** gkadam_ has joined #openstack-kayobe05:48
*** gkadam_ has quit IRC06:42
*** gkadam has joined #openstack-kayobe06:42
*** egonzalez_afk has joined #openstack-kayobe06:52
*** dougsz has joined #openstack-kayobe07:59
*** mgoddard has joined #openstack-kayobe08:03
*** raphael____ has joined #openstack-kayobe08:04
*** ktibi has joined #openstack-kayobe08:07
openstackgerritMark Goddard proposed openstack/kayobe master: WIP: Use the Zuul VM as the seed  https://review.openstack.org/55889108:09
*** raphael____ has quit IRC08:11
*** raphael____ has joined #openstack-kayobe08:15
*** raphael____ has quit IRC08:24
openstackgerritKevin Tibi proposed openstack/kayobe master: Add support for custom repo  https://review.openstack.org/55904508:31
ktibimgoddard, hi, live-restore enabled by default ?08:38
mgoddardktibi: no, it's disabled by default08:38
mgoddardbackwards compat08:38
ktibiok ;)08:38
openstackgerritKevin Tibi proposed openstack/kayobe master: Support docker live-restore option  https://review.openstack.org/55929208:40
mgoddardktibi: oh, you meant live-restore default in kayobe, not docker. Let's make it enabled, with an upgrade note08:43
ktibiyes08:43
ktibihum so when you upgrade08:43
ktibikayobe detect a change and restart all daemon docker ><08:44
ktibiand boom !08:44
ktibimgoddard, so https://docs.ansible.com/ansible/latest/user_guide/playbooks_delegation.html#rolling-update-batch-size :D08:44
mgoddardktibi: only if you run kayobe overcloud host configure08:45
ktibimgoddard, so how apply live restore ?08:45
ktibihost upgrade ?08:45
mgoddardhost configure08:45
ktibiyes so kayobe will restart all docker daemon ?08:45
mgoddardbut that's not in the standard upgrade procedure08:45
mgoddardyou can apply the setting by reloading docker rather than restarting: https://docs.docker.com/config/containers/live-restore/08:46
mgoddardunsure how we decide when to reload vs. restart08:47
ktibihttps://github.com/openstack/kayobe/blob/master/ansible/roles/docker/tasks/config.yml#L708:48
ktibihum I don't know why kayobe restart all my docker ><08:49
ktibior it's kolla I guess08:49
mgoddardcould also be kolla08:50
mgoddardneed to check the logic in bootstrap-servers08:50
ktibiI change docker_resgitry here08:51
ktibihttps://github.com/openstack/kolla-ansible/blob/stable/pike/ansible/roles/baremetal/templates/docker_systemd_service.j2#L408:51
ktibiand https://github.com/openstack/kolla-ansible/blob/stable/pike/ansible/roles/baremetal/tasks/post-install.yml#L5108:52
ktibiso it's kolla08:52
*** gkadam has quit IRC08:52
ktibiI'll open a bug I think08:53
*** gkadam has joined #openstack-kayobe08:53
ktibiok first test works, I stop daemon and all containers are alive :D08:57
ktibimgoddard, so I enabled by default in kayobe and we need to make a reload or a restart but with update size feature of ansible08:59
mgoddardktibi: you can't do serial per-task, but there are some workarounds that could be considered here: https://github.com/ansible/ansible/issues/1217009:08
mgoddardktibi: the daemon.json could be parsed before & after, then check for changed fields that support reload09:12
ktibiwhy can't add serial on https://github.com/openstack/kayobe/blob/master/ansible/roles/docker/tasks/config.yml#L2 ????09:12
ktibihum ho you call handler so all handler will call after the play :/09:13
mgoddardyou can't add serial to a task, only a play09:14
mgoddardwe could run the play serially, will just be a bit slower09:14
mgoddardslow > unsafe09:14
mgoddardusing reload would be nice though if possible09:15
mgoddardwe only need to ensure we reload to add live-restore - after that we can restart09:15
ktibiI think when you change the storage of docker we need restart :/09:15
ktibiis it possible to make that with ansible ?09:16
openstackgerritKevin Tibi proposed openstack/kayobe master: Support docker live-restore option  https://review.openstack.org/55929209:17
mgoddardyes09:18
mgoddardktibi: anything's possible09:18
mgoddardsee my comment earlier about a JSON diff09:19
ktibimgoddard, where is your comment ?10:01
mgoddardktibi: the daemon.json could be parsed before & after, then check for changed fields that support reload10:02
*** dasTor has quit IRC10:35
*** gkadam has quit IRC10:47
openstackgerritMark Goddard proposed openstack/kayobe master: WIP: Use the Zuul VM as the seed  https://review.openstack.org/55889112:14
*** egonzalez_afk has quit IRC13:02
*** gkadam has joined #openstack-kayobe13:30
ktibimgoddard, good to read http://www.juliosblog.com/high-performance-computing-hpc-on-openstack-a-few-recommendations/14:34
*** gkadam_ has joined #openstack-kayobe14:51
*** gkadam has quit IRC14:53
mgoddardthanks ktibi: looks interesting14:53
*** gkadam_ has quit IRC15:10
*** openstackgerrit has quit IRC15:33
*** ktibi has quit IRC16:11
*** mgoddard has quit IRC16:56
*** dougsz has quit IRC17:24
*** mgoddard has joined #openstack-kayobe17:28
*** openstackgerrit has joined #openstack-kayobe18:34
openstackgerritMark Goddard proposed openstack/kayobe master: WIP: Use the Zuul VM as the seed  https://review.openstack.org/55889118:34
*** mgoddard has quit IRC18:53
*** mgoddard has joined #openstack-kayobe19:31
*** mgoddard has quit IRC19:37

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!