Tuesday, 2019-06-18

* Ramereth swears he's either doing the Depends-On incorrectly or it's not working properly00:15
openstackgerritMerged openstack/cookbook-openstack-dns master: Drop auth version attribute  https://review.opendev.org/66549604:04
*** asidorenko has joined #openstack-chef09:09
jklareRamereth please stop merging your own patches without any reviews10:37
jklareRamereth i know we had some patches a while ago where we all agreed that +2 from just one reviewer is enough because there were not so many reviewer around, but we stopped that and we never merge our own patches without reviews10:38
jklareRamereth no hard feelings, but if we continue on this road, it gets harder and harder to keep track of what needs to change and what is already changed10:40
Ramerethjklare calbers: apologies for merging without any reviews. I won't be doing that anymore and no hard feelings (I hope).15:30
RamerethI just hadn't seen any activity in a while so I wasn't sure what the status was of the group for reviews and was wanting to make some progress on some fixes.15:31
jklareRamereth i totally understand why you did it15:31
jklareand as i said, no big deal15:32
jklarecalbers, frickler and me are usually around but do not post too much if there is nothing to discuss15:32
jklarebut we usually try to review stuff as fast as possible15:32
jklareif you want to get something done faster, just ping one or all of us here in the channel15:33
Ramerethwill do!15:33
jklarewe are usually available during european working hours15:33
Ramerethwhat is that typically?15:34
jklarebut all have a irc. bouncer, so will get your messages either way15:34
jklareusually between 8am and 4pm UTC15:35
* Ramereth nodes15:35
Ramereth*nods15:35
jklarewhich is currently 10am to 6pm CEST for us :)15:35
Ramereth:)15:36
jklarenot sure if they actually agreed on throwing away this idea of daylight saving time15:36
jklaredid not follow the discussion lately15:36
Ramerethso while I have you here, what is the current status of getting rocky done? What is left to do from your point of view?15:36
jklarei think the biggest blocker for getting rocky done is that nobody of us has installed in on a real production grade hardware setup15:37
RamerethI will be doing that soon for sure15:37
jklarewe usually try to get the feedback of at least one company or person if everything works in a real deployment before releasing it15:37
RamerethDo you normally go through the install guides on each release and make any changes based on that?15:38
jklareso i think frickler was planning on installing it in one of our staging environments in the next weeks15:38
jklareyeah, we try to follow the changes and recommendations from the projects15:38
jklareat least the big ones15:38
*** mgagne has quit IRC15:39
Ramerethright15:39
jklareand try to change the cookbooks accordingly15:39
jklarefrickler is also a infra core reviewer15:39
jklareso he usually has a pretty good idea of what changed in other projects15:39
RamerethI was hoping to add centos back to the integration tests at some point since that's what we use15:40
jklareyeah, sounds good15:40
Ramerethi'm also confused about how depends-on works15:40
jklarei would love to see the testing for centos back in place15:40
RamerethI tested the recent fog-openstack changes locally w/o any issue but it fails on zuul15:40
jklarewhat confuses you?15:40
Ramerethhow the changes actually get pulled in. I see bootstrap.sh does the magic, but I can't see where that gets triggered in the pipeline15:41
jklareiirc its done via zuul environment variables15:41
jklarebut i did not look into it for quite some time15:41
RamerethI suspect the problem is that the integration test is running it with chefdk rather than chef proper and fog-openstack is already baked in and locked in the older version15:42
*** mgagne has joined #openstack-chef15:42
RamerethI was going to test that theory locally to see if that was the case15:42
Ramerethis the only reason we use chefdk on the integration test job is for berks to do cookbook downloading?15:43
jklareinteresting question15:44
jklarei have no instant answer to that15:44
jklarebut might be15:44
Ramerethbecause if that's the case, it wouldn't be hard to just add a gem install for berkshelf15:44
jklarei will need to read the code to figure that out15:44
Ramerethat any rate, i was going to test that today and figure it out15:44
jklarei think we stopped rolling our own ruby and gems to avoid the dependency mess that comes with it15:45
Ramerethright, doing that with chefdk is a nightmare15:45
jklarebefore chefdk we had gemfiles in all of our repos15:45
jklareand used to install ruby on all testing vms15:46
jklarethat was even worse15:46
Ramerethyuck15:46
RamerethFWIW not sure if you've been following the licensing changes with Chef, but i'm also helping with the community build of Chef 1515:46
jklarei think we moved to whatever comes with chefdk to ease the bootstrapping process15:46
jklarei saw the changes for the chef license15:47
jklarebut did not read all of it15:47
jklareso basically everything is open source now15:47
jklarebut the binaries are proprietary15:47
jklareright?15:47
Ramerethcorrect, similar to the RHEL model15:47
jklareok15:48
Ramerethany binaries created need not infringe on the Chef branding and trademark15:48
Ramerethour working name is cinc (Cinc Is Not Chef) but that may change. The org has been called cc-builds or Chef Community Builds15:48
Ramerethhttp://downloads.cc-build.org/ for some initial alpha builds15:49
Ramerethanyway, I wanted to bring that up15:49
jklarelooks interesting15:51
jklareand this cinc binary is a chefdk or a chef-server?15:51
jklareor a chef-client?15:51
Ramerethhttps://gitlab.com/cc-build15:51
Ramerethcurrently client but we'll work on chefdk/chef-server eventually15:51
jklareok15:51
jklarecool15:51
Ramerethwe have a WIP with our inspec replacement (outspec) but we can't use that name as Chef already told us no.15:51
jklarelol15:52
jklareyeah, i guess its too close15:52
Ramerethwhich is sad as I loved that name15:52
Ramerethbtw, the apache2 cookbook finally got converted to resource/library based so that will be a fun thing to migrate to. I was hoping we'd wait until Stein for that15:53
jklareoh wow, finally15:53
jklarethats cool15:54
RamerethI met several of the sous-chef folks at ChefConf last month, so i've been trying to get synced up with them on things15:54
jklareyeah, i think we will wait for the stein cycle to refacor based on that15:54
RamerethI was having some issues with the apache restart code we have and was goign to propose some changes on that after I get this fog-openstack fixed15:55
Ramerethit's not good to restart apache on the controller node after every run :|15:55
jklareyeah15:57
jklareagree15:57
jklarehow about the name kitchencritic for the inspec project?15:57
RamerethI ended up having to use delete_resource in my wrapper cookbook to work around that on queens15:57
jklareor would that also be too close to chef stuff15:57
Ramerethhttps://gitlab.com/cc-build/organization-subjects/issues/1 for the latest on the naming15:57
jklarethanks15:59
jklarei will call it a day now, but will try to read up on your initial questions regarding depends-on tomorrow15:59
jklarei will hopefully have a good answer for you tomorrow16:00
Ramerethnp, thanks for the feedback and again sorry for the headaches I may have caused16:00
jklareno worries, thanks for the contributions :)16:00
*** altlogbot_1 has quit IRC17:00
*** altlogbot_2 has joined #openstack-chef17:01
*** irclogbot_0 has quit IRC17:02
*** irclogbot_1 has joined #openstack-chef17:02
RamerethI was able to duplicate the issue with using chefdk. Looks like upgrading chefdk doesn't fix it either. I'll propose something that uses chef proper for integration tests19:10
openstackgerritLance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests  https://review.opendev.org/66617622:00
Ramereththat will likely fail due to the fog-openstack issues ^22:01
*** calbers has quit IRC22:06
*** calbers has joined #openstack-chef22:08
openstackgerritLance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests  https://review.opendev.org/66617622:27
Ramerethlooks like adding the depends there works. I'm going to push my other fog-openstack PRs and then link those back to this openstack-chef repo PR22:45
openstackgerritLance Albertson proposed openstack/cookbook-openstack-image master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618222:47
openstackgerritLance Albertson proposed openstack/cookbook-openstack-network master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618322:49
openstackgerritLance Albertson proposed openstack/cookbook-openstack-compute master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618422:50
openstackgerritLance Albertson proposed openstack/cookbook-openstack-bare-metal master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618522:51
openstackgerritLance Albertson proposed openstack/cookbook-openstack-integration-test master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618822:52
openstackgerritLance Albertson proposed openstack/cookbook-openstack-block-storage master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66618922:53
openstackgerritLance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests  https://review.opendev.org/66617622:56
*** calbers has quit IRC23:25
*** jklare has quit IRC23:25
*** calbers has joined #openstack-chef23:26
*** jklare has joined #openstack-chef23:26
openstackgerritLance Albertson proposed openstack/cookbook-openstack-identity master: Fixes to support fog-openstack-1.x  https://review.opendev.org/66582723:44

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