| *** shakamunyi has joined #openstack-chef | 00:18 | |
| *** shakamunyi has quit IRC | 00:23 | |
| *** Ryjedo has joined #openstack-chef | 00:28 | |
| *** Ryjedo has quit IRC | 00:37 | |
| *** ChrissKO_ has joined #openstack-chef | 00:50 | |
| *** ChrissKO_ has quit IRC | 00:55 | |
| *** zhiwei has joined #openstack-chef | 01:10 | |
| *** shakamunyi has joined #openstack-chef | 01:18 | |
| *** shakamunyi has quit IRC | 01:23 | |
| *** jmickle has quit IRC | 02:15 | |
| *** The_8th_Dwarf has quit IRC | 02:17 | |
| *** jmickle has joined #openstack-chef | 02:17 | |
| *** shakamunyi has joined #openstack-chef | 02:19 | |
| *** openstackstatus has quit IRC | 02:19 | |
| *** shakamunyi has quit IRC | 02:24 | |
| *** The_8th_Dwarf has joined #openstack-chef | 02:30 | |
| *** ChrissKO_ has joined #openstack-chef | 02:38 | |
| *** ChrissKO_ has quit IRC | 02:43 | |
| *** shakamunyi has joined #openstack-chef | 03:18 | |
| *** shakamunyi has quit IRC | 03:23 | |
| *** jmickle has quit IRC | 04:05 | |
| *** ChrissKO_ has joined #openstack-chef | 04:26 | |
| *** ChrissKO_ has quit IRC | 04:31 | |
| *** shakamunyi has joined #openstack-chef | 05:18 | |
| *** shakamunyi has quit IRC | 05:23 | |
| *** nkrinner has joined #openstack-chef | 05:27 | |
| *** infra-red has joined #openstack-chef | 05:33 | |
| *** infra-red has quit IRC | 06:07 | |
| *** ChrissKO_ has joined #openstack-chef | 06:17 | |
| *** shakamunyi has joined #openstack-chef | 06:18 | |
| *** shakamunyi has quit IRC | 06:23 | |
| *** infra-red has joined #openstack-chef | 06:53 | |
| *** infra-red has joined #openstack-chef | 06:57 | |
| *** infra-red has quit IRC | 07:04 | |
| *** infra-red has joined #openstack-chef | 07:07 | |
| *** shakamunyi has joined #openstack-chef | 07:18 | |
| *** shakamunyi has quit IRC | 07:23 | |
| openstackgerrit | Merged openstack/cookbook-openstack-image: Restart glance immediately https://review.openstack.org/302866 | 07:53 |
|---|---|---|
| openstackgerrit | Merged openstack/cookbook-openstack-network: Ubuntu mitaka package name changes for openvswitch agent https://review.openstack.org/302371 | 07:54 |
| openstackgerrit | Merged openstack/openstack-chef-repo: Differentiate between Ubuntu and CentOS for integration https://review.openstack.org/291939 | 07:54 |
| openstackgerrit | Merged openstack/openstack-chef-repo: Add a few more projects to the logs https://review.openstack.org/299496 | 07:54 |
| openstackgerrit | Merged openstack/cookbook-openstack-image: Use admin user for glance uploads https://review.openstack.org/301892 | 08:01 |
| openstackgerrit | Merged openstack/cookbook-openstack-common: Adds distro-specific changes to make CI work https://review.openstack.org/291921 | 08:11 |
| *** openstackgerrit has quit IRC | 08:33 | |
| *** openstackgerrit has joined #openstack-chef | 08:33 | |
| *** zhiwei has quit IRC | 09:32 | |
| *** ChrissKO_ has quit IRC | 10:49 | |
| *** ChrissKO_ has joined #openstack-chef | 10:49 | |
| *** jmb__ has quit IRC | 10:54 | |
| *** ChrissKO_ has quit IRC | 11:16 | |
| *** ChrissKO_ has joined #openstack-chef | 11:16 | |
| *** shakamunyi has joined #openstack-chef | 11:18 | |
| *** shakamunyi has quit IRC | 11:23 | |
| *** shakamunyi has joined #openstack-chef | 12:18 | |
| *** shakamunyi has quit IRC | 12:23 | |
| *** zhiwei has joined #openstack-chef | 12:28 | |
| *** zhiwei has quit IRC | 12:33 | |
| *** Ryjedo has joined #openstack-chef | 12:37 | |
| *** ovaistariq has joined #openstack-chef | 13:04 | |
| *** zhiwei has joined #openstack-chef | 13:11 | |
| *** zhiwei has quit IRC | 13:16 | |
| *** ChrissKO_ has quit IRC | 13:25 | |
| *** nkrinner has quit IRC | 13:25 | |
| *** os-chef-bot has quit IRC | 13:34 | |
| *** os-chef-bot has joined #openstack-chef | 13:34 | |
| openstackgerrit | Jens Rosenboom proposed openstack/cookbook-openstack-identity: Remove the deprecated "verbose" option https://review.openstack.org/303443 | 14:01 |
| *** ovaistariq has quit IRC | 14:21 | |
| *** ovaistariq has joined #openstack-chef | 14:22 | |
| *** ovaistariq has quit IRC | 14:22 | |
| sc` | frickler: looks like that identity change is failing a rspec test | 14:24 |
| markvan | sc`: I respun a few to get the latest test results for them... | 14:25 |
| sc` | yup, see 'em in zuul queued up | 14:25 |
| sc` | looks like we're ahead of puppet in terms of what rdo build we're using. i'm pulling from testing, which is destined to become ga packages if/when their ci passses | 14:27 |
| *** jmb_ has joined #openstack-chef | 14:33 | |
| *** jmb_ has quit IRC | 14:38 | |
| openstackgerrit | Jens Rosenboom proposed openstack/cookbook-openstack-network: Do not hardcode any DHCP options for Neutron https://review.openstack.org/303481 | 15:06 |
| sc` | ubuntu integration is failing to start apache for keystone. maybe another race condition | 15:09 |
| markvan | sc`: looking at the ubuntu logs now | 15:10 |
| markvan | sc`: I've noticed that the combined 3 pass is quite large now, maybe we should consider breaking that up into the pass logs as well at some point | 15:11 |
| sc` | looking through some of the logs, i see lots of deprecation notices. going to have to spend some time in newton updating that stuff | 15:14 |
| markvan | sc`: humm, failing on the second pass, so it was up and running after the first pass tests. And it's the dashboard port 80 it's complaining about. | 15:15 |
| markvan | I recall we had an issue like this before with keystone and horizon fighting over apache ports | 15:15 |
| sc` | if it is, it shouldn't be. horizon should be taking port 80/443 and keystone 5000/35357 | 15:16 |
| sc` | troubling | 15:16 |
| markvan | yeah, I think the issue then was about restarting apache during certain config steps, which caused it to overlap with itself | 15:17 |
| markvan | actually, why does the 2nd,3rd pass need to restart apache? Maybe that is part of the issue? | 15:18 |
| sc` | it's the notifies | 15:18 |
| sc` | it doesn't _need_ to, but seems to do so regardless | 15:18 |
| markvan | yeah, that is a bug imo | 15:19 |
| sc` | i had an issue with an adapted server-apache.rb where it would restart apache every chef run | 15:19 |
| sc` | we had to do some ugly hacks | 15:19 |
| sc` | now, it checks to see if a file has been modified to determine if it should reload | 15:19 |
| markvan | humm, look at the ports.conf, 80 is specified twice, oops that I think that is the issue | 15:19 |
| sc` | yeah. i see Listen *:80 and then a Listen 0.0.0.0:80 | 15:20 |
| markvan | yup, that *:80 is a default thingy we need to get outa there | 15:21 |
| sc` | seems like it'd make apache barf. i haven't deliberately misconfigured apache in ages | 15:21 |
| sc` | https://github.com/svanzoest-cookbooks/apache2/blob/master/templates/default/ports.conf.erb | 15:22 |
| sc` | it's coming from a node attribute | 15:22 |
| sc` | smells like a bug in our stuff | 15:23 |
| markvan | sc`: I think it's here: https://github.com/openstack/cookbook-openstack-dashboard/blob/master/recipes/apache2-server.rb#L49 our unique port checking is not covering the diff between *:80 and 0.0.0.0:80 | 15:24 |
| sc` | nopr | 15:25 |
| sc` | nope | 15:25 |
| sc` | bug in apache2 | 15:25 |
| sc` | we need to override the default node attribute for node['apache']['listen'] | 15:26 |
| sc` | https://github.com/svanzoest-cookbooks/apache2/blob/master/attributes/default.rb#L279 | 15:26 |
| sc` | we may as well work around it. the maintainer is reluctant to fixing things reported | 15:26 |
| markvan | Since we don't ever use the "*" address syntax, I think we should just remove that from the list explicitly for now? | 15:27 |
| sc` | yup | 15:27 |
| *** jmb_ has joined #openstack-chef | 15:27 | |
| sc` | 0.0.0.0:80 takes its place anyway. so we don't really need that default attribute from apache2 | 15:27 |
| *** zhiwei has joined #openstack-chef | 15:27 | |
| markvan | right, so maybe we could even just remove that first line #51 which pulls in the defaults? or could that be used by some other cookbook/recipe to setup a different port? | 15:28 |
| sc` | yeah, looks like that'd do it | 15:29 |
| markvan | maybe best to just remove this explicit one for now "*:80" | 15:29 |
| sc` | yup. it needs to be fixed at the apache2 level, but that ain't happening | 15:29 |
| markvan | ok, I'll pop up a patch to workaround this for now | 15:30 |
| *** zhiwei has quit IRC | 15:32 | |
| *** infra-red has quit IRC | 15:37 | |
| markvan | sc`: https://review.openstack.org/#/c/303512/ yeah, still don't like that we force keystone/apache to restart | 15:58 |
| sc` | yeah. was thinking it should be converted to a subscribes | 15:58 |
| sc` | that looks okay for now. ideally, we'd get that fixed upstream, but... | 15:59 |
| markvan | yup, we have the template/conf resource that should be triggering this, but I see the comment in the code about lack of apache cookbook support for this | 15:59 |
| sc` | yeah. that's from kilo, didn't get refactored | 16:00 |
| sc` | apache2 has been updated significantly since then, so it may be fixed | 16:00 |
| markvan | humm, to fix it, I think they would need to allow dynamic subscribe resources that we would provide | 16:01 |
| markvan | have not looked at apache2 cb in a while | 16:01 |
| sc` | i've only been looking into it very recently | 16:01 |
| sc` | just scratching the surface | 16:02 |
| markvan | can we add a subscribes to a definition? | 16:03 |
| sc` | right... definition... hm | 16:04 |
| sc` | probably not | 16:04 |
| markvan | from doc: "Starting with chef-client 12.5, it is recommended to build custom resources instead of definitions." and one reason is not support for notify/subscribe/not_if/only_if | 16:07 |
| markvan | apache2 cookbook needs a refactor.... | 16:07 |
| sc` | apache2 needs a new maintainer | 16:10 |
| markvan | humm, I see that we have the same code here for the default port: https://github.com/openstack/cookbook-openstack-identity/blob/master/recipes/server-apache.rb#L240 | 16:12 |
| markvan | probably should patch that as well for *:80 in case of keystone and horizon on diff nodes, right? | 16:13 |
| sc` | yup | 16:13 |
| markvan | k, on it | 16:13 |
| sc` | right on. thanks | 16:13 |
| openstackgerrit | Mark Vanderwiel proposed openstack/cookbook-openstack-identity: Remove default apache port https://review.openstack.org/303524 | 16:18 |
| *** shakamunyi has joined #openstack-chef | 16:18 | |
| *** shakamunyi has quit IRC | 16:23 | |
| *** jmickle has joined #openstack-chef | 16:40 | |
| openstackgerrit | Merged openstack/cookbook-openstack-compute: Cleanup options deprecated for mitaka https://review.openstack.org/301641 | 16:40 |
| sc` | markvan: looks like we have green builds once more | 16:53 |
| markvan | sc`: yup, was now looking into why the c7 side is taking 2x longer for pass 1, seems odd | 16:55 |
| sc` | is it consistently slower, or just slow because of the pipeline's load? | 16:56 |
| sc` | yesterday builds were dog slow for me | 16:56 |
| sc` | i respun common and it was enqueued for over 2 hours yesterday. builds were pretty similar in times. maybe a 10 minute variance | 16:57 |
| markvan | c7: 430/578 resources updated in 15 minutes u: 371/591 resources updated in 06 minutes | 16:58 |
| markvan | looks like a bit more resources are pulled in on c7 | 16:59 |
| openstackgerrit | Mark Vanderwiel proposed openstack/openstack-chef-repo: Only run setup tempest once https://review.openstack.org/303541 | 17:11 |
| openstackgerrit | Mark Vanderwiel proposed openstack/openstack-chef-repo: Only run setup tempest once https://review.openstack.org/303541 | 17:15 |
| openstackgerrit | Mark Vanderwiel proposed openstack/openstack-chef-repo: lint cleanups https://review.openstack.org/303547 | 17:20 |
| *** zhiwei has joined #openstack-chef | 17:44 | |
| *** zhiwei has quit IRC | 17:48 | |
| *** jmb_ has quit IRC | 17:55 | |
| *** jmb_ has joined #openstack-chef | 17:59 | |
| *** The_8th_Dwarf has quit IRC | 18:06 | |
| *** jmb_ has quit IRC | 18:07 | |
| sc` | nice. integrations are still all green with those patches | 18:27 |
| sc` | that's... weird. did a recheck on https://review.openstack.org/#/c/295061 and it's trying to kick off apt-get on the centos build | 18:33 |
| sc` | and the ubuntu one is using chefdk 0.4.0 | 18:35 |
| sc` | right... kilo. integration won't work there | 18:36 |
| * sc` +1 coffee... too cloudy and rainy | 18:40 | |
| *** jmb_ has joined #openstack-chef | 19:35 | |
| *** zhiwei has joined #openstack-chef | 20:00 | |
| *** zhiwei has quit IRC | 20:04 | |
| *** infra-red has joined #openstack-chef | 20:07 | |
| *** ovaistariq has joined #openstack-chef | 20:26 | |
| *** infra-re_ has joined #openstack-chef | 20:34 | |
| *** infra-red has quit IRC | 20:37 | |
| *** ovaistariq has quit IRC | 21:14 | |
| *** jmickle has quit IRC | 21:50 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/openstack-chef-repo: Minor fixups for test patch tool https://review.openstack.org/303639 | 22:18 |
| *** shakamunyi has joined #openstack-chef | 22:18 | |
| *** jmickle has joined #openstack-chef | 22:21 | |
| *** infra-re_ has quit IRC | 22:21 | |
| *** shakamunyi has quit IRC | 22:22 | |
| sc` | yum is timing out, failing the centos build | 23:29 |
| sc` | saw that this morning | 23:29 |
| sc` | the failure is at the same place, too. installing openstack-nova-common | 23:30 |
| *** jmickle has quit IRC | 23:30 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!