Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-02-11

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:03 openstackgerrit Merged openstack/puppet-heat: Support setting instance_user to an empty string  https://review.openstack.org/278450
00:03 crinkle_ joined #puppet-openstack
00:03 openstackgerrit Merged openstack/puppet-neutron: Creation of neutron::db::sync  https://review.openstack.org/278458
00:04 openstackgerrit Merged openstack/puppet-neutron: Use uppercase 'S' in word "OpenStack"  https://review.openstack.org/276135
00:08 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/256248
00:09 ducttape_ joined #puppet-openstack
00:19 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/256248
00:32 xarses joined #puppet-openstack
00:35 openstackgerrit Merged openstack/puppet-neutron: Provide support for VPNaaS service_provider  https://review.openstack.org/273934
00:43 xarses joined #puppet-openstack
00:50 baker joined #puppet-openstack
00:54 xarses joined #puppet-openstack
00:56 xarses_ joined #puppet-openstack
01:00 baker_ joined #puppet-openstack
01:06 openstackgerrit Merged openstack/puppet-trove: Use uppercase 'S' in word "OpenStack"  https://review.openstack.org/276607
01:07 nlahouti joined #puppet-openstack
01:09 morazi joined #puppet-openstack
01:14 baker joined #puppet-openstack
01:15 openstackgerrit Merged openstack/puppet-vswitch: Add require ovs_redhat.rb to ovs_redhat_el6.rb  https://review.openstack.org/278560
01:36 sid14726 joined #puppet-openstack
01:37 jasondotstar joined #puppet-openstack
01:59 baker joined #puppet-openstack
01:59 ducttape_ joined #puppet-openstack
02:13 vinsh joined #puppet-openstack
02:13 foxhunt7z joined #puppet-openstack
02:18 vinsh joined #puppet-openstack
02:24 openstackgerrit joined #puppet-openstack
02:27 ducttape_ joined #puppet-openstack
02:28 openstackgerrit Andy Botting proposed openstack/puppet-rally: Add config file settings and tests for benchmarking  https://review.openstack.org/271138
02:29 baker joined #puppet-openstack
02:40 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
02:55 rahulait joined #puppet-openstack
03:06 haplo37 joined #puppet-openstack
03:15 sid14726 joined #puppet-openstack
03:17 rahulait joined #puppet-openstack
03:26 haplo37 joined #puppet-openstack
03:58 ducttape_ joined #puppet-openstack
04:09 rahulait joined #puppet-openstack
04:13 baker joined #puppet-openstack
04:28 openstackgerrit Matt Fischer proposed openstack/puppet-cinder: module to set quotas per volume type  https://review.openstack.org/197572
04:35 mfisch crinkle: would love to discuss the admin token removal stuff with you tomorrow
04:35 mfisch just saw the review
04:38 crinkle mfisch: sure
04:38 crinkle mfisch: I know you were working on something related so i'm not sure if mine is still necessary
04:38 mfisch I ended up using a new pipeline filter to only allow its usage from the keystone node itself
04:38 mfisch I left some comments in there about the v2 endpoints too
04:39 mfisch in the end we decided that admin token on disk or openrc on disk was the same difference really
04:39 mfisch crinkle: this is what we used: https://gist.github.com/matthewfischer/b170ee96a9005d5081ca
04:40 mfisch lines 41-45 are the meat
04:41 crinkle mfisch: hmm that's not very puppetty, could that get proposed in upstream keystone?
04:41 mfisch I dont think so
04:42 mfisch I mean maybe?
04:42 mfisch I think that their response would be to disable it entirely
04:43 crinkle my 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 it
04:43 mfisch thats how I was stuck
04:43 mfisch I considered lots of crazy stuff
04:43 mfisch removing all keystone resources and using ansible
04:43 mfisch dropping special flags for puppet
04:43 mfisch removing the token and having the providers ignore auth failures after the 1st run
04:43 mfisch and a few more
04:46 mfisch crinkle: the main issue you have is that you're trying to solve a problem that puppet isnt great at solving
04:46 mfisch doing something on a 2nd run special
04:47 crinkle this should work on the first run
04:49 mfisch oh at the end of the keystone resources then
04:53 crinkle mfisch: 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 mfisch asorry no
04:53 mfisch to remove admin_token_auth from those APIs
04:53 mfisch safe in that puppet doesnt use the v2 APIs
04:54 mfisch only 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 deployment
04:59 crinkle well we want it removed from the v3 pipeline too
04:59 mfisch for sure
04:59 mfisch I guess I just mean you can do v2 for "free" without impacting puppet
05:00 crinkle ah ok
05:04 sid14726 joined #puppet-openstack
05:08 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/256248
05:49 sid14726 joined #puppet-openstack
05:52 fedexo joined #puppet-openstack
05:53 fedexo_ joined #puppet-openstack
05:57 nlahouti joined #puppet-openstack
05:59 nlahouti1 joined #puppet-openstack
06:00 cwolferh joined #puppet-openstack
06:04 markvoelker joined #puppet-openstack
06:10 jaosorior joined #puppet-openstack
06:30 sid14726 joined #puppet-openstack
06:54 nlahouti joined #puppet-openstack
07:05 rahulait joined #puppet-openstack
07:15 sid14726 joined #puppet-openstack
07:20 jaosorior joined #puppet-openstack
07:33 dgurtner joined #puppet-openstack
07:35 jtomasek joined #puppet-openstack
07:36 leanderthal joined #puppet-openstack
07:45 sid14726 joined #puppet-openstack
07:46 hjensas joined #puppet-openstack
07:46 hjensas joined #puppet-openstack
07:55 richm joined #puppet-openstack
08:04 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow changing list of processing hooks  https://review.openstack.org/278916
08:05 jaosorior joined #puppet-openstack
08:25 jpena joined #puppet-openstack
08:28 sid14726 joined #puppet-openstack
08:29 yottatsa joined #puppet-openstack
08:34 rcernin joined #puppet-openstack
08:47 openstackgerrit_ joined #puppet-openstack
08:48 dgurtner joined #puppet-openstack
08:50 yottatsa joined #puppet-openstack
08:53 yottatsa joined #puppet-openstack
08:54 rmart04 joined #puppet-openstack
08:57 jtomasek joined #puppet-openstack
09:00 derekh joined #puppet-openstack
09:05 rmart04 left #puppet-openstack
09:20 yottatsa joined #puppet-openstack
09:20 priteau joined #puppet-openstack
09:24 sid14726 joined #puppet-openstack
09:25 openstackgerrit Gael Chamoulaud proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/220208
09:31 markvoelker joined #puppet-openstack
09:35 openstackgerrit Gael Chamoulaud proposed openstack/puppet-neutron: Remove use of template based config from cisco ml2 nexus plugin.  https://review.openstack.org/278952
09:38 paramite joined #puppet-openstack
09:38 gildub joined #puppet-openstack
09:47 yottatsa joined #puppet-openstack
09:55 gfidente joined #puppet-openstack
09:59 yottatsa joined #puppet-openstack
10:02 gildub joined #puppet-openstack
10:05 openstackgerrit Gael Chamoulaud proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/278960
10:12 IvanBerezovskiy joined #puppet-openstack
10:13 openstackgerrit Muhammad Salman Khan proposed openstack/puppet-neutron: Change of PLUMgrid plugin package and addition of new parameters.  https://review.openstack.org/278522
10:16 openstackgerrit Gael Chamoulaud proposed openstack/puppet-neutron: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/278962
10:20 openstackgerrit Merged openstack/puppet-ceph: Bump required version of puppetlabs-apt  https://review.openstack.org/276970
10:22 gildub joined #puppet-openstack
10:24 sid14726 joined #puppet-openstack
10:33 priteau joined #puppet-openstack
10:39 chem joined #puppet-openstack
10:41 morazi joined #puppet-openstack
10:50 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
10:55 morazi_ joined #puppet-openstack
10:56 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
10:57 yottatsa joined #puppet-openstack
11:02 openstackgerrit Merged openstack/puppet-nova: Allow to set max_concurrent_build for ironic compute  https://review.openstack.org/276422
11:08 sid14726 joined #puppet-openstack
11:32 markvoelker joined #puppet-openstack
11:45 sid14726 joined #puppet-openstack
11:55 dprince joined #puppet-openstack
11:58 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/271260
12:09 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Deprecate router[dhcp]_delete_namespaces parameter  https://review.openstack.org/276674
12:15 chem EmilienM: 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:16 degorenko chem, don't worry, i will update change
12:16 degorenko it depends-on another change
12:16 degorenko it will not be merged
12:17 chem degorenko: ah, oki, cool :)
12:17 chem degorenko: sorry for the very late review ...
12:28 openstackgerrit narasimha18sv proposed openstack/puppet-neutron: Removing Keystone middleware configurations in neutron paste file  https://review.openstack.org/279014
12:30 johnmilton joined #puppet-openstack
12:33 markvoelker joined #puppet-openstack
12:35 degorenko chem, no problem :)
12:35 chem cool
12:37 EmilienM good morning!
12:39 degorenko chem,  btw, thanks for regexp :D
12:39 degorenko Morning :)
12:39 chem degorenko: you're welcome, kinda like regexp :)
12:41 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/264927
12:42 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
12:45 sid14726 joined #puppet-openstack
12:50 openstackgerrit Merged openstack/puppet-tripleo: Make haproxy balancer default options configurable  https://review.openstack.org/265298
12:51 openstackgerrit Merged openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6  https://review.openstack.org/265309
12:57 EmilienM dprince: can you look https://review.openstack.org/#/c/278795/ please ?
12:59 dprince EmilienM: +A
13:02 EmilienM thanks
13:10 delattec joined #puppet-openstack
13:11 ducttape_ joined #puppet-openstack
13:18 morazi joined #puppet-openstack
13:20 jtomasek_ joined #puppet-openstack
13:22 zeus joined #puppet-openstack
13:23 markvoelker joined #puppet-openstack
13:31 openstackgerrit Giulio Fidente proposed openstack/puppet-nova: Set default branch to stable/liberty in .gitreview  https://review.openstack.org/279053
13:32 delatte joined #puppet-openstack
13:32 openstackgerrit Giulio Fidente proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option  https://review.openstack.org/279055
13:34 duritong joined #puppet-openstack
13:42 openstackgerrit Radoslaw Smigielski proposed openstack/puppet-keystone: Update init.pp examples to something what really works  https://review.openstack.org/279066
13:44 sid14726 joined #puppet-openstack
14:05 jaosorior joined #puppet-openstack
14:09 yolanda hi, EmilienM or any other that may know... have you ever had problems with self-signed certs and requests
14:09 yolanda i'm having a funny problem with puppet-neutron
14:09 EmilienM hello
14:09 yolanda first call of neutron net-list, shows an InsecurePlatformWarning
14:10 yolanda then neutron tries to create a network called "InsecurePlatformWarning"
14:10 EmilienM there is a flag to enable IIRC, let me check
14:10 yolanda i tried setting REQUESTS_CA_BUNDLE, but no luck
14:10 EmilienM maybe auth_insecure ?
14:10 EmilienM ah no auth_insecure is for metadata agent
14:10 yolanda requests is able to retrieve the content properly, but as it throws that warning first, neutron gets confused
14:11 rcarrillocruz joined #puppet-openstack
14:11 EmilienM yolanda: the provider is confused?
14:11 EmilienM the puppet provider to create networks/subnets?
14:11 yolanda yep
14:11 EmilienM you found a bug then
14:11 yolanda looks like
14:11 yolanda i have a log i can share
14:11 EmilienM iirc we parse the output
14:11 yolanda it's on stable/liberty
14:11 EmilienM can you file a LP ?
14:11 EmilienM the bug is prob on master tooo
14:12 yolanda sure
14:12 EmilienM I know chem and degorenko worked on this code a little, maybe they can tell if that's a bug
14:12 yolanda ok going to check in launchpad, for something similar, or file a new one
14:13 degorenko what's code? :)
14:13 mfisch chem: unsure if it will be stopped or not
14:13 yolanda hi degorenko, in puppet-neutron
14:14 degorenko yolanda, hi, what's exactly wrong?
14:14 yolanda so i am using a self signed cert for that
14:14 yolanda requests works ok, but throws a warning first
14:14 EmilienM degorenko: I think she's having issue with providers
14:14 yolanda something like "InsecurePlatformWarning"
14:14 yolanda then neutron goes and creates a network named InsecurePlatformWarning
14:15 EmilienM yep, it sounds like a bug in our providers that manage neutron_network resource
14:15 rcarrillocruz heh
14:15 degorenko wow :)
14:15 degorenko probably yeah
14:15 EmilienM why don't you buy a certificate? :-P
14:15 rcarrillocruz :D
14:15 degorenko we have had a bunch patches and reverts for neutron_network resource
14:15 yolanda going to file the bug and attach logs i have
14:16 degorenko yes, please :)
14:16 degorenko and also, can you please attach output from client?
14:16 degorenko openstack or neutron
14:16 yolanda yep, i'll paste the logs
14:16 degorenko nice
14:16 yolanda and show you a sample of requests
14:16 EmilienM yolanda: it's for infra-cloud, right?
14:17 rcarrillocruz y
14:17 yolanda yep
14:17 EmilienM we'll try to fix it asap
14:18 EmilienM I think we already had this issue in other providers
14:18 EmilienM probably in keystone
14:18 EmilienM yolanda: how keystone is working?
14:18 yolanda EmilienM, keystone looks good
14:18 richm joined #puppet-openstack
14:19 EmilienM I suspect we need to patch openstacklib/lib/puppet/provider/openstack.rb
14:21 yolanda EmilienM https://bugs.launchpad.net/puppet-neutron/+bug/1544573
14:21 openstack Launchpad bug 1544573 in puppet-neutron "puppet-neutron not properly parsing output of requests when warnings are thrown" [Undecided,New]
14:22 EmilienM yolanda: thanks
14:22 yolanda thanks for the quick support on that
14:23 rcarrillocruz ++
14:23 degorenko EmilienM, we have neutron cli for neutron providers
14:24 EmilienM oh :)
14:24 degorenko for keystone it is openstack cli
14:24 degorenko :)
14:24 EmilienM we don't use openstacklib
14:24 degorenko yes
14:24 EmilienM ok so yeah, the fix will be in puppet-neutron
14:24 rcarrillocruz ah right
14:25 rcarrillocruz cos openstack cli doesn't have yet key neutron resources merged
14:25 rcarrillocruz i remember seeing changes pending merge
14:25 degorenko but i think we anyway has wrong parsing in neutron
14:25 degorenko even we use neutron cli
14:25 EmilienM degorenko: do you want to assign the bug to yourself? :-)
14:26 degorenko no :D let me check before
14:26 degorenko how this problem is big
14:26 EmilienM lol
14:26 EmilienM "too big, I don't take it!"
14:26 degorenko no :)
14:26 degorenko just want be sure
14:26 degorenko that i have enough time for this
14:26 degorenko i can help - yes
14:27 yolanda oh that explains the different behaviour from others
14:27 degorenko looks like we need fix this function: https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L148-L168
14:28 degorenko and others
14:28 degorenko in this file
14:28 degorenko list
14:28 degorenko it will be really cool - when we will have all features of neutron client in openstack client
14:29 degorenko it will be easy to switch
14:29 dprince EmilienM: any ideas on this? http://paste.openstack.org/show/486695/
14:29 yolanda degorenko, why neutron is not yet on openstack client? is there some blocker, or just lack of implementation?
14:30 rcarrillocruz lack of implementation
14:30 degorenko yolanda, because not all functions of neutron cli have been moved to openstack clie
14:30 rcarrillocruz changes are on review
14:30 degorenko rcarrillocruz, can you share link on this review here? :)
14:30 rcarrillocruz sure, i can have a look
14:30 EmilienM dprince: run 'bundle install' before
14:31 EmilienM dprince: or 'bundle update'
14:31 dprince EmilienM: I just created it :)
14:31 EmilienM ah
14:31 dprince EmilienM: totally clean checkout, bundle install, etc.
14:31 EmilienM let me try
14:31 degorenko EmilienM, dprince also you may to set GEM_HOME var
14:32 EmilienM right you need that ^
14:32 rcarrillocruz they had a bluprint opened to put all verbs from neutron
14:32 rcarrillocruz https://review.openstack.org/#/q/topic:bp/neutron-client
14:32 rcarrillocruz degorenko: ^
14:32 dprince degorenko: okay, I don't set this for other projects. But I'll try
14:32 rcarrillocruz some merged
14:32 rcarrillocruz some pending
14:33 degorenko dprince, set it via export GEM_HOME=<yuor bundler dir>
14:33 degorenko rcarrillocruz, thanks
14:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/279100
14:37 EmilienM dprince, degorenko: I use that script https://gist.github.com/EmilienM/61d590a89c44364d0436
14:37 EmilienM very dumb but helpful
14:37 EmilienM you can override the vesion of puppet in $1
14:37 degorenko EmilienM, yes, i have the same, but also i have hack for choosing tests what i need :P
14:38 EmilienM degorenko: oh nice, please send a PR :-P
14:39 degorenko EmilienM, :P http://paste.openstack.org/show/486699/
14:41 degorenko EmilienM, yolanda  probably i found solution for neutron :)
14:41 yolanda degorenko, that sounds good
14:41 degorenko we should use these functions: https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L225-L236
14:41 degorenko they already implemented for list call
14:42 degorenko yolanda, if i will upload patch - do you have some env to check it?
14:42 yolanda degorenko yes
14:42 degorenko nice
14:42 yolanda willing to check it :)
14:42 EmilienM nice catch
14:43 EmilienM degorenko: I'm stealing your code :-P
14:43 degorenko no problem ;)
14:44 rcarrillocruz degorenko: that was quick, thx :-)
14:48 openstackgerrit Gael Chamoulaud proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/279110
14:49 openstackgerrit Denis Egorenko proposed openstack/puppet-neutron: Clean up neutron cli output from warning lines  https://review.openstack.org/279111
14:49 degorenko yolanda, ^^^ it's still wip, all neutron calls should contain it
14:49 degorenko just to be sure that it will help
14:51 yolanda degorenko, i can test it in a sec
14:56 johnmilton joined #puppet-openstack
14:58 degorenko yolanda, it will not help :(
14:58 degorenko we have shell output
14:58 degorenko function is for csv
14:58 degorenko looks like we need new function *facepalm*
14:59 degorenko EmilienM, ^
14:59 yolanda i went just direct and i exported PYTHONWARNINGS=ignore :)
14:59 ducttape_ joined #puppet-openstack
15:11 degorenko EmilienM, 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 EmilienM i think the pattern comes from python-requests
15:11 EmilienM but I'm not sure
15:12 degorenko just trying to find solution for this problem
15:12 degorenko how to detect warning
15:12 degorenko may be by leading '/' in the line?
15:12 yolanda degorenko, i found a pair of warnings
15:12 yolanda with requests 2.7.0, it starts with InsecurePlatformWarning
15:12 yolanda but on requests 2.9.0, i can see some SNI warning first
15:13 degorenko yolanda, you also have '/usr/lib/python2.7/dist-packages/urllib3/util/ssl_.py:97: InsecurePlatformWarning:'
15:13 degorenko and then warning meesage
15:13 degorenko i don't care about type of warning
15:13 yolanda degorenko let me retry , i think it's a bit different
15:13 degorenko for your warning we have pass to file, where it was spawn
15:14 EmilienM actually not in requests but urllib3
15:14 EmilienM my bad
15:14 degorenko and it seems, that all cli warnings have same pattern - /path/to/file:line typewarning: message
15:14 degorenko for this case we can just drop all line with leading '/'
15:15 EmilienM maybe we can parse "InsecurePlatformWarning"
15:15 degorenko not case
15:15 degorenko we can have some DeprecationWarning
15:15 degorenko or have such network :D
15:15 degorenko for fun
15:15 rcarrillocruz lol
15:16 degorenko but we don't have properties with name beginning with '/'
15:20 yolanda EmilienM, degorenko, how do you do your tests? we are going to use self-signed certs instead of relying on an infra CA
15:20 EmilienM yolanda: we don't have SSL enabled in our CI
15:20 yolanda how that will work with puppet-openstack?
15:20 yolanda oh
15:20 degorenko yolanda, just using warnings from keystone :D and simple output from neutron
15:20 EmilienM yolanda: why not using infra CA?
15:20 degorenko on my env
15:20 yolanda degorenko
15:20 yolanda Notice: 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 th
15:20 yolanda e 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 SNIMissingWarning
15:21 yolanda that's first error i have when i upgrade to requests 2.9.0
15:21 yolanda EmilienM, that comes from conversations in infra channel, seems better to go simple and use simple self-signed certs
15:21 degorenko so, we have warning which can use my pattern :D
15:21 yolanda unless there is a good reason for not doing that (for example if that won't work...)
15:21 EmilienM not sure that's simple, regarding warnings we have now
15:22 degorenko pattern /path/to/file:line typewarning: message
15:22 rcarrillocruz yeah, probably just match 'Warning' ?
15:22 degorenko no
15:22 degorenko we can have some properties
15:22 nlahouti joined #puppet-openstack
15:22 degorenko with value warning
15:23 degorenko we can use some regexp, based on pattern above for csv and shell
15:24 ricardo_ joined #puppet-openstack
15:29 degorenko yolanda, EmilienM, rcarrillocruz may be regexp: ^\/.*:\s\w+Warning:.*$
15:29 degorenko http://rubular.com/r/F88PA0RNhF
15:29 angdraug joined #puppet-openstack
15:30 rcarrillocruz hmm, yeah
15:30 rcarrillocruz that could work
15:31 EmilienM nice
15:32 yolanda ++
15:35 xarses_ joined #puppet-openstack
15:35 dragomir joined #puppet-openstack
15:37 degorenko no, it will not work again. if some warning will move message to next line
15:40 dragomir joined #puppet-openstack
15:46 colonwq one 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 joined #puppet-openstack
15:47 degorenko recheck - in comment
15:47 EmilienM colonwq: what says logs?
15:47 degorenko EmilienM, gnocchi error :)
15:47 EmilienM degorenko: I want to see a trace
15:48 EmilienM like, an URL so we can debug
15:48 colonwq one moment
15:49 colonwq http://logs.openstack.org/48/254548/24/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/18fa099/console.html#_2016-02-11_03_13_34_752
15:49 degorenko yolanda, can you put few outputs to paste openstack from neutron with warnings for me?
15:50 colonwq had to get browser on the right computer
15:51 yolanda degorenko, the two i pasted earlier, i don't have more
15:52 degorenko no, i mean :) from cmd directly with output
15:52 rochaporto joined #puppet-openstack
15:52 degorenko yolanda, like: http://paste.openstack.org/show/486715/
15:52 yolanda ah just a sec, because i wiped my testenv :(
15:52 degorenko no problem
15:55 yolanda ah 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 fle
15:55 degorenko can you do that? :)
15:56 colonwq degorenko, would the recheck go in the Jenkins comment or the git commit message?
15:56 rcarrillocruz colonwq: gerrit change comment
15:56 degorenko colonwq, gerrit comment
15:56 degorenko :)
15:57 colonwq Thanks everyone.
15:57 thumpba joined #puppet-openstack
15:57 yolanda yep, redeploying the test env now
16:00 rcarrillocruz left #puppet-openstack
16:02 thumpba_ joined #puppet-openstack
16:05 yolanda degorenko, question.. if we know that output can come on a json format, why not strip everything tha comes before real json content?
16:05 degorenko yolanda, we can't use json :(
16:05 degorenko we have issue with it
16:06 degorenko yolanda, see this patch https://review.openstack.org/#/c/262809/
16:06 EmilienM joined #puppet-openstack
16:06 yolanda oh
16:08 higgins joined #puppet-openstack
16:09 dgurtner joined #puppet-openstack
16:10 degorenko yolanda, so until we have openstack client with neutron features - we should have some workaround for neutron :(
16:11 yolanda ok running puppet code again, i had to redeploy
16:12 openstackgerrit Gael Chamoulaud proposed openstack/puppet-heat: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/279154
16:13 haplo37 joined #puppet-openstack
16:14 darkxploit joined #puppet-openstack
16:21 mdorman joined #puppet-openstack
16:25 degorenko chem, according to your second comment here: https://review.openstack.org/#/c/273513/8/lib/puppet/type/cinder_type.rb
16:25 degorenko it will not work for: key=value1,value2
16:28 chem degorenko: the regex I propose won't work for this
16:28 chem degorenko: correct
16:29 chem degorenko: is that a requirement ?
16:29 degorenko chem, we can have key and couple of values for them
16:29 chem degorenko: if so, it's easy to modify the regex
16:29 chem degorenko: 2s
16:30 chem degorenko: the first regex won't work either ... let me adjust it
16:30 degorenko chem, first fine, what's wrong with him?
16:30 chem degorenko: sorry you're right, nothing wrong with it...
16:30 degorenko Ж)
16:31 degorenko :)
16:32 chem degorenko: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 degorenko yeah
16:32 degorenko that's why my regexp reject such lines
16:33 chem degorenko: string actually, not regex
16:33 chem degorenko: just use split(/\s*,\s*/) and you'll be fine
16:33 degorenko i'm for first one
16:33 degorenko no will not :)
16:33 degorenko <degorenko> it will not work for: key=value1,value2
16:34 chem degorenko: so value = "value1,value2", correct ?
16:35 degorenko key="value1,value2"
16:35 degorenko that's why i have space between keys
16:36 chem degorenko: oh, you mean you can have 'key1=value1,value2 ,key2=value3, ...'
16:36 degorenko нуз
16:36 degorenko yep
16:36 degorenko :D
16:36 chem hehe
16:37 chem ack
16:37 degorenko russian letters, sorry
16:37 chem degorenko: that's tricky :)
16:37 degorenko yes
16:37 degorenko :)
16:38 degorenko easier way always have space :) like key=v,1,2, key2=s,sd, key3=v3
16:41 holser_ joined #puppet-openstack
16:43 mwhahaha joined #puppet-openstack
16:43 chem degorenko: 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 degorenko yes, it makes me cry :D
16:43 chem degorenko: I think It may be late for such a change ... dunno :)
16:44 degorenko why?
16:45 degorenko we can have input as array, then mungle it to hash and that's it
16:45 chem degorenko: for no reason :)
16:45 degorenko ah
16:45 degorenko so, how then we can solve this problem?
16:45 chem degorenko: yeap ^^
16:45 chem degorenko: array -> mungle -> hash -> done :)
16:46 chem degorenko: the regex will be then much easier, and you won't have to explain that the ", " is mandatory
16:46 degorenko yep :)
16:47 degorenko and having an array is not so complex i guess for properties
16:47 degorenko string/array - no big difference
16:47 chem degorenko let me find you an example  ... 2s
16:47 degorenko it's no problem with array
16:48 degorenko chem, and we still can have validation on which item of array :)
16:48 degorenko on each*
16:49 chem degorenko: yes : check this https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_user_role.rb#L64-L70
16:50 degorenko yep :) i remember about issync
16:50 degorenko insync
16:50 chem degorenko: yep, required to not depend on order
16:51 degorenko yes
16:52 dmsimard EmilienM: tempest revert merged
16:52 dmsimard EmilienM: and we got scenario002 to pass on weirdo (delorean current), did you pin tempest ?
16:52 chem degorenko: 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-behaviour
16:53 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "pin tempest to keep setuping heat roles"  https://review.openstack.org/279180
16:53 degorenko i know :P
16:53 EmilienM dmsimard: ^
16:53 dmsimard EmilienM: ok
16:53 EmilienM I hope it pass
16:53 EmilienM dmsimard: fyi, we bumped again to a very recent url
16:53 EmilienM so we don't wait for promotions
16:53 dmsimard EmilienM: sure
16:54 dmsimard EmilienM: still working on packstack and tripleo .. :(
16:54 chem degorenko: you will have to modify the l53 of the provider as well
16:54 degorenko chem, btw, do i need validate method? or i can use string validate in munge section?
16:54 chem degorenko : hum .. no sorry ^
16:54 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
16:55 degorenko chem, you don't get it or i don't need validate method? :D
16:55 chem degorenko for the validate you should get one yes
16:55 chem degorenko validate and munge at not done at the same time in the processing
16:55 degorenko so oh, that's good to know
16:56 degorenko but both of them provider methods
16:56 degorenko why we can't use them in same time?
16:56 bnemec joined #puppet-openstack
16:57 chem degorenko:you can use them at the same time, it's just that values are validated first and *then* munged
16:57 degorenko oh, no, that's of course
16:58 degorenko i meant, another thing
16:58 degorenko that on munge stage i have value and i'll iterate on this value (array items)
16:58 degorenko i  can call item.match(regexp)
16:58 degorenko ^ that was i meant
16:59 degorenko but it will be in one cycle
16:59 chem degorenko: I would have to check, but I think munge get the array, not its element
16:59 degorenko yes :D
16:59 degorenko let me clarify
16:59 degorenko you don't get it
16:59 degorenko sec
16:59 chem degorenko:oki, i got it
17:00 chem degorenko: I see what you mean, like failing during munge to not have to iterate twice over the array
17:00 degorenko yeah :)
17:00 degorenko is it good?
17:01 samuelBartel joined #puppet-openstack
17:01 colonwq I 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 chem degorenko: If it was my patch, I would waste some cycle.  But you can try it that way and check how it works
17:01 degorenko colonwq, is your question for unit test or for specs tests?
17:02 nlahouti joined #puppet-openstack
17:02 colonwq degorenko, spec I believe. I am learning the terms you guys use.
17:02 degorenko colonwq, it is for puppet manifest or for provider?
17:03 colonwq I am writing a spec/defines/ceph_rgw_civetweb_spec.rb to test my civetweb settings.
17:03 chem colonwq: if obj.exists? make sense, then you can expect(obj).not_to exists
17:03 degorenko ah i  see
17:03 chem colonwq: 95% sure :)
17:03 degorenko colonwq, then you can use https://github.com/openstack/puppet-sahara/blob/master/spec/classes/sahara_init_spec.rb#L167
17:03 degorenko like that
17:03 degorenko is_expected.to_not contain_
17:04 colonwq ah. That is easier and shorter.
17:05 colonwq chem, degorenko Thanks for the pointers. I am sure I'll ask more questions that are simple.
17:05 degorenko chem, so you'd prefer one more cycle, right?
17:06 degorenko colonwq, you're welcome
17:06 chem colonwq: for reference "expect(actual).to exist        # passes if actual.exist? and/or actual.exists? are truthy"
17:06 chem colonwq: if one day you need it
17:06 chem degorenko: yep
17:07 chem degorenko: easier to read
17:07 degorenko chem, ack, thanks for help man o/
17:08 chem degorenko: np :)
17:11 baker joined #puppet-openstack
17:12 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
17:14 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
17:26 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers  https://review.openstack.org/273513
17:28 degorenko chem, ^ should be good now
17:30 chem degorenko: looking
17:32 openstackgerrit Ben Nemec proposed openstack/puppet-tripleo: Handle redirects for Horizon  https://review.openstack.org/268288
17:37 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
17:37 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
17:39 sid14726 joined #puppet-openstack
17:42 openstackgerrit Denis Egorenko proposed openstack/puppet-neutron: Add clean up for shell format for neutron output  https://review.openstack.org/279111
17:43 degorenko yolanda, now it should work ^
17:43 degorenko chem, we have problem with neutron output :) in case you have time ^
17:48 chem degorenko: reviewed :)
17:49 degorenko chem, thanks :) i will update it tomorrow
17:51 chem degorenko: looking into the neutron stuff.
17:51 chem degorenko: have a good one and see you tomorrow
17:52 degorenko chem, i'm here for ~10-15 minutes yet
17:52 chem degorenko: same here :)
17:53 degorenko :D same what?
18:03 chem degorenko: I'm leaving as well :)  I've reviewed the neutron patch.
18:04 degorenko chem, thanks again, good night :) see you tomorrow
18:04 chem degorenko:see ya
18:09 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
18:13 openstackgerrit Merged openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/264927
18:16 rcernin joined #puppet-openstack
18:22 rmart04 joined #puppet-openstack
18:27 IvanBerezovskiy left #puppet-openstack
18:40 dgurtner joined #puppet-openstack
18:40 dgurtner joined #puppet-openstack
19:03 nlahouti joined #puppet-openstack
19:04 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Update supported version information  https://review.openstack.org/277562
19:29 nlahouti joined #puppet-openstack
19:34 dkaigarodsev joined #puppet-openstack
19:43 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/256248
19:46 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
19:52 nlahouti joined #puppet-openstack
20:00 vinsh joined #puppet-openstack
20:07 vinsh_ joined #puppet-openstack
20:24 mewald joined #puppet-openstack
20:27 mewald Does puppet-ceph work with infernalis? My monitors are not starting and ceph.conf looks broken. Any experiences?
20:28 rmart04 joined #puppet-openstack
20:30 gchamoul joined #puppet-openstack
20:36 openstackgerrit Giulio Fidente proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option  https://review.openstack.org/279055
20:40 openstackgerrit Merged openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone admin endpoint  https://review.openstack.org/267417
20:57 sid14726 joined #puppet-openstack
21:01 xarses mewald: there was a note from EmilienM that it likely did not
21:01 richm joined #puppet-openstack
21:03 xarses mewald: its most likely due to the change to support systemd init
21:04 xarses and if you paste ceph.conf I can probably triage it
21:04 xarses but afaiu there was nothing that should break infernalis there
21:20 jfluhmann joined #puppet-openstack
21:31 morazi joined #puppet-openstack
21:43 yolanda degorenko, sorry, i finished with my testenv just now. I will test your patches tomorrow
21:44 rahulait joined #puppet-openstack
21:57 mewald xarses: thx, that helped :)
21:57 mewald does it work to use LVM logical volumes as journal disks?
22:00 _ducttape_ joined #puppet-openstack
22:06 nlahouti joined #puppet-openstack
22:06 xarses mewald: 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 want
22:09 johnmilton joined #puppet-openstack
22:10 johnmilton joined #puppet-openstack
22:29 openstackgerrit Sam Morrison proposed openstack/puppet-neutron: Support midonet type driver for ML2  https://review.openstack.org/278207
22:32 sid14726 joined #puppet-openstack
22:35 andybotting joined #puppet-openstack
22:38 ducttape_ joined #puppet-openstack
22:42 cdelatte joined #puppet-openstack
23:03 gildub joined #puppet-openstack
23:25 priteau_ joined #puppet-openstack
23:32 priteau joined #puppet-openstack
23:36 xarses joined #puppet-openstack
23:37 mewald joined #puppet-openstack
23:53 mewald joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary