*** tuanla____ has joined #openstack-chef | 06:22 | |
openstackgerrit | Merged openstack/openstack-chef-repo master: kitchen: add a public IP address https://review.openstack.org/558801 | 07:41 |
---|---|---|
*** jklare has quit IRC | 08:12 | |
*** jklare has joined #openstack-chef | 08:16 | |
*** jklare has quit IRC | 08:25 | |
*** jklare_ has joined #openstack-chef | 08:25 | |
*** jklare_ is now known as jklare | 08:25 | |
*** jklare has quit IRC | 08:28 | |
*** jklare has joined #openstack-chef | 08:31 | |
*** os-chef-bot has quit IRC | 09:45 | |
*** tuanla____ has quit IRC | 10:21 | |
jklare | scas good morning, time for a quick meeting? | 13:13 |
jklare | scas i would like to talk about the ceph patches you proposed | 13:17 |
scas | i literally just woke up 5 minutes ago :D | 13:48 |
scas | Thu Apr 5 06:48:58 PDT 2018 | 13:48 |
scas | jklare: happy to talk further about the patches. my main goal was to swap out the older ceph cookbook for ceph-chef | 13:53 |
jklare | scas 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 code | 13:55 |
jklare | scas for most of the other cookbooks we tried to remove the deployment specific code | 13:55 |
jklare | scas and i think it should be pretty easy to add ceph deployment specific code in a wrapper | 13:55 |
scas | yeah. at first, my notion was to simply remove the whole thing | 13:55 |
scas | my first instinct was probably the right one, before i proposed patches | 13:56 |
jklare | i 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 |
scas | it would. the dep graph is crazy | 13:59 |
jklare | i 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 cinder | 13:59 |
jklare | instead 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 itself | 13:59 |
scas | that could work. i won't be in a place to even proof of concept anything until next week | 14:00 |
scas | https://github.com/bloomberg/chef-bcs has its own implementation of ceph-chef that is just a bunch of wrapper recipes | 14:06 |
scas | i thought i fixed that commit msg last week | 14:09 |
openstackgerrit | Samuel Cassiba proposed openstack/cookbook-openstack-network master: Update DB migration command for Queens https://review.openstack.org/556348 | 14:09 |
*** os-chef-bot has joined #openstack-chef | 14:19 | |
jklare | scas 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 |
openstackgerrit | Jan Klare proposed openstack/cookbook-openstack-image master: remove unused ceph related attributes https://review.openstack.org/559097 | 14:26 |
scas | jklare: both, sort of. if you have code already working, that'll make it easier | 14:27 |
jklare | scas i will push some patches to remove all the ceph code and dependencies from glance, cinder and nova | 14:27 |
scas | works for me | 14:27 |
jklare | scas 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 backend | 14:28 |
openstackgerrit | Jan Klare proposed openstack/cookbook-openstack-block-storage master: remove all ceph related attributes and specs https://review.openstack.org/559101 | 14:30 |
openstackgerrit | Jan Klare proposed openstack/cookbook-openstack-compute master: remove all ceph related attributes,recipes,specs and the cookbook dependency https://review.openstack.org/559103 | 14:38 |
jklare | scas first set pushed, lets see how these changes fail :) i will update them later, was too lazy to run local tests ;) | 14:39 |
scas | :D | 14:40 |
*** openstackgerrit has quit IRC | 15:34 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!