Tuesday, 2019-03-05

*** michaelbeaver has quit IRC00:00
*** kranthikirang has joined #openstack-helm00:01
*** kranthikirang has quit IRC00:05
*** sdake has joined #openstack-helm00:08
*** sdake has quit IRC00:12
*** sdake has joined #openstack-helm00:14
*** openstackgerrit has joined #openstack-helm00:20
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Ensure ceph-rbd-pool job runs with MON IPs  https://review.openstack.org/63764900:20
*** aaronsheffield has quit IRC00:25
*** sreejithp has quit IRC00:37
*** sdake has quit IRC00:41
*** cheng1 has joined #openstack-helm00:48
*** cheng1 has quit IRC00:54
*** rchurch has joined #openstack-helm00:54
*** rchurch_ has quit IRC00:55
*** sdake has joined #openstack-helm01:07
openstackgerritKoffi Nogbe proposed openstack/openstack-helm-infra master: Add mariadb and postgresql backup and restore * backup script for mariadb and postgres * restore script for mariadb and postgres * cronjob for automated backup mariadb and postgres * add parameters to values.yaml  https://review.openstack.org/63829001:07
*** rchurch has quit IRC01:10
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Ensure ceph-rbd-pool job runs with MON IPs  https://review.openstack.org/63764901:10
*** rchurch has joined #openstack-helm01:11
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Switch to using ceph-volume for ceph-osd chart  https://review.openstack.org/63398101:13
*** supamatt has joined #openstack-helm01:19
*** cheng1 has joined #openstack-helm01:21
*** sthussey has quit IRC01:22
cheng1o/01:26
*** kranthikirang has joined #openstack-helm01:30
*** kranthikirang has quit IRC01:34
openstackgerritBob Church proposed openstack/openstack-helm master: Add resource_filters.json to Cinder's configmap  https://review.openstack.org/64091501:38
*** Nishant__ has quit IRC01:40
openstackgerritMerged openstack/openstack-helm master: Report the reason why the script hits a timeout.  https://review.openstack.org/63475101:44
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Ensure ceph-rbd-pool job runs with MON IPs  https://review.openstack.org/63764902:04
*** sdake has quit IRC02:05
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Switch to using ceph-volume for ceph-osd chart  https://review.openstack.org/63398102:15
openstackgerritKoffi Nogbe proposed openstack/openstack-helm-infra master: Add mariadb and postgresql backup and restore * backup script for mariadb and postgres * restore script for mariadb and postgres * cronjob for automated backup mariadb and postgres * add parameters to values.yaml  https://review.openstack.org/63829002:19
openstackgerritMatthew Heler proposed openstack/openstack-helm-images master: [CEPH] Make the Ceph user run with an interactive bash shell  https://review.openstack.org/64092302:27
*** sdake has joined #openstack-helm02:28
supamatt@portdirect ^ this is needed to run the ceph containers under a privileged user02:37
portdirectwhy? could we not just own the sudoer's config or is there another reason here?02:38
supamattwhat user do you want the pods to run under?02:40
portdirectideally `ceph` i would have thought02:41
supamattBy default it's configured with /bin/false02:41
portdirectyeah - though unsure why we would need to change this?02:42
supamatturmm.. I think I get what you are saying02:42
supamattSo you are saying setting 'runAsUser' with a non root uuid would still work if the shell is set to /bin/nologin or /bin/false?02:44
portdirectshould do02:44
*** hemanth_n has joined #openstack-helm02:46
portdirectthe only thing you'll probably see issues with are the lack of a tty on the containers, and wanting to specify the commands that can be used by sudo02:49
supamattwe need a tty if we intend to execute into the container02:49
portdirectnot on the container itself02:50
portdirectfor the sudo issues - the pattern here can help: https://github.com/openstack/openstack-helm/blob/master/cinder/values.yaml#L446-L45002:50
portdirectfor clarification - i mean open a tty when launching the container02:51
*** hemanth_n has quit IRC02:57
supamattgotcha, yea..03:01
supamattfails pretty quickly, there is going to be a little bit of massaging here03:02
*** roman_g has quit IRC03:17
*** kranthikirang has joined #openstack-helm03:18
*** kranthikirang has quit IRC03:22
*** cheng1 has quit IRC03:37
*** sdake has quit IRC03:51
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-osd] resolve name conflicts by appending release name  https://review.openstack.org/64087704:39
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Add init container to load apparmor profile  https://review.openstack.org/60882604:59
*** cheng1 has joined #openstack-helm05:31
*** sdake has joined #openstack-helm05:47
*** cheng1 has quit IRC05:48
*** sdake has quit IRC05:50
*** sdake has joined #openstack-helm05:58
*** cheng1 has joined #openstack-helm05:58
*** dims has quit IRC06:24
*** dims has joined #openstack-helm06:26
*** dims has quit IRC06:36
*** dims has joined #openstack-helm06:37
*** cheng1 has quit IRC06:40
*** cheng1 has joined #openstack-helm07:06
*** jsuchome has joined #openstack-helm07:07
*** pgaxatte has joined #openstack-helm07:16
*** aojea has joined #openstack-helm07:23
openstackgerritBob Church proposed openstack/openstack-helm master: Add resource_filters.json to Cinder's configmap  https://review.openstack.org/64091507:42
*** nmimi has quit IRC07:54
*** JangwonLee has quit IRC08:09
*** JangwonLee has joined #openstack-helm08:23
*** sdake has quit IRC08:36
*** kranthikirang has joined #openstack-helm08:38
*** kranthikirang has quit IRC08:42
*** roman_g has joined #openstack-helm08:43
openstackgerritcheng li proposed openstack/openstack-helm master: Support per-host overrides of auto_bridge_add  https://review.openstack.org/63908708:43
cheng1evgenyl, regarding the release note. Would mind to discuss in irc meeting today? If it's confirmed, I would like to add it in a separated patch.08:46
cheng1evrardjp, I meant to at you :)08:48
evrardjpcheng1: separate patch is fine08:48
evrardjpelse I would have voted -08:48
evrardjp-108:48
evrardjpI think the release notes work was already started in the past, I will query why this wasn't done08:48
evrardjpfinished*08:48
cheng1sure08:50
evrardjpif you are the only one to write reno, I am not even sure it's warranting a release note.08:50
evrardjpI just thought -- because we recently decided to use reno -- that it was nice to use them for this, as it is a customer facing new feature08:51
cheng1agree, but seems we even have release08:52
cheng1do you know why no release since 2017?08:52
*** witek has joined #openstack-helm08:54
*** happyhemant has joined #openstack-helm08:59
*** itxaka has joined #openstack-helm09:00
*** rha has quit IRC09:07
*** pgaxatte has quit IRC09:10
*** pgaxatte has joined #openstack-helm09:12
*** roman_g has quit IRC09:20
*** lemko has joined #openstack-helm09:23
*** cheng1 has quit IRC09:34
*** roman_g has joined #openstack-helm09:41
*** cheng1 has joined #openstack-helm09:49
*** sdake has joined #openstack-helm09:55
roman_gcheng1: this is a worklist for openstack-helm 1.0 release, created by Matt McEuen - https://storyboard.openstack.org/#!/worklist/341 ; may be he knows more about releases.09:58
cheng1roman_g, thanks for the info10:00
*** cheng1 has quit IRC10:08
*** Anticom has joined #openstack-helm10:25
*** hemanth_n has joined #openstack-helm10:29
*** sdake has quit IRC10:51
*** sdake has joined #openstack-helm10:54
*** cheng1 has joined #openstack-helm10:56
*** cheng1 has quit IRC11:05
*** hemanth_n has quit IRC11:15
*** dimitris_ has joined #openstack-helm11:23
*** cheng1 has joined #openstack-helm11:46
*** cheng1 has quit IRC11:54
*** sdake has quit IRC12:39
*** cheng1 has joined #openstack-helm12:40
evrardjproman_g: please note I don't think it prevents us from releasing quite the other way around, we should relase more frequently, even if it's not 1.012:40
roman_gevrardjp: I'm not against releases. Just wanted to say that it would be good to talk to Matt on this topic.12:53
evrardjproman_g: oh yeah I didn't mean it like that :)12:54
evrardjpI wasn't even aware we had a release tbh12:54
roman_gyou can add it for a discussion for the meeting https://etherpad.openstack.org/p/openstack-helm-meeting-2019-03-0512:54
roman_gin the end, releases play for marketing purposes.12:55
evrardjpI don't think so -- we could use release for versioning the helm charts and the built images12:55
roman_gwhich I think is good, as this is one of the way to get attention and get more users12:55
evrardjpsorry12:55
evrardjpI mean12:55
evrardjpI don't think it's only that --12:56
roman_gof course12:56
evrardjp:)12:56
*** cheng1_ has joined #openstack-helm13:19
*** sdake has joined #openstack-helm13:20
*** cheng1 has quit IRC13:22
*** hemanth_n has joined #openstack-helm13:31
*** levmorgan has joined #openstack-helm13:51
*** aaronsheffield has joined #openstack-helm13:55
*** Nishant_ has joined #openstack-helm13:57
*** nmimi has joined #openstack-helm13:58
*** cheng1_ has quit IRC13:58
*** sgrasley has joined #openstack-helm13:59
*** sthussey has joined #openstack-helm14:05
*** sdake has quit IRC14:11
*** howell has joined #openstack-helm14:13
*** dustinspecker has joined #openstack-helm14:13
*** michael-beaver has joined #openstack-helm14:17
*** sdake has joined #openstack-helm14:21
*** lemko has quit IRC14:22
*** sdake has quit IRC14:23
portdirectsrwilkers, lrensing, v1k0d3n, lamt, portdirect, renmak, jayahn, korzen, alanmeadows, mattmceuen, raymaika, evrardjp, MegHeisler, howell, georgk, arunkant, roman_g: meeting in 30 mins folks, over in #openstack-meeting-414:32
portdirectAgenda is here: https://etherpad.openstack.org/p/openstack-helm-meeting-2019-03-0514:33
*** sdake has joined #openstack-helm14:34
srwilkerso/14:35
evrardjpthanks portdirect14:36
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: HTK: Refactor kubernetes security_context macro to allow scoping  https://review.openstack.org/63734514:39
*** kranthikirang has joined #openstack-helm14:41
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: HTK: Refactor kubernetes security_context macro(s) to allow scoping  https://review.openstack.org/63734514:42
*** sdake has quit IRC14:48
*** sdake has joined #openstack-helm14:49
openstackgerritMerged openstack/openstack-helm-images master: Zuul: Update dockerhub credentials and remove newline from secret  https://review.openstack.org/64078214:55
openstackgerritAngie Wang proposed openstack/openstack-helm master: Ceilometer chart: replace the obsolete ceilometer upgrade command  https://review.openstack.org/63973314:55
openstackgerritAngie Wang proposed openstack/openstack-helm master: Ceilometer chart: upgrade the default to ocata  https://review.openstack.org/64042714:55
*** Darcidride has joined #openstack-helm14:55
openstackgerritAngie Wang proposed openstack/openstack-helm master: Ceilometer chart: Add missing definition files  https://review.openstack.org/64082814:55
openstackgerritAngie Wang proposed openstack/openstack-helm master: Ceilometer chart: Add polling process with ipmi functionality  https://review.openstack.org/64082914:56
openstackgerritAngie Wang proposed openstack/openstack-helm master: Ceilometer chart: update messaging_urls and cache server  https://review.openstack.org/64042614:56
openstackgerritAngie Wang proposed openstack/openstack-helm master: Spec: Support docker registry with authentication turned on  https://review.openstack.org/63622314:56
*** cheng1 has joined #openstack-helm15:01
*** sdake has quit IRC15:04
*** sdake has joined #openstack-helm15:06
*** michael-beaver has quit IRC15:07
openstackgerritItxaka Serrano Garcia proposed openstack/openstack-helm master: make horizon work on suse based images  https://review.openstack.org/64103315:20
openstackgerritKoffi Nogbe proposed openstack/openstack-helm-infra master: Add mariadb and postgresql backup and restore * backup script for mariadb and postgres * restore script for mariadb and postgres * cronjob for automated backup mariadb and postgres * add parameters to values.yaml  https://review.openstack.org/63829015:23
*** levmorgan has quit IRC15:27
*** michael-beaver has joined #openstack-helm15:27
*** openstackgerrit has quit IRC15:28
*** Darcidride has quit IRC15:29
*** openstackgerrit has joined #openstack-helm15:30
openstackgerritHemanth Nakkina proposed openstack/openstack-helm-infra master: Add annotations to podsecuritiypolicy  https://review.openstack.org/64066415:30
openstackgerritHemanth Nakkina proposed openstack/openstack-helm-infra master: Add annotations to podsecuritypolicy  https://review.openstack.org/64066415:30
openstackgerritPRATEEK REDDY DODDA proposed openstack/openstack-helm master: readOnlyFilesystem: true for glance chart  https://review.openstack.org/63944415:38
*** pgaxatte has quit IRC15:43
evrardjpDavidSull: wow15:43
evrardjpthat wasn't in heat?15:43
evrardjphow have we missed that15:43
evrardjp:p15:43
portdirectevrardjp: it would be great to get more eyes on the charts for things like this... ;D15:46
evrardjpportdirect: don't worry I will scrutinize everything when the time comes :p15:46
evrardjpI thought the engine cleaner was doing this tbh, so I will check15:46
evrardjpportdirect: if you could explain here how you do your review to test the syntax of this new feature, it would be helpful for everyone15:47
evrardjpjust saying15:47
portdirectjust reviewing the gotpl, and rendered k8s objects15:48
evrardjplamt: I think we should update the .gitignore for those language string15:48
evrardjpyou  want me to do it?15:48
evrardjpportdirect: how do you render them before applying it, is there some kind of dry run thing?15:48
evrardjpjust tell me yes or now, I will then google it :)15:49
portdirecthelm template15:49
*** sdake has quit IRC15:49
evrardjpok15:49
evrardjpcool15:49
evrardjpthanks15:49
evrardjpjsuchome: rha FYI ^15:49
portdirectand helm lint15:49
evrardjpthis one was known15:51
evrardjpor presumed known15:51
evrardjpbut yeah my bad, I shouldn't assume15:51
portdirect`helm install --dry-run --debug` is also great15:55
portdirectas it runs it against the k8s api to validate15:55
*** michael-beaver has quit IRC15:56
evrardjpoh so that's a little further than helm template15:59
evrardjpI will try those15:59
evrardjpI hope my colleagues are reading :)15:59
*** belmoreira has quit IRC16:00
*** sdake has joined #openstack-helm16:01
*** cheng1 has quit IRC16:01
DavidSullevrardjp: Yeah as far as I know it's not in heat. We've had to do this type of thing since newton I think16:04
evrardjpDavidSull: I meant in the heat cron job we have already16:04
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: Add default AppArmor profile to prometheus-node-exporter  https://review.openstack.org/63981016:13
*** michael-beaver has joined #openstack-helm16:18
DavidSullevrardjp: Ahh. As far as I can see the existing job only calls service clean, which performs separate operations from purge_deleted.16:18
portdirectDavidSull: yup, i think putting it in a seperate job like you have is the right move here16:19
evrardjpDavidSull: yes that's most likely16:20
evrardjpportdirect: I guess it's indeed clearer, but I don't know the content of the cronjob yet, so it means I had to dig.. that's all I was saying16:20
*** michael-beaver has quit IRC16:22
*** michaelbeaver has joined #openstack-helm16:22
*** michaelbeaver has quit IRC16:26
*** hemanth_n has quit IRC16:27
*** hemanth_n has joined #openstack-helm16:29
*** hemanth_n has quit IRC16:31
openstackgerritDrew Walters proposed openstack/openstack-helm-images master: mini-mirror: Allow per-source Aptly config files  https://review.openstack.org/64107416:31
*** roman_g has quit IRC16:51
*** roman_g has joined #openstack-helm16:53
-openstackstatus- NOTICE: Gerrit is being restarted for a configuration change, it will be briefly offline.17:10
*** michael-beaver has joined #openstack-helm17:10
*** jistr|sick is now known as jistr17:14
*** DavidSull has quit IRC17:16
*** aojea has quit IRC17:34
*** sdake has quit IRC17:35
*** Anticom has quit IRC17:49
*** DavidSull has joined #openstack-helm17:49
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: readOnlyRootFilesystem: true for postgresql chart (cloud infra)  https://review.openstack.org/64049218:15
openstackgerritdiwakar thyagaraj proposed openstack/openstack-helm-infra master: Add default AppArmor profile to prometheus-node-exporter  https://review.openstack.org/63981018:30
openstackgerritKoffi Nogbe proposed openstack/openstack-helm-infra master: Add mariadb and postgresql backup and restore * backup script for mariadb and postgres * restore script for mariadb and postgres * cronjob for automated backup mariadb and postgres * add parameters to values.yaml  https://review.openstack.org/63829018:38
openstackgerritScott Hussey proposed openstack/openstack-helm-infra master: (postgresql) set db admin password at startup  https://review.openstack.org/63507018:38
*** happyhemant has quit IRC18:39
openstackgerritMunish proposed openstack/openstack-helm-infra master: Adding text collector for i40e VF stats from node. The stats are generated by divingbell job and node-exporter picks the stats from the host file system.  https://review.openstack.org/62870018:59
openstackgerritMunish proposed openstack/openstack-helm-infra master: Adding text collector for i40e VF stats from node. The stats are generated by divingbell job and node-exporter picks the stats from the host file system.  https://review.openstack.org/62870019:01
openstackgerritMunish proposed openstack/openstack-helm-infra master: Adding text collector for i40e VF stats from node. The stats are generated by divingbell job and node-exporter picks the stats from the host file system.  https://review.openstack.org/62870019:02
*** witek has quit IRC19:16
*** sdake has joined #openstack-helm19:33
*** jsuchome has quit IRC19:43
openstackgerritPRATEEK REDDY DODDA proposed openstack/openstack-helm master: readOnlyFilesystem: true for neutron chart  https://review.openstack.org/63944319:44
*** unicell has joined #openstack-helm19:45
roman_gsthussey, mattmceuen, commit which broke overrides is this one: https://github.com/helm/helm/commit/de5a6a931bfc47c1fa3a72db038002cbf68315ff19:45
portdirectgithub.com/imdario/mergo19:47
portdirectI'd be looking there to see what changed roman_g19:47
roman_gportdirect: thanks. So, somewhere in between 6633656539c1639d9d78127b7d47c622b5d7b6dc and v0.3.519:48
portdirectthis is where my gut leads me, but nothing more im afraid19:49
roman_gOh, this 6633656539c1639d9d78127b7d47c622b5d7b6dc is committed on Dec 6, 201419:53
openstackgerritPRATEEK REDDY DODDA proposed openstack/openstack-helm master: readOnlyFilesystem: true for horizon chart  https://review.openstack.org/64111319:56
roman_gportdirect: changeing github.com/imdario/mergo from v0.3.5 back to 6633656539c1639d9d78127b7d47c622b5d7b6dc in glide.yaml and rebuilding latest helm does not make anything better20:16
*** sdake has quit IRC20:17
*** sdake has joined #openstack-helm20:19
*** georgk has quit IRC20:52
*** georgk has joined #openstack-helm20:53
openstackgerritPRATEEK REDDY DODDA proposed openstack/openstack-helm master: readOnlyFilesystem: true for horizon chart  https://review.openstack.org/64111321:06
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: HTK: Refactor kubernetes security_context macro(s) to allow scoping  https://review.openstack.org/63734521:42
*** rchurch_ has joined #openstack-helm21:45
*** rchurch has quit IRC21:47
openstackgerritKoffi Nogbe proposed openstack/openstack-helm-infra master: Add mariadb and postgresql backup and restore * backup script for mariadb and postgres * restore script for mariadb and postgres * cronjob for automated backup mariadb and postgres * add parameters to values.yaml  https://review.openstack.org/63829021:48
openstackgerritMerged openstack/openstack-helm-infra master: [ceph-osd] resolve name conflicts by appending release name  https://review.openstack.org/64087721:48
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: readOnlyRootFilesystem: true for Prometheus chart  https://review.openstack.org/64054321:57
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: readOnlyRootFilesystem: true for Prometheus exporters charts  https://review.openstack.org/63947321:58
openstackgerritPRATEEK REDDY DODDA proposed openstack/openstack-helm master: readOnlyFilesystem: true for horizon chart  https://review.openstack.org/64111322:17
openstackgerritAngie Wang proposed openstack/openstack-helm master: Add Panko Chart  https://review.openstack.org/64114422:18
*** sdake has quit IRC22:28
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-osd] fix ceph jounral partition creation  https://review.openstack.org/64114822:31
*** howell has quit IRC22:36
*** sdake has joined #openstack-helm22:38
*** kranthikirang has quit IRC22:42
*** dustinspecker has quit IRC22:44
*** sdake has quit IRC22:45
*** sdake has joined #openstack-helm22:51
*** sdake has quit IRC23:02
*** sgrasley has quit IRC23:32
*** sdake has joined #openstack-helm23:33
*** michael-beaver has quit IRC23:33
roman_gportdirect: I think I'm mistaken. I forgot to update glide lock file (glide.lock). If I update it to the older version, helm build totally breaks.23:39
roman_gWill work on that tomorrow.23:40

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!