Tuesday, 2018-04-24

*** os-chef-bot has quit IRC03:30
*** os-chef-bot has joined #openstack-chef03:45
*** noodle has quit IRC06:25
*** noodle has joined #openstack-chef06:49
openstackgerritJens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore  https://review.openstack.org/56388710:55
openstackgerritMerged openstack/cookbook-openstack-compute master: Refactor libvirtd.conf creation  https://review.openstack.org/56358212:19
openstackgerritJens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore  https://review.openstack.org/56388712:41
openstackgerritJens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation  https://review.openstack.org/56393212:43
openstackgerritJens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation  https://review.openstack.org/56393212:44
openstackgerritJens Harbott (frickler) proposed openstack/openstack-chef-repo master: Increase timeout for rake job  https://review.openstack.org/56395213:22
jklarescas hi13:29
jklarescas any chance you remember why we dropped this "package_override" options for multiple cookbooks?13:33
jklarescas ah, nvm, found it13:46
jklarescas the logic has been moved to common13:46
scasthe overrides only need to be set once, instead of command-line time13:59
openstackgerritSamuel Cassiba proposed openstack/openstack-chef-repo master: Support Chef 14 in Test Kitchen scenarios  https://review.openstack.org/56315414:01
scasin practice, i found that in later revisions of chef 12, leading up to 13, got confused on package_overrides. specifically, the single versus double quotes required14:05
scasswitching gears, for openstackclient, i would like to use that one to kick off the renaming to eliminate hyphens from the cookbook names14:18
scascookbook-openstack-client was never fully retired, but left at openstack_client14:19
scasto me, it seems like the best home with the least amount of confusion between an overcloud deployer and python-openstackclient14:19
jklarei hate renaming14:44
jklare:)14:44
jklareit always means a lot of refactoring in a lot of places14:45
scas:D14:52
scaschef-client hates hyphens more14:52
scasthe main reason i'm bringing up renaming again is because it would have eventually happened if the attrition did not14:56
jklarescas so if i understand correctly you want to rename "cookbook-openstackclient" to just "openstackclient" ?15:37
jklarescas which would be just renaming the repo15:38
scasno. i'd to follow what was done here and rename cookbook-openstackclient accordingly: https://github.com/openstack/cookbook-openstack-client/blob/master/metadata.rb#L215:39
jklarescas but why not just keep the name "openstackclient" ?15:39
jklarescas i see no reason to add an "_" to the mix15:40
jklarescas in addition to that, i am not sure if we want to reuse the repo of the old "openstack_client" cookbook for the new one15:41
jklarescas i think its easier to create a new repo named "cookbook-openstackclient" under the openstack namespace and just deprecated and remove the old "cookbook-openstack-client" repo15:42
scasthat seems reasonable enough. i was overthinking it, of course15:46
jklarei think this way we also avoid confusion, since its a new cookbook with a new name in a new repo15:47
jklare:)15:47
scasit does continue to be a pain with regular expressions that don't key off of just 'cookbook-**'15:47
scasi've completed my most recent relocation, but i'm not fully set up, but i'll see about retiring the old one and importing the new15:48
jklarescas no rush, i think the number of community members that depend on a quick addition of this cookbook is quite limited ;)15:50
scasthe constraints are primarily on my own time15:56
openstackgerritMerged openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore  https://review.openstack.org/56388716:39
openstackgerritMerged openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation  https://review.openstack.org/56393216:39
openstackgerritMerged openstack/openstack-chef-repo master: Increase timeout for rake job  https://review.openstack.org/56395216:45
*** noodle has quit IRC18:09
*** spheromak has quit IRC20:04
*** spheromak has joined #openstack-chef20:06
*** noodle has joined #openstack-chef20:33
*** os-chef-bot has quit IRC23:19

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