Friday, 2021-02-26

*** jinyuanliu has quit IRC00:04
*** jinyuanliu has joined #openstack-helm00:05
*** ianychoi__ is now known as ianychoi00:34
openstackgerritChi Lo proposed openstack/openstack-helm-infra master: Enable TLS for grafana  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77434600:39
openstackgerritMerged openstack/openstack-helm master: Update rbac api version for nova  https://review.opendev.org/c/openstack/openstack-helm/+/77322700:57
openstackgerritjinyuanliu proposed openstack/openstack-helm-infra master: Update elasticsearch chart releasenotes to latest  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77770301:04
*** jinyuanliu has quit IRC01:06
*** jinyuanliu has joined #openstack-helm01:07
openstackgerritMerged openstack/openstack-helm master: Move rabbit-init to dynamic dependency  https://review.opendev.org/c/openstack/openstack-helm/+/77453301:28
*** jhesketh_ is now known as jhesketh01:49
openstackgerritjinyuanliu proposed openstack/openstack-helm-infra master: Update ceph-client chart releasenotes to latest  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77770701:50
openstackgerritWenping Song proposed openstack/openstack-helm master: Update rbac api version for keystone  https://review.opendev.org/c/openstack/openstack-helm/+/77323502:33
openstackgerritNafiz Haider proposed openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281302:36
openstackgerritNafiz Haider proposed openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281302:39
openstackgerritNafiz Haider proposed openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281302:39
openstackgerritNafiz Haider proposed openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281302:42
openstackgerritNafiz Haider proposed openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281302:43
openstackgerritWenping Song proposed openstack/openstack-helm master: BUG for deploying multiple compute nodes  https://review.opendev.org/c/openstack/openstack-helm/+/77355802:49
*** sreejithp has quit IRC02:59
*** ricolin has joined #openstack-helm03:15
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771103:59
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:03
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:07
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:11
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:23
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:43
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771104:44
openstackgerritChi Lo proposed openstack/openstack-helm-infra master: Enable TLS for grafana  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77434605:14
openstackgerritChi Lo proposed openstack/openstack-helm-infra master: Enable TLS for grafana  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77434605:16
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Glance  https://review.opendev.org/c/openstack/openstack-helm/+/77553306:22
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Keystone  https://review.opendev.org/c/openstack/openstack-helm/+/77553606:24
*** jinyuanliu has quit IRC06:25
*** jinyuanliu has joined #openstack-helm06:25
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Nova  https://review.opendev.org/c/openstack/openstack-helm/+/77553806:27
*** songwenping__ has joined #openstack-helm07:23
*** vsaienk0 has joined #openstack-helm07:34
*** jinyuanliu has quit IRC08:07
*** jinyuanliu has joined #openstack-helm08:07
*** roman_g has joined #openstack-helm08:59
*** jinyuanliu has quit IRC09:03
*** jinyuanliu has joined #openstack-helm09:03
*** songwenping__ has quit IRC09:35
*** gagehugo has quit IRC09:37
*** gagehugo has joined #openstack-helm09:37
*** vsaienk0 has quit IRC09:53
*** vsaienk0 has joined #openstack-helm09:54
*** Anticom has joined #openstack-helm10:00
*** vsaienk0 has quit IRC10:44
*** vsaienk0 has joined #openstack-helm11:14
*** jinyuanliu has quit IRC11:17
*** vsaienk0 has quit IRC11:41
*** roman_g has quit IRC11:57
*** vsaienk0 has joined #openstack-helm12:55
*** vsaienk0 has quit IRC13:06
*** vsaienk0 has joined #openstack-helm13:26
*** jinyuanliu has joined #openstack-helm13:33
*** sreejithp has joined #openstack-helm14:06
Open10K8SHi team. Is there any chance to discuss about https://review.opendev.org/c/openstack/openstack-helm-infra/+/776466 ? Thank you14:32
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771114:45
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771114:45
openstackgerritFrank Ritchie proposed openstack/openstack-helm-infra master: [WIP] feat(tls): Enable TLS for Ceph RGW  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77745114:55
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm master: Added post-install and post-upgrade annotations on job templates  https://review.opendev.org/c/openstack/openstack-helm/+/77665715:00
jinyuanliuupdate reno15:05
openstackgerritFrank Ritchie proposed openstack/openstack-helm-infra master: [WIP] feat(tls): Enable TLS for Ceph RGW  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77776015:12
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm master: Added post-install and post-upgrade annotations on job templates  https://review.opendev.org/c/openstack/openstack-helm/+/77665715:14
gagehugoOpen10K8S: Needs a version update15:30
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm-infra master: Added helm hooks post-install and post-upgrade for jobs  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77646615:30
Open10K8Sgagehugo: jinyuanliu: updated PS15:32
Open10K8SI want to discuss on the job upgrade problem and the purpose of this helm hook stuff with you15:33
Open10K8Ssusanta: can you explain the details? thank you15:38
susantaSure, As we were having the issue with the helm upgrade due to some resources that needs to be upgraded earlier then the jobs, we thought we can add the helm hook post-install and post-upgrade so that jobs are created after all others resources. So With post-install and post-upgrade hooks only we got issue that jobs with hooks needs to be completed before another resource with hook is applied. So we thought to15:41
susantaorder the resources creation with helm hook with weight.15:41
susantaAnd with maintaining the order of the jobs with the hook weight we were able to install and upgrade the helm without any issues.15:42
Open10K8Syeah, so to upgrade charts, more specifically keystone chart, need to delete the jobs manually because of immutable fields15:45
Open10K8Sit means soft-upgrade on helm release is impossible15:45
gagehugothose hooks break helm2 upgrade functionality15:46
Open10K8Sso tried to figure it out and ended up with using helm hook and weights15:46
gagehugoI assume you are using helm315:46
Open10K8Saha yes15:46
susantayes. we have been using helm315:48
Open10K8Shmm... I think helmv2 is also providing hook functions. the thing is helm-toolkit is using in all charts so once we use helm hook option, need to check all charts if they have any other jobs order15:48
Open10K8Sjust tested on keystone, nova, neutron, glance, placement charts and made a PS on helm-toolkit15:49
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm-infra master: Added helm hooks post-install and post-upgrade for jobs  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77646615:53
jinyuanliuI guess a large proportion of people who use openstack-helm to deploy openstack environment, now use helm315:54
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771115:56
Open10K8Syes, I also think so, anyway, we can test the hook functioning with helmv2 shortly15:56
Open10K8Ssusanta: can you test keystone chart hook using helmv2 now?15:57
susantaI am setting up helm2. I will test it shortly.15:57
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771115:58
openstackgerritCliff Parsons proposed openstack/openstack-helm-infra master: [Ceph] Add Ceph CSI plugin  https://review.opendev.org/c/openstack/openstack-helm-infra/+/75084415:59
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771116:16
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771116:31
openstackgerritSusanta gautam proposed openstack/openstack-helm master: Added post-install and post-upgrade annotations on job templates  https://review.opendev.org/c/openstack/openstack-helm/+/77665716:34
*** jinyuanliu has quit IRC16:46
*** jinyuanliu has joined #openstack-helm16:46
*** hemna has quit IRC16:47
*** hemna has joined #openstack-helm16:49
openstackgerritStephen Taylor proposed openstack/openstack-helm-infra master: [ceph-client] Separate pool quotas from pg_num calculations  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77491216:49
openstackgerritMerged openstack/openstack-helm master: Re-enable "feat(tls): Change Issuer to ClusterIssuer""  https://review.opendev.org/c/openstack/openstack-helm/+/77281316:52
openstackgerritChi Lo proposed openstack/openstack-helm-infra master: Enable TLS for grafana  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77434617:46
openstackgerritMerged openstack/openstack-helm-deployments master: Adjust Gate for Airshipctl secret-generate additions  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/77592517:48
openstackgerritFrank Ritchie proposed openstack/openstack-helm-infra master: [WIP] Ceph mgr prometheus tls sidecar  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77780018:02
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm master: Added post-install and post-upgrade annotations on job templates  https://review.opendev.org/c/openstack/openstack-helm/+/77665718:16
openstackgerritSangeet Gupta proposed openstack/openstack-helm-infra master: TLS gates from infra monitoring  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77473818:20
openstackgerritSusanta gautam proposed openstack/openstack-helm master: Added post-install and post-upgrade annotations on job templates  https://review.opendev.org/c/openstack/openstack-helm/+/77665718:24
openstackgerritSusanta gautam proposed openstack/openstack-helm-infra master: Added helm hooks post-install and post-upgrade for jobs  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77646618:30
susantajinyuanliu: We have tested with helmv2 too. It seems we will need to add the hook-delete-policy annotation too for helm v2. I can see that there is no issue in helm install and helm upgrade with post-install and post-upgrade annotation in keystone.18:34
Open10K8Ssusanta ok cool18:35
Open10K8Sgagehugo: so helm hook is fully functioning with helmv2 also. Just the difference between v2 and v3 is that v3 doesn't have default hook-delete-policy18:36
susantayes. So we added the hook-delete-policy and it start working fine.18:40
gagehugoOk18:42
*** vsaienk0 has quit IRC18:44
openstackgerritMerged openstack/openstack-helm-infra master: [ceph-client] Separate pool quotas from pg_num calculations  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77491219:03
openstackgerritBrian Wickersham proposed openstack/openstack-helm-infra master: [ceph-client] enhance logic to enable and disable the autoscaler  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77771119:15
*** sreejithp has quit IRC19:16
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Cinder  https://review.opendev.org/c/openstack/openstack-helm/+/77177519:44
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Glance  https://review.opendev.org/c/openstack/openstack-helm/+/77553319:46
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Heat  https://review.opendev.org/c/openstack/openstack-helm/+/77553419:47
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Horizon  https://review.opendev.org/c/openstack/openstack-helm/+/77553519:49
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Keystone  https://review.opendev.org/c/openstack/openstack-helm/+/77553619:50
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Neutron  https://review.opendev.org/c/openstack/openstack-helm/+/77553719:52
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Nova  https://review.opendev.org/c/openstack/openstack-helm/+/77553819:53
openstackgerritKsawery Dziekoński proposed openstack/openstack-helm master: Added PostgreSQL TLS support and snippets fixing cert permissions for Placement  https://review.opendev.org/c/openstack/openstack-helm/+/77553919:55
openstackgerritSangeet Gupta proposed openstack/openstack-helm-infra master: TLS gates from infra monitoring  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77473820:01
openstackgerritChi Lo proposed openstack/openstack-helm-infra master: Enable TLS for grafana  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77434620:15
*** Anticom has quit IRC20:37
openstackgerritSangeet Gupta proposed openstack/openstack-helm-infra master: TLS gates from infra monitoring  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77473820:37
openstackgerritSangeet Gupta proposed openstack/openstack-helm-infra master: TLS gates from infra monitoring  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77473821:07
openstackgerritGage Hugo proposed openstack/openstack-helm-deployments master: Add keystone to osh-deployments  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/76258621:50
openstackgerritGage Hugo proposed openstack/openstack-helm-deployments master: Add keystone to osh-deployments  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/76258621:50
*** sreejithp has joined #openstack-helm22:21
openstackgerritNafiz Haider proposed openstack/openstack-helm-infra master: [WIP] feat(tls): Enable TLS for OpenStack RabbitMQ  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77067822:23
openstackgerritNafiz Haider proposed openstack/openstack-helm master: [WIP] feat(tls): Make openstack services compatible with rabbitmq with TLS  https://review.opendev.org/c/openstack/openstack-helm/+/77283022:24
openstackgerritNafiz Haider proposed openstack/openstack-helm master: [WIP] feat(tls): Make openstack services compatible with rabbitmq with TLS  https://review.opendev.org/c/openstack/openstack-helm/+/77283022:35
openstackgerritNafiz Haider proposed openstack/openstack-helm-infra master: [WIP] feat(tls): Enable TLS for OpenStack RabbitMQ  https://review.opendev.org/c/openstack/openstack-helm-infra/+/77067822:35

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!