* Ramereth swears he's either doing the Depends-On incorrectly or it's not working properly | 00:15 | |
openstackgerrit | Merged openstack/cookbook-openstack-dns master: Drop auth version attribute https://review.opendev.org/665496 | 04:04 |
---|---|---|
*** asidorenko has joined #openstack-chef | 09:09 | |
jklare | Ramereth please stop merging your own patches without any reviews | 10:37 |
jklare | Ramereth 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 reviews | 10:38 |
jklare | Ramereth 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 changed | 10:40 |
Ramereth | jklare calbers: apologies for merging without any reviews. I won't be doing that anymore and no hard feelings (I hope). | 15:30 |
Ramereth | I 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 |
jklare | Ramereth i totally understand why you did it | 15:31 |
jklare | and as i said, no big deal | 15:32 |
jklare | calbers, frickler and me are usually around but do not post too much if there is nothing to discuss | 15:32 |
jklare | but we usually try to review stuff as fast as possible | 15:32 |
jklare | if you want to get something done faster, just ping one or all of us here in the channel | 15:33 |
Ramereth | will do! | 15:33 |
jklare | we are usually available during european working hours | 15:33 |
Ramereth | what is that typically? | 15:34 |
jklare | but all have a irc. bouncer, so will get your messages either way | 15:34 |
jklare | usually between 8am and 4pm UTC | 15:35 |
* Ramereth nodes | 15:35 | |
Ramereth | *nods | 15:35 |
jklare | which is currently 10am to 6pm CEST for us :) | 15:35 |
Ramereth | :) | 15:36 |
jklare | not sure if they actually agreed on throwing away this idea of daylight saving time | 15:36 |
jklare | did not follow the discussion lately | 15:36 |
Ramereth | so 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 |
jklare | i think the biggest blocker for getting rocky done is that nobody of us has installed in on a real production grade hardware setup | 15:37 |
Ramereth | I will be doing that soon for sure | 15:37 |
jklare | we usually try to get the feedback of at least one company or person if everything works in a real deployment before releasing it | 15:37 |
Ramereth | Do you normally go through the install guides on each release and make any changes based on that? | 15:38 |
jklare | so i think frickler was planning on installing it in one of our staging environments in the next weeks | 15:38 |
jklare | yeah, we try to follow the changes and recommendations from the projects | 15:38 |
jklare | at least the big ones | 15:38 |
*** mgagne has quit IRC | 15:39 | |
Ramereth | right | 15:39 |
jklare | and try to change the cookbooks accordingly | 15:39 |
jklare | frickler is also a infra core reviewer | 15:39 |
jklare | so he usually has a pretty good idea of what changed in other projects | 15:39 |
Ramereth | I was hoping to add centos back to the integration tests at some point since that's what we use | 15:40 |
jklare | yeah, sounds good | 15:40 |
Ramereth | i'm also confused about how depends-on works | 15:40 |
jklare | i would love to see the testing for centos back in place | 15:40 |
Ramereth | I tested the recent fog-openstack changes locally w/o any issue but it fails on zuul | 15:40 |
jklare | what confuses you? | 15:40 |
Ramereth | how the changes actually get pulled in. I see bootstrap.sh does the magic, but I can't see where that gets triggered in the pipeline | 15:41 |
jklare | iirc its done via zuul environment variables | 15:41 |
jklare | but i did not look into it for quite some time | 15:41 |
Ramereth | I 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 version | 15:42 |
*** mgagne has joined #openstack-chef | 15:42 | |
Ramereth | I was going to test that theory locally to see if that was the case | 15:42 |
Ramereth | is the only reason we use chefdk on the integration test job is for berks to do cookbook downloading? | 15:43 |
jklare | interesting question | 15:44 |
jklare | i have no instant answer to that | 15:44 |
jklare | but might be | 15:44 |
Ramereth | because if that's the case, it wouldn't be hard to just add a gem install for berkshelf | 15:44 |
jklare | i will need to read the code to figure that out | 15:44 |
Ramereth | at any rate, i was going to test that today and figure it out | 15:44 |
jklare | i think we stopped rolling our own ruby and gems to avoid the dependency mess that comes with it | 15:45 |
Ramereth | right, doing that with chefdk is a nightmare | 15:45 |
jklare | before chefdk we had gemfiles in all of our repos | 15:45 |
jklare | and used to install ruby on all testing vms | 15:46 |
jklare | that was even worse | 15:46 |
Ramereth | yuck | 15:46 |
Ramereth | FWIW not sure if you've been following the licensing changes with Chef, but i'm also helping with the community build of Chef 15 | 15:46 |
jklare | i think we moved to whatever comes with chefdk to ease the bootstrapping process | 15:46 |
jklare | i saw the changes for the chef license | 15:47 |
jklare | but did not read all of it | 15:47 |
jklare | so basically everything is open source now | 15:47 |
jklare | but the binaries are proprietary | 15:47 |
jklare | right? | 15:47 |
Ramereth | correct, similar to the RHEL model | 15:47 |
jklare | ok | 15:48 |
Ramereth | any binaries created need not infringe on the Chef branding and trademark | 15:48 |
Ramereth | our working name is cinc (Cinc Is Not Chef) but that may change. The org has been called cc-builds or Chef Community Builds | 15:48 |
Ramereth | http://downloads.cc-build.org/ for some initial alpha builds | 15:49 |
Ramereth | anyway, I wanted to bring that up | 15:49 |
jklare | looks interesting | 15:51 |
jklare | and this cinc binary is a chefdk or a chef-server? | 15:51 |
jklare | or a chef-client? | 15:51 |
Ramereth | https://gitlab.com/cc-build | 15:51 |
Ramereth | currently client but we'll work on chefdk/chef-server eventually | 15:51 |
jklare | ok | 15:51 |
jklare | cool | 15:51 |
Ramereth | we have a WIP with our inspec replacement (outspec) but we can't use that name as Chef already told us no. | 15:51 |
jklare | lol | 15:52 |
jklare | yeah, i guess its too close | 15:52 |
Ramereth | which is sad as I loved that name | 15:52 |
Ramereth | btw, 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 that | 15:53 |
jklare | oh wow, finally | 15:53 |
jklare | thats cool | 15:54 |
Ramereth | I met several of the sous-chef folks at ChefConf last month, so i've been trying to get synced up with them on things | 15:54 |
jklare | yeah, i think we will wait for the stein cycle to refacor based on that | 15:54 |
Ramereth | I 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 fixed | 15:55 |
Ramereth | it's not good to restart apache on the controller node after every run :| | 15:55 |
jklare | yeah | 15:57 |
jklare | agree | 15:57 |
jklare | how about the name kitchencritic for the inspec project? | 15:57 |
Ramereth | I ended up having to use delete_resource in my wrapper cookbook to work around that on queens | 15:57 |
jklare | or would that also be too close to chef stuff | 15:57 |
Ramereth | https://gitlab.com/cc-build/organization-subjects/issues/1 for the latest on the naming | 15:57 |
jklare | thanks | 15:59 |
jklare | i will call it a day now, but will try to read up on your initial questions regarding depends-on tomorrow | 15:59 |
jklare | i will hopefully have a good answer for you tomorrow | 16:00 |
Ramereth | np, thanks for the feedback and again sorry for the headaches I may have caused | 16:00 |
jklare | no worries, thanks for the contributions :) | 16:00 |
*** altlogbot_1 has quit IRC | 17:00 | |
*** altlogbot_2 has joined #openstack-chef | 17:01 | |
*** irclogbot_0 has quit IRC | 17:02 | |
*** irclogbot_1 has joined #openstack-chef | 17:02 | |
Ramereth | I 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 tests | 19:10 |
openstackgerrit | Lance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests https://review.opendev.org/666176 | 22:00 |
Ramereth | that will likely fail due to the fog-openstack issues ^ | 22:01 |
*** calbers has quit IRC | 22:06 | |
*** calbers has joined #openstack-chef | 22:08 | |
openstackgerrit | Lance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests https://review.opendev.org/666176 | 22:27 |
Ramereth | looks 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 PR | 22:45 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-image master: Fixes to support fog-openstack-1.x https://review.opendev.org/666182 | 22:47 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-network master: Fixes to support fog-openstack-1.x https://review.opendev.org/666183 | 22:49 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-compute master: Fixes to support fog-openstack-1.x https://review.opendev.org/666184 | 22:50 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-bare-metal master: Fixes to support fog-openstack-1.x https://review.opendev.org/666185 | 22:51 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-integration-test master: Fixes to support fog-openstack-1.x https://review.opendev.org/666188 | 22:52 |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-block-storage master: Fixes to support fog-openstack-1.x https://review.opendev.org/666189 | 22:53 |
openstackgerrit | Lance Albertson proposed openstack/openstack-chef master: Install and use Chef Client instead of ChefDK on integration tests https://review.opendev.org/666176 | 22:56 |
*** calbers has quit IRC | 23:25 | |
*** jklare has quit IRC | 23:25 | |
*** calbers has joined #openstack-chef | 23:26 | |
*** jklare has joined #openstack-chef | 23:26 | |
openstackgerrit | Lance Albertson proposed openstack/cookbook-openstack-identity master: Fixes to support fog-openstack-1.x https://review.opendev.org/665827 | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!