Thursday, 2020-09-10

openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [WIP] Add Ceph CSI plugin  https://review.opendev.org/75084400:04
openstackgerritGage Hugo proposed openstack/openstack-helm-deployments master: Add README and CONTRIBUTING  https://review.opendev.org/75084500:22
openstackgerritGage Hugo proposed openstack/openstack-helm-deployments master: Add initial zuul config yaml  https://review.opendev.org/75084700:30
openstackgerritMerged openstack/openstack-helm-deployments master: Add initial zuul config yaml  https://review.opendev.org/75084700:31
openstackgerritGage Hugo proposed openstack/openstack-helm-deployments master: Add README and CONTRIBUTING  https://review.opendev.org/75084500:31
openstackgerritMerged openstack/openstack-helm-infra master: Alerta: Fix files with CRLF eol characters  https://review.opendev.org/75066700:49
openstackgerritMerged openstack/openstack-helm master: Implement missing security context for nginx container  https://review.opendev.org/74932300:57
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Fix elasticsearch issue with JVM 14  https://review.opendev.org/75064701:08
*** yingjisun has joined #openstack-helm01:14
openstackgerritMerged openstack/openstack-helm-infra master: Alerta: Fix secret name and overrides  https://review.opendev.org/75066801:50
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: [tls] Add missing mysql certs to heat-purge-deleted cj  https://review.opendev.org/75085301:51
openstackgerritMerged openstack/openstack-helm master: Imported Translations from Zanata  https://review.opendev.org/75059401:52
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: [tls] Add missing mysql certs to heat-purge-deleted cj  https://review.opendev.org/75085302:04
openstackgerritMerged openstack/openstack-helm-deployments master: Add README and CONTRIBUTING  https://review.opendev.org/75084502:34
openstackgerritMerged openstack/openstack-helm-infra master: Add default reject rule at the end in Postgres pg_hba.conf to ensure all connections must be explicitly allowed.  https://review.opendev.org/74920202:52
openstackgerritMerged openstack/openstack-helm-images master: Fix TLS support in openstack-exporter osclient  https://review.opendev.org/75001003:28
*** stac- has quit IRC03:32
*** stac_ has joined #openstack-helm03:33
*** yingjisun has quit IRC03:38
*** yingjisun has joined #openstack-helm03:38
openstackgerritMerged openstack/openstack-helm master: [tls] Add missing mysql certs to heat-purge-deleted cj  https://review.opendev.org/75085303:47
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-helm04:33
*** yingjisun has joined #openstack-helm05:02
*** suryasingh has joined #openstack-helm06:33
openstackgerritMerged openstack/openstack-helm-infra master: Upgrade etcd to 3.4.3  https://review.opendev.org/74980107:26
*** yingjisun_ has joined #openstack-helm08:45
*** yingjisun has quit IRC08:46
*** yingjisun_ is now known as yingjisun08:46
mnasergagehugo: wondering about my question a little while back above about using openstack-helm charts in something like helm-operator when the deps are pointing to localhost ^12:34
DeaddyI just have made a small chart repo for that on localhost which I then serve with python -m http.server <helm port>12:37
mnaserDeaddy: yeah but in my case, if i'm using something like helm-operator.. i guess i'd have to maybe run.. a sidecar or something that runs it?12:38
*** yingjisun has quit IRC12:39
Deaddyah, I see12:39
portdirectWhy server them over http at all in this case?12:39
portdirect*serve12:39
mnaserportdirect: i haven't gotten that far yet but a lot of the helm charts have dependencies: http://localhost..12:39
mnaserso i cant point to the git repo commit as it'll probably try and do helm dep up and error out12:40
portdirectOnce built, you could just host the tarballs in the operator12:40
mnaserportdirect: right, i was hoping to be able to point towards the git repo / specific commit and avoid having a 'build' step :)12:41
portdirectAh, I see12:41
mnaserso this is where it (i suspect) will blow up: https://github.com/openstack/openstack-helm/blob/master/senlin/requirements.yaml12:41
portdirectHow are you constructing the charts then?12:41
portdirectAre you using a helm plug in or something to allow you to point to a git repo?12:42
mnaserportdirect: right now, we never do.  helm-operator/argocd pointing directly at a git repo, and instead of using subcharts, we use another application(argocd)/helmrelease(operator) inside the helm chart itself12:42
mnasernow i guess the issue is for a 'library', this won't cut it12:43
portdirectBy helm-operator, I assume you mean flux one?12:43
mnaseryep12:43
portdirectOk, yeah that would need some work12:44
mnaserportdirect: off the top of your head, what would you do in 'an ideal world'?12:44
portdirectIt would be nice if osh just published tarballs, which would make this easy12:44
mnaseri'm wondering if people would be comfortable with the idea of running chart-testing against the repos12:44
mnaserand enforce version bumps of charts on every change12:44
mnaseri've already have that implemented as a zuul job12:44
portdirectNice, a ps would be awesome!12:45
mnaserthat would be step 1, and then perhaps after we can publish tarballs/repo somewhere like tarballs.openstack.org12:45
portdirectYup12:45
mnaserthe only pain is well, people's changes will fail if they don't bump the chart version12:45
mnaserbut it seems that openstack-helm is relatively low velocity12:45
portdirectIts pretty mature at this point12:45
mnaserthe only other pita is going to be version bumps if helm-toolkit sees changes12:46
mnaserso it would be a 2 stage change12:46
mnaserbump helm-toolkit, then bump consumers of it.. which is reasonable and makes sure we don't ever break each other i guess12:46
portdirect++12:47
mnaserportdirect: i'll hack on this a bit, also btw i submitted a patch to loci to add re-add debian support under buster12:47
mnaserbut that's.. "busted" because we need octopus mirrors in infra12:47
mnaseras ceph did not publish nautilus buster debs12:48
portdirectRoger, will take a look today12:48
portdirectSorry for slow responses, I've been snowed under at work recently12:48
mnaserno worries, i understand12:49
gagehugomnaser: would love to see a ps for that!13:10
mnasergagehugo: will have something for this soon13:11
*** mfixtex has joined #openstack-helm13:18
*** vsaienk0 has quit IRC13:34
*** belmoreira has quit IRC13:36
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: Implement missing security context template for daemonjob-controller  https://review.opendev.org/74804714:11
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: Implement missing security context template for daemonjob-controller  https://review.opendev.org/74804714:12
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Move tls job to gate pipeline  https://review.opendev.org/75100214:32
*** sreejithp has joined #openstack-helm14:56
openstackgerritRahul Khiyani proposed openstack/openstack-helm-infra master: Implement missing security context template for daemonjob-controller  https://review.opendev.org/74804715:21
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103515:42
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103515:43
*** mfixtex has quit IRC15:51
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103515:53
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103515:55
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Move tls job to gate pipeline  https://review.opendev.org/75100215:59
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103515:59
openstackgerritMeghan Heisler proposed openstack/openstack-helm-images master: Hide the password in nagios error message  https://review.opendev.org/74828016:42
openstackgerritMerged openstack/openstack-helm-images master: Upgrade etcd to 3.4.3  https://review.opendev.org/74980316:55
*** guilhermesp has joined #openstack-helm17:11
openstackgerritRADHIKA PAI proposed openstack/openstack-helm-infra master: [update] Node problem detector path for conntrack  https://review.opendev.org/75106417:26
openstackgerritMerged openstack/openstack-helm-infra master: [ceph-osd] Allow logical disk labels to change for Ceph OSD disks  https://review.opendev.org/74516617:27
openstackgerritMerged openstack/openstack-helm-infra master: Implement missing security context template for daemonjob-controller  https://review.opendev.org/74804717:27
*** suryasingh has quit IRC17:35
*** thiagop has joined #openstack-helm17:52
*** thiagop has quit IRC17:54
*** thiagop has joined #openstack-helm17:54
openstackgerritMerged openstack/openstack-helm-images master: Alerta image  https://review.opendev.org/74658818:00
*** thiagop has quit IRC18:04
*** thiagop has joined #openstack-helm18:05
*** gmann is now known as gmann_afk18:11
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103518:28
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107518:32
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107518:33
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107518:34
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103518:44
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107518:44
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103518:49
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107518:52
openstackgerritMohammed Naser proposed openstack/openstack-helm master: WIP: add chart-testing linter  https://review.opendev.org/75108318:55
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-client] Update queries in wait_for_pgs function  https://review.opendev.org/75108418:56
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-client] Update queries in wait_for_pgs function  https://review.opendev.org/75108418:56
openstackgerritMohammed Naser proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107519:00
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103519:08
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103519:09
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103519:14
openstackgerritMeghan Heisler proposed openstack/openstack-helm-images master: Hide the password in nagios error message  https://review.opendev.org/74828019:28
*** yingjisun has joined #openstack-helm19:29
*** yingjisun has quit IRC19:43
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: Alerta: Add ingress  https://review.opendev.org/75066919:48
openstackgerritApurva Gokani proposed openstack/openstack-helm-infra master: adding archiving to postgres  https://review.opendev.org/74985019:50
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Optimize tls job to reduce build time  https://review.opendev.org/75109419:52
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-client] Update queries in wait_for_pgs function  https://review.opendev.org/75108419:54
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: Alerta: Add ingress  https://review.opendev.org/75066920:19
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107520:27
*** openstackgerrit has quit IRC20:36
*** thiagop has quit IRC20:55
*** outbrito has joined #openstack-helm20:55
*** outbrito has quit IRC21:11
*** openstackgerrit has joined #openstack-helm21:54
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-client] Update queries in wait_for_pgs function  https://review.opendev.org/75108421:54
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103522:12
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Optimize tls job to reduce build time  https://review.opendev.org/75109422:13
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Add force cron jobs test run  https://review.opendev.org/75103522:15
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Optimize tls job to reduce build time  https://review.opendev.org/75109422:15
openstackgerritOleksandr Kozachenko proposed openstack/openstack-helm-infra master: Run chart-testing on all charts  https://review.opendev.org/75107522:31
openstackgerritSophie Huang proposed openstack/openstack-helm master: [Cinder] Configuration for external ceph backend  https://review.opendev.org/75052022:31
*** sreejithp has quit IRC22:38
*** gmann_afk is now known as gmann22:49
openstackgerritDmitrii Kabanov proposed openstack/openstack-helm-infra master: [WIP] Add Ceph CSI plugin  https://review.opendev.org/75084423:32

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