Thursday, 2021-03-25

« Wednesday, 2021-03-24 Index Friday, 2021-03-26 »
jinyuanliuWho has the time to review for me? They are all small problems00:54
jinyuanliuhttps://review.opendev.org/c/openstack/openstack-helm/+/77268300:55
jinyuanliuhttps://review.opendev.org/c/openstack/openstack-helm/+/77884600:55
*** jinyuanliu has quit IRC01:03
*** jinyuanliu has joined #openstack-helm01:05
openstackgerritMerged openstack/openstack-helm-images master: Prevent python2 installation  https://review.opendev.org/c/openstack/openstack-helm-images/+/78284904:39
openstackgerritThirunavukkarasu Palani proposed openstack/openstack-helm-infra master: Updating python in osd-common-ceph-disk.sh  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78289705:07
openstackgerritSusanta gautam proposed openstack/openstack-helm-infra master: Removed keysize from certificate  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78290106:13
openstackgerritSusanta gautam proposed openstack/openstack-helm master: Fixed the jobAnnotation metadata position  https://review.opendev.org/c/openstack/openstack-helm/+/78290506:32
sgautamHello Everyone, Is there someone that can review the above two patches. For helm toolkit chart, `keySize` field has been removed from `cert-manager.io/v1` api, resulting in the job build failure in zuul for my keystone chart. And for keystone `jobAnnotations` metadata position has been fixed.06:35
openstackgerritSusanta gautam proposed openstack/openstack-helm-infra master: Removed keysize from certificate  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78290107:19
*** hemna has quit IRC07:49
*** hemna has joined #openstack-helm07:50
*** jinyuanliu has quit IRC12:05
*** jinyuanliu has joined #openstack-helm12:06
*** dansmith has quit IRC13:22
openstackgerritRADHIKA PAI proposed openstack/openstack-helm-infra master: [WIP][Update] Enable icmp module in Prometheus blackbox exporter  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78274013:23
*** dansmith has joined #openstack-helm13:24
openstackgerritRADHIKA PAI proposed openstack/openstack-helm-infra master: [WIP][Update] Enable icmp module in Prometheus blackbox exporter  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78274013:26
openstackgerritStephen Taylor proposed openstack/openstack-helm-infra master: [ceph-osd] Update directory-based OSD deployment for image changes  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78285713:30
openstackgerritStephen Taylor proposed openstack/openstack-helm-infra master: [ceph-osd] Update directory-based OSD deployment for image changes  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78285713:31
openstackgerritStephen Taylor proposed openstack/openstack-helm-infra master: [ceph-osd] Update directory-based OSD deployment for image changes  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78285713:33
sgautamHello Everyone, Is there someone that can review the above two patches. For helm toolkit chart, `keySize` field has been removed from `cert-manager.io/v1` api, resulting in the job build failure in zuul for keystone chart. And for keystone `jobAnnotations` metadata position has been fixed.13:59
sgautamhelm-tooklit: https://review.opendev.org/c/openstack/openstack-helm-infra/+/78290114:00
sgautamkeystone: https://review.opendev.org/c/openstack/openstack-helm/+/78290514:00
sgautamplease review them if you have time.14:00
*** miniroy has joined #openstack-helm14:24
*** roman_g has joined #openstack-helm15:59
openstackgerritRADHIKA PAI proposed openstack/openstack-helm-infra master: [WIP][Update] Enable icmp module in Prometheus blackbox exporter  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78274016:39
*** jhesketh has quit IRC17:20
*** anticw has quit IRC17:20
*** anticw has joined #openstack-helm17:20
*** jhesketh has joined #openstack-helm17:20
openstackgerritMerged openstack/openstack-helm master: Add helm.sh/hook related annotation for neutron  https://review.opendev.org/c/openstack/openstack-helm/+/78085117:26
openstackgerritJeremy Houser proposed openstack/openstack-helm-deployments master: Feature: Add Rabbitmq to Openstack-Helm-Deployments  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/76588417:35
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: DMN - Test Values for Elastic Search  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78307117:38
*** v1k0d3n has quit IRC17:42
*** sgautam has quit IRC17:42
*** zugwan has quit IRC17:42
*** thansen has quit IRC17:42
*** stac has quit IRC17:42
*** spy has quit IRC17:42
*** lamt has quit IRC17:42
*** ricolin has quit IRC17:42
*** lipoqil has quit IRC17:42
*** openstackgerrit has quit IRC17:42
*** mchlumsky has quit IRC17:42
*** lemko has quit IRC17:42
*** Deaddy has quit IRC17:42
*** dansmith has quit IRC17:42
*** jinyuanliu has quit IRC17:42
*** ianw has quit IRC17:42
*** gmann has quit IRC17:42
*** portdirect has quit IRC17:42
*** alanmeadows has quit IRC17:42
*** Open10K8S has quit IRC17:42
*** thansen has joined #openstack-helm17:42
*** spy has joined #openstack-helm17:43
*** jinyuanliu has joined #openstack-helm17:44
*** stac has joined #openstack-helm17:44
*** ianw has joined #openstack-helm17:44
*** lipoqil has joined #openstack-helm17:44
*** dansmith has joined #openstack-helm17:44
*** gmann has joined #openstack-helm17:46
*** jhesketh has quit IRC17:52
*** jhesketh has joined #openstack-helm17:52
*** Dantalion has quit IRC17:57
*** Iambchop has quit IRC17:57
*** nicolasbock has quit IRC17:57
*** mnaser has quit IRC17:57
*** Iambchop has joined #openstack-helm17:57
*** nicolasbock has joined #openstack-helm17:57
*** mnaser has joined #openstack-helm17:58
*** dasp_ has quit IRC17:58
*** srwilkers has quit IRC17:58
*** Dantalion has joined #openstack-helm17:58
*** dasp has joined #openstack-helm17:58
*** srwilkers has joined #openstack-helm17:58
*** openstack has joined #openstack-helm18:02
*** ChanServ sets mode: +o openstack18:02
*** kgz has joined #openstack-helm18:03
*** dasp has quit IRC18:03
*** stevthedev has quit IRC18:03
*** megheisler has quit IRC18:03
*** jayahn has quit IRC18:03
*** guilhermesp has quit IRC18:03
*** nicolasbock has quit IRC18:03
*** Iambchop has quit IRC18:03
*** dasp has joined #openstack-helm18:04
*** jayahn has joined #openstack-helm18:04
*** guilhermesp has joined #openstack-helm18:04
*** nicolasbock has joined #openstack-helm18:04
*** Iambchop has joined #openstack-helm18:04
*** stevthedev has joined #openstack-helm18:04
*** megheisler has joined #openstack-helm18:04
*** openstackstatus has joined #openstack-helm18:04
*** ChanServ sets mode: +v openstackstatus18:04
*** nicolasbock has quit IRC18:05
*** Iambchop has quit IRC18:05
*** gmann has quit IRC18:06
*** gagehugo has quit IRC18:06
*** gagehugo has joined #openstack-helm18:06
*** gmann has joined #openstack-helm18:06
*** nicolasbock has joined #openstack-helm18:08
*** Iambchop has joined #openstack-helm18:09
*** anticw has quit IRC18:16
*** miniroy has quit IRC18:16
*** roman_g has quit IRC18:16
*** hemna has quit IRC18:16
*** irclogbot_3 has quit IRC18:16
*** hemna has joined #openstack-helm18:17
*** anticw has joined #openstack-helm18:21
*** irclogbot_1 has joined #openstack-helm18:21
*** roman_g has joined #openstack-helm18:35
*** roman_g has quit IRC18:55
*** Dantalion has quit IRC19:02
*** Dantalion has joined #openstack-helm19:02
*** openstackgerrit has joined #openstack-helm19:08
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: WIP Elasticsearch S3 stuff  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78045319:08
openstackgerritJeremy Houser proposed openstack/openstack-helm-deployments master: Update Gates  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/78308519:43
openstackgerritJeremy Houser proposed openstack/openstack-helm-deployments master: Feature: Add Rabbitmq to Openstack-Helm-Deployments  https://review.opendev.org/c/openstack/openstack-helm-deployments/+/76588419:57
*** roman_g has joined #openstack-helm20:00
*** mchlumsky has joined #openstack-helm20:02
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: WIP Elasticsearch S3 stuff  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78045320:24
openstackgerritSangeet Gupta proposed openstack/openstack-helm-infra master: HTK: Override the expiry of Ingress TLS certificate  https://review.opendev.org/c/openstack/openstack-helm-infra/+/78311722:19
openstackgerritGuilherme proposed openstack/openstack-helm master: Update glance default policy values  https://review.opendev.org/c/openstack/openstack-helm/+/78311922:34
guilhermespgagehugo: hey there! How are you doing? Could give your thoughts ^ :)22:36
gagehugooh they modified the wiki page22:44
gagehugoUnless you are strict with who you give admin to, I still think that is not great22:45
gagehugoguilhermesp: are you using the default policies btw?22:46
guilhermespgagehugo: you mean the default that we are setting now right? ( aka '!' ) if so yeah, im facing the same horizon behavior in multi envs :P23:14
guilhermespbut indeed i agree with you23:14
gagehugoI was going off the first revision, so updating it to the latest is fine23:15
guilhermespwe are minimizing the scope of the issue restricting only "normal" users and we sill run into the issue with "untrusted" admins23:15
gagehugothanks for remembering horizon has all those policies too, I forget that it has those23:15
guilhermespat my operator point of view, i'd only give and admin role to a trusted member23:15
guilhermespyeah that's another thing you might know maybe23:15
gagehugo:)23:15
guilhermespi tried to23:15
guilhermespreplicate the disabling on horizon policies23:16
guilhermespbut i had no effect at all23:16
gagehugolol23:16
gagehugomy knowledge of horizon is lacking23:16
guilhermespi can always triple check :P let me test again replicating this https://github.com/openstack/openstack-helm/blob/master/glance/values.yaml#L223-L242 here https://github.com/openstack/openstack-helm/blob/bd476a66778ca11e2e4c6248e776943032cf8cf3/horizon/values.yaml#L103823:20
guilhermespyep, tripled checked, triple disappointed with horizon lol23:26
guilhermespbut yeah that was my first approach indeed23:26
guilhermespso i have up for loosen a bit the restrictions23:26
gagehugoyeah that's fine, it's what the OSSN recommends23:28
guilhermespk cool. i appreciate a lot the discussion gagehugo ! ps that's my first direct contribution :P23:28
gagehugooh nice23:29
*** roman_g has quit IRC23:49
« Wednesday, 2021-03-24 Index Friday, 2021-03-26 »

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