Wednesday, 2019-07-24

*** mfuller has quit IRC00:17
*** ianychoi has quit IRC00:18
*** ianychoi has joined #openstack-helm00:20
*** AlexNoskov has quit IRC00:26
*** Liang__ has joined #openstack-helm00:59
openstackgerritzhipeng liu proposed openstack/openstack-helm master: Add placement chart  https://review.opendev.org/66222901:33
openstackgerritzhipeng liu proposed openstack/openstack-helm master: Add placement chart  https://review.opendev.org/66222901:33
openstackgerritzhipeng liu proposed openstack/openstack-helm master: Add placement chart  https://review.opendev.org/66222903:08
*** gkadam has joined #openstack-helm03:50
*** gkadam has quit IRC03:50
*** vsaienk0 has joined #openstack-helm04:16
*** vsaienk0 has quit IRC04:26
*** michael-beaver has quit IRC04:31
*** jaypipes has joined #openstack-helm04:49
*** vsaienk0 has joined #openstack-helm04:49
*** vsaienk0 has quit IRC05:05
*** mfuller has joined #openstack-helm05:14
*** vsaienk0 has joined #openstack-helm05:30
*** vsaienk0 has quit IRC05:41
*** vsaienk0 has joined #openstack-helm05:46
*** vsaienk0 has quit IRC05:56
*** pgaxatte has joined #openstack-helm06:01
*** pgaxatte has quit IRC06:05
*** pgaxatte has joined #openstack-helm06:05
openstackgerritzhipeng liu proposed openstack/openstack-helm master: Add placement chart  https://review.opendev.org/66222906:12
*** jsuchome has joined #openstack-helm06:16
*** vsaienk0 has joined #openstack-helm06:16
*** mfuller has quit IRC06:29
*** vsaienk0 has quit IRC06:33
*** pgaxatte has quit IRC06:39
*** witek has joined #openstack-helm06:40
*** pgaxatte has joined #openstack-helm06:41
*** vsaienk0 has joined #openstack-helm06:50
*** dpawlik has joined #openstack-helm07:07
openstackgerritzhipeng liu proposed openstack/openstack-helm master: Add placement chart  https://review.opendev.org/66222907:34
*** mfuller has joined #openstack-helm07:37
*** vsaienk0 has quit IRC07:44
*** vsaienk0 has joined #openstack-helm07:46
*** mfuller has quit IRC07:56
*** vsaienk0 has quit IRC07:58
*** belmoreira has joined #openstack-helm07:59
openstackgerritManuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk  https://review.opendev.org/67146208:00
openstackgerritManuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk  https://review.opendev.org/67146208:00
openstackgerritOleh Hryhorov proposed openstack/openstack-helm master: Remove dependency for db_sync on keystone-rabbit-init  https://review.opendev.org/67241808:11
ohryhorovfolks, could you please review https://review.opendev.org/#/c/670493/ and https://review.opendev.org/#/c/671727/ ?08:16
*** vsaienk0 has joined #openstack-helm08:29
*** belmoreira has quit IRC08:34
*** vsaienk0 has quit IRC08:39
*** vsaienk0 has joined #openstack-helm08:50
*** altlogbot_2 has quit IRC08:51
*** irclogbot_1 has quit IRC08:51
*** altlogbot_2 has joined #openstack-helm08:52
*** irclogbot_1 has joined #openstack-helm08:52
*** belmoreira has joined #openstack-helm09:06
*** vsaienk0 has quit IRC09:11
*** Liang__ has quit IRC09:25
*** rezroo has quit IRC10:02
*** zhipeng[m] has quit IRC10:09
*** obravo has joined #openstack-helm10:13
*** pgaxatte has quit IRC10:24
*** vsaienk0 has joined #openstack-helm10:25
*** dpawlik has quit IRC10:39
*** zhipeng[m] has joined #openstack-helm10:46
cheng1Hello guys, anyone could have a review of this patch. It's a tiny patch https://review.opendev.org/#/c/671696/10:59
*** dpawlik has joined #openstack-helm11:08
openstackgerritManuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk  https://review.opendev.org/67146211:16
mbuilHi guys. I'd need reviews on https://review.opendev.org/#/c/671466/ and https://review.opendev.org/#/c/671462/11:23
*** irclogbot_1 has quit IRC11:33
*** irclogbot_1 has joined #openstack-helm11:34
*** belmoreira has quit IRC11:34
*** vsaienk0 has quit IRC11:41
*** belmoreira has joined #openstack-helm11:46
*** vsaienk0 has joined #openstack-helm11:55
*** pgaxatte has joined #openstack-helm12:03
*** vsaienk0 has quit IRC12:10
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: Restore overrides functionality after regression  https://review.opendev.org/67187512:18
*** roman_g has joined #openstack-helm12:20
*** vsaienk0 has joined #openstack-helm12:28
*** obravo has quit IRC12:30
*** obravo has joined #openstack-helm12:32
*** belmoreira has quit IRC12:54
*** aaronsheffield has joined #openstack-helm13:01
*** vsaienk0 has quit IRC13:13
*** vsaienk0 has joined #openstack-helm13:15
*** vsaienk0 has quit IRC13:28
*** vsaienk0 has joined #openstack-helm13:36
*** vsaienk0 has quit IRC13:48
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: [WIP] Nova: Update DB sync job to update transport url  https://review.opendev.org/67252713:49
*** belmoreira has joined #openstack-helm14:00
*** rezroo has joined #openstack-helm14:00
*** vsaienk0 has joined #openstack-helm14:01
*** HagunKim has quit IRC14:07
*** dpawlik has quit IRC14:25
*** michael-beaver has joined #openstack-helm14:29
openstackgerritMerged openstack/openstack-helm-infra master: Ceph-RGW: fix file permision issue  https://review.opendev.org/67232214:30
*** dpawlik has joined #openstack-helm14:33
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252714:37
*** belmoreira has quit IRC14:42
*** belmoreira has joined #openstack-helm14:47
*** dpawlik has quit IRC14:48
*** belmoreira has quit IRC14:50
evrardjpo/14:51
lamto/14:53
*** belmoreira has joined #openstack-helm14:53
*** belmoreira has quit IRC14:57
*** belmoreira has joined #openstack-helm15:01
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252715:04
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-OSD: Move to 'OnDelete' upgrade strategy for ceph-osd daemonsets  https://review.opendev.org/66464215:05
*** pgaxatte has quit IRC15:07
srwilkershello15:11
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-OSD: Move to 'OnDelete' upgrade strategy for ceph-osd daemonsets  https://review.opendev.org/66464215:17
*** belmoreira has quit IRC15:25
*** vsaienk0 has quit IRC15:30
*** Liang__ has joined #openstack-helm15:54
*** vsaienk0 has joined #openstack-helm15:57
*** Liang__ has left #openstack-helm15:58
*** vsaienk0 has quit IRC16:12
openstackgerritMerged openstack/openstack-helm master: Fix typo  https://review.opendev.org/67146616:16
openstackgerritMerged openstack/openstack-helm-infra master: Restore overrides functionality after regression  https://review.opendev.org/67187516:25
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: [WIP] DO NOT MERGE  https://review.opendev.org/67256116:30
*** obravo has quit IRC16:34
*** AlexNoskov has joined #openstack-helm16:37
*** witek has quit IRC16:44
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Validate Elasticsearch snapshot repositories  https://review.opendev.org/67256316:46
*** witek has joined #openstack-helm16:47
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252716:51
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252717:01
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252717:01
*** witek has quit IRC17:37
*** KeithMnemonic has joined #openstack-helm17:54
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Validate Elasticsearch snapshot repositories  https://review.opendev.org/67256318:00
*** obravo has joined #openstack-helm18:19
*** jsuchome has quit IRC18:45
*** rihabb2 has quit IRC18:45
*** georgk has quit IRC18:45
*** rihabb2 has joined #openstack-helm18:46
*** georgk has joined #openstack-helm18:46
*** witek has joined #openstack-helm18:52
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories  https://review.opendev.org/67257818:54
*** witek has quit IRC18:56
*** obravo has quit IRC19:06
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: [WIP] DO NOT MERGE  https://review.opendev.org/67257919:18
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252719:19
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252719:21
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: Gate: Deploy default number of rabbit replicas  https://review.opendev.org/67257919:21
evgenylHi everyone, I've added a new template to Airship component (shipyard) that is specific to Airflow/Kombu, I was suggested to add this template to helm-toolkit, however I'm not sure if htk should contain all possible url formats that are not related to OpenStack and its components https://review.opendev.org/#/c/671575/8/charts/shipyard/templates/endpoints/_authenticated_transport_endpoint_uri_lookup.tpl19:32
evgenylWhat does the team think about it?19:33
srwilkersevgenyl: that is nearly identical to https://github.com/openstack/openstack-helm-infra/blob/master/helm-toolkit/templates/endpoints/_authenticated_transport_endpoint_uri_lookup.tpl19:44
srwilkersdid you just copy that template and modify it to work specifically with a single service?19:44
portdirectwhat is the delta?19:44
portdirectsimply ; vs , ?19:45
srwilkershttps://www.diffchecker.com/xOmpMn4t19:45
evgenylsrwilkers: Yes, it is based on the patchset, the delta is semicolons, also for Kombu every element should be a complete url that contains schema + user + password + vhost.19:46
evgenylFor oslo messaging we could just add hosts + ports separated with commas.19:47
srwilkersi was under the impression the current helm-toolkit snippet returned exactly that: https://github.com/openstack/openstack-helm-infra/blob/master/helm-toolkit/templates/endpoints/_authenticated_transport_endpoint_uri_lookup.tpl#L4519:47
evgenylIt does not return schema + vhost for every url.19:48
srwilkersokay.  the example return you provide in your change doesn't make that clear, as it's still the same format returned with what that template used as inspiration19:56
evgenylIt has the schema, but you are right about vhosts, let me update that.19:59
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252720:02
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252720:02
evgenylUpdated the example with vhost.20:03
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252720:03
evgenylsrwilkers: portdirect so do you think it should be a part of htk?20:04
srwilkersevgenyl: if its specific just to airflow, i dont think it should be20:15
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories  https://review.opendev.org/67257820:29
openstackgerritPete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252720:35
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories  https://review.opendev.org/67257820:37
openstackgerritSangeet Gupta proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252720:49
portdirectevgenyl: i tend to agree with srwilkers here21:16
portdirectif its got use in two charts - then htk is the place21:16
portdirecta singleton - probably should like with its consumer21:16
portdirect*/like/live21:16
evgenylportdirect: srwilkers thank you guys for the feedback, I will leave it as is.21:19
openstackgerritMerged openstack/openstack-helm master: Gate: Deploy default number of rabbit replicas  https://review.opendev.org/67257921:48
openstackgerritMerged openstack/openstack-helm master: Nova: Update DB sync job to update transport url  https://review.opendev.org/67252722:29
*** michael-beaver has quit IRC23:24
*** aaronsheffield has quit IRC23:30

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