*** cheng1 has quit IRC | 00:24 | |
*** cheng1 has joined #openstack-helm | 00:26 | |
*** roman_g has quit IRC | 00:39 | |
*** Liang__ has joined #openstack-helm | 00:48 | |
*** unicell has joined #openstack-helm | 01:29 | |
*** unicell has quit IRC | 01:33 | |
*** unicell has joined #openstack-helm | 02:06 | |
openstackgerrit | Tin Lam proposed openstack/openstack-helm-infra master: Revert "readOnlyRootFilesystem: true for openvswitch chart" https://review.openstack.org/652792 | 02:28 |
---|---|---|
*** cfriesen has quit IRC | 02:52 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: Revert "readOnlyRootFilesystem: true for openvswitch chart" https://review.openstack.org/652792 | 04:43 |
openstackgerrit | Taewoo Lee proposed openstack/openstack-helm master: Add ipxe file path variable and installation codes for CentOS https://review.openstack.org/652822 | 05:04 |
*** jsuchome has joined #openstack-helm | 06:03 | |
*** hogepodge has quit IRC | 06:32 | |
*** aojea has joined #openstack-helm | 06:41 | |
*** pgaxatte has joined #openstack-helm | 06:52 | |
*** hogepodge has joined #openstack-helm | 06:56 | |
*** Talion has joined #openstack-helm | 07:01 | |
pgaxatte | hello | 07:02 |
pgaxatte | just read the irc logs, thanks fdegir for the tip on Depends-On ;) | 07:02 |
*** pma78 has joined #openstack-helm | 07:04 | |
openstackgerrit | Pierre Gaxatte proposed openstack/openstack-helm-infra master: Add registry authentication support https://review.openstack.org/651585 | 07:05 |
fdegir | pgaxatte: sure | 07:13 |
fdegir | pgaxette: one thing i’m not so sure is whether the location of Depends-On matters or not | 07:13 |
fdegir | pgaxette: i normally put it before change-id | 07:14 |
pgaxatte | fdegir: i put the depends-on in the commit message of https://review.openstack.org/#/c/651585 | 07:14 |
pgaxatte | in the hope that zuul will understand that it depends on another repo | 07:14 |
pgaxatte | because 651585 is a change on openstack-helm-infra and 651587 on openstack-helm | 07:14 |
fdegir | pgaxatte: just checked your change on zuul and it doesn’t look like it worked | 07:15 |
fdegir | yes, zuul can do work across multiple repos | 07:15 |
fdegir | i have a similar review and it got tested it correctly | 07:16 |
fdegir | https://review.openstack.org/652063 | 07:16 |
fdegir | in 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 want | 07:17 |
pgaxatte | ok we'll see if this works :) | 07:17 |
*** alisanhaji has joined #openstack-helm | 07:33 | |
*** Darcidride has joined #openstack-helm | 07:47 | |
openstackgerrit | Itxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job https://review.openstack.org/652705 | 07:49 |
*** roman_g has joined #openstack-helm | 07:58 | |
nmimi | Hi all! just a dumb question if someone knows: | 08:05 |
nmimi | I am working in task : XCI Introduction of Openstack-helm as installer(multinode) and currently i am trying to deploy tempest chart. | 08:06 |
nmimi | In tempest/values.yaml there is a section(line 125) tempest: auth: # admin_username value set by configmap-etc | 08:06 |
nmimi | admin_username: None | 08:06 |
nmimi | and 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 |
nmimi | My 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 |
nmimi | If i change the tempest/values.yaml from admin_username: None -> admin_username: '' , everything works fine. | 08:06 |
nmimi | Have you seen this issue before? Is there another way to get the correct values? | 08:06 |
*** gkadam has joined #openstack-helm | 08:09 | |
*** gkadam has quit IRC | 08:10 | |
pgaxatte | fdegir: 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_718353 | 08:27 |
pgaxatte | it is taking the master branch | 08:27 |
openstackgerrit | Pierre Gaxatte proposed openstack/openstack-helm-infra master: Add registry authentication support https://review.openstack.org/651585 | 08:37 |
nick_kar | Hi all | 08:44 |
nick_kar | working 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 found | 08:46 |
nick_kar | changing 4.0.3 to 4.0.0 gets an image but fails when trying to run the tests | 08:47 |
nick_kar | have you seen that before? any suggestion on the correct tempest image? | 08:48 |
jsuchome | itxaka: are you reading this ^ ? | 08:48 |
itxaka | ahh nice :D | 09:06 |
itxaka | nmimi, just in time, we are currently fixing that stuff | 09:06 |
itxaka | indeed, you are rigth, the image is wrong and the values are incorrect | 09:07 |
itxaka | you would need this for the values: https://review.openstack.org/#/c/650948/ | 09:07 |
itxaka | and the image is being changed to our own openstackhelm image on https://review.openstack.org/#/c/652698/ | 09:08 |
itxaka | unfortunately, 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 working | 09:09 |
itxaka | Im 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 failing | 09:09 |
*** Talion has quit IRC | 09:10 | |
nick_kar | itxaka: great, thanks | 09:10 |
itxaka | so if you want to apply both patches and let me know the output of the tempest job it would be great to troubleshoot :D | 09:10 |
nick_kar | itxaka: we'll try the new image and let you know | 09:10 |
itxaka | :+1 | 09:10 |
itxaka | nick_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_kar | itxaka: even better :) thanks | 09:13 |
openstackgerrit | Itxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job https://review.openstack.org/652705 | 09:37 |
pgaxatte | fdegir: ok finally it works but you need to specify the change id in depends-on, not the URL of the change ;) | 09:43 |
pgaxatte | angiewang: the tests are now OK on https://review.openstack.org/#/c/651585/8 | 09:44 |
fdegir | pgaxatte: i am not sure | 09:47 |
fdegir | pgaxatte: in my case, it worked the way i described | 09:47 |
fdegir | pgaxatte: which is what is written in the docs as well: https://zuul-ci.org/docs/zuul/user/gating.html#cross-project-dependencies | 09:47 |
fdegir | To use them, include Depends-On: <change-url> in the footer of a commit message or pull request. | 09:48 |
pgaxatte | fdegir: i found another discussion in a mailing list using the change-id, maybe the documentation is not up-to-date? | 09:52 |
pgaxatte | or zuul is not up-to-date itself | 09:54 |
pgaxatte | anyway 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_588161 | 09:54 |
*** Liang__ has quit IRC | 10:05 | |
openstackgerrit | Itxaka Serrano Garcia proposed openstack/openstack-helm master: Add logging.conf to tempest https://review.openstack.org/652963 | 10:12 |
openstackgerrit | Gerry Kopec proposed openstack/openstack-helm master: Nova console/ip address search optionality https://review.openstack.org/648307 | 11:05 |
evrardjp | I am off this week, so I wont' attend the meeting, FYI. | 11:28 |
*** nguyenhai has joined #openstack-helm | 11:47 | |
*** nguyenhai has quit IRC | 11:52 | |
*** happyhemant has joined #openstack-helm | 12:11 | |
*** lemko has joined #openstack-helm | 12:23 | |
*** Liang__ has joined #openstack-helm | 12:45 | |
*** bh526r has joined #openstack-helm | 12:53 | |
*** aaronsheffield has joined #openstack-helm | 12:57 | |
openstackgerrit | diwakar thyagaraj proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch. https://review.openstack.org/649990 | 13:17 |
*** howell has joined #openstack-helm | 13:23 | |
*** sgrasley has joined #openstack-helm | 13:35 | |
*** alisanhaji has quit IRC | 13:36 | |
*** sthussey has joined #openstack-helm | 13:44 | |
*** cfriesen has joined #openstack-helm | 13:47 | |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context https://review.openstack.org/647555 | 13:54 |
angiewang | pgaxatte, Great:) | 13:57 |
*** tao_liu has quit IRC | 13:59 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-images master: mini-mirror: Ignore src config for publishing https://review.openstack.org/643967 | 14:13 |
*** Liang__ has quit IRC | 14:17 | |
*** alisanhaji has joined #openstack-helm | 14:22 | |
*** dpawlik has quit IRC | 14:28 | |
srwilkers | thanks for the heads up evrardjp :) | 14:28 |
*** sgrasley has quit IRC | 14:29 | |
*** sgrasley has joined #openstack-helm | 14:36 | |
openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: ceph-client: Add pod/container security context https://review.openstack.org/648552 | 14:53 |
portdirect | srwilkers, lrensing, v1k0d3n, lamt, portdirect, renmak, jayahn, korzen, alanmeadows, mattmceuen, raymaika, evrardjp, MegHeisler, howell, georgk, arunkant, roman_g: it's meeting time in 5 mins folks | 14:55 |
portdirect | over in #openstack-meeting-4 | 14:55 |
*** pgaxatte has quit IRC | 14:56 | |
portdirect | the agenda is here: https://etherpad.openstack.org/p/openstack-helm-meeting-2019-04-16 | 14:57 |
*** Liang__ has joined #openstack-helm | 15:06 | |
*** Liang__ is now known as LiangFang | 15:08 | |
*** sreejithp has joined #openstack-helm | 15:09 | |
openstackgerrit | Doug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres https://review.openstack.org/644388 | 15:11 |
openstackgerrit | Doug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres https://review.openstack.org/644388 | 15:12 |
*** Talion has joined #openstack-helm | 15:15 | |
*** Darcidride has quit IRC | 15:16 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers https://review.openstack.org/603229 | 15:20 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: Tiller: Add pod/container security context https://review.openstack.org/645884 | 15:22 |
*** sgrasley has quit IRC | 15:36 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Add wait.resource for LMA services to armada manifest https://review.openstack.org/649141 | 15:37 |
*** sgrasley has joined #openstack-helm | 15:45 | |
*** sgrasley has quit IRC | 16:00 | |
srwilkers | hey itxaka | 16:01 |
jsuchome | any chance we could have https://review.openstack.org/#/c/651491/ reviewed? portdirect srwilkers ? | 16:01 |
srwilkers | one thing i wanted to bring up during round table also | 16:01 |
itxaka | yup yup | 16:02 |
srwilkers | was 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 + tempest | 16:02 |
itxaka | umm good point | 16:02 |
itxaka | Im pretty ignorant on what is behind the scenes on the zuul jobs so never thought about it | 16:03 |
srwilkers | ive 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 them | 16:03 |
itxaka | I guess that once the tempest job *works* we can see if its just killing the node | 16:03 |
*** sgrasley has joined #openstack-helm | 16:03 | |
srwilkers | yeah -- that's a good point too | 16:04 |
itxaka | FYI nova+neutron are disabled on tempest by default due some bugs so there is only about 80 tests I think | 16:04 |
srwilkers | okay, might not be so bad then | 16:04 |
itxaka | yeah I guess that if we solve those issue and want to enable both then we will need to move to multinode asap | 16:04 |
srwilkers | agreed | 16:04 |
itxaka | unless we want every zuul run to be 90 minutes :P | 16:04 |
itxaka | srwilkers, RE: docs: would be greatly appreciated! | 16:05 |
openstackgerrit | Gage Hugo proposed openstack/openstack-helm master: Add wait.resources.type to armada manifest https://review.openstack.org/648503 | 16:05 |
srwilkers | itxaka: we used to have mostly multinode jobs, and it wasn't uncommon for the total run time for all checks to be >2hrs | 16:07 |
srwilkers | was a sad time :( | 16:07 |
itxaka | aint nobody got time for that | 16:07 |
srwilkers | :D | 16:07 |
*** neolee has joined #openstack-helm | 16:09 | |
itxaka | gotta drop, talk to y'all tomorrow, enjoy your day! | 16:09 |
srwilkers | later itxaka :) | 16:09 |
neolee | Hello all, few hours ago, I spun up an ec2 instance on aws to try out the openstack helm, but I got stuck at installing MariaDB | 16:09 |
neolee | the exporter-create-sql-server unable to connect to the mariadb service | 16:10 |
neolee | any clue? | 16:10 |
neolee | sorry i meant exporter-create-sql-user* | 16:10 |
openstackgerrit | Liang Fang proposed openstack/openstack-helm master: Add internal tenant id in conf https://review.openstack.org/647493 | 16:10 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause https://review.openstack.org/653038 | 16:13 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause https://review.openstack.org/653038 | 16:13 |
*** flaviosr_ has quit IRC | 16:21 | |
*** bh526r has quit IRC | 16:22 | |
*** sgrasley_ has joined #openstack-helm | 16:22 | |
*** unicell has joined #openstack-helm | 16:24 | |
*** sgrasley has quit IRC | 16:26 | |
*** unicell has quit IRC | 16:28 | |
*** LiangFang has quit IRC | 16:30 | |
*** neolee has quit IRC | 16:37 | |
*** sgrasley__ has joined #openstack-helm | 16:50 | |
openstackgerrit | Doug Aaser proposed openstack/openstack-helm-infra master: [WIP] Patroni inclusion work for HA Postgres https://review.openstack.org/644388 | 16:51 |
*** sgrasley_ has quit IRC | 16:53 | |
*** itlinux has joined #openstack-helm | 16:59 | |
*** aojea has quit IRC | 17:20 | |
*** sgrasley_ has joined #openstack-helm | 17:27 | |
*** sgrasley_ has quit IRC | 17:30 | |
*** sgrasley__ has quit IRC | 17:31 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: Fix YAML indentation https://review.openstack.org/650556 | 17:34 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context https://review.openstack.org/647555 | 17:57 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: ceph-client: Add pod/container security context https://review.openstack.org/648552 | 17:58 |
*** lemko has quit IRC | 18:13 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause https://review.openstack.org/653038 | 18:15 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm master: DNM: set tempest test backofflimit high to determine failure cause https://review.openstack.org/653038 | 18:16 |
openstackgerrit | Randeep Jalli proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-node-exporter. https://review.openstack.org/650386 | 18:22 |
*** unicell has joined #openstack-helm | 18:25 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: Tiller: Add pod/container security context https://review.openstack.org/645884 | 18:27 |
*** happyhemant has quit IRC | 18:31 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver https://review.openstack.org/653080 | 18:37 |
*** irclogbot_1 has quit IRC | 18:39 | |
*** irclogbot_3 has joined #openstack-helm | 18:40 | |
openstackgerrit | Randeep 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/650373 | 18:43 |
openstackgerrit | Randeep Jalli proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-openstack-exporter. https://review.openstack.org/650388 | 18:51 |
*** angiewang has quit IRC | 18:51 | |
openstackgerrit | Randeep Jalli proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch. https://review.openstack.org/649990 | 19:02 |
openstackgerrit | Randeep 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/647881 | 19:02 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph. https://review.openstack.org/652731 | 19:20 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph. https://review.openstack.org/652731 | 19:20 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph. https://review.openstack.org/652731 | 19:24 |
openstackgerrit | diwakar thyagaraj proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-node-exporter. https://review.openstack.org/650386 | 19:28 |
openstackgerrit | diwakar 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/650373 | 19:28 |
openstackgerrit | diwakar thyagaraj proposed openstack/openstack-helm-infra master: This commit adds docker-default apparmor profile for prometheus-openstack-exporter. https://review.openstack.org/650388 | 19:28 |
openstackgerrit | diwakar thyagaraj proposed openstack/openstack-helm-infra master: Add docker-default apparmor profile for elasticsearch. https://review.openstack.org/649990 | 19:28 |
openstackgerrit | diwakar 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/647881 | 19:28 |
*** itlinux has quit IRC | 19:30 | |
*** itlinux has joined #openstack-helm | 19:34 | |
*** aric49 has joined #openstack-helm | 19:36 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: ceph-provisioners: Add pod/container security context https://review.openstack.org/647555 | 19:37 |
openstackgerrit | Merged openstack/openstack-helm-images master: Adding support for DPDK to openvswitch image https://review.openstack.org/650152 | 19:43 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch, Fluent-logging, Kibana Ingress Policy https://review.openstack.org/634078 | 19:44 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver https://review.openstack.org/653080 | 19:52 |
*** diablo_rojo has joined #openstack-helm | 19:55 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers https://review.openstack.org/603229 | 19:58 |
*** aric49 has quit IRC | 20:02 | |
itxaka | unfortunately 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 |
openstackgerrit | Colbert Philippe proposed openstack/openstack-helm-infra master: This is the first change to explore the system https://review.openstack.org/653143 | 20:25 |
*** alisanhaji has quit IRC | 20:25 | |
openstackgerrit | Colbert Philippe proposed openstack/openstack-helm-infra master: This is the first change to explore the system https://review.openstack.org/653143 | 20:31 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Move single node jobs to use journald docker logging driver https://review.openstack.org/653080 | 20:32 |
*** jsuchome has quit IRC | 20:52 | |
*** howell has quit IRC | 21:21 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: Mariadb: Disable monitoring by default https://review.openstack.org/651350 | 21:26 |
*** diablo_rojo has left #openstack-helm | 21:39 | |
srwilkers | Hey 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 |
srwilkers | Just verified it’s got logs from the tempest-run-tests job | 21:41 |
srwilkers | Hopefully that’ll at least help troubleshoot the tempest failures | 21:42 |
*** itlinux has quit IRC | 21:53 | |
*** itlinux has joined #openstack-helm | 22:07 | |
*** sreejithp has quit IRC | 22:10 | |
itxaka | nice, thanks srwilkers, really really useful! | 22:28 |
itxaka | interesting so it actually runs but just fails 5 tests :) | 22:29 |
openstackgerrit | Itxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job https://review.openstack.org/652705 | 22:34 |
itxaka | its soo good I already found the issue :D | 22:35 |
srwilkers | Awesome :D | 22:36 |
openstackgerrit | Itxaka Serrano Garcia proposed openstack/openstack-helm master: add tempest zuul job https://review.openstack.org/652705 | 22:46 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: Update image links for Ceph. https://review.openstack.org/652731 | 23:28 |
*** itlinux has quit IRC | 23:35 | |
*** Liang__ has joined #openstack-helm | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!