Thursday, 2016-02-11

openstackgerritMerged openstack/puppet-heat: Support setting instance_user to an empty string  https://review.openstack.org/27845000:03
*** crinkle_ has joined #puppet-openstack00:03
*** crinkle has quit IRC00:03
openstackgerritMerged openstack/puppet-neutron: Creation of neutron::db::sync  https://review.openstack.org/27845800:03
openstackgerritMerged openstack/puppet-neutron: Use uppercase 'S' in word "OpenStack"  https://review.openstack.org/27613500:04
*** crinkle_ is now known as crinkle00:04
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/25624800:08
*** ducttape_ has joined #puppet-openstack00:09
*** dgurtner has quit IRC00:12
*** sid14726 has quit IRC00:18
*** mdorman has quit IRC00:18
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/25624800:19
*** xarses has quit IRC00:24
*** xarses has joined #puppet-openstack00:32
openstackgerritMerged openstack/puppet-neutron: Provide support for VPNaaS service_provider  https://review.openstack.org/27393400:35
*** alee_ has quit IRC00:38
*** xarses has quit IRC00:43
*** xarses has joined #puppet-openstack00:43
*** xarses has quit IRC00:48
*** baker has joined #puppet-openstack00:50
*** xarses has joined #puppet-openstack00:54
*** ducttape_ has quit IRC00:55
*** xarses_ has joined #puppet-openstack00:56
*** xarses has quit IRC00:59
*** baker_ has joined #puppet-openstack01:00
*** baker has quit IRC01:03
openstackgerritMerged openstack/puppet-trove: Use uppercase 'S' in word "OpenStack"  https://review.openstack.org/27660701:06
*** baker_ has quit IRC01:06
*** nlahouti has quit IRC01:06
*** nlahouti has joined #puppet-openstack01:07
*** saneax is now known as saneax_AFK01:07
*** morazi has joined #puppet-openstack01:09
*** baker has joined #puppet-openstack01:14
openstackgerritMerged openstack/puppet-vswitch: Add require ovs_redhat.rb to ovs_redhat_el6.rb  https://review.openstack.org/27856001:16
*** baker has quit IRC01:17
*** angdraug has quit IRC01:20
*** morazi has quit IRC01:33
*** jasondotstar has quit IRC01:35
*** sid14726 has joined #puppet-openstack01:36
*** jasondotstar has joined #puppet-openstack01:37
*** gildub has quit IRC01:41
*** baker has joined #puppet-openstack01:59
*** ducttape_ has joined #puppet-openstack01:59
*** baker has quit IRC02:02
*** vinsh has quit IRC02:12
*** vinsh has joined #puppet-openstack02:13
*** foxhunt7z has joined #puppet-openstack02:13
*** openstackgerrit has quit IRC02:15
*** foxhunt_away has quit IRC02:15
*** vinsh has quit IRC02:18
*** vinsh has joined #puppet-openstack02:18
*** fvollero has quit IRC02:20
*** openstackgerrit has joined #puppet-openstack02:24
*** ducttape_ has quit IRC02:26
*** ducttape_ has joined #puppet-openstack02:27
openstackgerritAndy Botting proposed openstack/puppet-rally: Add config file settings and tests for benchmarking  https://review.openstack.org/27113802:28
*** baker has joined #puppet-openstack02:29
*** baker has quit IRC02:33
*** ducttape_ has quit IRC02:35
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454802:40
*** sid14726 has quit IRC02:47
*** rahulait has joined #puppet-openstack02:55
*** rahulait has quit IRC03:05
*** haplo37 has joined #puppet-openstack03:06
*** sid14726 has joined #puppet-openstack03:15
*** rahulait has joined #puppet-openstack03:17
*** haplo37 has quit IRC03:25
*** haplo37 has joined #puppet-openstack03:26
*** rahulait has quit IRC03:46
*** ducttape_ has joined #puppet-openstack03:58
*** rahulait has joined #puppet-openstack04:09
*** baker has joined #puppet-openstack04:13
*** sid14726 has quit IRC04:27
openstackgerritMatt Fischer proposed openstack/puppet-cinder: module to set quotas per volume type  https://review.openstack.org/19757204:28
*** ducttape_ has quit IRC04:29
mfischcrinkle: would love to discuss the admin token removal stuff with you tomorrow04:35
mfischjust saw the review04:35
crinklemfisch: sure04:38
crinklemfisch: I know you were working on something related so i'm not sure if mine is still necessary04:38
mfischI ended up using a new pipeline filter to only allow its usage from the keystone node itself04:38
mfischI left some comments in there about the v2 endpoints too04:38
mfischin the end we decided that admin token on disk or openrc on disk was the same difference really04:39
mfischcrinkle: this is what we used: https://gist.github.com/matthewfischer/b170ee96a9005d5081ca04:40
mfischlines 41-45 are the meat04:40
crinklemfisch: hmm that's not very puppetty, could that get proposed in upstream keystone?04:41
mfischI dont think so04:41
mfischI mean maybe?04:42
mfischI think that their response would be to disable it entirely04:42
crinklemy proposal is kind of janky because if anything goes wrong between disabling the token and adding an openrc file then puppet won't really be able to fix it04:43
mfischthats how I was stuck04:43
mfischI considered lots of crazy stuff04:43
mfischremoving all keystone resources and using ansible04:43
mfischdropping special flags for puppet04:43
mfischremoving the token and having the providers ignore auth failures after the 1st run04:43
mfischand a few more04:43
*** haplo37 has quit IRC04:44
mfischcrinkle: the main issue you have is that you're trying to solve a problem that puppet isnt great at solving04:46
mfischdoing something on a 2nd run special04:46
crinklethis should work on the first run04:47
mfischoh at the end of the keystone resources then04:49
crinklemfisch: when you say "it's completely safe to remove the v2 admin_token references" what do you mean? are you talking about the entire pipline:public_api and pipeline:admin_api sections?04:53
mfischasorry no04:53
mfischto remove admin_token_auth from those APIs04:53
mfischsafe in that puppet doesnt use the v2 APIs04:53
mfischonly puppet uses admin_token, puppet only uses v3, ergo you can remove admin_token_auth from v2 pipelines. First statement is NOT true for every deployment04:54
crinklewell we want it removed from the v3 pipeline too04:59
mfischfor sure04:59
mfischI guess I just mean you can do v2 for "free" without impacting puppet04:59
crinkleah ok05:00
*** markvoelker has quit IRC05:03
*** sid14726 has joined #puppet-openstack05:04
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/25624805:08
*** baker has quit IRC05:32
*** sid14726 has quit IRC05:39
*** sid14726 has joined #puppet-openstack05:49
*** fedexo has joined #puppet-openstack05:52
*** fedexo_ has joined #puppet-openstack05:53
*** nlahouti has joined #puppet-openstack05:57
*** nlahouti1 has joined #puppet-openstack05:59
*** cwolferh has quit IRC06:00
*** cwolferh has joined #puppet-openstack06:00
*** nlahouti has quit IRC06:01
*** markvoelker has joined #puppet-openstack06:04
*** jaosorior has joined #puppet-openstack06:10
*** sid14726 has quit IRC06:21
*** a7ndrew_ is now known as a7ndrew06:25
*** sid14726 has joined #puppet-openstack06:30
*** markvoelker has quit IRC06:34
*** saneax_AFK is now known as saneax06:53
*** nlahouti1 has quit IRC06:54
*** nlahouti has joined #puppet-openstack06:54
*** jaosorior has quit IRC06:56
*** rahulait has quit IRC07:03
*** rahulait has joined #puppet-openstack07:05
*** sid14726 has quit IRC07:05
*** fedexo has quit IRC07:14
*** sid14726 has joined #puppet-openstack07:15
*** jaosorior has joined #puppet-openstack07:20
*** jaosorior has quit IRC07:32
*** dgurtner has joined #puppet-openstack07:33
*** jtomasek has joined #puppet-openstack07:36
*** sid14726 has quit IRC07:36
*** leanderthal has joined #puppet-openstack07:36
*** sid14726 has joined #puppet-openstack07:45
*** hjensas has joined #puppet-openstack07:46
*** hjensas has quit IRC07:46
*** hjensas has joined #puppet-openstack07:46
*** rahulait has quit IRC07:49
*** richm has joined #puppet-openstack07:55
*** nlahouti has quit IRC08:02
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow changing list of processing hooks  https://review.openstack.org/27891608:04
*** jaosorior has joined #puppet-openstack08:05
*** sid14726 has quit IRC08:13
*** jpena has joined #puppet-openstack08:25
*** sid14726 has joined #puppet-openstack08:28
*** dgurtner has quit IRC08:29
*** yottatsa has joined #puppet-openstack08:29
*** rcernin has joined #puppet-openstack08:35
*** yottatsa has quit IRC08:43
*** openstackgerrit has quit IRC08:47
*** openstackgerrit_ has joined #puppet-openstack08:47
*** openstackgerrit_ is now known as openstackgerrit08:48
*** dgurtner has joined #puppet-openstack08:48
*** dgurtner has quit IRC08:48
*** dgurtner has joined #puppet-openstack08:48
*** yottatsa has joined #puppet-openstack08:50
*** jtomasek has quit IRC08:52
*** yottatsa has quit IRC08:52
*** yottatsa has joined #puppet-openstack08:53
*** rmart04 has joined #puppet-openstack08:54
*** jtomasek has joined #puppet-openstack08:57
*** derekh has joined #puppet-openstack09:00
*** sid14726 has quit IRC09:04
*** rmart04 has left #puppet-openstack09:05
*** yottatsa has quit IRC09:07
*** yottatsa has joined #puppet-openstack09:20
*** priteau has joined #puppet-openstack09:20
*** yottatsa has quit IRC09:24
*** sid14726 has joined #puppet-openstack09:24
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/22020809:25
*** markvoelker has joined #puppet-openstack09:31
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Remove use of template based config from cisco ml2 nexus plugin.  https://review.openstack.org/27895209:35
*** markvoelker has quit IRC09:36
*** paramite has joined #puppet-openstack09:38
*** gildub has joined #puppet-openstack09:38
*** gildub has quit IRC09:45
*** yottatsa has joined #puppet-openstack09:47
*** yottatsa has quit IRC09:54
*** gfidente has joined #puppet-openstack09:55
*** gfidente has quit IRC09:55
*** gfidente has joined #puppet-openstack09:55
*** yottatsa has joined #puppet-openstack09:59
*** yottatsa has quit IRC10:00
*** gildub has joined #puppet-openstack10:02
openstackgerritGael Chamoulaud proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/27896010:05
*** IvanBerezovskiy has joined #puppet-openstack10:12
openstackgerritMuhammad Salman Khan proposed openstack/puppet-neutron: Change of PLUMgrid plugin package and addition of new parameters.  https://review.openstack.org/27852210:13
*** sid14726 has quit IRC10:13
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/27896210:16
openstackgerritMerged openstack/puppet-ceph: Bump required version of puppetlabs-apt  https://review.openstack.org/27697010:20
*** gildub has quit IRC10:21
*** gildub has joined #puppet-openstack10:22
*** sid14726 has joined #puppet-openstack10:24
*** fedexo_ has quit IRC10:24
*** priteau has quit IRC10:27
*** priteau has joined #puppet-openstack10:33
*** paramite is now known as paramite|afk10:37
*** chem has joined #puppet-openstack10:39
*** morazi has joined #puppet-openstack10:41
*** paramite|afk is now known as paramite10:50
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495110:50
*** morazi_ has joined #puppet-openstack10:55
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495110:56
*** yottatsa has joined #puppet-openstack10:57
*** sid14726 has quit IRC10:57
*** morazi has quit IRC10:58
openstackgerritMerged openstack/puppet-nova: Allow to set max_concurrent_build for ironic compute  https://review.openstack.org/27642211:02
*** sid14726 has joined #puppet-openstack11:08
*** morazi_ has quit IRC11:15
*** _degorenko|afk is now known as degorenko11:19
*** markvoelker has joined #puppet-openstack11:32
*** markvoelker has quit IRC11:37
*** sid14726 has quit IRC11:41
*** paramite is now known as paramite|afk11:41
*** sid14726 has joined #puppet-openstack11:45
*** dprince has joined #puppet-openstack11:55
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/27126011:58
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Deprecate router[dhcp]_delete_namespaces parameter  https://review.openstack.org/27667412:09
*** gildub has quit IRC12:14
chemEmilienM: mfisch : degorenko: there is a bug in https://review.openstack.org/#/c/273513/ I'm not sure how to proceed as the code is not yet merged.  Can it be rollback or do I need to wait and make a bug report/review ?12:15
degorenkochem, don't worry, i will update change12:16
degorenkoit depends-on another change12:16
degorenkoit will not be merged12:16
chemdegorenko: ah, oki, cool :)12:17
chemdegorenko: sorry for the very late review ...12:17
*** sid14726 has quit IRC12:21
openstackgerritnarasimha18sv proposed openstack/puppet-neutron: Removing Keystone middleware configurations in neutron paste file  https://review.openstack.org/27901412:28
*** johnmilton has joined #puppet-openstack12:30
*** markvoelker has joined #puppet-openstack12:33
degorenkochem, no problem :)12:35
chemcool12:35
*** markvoelker has quit IRC12:37
EmilienMgood morning!12:37
degorenkochem,  btw, thanks for regexp :D12:39
degorenkoMorning :)12:39
chemdegorenko: you're welcome, kinda like regexp :)12:39
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/26492712:41
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495112:42
*** sid14726 has joined #puppet-openstack12:45
*** paramite|afk is now known as paramite12:47
openstackgerritMerged openstack/puppet-tripleo: Make haproxy balancer default options configurable  https://review.openstack.org/26529812:50
openstackgerritMerged openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6  https://review.openstack.org/26530912:51
EmilienMdprince: can you look https://review.openstack.org/#/c/278795/ please ?12:57
dprinceEmilienM: +A12:59
EmilienMthanks13:02
*** delattec has joined #puppet-openstack13:10
*** ducttape_ has joined #puppet-openstack13:11
*** jayg|g0n3 is now known as jayg13:12
*** delatte has quit IRC13:13
*** morazi has joined #puppet-openstack13:18
*** jtomasek_ has joined #puppet-openstack13:20
*** sid14726 has quit IRC13:21
*** Guest57497 is now known as zeus13:22
*** zeus has joined #puppet-openstack13:22
*** markvoelker has joined #puppet-openstack13:23
*** ducttape_ has quit IRC13:27
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Set default branch to stable/liberty in .gitreview  https://review.openstack.org/27905313:31
*** delatte has joined #puppet-openstack13:32
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option  https://review.openstack.org/27905513:32
*** duritong has quit IRC13:33
*** duritong has joined #puppet-openstack13:34
*** delattec has quit IRC13:35
*** jtomasek_ has quit IRC13:37
openstackgerritRadoslaw Smigielski proposed openstack/puppet-keystone: Update init.pp examples to something what really works  https://review.openstack.org/27906613:42
*** sid14726 has joined #puppet-openstack13:44
*** jpena is now known as jpena|lunch13:48
*** johnmilton has quit IRC13:59
*** saneax is now known as saneax_AFK14:02
*** richm has quit IRC14:03
*** jaosorior has quit IRC14:05
*** jaosorior has joined #puppet-openstack14:05
yolandahi, EmilienM or any other that may know... have you ever had problems with self-signed certs and requests14:09
yolandai'm having a funny problem with puppet-neutron14:09
EmilienMhello14:09
yolandafirst call of neutron net-list, shows an InsecurePlatformWarning14:09
yolandathen neutron tries to create a network called "InsecurePlatformWarning"14:10
EmilienMthere is a flag to enable IIRC, let me check14:10
yolandai tried setting REQUESTS_CA_BUNDLE, but no luck14:10
EmilienMmaybe auth_insecure ?14:10
EmilienMah no auth_insecure is for metadata agent14:10
yolandarequests is able to retrieve the content properly, but as it throws that warning first, neutron gets confused14:10
*** rcarrillocruz has joined #puppet-openstack14:11
EmilienMyolanda: the provider is confused?14:11
EmilienMthe puppet provider to create networks/subnets?14:11
yolandayep14:11
EmilienMyou found a bug then14:11
yolandalooks like14:11
yolandai have a log i can share14:11
EmilienMiirc we parse the output14:11
yolandait's on stable/liberty14:11
EmilienMcan you file a LP ?14:11
EmilienMthe bug is prob on master tooo14:12
yolandasure14:12
EmilienMI know chem and degorenko worked on this code a little, maybe they can tell if that's a bug14:12
yolandaok going to check in launchpad, for something similar, or file a new one14:12
degorenkowhat's code? :)14:13
mfischchem: unsure if it will be stopped or not14:13
yolandahi degorenko, in puppet-neutron14:14
degorenkoyolanda, hi, what's exactly wrong?14:14
yolandaso i am using a self signed cert for that14:14
yolandarequests works ok, but throws a warning first14:14
EmilienMdegorenko: I think she's having issue with providers14:14
yolandasomething like "InsecurePlatformWarning"14:14
yolandathen neutron goes and creates a network named InsecurePlatformWarning14:14
EmilienMyep, it sounds like a bug in our providers that manage neutron_network resource14:15
rcarrillocruzheh14:15
degorenkowow :)14:15
degorenkoprobably yeah14:15
EmilienMwhy don't you buy a certificate? :-P14:15
rcarrillocruz:D14:15
degorenkowe have had a bunch patches and reverts for neutron_network resource14:15
yolandagoing to file the bug and attach logs i have14:15
degorenkoyes, please :)14:16
degorenkoand also, can you please attach output from client?14:16
degorenkoopenstack or neutron14:16
yolandayep, i'll paste the logs14:16
degorenkonice14:16
yolandaand show you a sample of requests14:16
EmilienMyolanda: it's for infra-cloud, right?14:16
rcarrillocruzy14:17
yolandayep14:17
EmilienMwe'll try to fix it asap14:17
EmilienMI think we already had this issue in other providers14:18
EmilienMprobably in keystone14:18
EmilienMyolanda: how keystone is working?14:18
yolandaEmilienM, keystone looks good14:18
*** richm has joined #puppet-openstack14:18
EmilienMI suspect we need to patch openstacklib/lib/puppet/provider/openstack.rb14:19
yolandaEmilienM https://bugs.launchpad.net/puppet-neutron/+bug/154457314:21
openstackLaunchpad bug 1544573 in puppet-neutron "puppet-neutron not properly parsing output of requests when warnings are thrown" [Undecided,New]14:21
EmilienMyolanda: thanks14:22
yolandathanks for the quick support on that14:22
rcarrillocruz++14:23
degorenkoEmilienM, we have neutron cli for neutron providers14:23
EmilienMoh :)14:24
degorenkofor keystone it is openstack cli14:24
degorenko:)14:24
EmilienMwe don't use openstacklib14:24
degorenkoyes14:24
EmilienMok so yeah, the fix will be in puppet-neutron14:24
rcarrillocruzah right14:25
rcarrillocruzcos openstack cli doesn't have yet key neutron resources merged14:25
rcarrillocruzi remember seeing changes pending merge14:25
degorenkobut i think we anyway has wrong parsing in neutron14:25
degorenkoeven we use neutron cli14:25
EmilienMdegorenko: do you want to assign the bug to yourself? :-)14:25
degorenkono :D let me check before14:26
degorenkohow this problem is big14:26
EmilienMlol14:26
EmilienM"too big, I don't take it!"14:26
degorenkono :)14:26
degorenkojust want be sure14:26
degorenkothat i have enough time for this14:26
degorenkoi can help - yes14:26
yolandaoh that explains the different behaviour from others14:27
degorenkolooks like we need fix this function: https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L148-L16814:27
degorenkoand others14:28
degorenkoin this file14:28
degorenkolist14:28
degorenkoit will be really cool - when we will have all features of neutron client in openstack client14:28
degorenkoit will be easy to switch14:29
dprinceEmilienM: any ideas on this? http://paste.openstack.org/show/486695/14:29
yolandadegorenko, why neutron is not yet on openstack client? is there some blocker, or just lack of implementation?14:29
rcarrillocruzlack of implementation14:30
degorenkoyolanda, because not all functions of neutron cli have been moved to openstack clie14:30
rcarrillocruzchanges are on review14:30
degorenkorcarrillocruz, can you share link on this review here? :)14:30
rcarrillocruzsure, i can have a look14:30
EmilienMdprince: run 'bundle install' before14:30
EmilienMdprince: or 'bundle update'14:31
dprinceEmilienM: I just created it :)14:31
EmilienMah14:31
dprinceEmilienM: totally clean checkout, bundle install, etc.14:31
EmilienMlet me try14:31
degorenkoEmilienM, dprince also you may to set GEM_HOME var14:31
EmilienMright you need that ^14:32
rcarrillocruzthey had a bluprint opened to put all verbs from neutron14:32
rcarrillocruzhttps://review.openstack.org/#/q/topic:bp/neutron-client14:32
rcarrillocruzdegorenko: ^14:32
dprincedegorenko: okay, I don't set this for other projects. But I'll try14:32
rcarrillocruzsome merged14:32
rcarrillocruzsome pending14:32
degorenkodprince, set it via export GEM_HOME=<yuor bundler dir>14:33
degorenkorcarrillocruz, thanks14:33
openstackgerritGael Chamoulaud proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/27910014:33
EmilienMdprince, degorenko: I use that script https://gist.github.com/EmilienM/61d590a89c44364d043614:37
EmilienMvery dumb but helpful14:37
EmilienMyou can override the vesion of puppet in $114:37
degorenkoEmilienM, yes, i have the same, but also i have hack for choosing tests what i need :P14:37
EmilienMdegorenko: oh nice, please send a PR :-P14:38
degorenkoEmilienM, :P http://paste.openstack.org/show/486699/14:39
degorenkoEmilienM, yolanda  probably i found solution for neutron :)14:41
yolandadegorenko, that sounds good14:41
degorenkowe should use these functions: https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L225-L23614:41
degorenkothey already implemented for list call14:41
degorenkoyolanda, if i will upload patch - do you have some env to check it?14:42
yolandadegorenko yes14:42
degorenkonice14:42
yolandawilling to check it :)14:42
EmilienMnice catch14:42
EmilienMdegorenko: I'm stealing your code :-P14:43
degorenkono problem ;)14:43
rcarrillocruzdegorenko: that was quick, thx :-)14:44
openstackgerritGael Chamoulaud proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/27911014:48
openstackgerritDenis Egorenko proposed openstack/puppet-neutron: Clean up neutron cli output from warning lines  https://review.openstack.org/27911114:49
degorenkoyolanda, ^^^ it's still wip, all neutron calls should contain it14:49
degorenkojust to be sure that it will help14:49
*** sid14726 has quit IRC14:50
yolandadegorenko, i can test it in a sec14:51
*** xarses_ has quit IRC14:56
*** johnmilton has joined #puppet-openstack14:56
degorenkoyolanda, it will not help :(14:58
degorenkowe have shell output14:58
degorenkofunction is for csv14:58
degorenkolooks like we need new function *facepalm*14:59
degorenkoEmilienM, ^14:59
yolandai went just direct and i exported PYTHONWARNINGS=ignore :)14:59
*** ducttape_ has joined #puppet-openstack14:59
*** jpena|lunch is now known as jpena15:05
*** jaosorior has quit IRC15:06
*** morazi has quit IRC15:08
degorenkoEmilienM, yolanda may be you know, is here one common pattern for cli warnings? i mean, all of them are begin with pass to some file?15:11
EmilienMi think the pattern comes from python-requests15:11
EmilienMbut I'm not sure15:11
degorenkojust trying to find solution for this problem15:12
degorenkohow to detect warning15:12
degorenkomay be by leading '/' in the line?15:12
yolandadegorenko, i found a pair of warnings15:12
yolandawith requests 2.7.0, it starts with InsecurePlatformWarning15:12
yolandabut on requests 2.9.0, i can see some SNI warning first15:12
*** paramite is now known as paramite|afk15:13
degorenkoyolanda, you also have '/usr/lib/python2.7/dist-packages/urllib3/util/ssl_.py:97: InsecurePlatformWarning:'15:13
degorenkoand then warning meesage15:13
degorenkoi don't care about type of warning15:13
*** paramite|afk is now known as paramite15:13
yolandadegorenko let me retry , i think it's a bit different15:13
degorenkofor your warning we have pass to file, where it was spawn15:13
EmilienMactually not in requests but urllib315:14
EmilienMmy bad15:14
*** paramite is now known as paramite|afk15:14
degorenkoand it seems, that all cli warnings have same pattern - /path/to/file:line typewarning: message15:14
degorenkofor this case we can just drop all line with leading '/'15:14
EmilienMmaybe we can parse "InsecurePlatformWarning"15:15
degorenkonot case15:15
degorenkowe can have some DeprecationWarning15:15
degorenkoor have such network :D15:15
degorenkofor fun15:15
rcarrillocruzlol15:15
degorenkobut we don't have properties with name beginning with '/'15:16
yolandaEmilienM, degorenko, how do you do your tests? we are going to use self-signed certs instead of relying on an infra CA15:20
EmilienMyolanda: we don't have SSL enabled in our CI15:20
yolandahow that will work with puppet-openstack?15:20
yolandaoh15:20
degorenkoyolanda, just using warnings from keystone :D and simple output from neutron15:20
EmilienMyolanda: why not using infra CA?15:20
degorenkoon my env15:20
yolandadegorenko15:20
yolandaNotice: Puppet::Type::Neutron_network::ProviderNeutron: Unable to complete neutron request due to non-fatal error: "Execution of '/usr/bin/neutron net-list --format=csv --column=id --quote=none' returned 1: /usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/util/ssl_.py:315: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause th15:20
yolandae server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning.15:20
yolanda  SNIMissingWarning15:20
yolandathat's first error i have when i upgrade to requests 2.9.015:21
yolandaEmilienM, that comes from conversations in infra channel, seems better to go simple and use simple self-signed certs15:21
degorenkoso, we have warning which can use my pattern :D15:21
yolandaunless there is a good reason for not doing that (for example if that won't work...)15:21
EmilienMnot sure that's simple, regarding warnings we have now15:21
degorenkopattern /path/to/file:line typewarning: message15:22
rcarrillocruzyeah, probably just match 'Warning' ?15:22
degorenkono15:22
degorenkowe can have some properties15:22
*** nlahouti has joined #puppet-openstack15:22
degorenkowith value warning15:22
degorenkowe can use some regexp, based on pattern above for csv and shell15:23
*** ricardo_ has joined #puppet-openstack15:24
*** ricardo_ has quit IRC15:25
*** nlahouti has quit IRC15:27
degorenkoyolanda, EmilienM, rcarrillocruz may be regexp: ^\/.*:\s\w+Warning:.*$15:29
degorenkohttp://rubular.com/r/F88PA0RNhF15:29
*** angdraug has joined #puppet-openstack15:29
rcarrillocruzhmm, yeah15:30
rcarrillocruzthat could work15:30
EmilienMnice15:31
yolanda++15:32
*** xarses_ has joined #puppet-openstack15:35
*** dragomir has joined #puppet-openstack15:35
*** dragomir has quit IRC15:37
degorenkono, it will not work again. if some warning will move message to next line15:37
*** dragomir has joined #puppet-openstack15:40
*** dgurtner has quit IRC15:41
*** dragomir has quit IRC15:42
colonwqone of my patches (254548) failed scenario001 on centos due to a gnocchi error. How can I get my patch retested without submitting an update to the patch?15:46
*** dmburmistrov has joined #puppet-openstack15:47
degorenkorecheck - in comment15:47
EmilienMcolonwq: what says logs?15:47
degorenkoEmilienM, gnocchi error :)15:47
EmilienMdegorenko: I want to see a trace15:47
EmilienMlike, an URL so we can debug15:48
colonwqone moment15:48
colonwqhttp://logs.openstack.org/48/254548/24/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/18fa099/console.html#_2016-02-11_03_13_34_75215:49
degorenkoyolanda, can you put few outputs to paste openstack from neutron with warnings for me?15:49
colonwqhad to get browser on the right computer15:50
*** xarses_ is now known as xarses15:51
yolandadegorenko, the two i pasted earlier, i don't have more15:51
degorenkono, i mean :) from cmd directly with output15:52
*** rochaporto has joined #puppet-openstack15:52
degorenkoyolanda, like: http://paste.openstack.org/show/486715/15:52
yolandaah just a sec, because i wiped my testenv :(15:52
degorenkono problem15:52
*** higgins has quit IRC15:53
*** EmilienM has quit IRC15:53
yolandaah degorenko , i also need to auth properly to get that. Because as i run on puppet, this output is not shown. To run from cmd i need to create some auth fle15:55
degorenkocan you do that? :)15:55
*** yottatsa has quit IRC15:56
colonwqdegorenko, would the recheck go in the Jenkins comment or the git commit message?15:56
rcarrillocruzcolonwq: gerrit change comment15:56
degorenkocolonwq, gerrit comment15:56
degorenko:)15:56
colonwqThanks everyone.15:57
*** thumpba has joined #puppet-openstack15:57
yolandayep, redeploying the test env now15:57
*** paramite|afk is now known as paramite15:58
*** rcarrillocruz has left #puppet-openstack16:00
*** thumpba has quit IRC16:01
*** thumpba_ has joined #puppet-openstack16:02
yolandadegorenko, question.. if we know that output can come on a json format, why not strip everything tha comes before real json content?16:05
degorenkoyolanda, we can't use json :(16:05
degorenkowe have issue with it16:05
degorenkoyolanda, see this patch https://review.openstack.org/#/c/262809/16:06
*** EmilienM has joined #puppet-openstack16:06
*** thumpba_ has quit IRC16:06
yolandaoh16:06
*** higgins has joined #puppet-openstack16:09
*** dgurtner has joined #puppet-openstack16:09
degorenkoyolanda, so until we have openstack client with neutron features - we should have some workaround for neutron :(16:10
yolandaok running puppet code again, i had to redeploy16:11
openstackgerritGael Chamoulaud proposed openstack/puppet-heat: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/27915416:12
*** haplo37 has joined #puppet-openstack16:13
*** darkxploit has joined #puppet-openstack16:14
*** leanderthal is now known as leanderthal|afk16:15
*** bnemec has quit IRC16:21
*** mdorman has joined #puppet-openstack16:21
degorenkochem, according to your second comment here: https://review.openstack.org/#/c/273513/8/lib/puppet/type/cinder_type.rb16:25
degorenkoit will not work for: key=value1,value216:25
chemdegorenko: the regex I propose won't work for this16:28
chemdegorenko: correct16:28
chemdegorenko: is that a requirement ?16:29
degorenkochem, we can have key and couple of values for them16:29
chemdegorenko: if so, it's easy to modify the regex16:29
chemdegorenko: 2s16:29
chemdegorenko: the first regex won't work either ... let me adjust it16:30
degorenkochem, first fine, what's wrong with him?16:30
chemdegorenko: sorry you're right, nothing wrong with it...16:30
degorenkoЖ)16:31
degorenko:)16:31
chemdegorenko:oki, I confuse the first and second comment.  Now for the second one /your/ regex won't work if no space is after the comma.16:32
degorenkoyeah16:32
degorenkothat's why my regexp reject such lines16:33
chemdegorenko: string actually, not regex16:33
chemdegorenko: just use split(/\s*,\s*/) and you'll be fine16:33
degorenkoi'm for first one16:33
degorenkono will not :)16:33
degorenko<degorenko> it will not work for: key=value1,value216:33
chemdegorenko: so value = "value1,value2", correct ?16:34
*** foxhunt7z is now known as foxhunt_away16:35
degorenkokey="value1,value2"16:35
degorenkothat's why i have space between keys16:35
chemdegorenko: oh, you mean you can have 'key1=value1,value2 ,key2=value3, ...'16:36
degorenkoнуз16:36
degorenkoyep16:36
degorenko:D16:36
chemhehe16:37
chemack16:37
degorenkorussian letters, sorry16:37
chemdegorenko: that's tricky :)16:37
degorenkoyes16:37
degorenko:)16:37
*** holser_ has quit IRC16:38
degorenkoeasier way always have space :) like key=v,1,2, key2=s,sd, key3=v316:38
*** mwhahaha has quit IRC16:39
*** richm has quit IRC16:39
*** holser_ has joined #puppet-openstack16:41
*** mwhahaha has joined #puppet-openstack16:43
chemdegorenko: hum, as <value> can have comma, I can't think of something else without changing the type from a string to an array ...16:43
degorenkoyes, it makes me cry :D16:43
chemdegorenko: I think It may be late for such a change ... dunno :)16:43
degorenkowhy?16:44
degorenkowe can have input as array, then mungle it to hash and that's it16:45
chemdegorenko: for no reason :)16:45
degorenkoah16:45
degorenkoso, how then we can solve this problem?16:45
chemdegorenko: yeap ^^16:45
chemdegorenko: array -> mungle -> hash -> done :)16:46
chemdegorenko: the regex will be then much easier, and you won't have to explain that the ", " is mandatory16:46
degorenkoyep :)16:46
degorenkoand having an array is not so complex i guess for properties16:47
degorenkostring/array - no big difference16:47
chemdegorenko let me find you an example  ... 2s16:47
degorenkoit's no problem with array16:47
degorenkochem, and we still can have validation on which item of array :)16:48
degorenkoon each*16:48
chemdegorenko: yes : check this https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_user_role.rb#L64-L7016:49
degorenkoyep :) i remember about issync16:50
degorenkoinsync16:50
chemdegorenko: yep, required to not depend on order16:50
degorenkoyes16:51
dmsimardEmilienM: tempest revert merged16:52
dmsimardEmilienM: and we got scenario002 to pass on weirdo (delorean current), did you pin tempest ?16:52
chemdegorenko: you need the :array_matching => :all as well as the default behavior of puppet is strang https://docs.puppetlabs.com/guides/custom_types.html#customizing-behaviour16:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "pin tempest to keep setuping heat roles"  https://review.openstack.org/27918016:53
degorenkoi know :P16:53
EmilienMdmsimard: ^16:53
dmsimardEmilienM: ok16:53
EmilienMI hope it pass16:53
EmilienMdmsimard: fyi, we bumped again to a very recent url16:53
EmilienMso we don't wait for promotions16:53
dmsimardEmilienM: sure16:53
dmsimardEmilienM: still working on packstack and tripleo .. :(16:54
*** holser_ is now known as holser16:54
chemdegorenko: you will have to modify the l53 of the provider as well16:54
degorenkochem, btw, do i need validate method? or i can use string validate in munge section?16:54
chemdegorenko : hum .. no sorry ^16:54
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614116:54
degorenkochem, you don't get it or i don't need validate method? :D16:55
chemdegorenko for the validate you should get one yes16:55
chemdegorenko validate and munge at not done at the same time in the processing16:55
degorenkoso oh, that's good to know16:55
*** rcernin has quit IRC16:56
degorenkobut both of them provider methods16:56
degorenkowhy we can't use them in same time?16:56
*** bnemec has joined #puppet-openstack16:56
chemdegorenko:you can use them at the same time, it's just that values are validated first and *then* munged16:57
degorenkooh, no, that's of course16:58
degorenkoi meant, another thing16:58
degorenkothat on munge stage i have value and i'll iterate on this value (array items)16:58
degorenkoi  can call item.match(regexp)16:58
degorenko^ that was i meant16:58
degorenkobut it will be in one cycle16:59
chemdegorenko: I would have to check, but I think munge get the array, not its element16:59
degorenkoyes :D16:59
degorenkolet me clarify16:59
degorenkoyou don't get it16:59
chemdegorenko:oki, i got it16:59
degorenkosec16:59
chemdegorenko: I see what you mean, like failing during munge to not have to iterate twice over the array17:00
degorenkoyeah :)17:00
degorenkois it good?17:00
*** samuelBartel has joined #puppet-openstack17:01
colonwqI have an rspec question: Can I do "expect(obj).not_to exist"? I want to make sure a setting is not in a config file.17:01
*** dgurtner has quit IRC17:01
chemdegorenko: If it was my patch, I would waste some cycle.  But you can try it that way and check how it works17:01
degorenkocolonwq, is your question for unit test or for specs tests?17:01
*** nlahouti has joined #puppet-openstack17:02
colonwqdegorenko, spec I believe. I am learning the terms you guys use.17:02
degorenkocolonwq, it is for puppet manifest or for provider?17:02
colonwqI am writing a spec/defines/ceph_rgw_civetweb_spec.rb to test my civetweb settings.17:03
chemcolonwq: if obj.exists? make sense, then you can expect(obj).not_to exists17:03
degorenkoah i  see17:03
chemcolonwq: 95% sure :)17:03
degorenkocolonwq, then you can use https://github.com/openstack/puppet-sahara/blob/master/spec/classes/sahara_init_spec.rb#L16717:03
degorenkolike that17:03
degorenkois_expected.to_not contain_17:03
colonwqah. That is easier and shorter.17:04
colonwqchem, degorenko Thanks for the pointers. I am sure I'll ask more questions that are simple.17:05
degorenkochem, so you'd prefer one more cycle, right?17:05
degorenkocolonwq, you're welcome17:06
chemcolonwq: for reference "expect(actual).to exist        # passes if actual.exist? and/or actual.exists? are truthy"17:06
chemcolonwq: if one day you need it17:06
chemdegorenko: yep17:06
chemdegorenko: easier to read17:07
degorenkochem, ack, thanks for help man o/17:07
chemdegorenko: np :)17:08
*** baker has joined #puppet-openstack17:11
*** derekh has quit IRC17:12
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614117:12
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614117:14
*** darkxploit has quit IRC17:18
openstackgerritDenis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers  https://review.openstack.org/27351317:26
degorenkochem, ^ should be good now17:28
chemdegorenko: looking17:30
openstackgerritBen Nemec proposed openstack/puppet-tripleo: Handle redirects for Horizon  https://review.openstack.org/26828817:32
*** samuelBartel has quit IRC17:33
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614117:37
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614117:37
*** sid14726 has joined #puppet-openstack17:39
openstackgerritDenis Egorenko proposed openstack/puppet-neutron: Add clean up for shell format for neutron output  https://review.openstack.org/27911117:42
degorenkoyolanda, now it should work ^17:43
degorenkochem, we have problem with neutron output :) in case you have time ^17:43
chemdegorenko: reviewed :)17:48
degorenkochem, thanks :) i will update it tomorrow17:49
*** paramite has quit IRC17:50
chemdegorenko: looking into the neutron stuff.17:51
chemdegorenko: have a good one and see you tomorrow17:51
degorenkochem, i'm here for ~10-15 minutes yet17:52
chemdegorenko: same here :)17:52
degorenko:D same what?17:53
*** rochaporto has quit IRC18:01
chemdegorenko: I'm leaving as well :)  I've reviewed the neutron patch.18:03
degorenkochem, thanks again, good night :) see you tomorrow18:04
chemdegorenko:see ya18:04
*** degorenko is now known as _degorenko|afk18:04
*** sid14726 has quit IRC18:07
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454818:09
openstackgerritMerged openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/26492718:13
*** jpena is now known as jpena|off18:15
*** rcernin has joined #puppet-openstack18:16
*** rmart04 has joined #puppet-openstack18:22
*** rmart04 has quit IRC18:27
*** IvanBerezovskiy has left #puppet-openstack18:27
*** chem has quit IRC18:38
*** angdraug has quit IRC18:39
*** dgurtner has joined #puppet-openstack18:40
*** dgurtner has quit IRC18:40
*** dgurtner has joined #puppet-openstack18:40
*** nlahouti has quit IRC18:51
*** nlahouti has joined #puppet-openstack19:03
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Update supported version information  https://review.openstack.org/27756219:04
*** nlahouti has quit IRC19:24
*** nlahouti has joined #puppet-openstack19:29
*** dkaigarodsev_ has quit IRC19:32
*** nlahouti has quit IRC19:32
*** dkaigarodsev has joined #puppet-openstack19:34
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/25624819:43
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/27614119:46
*** nlahouti has joined #puppet-openstack19:52
*** vinsh has quit IRC20:00
*** vinsh has joined #puppet-openstack20:00
*** vinsh_ has joined #puppet-openstack20:07
*** vinsh has quit IRC20:07
*** dgurtner has quit IRC20:18
*** rcernin has quit IRC20:24
*** mewald has joined #puppet-openstack20:25
mewaldDoes puppet-ceph work with infernalis? My monitors are not starting and ceph.conf looks broken. Any experiences?20:27
*** gchamoul has quit IRC20:27
*** rmart04 has joined #puppet-openstack20:28
*** gchamoul has joined #puppet-openstack20:30
*** jtomasek has quit IRC20:31
*** rmart04 has quit IRC20:32
*** nlahouti has quit IRC20:32
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option  https://review.openstack.org/27905520:36
*** gfidente has quit IRC20:38
openstackgerritMerged openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone admin endpoint  https://review.openstack.org/26741720:40
*** sid14726 has joined #puppet-openstack20:57
xarsesmewald: there was a note from EmilienM that it likely did not21:01
*** richm has joined #puppet-openstack21:01
xarsesmewald: its most likely due to the change to support systemd init21:03
xarsesand if you paste ceph.conf I can probably triage it21:04
xarsesbut afaiu there was nothing that should break infernalis there21:04
*** jpena|off has quit IRC21:14
*** jfluhmann has joined #puppet-openstack21:20
*** dprince has quit IRC21:31
*** morazi has joined #puppet-openstack21:31
*** sid14726 has quit IRC21:42
*** richm has quit IRC21:42
*** johnmilton has quit IRC21:42
yolandadegorenko, sorry, i finished with my testenv just now. I will test your patches tomorrow21:43
*** rahulait has joined #puppet-openstack21:44
mewaldxarses: thx, that helped :)21:57
mewalddoes it work to use LVM logical volumes as journal disks?21:57
*** delatte has quit IRC22:00
*** _ducttape_ has joined #puppet-openstack22:00
*** jayg is now known as jayg|g0n322:02
*** ducttape_ has quit IRC22:03
*** nlahouti has joined #puppet-openstack22:06
xarsesmewald: it should work with either a raw device (LVM LV counts) or a formatted filesystem on a partation, or a mounted path if you really want22:06
*** johnmilton has joined #puppet-openstack22:09
*** johnmilton has quit IRC22:10
*** johnmilton has joined #puppet-openstack22:10
openstackgerritSam Morrison proposed openstack/puppet-neutron: Support midonet type driver for ML2  https://review.openstack.org/27820722:29
*** sid14726 has joined #puppet-openstack22:32
*** andybotting has joined #puppet-openstack22:35
*** _ducttape_ has quit IRC22:37
*** ducttape_ has joined #puppet-openstack22:38
*** morazi has quit IRC22:42
*** cdelatte has joined #puppet-openstack22:42
*** jfluhmann has quit IRC22:51
*** gildub has joined #puppet-openstack23:03
*** xarses has quit IRC23:11
*** johnmilton has quit IRC23:17
*** sid14726 has quit IRC23:18
*** saneax_AFK is now known as saneax23:19
*** priteau has quit IRC23:25
*** priteau_ has joined #puppet-openstack23:25
*** mewald has quit IRC23:25
*** priteau_ has quit IRC23:27
*** baker has quit IRC23:31
*** priteau has joined #puppet-openstack23:32
*** priteau has quit IRC23:34
*** xarses has joined #puppet-openstack23:36
*** mewald has joined #puppet-openstack23:37
*** mewald has quit IRC23:42
*** haplo37 has quit IRC23:51
*** mewald has joined #puppet-openstack23:53
*** ducttape_ has quit IRC23:55
*** mewald has quit IRC23:57

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