| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm-infra master: Ceph-client: create pools after mgr up https://review.openstack.org/605841 | 01:07 |
|---|---|---|
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: Gate: Tolerate failures in log collection tasks https://review.openstack.org/608034 | 01:44 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: Gate: Remove unused helm chart deployment role and playbook https://review.openstack.org/608032 | 01:44 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: Gate: remove unused pull images role and playbook https://review.openstack.org/608033 | 01:44 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: WIP: refactor gate runner https://review.openstack.org/608045 | 01:46 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [Calico] Update to Calico v3.2 https://review.openstack.org/607065 | 01:50 |
| *** unicell has joined #openstack-helm | 02:18 | |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [DNM] impatient https://review.openstack.org/607612 | 02:19 |
| *** unicell has quit IRC | 02:22 | |
| *** unicell has joined #openstack-helm | 02:45 | |
| *** unicell has quit IRC | 02:46 | |
| *** unicell has joined #openstack-helm | 02:46 | |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm-infra master: Ceph-client: create pools after mgr up https://review.openstack.org/605841 | 02:55 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: WIP: refactor gate runner https://review.openstack.org/608045 | 02:58 |
| openstackgerrit | Tin Lam proposed openstack/openstack-helm master: Specification: Value File Ordering https://review.openstack.org/552485 | 02:59 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: WIP: refactor gate runner https://review.openstack.org/608045 | 03:05 |
| openstackgerrit | Merged openstack/openstack-helm master: Specification: Value File Ordering https://review.openstack.org/552485 | 03:13 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: WIP: refactor gate runner https://review.openstack.org/608045 | 03:17 |
| openstackgerrit | Tin Lam proposed openstack/openstack-helm master: Add apparmor security annotation https://review.openstack.org/602365 | 03:26 |
| openstackgerrit | Hari proposed openstack/openstack-helm-addons master: Inclusion annotation for upgrade and install chart changes https://review.openstack.org/604849 | 03:50 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: WIP: refactor gate runner https://review.openstack.org/608045 | 03:50 |
| *** unicell has joined #openstack-helm | 04:09 | |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Gate: Tolerate failures in log collection tasks https://review.openstack.org/608034 | 04:17 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm master: Libvirt: Remove libvirt dockerfile from repo https://review.openstack.org/607567 | 04:22 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Gate: refactor gate script running https://review.openstack.org/608045 | 04:30 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Gate: refactor gate script running https://review.openstack.org/608045 | 04:32 |
| *** roman_g has quit IRC | 04:41 | |
| *** kukacz has quit IRC | 04:52 | |
| *** openstackgerrit has quit IRC | 04:52 | |
| *** kukacz has joined #openstack-helm | 04:57 | |
| *** openstackgerrit has joined #openstack-helm | 04:57 | |
| *** unicell has quit IRC | 06:16 | |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Gate: Remove unused helm chart deployment role and playbook https://review.openstack.org/608032 | 06:43 |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Gate: remove unused pull images role and playbook https://review.openstack.org/608033 | 06:57 |
| openstackgerrit | Merged openstack/openstack-helm-infra master: [Calico] Update to Calico v3.2 https://review.openstack.org/607065 | 06:57 |
| *** roman_g has joined #openstack-helm | 07:07 | |
| *** tmorin has joined #openstack-helm | 07:10 | |
| *** tmorin has left #openstack-helm | 07:12 | |
| *** cfriesen has quit IRC | 08:21 | |
| *** aojeagarcia has joined #openstack-helm | 08:26 | |
| *** lemko has joined #openstack-helm | 09:01 | |
| *** rgogunskiy has joined #openstack-helm | 09:14 | |
| *** gkadam has joined #openstack-helm | 09:43 | |
| openstackgerrit | Achal proposed openstack/openstack-helm-infra master: Libvirt: Make --listen a configurable paramter for libvirtd https://review.openstack.org/607971 | 10:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup jobs https://review.openstack.org/608232 | 11:07 |
| evrardjp | portdirect: it seems our work will overlap | 11:17 |
| evrardjp | should we sync? | 11:17 |
| portdirect | Yeah | 11:18 |
| evrardjp | so I've cleaned up the jobs in openstack-helm, to see how bad it is. It's quite okay. I see you've worked there :) | 11:19 |
| evrardjp | I'd like to move all the job definitions into openstack-helm-infra | 11:19 |
| evrardjp | so we only keep the project definition in openstack-helm | 11:20 |
| evrardjp | what do you think of this? | 11:20 |
| evrardjp | the idea is to have really lightweight definitions, and standardized across all the repos | 11:20 |
| evrardjp | it reduces the amount of plays too -- your playbook osh-infra-gate-runner can become an osh-gate-runner | 11:21 |
| evrardjp | then, in said playbooks, we can be smart into what to run, so either gate_scripts is getting passed, or we have a series of defaults | 11:22 |
| evrardjp | alternatively, keeping the job definitions with the project BUT using the playbooks directly from the infra project would do the trick, and be easier to follow for people | 11:23 |
| portdirect | This sounds good, and where I was wanting to head too :D | 11:23 |
| portdirect | Though I think your skills will get us there more quickly on the zuul/ansible side | 11:24 |
| portdirect | I think the former option you mentioned sounds more manageable over time? | 11:25 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup jobs https://review.openstack.org/608232 | 11:25 |
| evrardjp | the former is more manageable over time, but is harder to read for new people: they will not know what the job do, nor how to add more | 11:27 |
| evrardjp | This can be fixed with proper documentation | 11:27 |
| evrardjp | also it removes the duplication everywhere -- openstack-helm-infra becomes the place where the gating data happens | 11:27 |
| evrardjp | could you clarify what is the purpose you wanted to give to openstack-helm-infra ? | 11:28 |
| evrardjp | I mean it's one or the other, it doesn't really matter. But I'd like to make sure I am heading to the right direction: 1) simplify jobs, 2) simplify their playbooks, 3) simplify how they run | 11:29 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup jobs https://review.openstack.org/608232 | 11:30 |
| evrardjp | what I like with the patch I have, is that I already see we have cross definitions, but it was not explicit | 11:32 |
| evrardjp | haha | 11:32 |
| evrardjp | catching bugs! :) | 11:32 |
| evrardjp | so maybe we should take a decision: keep the jobs definitions separate, or group them | 11:33 |
| evrardjp | maybe cores have an opinion srwilkers for example was working on gates too IIRC | 11:33 |
| portdirect | Yeah, I think this needs a consensus | 11:34 |
| portdirect | I'm drinking coffee and trying to make arbitary judgement - never a good thing ;) | 11:34 |
| portdirect | I think an etherpad may help here | 11:35 |
| evrardjp | ok I can draft my plan there maybe | 11:38 |
| portdirect | I think this would be best, it may also provide some opportunity to divide effort | 11:39 |
| evrardjp | https://etherpad.openstack.org/p/helm-gates-refactor | 11:39 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup jobs https://review.openstack.org/608232 | 12:16 |
| srwilkers | Morning | 12:19 |
| srwilkers | reading scrollback | 12:25 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Gate: refactor gate script running https://review.openstack.org/608045 | 12:27 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: [WIP] Wire linters to openstack-helm-infra plays https://review.openstack.org/608254 | 12:32 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Gate: Move to K8s 1.10.8 https://review.openstack.org/608255 | 12:45 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup project jobs for openstack-helm https://review.openstack.org/608256 | 12:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: Cleanup project jobs for openstack-helm https://review.openstack.org/608256 | 12:55 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm master: WIP: move default to Ocata https://review.openstack.org/608257 | 12:57 |
| *** dims_ has quit IRC | 12:58 | |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm master: WIP: move default to Ocata https://review.openstack.org/608257 | 12:58 |
| evrardjp | srwilkers: I asked a question on the etherpad | 12:59 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm master: WIP: move default to Ocata https://review.openstack.org/608257 | 13:02 |
| evrardjp | portdirect: don't worry about moving the default to O, this is gonna be a one liner in the future :p | 13:03 |
| evrardjp | :D | 13:04 |
| portdirect | if it does not get on your toes right now - I'd like to get that in | 13:05 |
| srwilkers | evrardjp: looking now | 13:12 |
| evrardjp | portdirect: it kinda hits on my toes :p | 13:17 |
| portdirect | ok - ignore it for now | 13:17 |
| portdirect | it was 3 sed's :) | 13:17 |
| portdirect | I can run them after your intial ps to bring some sanity to .zuul.yaml is in | 13:18 |
| evrardjp | well I think those defaults can be overriden -- I will explain :) | 13:19 |
| evrardjp | my idea is that by default we point to "latest" images so we catch issues in the image building early. Latest should not be latest openstack though, but a "latest stable" image built | 13:20 |
| evrardjp | so the "frozen" images would be newton/ocata/pike/queens for example | 13:20 |
| evrardjp | and these would ALWAYS be given in values overrides | 13:20 |
| evrardjp | it simplifies the gates: We always pass those :p | 13:20 |
| *** aaronsheffield has joined #openstack-helm | 13:22 | |
| openstackgerrit | Tin Lam proposed openstack/openstack-helm master: Upgrade default from newton to ocata https://review.openstack.org/607312 | 13:29 |
| openstackgerrit | Tin Lam proposed openstack/openstack-helm master: Add apparmor security annotation https://review.openstack.org/602365 | 13:29 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm-infra master: [DNM] test lint simplification https://review.openstack.org/608267 | 13:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: [DNM] test lint simplification https://review.openstack.org/608269 | 13:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm master: [DNM] test lint simplification https://review.openstack.org/608269 | 13:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm-infra master: [DNM] test lint simplification https://review.openstack.org/608267 | 14:00 |
| *** munimeha1 has joined #openstack-helm | 14:05 | |
| *** sthussey has joined #openstack-helm | 14:06 | |
| evrardjp | srwilkers: may I borrow you eyes for a second? | 14:13 |
| srwilkers | evrardjp: of course. whats up? | 14:14 |
| evrardjp | https://review.openstack.org/#/c/608232/4 seems to be failing a lot, and I am wondering if I did something really wrong and that's very hidden, or if that's normal case | 14:14 |
| evrardjp | It looks like the jobs are getting their variables correctly (see for example kolla) | 14:15 |
| evrardjp | (I do not ask you to review this yet, just have a look at the log and help me figure things out) | 14:15 |
| evrardjp | I have thought further about the jobs -- they should be all in the same repo (-infra). It will allow an easier reuse | 14:16 |
| srwilkers | looking through now | 14:18 |
| srwilkers | looks like most of the failures are due to libvirt and nova-compute crashing | 14:19 |
| srwilkers | http://logs.openstack.org/32/608232/4/check/openstack-helm-latest-dev_deploy_ceph-kolla/3839118/primary/pod-logs/openstack/libvirt-4pqr8/libvirt.txt | 14:19 |
| *** dansmith is now known as SteelyDan | 14:20 | |
| evrardjp | so that's not really a fault of mine then | 14:20 |
| evrardjp | wow then it's real bad | 14:20 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [DNM] impatient https://review.openstack.org/607612 | 14:21 |
| srwilkers | portdirect: ^ | 14:21 |
| evrardjp | I have seen this "impatient" patch -- I am curious if we could be smarter in the wait : Wait only for DB, then wait for keystone, then all the hell breaks loose? | 14:21 |
| evrardjp | :) | 14:22 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [MariaDB] Fix non-default setting for admin user https://review.openstack.org/608057 | 14:22 |
| srwilkers | looks like libvirt might just be crashing in the kolla gates. wondering if portdirect's change to the libvirt chart requires us to use the same libvirt image for the kolla gates that we do for the loci gates rather than the kolla image | 14:27 |
| srwilkers | evrardjp: thanks for the heads up | 14:30 |
| openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-client: make pool creation depedent on ceph-mgr service https://review.openstack.org/605841 | 14:41 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Libvirt: Remove libvirt dockerfile from repo https://review.openstack.org/607567 | 14:43 |
| evrardjp | what's the irc nick of Chris Wedgwood? | 14:57 |
| openstackgerrit | Gage Hugo proposed openstack/openstack-helm master: [DNM/WIP/POC] Add syntribos gate check https://review.openstack.org/607702 | 15:08 |
| *** rgogunskiy has quit IRC | 15:12 | |
| srwilkers | evrardjp: anticw | 15:21 |
| *** munimeha1 has quit IRC | 15:22 | |
| portdirect | evrardjp: I'd certainly like to parallise the deployment of charts a lot more than we do currently - this is what i do locally | 15:22 |
| portdirect | the trade off here is again io on the nodepool vms | 15:22 |
| portdirect | we used to cause a lot of lings to fall over on the db-syncs | 15:23 |
| evrardjp | yeah, and helm itself won't give a good orchestration enough | 15:27 |
| evrardjp | too bad we don't use ansible :p | 15:27 |
| *** munimeha1 has joined #openstack-helm | 15:28 | |
| evrardjp | #troll engaged | 15:28 |
| jamesgu__ | portdirect: there are a small number of Docker images in the tools/images directory and they are Ubuntu based only. We want to add suse support, would you recommend to add Suse based Docker file there. I noticed the libvirt has a name convention "Dockerfile.ubuntu.xenial" but wasn't sure that was still the design. | 15:29 |
| evrardjp | jamesgu__: If I understand what portdirect said above, I'd say that OSH wants to get rid of image building as much as possible. Instead I'd pass an override to the values to point to a suse image | 15:30 |
| evrardjp | said image can be built on LOCI or anywhere else (for example, for opensuse OBS kiwi) | 15:30 |
| evrardjp | I am thinking it's a very good idea to get away from building images in OSH | 15:31 |
| jamesgu__ | I'd agree | 15:31 |
| evrardjp | every project its scope | 15:31 |
| evrardjp | I am cleaning up the gates, so it will be easy, in the long term, to pass your own values for distro support | 15:32 |
| jamesgu__ | I'd agree that we push the tool/images to Loci, some of those may also be available from docker community, I don't know if osh needs specialty images of ovsswtich, libvirt etc. for example | 15:33 |
| evrardjp | I'd expect it would be nice to drop an override file in the current folder for example: https://github.com/openstack/openstack-helm/blob/master/tools/overrides/releases/pike/loci.yaml | 15:33 |
| evrardjp | jamesgu__: have a look at the file I linked | 15:34 |
| jamesgu__ | evrardjp: nice. the images I was asking specifically are infra iamges, https://github.com/openstack/openstack-helm/tree/master/tools/images | 15:37 |
| jamesgu__ | for example, libvirt, ovsswitch, kube aio etc. | 15:37 |
| evrardjp | IMO we should build all these images | 15:37 |
| evrardjp | I guess portdirect experience might be worth it there. But I believe the work to be done in OSH should be limited to the update of the value files | 15:39 |
| * portdirect reading | 15:48 | |
| portdirect | evrardjp: this is correct - i until recently was running centos for most images in my home lab | 15:49 |
| portdirect | jamesgu__: yeah the intent behind "Dockerfile.ubuntu.xenial" was that we could add a "Dockerfile.opensuse.tumbleweed" or whatever as desired | 15:50 |
| *** spiette has quit IRC | 15:58 | |
| *** spiette has joined #openstack-helm | 15:59 | |
| evrardjp | portdirect: looks like you'll have images to push in the future :) | 16:17 |
| jamesgu__ | portdirect: do you want to move those images to loci or stay in osh? | 16:17 |
| portdirect | jamesgu__: i think osh for these | 16:18 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: Adding Falco https://review.openstack.org/598297 | 16:18 |
| portdirect | as we have to do some pretty weird things with some of them (libvirt) | 16:19 |
| *** ianychoi_ is now known as ianychoi | 16:19 | |
| evrardjp | portdirect: have you thought of using loci_docker_login secret in OSH gates to built images? | 16:19 |
| portdirect | yes! just never got round to it | 16:19 |
| portdirect | but that would be the target state for these | 16:19 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: fix tox python3 overrides https://review.openstack.org/606688 | 16:19 |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: Disable Ceph RADOS Gateway dynamic bucket resharding. Problem was discovered regarding issues being caused by RGW dynamic bucket resharding. It is at this time recommended to disable this feature. https://review.openstack.org/607001 | 16:20 |
| jamesgu__ | I figured there must be some specialty sauce in those images. we will start to add Dockerfile.opensuse.xxx then. | 16:20 |
| *** gkadam has quit IRC | 16:21 | |
| *** aojeagarcia has quit IRC | 16:32 | |
| *** spiette has quit IRC | 16:34 | |
| *** spiette has joined #openstack-helm | 16:35 | |
| *** dims has joined #openstack-helm | 16:46 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm-infra master: [DNM] test lint simplification https://review.openstack.org/608267 | 16:53 |
| evrardjp | ok it seems my idea for the gates cleanup work | 16:54 |
| *** spiette has quit IRC | 16:59 | |
| *** spiette has joined #openstack-helm | 17:00 | |
| *** sawblade6 has joined #openstack-helm | 17:36 | |
| openstackgerrit | Tin Lam proposed openstack/openstack-helm-infra master: Add LDAP support for k8s-keystone-auth in gate https://review.openstack.org/601419 | 17:39 |
| *** sawblade6 has quit IRC | 17:41 | |
| openstackgerrit | Gage Hugo proposed openstack/openstack-helm master: [DNM/WIP/POC] Add syntribos gate check https://review.openstack.org/607702 | 17:42 |
| *** lemko has quit IRC | 19:37 | |
| *** sawblade6 has joined #openstack-helm | 19:37 | |
| *** sawblade6 has quit IRC | 19:46 | |
| *** sawblade6 has joined #openstack-helm | 20:14 | |
| *** sawblade6 has quit IRC | 20:22 | |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Adding Falco https://review.openstack.org/598297 | 20:24 |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Disable Ceph RADOS Gateway dynamic bucket resharding. Problem was discovered regarding issues being caused by RGW dynamic bucket resharding. It is at this time recommended to disable this feature. https://review.openstack.org/607001 | 20:32 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Libvirt: Fix image https://review.openstack.org/608355 | 20:42 |
| openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Libvirt: Remove libvirt dockerfile from repo https://review.openstack.org/607567 | 20:52 |
| *** munimeha1 has quit IRC | 20:52 | |
| openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Ceph: Remove fluentbit sidecars, mount hostpath for logs https://review.openstack.org/608356 | 20:53 |
| *** aaronsheffield has quit IRC | 20:56 | |
| openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [DNM] impatient https://review.openstack.org/607612 | 21:04 |
| openstackgerrit | Gage Hugo proposed openstack/openstack-helm-infra master: DNM - Replace docker-py with docker https://review.openstack.org/608358 | 21:06 |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm-infra master: Restrict Gnocchi Ceph access scope to what is needed only https://review.openstack.org/608364 | 21:28 |
| *** sawblade6 has joined #openstack-helm | 22:19 | |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm-infra master: Restrict libvirt Ceph access scope to what is needed only https://review.openstack.org/608373 | 22:23 |
| *** sawblade6 has quit IRC | 22:25 | |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm master: Restrict Cinder, Glance and libvirt Ceph access scope to what is needed only https://review.openstack.org/608374 | 22:26 |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm master: Restrict Cinder, Glance and libvirt Ceph access scope. https://review.openstack.org/608374 | 22:30 |
| openstackgerrit | Jean-Charles Lopez proposed openstack/openstack-helm-infra master: Restrict libvirt Ceph access scope to what is needed only. https://review.openstack.org/608373 | 22:31 |
| *** cfriesen has joined #openstack-helm | 22:44 | |
| openstackgerrit | Gage Hugo proposed openstack/openstack-helm-infra master: Replace docker-py with docker https://review.openstack.org/608358 | 23:01 |
| openstackgerrit | Renis Makadia proposed openstack/openstack-helm-infra master: [WIP] Ceph-RGW chart cleanup. https://review.openstack.org/608379 | 23:01 |
| openstackgerrit | Renis Makadia proposed openstack/openstack-helm-infra master: [WIP] Ceph-RGW chart cleanup. https://review.openstack.org/608379 | 23:26 |
| *** roman_g has quit IRC | 23:35 | |
| *** sthussey has quit IRC | 23:36 | |
| openstackgerrit | Renis Makadia proposed openstack/openstack-helm master: [WIP] Ceph-rgw cleanup - update deploy script https://review.openstack.org/608385 | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!