Thursday, 2019-06-20

*** asidorenko has quit IRC03:17
*** mutin-s has quit IRC08:39
*** mutin-s has joined #openstack-chef08:39
*** mutin-sa has joined #openstack-chef08:41
*** mutin-s has quit IRC08:45
*** mutin-sa has quit IRC08:45
*** mutin-sa has joined #openstack-chef08:45
*** mutin-sa has quit IRC09:26
*** mutin-sa has joined #openstack-chef09:27
*** asidorenko has joined #openstack-chef10:39
*** asidorenko has quit IRC10:41
*** asidorenko has joined #openstack-chef10:57
openstackgerritJan Klare proposed openstack/cookbook-openstackclient master: WIP: testpatch  https://review.opendev.org/66655912:56
jklareRamereth i am currently reviewing your patch here https://review.opendev.org/#/c/665826/ but i do not understand why this is needed13:33
jklareRamereth do you have a link to a log or something with the error message or issue?13:34
jklarei guess i do not really understand why we need these depends on13:42
openstackgerritJan Klare proposed openstack/cookbook-openstackclient master: WIP: testpatch  https://review.opendev.org/66655913:46
jklareRamereth ok, i understand the patch in libraries/openstack_base.rb , but i think there is no need for any depends-on in this patch13:52
jklareRamereth i think i am getting closer to understanding the issue, but will have to get back on this tomorrow :) i think the main issue we want to solve here is the locked version of fog-openstack in chefdk14:12
jklarebut i also think there might be an easier way than to install chef ourselves14:13
jklarei will try to put together a patch tomorrow14:13
Ramerethjklare: how else would you install chef ourselves?15:29
RamerethIMHO we shouldn't be using chefdk for our integration tests because of the differences that get packaged in gems between that and installing chef stand alone if that's what you're referring to15:32
RamerethThe depends-on isn't technically needed for the patch but to fix the integration job run which fails without everything being used15:33
RamerethThe fog-openstack version lock issue didn't show up in our tests until we switched over to using chef proper. Also this better aligns with how these cookbooks get used in production15:34
Ramerethit should hopefully catch issues like this more quickly in the future15:35
openstackgerritLance Albertson proposed openstack/cookbook-openstackclient master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66582615:50

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