Tuesday, 2019-04-16

*** cheng1 has quit IRC00:24
*** cheng1 has joined #openstack-helm00:26
*** roman_g has quit IRC00:39
*** Liang__ has joined #openstack-helm00:48
*** unicell has joined #openstack-helm01:29
*** unicell has quit IRC01:33
*** unicell has joined #openstack-helm02:06
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Revert "readOnlyRootFilesystem: true for openvswitch chart"  https://review.openstack.org/65279202:28
*** cfriesen has quit IRC02:52
openstackgerritMerged openstack/openstack-helm-infra master: Revert "readOnlyRootFilesystem: true for openvswitch chart"  https://review.openstack.org/65279204:43
openstackgerritTaewoo Lee proposed openstack/openstack-helm master: Add ipxe file path variable and installation codes for CentOS  https://review.openstack.org/65282205:04
*** jsuchome has joined #openstack-helm06:03
*** hogepodge has quit IRC06:32
*** aojea has joined #openstack-helm06:41
*** pgaxatte has joined #openstack-helm06:52
*** hogepodge has joined #openstack-helm06:56
*** Talion has joined #openstack-helm07:01
pgaxattehello07:02
pgaxattejust read the irc logs, thanks fdegir for the tip on Depends-On ;)07:02
*** pma78 has joined #openstack-helm07:04
openstackgerritPierre Gaxatte proposed openstack/openstack-helm-infra master: Add registry authentication support  https://review.openstack.org/65158507:05
fdegirpgaxatte: sure07:13
fdegirpgaxette: one thing i’m not so sure is whether the location of Depends-On matters or not07:13
fdegirpgaxette: i normally put it before change-id07:14
pgaxattefdegir: i put the depends-on in the commit message of https://review.openstack.org/#/c/65158507:14
pgaxattein the hope that zuul will understand that it depends on another repo07:14
pgaxattebecause 651585 is a change on openstack-helm-infra and 651587 on openstack-helm07:14
fdegirpgaxatte: just checked your change on zuul and it doesn’t look like it worked07:15
fdegiryes, zuul can do work across multiple repos07:15
fdegiri have a similar review and it got tested it correctly07:16
fdegirhttps://review.openstack.org/65206307:16
fdegirin my case, the change the above one depends on was displayed that way so that’s why i say your change might not be tested in the way you want07:17
pgaxatteok we'll see if this works :)07:17
*** alisanhaji has joined #openstack-helm07:33
*** Darcidride has joined #openstack-helm07:47
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job  https://review.openstack.org/65270507:49
*** roman_g has joined #openstack-helm07:58
nmimiHi all! just a dumb question if someone knows:08:05
nmimiI am working in task : XCI Introduction of Openstack-helm as installer(multinode) and currently i am trying to deploy tempest chart.08:06
nmimiIn tempest/values.yaml there is a section(line 125) tempest: auth:  # admin_username value set by configmap-etc08:06
nmimiadmin_username: None08:06
nmimiand in tempest/templates/configmap-etc.yaml :08:06
nmimi{{- if empty .Values.conf.tempest.auth.admin_username -}}08:06
nmimi{{- $_ := set .Values.conf.tempest.auth "admin_username" .Values.endpoints.identity.auth.admin.username -}}08:06
nmimiMy point is that during deployment, configmap-etc search for empty values (instead of None) and as a result it doesn't fill in the appropriate values of keystone authentication.08:06
nmimiIf i change the tempest/values.yaml from admin_username: None -> admin_username: '' , everything works fine.08:06
nmimiHave you seen this issue before? Is there another way to get the correct values?08:06
*** gkadam has joined #openstack-helm08:09
*** gkadam has quit IRC08:10
pgaxattefdegir: apparently the depends-on is not taken into account: http://logs.openstack.org/85/651585/7/check/openstack-helm-infra-openstack-support/151dca0/job-output.txt.gz#_2019-04-16_07_08_05_71835308:27
pgaxatteit is taking the master branch08:27
openstackgerritPierre Gaxatte proposed openstack/openstack-helm-infra master: Add registry authentication support  https://review.openstack.org/65158508:37
nick_karHi all08:44
nick_karworking on the same as nmimi, it is also found that the current tempest image used by multinode/900-tempest.sh https://github.com/openstack/openstack-helm/blob/94ac3569f5bca5509636f9c78f4108589b7dc2e9/tempest/values.yaml#L28 is not found08:46
nick_karchanging 4.0.3 to 4.0.0 gets an image but fails when trying to run the tests08:47
nick_karhave you seen that before? any suggestion on the correct tempest image?08:48
jsuchomeitxaka: are you reading this ^ ?08:48
itxakaahh nice :D09:06
itxakanmimi, just in time, we are currently fixing that stuff09:06
itxakaindeed, you are rigth, the image is wrong and the values are incorrect09:07
itxakayou would need this for the values: https://review.openstack.org/#/c/650948/09:07
itxakaand the image is being changed to our own openstackhelm image on https://review.openstack.org/#/c/652698/09:08
itxakaunfortunately, while I can test those locally and they seem to work, the current status is that we dont test the tempest image so we dont know if its working09:09
itxakaIm adding a tempest job here: https://review.openstack.org/#/c/652705/ but the tempest job is failnig and we dont recover kubernetes job logs so no idea what is failing09:09
*** Talion has quit IRC09:10
nick_karitxaka: great, thanks09:10
itxakaso if you want to apply both patches and let me know the output of the tempest job it would be great to troubleshoot :D09:10
nick_karitxaka: we'll try the new image and let you know09:10
itxaka:+109:10
itxakanick_kar, nmimi you migth need also the following which changes the script that is run to execute tempest so it picks up the config file properly https://review.openstack.org/#/c/652700/09:11
nick_karitxaka: even better :) thanks09:13
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job  https://review.openstack.org/65270509:37
pgaxattefdegir: ok finally it works but you need to specify the change id in depends-on, not the URL of the change ;)09:43
pgaxatteangiewang: the tests are now OK on https://review.openstack.org/#/c/651585/809:44
fdegirpgaxatte: i am not sure09:47
fdegirpgaxatte: in my case, it worked the way i described09:47
fdegirpgaxatte: which is what is written in the docs as well: https://zuul-ci.org/docs/zuul/user/gating.html#cross-project-dependencies09:47
fdegirTo use them, include Depends-On: <change-url> in the footer of a commit message or pull request.09:48
pgaxattefdegir: i found another discussion in a mailing list using the change-id, maybe the documentation is not up-to-date?09:52
pgaxatteor zuul is not up-to-date itself09:54
pgaxatteanyway it works: http://logs.openstack.org/85/651585/8/check/openstack-helm-infra-openstack-support/e9d6579/job-output.txt.gz#_2019-04-16_08_47_53_58816109:54
*** Liang__ has quit IRC10:05
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: Add logging.conf to tempest  https://review.openstack.org/65296310:12
openstackgerritGerry Kopec proposed openstack/openstack-helm master: Nova console/ip address search optionality  https://review.openstack.org/64830711:05
evrardjpI am off this week, so I wont' attend the meeting, FYI.11:28
*** nguyenhai has joined #openstack-helm11:47
*** nguyenhai has quit IRC11:52
*** happyhemant has joined #openstack-helm12:11
*** lemko has joined #openstack-helm12:23
*** Liang__ has joined #openstack-helm12:45
*** bh526r has joined #openstack-helm12:53
*** aaronsheffield has joined #openstack-helm12:57
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch.  https://review.openstack.org/64999013:17
*** howell has joined #openstack-helm13:23
*** sgrasley has joined #openstack-helm13:35
*** alisanhaji has quit IRC13:36
*** sthussey has joined #openstack-helm13:44
*** cfriesen has joined #openstack-helm13:47
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context  https://review.openstack.org/64755513:54
angiewangpgaxatte, Great:)13:57
*** tao_liu has quit IRC13:59
openstackgerritDrew Walters proposed openstack/openstack-helm-images master: mini-mirror: Ignore src config for publishing  https://review.openstack.org/64396714:13
*** Liang__ has quit IRC14:17
*** alisanhaji has joined #openstack-helm14:22
*** dpawlik has quit IRC14:28
srwilkersthanks for the heads up evrardjp :)14:28
*** sgrasley has quit IRC14:29
*** sgrasley has joined #openstack-helm14:36
openstackgerritChris Wedgwood proposed openstack/openstack-helm-infra master: ceph-client: Add pod/container security context  https://review.openstack.org/64855214:53
portdirectsrwilkers, lrensing, v1k0d3n, lamt, portdirect, renmak, jayahn, korzen, alanmeadows, mattmceuen, raymaika, evrardjp, MegHeisler, howell, georgk, arunkant, roman_g: it's meeting time in 5 mins folks14:55
portdirectover in #openstack-meeting-414:55
*** pgaxatte has quit IRC14:56
portdirectthe agenda is here: https://etherpad.openstack.org/p/openstack-helm-meeting-2019-04-1614:57
*** Liang__ has joined #openstack-helm15:06
*** Liang__ is now known as LiangFang15:08
*** sreejithp has joined #openstack-helm15:09
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres  https://review.openstack.org/64438815:11
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres  https://review.openstack.org/64438815:12
*** Talion has joined #openstack-helm15:15
*** Darcidride has quit IRC15:16
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers  https://review.openstack.org/60322915:20
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: Tiller: Add pod/container security context  https://review.openstack.org/64588415:22
*** sgrasley has quit IRC15:36
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Add wait.resource for LMA services to armada manifest  https://review.openstack.org/64914115:37
*** sgrasley has joined #openstack-helm15:45
*** sgrasley has quit IRC16:00
srwilkershey itxaka16:01
jsuchomeany chance we could have https://review.openstack.org/#/c/651491/ reviewed? portdirect srwilkers ?16:01
srwilkersone thing i wanted to bring up during round table also16:01
itxakayup yup16:02
srwilkerswas that we may want to explore using a multinode job for tempest runs -- i'm not sure a single node would suffice for running kubernetes + osh + tempest16:02
itxakaumm good point16:02
itxakaIm pretty ignorant on what is behind the scenes on the zuul jobs so never thought about it16:03
srwilkersive been chewing on the idea of adding documentation for our current jobs/layout -- i think it might add value, given we're seeing more attention being paid to them16:03
itxakaI guess that once the tempest job *works* we can see if its just killing the node16:03
*** sgrasley has joined #openstack-helm16:03
srwilkersyeah -- that's a good point too16:04
itxakaFYI nova+neutron are disabled on tempest by default due some bugs so there is only about 80 tests I think16:04
srwilkersokay, might not be so bad then16:04
itxakayeah I guess that if we solve those issue and want to enable both then we will need to move to multinode asap16:04
srwilkersagreed16:04
itxakaunless we want every zuul run to be 90 minutes :P16:04
itxakasrwilkers, RE: docs: would be greatly appreciated!16:05
openstackgerritGage Hugo proposed openstack/openstack-helm master: Add wait.resources.type to armada manifest  https://review.openstack.org/64850316:05
srwilkersitxaka: we used to have mostly multinode jobs, and it wasn't uncommon for the total run time for all checks to be >2hrs16:07
srwilkerswas a sad time :(16:07
itxakaaint nobody got time for that16:07
srwilkers:D16:07
*** neolee has joined #openstack-helm16:09
itxakagotta drop, talk to y'all tomorrow, enjoy your day!16:09
srwilkerslater itxaka :)16:09
neoleeHello all, few hours ago, I spun up an ec2 instance on aws to try out the openstack helm, but I got stuck at installing MariaDB16:09
neoleethe exporter-create-sql-server unable to connect to the mariadb service16:10
neoleeany clue?16:10
neoleesorry i meant exporter-create-sql-user*16:10
openstackgerritLiang Fang proposed openstack/openstack-helm master: Add internal tenant id in conf  https://review.openstack.org/64749316:10
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause  https://review.openstack.org/65303816:13
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause  https://review.openstack.org/65303816:13
*** flaviosr_ has quit IRC16:21
*** bh526r has quit IRC16:22
*** sgrasley_ has joined #openstack-helm16:22
*** unicell has joined #openstack-helm16:24
*** sgrasley has quit IRC16:26
*** unicell has quit IRC16:28
*** LiangFang has quit IRC16:30
*** neolee has quit IRC16:37
*** sgrasley__ has joined #openstack-helm16:50
openstackgerritDoug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres  https://review.openstack.org/64438816:51
*** sgrasley_ has quit IRC16:53
*** itlinux has joined #openstack-helm16:59
*** aojea has quit IRC17:20
*** sgrasley_ has joined #openstack-helm17:27
*** sgrasley_ has quit IRC17:30
*** sgrasley__ has quit IRC17:31
openstackgerritMerged openstack/openstack-helm-infra master: Fix YAML indentation  https://review.openstack.org/65055617:34
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context  https://review.openstack.org/64755517:57
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: ceph-client: Add pod/container security context  https://review.openstack.org/64855217:58
*** lemko has quit IRC18:13
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause  https://review.openstack.org/65303818:15
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause  https://review.openstack.org/65303818:16
openstackgerritRandeep Jalli proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-node-exporter.  https://review.openstack.org/65038618:22
*** unicell has joined #openstack-helm18:25
openstackgerritMerged openstack/openstack-helm-infra master: Tiller: Add pod/container security context  https://review.openstack.org/64588418:27
*** happyhemant has quit IRC18:31
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver  https://review.openstack.org/65308018:37
*** irclogbot_1 has quit IRC18:39
*** irclogbot_3 has joined #openstack-helm18:40
openstackgerritRandeep Jalli proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-alertmanager Add in prometheus-alertmanager gate script as a script  https://review.openstack.org/65037318:43
openstackgerritRandeep Jalli proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-openstack-exporter.  https://review.openstack.org/65038818:51
*** angiewang has quit IRC18:51
openstackgerritRandeep Jalli proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch.  https://review.openstack.org/64999019:02
openstackgerritRandeep Jalli proposed openstack/openstack-helm-infra master: Add default AppArmor profile to Fluentbit and Elasticsearch Make MAC profile annotation optional because nothing is currently defined in values.yaml Co-authored-by: ld366r@att.com Co-authored-by: rj2083@att.com Change-Id: If60f3b9c9015720825bd2ad7e5cbd8a6  https://review.openstack.org/64788119:02
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph.  https://review.openstack.org/65273119:20
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph.  https://review.openstack.org/65273119:20
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph.  https://review.openstack.org/65273119:24
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-node-exporter.  https://review.openstack.org/65038619:28
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-alertmanager Add in prometheus-alertmanager gate script as a script  https://review.openstack.org/65037319:28
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-openstack-exporter.  https://review.openstack.org/65038819:28
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch.  https://review.openstack.org/64999019:28
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: Add default AppArmor profile to Fluentbit and Elasticsearch Make MAC profile annotation optional because nothing is currently defined in values.yaml Co-authored-by: ld366r@att.com Co-authored-by: rj2083@att.com Change-Id: If60f3b9c9015720825bd2ad7e5cbd8a6  https://review.openstack.org/64788119:28
*** itlinux has quit IRC19:30
*** itlinux has joined #openstack-helm19:34
*** aric49 has joined #openstack-helm19:36
openstackgerritMerged openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context  https://review.openstack.org/64755519:37
openstackgerritMerged openstack/openstack-helm-images master: Adding support for DPDK to openvswitch image  https://review.openstack.org/65015219:43
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch, Fluent-logging, Kibana Ingress Policy  https://review.openstack.org/63407819:44
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver  https://review.openstack.org/65308019:52
*** diablo_rojo has joined #openstack-helm19:55
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers  https://review.openstack.org/60322919:58
*** aric49 has quit IRC20:02
itxakaunfortunately this was left over from the meeting due to time, but we are really interested into moving this forward as we feel its important to confirm that our changes to charts do not break on either OS (ubuntu+suse) so reviews are very very welcome and paid in beer at the next summit https://review.openstack.org/#/c/651491/20:07
openstackgerritColbert Philippe proposed openstack/openstack-helm-infra master: This is the first change to explore the system  https://review.openstack.org/65314320:25
*** alisanhaji has quit IRC20:25
openstackgerritColbert Philippe proposed openstack/openstack-helm-infra master: This is the first change to explore the system  https://review.openstack.org/65314320:31
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver  https://review.openstack.org/65308020:32
*** jsuchome has quit IRC20:52
*** howell has quit IRC21:21
openstackgerritMerged openstack/openstack-helm-infra master: Mariadb: Disable monitoring by default  https://review.openstack.org/65135021:26
*** diablo_rojo has left #openstack-helm21:39
srwilkersHey itxaka - when you’re back around, I rebased this change on your change that adds a tempest job. I also made it depend on my change that adds the gathering of previous logs for failed pods: https://review.openstack.org/#/c/653038/21:41
srwilkersJust verified it’s got logs from the tempest-run-tests job21:41
srwilkersHopefully that’ll at least help troubleshoot the tempest failures21:42
*** itlinux has quit IRC21:53
*** itlinux has joined #openstack-helm22:07
*** sreejithp has quit IRC22:10
itxakanice, thanks srwilkers, really really useful!22:28
itxakainteresting so it actually runs but just fails 5 tests :)22:29
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job  https://review.openstack.org/65270522:34
itxakaits soo good I already found the issue :D22:35
srwilkersAwesome :D22:36
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job  https://review.openstack.org/65270522:46
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph.  https://review.openstack.org/65273123:28
*** itlinux has quit IRC23:35
*** Liang__ has joined #openstack-helm23:53

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