*** ChanServ changes topic to "https://github.com/openstack/openstack-helm || Launchpad: https://launchpad.net/openstack-helm || General Review Dashboard: https://review.openstack.org/#/q/project:%255Eopenstack/openstack-helm.*+status:open,n,z" | 00:37 | |
-openstackstatus- NOTICE: The Gerrit service at review.opendev.org is back up and running; for outage details see analysis here: http://lists.opendev.org/pipermail/service-announce/2020-October/000011.html | 00:37 | |
*** sreejithp has joined #openstack-helm | 00:56 | |
*** sreejithp has quit IRC | 01:03 | |
*** sreejithp has joined #openstack-helm | 01:20 | |
*** sreejithp has quit IRC | 01:28 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-helm | 04:33 | |
*** sreejithp has joined #openstack-helm | 05:24 | |
*** sreejithp has quit IRC | 05:28 | |
*** vsaienk0 has joined #openstack-helm | 06:32 | |
*** vsaienk0 has quit IRC | 07:10 | |
*** vsaienk0 has joined #openstack-helm | 07:14 | |
*** sreejithp has joined #openstack-helm | 07:25 | |
*** sreejithp has quit IRC | 07:31 | |
*** openstackgerrit has quit IRC | 07:38 | |
*** roman_g has joined #openstack-helm | 08:11 | |
*** vsaienk0 has quit IRC | 08:16 | |
*** vsaienk0 has joined #openstack-helm | 08:29 | |
*** vsaienk0 has quit IRC | 09:04 | |
*** vsaienk0 has joined #openstack-helm | 09:07 | |
*** sreejithp has joined #openstack-helm | 09:27 | |
*** sreejithp has quit IRC | 09:32 | |
*** openstackgerrit has joined #openstack-helm | 10:21 | |
openstackgerrit | Gayathri Devi proposed openstack/openstack-helm master: Add "TMOUT" parameter in the pods https://review.opendev.org/754942 | 10:21 |
---|---|---|
*** sugaar has quit IRC | 10:24 | |
*** sugaar has joined #openstack-helm | 10:24 | |
openstackgerrit | David Smith proposed openstack/openstack-helm-infra master: Fix spacing inconsistencies with flags https://review.opendev.org/759017 | 12:12 |
*** andrii_ostapenko has quit IRC | 12:33 | |
*** sangeet has quit IRC | 12:41 | |
*** jamesgu has quit IRC | 13:33 | |
openstackgerrit | David Smith proposed openstack/openstack-helm-infra master: Fix spacing inconsistencies with flags https://review.opendev.org/759017 | 13:44 |
*** Reddy has joined #openstack-helm | 13:47 | |
openstackgerrit | Gayathri Devi proposed openstack/openstack-helm master: Add "TMOUT" parameter in the pods https://review.opendev.org/754942 | 13:48 |
*** reddy1 has joined #openstack-helm | 13:49 | |
*** Reddy has quit IRC | 13:52 | |
*** NafH has joined #openstack-helm | 14:05 | |
openstackgerrit | Brian Wickersham proposed openstack/openstack-helm-infra master: [ceph-osd] Update post apply job https://review.opendev.org/758930 | 14:28 |
*** NafH has quit IRC | 14:39 | |
*** NafH has joined #openstack-helm | 14:43 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 15:09 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 15:09 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 15:09 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759071 | 15:09 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 15:41 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 15:41 |
openstackgerrit | Merged openstack/openstack-helm-infra master: chore(pkg): updates the chart packaging https://review.opendev.org/758644 | 15:43 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: Remove the overrides which are not required https://review.opendev.org/759092 | 15:55 |
openstackgerrit | Tin Lam proposed openstack/openstack-helm master: fix(neutron): fixes tls issue https://review.opendev.org/758919 | 16:03 |
*** portdirect has quit IRC | 16:19 | |
*** portdirect has joined #openstack-helm | 16:20 | |
*** ChanServ sets mode: +o portdirect | 16:20 | |
*** portdirect has quit IRC | 16:21 | |
*** portdirect has joined #openstack-helm | 16:21 | |
*** ChanServ sets mode: +o portdirect | 16:21 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759071 | 16:44 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 16:44 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 16:44 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 16:44 |
*** vsaienk0 has quit IRC | 16:48 | |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 17:02 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 17:03 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 17:14 |
openstackgerrit | Cliff Parsons proposed openstack/openstack-helm-infra master: Add quotes around passwords for backup user credentials https://review.opendev.org/759109 | 17:16 |
openstackgerrit | Cliff Parsons proposed openstack/openstack-helm-infra master: Add quotes around passwords for backup user credentials https://review.opendev.org/759109 | 17:21 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 17:28 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 17:28 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 17:28 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: Remove the overrides which are not required https://review.opendev.org/759092 | 17:40 |
openstackgerrit | Merged openstack/openstack-helm master: Add blank access_rules file https://review.opendev.org/758466 | 17:40 |
openstackgerrit | Rahul Khiyani proposed openstack/openstack-helm-infra master: Add pod/contianer security context template to create_db.yaml https://review.opendev.org/759092 | 17:46 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-osd] Update post apply job https://review.opendev.org/758930 | 17:48 |
openstackgerrit | Merged openstack/openstack-helm-infra master: Fix ks-user script case matching for domain https://review.opendev.org/758501 | 17:51 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 17:51 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 17:53 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 17:53 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 17:53 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 17:56 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 17:56 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 17:56 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: zuul.d: Adding multinode job https://review.opendev.org/757915 | 18:06 |
openstackgerrit | Cliff Parsons proposed openstack/openstack-helm-infra master: Add quotes around passwords for backup user credentials https://review.opendev.org/759109 | 18:15 |
*** NafH has quit IRC | 18:19 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 18:24 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 18:24 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 18:24 |
*** jamesgu has joined #openstack-helm | 18:52 | |
*** NafH has joined #openstack-helm | 19:17 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759129 | 19:49 |
mnaser | portdirect, gagehugo: do you internally use different values for initial deploy vs upgrade deploys in terms of manifests? for example, at my side here, keystone fernet keys are all disappearing when i redeploy because the data key is empty in our manifest (as compared as when it gets filled in after the init stuff ran) | 19:55 |
mnaser | i.e. on redeploy, this seems to get wiped -- https://github.com/openstack/openstack-helm/blob/master/keystone/templates/secret-fernet-keys.yaml | 19:57 |
gagehugo | I mean we don't redeploy everything | 19:58 |
mnaser | well, sorry, for accuracy, i'm talking about a helm install --upgrade case | 19:58 |
gagehugo | specifically for fernet-keys, the keystone libraries should do a re-auth | 19:59 |
openstackgerrit | Dmitrii Kabanov proposed openstack/openstack-helm-infra master: [ceph-osd] Update post apply job https://review.opendev.org/758930 | 19:59 |
mnaser | i think credential keys is a little worry-some, because you could potentially lose credential data that cant be uenencrypted again | 20:00 |
gagehugo | yes | 20:00 |
gagehugo | I wasn't aware that doing --upgrade wiped all the secrets though | 20:01 |
mnaser | in my case it is, i'm using helmfile right now with the charts, oddly enough helmfile diff doesn't show a change but when i run helmfile apply it most certainly disappears | 20:01 |
mnaser | i wonder if creating those secrets could be tagged as a helm hook like on install | 20:02 |
mnaser | "Under the covers, Helmfile executes helm upgrade --install for each release declared in the manifest" | 20:03 |
mnaser | https://github.com/roboll/helmfile/blob/f5d098876a95cdc7f435623d7c10df1d46fdf036/pkg/helmexec/exec.go#L184 so seems to be `helm upgrade --install --reset-values <name> <chart>` | 20:04 |
mnaser | perhaps we need `"helm.sh/hook": pre-install` | 20:07 |
gagehugo | --reset-values seems sketchy | 20:08 |
mnaser | "In Helm 2, resources with hook annotations were unmanaged. In Helm 3, the owner reference is set on those, and they will be removed when the chart is deleted." | 20:08 |
mnaser | we use helm 3 so maybe it's got to do with that... | 20:09 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-helm master: [keystone] Fix fernet secret reset https://review.opendev.org/759136 | 20:14 |
gagehugo | that is possible | 20:16 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 20:17 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 20:17 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759129 | 20:19 |
openstackgerrit | Merged openstack/openstack-helm master: fix(neutron): fixes tls issue https://review.opendev.org/758919 | 20:56 |
*** NafH has quit IRC | 21:20 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759129 | 21:26 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: DNM https://review.opendev.org/759129 | 21:53 |
*** roman_g has quit IRC | 22:00 | |
openstackgerrit | Nafiz Haider proposed openstack/openstack-helm master: [WIP] [NOVA] Add WSGISocketPrefix to Nova Conf Override https://review.opendev.org/759148 | 22:01 |
*** reddy1 has quit IRC | 22:08 | |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 22:12 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 22:12 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: zuul.d: Adding multinode job https://review.opendev.org/757915 | 22:37 |
openstackgerrit | Sangeet Gupta proposed openstack/openstack-helm master: zuul.d: Adding multinode job https://review.opendev.org/757915 | 22:42 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 22:50 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 22:50 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 22:50 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 22:50 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Adapt code due to airshipctl changes https://review.opendev.org/758842 | 22:51 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add Docker registry https://review.opendev.org/757601 | 22:52 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 22:52 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 22:53 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 22:56 |
openstackgerrit | Drew Walters proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 22:56 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add helm-chart-collator https://review.opendev.org/757639 | 23:17 |
openstackgerrit | Andrii Ostapenko proposed openstack/openstack-helm-deployments master: Add memcached function and OpenStack composite https://review.opendev.org/756401 | 23:18 |
openstackgerrit | Nafiz Haider proposed openstack/openstack-helm master: [WIP] [NOVA] Add WSGISocketPrefix to Nova Conf Override https://review.opendev.org/759148 | 23:24 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!