| *** ntpttr_laptop has quit IRC | 00:12 | |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-modulesync-configs master: Update test-requirements reno release to 2.5.1 https://review.openstack.org/509665 | 00:35 |
|---|---|---|
| *** openstackstatus has quit IRC | 01:14 | |
| *** openstackstatus has joined #puppet-openstack | 01:16 | |
| *** ChanServ sets mode: +v openstackstatus | 01:16 | |
| *** zigo has quit IRC | 01:27 | |
| *** zigo has joined #puppet-openstack | 01:31 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** markvoelker has joined #puppet-openstack | 01:52 | |
| *** ilbot3 has joined #puppet-openstack | 01:55 | |
| *** afisher has quit IRC | 02:04 | |
| *** afisher has joined #puppet-openstack | 02:07 | |
| *** mtreinish has quit IRC | 02:08 | |
| *** jistr has quit IRC | 02:08 | |
| *** jistr has joined #puppet-openstack | 02:10 | |
| *** mtreinish has joined #puppet-openstack | 02:10 | |
| *** higgins` has joined #puppet-openstack | 02:11 | |
| *** higgins has quit IRC | 02:12 | |
| *** mgagne has quit IRC | 02:23 | |
| *** mgagne has joined #puppet-openstack | 02:24 | |
| *** mgagne is now known as Guest66098 | 02:24 | |
| *** markvoelker has quit IRC | 02:27 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Install python-cinderclient in integration https://review.openstack.org/508169 | 02:58 |
| openstackgerrit | Merged openstack/puppet-nova master: Remove Class nova::rabbitmq https://review.openstack.org/508672 | 02:58 |
| openstackgerrit | Merged openstack/puppet-oslo master: Add use_journal option for logging configuration https://review.openstack.org/508497 | 03:09 |
| *** morazi has quit IRC | 03:10 | |
| *** markvoelker has joined #puppet-openstack | 03:24 | |
| *** udesale has joined #puppet-openstack | 03:38 | |
| *** udesale has quit IRC | 03:38 | |
| *** udesale has joined #puppet-openstack | 03:42 | |
| *** markvoelker has quit IRC | 03:57 | |
| *** ykarel has joined #puppet-openstack | 04:02 | |
| *** bnemec has quit IRC | 04:16 | |
| *** jaosorior has joined #puppet-openstack | 04:21 | |
| *** asilenkov has quit IRC | 04:22 | |
| *** asilenkov has joined #puppet-openstack | 04:23 | |
| *** skramaja has joined #puppet-openstack | 04:48 | |
| *** markvoelker has joined #puppet-openstack | 04:54 | |
| *** udesale has quit IRC | 04:55 | |
| *** cgfbee has joined #puppet-openstack | 05:05 | |
| *** ratailor has joined #puppet-openstack | 05:16 | |
| *** udesale has joined #puppet-openstack | 05:18 | |
| *** yprokule has joined #puppet-openstack | 05:24 | |
| *** markvoelker has quit IRC | 05:27 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove revocation_cache_time parameter https://review.openstack.org/509971 | 05:27 |
| *** ykarel_ has joined #puppet-openstack | 05:38 | |
| *** ykarel has quit IRC | 05:41 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove useless method file_path https://review.openstack.org/509975 | 05:54 |
| *** spectr has joined #puppet-openstack | 05:56 | |
| *** jaganathan has joined #puppet-openstack | 05:57 | |
| openstackgerrit | yatin proposed openstack/puppet-zaqar master: Set *_domain_name vars to 'Default' by default https://review.openstack.org/509977 | 06:01 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc https://review.openstack.org/509978 | 06:16 |
| *** clayton has quit IRC | 06:16 | |
| *** ykarel__ has joined #puppet-openstack | 06:16 | |
| *** rcernin has joined #puppet-openstack | 06:17 | |
| *** clayton has joined #puppet-openstack | 06:18 | |
| *** ykarel_ has quit IRC | 06:20 | |
| *** markvoelker has joined #puppet-openstack | 06:24 | |
| *** karthiks has joined #puppet-openstack | 06:28 | |
| *** udesale has quit IRC | 06:31 | |
| *** udesale has joined #puppet-openstack | 06:33 | |
| *** dtantsur|afk has quit IRC | 06:40 | |
| *** dtantsur has joined #puppet-openstack | 06:40 | |
| *** rcernin has quit IRC | 06:41 | |
| *** rcernin has joined #puppet-openstack | 06:43 | |
| *** mandre is now known as mandre_afk | 06:51 | |
| *** markvoelker has quit IRC | 06:58 | |
| *** jtomasek has joined #puppet-openstack | 06:59 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-nova stable/ocata: Ensure that policy file exist https://review.openstack.org/509710 | 07:03 |
| openstackgerrit | yatin proposed openstack/puppet-ec2api master: Set *_domain_name vars to 'Default' by default https://review.openstack.org/509988 | 07:07 |
| *** agurenko has joined #puppet-openstack | 07:14 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-nova stable/ocata: Ensure that policy file exist https://review.openstack.org/509710 | 07:14 |
| *** amoralej|off is now known as amoralej | 07:18 | |
| *** ccamacho has joined #puppet-openstack | 07:20 | |
| *** gfidente has joined #puppet-openstack | 07:21 | |
| *** tesseract has joined #puppet-openstack | 07:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/509998 | 07:39 |
| *** ykarel__ is now known as ykarel | 07:47 | |
| *** jpena|off is now known as jpena | 07:50 | |
| *** markvoelker has joined #puppet-openstack | 07:54 | |
| *** mandre_afk is now known as mandre | 08:17 | |
| *** markvoelker has quit IRC | 08:28 | |
| *** derekh has joined #puppet-openstack | 08:46 | |
| *** salmankhan has joined #puppet-openstack | 08:52 | |
| openstackgerrit | Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest https://review.openstack.org/509807 | 09:01 |
| openstackgerrit | Merged openstack/puppet-nova master: Accept empty strings for log_dir https://review.openstack.org/509105 | 09:09 |
| *** ykarel is now known as ykarel|lunch | 09:12 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 09:13 |
| *** markvoelker has joined #puppet-openstack | 09:25 | |
| *** spectr has quit IRC | 09:26 | |
| amoralej | jaosorior, re https://review.openstack.org/#/c/510022/ i can move it to keystone::resources::authtoken if it's better | 09:32 |
| *** spectr has joined #puppet-openstack | 09:34 | |
| *** spectr has quit IRC | 09:34 | |
| *** spectr has joined #puppet-openstack | 09:36 | |
| *** spectr has quit IRC | 09:36 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 09:36 |
| *** spectr has joined #puppet-openstack | 09:37 | |
| *** spectr has quit IRC | 09:38 | |
| *** spectr has joined #puppet-openstack | 09:40 | |
| jaosorior | amoralej: that would be good :) | 09:40 |
| *** spectr has quit IRC | 09:40 | |
| *** spectr has joined #puppet-openstack | 09:41 | |
| *** ykarel|lunch is now known as ykarel | 09:42 | |
| *** spectr has quit IRC | 09:43 | |
| *** spectr has joined #puppet-openstack | 09:43 | |
| *** yprokule has quit IRC | 09:47 | |
| *** yprokule_ has joined #puppet-openstack | 09:47 | |
| *** agurenko has quit IRC | 09:47 | |
| *** yprokule_ is now known as yprokule | 09:48 | |
| amoralej | jaosorior, common pattern is to create a class <service>::keystone::authtoken to configure that and include it from init.pp | 09:48 |
| amoralej | however i don't want to break backwards compatibility | 09:48 |
| amoralej | so i'm calling keystone::resource::authtoken from init.pp directly, is that ok? | 09:49 |
| amoralej | is there a better way? | 09:49 |
| *** chem has quit IRC | 09:50 | |
| *** chem has joined #puppet-openstack | 09:51 | |
| *** spectr has quit IRC | 09:56 | |
| *** spectr has joined #puppet-openstack | 09:57 | |
| *** spectr has quit IRC | 09:57 | |
| *** spectr has joined #puppet-openstack | 09:58 | |
| *** markvoelker has quit IRC | 09:58 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ceilometer master: Accept empty strings for log_dir https://review.openstack.org/510043 | 10:09 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-cinder master: Accept empty strings for log_dir https://review.openstack.org/510044 | 10:09 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-glance master: Accept empty strings for log_dir https://review.openstack.org/510045 | 10:09 |
| jaosorior | amoralej: that's fine for me | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-heat master: Accept empty strings for log_dir https://review.openstack.org/510046 | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir https://review.openstack.org/510047 | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Accept empty strings for log_dir https://review.openstack.org/510048 | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-manila master: Accept empty strings for log_dir https://review.openstack.org/510049 | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-mistral master: Accept empty strings for log_dir https://review.openstack.org/510050 | 10:10 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-neutron master: Accept empty strings for log_dir https://review.openstack.org/510051 | 10:10 |
| *** ykarel_ has joined #puppet-openstack | 10:16 | |
| *** ykarel has quit IRC | 10:18 | |
| *** ykarel__ has joined #puppet-openstack | 10:21 | |
| *** ykarel__ is now known as ykarel | 10:21 | |
| *** ykarel_ has quit IRC | 10:22 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 10:28 |
| *** salmankhan has quit IRC | 10:47 | |
| *** markvoelker has joined #puppet-openstack | 10:56 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 10:57 |
| *** salmankhan has joined #puppet-openstack | 11:10 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 11:14 |
| openstackgerrit | Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest https://review.openstack.org/509807 | 11:15 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-barbican master: Allow setting log files for apache vhost https://review.openstack.org/510064 | 11:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ceilometer master: Allow setting log files for apache vhost https://review.openstack.org/510065 | 11:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-cinder master: Allow setting log files for apache vhost https://review.openstack.org/510066 | 11:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-gnocchi master: Allow setting log files for apache vhost https://review.openstack.org/510067 | 11:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-heat master: Allow setting log files for apache vhost https://review.openstack.org/510068 | 11:23 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost https://review.openstack.org/510069 | 11:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-mistral master: Allow setting log files for apache vhost https://review.openstack.org/510070 | 11:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-neutron master: Allow setting log files for apache vhost https://review.openstack.org/510071 | 11:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-panko master: Allow setting log files for apache vhost https://review.openstack.org/510072 | 11:24 |
| amoralej | mwhahaha, mnaser iurygregory could you review https://review.openstack.org/#/q/topic:keystone_v3+status:open+project:%255Eopenstack/puppet.* when you have a chance? | 11:24 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost https://review.openstack.org/510073 | 11:24 |
| amoralej | we need it to unblock promotions | 11:24 |
| *** amoralej is now known as amoralej|away | 11:25 | |
| *** markvoelker has quit IRC | 11:29 | |
| *** morazi has joined #puppet-openstack | 11:33 | |
| *** ansmith has quit IRC | 11:56 | |
| *** ansmith_ has quit IRC | 11:57 | |
| *** spectr has quit IRC | 12:02 | |
| *** spectr has joined #puppet-openstack | 12:03 | |
| *** dprince has joined #puppet-openstack | 12:03 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove Class nova::wsgi::apache https://review.openstack.org/510081 | 12:05 |
| *** jpena is now known as jpena|lunch | 12:05 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove Class nova::wsgi::apache https://review.openstack.org/510081 | 12:06 |
| *** trown|outtypewww is now known as trown | 12:10 | |
| *** salmankhan has quit IRC | 12:15 | |
| *** markvoelker has joined #puppet-openstack | 12:26 | |
| *** spectr has quit IRC | 12:28 | |
| *** markvoelker has quit IRC | 12:31 | |
| *** markvoelker has joined #puppet-openstack | 12:31 | |
| *** ratailor has quit IRC | 12:38 | |
| *** spectr has joined #puppet-openstack | 12:39 | |
| iurygregory | amoralej|away, sure | 12:44 |
| *** ansmith has joined #puppet-openstack | 12:45 | |
| *** ansmith_ has joined #puppet-openstack | 12:46 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon master: Add parameter to configure instance defaults https://review.openstack.org/510098 | 12:57 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings https://review.openstack.org/509021 | 12:57 |
| *** jpena|lunch is now known as jpena | 13:00 | |
| *** salmankhan has joined #puppet-openstack | 13:04 | |
| *** udesale has quit IRC | 13:05 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-barbican master: Configure *_domain_name to Default by default https://review.openstack.org/510102 | 13:06 |
| *** bnemec has joined #puppet-openstack | 13:12 | |
| *** skramaja has quit IRC | 13:18 | |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-ceilometer master: Configure *_domain_name to Default by default https://review.openstack.org/510114 | 13:33 |
| *** pradk has joined #puppet-openstack | 13:34 | |
| *** spectr has quit IRC | 13:35 | |
| *** spectr has joined #puppet-openstack | 13:36 | |
| *** trown is now known as trown|brb | 13:37 | |
| jaosorior | dmsimard: a release note for accepting empty string now? | 13:38 |
| jaosorior | dtantsur: not dmsimard | 13:38 |
| dtantsur | jaosorior: that's a feature right? now you can enable logging to stderr | 13:38 |
| jaosorior | I don't really see it as a feature to be honest | 13:39 |
| jaosorior | dtantsur: more like, it has always been possible, but it wasn't possible cause of the way the code was made | 13:39 |
| dtantsur | jaosorior: then it's a bug fix, right? both deserve a release note | 13:39 |
| jaosorior | .. | 13:40 |
| jaosorior | alright | 13:40 |
| *** ykarel is now known as ykarel|afk | 13:41 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost https://review.openstack.org/510069 | 13:43 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-cinder master: Configure *_domain_name to Default by default https://review.openstack.org/510119 | 13:43 |
| *** shardy has quit IRC | 13:44 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir https://review.openstack.org/510047 | 13:45 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost https://review.openstack.org/510073 | 13:46 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost https://review.openstack.org/510069 | 13:47 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-glance master: Configure *_domain_name to Default by default https://review.openstack.org/510121 | 13:50 |
| *** trown|brb is now known as trown | 13:55 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir https://review.openstack.org/510047 | 13:56 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-glance master: Configure *_domain_name to Default by default https://review.openstack.org/510121 | 13:57 |
| dtantsur | jaosorior: +2 to both, thanks | 13:58 |
| openstackgerrit | Bob Fournier proposed openstack/puppet-ironic stable/pike: Increase Ironic inspector dnsmasq lease time https://review.openstack.org/510125 | 13:59 |
| jaosorior | dtantsur: thanks for the reviews | 14:00 |
| dtantsur | np | 14:00 |
| *** ratailor has joined #puppet-openstack | 14:00 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Update reno to 2.5.1 https://review.openstack.org/510128 | 14:04 |
| *** archit has joined #puppet-openstack | 14:09 | |
| *** bnemec is now known as beekneemech | 14:11 | |
| *** amoralej|away is now known as amoralej | 14:11 | |
| *** spectr has quit IRC | 14:11 | |
| openstackgerrit | Merged openstack/puppet-murano master: Add parameters for keystone API v3 https://review.openstack.org/510022 | 14:14 |
| mnaser | jaosorior thanks for your patches for logging :D | 14:14 |
| jaosorior | mnaser: thanks for your reviews | 14:15 |
| *** ykarel|afk has quit IRC | 14:17 | |
| mnaser | :) | 14:19 |
| amoralej | mwhahaha, iurygregory thanks for all those reviews | 14:21 |
| iurygregory | amoralej, yw, did i missed something? | 14:21 |
| iurygregory | many tabs open here XD | 14:21 |
| amoralej | no, i mean, the keysone_v3 reviews | 14:21 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-ironic master: Configure *_domain_name to Default by default https://review.openstack.org/510134 | 14:23 |
| mnaser | github has so many issues :( | 14:26 |
| mnaser | i wonder if cloning over http might be more sane | 14:26 |
| mnaser | err, i mean over ssh | 14:26 |
| mnaser | reading online, folks seem to report that ssh is better | 14:29 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-manila master: Configure *_domain_name to Default by default https://review.openstack.org/510139 | 14:34 |
| mjblack | mwhahaha: got a question, what am I missing in my spec changes in https://review.openstack.org/#/c/510098 that is causing the errors | 14:39 |
| *** beekneemech has quit IRC | 14:40 | |
| mwhahaha | mjblack: not sure i'll see when tests come back but i noticed an issue with the verify_concat_fragment_contents function | 14:40 |
| mwhahaha | mjblack: if one of the lines is duplicated in that array it won't match | 14:40 |
| mjblack | oh | 14:41 |
| mwhahaha | ie "]" only matches once | 14:41 |
| mwhahaha | or "}" rather | 14:41 |
| mjblack | ah ok, now I understand | 14:41 |
| *** bnemec has joined #puppet-openstack | 14:42 | |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-octavia master: Configure *_domain_name to Default by default https://review.openstack.org/510141 | 14:42 |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove some useless parameters https://review.openstack.org/510143 | 14:45 |
| jaosorior | EmilienM: could you check this out https://review.openstack.org/#/c/509127/ ? | 14:48 |
| *** mwhahaha changes topic to "CI Status: YELLOWish - scenarios 1/4 failing (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/" | 14:50 | |
| *** ntpttr_laptop has joined #puppet-openstack | 14:50 | |
| mjblack | mwhahaha: not seeing any duplicates | 14:52 |
| mwhahaha | yea that bit is so hard to debug | 14:53 |
| mwhahaha | i'll take a look later when the results post | 14:53 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-panko master: Configure *_domain_name to Default by default https://review.openstack.org/510147 | 14:54 |
| EmilienM | jaosorior: done | 14:57 |
| openstackgerrit | Harry Rybacki proposed openstack/puppet-sahara master: Configure *_domain_name to Default by default https://review.openstack.org/510150 | 14:58 |
| *** ntpttr_laptop has quit IRC | 14:58 | |
| mjblack | mwhahaha: I found it, forgot to set a parameter in the hash | 14:59 |
| mwhahaha | oops | 14:59 |
| mjblack | heh, well at least it wasnt something major | 14:59 |
| *** dgurtner has joined #puppet-openstack | 14:59 | |
| *** dgurtner has quit IRC | 14:59 | |
| *** dgurtner has joined #puppet-openstack | 14:59 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon master: Add parameter to configure instance defaults https://review.openstack.org/510098 | 15:01 |
| *** dgurtner has quit IRC | 15:02 | |
| *** spectr has joined #puppet-openstack | 15:04 | |
| *** spectr has quit IRC | 15:04 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough https://review.openstack.org/510151 | 15:06 |
| *** bnemec has quit IRC | 15:08 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough https://review.openstack.org/510151 | 15:08 |
| *** ratailor has quit IRC | 15:09 | |
| *** bnemec has joined #puppet-openstack | 15:09 | |
| *** rcernin has quit IRC | 15:19 | |
| *** jaosorior has quit IRC | 15:24 | |
| *** andreww has joined #puppet-openstack | 15:25 | |
| *** xarses_ has quit IRC | 15:28 | |
| openstackgerrit | Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest https://review.openstack.org/509807 | 15:36 |
| *** dtantsur is now known as dtantsur|afk | 15:44 | |
| *** morazi has quit IRC | 15:47 | |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters https://review.openstack.org/510167 | 15:59 |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters https://review.openstack.org/510167 | 16:00 |
| openstackgerrit | zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters https://review.openstack.org/510167 | 16:01 |
| EmilienM | mnaser: can I get a review on https://review.openstack.org/#/c/509850/ please? | 16:04 |
| *** bnemec has quit IRC | 16:08 | |
| *** jaganathan has quit IRC | 16:10 | |
| mnaser | EmilienM sure one sec, just checking 1 thing | 16:13 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova master: Allow setting log files for nova-api apache vhost https://review.openstack.org/509127 | 16:16 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova master: Allow setting log files for nova-placement apache vhost https://review.openstack.org/509151 | 16:17 |
| openstackgerrit | Merged openstack/puppet-zaqar master: Set *_domain_name vars to 'Default' by default https://review.openstack.org/509977 | 16:18 |
| mwhahaha | murano appears broken | 16:19 |
| mwhahaha | http://logs.openstack.org/71/510071/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8172604/logs/puppet.txt.gz#_2017-10-06_14_41_18 | 16:19 |
| mwhahaha | fyi | 16:19 |
| EmilienM | BLAH | 16:22 |
| mnaser | isn't murano like | 16:22 |
| mnaser | unmaintained? i think? | 16:22 |
| mnaser | wait or did we break it with the keystone v3 changes | 16:22 |
| mwhahaha | probably broke it with v3 | 16:23 |
| mwhahaha | but yea we might need to drop it from scenario testing | 16:23 |
| mnaser | ooooh | 16:23 |
| mnaser | mwhahaha i found it | 16:23 |
| mwhahaha | k | 16:23 |
| mnaser | we got rid of admin_user etc | 16:23 |
| mnaser | instead its keystone_authtoken/username etc | 16:24 |
| mnaser | https://github.com/openstack/puppet-murano/blob/master/lib/puppet/provider/murano.rb#L68-L76 | 16:24 |
| mnaser | so these are wrong | 16:24 |
| mnaser | 1 sec ill push up a patch | 16:24 |
| openstackgerrit | Merged openstack/puppet-ec2api master: Set *_domain_name vars to 'Default' by default https://review.openstack.org/509988 | 16:29 |
| mnaser | lol | 16:33 |
| mnaser | puppet-murano was nonvoting on debian | 16:33 |
| mnaser | but it was only testing on debian | 16:33 |
| mnaser | and it was voting on centos | 16:33 |
| mnaser | and doing nothing | 16:33 |
| mnaser | im refactoring it to use p-o-i | 16:33 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider https://review.openstack.org/510178 | 16:34 |
| mnaser | EmilienM mwhahaha ^ | 16:34 |
| *** morazi has joined #puppet-openstack | 16:34 | |
| EmilienM | woops | 16:35 |
| EmilienM | mnaser: ok thx | 16:35 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-openstack-integration master: Add un-tested resources to Murano integration https://review.openstack.org/510183 | 16:42 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes https://review.openstack.org/510184 | 16:43 |
| mnaser | there, that should avoid us having that happen again | 16:43 |
| *** trown is now known as trown|lunch | 16:45 | |
| *** derekh has quit IRC | 16:52 | |
| *** bnemec has joined #puppet-openstack | 16:59 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-murano master: Switch murano auth provider to new authtoken params https://review.openstack.org/510187 | 17:03 |
| *** openstackstatus has quit IRC | 17:05 | |
| *** openstack has joined #puppet-openstack | 17:09 | |
| *** ChanServ sets mode: +o openstack | 17:09 | |
| amoralej | mnaser, oh, you catched it first | 17:10 |
| openstackgerrit | Merged openstack/puppet-cinder master: Read os_region_name from cinder.conf when doing requests with the Openstack client https://review.openstack.org/508926 | 17:12 |
| mwhahaha | mnaser: https://review.openstack.org/#/c/510178/ does it need to be OS_PROJECT_NAME instead? | 17:12 |
| mnaser | mwhahaha im waiting for xenial beaker to run, i think it will be okay (ill fix it in a follow up patch so we don't sit in queue for much longer) | 17:13 |
| mwhahaha | k | 17:13 |
| mnaser | if its not okay ill edit this patch but im about 90% sure itll beo kay | 17:13 |
| mwhahaha | xenial will probably pass because they haven't gotten the v2 removal yet | 17:13 |
| mwhahaha | which reminds me i think we still deploy pike | 17:14 |
| *** ntpttr_laptop has joined #puppet-openstack | 17:17 | |
| mnaser | mwhahaha it got the v2 removal, thats why it failed | 17:18 |
| mnaser | but because beaker was non voting (and failing for some other reason) | 17:18 |
| mnaser | it was merged, even though it failed for the same "missing credentials" but i guess it went unnoticed | 17:18 |
| *** jpena is now known as jpena|off | 17:19 | |
| mnaser | http://logs.openstack.org/22/510022/5/check/gate-puppet-murano-puppet-beaker-rspec-ubuntu-xenial-nv/036d362/console.html#_2017-10-06_11_26_17_689474 | 17:19 |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc https://review.openstack.org/509978 | 17:20 |
| *** bnemec is now known as beekneemech | 17:20 | |
| amoralej | mnaser, could we run p-o-i scenario003 on puppet-murano gate? | 17:21 |
| amoralej | that would have detected the issue in my previous patch | 17:21 |
| mnaser | amoralej i think we will do that but project-config is frozen right now due to zuulv3 stuff so i think that is a good path to go towards | 17:22 |
| amoralej | ahh, right | 17:23 |
| mnaser | amoralej also, i refactored the codebase to use the integration code for beaker tests, so it will run the same stuff and hopefully (properly) fail both in beaker and integration for now | 17:23 |
| amoralej | that'd be nice | 17:23 |
| amoralej | mnaser, EmilienM, i'm running pipeline job to check if we find more hidden issues with api v3 https://ci.centos.org/view/rdo/view/weirdo-pipelines/view/weirdo-promote-puppet/ | 17:24 |
| amoralej | so far only murano and designate should fail | 17:24 |
| EmilienM | amoralej: excellent | 17:24 |
| amoralej | all other are fixed | 17:24 |
| mnaser | amoralej awesome work | 17:30 |
| amoralej | not until we got a new promotion | 17:31 |
| amoralej | s/got/get/ | 17:33 |
| openstackgerrit | Merged openstack/puppet-ceilometer master: Accept empty strings for log_dir https://review.openstack.org/510043 | 17:33 |
| *** salmankhan has quit IRC | 17:35 | |
| mnaser | we should totally | 17:39 |
| mnaser | run all of our jobs in centos7 (The ones we can at least) | 17:39 |
| mnaser | way less jobs use them, which means we get vms much faster | 17:39 |
| mnaser | murano job is been in queue for an hour, all centos jobs of it were done loooong ago | 17:39 |
| mwhahaha | ehhhh we've had the opposite before | 17:43 |
| mwhahaha | we already do run most of them on centos7 :D | 17:43 |
| *** ntpttr_laptop has quit IRC | 17:45 | |
| *** ntpttr_laptop has joined #puppet-openstack | 17:45 | |
| *** gfidente is now known as gfidente|afk | 17:50 | |
| amoralej | https://ci.centos.org/view/rdo/view/weirdo-pipelines/view/weirdo-promote-puppet/ only scenario003 is failing because of the missing patches which are in the gates, so we should be ok soon | 17:58 |
| amoralej | mnaser, there seems to still be some issue with murano, scenario003 has failed for https://review.openstack.org/#/c/509978/ with depends on https://review.openstack.org/#/c/510178/ | 18:08 |
| amoralej | it seems it's not connecting to keystone properly | 18:08 |
| amoralej | i need to leave now | 18:08 |
| mnaser | amoralej ok thanks for investigating ill have a look | 18:08 |
| amoralej | it seems to try to connect using 127.0.0.1:35357 but it's only listening in ipv1 ::1:35357 | 18:08 |
| amoralej | ipv6 i meant | 18:09 |
| amoralej | http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/2a7d72c/ | 18:09 |
| mnaser | 2017-10-06 17:55:35 +0000 Puppet (err): Failed to apply catalog: Execution of '/bin/murano package-list' returned 1: 503 Service Unavailable: The server is currently unavailable. Please try again at a later time. (HTTP 503) | 18:09 |
| mnaser | indeed | 18:09 |
| mnaser | i think p-o-i might need a fix for that | 18:10 |
| mnaser | 2017-10-06 17:46:14.319 31434 DEBUG oslo_service.service [-] keystone_authtoken.auth_uri = https://[::1]:5000 log_opt_values /usr/lib/python2.7/site-packages/oslo_config/cfg.py:2887 | 18:10 |
| amoralej | mnaser, but authtoken config seems ok http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/2a7d72c/logs/etc/murano/murano.conf.txt.gz | 18:11 |
| amoralej | Unable to validate token: Unable to establish connection to https://127.0.0.1:35357: HTTPSConnectionPool(host='127.0.0.1', port=35357): Max retries exceeded with u | 18:11 |
| mnaser | 2017-10-06 17:46:14.218 31434 WARNING keystonemiddleware.auth_token [-] Configuring admin URI using auth fragments was deprecated in the Kilo release, and will be removed in the Newton release, use 'identity_uri\ instead. | 18:11 |
| amoralej | it's weird | 18:11 |
| mnaser | 2017-10-06 17:46:14.322 31434 DEBUG oslo_service.service [-] keystone_authtoken.identity_uri = None log_opt_values /usr/lib/python2.7/site-packages/oslo_config/cfg.py:2887 | 18:11 |
| amoralej | ups, that's my fault, :( | 18:12 |
| amoralej | sorry, i need to leave now | 18:12 |
| mnaser | amoralej np, ill fix it | 18:13 |
| *** amoralej is now known as amoralej|off | 18:13 | |
| mnaser | i think because murano is unmaintained, its relying on identity_uri and ignoring auth_uri | 18:13 |
| *** gfidente|afk has quit IRC | 18:16 | |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider https://review.openstack.org/510178 | 18:25 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes https://review.openstack.org/510184 | 18:25 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Add identity_uri back to keystone_authtoken options https://review.openstack.org/510206 | 18:25 |
| *** trown|lunch is now known as trown | 18:25 | |
| mnaser | that should fix it | 18:28 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings https://review.openstack.org/509021 | 18:54 |
| mnaser | uhoh | 19:05 |
| mnaser | EmilienM mwhahaha so it looks like murano codebase has not been updated, therefore still lives on the admin_* variables http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/1e2cbe6/logs/murano/murano-api.txt.gz#_2017-10-06_18_48_14_514 | 19:07 |
| mwhahaha | time to non-vote and exclude murano it would seem | 19:07 |
| mnaser | https://github.com/openstack/murano/commit/bd76070b138477013d737e85cd427efa1f6e5b3d | 19:08 |
| mnaser | it would seem that it was "fixed" but more like patched up | 19:08 |
| mnaser | it looks like it's pinned to v2, regardless | 19:08 |
| mwhahaha | https://review.openstack.org/#/c/504122/ | 19:08 |
| mwhahaha | was just dealing with sessions too | 19:09 |
| mnaser | i'll disable it with a note explaining the issue | 19:09 |
| mwhahaha | https://review.openstack.org/#/c/503598/ | 19:11 |
| mwhahaha | that'll fix it | 19:11 |
| mwhahaha | i think | 19:11 |
| openstackgerrit | Merged openstack/puppet-manila master: Accept empty strings for log_dir https://review.openstack.org/510049 | 19:11 |
| * mwhahaha doesn't know murano code but seems like it might | 19:11 | |
| mwhahaha | looks like CI ate the change | 19:11 |
| mwhahaha | it should have been merged days ago | 19:12 |
| mnaser | mwhahaha trying to understand what is the point of "murano_auth" | 19:13 |
| mwhahaha | murano didn't properly use keystoneauth originally | 19:14 |
| *** pester has joined #puppet-openstack | 19:14 | |
| mwhahaha | so they had their own murano_auth section | 19:14 |
| mwhahaha | seems that it's been updated at some point to use keystoneauth | 19:14 |
| mwhahaha | but they just got around to removing the murano_auth bits | 19:14 |
| mwhahaha | we haven't really touched the murano module much since fuel died | 19:14 |
| mwhahaha | fuel was where murano was being pushed | 19:14 |
| mnaser | i think they are using murano_auth instead of keystone_authtoken (not the other way around) | 19:15 |
| mnaser | "Since [murano_auth] section is introduced in Pike, no need to fallback to [keystone_authtoken] now." | 19:15 |
| mnaser | seems like they depend on it | 19:15 |
| mnaser | "_create_keystone_admin_client" uses those credentials | 19:15 |
| * mnaser looks at murano docs | 19:15 | |
| mwhahaha | oh you're right | 19:16 |
| mwhahaha | they've moved out of keystone auth to murano_auth | 19:16 |
| mnaser | <shrug ascii> | 19:16 |
| *** fxpester has quit IRC | 19:17 | |
| mnaser | mwhahaha i vote revert amoralej|off patch which made that change (because it's technically broken), remove murano out of p-o-i (till they fix their stuff?) | 19:17 |
| mwhahaha | i doubt it'll ever get fixed | 19:17 |
| mwhahaha | but sure | 19:17 |
| mwhahaha | might want to send an ML note about it being completely broken | 19:18 |
| mnaser | mwhahaha http://logs.openstack.org/98/503598/1/check/gate-tempest-dsvm-murano-api-ubuntu-xenial/f9cbaa0/logs/etc/murano/murano.conf.txt.gz this config passed their ci | 19:19 |
| *** pester has quit IRC | 19:20 | |
| mwhahaha | maybe we just need to s/keystoneauth/murano_auth? | 19:20 |
| * mwhahaha shrugs | 19:20 | |
| mnaser | wait wait | 19:20 |
| mnaser | mwhahaha is it possible we are not testing against new packages because we havent been able to get the puppet packages promoted? | 19:20 |
| mwhahaha | yes | 19:20 |
| mnaser | i think this might be the cause | 19:20 |
| mnaser | they have both murano_auth and keystone_authtoken configured | 19:21 |
| mwhahaha | well that drop change didn't merge yet | 19:21 |
| mwhahaha | it got lost | 19:21 |
| mnaser | mwhahaha ill attempt configuring both keystone_authtoken and murano_auth with the same info and maybe | 19:25 |
| mnaser | aw shucks, keystone::resource::authtoken is hardcoded with keystone_authtoken | 19:26 |
| *** tesseract has quit IRC | 19:31 | |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Add murano_auth configuration https://review.openstack.org/510206 | 19:31 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider https://review.openstack.org/510178 | 19:31 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes https://review.openstack.org/510184 | 19:31 |
| *** yprokule has quit IRC | 19:44 | |
| *** ansmith has quit IRC | 20:06 | |
| *** ansmith_ has quit IRC | 20:07 | |
| *** jtomasek has quit IRC | 20:08 | |
| *** spectr has joined #puppet-openstack | 20:34 | |
| *** spectr has quit IRC | 20:36 | |
| *** trown is now known as trown|outtypewww | 20:39 | |
| *** beekneemech has quit IRC | 20:45 | |
| *** chem has quit IRC | 20:45 | |
| *** archit has quit IRC | 20:47 | |
| mnaser | sigh, it failed against, so frustrating | 21:11 |
| mnaser | i'll look into it and try to take a decision later today | 21:12 |
| openstackgerrit | Merged openstack/puppet-panko master: Allow setting log files for apache vhost https://review.openstack.org/510072 | 21:23 |
| openstackgerrit | Merged openstack/puppet-ironic master: Allow setting log files for apache vhost https://review.openstack.org/510069 | 21:23 |
| openstackgerrit | Merged openstack/puppet-ironic master: Accept empty strings for log_dir https://review.openstack.org/510047 | 21:24 |
| openstackgerrit | Merged openstack/puppet-ironic stable/pike: Increase Ironic inspector dnsmasq lease time https://review.openstack.org/510125 | 21:24 |
| *** dprince has quit IRC | 21:25 | |
| *** ccamacho has quit IRC | 21:34 | |
| *** pradk has quit IRC | 21:57 | |
| *** ntpttr_laptop has quit IRC | 22:02 | |
| *** ntpttr_laptop has joined #puppet-openstack | 22:50 | |
| *** ntpttr_laptop has quit IRC | 23:50 | |
| *** markvoelker has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!