*** mfuller has quit IRC | 00:17 | |
*** ianychoi has quit IRC | 00:18 | |
*** ianychoi has joined #openstack-helm | 00:20 | |
*** AlexNoskov has quit IRC | 00:26 | |
*** Liang__ has joined #openstack-helm | 00:59 | |
openstackgerrit | zhipeng liu proposed openstack/openstack-helm master: Add placement chart https://review.opendev.org/662229 | 01:33 |
---|---|---|
openstackgerrit | zhipeng liu proposed openstack/openstack-helm master: Add placement chart https://review.opendev.org/662229 | 01:33 |
openstackgerrit | zhipeng liu proposed openstack/openstack-helm master: Add placement chart https://review.opendev.org/662229 | 03:08 |
*** gkadam has joined #openstack-helm | 03:50 | |
*** gkadam has quit IRC | 03:50 | |
*** vsaienk0 has joined #openstack-helm | 04:16 | |
*** vsaienk0 has quit IRC | 04:26 | |
*** michael-beaver has quit IRC | 04:31 | |
*** jaypipes has joined #openstack-helm | 04:49 | |
*** vsaienk0 has joined #openstack-helm | 04:49 | |
*** vsaienk0 has quit IRC | 05:05 | |
*** mfuller has joined #openstack-helm | 05:14 | |
*** vsaienk0 has joined #openstack-helm | 05:30 | |
*** vsaienk0 has quit IRC | 05:41 | |
*** vsaienk0 has joined #openstack-helm | 05:46 | |
*** vsaienk0 has quit IRC | 05:56 | |
*** pgaxatte has joined #openstack-helm | 06:01 | |
*** pgaxatte has quit IRC | 06:05 | |
*** pgaxatte has joined #openstack-helm | 06:05 | |
openstackgerrit | zhipeng liu proposed openstack/openstack-helm master: Add placement chart https://review.opendev.org/662229 | 06:12 |
*** jsuchome has joined #openstack-helm | 06:16 | |
*** vsaienk0 has joined #openstack-helm | 06:16 | |
*** mfuller has quit IRC | 06:29 | |
*** vsaienk0 has quit IRC | 06:33 | |
*** pgaxatte has quit IRC | 06:39 | |
*** witek has joined #openstack-helm | 06:40 | |
*** pgaxatte has joined #openstack-helm | 06:41 | |
*** vsaienk0 has joined #openstack-helm | 06:50 | |
*** dpawlik has joined #openstack-helm | 07:07 | |
openstackgerrit | zhipeng liu proposed openstack/openstack-helm master: Add placement chart https://review.opendev.org/662229 | 07:34 |
*** mfuller has joined #openstack-helm | 07:37 | |
*** vsaienk0 has quit IRC | 07:44 | |
*** vsaienk0 has joined #openstack-helm | 07:46 | |
*** mfuller has quit IRC | 07:56 | |
*** vsaienk0 has quit IRC | 07:58 | |
*** belmoreira has joined #openstack-helm | 07:59 | |
openstackgerrit | Manuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk https://review.opendev.org/671462 | 08:00 |
openstackgerrit | Manuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk https://review.opendev.org/671462 | 08:00 |
openstackgerrit | Oleh Hryhorov proposed openstack/openstack-helm master: Remove dependency for db_sync on keystone-rabbit-init https://review.opendev.org/672418 | 08:11 |
ohryhorov | folks, could you please review https://review.opendev.org/#/c/670493/ and https://review.opendev.org/#/c/671727/ ? | 08:16 |
*** vsaienk0 has joined #openstack-helm | 08:29 | |
*** belmoreira has quit IRC | 08:34 | |
*** vsaienk0 has quit IRC | 08:39 | |
*** vsaienk0 has joined #openstack-helm | 08:50 | |
*** altlogbot_2 has quit IRC | 08:51 | |
*** irclogbot_1 has quit IRC | 08:51 | |
*** altlogbot_2 has joined #openstack-helm | 08:52 | |
*** irclogbot_1 has joined #openstack-helm | 08:52 | |
*** belmoreira has joined #openstack-helm | 09:06 | |
*** vsaienk0 has quit IRC | 09:11 | |
*** Liang__ has quit IRC | 09:25 | |
*** rezroo has quit IRC | 10:02 | |
*** zhipeng[m] has quit IRC | 10:09 | |
*** obravo has joined #openstack-helm | 10:13 | |
*** pgaxatte has quit IRC | 10:24 | |
*** vsaienk0 has joined #openstack-helm | 10:25 | |
*** dpawlik has quit IRC | 10:39 | |
*** zhipeng[m] has joined #openstack-helm | 10:46 | |
cheng1 | Hello 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-helm | 11:08 | |
openstackgerrit | Manuel Buil proposed openstack/openstack-helm-infra master: Provide option to switch between dpdk and non-dpdk https://review.opendev.org/671462 | 11:16 |
mbuil | Hi 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 IRC | 11:33 | |
*** irclogbot_1 has joined #openstack-helm | 11:34 | |
*** belmoreira has quit IRC | 11:34 | |
*** vsaienk0 has quit IRC | 11:41 | |
*** belmoreira has joined #openstack-helm | 11:46 | |
*** vsaienk0 has joined #openstack-helm | 11:55 | |
*** pgaxatte has joined #openstack-helm | 12:03 | |
*** vsaienk0 has quit IRC | 12:10 | |
openstackgerrit | Pete Birley proposed openstack/openstack-helm-infra master: Restore overrides functionality after regression https://review.opendev.org/671875 | 12:18 |
*** roman_g has joined #openstack-helm | 12:20 | |
*** vsaienk0 has joined #openstack-helm | 12:28 | |
*** obravo has quit IRC | 12:30 | |
*** obravo has joined #openstack-helm | 12:32 | |
*** belmoreira has quit IRC | 12:54 | |
*** aaronsheffield has joined #openstack-helm | 13:01 | |
*** vsaienk0 has quit IRC | 13:13 | |
*** vsaienk0 has joined #openstack-helm | 13:15 | |
*** vsaienk0 has quit IRC | 13:28 | |
*** vsaienk0 has joined #openstack-helm | 13:36 | |
*** vsaienk0 has quit IRC | 13:48 | |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: [WIP] Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 13:49 |
*** belmoreira has joined #openstack-helm | 14:00 | |
*** rezroo has joined #openstack-helm | 14:00 | |
*** vsaienk0 has joined #openstack-helm | 14:01 | |
*** HagunKim has quit IRC | 14:07 | |
*** dpawlik has quit IRC | 14:25 | |
*** michael-beaver has joined #openstack-helm | 14:29 | |
openstackgerrit | Merged openstack/openstack-helm-infra master: Ceph-RGW: fix file permision issue https://review.opendev.org/672322 | 14:30 |
*** dpawlik has joined #openstack-helm | 14:33 | |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 14:37 |
*** belmoreira has quit IRC | 14:42 | |
*** belmoreira has joined #openstack-helm | 14:47 | |
*** dpawlik has quit IRC | 14:48 | |
*** belmoreira has quit IRC | 14:50 | |
evrardjp | o/ | 14:51 |
lamt | o/ | 14:53 |
*** belmoreira has joined #openstack-helm | 14:53 | |
*** belmoreira has quit IRC | 14:57 | |
*** belmoreira has joined #openstack-helm | 15:01 | |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 15:04 |
openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-OSD: Move to 'OnDelete' upgrade strategy for ceph-osd daemonsets https://review.opendev.org/664642 | 15:05 |
*** pgaxatte has quit IRC | 15:07 | |
srwilkers | hello | 15:11 |
openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: Ceph-OSD: Move to 'OnDelete' upgrade strategy for ceph-osd daemonsets https://review.opendev.org/664642 | 15:17 |
*** belmoreira has quit IRC | 15:25 | |
*** vsaienk0 has quit IRC | 15:30 | |
*** Liang__ has joined #openstack-helm | 15:54 | |
*** vsaienk0 has joined #openstack-helm | 15:57 | |
*** Liang__ has left #openstack-helm | 15:58 | |
*** vsaienk0 has quit IRC | 16:12 | |
openstackgerrit | Merged openstack/openstack-helm master: Fix typo https://review.opendev.org/671466 | 16:16 |
openstackgerrit | Merged openstack/openstack-helm-infra master: Restore overrides functionality after regression https://review.opendev.org/671875 | 16:25 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: [WIP] DO NOT MERGE https://review.opendev.org/672561 | 16:30 |
*** obravo has quit IRC | 16:34 | |
*** AlexNoskov has joined #openstack-helm | 16:37 | |
*** witek has quit IRC | 16:44 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Validate Elasticsearch snapshot repositories https://review.opendev.org/672563 | 16:46 |
*** witek has joined #openstack-helm | 16:47 | |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 16:51 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 17:01 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 17:01 |
*** witek has quit IRC | 17:37 | |
*** KeithMnemonic has joined #openstack-helm | 17:54 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Validate Elasticsearch snapshot repositories https://review.opendev.org/672563 | 18:00 |
*** obravo has joined #openstack-helm | 18:19 | |
*** jsuchome has quit IRC | 18:45 | |
*** rihabb2 has quit IRC | 18:45 | |
*** georgk has quit IRC | 18:45 | |
*** rihabb2 has joined #openstack-helm | 18:46 | |
*** georgk has joined #openstack-helm | 18:46 | |
*** witek has joined #openstack-helm | 18:52 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories https://review.opendev.org/672578 | 18:54 |
*** witek has quit IRC | 18:56 | |
*** obravo has quit IRC | 19:06 | |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: [WIP] DO NOT MERGE https://review.opendev.org/672579 | 19:18 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 19:19 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 19:21 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: Gate: Deploy default number of rabbit replicas https://review.opendev.org/672579 | 19:21 |
evgenyl | Hi 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.tpl | 19:32 |
evgenyl | What does the team think about it? | 19:33 |
srwilkers | evgenyl: that is nearly identical to https://github.com/openstack/openstack-helm-infra/blob/master/helm-toolkit/templates/endpoints/_authenticated_transport_endpoint_uri_lookup.tpl | 19:44 |
srwilkers | did you just copy that template and modify it to work specifically with a single service? | 19:44 |
portdirect | what is the delta? | 19:44 |
portdirect | simply ; vs , ? | 19:45 |
srwilkers | https://www.diffchecker.com/xOmpMn4t | 19:45 |
evgenyl | srwilkers: 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 |
evgenyl | For oslo messaging we could just add hosts + ports separated with commas. | 19:47 |
srwilkers | i 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#L45 | 19:47 |
evgenyl | It does not return schema + vhost for every url. | 19:48 |
srwilkers | okay. 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 inspiration | 19:56 |
evgenyl | It has the schema, but you are right about vhosts, let me update that. | 19:59 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 20:02 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 20:02 |
evgenyl | Updated the example with vhost. | 20:03 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 20:03 |
evgenyl | srwilkers: portdirect so do you think it should be a part of htk? | 20:04 |
srwilkers | evgenyl: if its specific just to airflow, i dont think it should be | 20:15 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories https://review.opendev.org/672578 | 20:29 |
openstackgerrit | Pete Birley proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 20:35 |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: Elasticsearch: Manually verify snapshot repositories https://review.opendev.org/672578 | 20:37 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 20:49 |
portdirect | evgenyl: i tend to agree with srwilkers here | 21:16 |
portdirect | if its got use in two charts - then htk is the place | 21:16 |
portdirect | a singleton - probably should like with its consumer | 21:16 |
portdirect | */like/live | 21:16 |
evgenyl | portdirect: srwilkers thank you guys for the feedback, I will leave it as is. | 21:19 |
openstackgerrit | Merged openstack/openstack-helm master: Gate: Deploy default number of rabbit replicas https://review.opendev.org/672579 | 21:48 |
openstackgerrit | Merged openstack/openstack-helm master: Nova: Update DB sync job to update transport url https://review.opendev.org/672527 | 22:29 |
*** michael-beaver has quit IRC | 23:24 | |
*** aaronsheffield has quit IRC | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!