Monday, 2017-10-09

*** archit has joined #puppet-openstack01:36
*** ilbot3 has quit IRC01:45
*** markvoelker has quit IRC01:54
*** ilbot3 has joined #puppet-openstack01:55
*** archit has quit IRC02:24
*** atoth has quit IRC02:32
*** guimaluf has quit IRC03:26
*** udesale has joined #puppet-openstack03:40
openstackgerritMerged openstack/puppet-neutron master: Allow setting log files for apache vhost  https://review.openstack.org/51007103:46
openstackgerritMerged openstack/puppet-nova master: Remove nova-start  https://review.openstack.org/50875403:47
*** markvoelker has joined #puppet-openstack03:55
openstackgerritMerged openstack/puppet-nova master: Allow setting log files for nova-placement apache vhost  https://review.openstack.org/50915103:58
openstackgerritMerged openstack/puppet-nova master: Remove some useless parameters  https://review.openstack.org/51014303:58
*** jaosorior has joined #puppet-openstack04:07
*** etingof has quit IRC04:27
*** etingof has joined #puppet-openstack04:28
*** markvoelker has quit IRC04:28
*** ratailor has joined #puppet-openstack04:32
*** ratailor_ has joined #puppet-openstack04:33
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-glance master: Accept empty strings for log_file  https://review.openstack.org/51040404:34
*** ratailor_ has quit IRC04:35
*** ratailor_ has joined #puppet-openstack04:37
*** ratailor__ has joined #puppet-openstack04:39
*** ratailor has quit IRC04:40
*** ykarel_ has joined #puppet-openstack04:42
*** ratailor_ has quit IRC04:42
*** ykarel_ is now known as ykarel04:42
*** ntpttr_laptop has joined #puppet-openstack04:48
*** ratailor_ has joined #puppet-openstack04:48
*** ratailor__ has quit IRC04:52
*** ntpttr_laptop has quit IRC04:54
*** ratailor_ has quit IRC05:05
*** ratailor__ has joined #puppet-openstack05:05
openstackgerritTrygve Vea proposed openstack/puppet-nova master: Deprecate dedicated database for placement API  https://review.openstack.org/51037205:23
*** markvoelker has joined #puppet-openstack05:26
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnc-backend  https://review.openstack.org/51040605:40
*** ratailor_ has joined #puppet-openstack05:41
*** ratailor__ has quit IRC05:44
*** etingof has quit IRC05:51
*** spectr has joined #puppet-openstack05:54
*** jtomasek has joined #puppet-openstack05:56
*** markvoelker has quit IRC05:58
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/51040906:00
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnx-backend  https://review.openstack.org/51040606:02
*** udesale__ has joined #puppet-openstack06:03
*** udesale has quit IRC06:05
openstackgerritMerged openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough  https://review.openstack.org/51015106:07
*** etingof has joined #puppet-openstack06:12
*** skramaja has joined #puppet-openstack06:21
*** yprokule has joined #puppet-openstack06:27
*** ykarel_ has joined #puppet-openstack06:28
*** ykarel has quit IRC06:31
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Fix README, was referring to deprecated class  https://review.openstack.org/51041706:32
*** ratailor_ has quit IRC06:35
*** ratailor__ has joined #puppet-openstack06:35
*** rcernin has joined #puppet-openstack06:40
*** ccamacho has joined #puppet-openstack06:52
*** markvoelker has joined #puppet-openstack06:55
*** udesale has joined #puppet-openstack06:59
*** udesale has quit IRC06:59
*** udesale has joined #puppet-openstack06:59
*** udesale has quit IRC07:00
*** udesale__ has quit IRC07:01
*** jaganathan has joined #puppet-openstack07:01
*** udesale has joined #puppet-openstack07:01
*** etingof has quit IRC07:11
*** etingof has joined #puppet-openstack07:13
*** ratailor_ has joined #puppet-openstack07:18
*** ratailor__ has quit IRC07:22
*** ratailor__ has joined #puppet-openstack07:25
*** ratailor_ has quit IRC07:28
*** ratailor has joined #puppet-openstack07:29
*** markvoelker has quit IRC07:29
*** ratailor__ has quit IRC07:31
*** jpena|off is now known as jpena07:39
*** ykarel_ is now known as ykarel07:45
*** amoralej|off is now known as amoralej07:50
*** etingof has quit IRC07:51
*** shardy has joined #puppet-openstack07:55
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Fix README, was referring to deprecated class  https://review.openstack.org/51041707:59
*** etingof has joined #puppet-openstack08:03
*** ykarel_ has joined #puppet-openstack08:14
*** ykarel_ has quit IRC08:16
*** markvoelker has joined #puppet-openstack08:26
*** derekh has joined #puppet-openstack08:32
*** ratailor_ has joined #puppet-openstack08:34
*** gfidente has joined #puppet-openstack08:35
*** chem has joined #puppet-openstack08:36
*** ratailor has quit IRC08:38
*** ratailor_ has quit IRC08:50
*** ratailor has joined #puppet-openstack08:50
*** salmankhan has joined #puppet-openstack08:51
openstackgerritSaverio Proto proposed openstack/puppet-cinder stable/pike: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/51044608:55
openstackgerritSaverio Proto proposed openstack/puppet-cinder stable/ocata: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/51044708:55
*** salmankhan has quit IRC08:55
openstackgerritSaverio Proto proposed openstack/puppet-cinder stable/newton: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/51044808:55
*** salmankhan has joined #puppet-openstack08:57
*** markvoelker has quit IRC08:59
*** ykarel is now known as ykarel|lunch09:05
*** salmankhan has quit IRC09:05
*** salmankhan has joined #puppet-openstack09:15
*** ccamacho has quit IRC09:42
*** ccamacho has joined #puppet-openstack09:43
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/51040909:53
*** markvoelker has joined #puppet-openstack09:56
*** ykarel|lunch is now known as ykarel10:07
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/51040910:15
* Kvisle . o O ( How did that happen? I'm in the tree for 510406. )10:18
KvisleAnyone with gerrit-fu who knows how to make 510406 depend on 510409? The tests in 510406 fail, because 510409 needs to be merged first.10:18
ykarelKvisle, you can add Depends-On: If09687d9b58c6ac8ea08a4cca94956f0e1ccbc9c in 51040610:21
Kvislethanks!10:21
ykarelcommit message10:21
KvisleI tried with git review -d 510409 -- but that didn't seem to work10:21
ykarelKvisle, this would checkout 510409 locally10:22
openstackgerritTrygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnx-backend  https://review.openstack.org/51040610:23
*** udesale has quit IRC10:25
*** markvoelker has quit IRC10:30
*** dtantsur|afk is now known as dtantsur10:55
*** morazi has quit IRC11:03
*** jaosorior has quit IRC11:05
*** jaosorior has joined #puppet-openstack11:05
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902111:06
openstackgerritTrygve Vea proposed openstack/puppet-nova master: Deprecate dedicated database for placement API  https://review.openstack.org/51037211:14
*** markvoelker has joined #puppet-openstack11:27
*** shardy is now known as shardy_lunch11:40
*** ansiwen[q] has joined #puppet-openstack11:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova master: Expose log_outputs parameter for libvirt  https://review.openstack.org/51054411:51
*** trown|outtypewww is now known as trown11:59
*** markvoelker has quit IRC12:00
*** etingof has quit IRC12:07
*** udesale has joined #puppet-openstack12:13
*** etingof has joined #puppet-openstack12:21
*** markvoelker has joined #puppet-openstack12:21
*** ratailor has quit IRC12:24
*** ratailor has joined #puppet-openstack12:28
*** iurygregory has joined #puppet-openstack12:32
*** morazi has joined #puppet-openstack12:32
*** ratailor_ has joined #puppet-openstack12:34
*** ratailor has quit IRC12:38
*** shardy_lunch is now known as shardy12:40
*** jpena is now known as jpena|lunch12:58
*** ansmith has joined #puppet-openstack13:01
*** ratailor_ has quit IRC13:10
*** dprince has joined #puppet-openstack13:10
*** ratailor_ has joined #puppet-openstack13:10
*** skramaja has quit IRC13:18
*** ratailor__ has joined #puppet-openstack13:35
*** ratailor_ has quit IRC13:37
*** ratailor_ has joined #puppet-openstack13:40
*** bnemec has joined #puppet-openstack13:40
*** ratailor__ has quit IRC13:43
*** etingof has quit IRC13:50
*** jpena|lunch is now known as jpena13:56
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902113:57
*** morazi has quit IRC14:05
*** ykarel is now known as ykarel|away14:07
*** etingof has joined #puppet-openstack14:07
*** morazi has joined #puppet-openstack14:07
*** ratailor__ has joined #puppet-openstack14:09
*** ratailor__ has quit IRC14:11
*** jaganathan has quit IRC14:12
*** ratailor_ has quit IRC14:13
*** spectr has quit IRC14:30
amoralejmnaser, i'm removing the depends-on https://review.openstack.org/#/c/50997814:31
amoralejas murano is disabled14:32
openstackgerritAlfredo Moralejo proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/50997814:32
*** ntpttr_laptop has joined #puppet-openstack14:39
*** archit has joined #puppet-openstack14:40
*** ntpttr_laptop has quit IRC14:45
mnaseramoralej yep!  thank you14:50
*** morazi has quit IRC14:50
mnaseramoralej i wonder if i should reapprove or it went into gate14:50
*** morazi has joined #puppet-openstack14:51
amoralejmnaser, it's in gate queue14:52
mnaseroh cool14:52
*** trown is now known as trown|outtypewww15:12
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061115:44
*** jpena is now known as jpena|brb15:45
*** ntpttr_laptop has joined #puppet-openstack15:53
*** ntpttr_laptop has quit IRC15:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost  https://review.openstack.org/51007315:54
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061115:57
*** yprokule has quit IRC16:03
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061116:09
openstackgerritMerged openstack/puppet-glance master: Accept empty strings for log_file  https://review.openstack.org/51040416:14
openstackgerritMerged openstack/puppet-horizon master: Add parameter to configure instance defaults  https://review.openstack.org/51009816:14
*** jpena|brb is now known as jpena16:15
jaosoriormwhahaha: could you check this out https://review.openstack.org/#/c/510066/ ?16:15
*** udesale has quit IRC16:16
*** ykarel|away is now known as ykarel16:26
*** jaosorior has quit IRC16:33
*** jaosorior has joined #puppet-openstack16:34
*** jaosorior has quit IRC16:38
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902116:40
*** jaosorior has joined #puppet-openstack16:40
*** rcernin has quit IRC16:49
EmilienMmnaser, mwhahaha: do we plan to release puppet-openstack newton/ocata/pike more often?16:49
EmilienMI do tripleo every 2 weeks16:49
mwhahahano16:49
EmilienMbou16:50
mwhahahawould just need to evaulate the changes in puppet-tripleo to see which modules to release16:50
mwhahahawe should not do all of them because we do not backport enough to warrant that16:50
*** jaosorior has quit IRC16:53
*** derekh has quit IRC17:01
*** xarses has joined #puppet-openstack17:10
*** jpena is now known as jpena|away17:17
*** shardy has quit IRC17:18
*** amoralej is now known as amoralej|off17:29
*** jtomasek has quit IRC17:30
*** jtomasek has joined #puppet-openstack17:31
*** salmankhan has quit IRC17:37
*** dtantsur is now known as dtantsur|afk18:06
mnaseri think evaluating puppet-tripleo changes and releasing as-need-be works for everyone, it avoids the extra churn18:08
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061118:10
Kvisleis there a term one could use for a parameter that is not deprecated, does not do anything, but may do something in the future? (this is about https://review.openstack.org/#/c/510372/, I'm the proposer)18:12
*** ykarel has quit IRC18:14
*** dprince has quit IRC18:16
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061118:35
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065118:35
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065218:35
mnaserKvisle i would just warn that it is currently un-used and remove any mention of deprecation.  it is a very weird scenario18:35
Kvislemnaser: as in have a warning() whenever it's in use?18:36
Kvisleor warning through a line in the docs?18:36
mnaserKvisle yes, similar to how you've done but avoid saying that it's deprecated18:36
Kvisleack.18:36
Kvisle"has no effect as of pike, and may be removed in a future release"18:39
*** dprince has joined #puppet-openstack18:49
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061118:49
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065118:49
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065218:50
mnaserKvisle but also maybe readded according to Nova's team, that sounds okay to me though18:50
mnaserin other good news im testing out the patch above and it might eliminate out r10k github timeouts 100% ^^ :D18:51
openstackgerritTrygve Vea proposed openstack/puppet-nova master: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/51037218:56
mwhahahathere's a deprecation function we should be using instead of a warning18:57
mwhahahajfyi18:57
mwhahahahttps://github.com/puppetlabs/puppetlabs-stdlib#deprecation18:57
Kvisleoh, TIL18:58
Kvisle... this is however, not a deprecation warning, though.18:59
Kvislebut nice to know.18:59
mwhahahayea just incase it was18:59
* Kvisle have had two patches approved today, maybe not wait almost a year until next time.19:00
mnaserKvisle fyi no need to recheck zuul :) the only failing job was a tripleo job and i think tripleo folks are working to get those fixed up19:07
mnaserzuul isn't voting at the moment anyawys19:07
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/51061119:07
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/51065119:07
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/51065219:07
Kvislemnaser: it doesn't count? I saw a big '-1' on the V, so I assumed it counted.19:07
mnaserKvisle nope, because infra rolled back the deployment of zuulv3, it is currently not meaning much other than "hey your zuul jobs are ok"19:08
mnaserit's a bit confusing for reviewing code i agree, cause you dont know if the -1 is really zuul failing or jenkins19:08
Kvislecool!19:09
*** jtomasek has quit IRC19:13
mnaserEmilienM https://review.openstack.org/#/c/508170/ can i borrow a +w?19:25
openstackgerritMohammed Naser proposed openstack/puppet-cinder stable/pike: Always set keymgr_api_class  https://review.openstack.org/51066519:25
EmilienMmnaser: sure thing19:25
mnaserthe -1s left by zuulv3 make finding things +1d by jenkins so much harder19:26
EmilienMmnaser: done19:26
mnaserthank you19:26
openstackgerritMohammed Naser proposed openstack/puppet-cinder stable/ocata: Always set keymgr_api_class  https://review.openstack.org/51066619:26
openstackgerritMohammed Naser proposed openstack/puppet-cinder stable/newton: Always set keymgr_api_class  https://review.openstack.org/51066719:27
*** morazi has quit IRC19:38
mnaseroh this is so awesome19:42
mnaserwith that r10k + ansible patch, we make 5 attempts with a 5 minute timeout, which means that we have a chance to recover github timeouts19:44
mnaserand if it fails completetly then its only 25 minutes to failure, not JOB_TIMEOUT19:44
mwhahahathat's handy19:48
openstackgerritTrygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/51067519:55
openstackgerritTrygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/51067519:58
*** jpena|away is now known as jpena|off20:01
*** xarses has quit IRC20:18
*** dprince has quit IRC20:19
*** xarses has joined #puppet-openstack20:19
*** andreww has joined #puppet-openstack20:22
*** morazi has joined #puppet-openstack20:22
*** xarses has quit IRC20:23
*** ansmith has quit IRC20:58
*** chem has quit IRC21:06
*** archit has quit IRC21:17
*** ansmith has joined #puppet-openstack21:49
*** etingof has quit IRC21:53
openstackgerritMerged openstack/puppet-cinder master: Configure *_domain_name to Default by default  https://review.openstack.org/51011921:55
*** etingof has joined #puppet-openstack22:06
openstackgerritTrygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/51067522:32
*** gfidente has quit IRC22:35
*** andreww has quit IRC22:46
*** etingof has quit IRC22:47
*** etingof has joined #puppet-openstack23:01
openstackgerritMerged openstack/puppet-cinder master: Allow setting log files for apache vhost  https://review.openstack.org/51006623:58
*** xarses has joined #puppet-openstack23:59

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