| *** rcernin has quit IRC | 01:37 | |
| *** rcernin has joined #puppet-openstack | 01:39 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-barbican master: Missing supported_os fact https://review.opendev.org/c/openstack/puppet-barbican/+/765032 | 01:51 |
|---|---|---|
| openstackgerrit | zhongshengping proposed openstack/puppet-barbican master: Missing supported_os fact https://review.opendev.org/c/openstack/puppet-barbican/+/765032 | 01:59 |
| *** ysandeep|away is now known as ysandeep | 02:18 | |
| *** rcernin has quit IRC | 02:36 | |
| *** iurygregory|pto has quit IRC | 03:01 | |
| *** rcernin has joined #puppet-openstack | 03:06 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Enable Ceph in CentOS8 scenario001/004 job https://review.opendev.org/c/openstack/puppet-openstack-integration/+/763385 | 03:14 |
| *** ykarel has joined #puppet-openstack | 05:23 | |
| *** rcernin has quit IRC | 05:43 | |
| *** rcernin has joined #puppet-openstack | 05:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/c/openstack/puppet-openstack-integration/+/765065 | 07:02 |
| *** rcernin has quit IRC | 07:38 | |
| *** rcernin has joined #puppet-openstack | 07:44 | |
| *** rcernin has quit IRC | 07:48 | |
| *** rcernin has joined #puppet-openstack | 08:01 | |
| *** rcernin has quit IRC | 08:16 | |
| *** rpittau|afk is now known as rpittau | 08:28 | |
| *** rcernin has joined #puppet-openstack | 08:32 | |
| *** rcernin has quit IRC | 08:46 | |
| *** tkajinam has quit IRC | 08:49 | |
| *** jpena|off is now known as jpena | 08:57 | |
| *** obre has joined #puppet-openstack | 08:59 | |
| *** tosky has joined #puppet-openstack | 09:05 | |
| *** derekh has joined #puppet-openstack | 09:07 | |
| *** rcernin has joined #puppet-openstack | 09:42 | |
| *** rcernin has quit IRC | 09:42 | |
| *** gfidente has joined #puppet-openstack | 10:21 | |
| LarsErikP | hi | 10:26 |
| LarsErikP | why is this set to ussuri in stable/victoria? https://github.com/openstack/puppet-openstack_extras/blob/stable/victoria/manifests/repo/redhat/params.pp#L8 | 10:26 |
| LarsErikP | actually, here as well: https://github.com/openstack/puppet-openstack_extras/blob/stable/victoria/manifests/repo/debian/params.pp#L7 | 10:27 |
| *** ykarel_ has joined #puppet-openstack | 10:27 | |
| *** ykarel_ is now known as ykarel|lunch | 10:28 | |
| *** ykarel has quit IRC | 10:29 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-ceilometer master: [WiP] Add TLS options to oslo.cache https://review.opendev.org/c/openstack/puppet-ceilometer/+/765100 | 10:35 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-nova master: [WiP] Add TLS options to oslo.cache https://review.opendev.org/c/openstack/puppet-nova/+/764763 | 10:41 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-ironic master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-ironic/+/765103 | 10:45 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-aodh master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-aodh/+/765107 | 10:53 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-manila master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-manila/+/765111 | 10:58 |
| *** ysandeep is now known as ysandeep|afk | 11:04 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-octavia master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-octavia/+/765113 | 11:08 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-cinder master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-cinder/+/765115 | 11:14 |
| *** dtantsur|afk is now known as dtantsur | 11:19 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-glance master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-glance/+/765118 | 11:19 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-gnocchi master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-gnocchi/+/765119 | 11:22 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-barbican master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-barbican/+/765120 | 11:25 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-heat master: [WiP] Add TLS options to oslo.cache https://review.opendev.org/c/openstack/puppet-heat/+/764764 | 11:28 |
| *** ykarel|lunch is now known as ykarel | 11:30 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-neutron master: [WiP] Add TLS options to keystone authtoken/oslo.cache https://review.opendev.org/c/openstack/puppet-neutron/+/765121 | 11:35 |
| *** ykarel_ has joined #puppet-openstack | 11:35 | |
| *** ykarel has quit IRC | 11:38 | |
| LarsErikP | tobias-urdin: do you know? ^ (about the openstack_extras) | 11:45 |
| *** iurygregory has joined #puppet-openstack | 12:03 | |
| *** dtantsur is now known as dtantsur|brb | 12:31 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Do not use pick() for dhcp_domain as it can be empty string https://review.opendev.org/c/openstack/puppet-nova/+/765131 | 12:35 |
| *** jpena is now known as jpena|lunch | 12:36 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Handle opendev.org sources in copy_logs.sh https://review.opendev.org/c/openstack/puppet-openstack-integration/+/765135 | 13:15 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-oslo master: Add TLS options to oslo.cache https://review.opendev.org/c/openstack/puppet-oslo/+/761604 | 13:21 |
| *** ykarel_ is now known as ykarel | 13:30 | |
| *** ysandeep|afk is now known as ysandeep | 13:30 | |
| *** dtantsur|brb is now known as dtantsur | 13:46 | |
| *** lbragstad has quit IRC | 13:50 | |
| *** jpena|lunch is now known as jpena | 13:51 | |
| *** lbragstad has joined #puppet-openstack | 13:55 | |
| *** ysandeep is now known as ysandeep|away | 15:23 | |
| *** redrobot has quit IRC | 16:11 | |
| mwhahaha | owalsh: https://review.opendev.org/c/openstack/puppet-nova/+/765131 isn't working? i thought pick would pick '' tho | 16:29 |
| owalsh | mwhahaha: docs say pick will not choose an empty string https://forge.puppet.com/modules/puppetlabs/stdlib/reference#pick-1 | 16:30 |
| owalsh | "first value in a list of values that is not undefined or an empty string" | 16:30 |
| mwhahaha | thats annoying | 16:30 |
| owalsh | wasn't everything like that back in puppet 3.x | 16:31 |
| owalsh | mwhahaha: hrmm, https://ce99744a61fc72e05a32-bd54a9781d6bc663ca8af93b25749dfd.ssl.cf2.rackcdn.com/765131/1/check/puppet-nova-tripleo-standalone/193c98b/logs/undercloud/var/log/containers/stdouts/container-puppet-nova_metadata.log | 16:38 |
| owalsh | mwhahaha: I don't see dhcp_domain set there | 16:38 |
| mwhahaha | if it's the default in teh file it won't be updated | 16:38 |
| mwhahaha | also is it '' or "''" | 16:39 |
| mwhahaha | 2020-12-02T13:55:02.487758452+00:00 stderr F <13>Dec 2 13:55:02 puppet-user: Warning: Scope(Class[Nova::Metadata]): dhcp_domain in nova::metadata is deprecated, use nova::dhcp_domain instead. | 16:39 |
| mwhahaha | doesn't look like it picked up the change? | 16:39 |
| owalsh | that's the logs for https://review.opendev.org/c/openstack/puppet-nova/+/765131 | 16:39 |
| mwhahaha | owalsh: oh are the patches backwards | 16:39 |
| mwhahaha | do you need puppet-nova to depends on tht? | 16:40 |
| owalsh | mwhahaha: would be more correct, but CI doens't care | 16:40 |
| owalsh | mwhahaha: or not depend on each other | 16:41 |
| mwhahaha | well the standalone job will have the old value | 16:41 |
| mwhahaha | my thought is that we dont' include ::nova in that execution | 16:41 |
| mwhahaha | because the warning is generated elsewhere | 16:41 |
| owalsh | mwhahaha: warning comes from nova::metadata, but I think we are not setting dhcp_domain because nova is included before nova::metadata in t-h-t | 16:42 |
| owalsh | s/t-h-t/puppet-tripleo/ | 16:42 |
| mwhahaha | we don't include nova | 16:43 |
| mwhahaha | so that puppet execution include the tripleo::profile::base::nova::metadata | 16:44 |
| mwhahaha | we nm i see it | 16:44 |
| mwhahaha | sin't the default value ''? | 16:45 |
| mwhahaha | eh this stuff makes my brain hurt | 16:45 |
| mwhahaha | my only assuption is that '' ends but not actually being honored by the provider | 16:46 |
| mwhahaha | and gets interpreted as blank so it doesn't set it | 16:46 |
| mwhahaha | you might have to set "''" to get an empty value | 16:46 |
| owalsh | mwhahaha: pretty sure it works (I'll check) | 16:46 |
| owalsh | I think it's just the include order in tripleo, which i could change but easier to just stop using the old hiera in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/765132 | 16:47 |
| mwhahaha | it could also be a load order issue | 16:47 |
| owalsh | mwhahaha: also the include order only matters for nova-metadata, for nova-compute we don't (and can't) include nova::metadata, so the t-h-t patch is required | 16:48 |
| mwhahaha | i think it's a load order problem | 16:48 |
| mwhahaha | yea we need the tht patch | 16:48 |
| mwhahaha | but we need the puppet-nova one or the pick will fail | 16:48 |
| owalsh | ack +1 | 16:49 |
| mwhahaha | i don't think $::nova::metadata::dhcp_domain is set when we class {'nova': } | 16:49 |
| mwhahaha | because puppet evaulates it top down i believe so we don't get nova::metadata until after nova is included already | 16:49 |
| owalsh | +1 exactly what I figure | 16:50 |
| *** valleedelisle has joined #puppet-openstack | 16:52 | |
| *** Guest18921 has joined #puppet-openstack | 16:58 | |
| *** Guest18921 is now known as redrobot | 17:01 | |
| *** ykarel is now known as ykarel|away | 17:04 | |
| *** rpittau is now known as rpittau|afk | 17:04 | |
| *** ykarel|away has quit IRC | 17:09 | |
| *** ykarel|away has joined #puppet-openstack | 17:17 | |
| *** ykarel|away has quit IRC | 17:22 | |
| *** camelCaser has joined #puppet-openstack | 17:48 | |
| *** jpena is now known as jpena|off | 18:02 | |
| *** derekh has quit IRC | 18:14 | |
| *** gfidente is now known as gfidente|afk | 18:40 | |
| *** dtantsur is now known as dtantsur|afk | 18:47 | |
| *** iurygregory has quit IRC | 19:29 | |
| openstackgerrit | David Vallee Delisle proposed openstack/puppet-nova master: Removing scheduler_default_filters https://review.opendev.org/c/openstack/puppet-nova/+/765199 | 19:59 |
| openstackgerrit | David Vallee Delisle proposed openstack/puppet-nova master: Removing scheduler_default_filters https://review.opendev.org/c/openstack/puppet-nova/+/765199 | 20:08 |
| *** iurygregory has joined #puppet-openstack | 21:28 | |
| *** yann-kaelig has joined #puppet-openstack | 22:16 | |
| *** yann-kaelig has quit IRC | 22:21 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Allow both DEFAULT/dhcp_domain and api/dhcp_domain to be set https://review.opendev.org/c/openstack/puppet-nova/+/765131 | 22:32 |
| *** rcernin has joined #puppet-openstack | 22:34 | |
| *** tkajinam has joined #puppet-openstack | 23:00 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!