Thursday, 2018-04-05

*** tuanla____ has joined #openstack-chef06:22
openstackgerritMerged openstack/openstack-chef-repo master: kitchen: add a public IP address  https://review.openstack.org/55880107:41
*** jklare has quit IRC08:12
*** jklare has joined #openstack-chef08:16
*** jklare has quit IRC08:25
*** jklare_ has joined #openstack-chef08:25
*** jklare_ is now known as jklare08:25
*** jklare has quit IRC08:28
*** jklare has joined #openstack-chef08:31
*** os-chef-bot has quit IRC09:45
*** tuanla____ has quit IRC10:21
jklarescas good morning, time for a quick meeting?13:13
jklarescas i would like to talk about the ceph patches you proposed13:17
scasi literally just woke up 5 minutes ago :D13:48
scasThu Apr  5 06:48:58 PDT 201813:48
scasjklare: happy to talk further about the patches. my main goal was to swap out the older ceph cookbook for ceph-chef13:53
jklarescas i think removing the old ceph cookbook is a good idea, but i am not sure if we want to add a newer or or rather completely remove the rbd/ceph/backendspecific code13:55
jklarescas for most of the other cookbooks we tried to remove the deployment specific code13:55
jklarescas and i think it should be pretty easy to add ceph deployment specific code in a wrapper13:55
scasyeah. at first, my notion was to simply remove the whole thing13:55
scasmy first instinct was probably the right one, before i proposed patches13:56
jklarei think just removing the code would simplify the dependency chain and also free us from maintaining any code related to this ceph cookbook (which also might not be to well maintained itself)13:58
scasit would. the dep graph is crazy13:59
jklarei looked through our deployment today and realised that we do not use any of the openstack-* cookbook recipes to configure the ceph backend for glance and cinder13:59
jklareinstead we just use a pretty simple wrapper recipe which adds the correct backends for cinder and glance and another one which takes care of the ceph deployment itself13:59
scasthat could work. i won't be in a place to even proof of concept anything until next week14:00
scashttps://github.com/bloomberg/chef-bcs has its own implementation of ceph-chef that is just a bunch of wrapper recipes14:06
scasi thought i fixed that commit msg last week14:09
openstackgerritSamuel Cassiba proposed openstack/cookbook-openstack-network master: Update DB migration command for Queens  https://review.openstack.org/55634814:09
*** os-chef-bot has joined #openstack-chef14:19
jklarescas do you mean a "proof of concept" for such a ceph wrapper recipe or the removal of all ceph related code in the nova,glance and cinder cookbooks?14:21
openstackgerritJan Klare proposed openstack/cookbook-openstack-image master: remove unused ceph related attributes  https://review.openstack.org/55909714:26
scasjklare: both, sort of. if you have code already working, that'll make it easier14:27
jklarescas i will push some patches to remove all the ceph code and dependencies from glance, cinder and nova14:27
scasworks for me14:27
jklarescas and also look though our cookbooks and see if i can publish some examples on how to wrap the cookbooks to deploy openstack with ceph as storage backend14:28
openstackgerritJan Klare proposed openstack/cookbook-openstack-block-storage master: remove all ceph related attributes and specs  https://review.openstack.org/55910114:30
openstackgerritJan Klare proposed openstack/cookbook-openstack-compute master: remove all ceph related attributes,recipes,specs and the cookbook dependency  https://review.openstack.org/55910314:38
jklarescas first set pushed, lets see how these changes fail :) i will update them later, was too lazy to run local tests ;)14:39
scas:D14:40
*** openstackgerrit has quit IRC15:34

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