| *** jamesgu_ has joined #openstack-helm | 00:37 | |
| *** aaronsheffield has quit IRC | 00:50 | |
| *** sawblade6 has quit IRC | 01:07 | |
| *** sawblade6 has joined #openstack-helm | 01:08 | |
| *** kranthikirang has quit IRC | 01:50 | |
| *** cfriesen has quit IRC | 01:55 | |
| *** kranthikirang has joined #openstack-helm | 02:22 | |
| *** kranthikirang has quit IRC | 02:26 | |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/openstack-helm-infra master: limiting the Rabbitmq queue length https://review.openstack.org/632759 | 02:38 |
|---|---|---|
| openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: ceph-rgw: Add network policy for ceph-rgw pods https://review.openstack.org/632567 | 03:34 |
| *** kranthikirang has joined #openstack-helm | 04:10 | |
| *** kranthikirang has quit IRC | 04:14 | |
| *** georgk has joined #openstack-helm | 04:16 | |
| *** georgk has quit IRC | 04:20 | |
| openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP/DNM: Add armada experimental job to osh-infra https://review.openstack.org/633067 | 04:23 |
| openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm master: WIP/DNM: Armada check to test password rotation https://review.openstack.org/632558 | 04:29 |
| openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm master: OSH: Add ingress netpol for ceph-rgw pods https://review.openstack.org/633045 | 04:51 |
| *** spsurya has joined #openstack-helm | 05:40 | |
| *** mdih has joined #openstack-helm | 05:48 | |
| *** kranthikirang has joined #openstack-helm | 05:58 | |
| *** kranthikirang has quit IRC | 06:03 | |
| *** jamesgu_ has quit IRC | 06:20 | |
| *** jamesgu has quit IRC | 06:21 | |
| openstackgerrit | Deokjin Kim proposed openstack/openstack-helm-infra master: [prometheus-node-exporter] Deploy pods on all nodes by default https://review.openstack.org/633147 | 06:32 |
| openstackgerrit | Deokjin Kim proposed openstack/openstack-helm-infra master: [prometheus-node-exporter] Deploy pods on all nodes by default https://review.openstack.org/633147 | 06:34 |
| *** roman_g has joined #openstack-helm | 07:29 | |
| *** kranthikirang has joined #openstack-helm | 07:47 | |
| *** kranthikirang has quit IRC | 07:51 | |
| *** roman_g has quit IRC | 08:05 | |
| *** georgk has joined #openstack-helm | 08:11 | |
| *** kranthikirang has joined #openstack-helm | 09:35 | |
| *** roman_g has joined #openstack-helm | 09:35 | |
| *** kranthikirang has quit IRC | 09:39 | |
| *** georgk has quit IRC | 09:40 | |
| evrardjp | o/ | 09:45 |
| *** dimitris_ has joined #openstack-helm | 09:50 | |
| *** georgk has joined #openstack-helm | 09:55 | |
| *** roman_g has quit IRC | 10:03 | |
| *** kranthikirang has joined #openstack-helm | 11:23 | |
| *** kranthikirang has quit IRC | 11:27 | |
| *** mdih has quit IRC | 11:43 | |
| *** georgk has quit IRC | 11:44 | |
| *** georgk has joined #openstack-helm | 11:58 | |
| srwilkers | o/ | 13:05 |
| srwilkers | How’s it going evrardjp? | 13:16 |
| *** georgk has quit IRC | 13:19 | |
| evrardjp | still on my CI | 13:22 |
| evrardjp | when CI will be setup to integrate all the components to install OSH on top, then I will be more chill | 13:23 |
| evrardjp | :) | 13:23 |
| *** bodgix has joined #openstack-helm | 13:40 | |
| openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP/DNM: Add armada experimental job to osh-infra https://review.openstack.org/633067 | 13:48 |
| *** aaronsheffield has joined #openstack-helm | 13:50 | |
| *** jaypipes is now known as leakypipes | 13:50 | |
| *** howell has joined #openstack-helm | 14:10 | |
| openstackgerrit | Deokjin Kim proposed openstack/openstack-helm master: Allow prometheus-openstack-exporter to access keystone https://review.openstack.org/632190 | 14:12 |
| *** cfriesen has joined #openstack-helm | 15:00 | |
| *** spsurya has quit IRC | 15:07 | |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/openstack-helm-infra master: limiting the Rabbitmq queue length https://review.openstack.org/632759 | 15:12 |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/openstack-helm-infra master: limiting the Rabbitmq queue length https://review.openstack.org/632759 | 15:15 |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/openstack-helm-infra master: limiting the Rabbitmq queue length https://review.openstack.org/632759 | 15:17 |
| evrardjp | srwilkers: is it correct to think that multiple value files can be passed to helm, and there is a hash merge of those, with the latest that wins? | 15:18 |
| evrardjp | srwilkers: is it correct to think that multiple value files can be passed to helm, and there is a hash merge of those, with the latest that wins in case of ties? | 15:18 |
| evrardjp | sorry for the double ping | 15:19 |
| *** kranthikirang has joined #openstack-helm | 15:22 | |
| srwilkers | evrardjp: i know multiple values files can be passed, but i'm not sure how variable precedence is handled there. my assumption would be that the latest would win | 15:27 |
| *** lemko has joined #openstack-helm | 15:27 | |
| evrardjp | I remember we can pass multiple value files, and in case of no conflict, a merge happens | 15:28 |
| srwilkers | i guess there's one way to find out | 15:28 |
| openstackgerrit | Merged openstack/openstack-helm master: Allow prometheus-openstack-exporter to access keystone https://review.openstack.org/632190 | 15:28 |
| evrardjp | srwilkers: yeah, but I don't have anything real quick to test this :) | 15:28 |
| srwilkers | i do. give me a few | 15:28 |
| evrardjp | cool thanks | 15:28 |
| *** spsurya has joined #openstack-helm | 15:30 | |
| *** nick_kar has quit IRC | 15:35 | |
| cfriesen | maybe a weird question, but what's the intent for dealing with PCI-passthrough and SR-IOV where we need per-host config? | 15:41 |
| *** GregWaines has joined #openstack-helm | 15:41 | |
| cfriesen | Is it expected that the operator will specify per-host override data? | 15:41 |
| srwilkers | evrardjp: a quick test there shows when multiple values are passed, the last file in the list will take precedence | 15:42 |
| srwilkers | cfriesen: yeah, that would be the case as far as i'm aware | 15:44 |
| evrardjp | srwilkers: yes that was my understanding,thanks for confirming | 15:47 |
| srwilkers | no problem | 15:48 |
| GregWaines | hey there ... general question on the topic of interworking with a Docker Registry with authentication turned on | 15:48 |
| GregWaines | Has anyone looked at how to extend the helm-toolkit function to support docker registry credentials ? | 15:49 |
| GregWaines | e.g. we were thinking of adding an optional imagePullSecret entry in the serviceAccount template ? | 15:49 |
| GregWaines | Although don't understand how we could put this in an 'optional' manner ? | 15:49 |
| GregWaines | Any thoughts ? | 15:49 |
| GregWaines | should I re-ask during office hours in 5 mins ? | 15:55 |
| *** dimitris_ has quit IRC | 15:59 | |
| *** jamesgu has joined #openstack-helm | 16:16 | |
| *** jamesgu_ has joined #openstack-helm | 16:16 | |
| *** jamesgu_ has quit IRC | 16:16 | |
| srwilkers | hey GregWaines -- it could be handled as optional by wrapping that section of the template in a conditional. we do that for other optional fields, like tolerations on daemonsets | 16:30 |
| srwilkers | let me grab a link | 16:30 |
| srwilkers | https://github.com/openstack/openstack-helm-infra/blob/master/fluent-logging/templates/daemonset-fluent-bit.yaml#L96-L98 | 16:31 |
| GregWaines | the other option we just experimented with .... | 16:33 |
| GregWaines | if you ALWAYS put in the ImagePullSecret in the serviceAccount template ... with a well-known secret name | 16:33 |
| GregWaines | then it appears that this STILL works with a Registry with noauth ....if the secret does not exist or even if the secret exists | 16:34 |
| GregWaines | ... and then would also work with a Registry with auth turned on ... as long as the secret exists with the proper credentials | 16:34 |
| GregWaines | would that be acceptable upstream ? | 16:35 |
| GregWaines | i.e. would require no change to upstream operational model if using noauth Registry | 16:35 |
| GregWaines | but if using a tokenAuth Registry ... would require that user first create that secret and then apply the helm charts | 16:36 |
| GregWaines | srwilkers: we looked at doing something similar to your example .... but in the serviceAccount template, I think the only env variables that can be checked are from the specific helm chart ... and there really isn't a variable common across all helm charts that we could use | 16:51 |
| openstackgerrit | Mohan V proposed openstack/openstack-helm-infra master: Adding logging section to PostgreSQL https://review.openstack.org/628336 | 16:53 |
| srwilkers | GregWaines: well, this would require adding something common across all charts to take advantage of. ideally, this would start small (ie, create a helm-toolkit function, then added it to a chart as a RFC upstream), then once proved out it could be rolled out across the rest of the charts | 16:55 |
| srwilkers | preferably, something under the current images: key in the charts probably | 16:56 |
| GregWaines | srwilkers: k, thanks for your input ... we'll probably work on suggesting something upstream in a SPEC in the near future | 16:59 |
| srwilkers | i think that might be the best way forward GregWaines :) | 16:59 |
| srwilkers | let me know when you're ready to throw a spec up and want some eyes on it | 16:59 |
| GregWaines | srwilkers: will do. | 17:47 |
| *** lemko has quit IRC | 17:47 | |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Add liveness probe to fluentd https://review.openstack.org/631336 | 19:39 |
| openstackgerrit | Merged openstack/openstack-helm-infra master: Prometheus: Update pod container status alerts https://review.openstack.org/631814 | 19:39 |
| openstackgerrit | chinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: ceph-rgw: Add network policy for ceph-rgw pods https://review.openstack.org/632567 | 19:47 |
| openstackgerrit | Dustin Specker proposed openstack/openstack-helm-addons master: add post-upgrade hook to readonly SA token secret https://review.openstack.org/633268 | 20:18 |
| *** GregWaines has quit IRC | 20:23 | |
| *** unicell has joined #openstack-helm | 20:27 | |
| *** spsurya has quit IRC | 20:37 | |
| *** unicell has quit IRC | 20:59 | |
| *** howell has quit IRC | 21:22 | |
| *** unicell has joined #openstack-helm | 21:40 | |
| *** unicell has quit IRC | 22:57 | |
| openstackgerrit | Hemachandra Reddy proposed openstack/openstack-helm master: Livenss probe for Neutron agents https://review.openstack.org/632200 | 22:59 |
| *** aaronsheffield has quit IRC | 23:06 | |
| *** unicell has joined #openstack-helm | 23:18 | |
| *** unicell has joined #openstack-helm | 23:50 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!