15:00:43 <gagehugo> #startmeeting openstack-helm
15:00:44 <openstack> Meeting started Tue Oct  6 15:00:43 2020 UTC and is due to finish in 60 minutes.  The chair is gagehugo. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:45 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:48 <openstack> The meeting name has been set to 'openstack_helm'
15:00:50 <stevthedev> o/
15:00:52 <gagehugo> #link https://etherpad.opendev.org/p/openstack-helm-weekly-meeting agenda
15:00:53 <lamt> \o
15:00:56 <gagehugo> o/
15:00:57 <miniroy> o/
15:01:06 <reddy1> o/
15:01:08 <sangeet> \o
15:01:24 <fungi> heh
15:01:36 <fungi> not the security meeting indeed
15:01:49 <gagehugo> I can't tell the days apart anymore :(
15:02:08 <stevthedev> I felt that
15:03:01 <miniroy> but the days do not come apart, that's the problem
15:03:05 <megheisler> \o
15:03:08 <miniroy> it just hit your continuously  :)
15:04:05 <stevthedev> Apologies in advance but I am gonna beg for reviews today :)
15:04:45 <gagehugo> stevthedev put them in the agenda :p
15:05:01 <gagehugo> #topic Open Infrastructure PTG
15:05:16 <gagehugo> Just a reminder the PTG is in a few weeks
15:05:19 <gagehugo> #link https://etherpad.opendev.org/p/openstack-helm-wallaby-ptg
15:05:25 <gagehugo> ^ list of topics
15:06:50 <gagehugo> #topic Upgrade k8s in gate to 1.18
15:07:14 <sangeet> Will have to make sure ther are no greps, they will all have to converted to jq
15:07:18 <sangeet> Else we will see faiures
15:07:29 <gagehugo> hmm
15:07:54 <lamt> besides that - is there any other reasons we can't move up from the 1.16 minikube ?
15:07:57 <sangeet> I think managedFields get added by default
15:08:10 <gagehugo> none that I can think of
15:08:19 <gagehugo> unless it just all falls apart
15:08:28 <miniroy> gagehugo how is the upgrade work coordinated usually?
15:08:43 <lamt> we should fix any fall out
15:09:16 <gagehugo> miniroy: someone is free to pick it up and we can track it via the weekly meetings
15:10:37 <miniroy> probably no earlier than next week... I like to see if it is stable enough in our env this week first
15:10:39 <sangeet> I can try
15:10:49 <gagehugo> ok cool
15:10:59 <reddy1> I can help too
15:12:10 <gagehugo> awesome
15:12:34 <lamt> miniroy: at some point, we can revive that terrible multinode gate patch I have
15:12:48 <miniroy> oh yes... that seems more fun  :)
15:12:57 <gagehugo> yes we will need to do that too
15:12:59 <miniroy> I do like another crack at that
15:13:02 <lamt> something wonky happened and I got side tracked - the worker nodes won't join properly
15:14:12 <miniroy> lamt I will sync up with you later
15:14:39 <gagehugo> lamt miniroy: I started a ps to get that multinode working again, but reviving lamt's ps might be better
15:14:54 <gagehugo> I'll help out too there
15:15:05 <gagehugo> #topic Reviews
15:15:15 <gagehugo> #link https://review.opendev.org/#/c/755418 - Alertmanager Command Line Flags
15:15:24 <gagehugo> #link https://review.opendev.org/#/c/756262 - Alertmanager LDAP / Auth Sidecar
15:15:34 <gagehugo> #link https://review.opendev.org/#/c/754634 - Prometheus Probe Template
15:15:42 <gagehugo> #link https://review.opendev.org/#/c/756241 - Revert "Prometheus: Render Rules as Templates"
15:15:53 <stevthedev> I have a few PS pending today. The two Alertmanager ones are more pressing than the prometheus ones, but they would be nice to have as well
15:15:54 <gagehugo> plenty to look at :)
15:16:08 <stevthedev> Thanks for posting the links gagehugo
15:17:06 <stevthedev> The last one (the revert) would be especially nice. I first submitted that as a way to add dynamic capability to the prometheus rules (they infamously don't support ENV variable substitution there)
15:17:33 <gagehugo> stevthedev: can you update the commit message real quick then with that reason you provided?
15:17:49 <stevthedev> But last night realized this could be easily done with another solution, and would clean up the prometheus chart values significantly
15:17:51 <stevthedev> Sure thing
15:19:03 <stevthedev> Roy to your comment about the chart version - The initial change was made while everything sat at 0.1.0, so the lint gate is kind of unaware of that
15:19:07 <stevthedev> It asked to bump the version
15:19:50 <stevthedev> This is why I based it on the probe PS and incremented the version in both
15:19:55 <stevthedev> What do you all think?
15:19:58 <gagehugo> yeah, that's a good question there
15:20:04 <gagehugo> I'd say just bump it
15:20:15 <gagehugo> keep moving forward
15:21:07 <miniroy> stevthedev I am ok with bumping it obviously, I am just a fatalist by nature... lol
15:21:30 <stevthedev> Commit message updated
15:21:53 <gagehugo> cool
15:22:06 <gagehugo> #topic Open Discussion
15:22:11 <gagehugo> Anyone have anything else for this week?
15:24:35 <stevthedev> Thanks for the reviews everyone. I will address your comments and maybe bug you later :)
15:26:55 <gagehugo> thanks everyone, have a good week!
15:26:59 <gagehugo> #endmeeting