Thursday, 2019-05-16

*** unicell has joined #openstack-helm00:09
*** unicell has left #openstack-helm00:09
*** dpawlik has joined #openstack-helm00:38
*** dpawlik has quit IRC00:42
*** dpawlik has joined #openstack-helm00:54
*** dpawlik has quit IRC00:59
openstackgerritKim Ha-gun proposed openstack/openstack-helm master: Octavia: add octavia chart and deployment scripts  https://review.opendev.org/63082102:06
*** cfriesen has quit IRC02:10
*** dpawlik has joined #openstack-helm03:11
*** dpawlik has quit IRC03:15
*** unicell has joined #openstack-helm04:34
*** unicell has quit IRC04:43
*** unicell has joined #openstack-helm05:10
*** unicell has quit IRC05:17
*** dpawlik has joined #openstack-helm05:28
*** gkadam has quit IRC05:49
*** gkadam has joined #openstack-helm05:51
*** aojea has joined #openstack-helm06:02
*** pgaxatte has joined #openstack-helm06:41
*** openstackgerrit has quit IRC07:03
*** mbuil has joined #openstack-helm07:17
mbuil#join #opnfv-pharos07:17
*** alisanhaji has left #openstack-helm07:19
*** witek has joined #openstack-helm07:21
evrardjpmbuil:  /join :p07:23
mbuilevrardjp: I know :P07:23
mbuilmornings...07:24
*** ataraday has joined #openstack-helm07:39
*** gkadam has quit IRC07:55
*** jsuchome has joined #openstack-helm07:57
*** gkadam has joined #openstack-helm08:45
*** openstackgerrit has joined #openstack-helm08:45
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: Wait for pods for openstack-support scenario  https://review.opendev.org/65928408:45
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: Point to OSH-images images  https://review.opendev.org/65876608:50
evrardjpgeorgk: hey10:31
evrardjpit seems the ovs image you updated for dpdk is failing10:31
evrardjpif I run with previous image, everything works10:31
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: [DNM] Test if 2.8.1 is behaving better.  https://review.opendev.org/65882010:43
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Revert "Adding support for DPDK to openvswitch image"  https://review.opendev.org/65952610:47
evrardjpgeorgk: ^10:48
evrardjpHagunKim: I believe that we can use a similar method as OpenStack-Ansible, fetching from https://github.com/openstack/openstack-ansible-os_octavia/blob/0b5766859e465e1a26eb64e66888fd2777c5cbb2/defaults/main.yml#L24111:26
evrardjp(for test purposes only)11:26
georgkevrardjp: hi. are you refering to the failing gate jobs?11:37
evrardjpgeorgk: yes11:48
evrardjpthe openstack-helm-infra-openstack-support job11:49
evrardjpmy goal is, as I said in the commit message, arrive to a fully integrated pipeline from image building to image consumption in the helm charts11:49
evrardjpbut we are not there yet11:49
evrardjpso it's possible that images break stuff11:49
evrardjpcurrently we haven't moved to consume images from osh-images, so it's possible that we completely forget stuff and things break... But I want to change that to improve stability11:50
*** aaronsheffield has joined #openstack-helm12:50
openstackgerrityann degat proposed openstack/openstack-helm-infra master: helm-toolkit: fix k8s rolebinding bad SA ref  https://review.opendev.org/65955613:01
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm-images master: ceph: bump kubernetes version for ceph images  https://review.opendev.org/65955713:07
evrardjpone vote remaining for Ocata fixing: https://review.opendev.org/#/c/659278/ ... thanks lamt for already voting! :)13:08
*** howell has joined #openstack-helm13:09
*** itlinux has quit IRC13:10
evrardjpat the contrary, lamt could you vote on https://review.opendev.org/#/c/659526/ ? :)13:10
openstackgerritYi Wang proposed openstack/openstack-helm-infra master: Add a configmap for mariadb ingress controller  https://review.opendev.org/65956013:15
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm-images master: ceph: bump kubernetes version for ceph images  https://review.opendev.org/65955713:16
*** kranthikirang has joined #openstack-helm13:21
*** ianychoi has joined #openstack-helm13:26
*** kguttikonda has joined #openstack-helm13:27
*** kranthikirang has quit IRC13:30
openstackgerritMerged openstack/openstack-helm master: Extend OpenSUSE Leap15 testing  https://review.opendev.org/65367613:31
*** gkadam has quit IRC13:33
*** sreejithp has joined #openstack-helm13:34
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: Patroni inclusion work for HA Postgres  https://review.opendev.org/64438813:36
*** nmimi has quit IRC13:53
openstackgerritJiri Suchomel proposed openstack/openstack-helm-infra master: Sync wait-for-pods script with the one from openstack-helm  https://review.opendev.org/65956813:53
georgkevrardjp: I noticed this also some time ago14:17
georgkevrardjp: I looked into this, but when I run the gate jobs locally, it worked fine14:17
evrardjpgeorgk: likewise14:17
evrardjpwhich means somethign very weird is happening in gates14:18
georgkevrardjp: mmh14:18
evrardjpin the meantime I propose a revert -- and I think we should have this in a different image14:18
evrardjpthe revert works fine14:18
georgka dpdk-specific image?14:18
evrardjpyes14:18
evrardjpuntil we have a pipeline that tests things properly I mean14:19
georgkso, since I actually need this image, I am wondering if I should just create this separate image for the time being14:19
*** aojea has quit IRC14:20
evrardjpyes this is what I meant :)14:20
*** AOjeaGarcia has joined #openstack-helm14:20
evrardjphaving a separate dpdk image temporarily14:21
georgkok, wanted to make sure14:21
evrardjpor definitively that's your pick :)14:21
evrardjpI could totally imagine a "dpdk" scenario where we override the default image to that image14:22
georgkabsolutely14:22
*** itlinux has joined #openstack-helm14:27
georgkll14:30
georgkopps14:30
*** itlinux has quit IRC14:31
evrardjp:)14:32
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: [WIP] Freeze Images with a date  https://review.opendev.org/65957814:33
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: [WIP] Freeze Images with a date  https://review.opendev.org/65957814:38
openstackgerritMerged openstack/openstack-helm-images master: Revert "Adding support for DPDK to openvswitch image"  https://review.opendev.org/65952614:38
evrardjpportdirect: can you vote on this one too? https://review.opendev.org/#/c/659278/14:40
evrardjpwith this one in, and the ovs rebuilt, all should be back to normal14:40
openstackgerritGeorg Kunz proposed openstack/openstack-helm-images master: Adding temporary individual OVS image with DPDK support  https://review.opendev.org/65958114:41
georgkevrardjp: could you take a look at this and let me know if this works for you? ^14:42
evrardjpalready did14:42
evrardjp:)14:42
evrardjphavent' seen any typo :p14:43
georgk:-)14:43
georgkwell, copy and paste mostly14:43
evrardjpmost of my errors come from copy paste :p14:43
evrardjpjoke aside...14:44
portdirectevrardjp: done :)14:44
evrardjpthis is good14:44
evrardjpportdirect: thanks for the help and dedication there14:44
georgkok, thanks14:44
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: Add tempest job and gate to zuul as non-voting  https://review.opendev.org/65342814:57
*** itxaka has joined #openstack-helm15:04
evrardjpitxaka: reviewed ^15:12
*** cfriesen has joined #openstack-helm15:12
*** itxaka has quit IRC15:13
*** itxaka has joined #openstack-helm15:13
itxakaevrardjp: answered. It was agreed on the meeting to have it on the gate, the idea was to enable it asap, even on that patch. But we keep running in circles with people saying "enable voting" and other people "disable voting" all the time15:17
itxakaIm keeping it like that for now, not gonna change it back again for the third of fourth time from non-voting to voting ¬_¬15:18
itxakaI just want a simple enabled job for tempest, dont care if it votes for now, we can always enable it later15:19
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: [WIP] Freeze images with a date  https://review.opendev.org/65957815:23
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: [WIP] Freeze images with a date  https://review.opendev.org/65957815:24
*** mbuil has quit IRC15:25
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph : Add wait.resources.type to armada manifest  https://review.opendev.org/65787315:26
*** witek has quit IRC15:35
georgkportdirect: in the context of reverting the OVS DPDK changes, could you please take a look at this temporary solution so that I keep an image I can pull :-)  https://review.opendev.org/65958115:35
evrardjpitxaka: it consumes infra power and gating pipelines for no reason. I can't vote positively on that15:35
*** roman_g has quit IRC15:36
evrardjpEither we make it voting or we don't, I agree.15:36
evrardjpAnd I rather make it voting.15:36
evrardjpBut non-voting and in gates pipelines is nonsense15:36
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Freeze images with a date  https://review.opendev.org/65957815:40
itxakaI do agree, but I dont want to keep flipping it between vt and nvt, so Im changing it one last time to voting and if someone has a problem with it then the patch can be left there for that person to take care of it, something so simple should not be a timesink like this!15:42
evrardjpagreed15:43
evrardjpbut let's be respectful of other projects too15:43
evrardjpI don't criticize the back and forth15:43
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: Add tempest job and gate to zuul as voting  https://review.opendev.org/65342815:43
evrardjpI criticized the fact that you set it in gates for non-voting.15:43
evrardjpnot sure if I was clear or not15:43
evrardjpit seems I wasn't15:44
itxakano, dont get me wrong, Im not mad at you or at your request, in fact its pretty sensible to have either nvt on jobs only or full vt on jobs+gate, makes no sense otherwise15:45
itxakaIm just mad at a simple patch taking so much of my time15:45
itxakanext time I just need to learn to stand my ground and say no to request for changes if they dont make much sense or I will end like with this patch, back and forth15:47
*** pgaxatte has quit IRC15:56
openstackgerritMerged openstack/openstack-helm-images master: Add pycrypto into Ocata images explicitly  https://review.opendev.org/65927816:00
evrardjpitxaka: I know the feeling16:03
*** witek has joined #openstack-helm16:03
*** dims has quit IRC16:05
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: Point to OSH-images images  https://review.opendev.org/65876616:06
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: Point to OSH-images images  https://review.opendev.org/65876616:07
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Do not hardcode libvirt version  https://review.opendev.org/65961516:18
evrardjpgreat, libvirt is broken too. Fix here ^16:18
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Do not hardcode libvirt version  https://review.opendev.org/65961516:19
evrardjpgod that little libvirt failure looks like an ugly red dot in a sea of greens in gating :)16:22
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-infra master: Adapt rabbitmq test for py2 and py3 support  https://review.opendev.org/65930116:29
*** witek has quit IRC16:30
*** AOjeaGarcia has quit IRC16:30
evrardjpportdirect: It could be nice if you could answer the question in https://review.opendev.org/#/c/659284/ , so that I can adapt my commit.16:31
cfriesenis there documentation describing how to use helm-toolkit.utils.daemonset_overrides or do I just try to follow the existing usages?16:32
evrardjpI am following the existing usages16:32
evrardjpbut I find that this is lacking16:32
evrardjppatches welcomed? :D16:32
cfriesen:)16:32
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Add periodic building  https://review.opendev.org/65542616:34
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Freeze images with a date  https://review.opendev.org/65957816:35
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Add periodic building  https://review.opendev.org/65542616:35
evrardjpOne last series of things to review from my side: This stack of patches: https://review.opendev.org/#/c/655426/ , the one on the bottom is basically fixing libvirt jobs, the rest are new features to improve stability.16:36
evrardjpcores ^16:37
evrardjpthanks evgenyl for the reviews already :)16:37
evrardjpevgenyl: if you need anything, please ping16:37
*** michael-beaver has joined #openstack-helm16:55
*** KeithMnemonic has joined #openstack-helm17:01
*** dims has joined #openstack-helm17:03
*** dims has quit IRC17:12
*** dims has joined #openstack-helm17:13
openstackgerritMerged openstack/openstack-helm-infra master: Kibana: set read-only-fs  https://review.opendev.org/65766617:17
openstackgerritMerged openstack/openstack-helm-infra master: Expose Anti-Affinity Weight Setting  https://review.opendev.org/65851717:17
-openstackstatus- NOTICE: Gerrit is being restarted to add gitweb links back to Gerrit. Sorry for the noise.17:36
*** HagunKim has quit IRC18:13
*** ianychoi has quit IRC18:15
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-images master: Build Script and Docker file for openstack-utility containers  https://review.opendev.org/65640718:45
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: [WIP] Add helm tests for Ceph Provisioners chart  https://review.opendev.org/63673518:46
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: Add helm tests for Ceph Provisioners chart  https://review.opendev.org/63673518:46
openstackgerritJagan Mohan Kavva proposed openstack/openstack-helm-images master: Add Dockerfile and make file for etcdctl-Utility container  https://review.opendev.org/65933618:47
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: Add helm tests for Ceph Provisioners chart  https://review.opendev.org/63673518:52
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: Fluent-logging: Fix read-only-fs  https://review.opendev.org/65964219:12
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: Patroni inclusion work for HA Postgres  https://review.opendev.org/64438819:17
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: Ceph-Client: Update, Enable and Cleanup helm tests  https://review.opendev.org/64759619:32
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: Patroni inclusion work for HA Postgres  https://review.opendev.org/64438819:32
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: Patroni inclusion work for HA Postgres  https://review.opendev.org/64438819:38
openstackgerritJagan Mohan Kavva proposed openstack/openstack-helm-images master: Add Dockerfile and make file for etcdctl-Utility container  https://review.opendev.org/65933619:54
*** jsuchome has quit IRC20:03
*** kguttikonda has quit IRC20:18
openstackgerritMerged openstack/openstack-helm-images master: mini-mirror: Ignore src config for publishing  https://review.opendev.org/64396720:43
openstackgerritKrishna Venkata proposed openstack/openstack-helm-infra master: [WIP] Implementing and parameterize Unavailable value for Ceph components  https://review.opendev.org/65849221:07
*** fdegir has quit IRC21:10
*** fdegir has joined #openstack-helm21:11
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: Rabbit: Ensure node has joined cluster on initial startup  https://review.opendev.org/65967121:12
*** howell has quit IRC21:13
*** howell has joined #openstack-helm21:13
*** nishantkr has joined #openstack-helm21:29
*** howell has quit IRC21:32
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-Client: Update, Enable and Cleanup helm tests  https://review.opendev.org/64759621:41
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Add helm tests for Ceph Provisioners chart  https://review.opendev.org/63673521:45
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: Rabbit: Ensure node has joined cluster on initial startup  https://review.opendev.org/65967121:47
openstackgerritCliff Parsons proposed openstack/openstack-helm-infra master: Postgresql backup/restore enhancements  https://review.opendev.org/65523121:58
*** sreejithp has quit IRC22:06
*** aaronsheffield has quit IRC23:30

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