*** os-chef-bot has quit IRC | 03:30 | |
*** os-chef-bot has joined #openstack-chef | 03:45 | |
*** noodle has quit IRC | 06:25 | |
*** noodle has joined #openstack-chef | 06:49 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore https://review.openstack.org/563887 | 10:55 |
---|---|---|
openstackgerrit | Merged openstack/cookbook-openstack-compute master: Refactor libvirtd.conf creation https://review.openstack.org/563582 | 12:19 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore https://review.openstack.org/563887 | 12:41 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation https://review.openstack.org/563932 | 12:43 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation https://review.openstack.org/563932 | 12:44 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/openstack-chef-repo master: Increase timeout for rake job https://review.openstack.org/563952 | 13:22 |
jklare | scas hi | 13:29 |
jklare | scas any chance you remember why we dropped this "package_override" options for multiple cookbooks? | 13:33 |
jklare | scas ah, nvm, found it | 13:46 |
jklare | scas the logic has been moved to common | 13:46 |
scas | the overrides only need to be set once, instead of command-line time | 13:59 |
openstackgerrit | Samuel Cassiba proposed openstack/openstack-chef-repo master: Support Chef 14 in Test Kitchen scenarios https://review.openstack.org/563154 | 14:01 |
scas | in 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 required | 14:05 |
scas | switching gears, for openstackclient, i would like to use that one to kick off the renaming to eliminate hyphens from the cookbook names | 14:18 |
scas | cookbook-openstack-client was never fully retired, but left at openstack_client | 14:19 |
scas | to me, it seems like the best home with the least amount of confusion between an overcloud deployer and python-openstackclient | 14:19 |
jklare | i hate renaming | 14:44 |
jklare | :) | 14:44 |
jklare | it always means a lot of refactoring in a lot of places | 14:45 |
scas | :D | 14:52 |
scas | chef-client hates hyphens more | 14:52 |
scas | the main reason i'm bringing up renaming again is because it would have eventually happened if the attrition did not | 14:56 |
jklare | scas so if i understand correctly you want to rename "cookbook-openstackclient" to just "openstackclient" ? | 15:37 |
jklare | scas which would be just renaming the repo | 15:38 |
scas | no. i'd to follow what was done here and rename cookbook-openstackclient accordingly: https://github.com/openstack/cookbook-openstack-client/blob/master/metadata.rb#L2 | 15:39 |
jklare | scas but why not just keep the name "openstackclient" ? | 15:39 |
jklare | scas i see no reason to add an "_" to the mix | 15:40 |
jklare | scas in addition to that, i am not sure if we want to reuse the repo of the old "openstack_client" cookbook for the new one | 15:41 |
jklare | scas 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" repo | 15:42 |
scas | that seems reasonable enough. i was overthinking it, of course | 15:46 |
jklare | i think this way we also avoid confusion, since its a new cookbook with a new name in a new repo | 15:47 |
jklare | :) | 15:47 |
scas | it does continue to be a pain with regular expressions that don't key off of just 'cookbook-**' | 15:47 |
scas | i'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 new | 15:48 |
jklare | scas no rush, i think the number of community members that depend on a quick addition of this cookbook is quite limited ;) | 15:50 |
scas | the constraints are primarily on my own time | 15:56 |
openstackgerrit | Merged openstack/cookbook-openstack-compute stable/pike: cleanup attribute leftovers that are not used anymore https://review.openstack.org/563887 | 16:39 |
openstackgerrit | Merged openstack/cookbook-openstack-compute stable/pike: Refactor libvirtd.conf creation https://review.openstack.org/563932 | 16:39 |
openstackgerrit | Merged openstack/openstack-chef-repo master: Increase timeout for rake job https://review.openstack.org/563952 | 16:45 |
*** noodle has quit IRC | 18:09 | |
*** spheromak has quit IRC | 20:04 | |
*** spheromak has joined #openstack-chef | 20:06 | |
*** noodle has joined #openstack-chef | 20:33 | |
*** os-chef-bot has quit IRC | 23:19 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!