Friday, 2017-10-13

*** archit has joined #puppet-openstack00:05
*** gfidente|afk has quit IRC00:11
*** ilbot3 has quit IRC01:46
*** markvoelker has joined #puppet-openstack01:47
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51166601:55
*** ilbot3 has joined #puppet-openstack01:55
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Refactor breaker into openstack_integration  https://review.openstack.org/51166902:01
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Remove driver LibvirtOpenVswitchDriver  https://review.openstack.org/51145002:02
openstackgerritMohammed Naser proposed openstack/puppet-cloudkitty master: Configure *_domain_name to Default by default  https://review.openstack.org/51167102:09
openstackgerritMohammed Naser proposed openstack/puppet-cloudkitty master: Allow setting log files for apache vhost  https://review.openstack.org/51136202:09
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51166602:12
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Refactor breaker into openstack_integration  https://review.openstack.org/51166902:12
*** mgagne has quit IRC02:19
*** markvoelker has quit IRC02:21
*** mgagne has joined #puppet-openstack02:22
*** mgagne is now known as Guest5368002:22
openstackgerritMohammed Naser proposed openstack/puppet-nova master: Remove driver LibvirtOpenVswitchDriver  https://review.openstack.org/51145002:27
*** archit has quit IRC02:41
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061102:56
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065102:56
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065202:56
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Refactor package installations to bindep.txt  https://review.openstack.org/51168702:56
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Refactor beaker to use integration classes  https://review.openstack.org/51169103:09
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Refactor package installations to bindep.txt  https://review.openstack.org/51168703:10
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061103:10
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065103:10
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065203:10
*** udesale has joined #puppet-openstack03:12
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Add support to set host param  https://review.openstack.org/50918003:13
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Refactor package installations to bindep.txt  https://review.openstack.org/51168703:18
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061103:18
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065103:18
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065203:18
*** markvoelker has joined #puppet-openstack03:18
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Manage correct rados package depending on operating system  https://review.openstack.org/51169203:24
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Refactor beaker to use integration classes  https://review.openstack.org/51169103:24
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Manage correct rados package depending on operating system  https://review.openstack.org/51169203:24
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Refactor package installations to bindep.txt  https://review.openstack.org/51168703:24
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061103:24
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065103:24
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065203:24
mnaserim sorry for all teh damn spam03:25
* mnaser is taking advantage of an empty gate03:25
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Add support to set host param  https://review.openstack.org/50918003:25
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty master: [DNM] test  https://review.openstack.org/51169403:38
openstackgerritMerged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/51077803:41
openstackgerritMerged openstack/puppet-watcher master: Allow setting log files for apache vhost  https://review.openstack.org/51137403:43
*** jaosorior has joined #puppet-openstack03:45
mnaserhere comes a stack :>03:50
*** markvoelker has quit IRC03:51
openstackgerritMerged openstack/puppet-vitrage master: Allow setting log files for apache vhost  https://review.openstack.org/51137303:51
openstackgerritMerged openstack/puppet-vitrage master: Accept empty strings for log_dir  https://review.openstack.org/51137103:52
openstackgerritMerged openstack/puppet-trove master: Accept empty strings for log_dir and log_file  https://review.openstack.org/51137003:52
openstackgerritMerged openstack/puppet-neutron master: Add ability to set load balancer quota  https://review.openstack.org/51138703:52
*** ykarel|afk has joined #puppet-openstack04:06
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty master: [DNM] test  https://review.openstack.org/51169404:13
EmilienMmnaser: what's up04:15
EmilienMyou're a spammer !!04:15
EmilienMmnaser: do you sleep sometimes?04:24
EmilienMI hope so04:24
openstackgerritMerged openstack/puppet-cloudkitty master: Configure *_domain_name to Default by default  https://review.openstack.org/51167104:39
openstackgerritMerged openstack/puppet-cloudkitty master: Allow setting log files for apache vhost  https://review.openstack.org/51136204:40
openstackgerritMerged openstack/puppet-horizon master: Added customization-module option to Horizon config  https://review.openstack.org/51105404:43
*** ratailor has joined #puppet-openstack04:46
*** markvoelker has joined #puppet-openstack04:49
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170304:56
*** ratailor_ has joined #puppet-openstack04:57
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-keystone master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170405:00
*** udesale has quit IRC05:01
*** ratailor has quit IRC05:01
openstackgerritMerged openstack/puppet-gnocchi master: Accept empty strings for log_dir  https://review.openstack.org/51136605:06
openstackgerritMerged openstack/puppet-designate master: Accept empty strings for log_dir  https://review.openstack.org/51136305:06
openstackgerritMerged openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/51067505:06
openstackgerritMerged openstack/puppet-openstack_extras master: Added OS_AUTH_TYPE parameter to auth file  https://review.openstack.org/51089905:07
openstackgerritMerged openstack/puppet-neutron master: Fixes vhostuser port prefix with ODL OVS host config  https://review.openstack.org/51133705:07
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-zaqar master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170505:14
*** ratailor__ has joined #puppet-openstack05:16
*** jaosorior has quit IRC05:17
*** ratailor_ has quit IRC05:20
*** ratailor has joined #puppet-openstack05:20
*** ykarel|afk has quit IRC05:21
*** markvoelker has quit IRC05:22
*** ratailor__ has quit IRC05:23
*** jaosorior has joined #puppet-openstack05:30
*** ykarel has joined #puppet-openstack05:32
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170805:34
*** skramaja has joined #puppet-openstack05:36
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-aodh master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171005:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-ironic master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171105:38
*** jaganathan has joined #puppet-openstack05:38
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-panko master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171205:40
*** rcernin has joined #puppet-openstack05:41
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-neutron master: Fix workers/threads documentation  https://review.openstack.org/51171305:42
*** ratailor_ has joined #puppet-openstack05:46
*** ratailor has quit IRC05:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171605:49
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-ceilometer master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171705:51
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-barbican master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171805:52
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170305:55
*** yprokule has joined #puppet-openstack05:57
*** spectr has quit IRC05:58
*** spectr has joined #puppet-openstack05:58
openstackgerritMerged openstack/puppet-gnocchi master: Add processing delay param for metricd  https://review.openstack.org/51123306:03
openstackgerritMerged openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest  https://review.openstack.org/50980706:03
*** ykarel_ has joined #puppet-openstack06:16
*** ykarel has quit IRC06:17
*** markvoelker has joined #puppet-openstack06:19
openstackgerritzhongshengping proposed openstack/puppet-congress master: Configure *_domain_name to Default by default  https://review.openstack.org/51172406:29
openstackgerritzhongshengping proposed openstack/puppet-freezer master: Configure *_domain_name to Default by default  https://review.openstack.org/51172506:31
openstackgerritzhongshengping proposed openstack/puppet-glare master: Configure *_domain_name to Default by default  https://review.openstack.org/51172606:33
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Configure *_domain_name to Default by default  https://review.openstack.org/51172706:36
openstackgerritzhongshengping proposed openstack/puppet-openstack-cookiecutter master: Configure *_domain_name to Default by default  https://review.openstack.org/51172806:39
*** ccamacho has joined #puppet-openstack06:39
openstackgerritzhongshengping proposed openstack/puppet-tacker master: Configure *_domain_name to Default by default  https://review.openstack.org/51172906:40
openstackgerritMerged openstack/puppet-nova master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51166606:43
openstackgerritMerged openstack/puppet-nova master: Remove driver LibvirtOpenVswitchDriver  https://review.openstack.org/51145006:43
openstackgerritTrygve Vea proposed openstack/puppet-neutron stable/pike: Deprecate parameters deprecated in Ocata  https://review.openstack.org/51173006:43
openstackgerritMerged openstack/puppet-openstack-integration master: Manage correct rados package depending on operating system  https://review.openstack.org/51169206:44
openstackgerritzhongshengping proposed openstack/puppet-trove master: Configure *_domain_name to Default by default  https://review.openstack.org/51173206:46
openstackgerritzhongshengping proposed openstack/puppet-vitrage master: Configure *_domain_name to Default by default  https://review.openstack.org/51173406:48
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-ironic master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171106:50
*** markvoelker has quit IRC06:52
*** shardy has joined #puppet-openstack06:54
*** ykarel has joined #puppet-openstack06:55
*** ykarel_ has quit IRC06:57
openstackgerritzhongshengping proposed openstack/puppet-nova master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170306:58
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-zaqar master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170506:59
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170807:01
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-aodh master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171007:02
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-ironic master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171107:02
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-panko master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171207:03
openstackgerritzhongshengping proposed openstack/puppet-watcher master: Configure *_domain_name to Default by default  https://review.openstack.org/51174407:19
*** tesseract has joined #puppet-openstack07:22
*** dtantsur|afk has quit IRC07:26
*** spectr has quit IRC07:26
*** tesseract has quit IRC07:26
*** rcernin has quit IRC07:26
*** pradk has quit IRC07:28
*** dtantsur has joined #puppet-openstack07:31
*** amoralej|off is now known as amoralej07:38
*** spectr has joined #puppet-openstack07:39
*** rcernin has joined #puppet-openstack07:41
*** jpena|off is now known as jpena07:41
*** spectr has quit IRC07:43
*** spectr has joined #puppet-openstack07:44
*** markvoelker has joined #puppet-openstack07:50
*** rmart04 has joined #puppet-openstack08:10
*** vnogin has joined #puppet-openstack08:12
*** vnogin has quit IRC08:16
openstackgerritzhongshengping proposed openstack/puppet-barbican master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51176008:20
*** markvoelker has quit IRC08:23
*** derekh has joined #puppet-openstack08:26
*** ykarel is now known as ykarel|lunch08:36
*** gfidente has joined #puppet-openstack08:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-aodh master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171008:47
*** salmankhan has joined #puppet-openstack08:53
*** salmankhan1 has joined #puppet-openstack08:57
*** salmankhan has quit IRC08:57
*** salmankhan1 is now known as salmankhan08:57
*** dtantsur has quit IRC08:57
*** dtantsur has joined #puppet-openstack08:58
*** rcernin has quit IRC08:59
*** spectr has quit IRC08:59
*** spectr has joined #puppet-openstack09:12
*** rcernin has joined #puppet-openstack09:13
*** ykarel|lunch is now known as ykarel09:18
*** markvoelker has joined #puppet-openstack09:20
*** salmankhan has quit IRC09:26
*** salmankhan has joined #puppet-openstack09:31
*** rmart04 has quit IRC09:33
*** vnogin has joined #puppet-openstack09:35
*** rmart04 has joined #puppet-openstack09:36
*** vnogin has quit IRC09:40
*** ratailor__ has joined #puppet-openstack09:46
*** ratailor_ has quit IRC09:49
*** vnogin has joined #puppet-openstack09:50
*** markvoelker has quit IRC09:53
*** shardy has quit IRC09:59
*** vnogin has quit IRC10:00
*** vnogin has joined #puppet-openstack10:00
*** rmart04 has quit IRC10:05
*** rmart04 has joined #puppet-openstack10:36
*** morazi has quit IRC10:36
*** salmankhan has quit IRC10:45
*** jaosorior has quit IRC10:47
*** markvoelker has joined #puppet-openstack10:50
*** salmankhan has joined #puppet-openstack11:02
*** markvoelker has quit IRC11:23
*** morazi has joined #puppet-openstack11:36
*** salmankhan has quit IRC11:38
*** salmankhan has joined #puppet-openstack11:39
*** morazi has quit IRC11:42
*** morazi has joined #puppet-openstack11:42
*** dprince has joined #puppet-openstack11:54
*** pradk has joined #puppet-openstack11:55
*** jaganathan has quit IRC12:10
*** skramaja has quit IRC12:13
*** markvoelker has joined #puppet-openstack12:15
*** shardy has joined #puppet-openstack12:19
*** trown|outtypewww is now known as trown12:28
*** pradk has quit IRC12:28
*** jpena is now known as jpena|lunch12:43
zioprotohello12:51
zioprotoI am trying to use this thing here: https://github.com/openstack/puppet-oslo/blob/stable/newton/manifests/messaging/notifications.pp#L3012:51
zioprotobut it is not a class12:51
zioprotoso I cant include it like12:51
zioprotoclass { '::oslo::messaging::notifications':12:51
zioprotohow am I supposed to use it ? is there an example ?12:51
zioprotoMaybe I found an example: https://github.com/openstack/puppet-sahara/blob/3f5761ed2a48701863f4dcf520022201afdd4488/manifests/notify.pp#L3112:54
*** ratailor__ has quit IRC12:54
*** ratailor__ has joined #puppet-openstack12:55
*** amoralej is now known as amoralej|lunch12:55
*** spectr has quit IRC12:57
*** spectr has joined #puppet-openstack13:01
KvisleHi, I proposed a change (https://review.openstack.org/#/c/511730/), but Jenkins failed -- and I did a recheck, because I couldn't see what those errors had to do with my change ---- does anyone have an oppinion of this?13:06
*** jistr is now known as jistr|mtg13:08
*** ratailor_ has joined #puppet-openstack13:14
*** ratailor__ has quit IRC13:17
openstackgerritMerged openstack/puppet-sahara master: Accept empty strings for log_dir  https://review.openstack.org/51136813:18
*** karthiks has quit IRC13:26
*** ratailor__ has joined #puppet-openstack13:27
*** ratailor_ has quit IRC13:30
*** rmart04 has quit IRC13:30
*** jpena|lunch is now known as jpena13:41
*** rcernin has quit IRC13:44
*** amoralej|lunch is now known as amoralej13:50
*** archit has joined #puppet-openstack13:56
*** bnemec is now known as beekneemech14:00
*** trown is now known as trown|brb14:08
openstackgerritTim Rozet proposed openstack/puppet-neutron stable/pike: Fixes vhostuser port prefix with ODL OVS host config  https://review.openstack.org/51186214:13
openstackgerritLokesh Jain proposed openstack/puppet-horizon stable/pike: Added customization-module option to Horizon config  https://review.openstack.org/51186314:14
*** trown|brb is now known as trown14:15
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Refactor beaker to use integration classes  https://review.openstack.org/51169114:16
openstackgerritMohammed Naser proposed openstack/puppet-barbican master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51176014:22
openstackgerritLokesh Jain proposed openstack/puppet-horizon stable/ocata: Added customization-module option to Horizon config  https://review.openstack.org/51187514:23
openstackgerritMohammed Naser proposed openstack/puppet-barbican master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171814:23
*** ratailor__ has quit IRC14:24
openstackgerritDan Radez proposed openstack/puppet-ironic master: Adding support for UEFI client arch type 11 (EFI aarch64)  https://review.openstack.org/51187714:27
openstackgerritLokesh Jain proposed openstack/puppet-horizon stable/newton: Added customization-module option to Horizon config  https://review.openstack.org/51187814:28
*** vnogin has quit IRC14:30
KvisleCan this get a review? https://review.openstack.org/#/c/510372/14:31
mnaserKvisle /me looks14:34
Kvisle\o/14:34
mnaserKvisle can you drop the warning from init.pp? because db.pp actually is included from init.pp anyways so its causing double warnings14:34
mnaserhttp://logs.openstack.org/72/510372/4/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/275d4e7/logs/puppet-20171009_192827.txt.gz#_2017-10-09_19_02_4314:35
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061114:36
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065114:36
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065214:36
openstackgerritTrygve Vea proposed openstack/puppet-nova master: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/51037214:37
Kvislemnaser: thanks, done14:37
* Kvisle would like to cherry pick the commit for ocata and pike, if/when merged.14:37
mnaserKvisle cool, +2 :)14:37
mnaserdmsimard https://review.openstack.org/#/c/510611/ -- addressed your (nice big) review :)14:38
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer master: Add support to set host param  https://review.openstack.org/50918014:40
dmsimardmnaser: cool I'll check sometime later14:41
mnaserok looks like stable/pike is broken14:41
mnaserhttp://logs.openstack.org/62/511862/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/00240f9/logs/puppet.txt.gz#_2017-10-13_14_19_5714:42
mnaseroh crap14:44
mnasermwhahaha it looks like puppet-vitrage stable/pike was not cut?14:44
mnaserso stable/pike integration jobs are falling back to master14:44
mnaserand then complaining about the access log stuff not working14:44
mnasershould we cut stable/pike now, and revert that log file change in it?14:44
mwhahahalet me see14:45
*** rcernin has joined #puppet-openstack14:45
mwhahahamnaser: we haven't released puppet-vitrage at all14:45
mwhahahaso we've never released it14:46
mwhahahawe could start for queens m1 i guess14:46
*** spectr has quit IRC14:46
mwhahahacloudkitty is another one like that with no releases14:46
* mwhahaha is unsure of the dev status of these modules14:46
mnasermwhahaha so drop them from p-o-i in the stable branches?14:47
mwhahahacloudkitty isn't in p-o-i14:47
mwhahahasince vitrage is, maybe we can start cutting them for queens14:47
mwhahahabut pull it from pike14:47
mnasercloudkitty isnt even in intergration so i think this is why its not an issue14:48
mwhahahaor we can release a pike version14:48
* mwhahaha shrugs14:48
mwhahahaprobably pulling it from pike p-o-i is the best action right now14:48
mnaserill do that14:48
mwhahahaor just turn it off14:48
mwhahahaleave the code but turn it off14:49
dmsimardmnaser: lgtm, just one comment14:53
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration stable/pike: Add Zuul V3 support for Puppet Jobs  https://review.openstack.org/50920214:56
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration stable/pike: [pike only] remove vitrage testing code  https://review.openstack.org/51189614:56
*** jistr|mtg is now known as jistr14:58
*** rcernin has quit IRC15:03
EmilienMmnaser, mwhahaha: just to sync - next week is queens-1 (yeah alraedy) - who's working on it?15:06
mnaserEmilienM mwhahaha i dont mind either shadow-ing someone doing it to see how the release goes, or doing it myself with someone's guidance15:07
mwhahahaI'll prep it next week15:07
EmilienMcool15:07
EmilienMI'll do tripleo15:08
EmilienMmwhahaha: if you don't mind15:08
mwhahahaK15:08
EmilienMcool15:08
mwhahahaThat's fine15:08
*** archit has quit IRC15:16
*** ratailor has joined #puppet-openstack15:22
*** ratailor has quit IRC15:23
*** ntpttr_laptop has joined #puppet-openstack15:24
openstackgerritMohammed Naser proposed openstack/puppet-murano master: Add murano_auth configuration  https://review.openstack.org/51020615:27
openstackgerritMohammed Naser proposed openstack/puppet-murano master: Fix murano base provider  https://review.openstack.org/51017815:27
openstackgerritMohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes  https://review.openstack.org/51018415:27
*** archit has joined #puppet-openstack15:29
*** vnogin has joined #puppet-openstack15:29
openstackgerritWilliam Huang proposed openstack/puppet-cloudkitty master: Create rating role in keystone/auth.pp  https://review.openstack.org/51190915:36
*** dtantsur is now known as dtantsur|afk15:39
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Add option to enable or disable compute namespace for Ceilometer  https://review.openstack.org/51191015:39
openstackgerritMohammed Naser proposed openstack/puppet-ceilometer master: Refactor beaker to use integration classes  https://review.openstack.org/51169115:41
*** fultonj has quit IRC15:59
*** makowals has quit IRC16:09
*** ykarel is now known as ykarel|afk16:12
*** lnxnut_ has quit IRC16:16
*** ykarel|afk has quit IRC16:20
mnaserEmilienM mwhahaha https://review.openstack.org/#/c/511896/ to unblock stable/pike for now16:25
*** derekh has quit IRC16:33
*** xarses has joined #puppet-openstack16:33
*** rpittau has quit IRC16:34
*** salmankhan has quit IRC16:42
*** trown is now known as trown|lunch16:45
EmilienMmnaser: ok16:46
EmilienMmnaser: will be reverted at some point?16:46
openstackgerritMerged openstack/puppet-aodh master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171016:48
openstackgerritMerged openstack/puppet-nova master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170316:48
*** EmilienM changes topic to "CI Status: Green | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/"16:48
openstackgerritMerged openstack/puppet-ironic master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171116:48
*** vnogin has quit IRC16:48
*** ntpttr_laptop has quit IRC16:48
*** jtomasek has quit IRC16:49
*** yprokule has quit IRC16:59
*** ntpttr_laptop has joined #puppet-openstack16:59
*** jpena is now known as jpena|off17:03
*** shardy has quit IRC17:05
mnaserEmilienM no, because i think we'll just release watcher in queens17:06
EmilienMmnaser: wasn't it vitrage?17:06
mnaseroh yes17:06
mnasersorry17:06
*** shardy has joined #puppet-openstack17:06
EmilienMwhy don't we release it in pike?17:07
EmilienMand re-establish ci?17:07
mnaserwe will have to revert a patch in it, cut pike release, and then revert revert the patch17:07
EmilienMyeah17:07
EmilienMworks for me17:07
EmilienMI mean, if we have vitrage users in pike17:07
EmilienMwe want to keep them happy by making sure puppet-vitrage is still well tested17:07
mnaservalid17:08
EmilienMeven if not much patches are sent to this module17:08
EmilienMI think Nokia is using it17:08
EmilienMthough we'll have to ask for a late Pike release17:08
EmilienMbut I don't think release-managers would mind17:08
mnaserEmilienM ok so what would be the best way to go about this, ask in #openstack-release if we can get this permission, revert the patch that breaks it, release pike, cut stable/pike, and then revert the revert?17:09
EmilienMyeah17:09
EmilienMping sean or doug17:09
*** jtomasek has joined #puppet-openstack17:09
mnaserEmilienM should i propose a release with the commit hash *before* the change that broke things?17:20
mnaserand so stable/pike will start from there?17:21
EmilienMmnaser: ++17:21
EmilienMmnaser: please patch puppet-vitrage first17:22
EmilienMto prepare the metadata.json17:22
EmilienMoh but17:22
EmilienMit won't work17:22
EmilienMunless the tag is good, let me see17:22
mnaserversion 0.0.117:23
EmilienMyeah we would have to release 0.0.1 ...17:23
EmilienMwhich is fine I guess17:23
EmilienMfrom a CI perspective, that's not a blocker.17:23
mnasereh lets do it right17:23
EmilienMthe best way to do it right would be to17:23
EmilienMrevert the broken patch17:23
EmilienMpatch metadata with right tag17:23
EmilienMmake a release17:23
EmilienMrevert the revert17:23
mnaseragreed17:23
openstackgerritMerged openstack/puppet-zaqar master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170517:28
openstackgerritMerged openstack/puppet-neutron master: Fix workers/threads documentation  https://review.openstack.org/51171317:28
openstackgerritMerged openstack/puppet-heat master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171617:28
openstackgerritMerged openstack/puppet-keystone master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51170417:28
openstackgerritMohammed Naser proposed openstack/puppet-vitrage master: Revert "Allow setting log files for apache vhost"  https://review.openstack.org/51192617:29
openstackgerritMohammed Naser proposed openstack/puppet-vitrage master: Prepare for Vitrage 1.0.0  https://review.openstack.org/51192717:29
openstackgerritMohammed Naser proposed openstack/puppet-vitrage master: Revert "Revert "Allow setting log files for apache vhost""  https://review.openstack.org/51192817:29
mnaserEmilienM ^17:29
EmilienMmnaser: awesome17:35
EmilienMlet me take a look17:35
*** vnogin has joined #puppet-openstack17:41
*** trown|lunch is now known as trown17:41
*** gfidente has quit IRC17:43
*** jtomasek has quit IRC17:45
EmilienMmnaser: I'm confused why https://review.openstack.org/511926 breaks things17:49
mnaserEmilienM: the scenario - p-o-i runs in stable/pike, clones stable/pike for all modules (including openstacklib stable/pike), clones master for vitrage, master of vitrage uses those options which are *not* in openstacklib stable/pike but in master, so puppet fails because the attributes dont exist17:51
EmilienMah17:55
EmilienMthat17:55
EmilienMhehe17:55
EmilienMgo4it17:55
openstackgerritMerged openstack/puppet-freezer master: Configure *_domain_name to Default by default  https://review.openstack.org/51172517:57
*** jtomasek has joined #puppet-openstack17:57
openstackgerritMerged openstack/puppet-glare master: Configure *_domain_name to Default by default  https://review.openstack.org/51172617:59
*** salmankhan has joined #puppet-openstack18:04
openstackgerritMerged openstack/puppet-magnum master: Configure *_domain_name to Default by default  https://review.openstack.org/51172718:08
*** salmankhan has quit IRC18:08
*** Guest53680 is now known as mgagne18:14
*** mgagne has quit IRC18:14
*** mgagne has joined #puppet-openstack18:14
*** ntpttr_laptop has quit IRC18:22
*** amoralej is now known as amoralej|off18:27
*** dprince has quit IRC18:33
mnaserEmilienM mwhahaha https://review.openstack.org/#/c/511926/ and https://review.openstack.org/#/c/511927/ are ready if possible :)18:35
EmilienMlooking18:38
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Add option to enable or disable compute namespace for Ceilometer  https://review.openstack.org/51191018:38
openstackgerritMerged openstack/puppet-openstack-cookiecutter master: Configure *_domain_name to Default by default  https://review.openstack.org/51172819:03
mnaserEmilienM mwhahaha can we come to consensus about the version #?  also, i checked and the reno config had no version so it should naturally pick it up (this one was fixed)19:22
EmilienMmnaser: approved19:24
EmilienMI didn't know about that19:24
mwhahahaK19:26
mwhahahaYea they have complained the last few times about it. So for new modules they start at 1.0.019:26
mwhahahaI've argued about it a bunch but with automated release stuff now it's not too baf19:27
EmilienMok19:28
mwhahahas/baf/bad19:29
EmilienMk19:29
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration stable/pike: [pike only] remove vitrage testing code  https://review.openstack.org/51189619:34
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration stable/pike: Add Zuul V3 support for Puppet Jobs  https://review.openstack.org/50920219:34
mnaser^ and thats a much cleaner disable as proposed19:34
mnaserDetails: {u'message': u'Unable to associate floating IP 172.24.5.21 to fixed IP 10.100.0.12 for instance 8857c778-521c-46ea-b55a-d618144fa8a4. Error: Request to https://127.0.0.1:9696/v2.0/ports?tenant_id=8010df8cfcfe43bfb8972e8646c97207&device_id=8857c778-521c-46ea-b55a-d618144fa8a4 timed out', u'code': 400}19:38
mnaserbeen seeing tis more than once now19:38
*** ntpttr_laptop has joined #puppet-openstack19:39
*** ccamacho has quit IRC19:46
*** vnogin has quit IRC19:47
*** morazi has quit IRC19:55
*** vnogin has joined #puppet-openstack19:56
*** vnogin has quit IRC20:00
dmsimardmnaser: left a comment in https://review.openstack.org/#/c/510611/20:16
dmsimardmnaser: but now I'm thinking, we will likely have a lot of those zuul variables that we expect and need throughout the different roles20:17
dmsimardp-o-i gate jobs are "special" in the sense that they are designed to be usable outside the gate (amongst other things for collaboration with RDO)20:17
dmsimardWhat I'm thinking right now is that we could perhaps have a role that validates that we have the required variables and then declare those by means of a meta dependency20:18
dmsimardlet me put up a quick patch to show what I mean20:19
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration master: WIP: Add role to validate Zuul variables  https://review.openstack.org/51195820:29
dmsimardmnaser: ^20:29
dmsimardmaybe it's okay to expect zuul variables to be set -- people running it outside a gate can provide them as required20:29
*** trown is now known as trown|outtypewww20:30
mnaserdmsimard the way i imagined things to be is that run_tests.sh becomes the entry point for people running it outside the gate20:35
dmsimardPretty much what happens today20:35
mnaseryeah but today run_tests.sh is the entry point of the gate as well.  in zuulv3, i imagine our entry point will be a playbook where the vars will be filled in by zuul. and run_tests.sh runs that same playbook but with whatever vars zuul provides instead.  we wouldnt call run_tests.sh in the gate anymore, or should we.. idk, its friday and my brain is toast20:37
openstackgerritMerged openstack/puppet-barbican master: Switch beaker tests to use Keystone V3  https://review.openstack.org/51176021:02
openstackgerritMerged openstack/puppet-congress master: Configure *_domain_name to Default by default  https://review.openstack.org/51172421:02
openstackgerritMerged openstack/puppet-horizon stable/pike: Added customization-module option to Horizon config  https://review.openstack.org/51186321:02
openstackgerritMerged openstack/puppet-horizon stable/ocata: Added customization-module option to Horizon config  https://review.openstack.org/51187521:02
openstackgerritMerged openstack/puppet-horizon stable/newton: Added customization-module option to Horizon config  https://review.openstack.org/51187821:03
openstackgerritMerged openstack/puppet-nova master: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/51037221:03
openstackgerritMerged openstack/puppet-nova master: Refactor breaker into openstack_integration  https://review.openstack.org/51166921:03
*** morazi has joined #puppet-openstack21:16
*** archit has quit IRC21:25
openstackgerritMerged openstack/puppet-tacker master: Configure *_domain_name to Default by default  https://review.openstack.org/51172921:38
openstackgerritMerged openstack/puppet-trove master: Configure *_domain_name to Default by default  https://review.openstack.org/51173221:38
openstackgerritMerged openstack/puppet-vitrage master: Revert "Allow setting log files for apache vhost"  https://review.openstack.org/51192621:38
openstackgerritMerged openstack/puppet-watcher master: Configure *_domain_name to Default by default  https://review.openstack.org/51174421:38
openstackgerritMerged openstack/puppet-vitrage master: Prepare for Vitrage 1.0.0  https://review.openstack.org/51192721:55
openstackgerritTrygve Vea proposed openstack/puppet-nova stable/pike: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/51197222:11
openstackgerritTrygve Vea proposed openstack/puppet-nova stable/ocata: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/51197322:11
EmilienMmnaser: no unit tests on https://review.openstack.org/#/c/510206/ ?22:21
*** vnogin has joined #puppet-openstack22:30
*** xarses has quit IRC23:08
*** ntpttr_laptop has quit IRC23:12
*** vnogin has quit IRC23:24
mnaserEmilienM this was because im still not even sure its the right config.. so im still looking if this will solve our murano woes in other patches so im waiting till i figure out what works and what doesnt23:40
EmilienMcool23:40
EmilienMI can approve it if you like23:40
*** markvoelker has quit IRC23:49

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