| *** iniazi has quit IRC | 00:15 | |
| *** iniazi has joined #openstack-kolla | 00:20 | |
| *** zhanglong has quit IRC | 00:32 | |
| *** jbadiapa has joined #openstack-kolla | 00:41 | |
| *** k_mouza has joined #openstack-kolla | 01:23 | |
| *** k_mouza has quit IRC | 01:32 | |
| *** unicell has joined #openstack-kolla | 02:07 | |
| *** unicell has quit IRC | 02:07 | |
| *** unicell has joined #openstack-kolla | 02:07 | |
| *** unicell has quit IRC | 02:08 | |
| *** janki has quit IRC | 02:44 | |
| *** wolverineav has joined #openstack-kolla | 02:55 | |
| *** Sravan has quit IRC | 03:21 | |
| *** Sravan has joined #openstack-kolla | 03:22 | |
| *** vblando has joined #openstack-kolla | 03:23 | |
| vblando | tried doing a 2 node setup (controller, compute) and disabled haproxy (enable_haproxy=no) but kola still trying to use VIP, is this a normal behavior? Is setting up VIP still required even without haproxy on? | 03:24 |
|---|---|---|
| *** dciabrin has quit IRC | 03:33 | |
| *** dciabrin has joined #openstack-kolla | 03:33 | |
| *** andrein has joined #openstack-kolla | 03:59 | |
| *** andrein has quit IRC | 03:59 | |
| *** janki has joined #openstack-kolla | 04:00 | |
| *** unicell has joined #openstack-kolla | 04:49 | |
| *** Sravan has joined #openstack-kolla | 04:56 | |
| *** wolverineav has quit IRC | 05:02 | |
| *** wolverineav has joined #openstack-kolla | 05:02 | |
| *** wolverineav has quit IRC | 05:08 | |
| *** JamesBenson has joined #openstack-kolla | 05:11 | |
| *** Sravan has quit IRC | 05:15 | |
| *** JamesBenson has quit IRC | 05:16 | |
| *** skramaja has joined #openstack-kolla | 05:18 | |
| *** ivve has quit IRC | 05:18 | |
| *** janki has quit IRC | 05:20 | |
| *** Sravan has joined #openstack-kolla | 05:21 | |
| *** wolverineav has joined #openstack-kolla | 05:34 | |
| *** JamesBenson has joined #openstack-kolla | 05:37 | |
| *** JamesBenson has quit IRC | 05:41 | |
| mnasiadka | vblando: tried setting the vip to first node ip? | 05:48 |
| *** janki has joined #openstack-kolla | 05:51 | |
| *** jbadiapa has quit IRC | 05:51 | |
| *** unicell has quit IRC | 06:00 | |
| *** wolverineav has quit IRC | 06:07 | |
| *** pcaruana has joined #openstack-kolla | 06:11 | |
| *** Sravan has quit IRC | 06:17 | |
| *** ivve has joined #openstack-kolla | 06:22 | |
| *** wolverineav has joined #openstack-kolla | 06:41 | |
| *** Luzi has joined #openstack-kolla | 06:44 | |
| *** ivve has quit IRC | 06:58 | |
| *** ivve has joined #openstack-kolla | 07:03 | |
| *** JamesBenson has joined #openstack-kolla | 07:03 | |
| *** JamesBenson has quit IRC | 07:07 | |
| *** hjensas has quit IRC | 07:25 | |
| hrw | morning | 07:35 |
| hrw | karbor change merged | 07:36 |
| *** luksky has joined #openstack-kolla | 07:46 | |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: move to Python 3 for binary/source images https://review.openstack.org/642375 | 07:50 |
| hrw | now it should pass | 07:51 |
| hrw | ubuntu builds I mean ;D | 07:51 |
| *** wolverineav has quit IRC | 07:54 | |
| *** hjensas has joined #openstack-kolla | 08:03 | |
| *** openstackgerrit has quit IRC | 08:17 | |
| *** chrizl has joined #openstack-kolla | 08:18 | |
| *** chrizl has quit IRC | 08:22 | |
| *** gfidente has joined #openstack-kolla | 08:24 | |
| *** tosky has joined #openstack-kolla | 08:27 | |
| hrw | kolla-build-ubuntu passed for both binary and source | 08:29 |
| hrw | k-a jobs fail but good to see that it is finally buildable | 08:29 |
| vblando | @mnasiadka: i'll try that | 08:30 |
| mnasiadka | hrw: I was looking at this placement-api wsgi bug... but I need more coffee to understand what is causing it ;) | 08:31 |
| vblando | in the inventory file, there's an entry for [deployment] | 08:33 |
| vblando | localhost ansible_connection=local | 08:33 |
| hrw | mnasiadka: can you look at http://logs.openstack.org/75/642375/15/check/kolla-ansible-ubuntu-source/45ded2d/ and tell me what is wrong? | 08:33 |
| vblando | is that needed even if you deployer is remote? | 08:33 |
| *** gkadam has joined #openstack-kolla | 08:33 | |
| hrw | vblando: that's for all-in-one | 08:33 |
| *** gkadam has quit IRC | 08:33 | |
| hrw | vblando: when you deploy on deployer | 08:34 |
| vblando | hrw: ah, so for multinode, it can be commented out | 08:34 |
| hrw | vblando: if you want all-in-one on remote then change inventory | 08:34 |
| mnasiadka | hrw: that one seems like some network issue, it was waiting for vip to appear after starting/restarting keepalived, but it timed out... | 08:34 |
| *** iniazi has quit IRC | 08:35 | |
| hrw | vblando: that's how I always did. in-repo all-in-one for 'deploy all-in-one on deployer', own all-in-one for 'deploy on remote' and multinode for multi | 08:35 |
| hrw | mnasiadka: dzięki | 08:35 |
| hrw | mnasiadka: it is not first time I see such failure ;( | 08:35 |
| vblando | hrw: both sample inventory files for all-in-one and multinode has that entry | 08:35 |
| mnasiadka | hrw: and it's single node, so that's even more weird ;) | 08:36 |
| mnasiadka | hrw: ah, it's ovh | 08:36 |
| mnasiadka | hrw: try recheck to land at some different nodepool provider ;) | 08:36 |
| mnasiadka | hrw: or ask ovh guys on #openstack-pl to fix their env :D | 08:37 |
| hrw | mnasiadka: ;D) | 08:37 |
| egonzalez | hrw k-a centos binary is failing due an error in the deployment of placement-api | 08:37 |
| hrw | egonzalez: thx | 08:37 |
| hrw | I am happy anyway - my py3 patch passed build phase for first time | 08:37 |
| egonzalez | the package bundles the wsgi file and the name conflict with our deployed wsgi process name | 08:38 |
| mnasiadka | egonzalez: ah, that's the reason - so we should remove the bundled file? ;) | 08:38 |
| egonzalez | asked to tripleo guys and they dont use it, they copy their own file too, so probably is the option | 08:38 |
| egonzalez | or copy ours as 00-placement-api.conf to override the original | 08:39 |
| mnasiadka | maybe that's even better | 08:39 |
| egonzalez | or change wsgi process name to something else | 08:39 |
| mnasiadka | but then it's not in the same manner as all others | 08:40 |
| mnasiadka | or raise a bug to RDO to remove that bundled file :) | 08:40 |
| egonzalez | other rdo packages doesnt have the wsgi file on it, so is probably better remove the original file | 08:40 |
| *** JamesBenson has joined #openstack-kolla | 08:44 | |
| *** JamesBenson has quit IRC | 08:49 | |
| *** dougsz has joined #openstack-kolla | 08:55 | |
| *** iniazi has joined #openstack-kolla | 09:03 | |
| dougsz | thanks for reviewing those kolla patches hrw! | 09:09 |
| hrw | dougsz: yw | 09:10 |
| *** jbadiapa has joined #openstack-kolla | 09:13 | |
| *** dteselkin has quit IRC | 09:32 | |
| *** dteselkin has joined #openstack-kolla | 09:33 | |
| egonzalez | mgoddard hrw mnasiadka i think we cannot make the tripleo job voting because is 3rd party job, trying to find some gobernance rule for that, but other projects doesnt allow -1 to 3rd party job | 09:35 |
| hrw | ok, then stays as is | 09:35 |
| *** priteau has joined #openstack-kolla | 09:37 | |
| egonzalez | jobs can -1 but not -2 | 09:39 |
| egonzalez | dont know if this still applies to zuul v3 but neutron have this in their docs https://docs.openstack.org/neutron/latest/contributor/policies/thirdparty-ci.html#third-party-ci-voting | 09:40 |
| mgoddard | egonzalez: so we could make it voting but not gating? does that even make sense? | 09:40 |
| mgoddard | egonzalez: is third party more about vendors than cross-project though? | 09:41 |
| *** pcaruana has quit IRC | 09:45 | |
| *** pcaruana has joined #openstack-kolla | 09:46 | |
| *** wolverineav has joined #openstack-kolla | 09:51 | |
| *** jbadiapa has quit IRC | 09:55 | |
| *** zbr has quit IRC | 10:08 | |
| *** zbr has joined #openstack-kolla | 10:10 | |
| *** jbadiapa has joined #openstack-kolla | 10:15 | |
| *** zbr has quit IRC | 10:16 | |
| *** jbadiapa has quit IRC | 10:16 | |
| *** k_mouza has joined #openstack-kolla | 10:17 | |
| *** jbadiapa has joined #openstack-kolla | 10:18 | |
| *** zbr has joined #openstack-kolla | 10:22 | |
| *** zbr has quit IRC | 10:22 | |
| *** jbadiapa has quit IRC | 10:25 | |
| *** wolverineav has quit IRC | 10:25 | |
| *** jbadiapa has joined #openstack-kolla | 10:37 | |
| mnasiadka | egonzalez: https://docs.openstack.org/infra/system-config/third_party.html#permissions-on-your-third-party-system | 10:53 |
| *** openstackgerrit has joined #openstack-kolla | 11:02 | |
| openstackgerrit | Michal Nasiadka proposed openstack/kolla-ansible master: Fix placement-api WSGI error https://review.openstack.org/645081 | 11:02 |
| *** iniazi_ has joined #openstack-kolla | 11:05 | |
| *** iniazi has quit IRC | 11:07 | |
| *** Sravan has joined #openstack-kolla | 11:11 | |
| *** Sravan has quit IRC | 11:15 | |
| *** priteau has quit IRC | 11:19 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 11:21 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible stable/rocky: Fix swift reconfigure https://review.openstack.org/645131 | 11:21 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible stable/queens: Fix swift reconfigure https://review.openstack.org/645132 | 11:21 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible stable/pike: Fix swift reconfigure https://review.openstack.org/645133 | 11:21 |
| *** wolverineav has joined #openstack-kolla | 11:23 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 11:31 |
| erolg | Guys when we enable fluentd, we are not configuring oslo.log fluent handler for os services right? Alternatively we are using tail from files with fluentd. | 11:35 |
| erolg | I wondering is there any reason to use this method instead of configure oslo.log fluentd handler | 11:36 |
| mgoddard | erolg: I'm not sure about that. Perhaps the fluentd handler was not available when the logging system was created? | 11:38 |
| *** zbr has joined #openstack-kolla | 11:39 | |
| mgoddard | erolg: I quite like the current setup. It means that if fluentd or your log aggregation system fails you still have a copy of logs | 11:39 |
| mgoddard | erolg: it's quite configurable though, I expect you could probably change it to work how you want via config | 11:39 |
| *** wolverineav has quit IRC | 11:40 | |
| *** luksky has quit IRC | 11:41 | |
| *** k_mouza has quit IRC | 11:46 | |
| erolg | mgoddard, actually we can provide both (file and fluent handler). What do you think? Is this more suitable way for our fully containerized environment? | 11:47 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Fix MariaDB 10.3 upgrade https://review.openstack.org/644250 | 11:48 |
| *** dave-mccowan has joined #openstack-kolla | 11:49 | |
| mgoddard | erolg: what happens if fluentd is down? Do logs get dropped by oslo.log? | 11:50 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 11:52 |
| *** dave-mccowan has quit IRC | 11:53 | |
| *** k_mouza has joined #openstack-kolla | 11:56 | |
| *** k_mouza has quit IRC | 12:05 | |
| erolg | I have nothing to say about that, I have not used fluent with oslo.log before I will digging into it in my free time and compare this two scenario. | 12:05 |
| *** k_mouza_ has joined #openstack-kolla | 12:05 | |
| openstackgerrit | Brent Eagles proposed openstack/kolla stable/queens: Install packages into octavia-api image to support wsgi https://review.openstack.org/645154 | 12:08 |
| openstackgerrit | Brent Eagles proposed openstack/kolla stable/queens: Fix Octavia-API extend_start.sh https://review.openstack.org/645155 | 12:08 |
| *** iniazi_ has quit IRC | 12:09 | |
| *** jbadiapa has quit IRC | 12:10 | |
| *** luksky has joined #openstack-kolla | 12:20 | |
| *** iniazi_ has joined #openstack-kolla | 12:21 | |
| *** k_mouza_ has quit IRC | 12:22 | |
| *** beagles has joined #openstack-kolla | 12:25 | |
| mgoddard | erolg: one nice thing about using the fluentd support in oslo.log is (I assume) we would send logs in a more machine-friendly format | 12:30 |
| *** goldyfruit has quit IRC | 12:31 | |
| erolg | mgoddard, and cpu time? fluentd will not parse the files. | 12:34 |
| mgoddard | erolg: no, but it will have to process whatever is sent to it | 12:34 |
| erolg | mgoddard, hmm ok. | 12:37 |
| *** skramaja has quit IRC | 12:40 | |
| *** S4ren has joined #openstack-kolla | 12:41 | |
| openstackgerrit | Merged openstack/kolla master: install ncurses explicitly for tput for rpm distro https://review.openstack.org/644853 | 12:44 |
| openstackgerrit | Merged openstack/kolla master: Introduce clean_package_cache https://review.openstack.org/636646 | 12:44 |
| openstackgerrit | Merged openstack/kolla master: Update programming language mentioned in setup.cfg https://review.openstack.org/643298 | 12:47 |
| openstackgerrit | Merged openstack/kolla master: Install qemu-system in nova-libvirt on ubuntu https://review.openstack.org/643563 | 12:47 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Fix neutron rolling upgrade https://review.openstack.org/645102 | 12:47 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 12:47 |
| openstackgerrit | Merged openstack/kolla master: Use virtual package for ansible https://review.openstack.org/642874 | 12:48 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/rocky: Pin sensu mailer and slack plugins https://review.openstack.org/645164 | 12:53 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/queens: Pin sensu mailer and slack plugins https://review.openstack.org/645165 | 12:53 |
| *** irclogbot_1 has quit IRC | 13:07 | |
| *** irclogbot_1 has joined #openstack-kolla | 13:09 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Add initial support for upgrade checkers https://review.openstack.org/644528 | 13:13 |
| *** altlogbot_2 has quit IRC | 13:23 | |
| *** iniazi has joined #openstack-kolla | 13:23 | |
| *** altlogbot_0 has joined #openstack-kolla | 13:24 | |
| *** iniazi_ has quit IRC | 13:25 | |
| *** janki has quit IRC | 13:28 | |
| *** k_mouza has joined #openstack-kolla | 13:29 | |
| *** k_mouza has quit IRC | 13:29 | |
| *** jbadiapa has joined #openstack-kolla | 13:29 | |
| *** lemko has joined #openstack-kolla | 13:36 | |
| *** wolverineav has joined #openstack-kolla | 13:37 | |
| *** altlogbot_0 has quit IRC | 13:39 | |
| *** altlogbot_1 has joined #openstack-kolla | 13:40 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Add initial support for upgrade checkers https://review.openstack.org/644528 | 13:40 |
| *** baha has joined #openstack-kolla | 13:41 | |
| *** irclogbot_1 has quit IRC | 13:45 | |
| *** irclogbot_3 has joined #openstack-kolla | 13:46 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 13:49 |
| openstackgerrit | Merged openstack/kolla-ansible master: Fix placement-api WSGI error https://review.openstack.org/645081 | 13:52 |
| *** jistr is now known as jistr|call | 13:52 | |
| *** hjensas has quit IRC | 13:52 | |
| *** jistr|call is now known as jistr | 14:00 | |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: introduce install_packages chain option https://review.openstack.org/645185 | 14:02 |
| *** mjturek has joined #openstack-kolla | 14:02 | |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: Move top package installed in every *-base container to openstack-base https://review.openstack.org/645187 | 14:05 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Fix MariaDB 10.3 upgrade https://review.openstack.org/644250 | 14:09 |
| *** wolverineav has quit IRC | 14:12 | |
| openstackgerrit | Merged openstack/kolla master: monasca, tempest: install librdkafka-dev to get it built for non-x86/source https://review.openstack.org/642826 | 14:12 |
| openstackgerrit | Merged openstack/kolla master: Fix MariaDB 10.3 upgrade https://review.openstack.org/644244 | 14:12 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/rocky: Fix MariaDB 10.3 upgrade https://review.openstack.org/645191 | 14:17 |
| *** iniazi has quit IRC | 14:27 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/rocky: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645164 | 14:33 |
| *** iniazi has joined #openstack-kolla | 14:33 | |
| *** mjturek has quit IRC | 14:33 | |
| *** gfidente has quit IRC | 14:33 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/queens: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645165 | 14:35 |
| *** altlogbot_1 has quit IRC | 14:35 | |
| *** altlogbot_0 has joined #openstack-kolla | 14:36 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/pike: rabbitmq container error. https://review.openstack.org/645197 | 14:37 |
| *** irclogbot_3 has quit IRC | 14:38 | |
| *** gfidente has joined #openstack-kolla | 14:38 | |
| *** irclogbot_1 has joined #openstack-kolla | 14:39 | |
| *** mjturek has joined #openstack-kolla | 14:39 | |
| *** hjensas has joined #openstack-kolla | 14:44 | |
| *** JamesBenson has joined #openstack-kolla | 14:45 | |
| hrw | it is visible that release time is coming ;d | 14:53 |
| hrw | jschlueter: https://review.openstack.org/645187 - that oslo-upgradecheck gets installed as popular dependency in images, not directly - right? | 14:58 |
| jschlueter | hrw: yes as a dep of nova/cinder and others | 15:01 |
| hrw | jschlueter: thx | 15:02 |
| hrw | jschlueter: commented https://review.openstack.org/#/c/645185 | 15:02 |
| *** hjensas has quit IRC | 15:07 | |
| jschlueter | hrw: ack thanks, I don't have a good patch yet for kolla but I'm using something like this in a local branch for introducing yum/dnf -y update all in one line to keep layer count and size down | 15:08 |
| *** wolverineav has joined #openstack-kolla | 15:08 | |
| hrw | jschlueter: will wait for patch which uses it then | 15:08 |
| jschlueter | just this morning I was contemplating introducing a macro or option to the macro to do the update and cleanup ... so we don't have repeated code in multiple spots | 15:09 |
| jschlueter | hrw: ack | 15:09 |
| hrw | jschlueter: imho chaining install_package() is just wrong. list should be joined and installed once | 15:09 |
| jschlueter | hrw: chaining would be ... RUN <do something> && dnf -y update && {{ macros.package_install.... }} | 15:10 |
| hrw | jschlueter: I see that it may be useful for base image. DEB systems first install packages to get APT https working, then install standard stuff. RPM systems install *-release packages and then some stuff | 15:10 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 15:10 |
| jschlueter | hrw: currently the deb has update entry for all calls | 15:11 |
| *** Sravan has joined #openstack-kolla | 15:11 | |
| hrw | jschlueter: yum/dnf does metadata fetching when needed. apt has it explicit | 15:11 |
| jschlueter | hrw: yea I think that area could benifet from something this way but haven't worked through all of the options and use cases yet there | 15:12 |
| jschlueter | hrw: that's right ... | 15:12 |
| hrw | jschlueter: https://review.openstack.org/462567 | 15:12 |
| jschlueter | so atp-get upgrade would be the thing to map through to match dnf -y update | 15:12 |
| hrw | jschlueter: yum/dnf update == (apt update + ) apt upgrade | 15:13 |
| * jschlueter nods | 15:13 | |
| jschlueter | thanks for reminding me ... I knew that but wasn't connecting it when I was reading the code this morning | 15:13 |
| hrw | jschlueter: ;) | 15:14 |
| *** Sravan has quit IRC | 15:15 | |
| hrw | jschlueter: speaking of yum and macros... base image may benefit some cleanups for rpm systems. there are few places asking for macro | 15:15 |
| jschlueter | hrw: yea that's what I was thinking ... let me start there and see what I come up with | 15:18 |
| *** baha has quit IRC | 15:18 | |
| hrw | jschlueter: I wrote a comment about it in some patch but forgot which one | 15:18 |
| jschlueter | hrw: I think the python3 bp that Alex proposed | 15:19 |
| hrw | possible | 15:19 |
| jschlueter | but was out of scope for that patch | 15:19 |
| hrw | jschlueter: still prefer to leave such comment so maybe patch author will follow with another one | 15:20 |
| jschlueter | yep | 15:20 |
| jschlueter | I alway appreciate it when they are pointed out | 15:21 |
| *** Luzi has quit IRC | 15:25 | |
| *** unicell has joined #openstack-kolla | 15:26 | |
| *** jbadiapa has quit IRC | 15:26 | |
| *** unicell has quit IRC | 15:27 | |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: refactor to pull out rpm_security_update macro https://review.openstack.org/645209 | 15:39 |
| jschlueter | hrw, mandre: iterating on the theme here ^^ first pass at that refactor | 15:40 |
| jschlueter | by inspection of a couple of quick template only runs looks to be a clean refacter but feedback welcome | 15:41 |
| *** wolverineav has quit IRC | 15:42 | |
| *** ivve has quit IRC | 15:42 | |
| jschlueter | hrw: in that block there is where I could make use of the chain macro entry for package_install line 175 of base/Dockerfile.j2 ... | 15:43 |
| jschlueter | but I would want to also add cleanup option so you could override and not do cleanup until last one in list of chained ... | 15:43 |
| *** baha has joined #openstack-kolla | 15:44 | |
| *** spiette has quit IRC | 15:47 | |
| *** spiette has joined #openstack-kolla | 15:48 | |
| hrw | jschlueter: commented | 15:52 |
| jschlueter | hrw: thanks | 15:53 |
| *** unicell has joined #openstack-kolla | 15:55 | |
| jschlueter | mandre, hrw: are there any unittests for the macros that I could add a test for the new macro to make sure we don't add extra whitespace at end of the macro regardless of how it's called? | 15:57 |
| *** hjensas has joined #openstack-kolla | 15:58 | |
| hrw | jschlueter: do not know but sounds like very good idea | 15:58 |
| hrw | jschlueter: especially when you read mandre's comments | 15:58 |
| * jschlueter nods | 16:00 | |
| mandre | jschlueter: I do not know off the top of my head about unit testing macros but it would indeed be nice to have | 16:01 |
| * jschlueter nods | 16:02 | |
| *** unicell has quit IRC | 16:06 | |
| *** goldyfruit has joined #openstack-kolla | 16:13 | |
| *** mjturek has quit IRC | 16:16 | |
| openstackgerrit | Merged openstack/kolla-ansible master: Use endpoint_override for nova-compute-ironic https://review.openstack.org/643911 | 16:22 |
| openstackgerrit | Merged openstack/kolla-ansible stable/queens: Create /volumes when using external ceph https://review.openstack.org/643087 | 16:22 |
| openstackgerrit | Merged openstack/kolla-ansible stable/queens: Fix failure to mount cephfs https://review.openstack.org/643088 | 16:22 |
| *** mjturek has joined #openstack-kolla | 16:24 | |
| *** hjensas has quit IRC | 16:24 | |
| *** baha has quit IRC | 16:24 | |
| *** baha has joined #openstack-kolla | 16:35 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/queens: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645165 | 16:38 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/rocky: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645164 | 16:39 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/queens: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645165 | 16:39 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/pike: Fix rabbitmq container error and oraclelinux https://review.openstack.org/645197 | 16:44 |
| *** Sravan has joined #openstack-kolla | 16:48 | |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: refactor to pull out rpm_security_update macro https://review.openstack.org/645209 | 16:49 |
| *** ivve has joined #openstack-kolla | 16:54 | |
| *** luksky has quit IRC | 17:02 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 17:03 |
| *** henriqueof has joined #openstack-kolla | 17:06 | |
| hrw | see you on Monday | 17:12 |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: Fixup whitespace issue in chaining https://review.openstack.org/645242 | 17:17 |
| jschlueter | mandre: found one whitespace issue not in my patch but related to what I was digging on | 17:17 |
| mandre | jschlueter: wow, nice one | 17:24 |
| openstackgerrit | Jon Schlueter proposed openstack/kolla master: cleanup in base for yum install entries https://review.openstack.org/645185 | 17:24 |
| jschlueter | mandre: and here is the other piece one that actually does cleanup for the yum -y install entries in base | 17:25 |
| jschlueter | mandre: yea not sure how to do regex search but shouldn't be too hard if we have construct for multi-line searches .... look for \ continuation mark at end of line followed by 2 line returns in any of the rendered Dockerfiles | 17:26 |
| jschlueter | hrw: ^^ found usage of the chain option in install_packages macro ... to do cleanup of the extra yum -y install entries | 17:28 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Fix MariaDB 10.3 upgrade https://review.openstack.org/644250 | 17:35 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Wait for cinder volume to become available in CI https://review.openstack.org/645277 | 17:35 |
| *** S4ren has quit IRC | 17:41 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 17:42 |
| jschlueter | mgoddard: question do you guys prefer smaller easily verified by visual inspection patches and a patch series if they are tightly related or would you prefer them not stacked up like this set is? | 17:42 |
| *** gmann is now known as gmann_afk | 17:43 | |
| * jschlueter just checking so I'm not flooding the system or causing issues | 17:43 | |
| mgoddard | jschlueter: depends, I can deal with either. If there are going to be lots of iterations then a big patch will be easier on resources | 17:44 |
| * jschlueter nods | 17:44 | |
| jschlueter | ack if I get more than a couple of iterations on this I'll work on collapsing it down otherwise I'll leave them individual improvements that stand alone but don't collide if applied in this order | 17:45 |
| jschlueter | I see another patch but it will need some thinking about it ... so that we can get both rpm and deb blocks to call through to a macro for the upgrade with or without installing packages and cleaning up caches afterwards | 17:46 |
| jschlueter | both tripleo and deb/ubuntu has room for improvement here ... | 17:47 |
| jschlueter | but that's for another day | 17:47 |
| mgoddard | jschlueter: splitting is also good if earlier patches can get merged before later ones | 17:47 |
| jschlueter | incremental improvements is good | 17:47 |
| * jschlueter nods | 17:47 | |
| mgoddard | +1 | 17:47 |
| mgoddard | nice to see you getting more involved in kolla recently jschlueter | 17:47 |
| jschlueter | right now all of these are incrementally refactors that should preserve the same code paths just reduce duplicated code | 17:48 |
| jschlueter | mgoddard: yea I've been learning/using it more and the more that I find problems using it the more I dig in, and for the most part it's been a matter of figuring out how to adjust or fix holes in the scripts that I'm hitting | 17:48 |
| * jschlueter involved with Red Hat OSP | 17:49 | |
| mgoddard | always nice too keep thinks DRY | 17:49 |
| jschlueter | :-) | 17:49 |
| mgoddard | Good stuff, I sometimes wonder how big the tripleo template overrides file is these days, and whether there is stuff in there we could all benefit from in kolla | 17:50 |
| jschlueter | mgoddard: it's bigger than it should be and likely yes | 17:51 |
| jschlueter | mgoddard: this cycle I've collapsed most of the OSP specific overrides into tripleo-common overrides or patches to it ... or into kolla to make life easier | 17:52 |
| mgoddard | jschlueter: maybe something we can discuss at the PTG, if you're there? | 17:52 |
| mgoddard | we're not doing any kolla meetings at the PTG, but I'll be there and could join TripleO people at some point | 17:53 |
| jschlueter | I'm not going to be this year but at some future one, mandre, mwhahaha, or Emillian would likely be interested in it | 17:53 |
| jschlueter | mgoddard: https://github.com/openstack/tripleo-common/blob/master/container-images/tripleo_kolla_template_overrides.j2 750 lines long | 17:54 |
| mgoddard | that would be good. I'll follow up with them | 17:54 |
| *** ivve has quit IRC | 17:54 | |
| mgoddard | jschlueter: ... | 17:54 |
| jschlueter | a lot of healthcheck entries, a STOPSIGNAL entry, a yum -y update in base | 17:54 |
| jschlueter | some of them are naming fixups | 17:55 |
| mgoddard | would love me some healthchecks | 17:55 |
| jschlueter | IIRC the healthcheck scripts are currently in one of the tripleo packages I think tripleo-common | 17:56 |
| jschlueter | also a few blocks adding wsgi type wrappers ... that might be low hanging fruit | 17:56 |
| mgoddard | https://github.com/openstack/tripleo-common/blob/master/healthcheck/cinder-volume | 17:56 |
| jschlueter | but I don't know enough about that just that I've seen it updated some places in kolla and sometimes only in tripleo-common | 17:56 |
| mgoddard | would be great if there were a common way of doing healthchecks | 17:57 |
| * jschlueter nods | 17:57 | |
| jschlueter | https://github.com/openstack/tripleo-common/blob/master/healthcheck/common.sh | 17:57 |
| mgoddard | I seem to remember it being discussed as a cross project goal at some point, not sure what happened | 17:57 |
| * jschlueter nods | 17:59 | |
| jschlueter | I think part of difficulty is healthchecks are dependent on how you deploy/configure things | 17:59 |
| jschlueter | mgoddard: I'm not sure why ect_api doesn't have the httpd/mod_ssl + runtime tweaks embedded in kolla | 18:00 |
| jschlueter | *ec2_api | 18:00 |
| mgoddard | seems like it should do | 18:01 |
| jschlueter | but nova-api does | 18:01 |
| jschlueter | I think there are some patches I've seen in the queue unmerged that add similar httpd/mod_ssl + tweaks ... | 18:02 |
| *** ivve has joined #openstack-kolla | 18:02 | |
| jschlueter | which should almost be a macros at this point | 18:02 |
| jschlueter | since it's repeated in almost the same exact form across many images | 18:02 |
| mgoddard | +1 | 18:02 |
| jschlueter | but beyond my current grasp to make it just work yet | 18:03 |
| mgoddard | I proposed a patch that added a common variables file, similar to macros, but just some shared variables | 18:03 |
| mgoddard | https://review.openstack.org/642713 | 18:04 |
| mgoddard | could be used to create e.g. wsgi_packages variable | 18:04 |
| jschlueter | ack | 18:05 |
| * jschlueter on wind down for this week ... I'm taking PTO next week | 18:05 | |
| * jschlueter EBRAINFULL | 18:06 | |
| mgoddard | nice, enjoy jschlueter | 18:06 |
| jschlueter | mgoddard: but good idea and might be useful for where we have repeated package lists across a lot of container images | 18:06 |
| *** gfidente has quit IRC | 18:14 | |
| *** dougsz has quit IRC | 18:23 | |
| *** ivve has quit IRC | 18:36 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Add CI job for ironic https://review.openstack.org/568829 | 18:36 |
| *** gmann_afk is now known as gmann | 18:40 | |
| *** itlinux has quit IRC | 18:55 | |
| *** sshnaidm|rover is now known as sshnaidm|off | 19:03 | |
| jschlueter | http://logs.openstack.org/09/645209/2/check/kolla-build-oraclelinux-source/b4b62ca/logs/build/000_FAILED_grafana.txt.gz | 19:10 |
| jschlueter | got failure on trying to download/extract INFO:kolla.common.utils.grafana:from url: https://grafana.com/api/plugins/gnocchixyz-gnocchi-datasource/versions/1.7.0/download | 19:10 |
| jschlueter | is that known? | 19:10 |
| openstackgerrit | Mark Goddard proposed openstack/kolla stable/pike: Fix rabbitmq container error, oraclelinux and percona https://review.openstack.org/645197 | 19:17 |
| *** vblando has quit IRC | 19:21 | |
| *** hjensas has joined #openstack-kolla | 19:28 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Add CI job for ironic https://review.openstack.org/568829 | 19:28 |
| *** luksky has joined #openstack-kolla | 19:29 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 19:31 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Check containers during kolla-ansible check https://review.openstack.org/599735 | 19:33 |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Check containers during kolla-ansible check https://review.openstack.org/599735 | 19:33 |
| *** iniazi_ has joined #openstack-kolla | 19:45 | |
| *** iniazi has quit IRC | 19:48 | |
| hrw | jschlueter: I prefer granular patches if possible as they are easier to review. But if you need to make a series of small patches (depending on order) then sometimes it is better to collapse them into smaller amount of bigger patches. | 19:49 |
| hrw | jschlueter: longer the series, longer to review/CI/merge | 19:50 |
| hrw | jschlueter: you do not want to beat my record of 50 (fifty) revisions of one patch | 19:50 |
| openstackgerrit | Merged openstack/kolla stable/rocky: Pin sensu mailer and slack plugins, fix rabbitmq https://review.openstack.org/645164 | 19:51 |
| *** Sravan has quit IRC | 20:00 | |
| *** Sravan has joined #openstack-kolla | 20:04 | |
| *** Sravan has quit IRC | 20:05 | |
| *** Sravan has joined #openstack-kolla | 20:07 | |
| jschlueter | hrw: ack thanks, got it | 20:09 |
| jschlueter | as long as I can keep them clean for number of revisions, conflicts keep them small, and if some order keep them stacked otherwise start collecting into larger patches for review/work load for check/gate jobs | 20:10 |
| jschlueter | hmm looks like there was an issue with grafana plugin install | 20:20 |
| jschlueter | do you guys track what failure modes are most common transient failure modes? | 20:20 |
| *** dciabrin_ has joined #openstack-kolla | 20:25 | |
| *** dciabrin has quit IRC | 20:27 | |
| rwellum | Hi folks, is Kolla-Ansible ok installing on Ubuntu 18.04? I ask because 'bootstrap-servers' is failing due to this version of Ubuntu having removed the 'easy_install' scripts (https://bugs.launchpad.net/ubuntu/+source/python-setuptools/+bug/1774419) | 20:55 |
| openstack | Launchpad bug 1774419 in python-setuptools (Ubuntu) "easy_install command is missing" [Undecided,Confirmed] | 20:55 |
| *** iniazi_ has quit IRC | 21:02 | |
| *** mjturek has quit IRC | 21:06 | |
| *** henriqueof has quit IRC | 21:07 | |
| *** baha has quit IRC | 21:11 | |
| *** iniazi_ has joined #openstack-kolla | 21:12 | |
| *** S4ren has joined #openstack-kolla | 21:31 | |
| *** pcaruana has quit IRC | 21:33 | |
| *** S4ren has quit IRC | 21:33 | |
| *** JamesBenson has quit IRC | 22:01 | |
| *** yankcrime has quit IRC | 22:04 | |
| *** irclogbot_1 has quit IRC | 22:05 | |
| *** mgoddard has quit IRC | 22:08 | |
| *** mgoddard has joined #openstack-kolla | 22:09 | |
| *** ivve has joined #openstack-kolla | 22:11 | |
| *** dciabrin_ has quit IRC | 22:13 | |
| *** dciabrin__ has joined #openstack-kolla | 22:13 | |
| openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: WIP: Test upgrades in CI https://review.openstack.org/616640 | 22:19 |
| *** tolisbar has joined #openstack-kolla | 22:56 | |
| *** tosky has quit IRC | 22:59 | |
| *** ivve has quit IRC | 23:27 | |
| *** luksky has quit IRC | 23:32 | |
| *** goldyfruit has quit IRC | 23:35 | |
| *** JamesBenson has joined #openstack-kolla | 23:40 | |
| *** wolverineav has joined #openstack-kolla | 23:40 | |
| *** JamesBenson has quit IRC | 23:45 | |
| *** wolverineav has quit IRC | 23:45 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!