openstackgerrit | Merged openstack/puppet-heat: Support setting instance_user to an empty string https://review.openstack.org/278450 | 00:03 |
---|---|---|
*** crinkle_ has joined #puppet-openstack | 00:03 | |
*** crinkle has quit IRC | 00:03 | |
openstackgerrit | Merged openstack/puppet-neutron: Creation of neutron::db::sync https://review.openstack.org/278458 | 00:03 |
openstackgerrit | Merged openstack/puppet-neutron: Use uppercase 'S' in word "OpenStack" https://review.openstack.org/276135 | 00:04 |
*** crinkle_ is now known as crinkle | 00:04 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class https://review.openstack.org/256248 | 00:08 |
*** ducttape_ has joined #puppet-openstack | 00:09 | |
*** dgurtner has quit IRC | 00:12 | |
*** sid14726 has quit IRC | 00:18 | |
*** mdorman has quit IRC | 00:18 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class https://review.openstack.org/256248 | 00:19 |
*** xarses has quit IRC | 00:24 | |
*** xarses has joined #puppet-openstack | 00:32 | |
openstackgerrit | Merged openstack/puppet-neutron: Provide support for VPNaaS service_provider https://review.openstack.org/273934 | 00:35 |
*** alee_ has quit IRC | 00:38 | |
*** xarses has quit IRC | 00:43 | |
*** xarses has joined #puppet-openstack | 00:43 | |
*** xarses has quit IRC | 00:48 | |
*** baker has joined #puppet-openstack | 00:50 | |
*** xarses has joined #puppet-openstack | 00:54 | |
*** ducttape_ has quit IRC | 00:55 | |
*** xarses_ has joined #puppet-openstack | 00:56 | |
*** xarses has quit IRC | 00:59 | |
*** baker_ has joined #puppet-openstack | 01:00 | |
*** baker has quit IRC | 01:03 | |
openstackgerrit | Merged openstack/puppet-trove: Use uppercase 'S' in word "OpenStack" https://review.openstack.org/276607 | 01:06 |
*** baker_ has quit IRC | 01:06 | |
*** nlahouti has quit IRC | 01:06 | |
*** nlahouti has joined #puppet-openstack | 01:07 | |
*** saneax is now known as saneax_AFK | 01:07 | |
*** morazi has joined #puppet-openstack | 01:09 | |
*** baker has joined #puppet-openstack | 01:14 | |
openstackgerrit | Merged openstack/puppet-vswitch: Add require ovs_redhat.rb to ovs_redhat_el6.rb https://review.openstack.org/278560 | 01:16 |
*** baker has quit IRC | 01:17 | |
*** angdraug has quit IRC | 01:20 | |
*** morazi has quit IRC | 01:33 | |
*** jasondotstar has quit IRC | 01:35 | |
*** sid14726 has joined #puppet-openstack | 01:36 | |
*** jasondotstar has joined #puppet-openstack | 01:37 | |
*** gildub has quit IRC | 01:41 | |
*** baker has joined #puppet-openstack | 01:59 | |
*** ducttape_ has joined #puppet-openstack | 01:59 | |
*** baker has quit IRC | 02:02 | |
*** vinsh has quit IRC | 02:12 | |
*** vinsh has joined #puppet-openstack | 02:13 | |
*** foxhunt7z has joined #puppet-openstack | 02:13 | |
*** openstackgerrit has quit IRC | 02:15 | |
*** foxhunt_away has quit IRC | 02:15 | |
*** vinsh has quit IRC | 02:18 | |
*** vinsh has joined #puppet-openstack | 02:18 | |
*** fvollero has quit IRC | 02:20 | |
*** openstackgerrit has joined #puppet-openstack | 02:24 | |
*** ducttape_ has quit IRC | 02:26 | |
*** ducttape_ has joined #puppet-openstack | 02:27 | |
openstackgerrit | Andy Botting proposed openstack/puppet-rally: Add config file settings and tests for benchmarking https://review.openstack.org/271138 | 02:28 |
*** baker has joined #puppet-openstack | 02:29 | |
*** baker has quit IRC | 02:33 | |
*** ducttape_ has quit IRC | 02:35 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 02:40 |
*** sid14726 has quit IRC | 02:47 | |
*** rahulait has joined #puppet-openstack | 02:55 | |
*** rahulait has quit IRC | 03:05 | |
*** haplo37 has joined #puppet-openstack | 03:06 | |
*** sid14726 has joined #puppet-openstack | 03:15 | |
*** rahulait has joined #puppet-openstack | 03:17 | |
*** haplo37 has quit IRC | 03:25 | |
*** haplo37 has joined #puppet-openstack | 03:26 | |
*** rahulait has quit IRC | 03:46 | |
*** ducttape_ has joined #puppet-openstack | 03:58 | |
*** rahulait has joined #puppet-openstack | 04:09 | |
*** baker has joined #puppet-openstack | 04:13 | |
*** sid14726 has quit IRC | 04:27 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-cinder: module to set quotas per volume type https://review.openstack.org/197572 | 04:28 |
*** ducttape_ has quit IRC | 04:29 | |
mfisch | crinkle: would love to discuss the admin token removal stuff with you tomorrow | 04:35 |
mfisch | just saw the review | 04:35 |
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:38 |
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:40 |
crinkle | mfisch: hmm that's not very puppetty, could that get proposed in upstream keystone? | 04:41 |
mfisch | I dont think so | 04:41 |
mfisch | I mean maybe? | 04:42 |
mfisch | I think that their response would be to disable it entirely | 04:42 |
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:43 |
*** haplo37 has quit IRC | 04:44 | |
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:46 |
crinkle | this should work on the first run | 04:47 |
mfisch | oh at the end of the keystone resources then | 04:49 |
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:53 |
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:54 |
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 | 04:59 |
crinkle | ah ok | 05:00 |
*** markvoelker has quit IRC | 05:03 | |
*** sid14726 has joined #puppet-openstack | 05:04 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class https://review.openstack.org/256248 | 05:08 |
*** baker has quit IRC | 05:32 | |
*** sid14726 has quit IRC | 05:39 | |
*** sid14726 has joined #puppet-openstack | 05:49 | |
*** fedexo has joined #puppet-openstack | 05:52 | |
*** fedexo_ has joined #puppet-openstack | 05:53 | |
*** nlahouti has joined #puppet-openstack | 05:57 | |
*** nlahouti1 has joined #puppet-openstack | 05:59 | |
*** cwolferh has quit IRC | 06:00 | |
*** cwolferh has joined #puppet-openstack | 06:00 | |
*** nlahouti has quit IRC | 06:01 | |
*** markvoelker has joined #puppet-openstack | 06:04 | |
*** jaosorior has joined #puppet-openstack | 06:10 | |
*** sid14726 has quit IRC | 06:21 | |
*** a7ndrew_ is now known as a7ndrew | 06:25 | |
*** sid14726 has joined #puppet-openstack | 06:30 | |
*** markvoelker has quit IRC | 06:34 | |
*** saneax_AFK is now known as saneax | 06:53 | |
*** nlahouti1 has quit IRC | 06:54 | |
*** nlahouti has joined #puppet-openstack | 06:54 | |
*** jaosorior has quit IRC | 06:56 | |
*** rahulait has quit IRC | 07:03 | |
*** rahulait has joined #puppet-openstack | 07:05 | |
*** sid14726 has quit IRC | 07:05 | |
*** fedexo has quit IRC | 07:14 | |
*** sid14726 has joined #puppet-openstack | 07:15 | |
*** jaosorior has joined #puppet-openstack | 07:20 | |
*** jaosorior has quit IRC | 07:32 | |
*** dgurtner has joined #puppet-openstack | 07:33 | |
*** jtomasek has joined #puppet-openstack | 07:36 | |
*** sid14726 has quit IRC | 07:36 | |
*** leanderthal has joined #puppet-openstack | 07:36 | |
*** sid14726 has joined #puppet-openstack | 07:45 | |
*** hjensas has joined #puppet-openstack | 07:46 | |
*** hjensas has quit IRC | 07:46 | |
*** hjensas has joined #puppet-openstack | 07:46 | |
*** rahulait has quit IRC | 07:49 | |
*** richm has joined #puppet-openstack | 07:55 | |
*** nlahouti has quit IRC | 08:02 | |
openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow changing list of processing hooks https://review.openstack.org/278916 | 08:04 |
*** jaosorior has joined #puppet-openstack | 08:05 | |
*** sid14726 has quit IRC | 08:13 | |
*** jpena has joined #puppet-openstack | 08:25 | |
*** sid14726 has joined #puppet-openstack | 08:28 | |
*** dgurtner has quit IRC | 08:29 | |
*** yottatsa has joined #puppet-openstack | 08:29 | |
*** rcernin has joined #puppet-openstack | 08:35 | |
*** yottatsa has quit IRC | 08:43 | |
*** openstackgerrit has quit IRC | 08:47 | |
*** openstackgerrit_ has joined #puppet-openstack | 08:47 | |
*** openstackgerrit_ is now known as openstackgerrit | 08:48 | |
*** dgurtner has joined #puppet-openstack | 08:48 | |
*** dgurtner has quit IRC | 08:48 | |
*** dgurtner has joined #puppet-openstack | 08:48 | |
*** yottatsa has joined #puppet-openstack | 08:50 | |
*** jtomasek has quit IRC | 08:52 | |
*** yottatsa has quit IRC | 08:52 | |
*** yottatsa has joined #puppet-openstack | 08:53 | |
*** rmart04 has joined #puppet-openstack | 08:54 | |
*** jtomasek has joined #puppet-openstack | 08:57 | |
*** derekh has joined #puppet-openstack | 09:00 | |
*** sid14726 has quit IRC | 09:04 | |
*** rmart04 has left #puppet-openstack | 09:05 | |
*** yottatsa has quit IRC | 09:07 | |
*** yottatsa has joined #puppet-openstack | 09:20 | |
*** priteau has joined #puppet-openstack | 09:20 | |
*** yottatsa has quit IRC | 09:24 | |
*** sid14726 has joined #puppet-openstack | 09:24 | |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/220208 | 09:25 |
*** markvoelker has joined #puppet-openstack | 09:31 | |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-neutron: Remove use of template based config from cisco ml2 nexus plugin. https://review.openstack.org/278952 | 09:35 |
*** markvoelker has quit IRC | 09:36 | |
*** paramite has joined #puppet-openstack | 09:38 | |
*** gildub has joined #puppet-openstack | 09:38 | |
*** gildub has quit IRC | 09:45 | |
*** yottatsa has joined #puppet-openstack | 09:47 | |
*** yottatsa has quit IRC | 09:54 | |
*** gfidente has joined #puppet-openstack | 09:55 | |
*** gfidente has quit IRC | 09:55 | |
*** gfidente has joined #puppet-openstack | 09:55 | |
*** yottatsa has joined #puppet-openstack | 09:59 | |
*** yottatsa has quit IRC | 10:00 | |
*** gildub has joined #puppet-openstack | 10:02 | |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat https://review.openstack.org/278960 | 10:05 |
*** IvanBerezovskiy has joined #puppet-openstack | 10:12 | |
openstackgerrit | Muhammad Salman Khan proposed openstack/puppet-neutron: Change of PLUMgrid plugin package and addition of new parameters. https://review.openstack.org/278522 | 10:13 |
*** sid14726 has quit IRC | 10:13 | |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-neutron: Add support for RabbitMQ connection heartbeat https://review.openstack.org/278962 | 10:16 |
openstackgerrit | Merged openstack/puppet-ceph: Bump required version of puppetlabs-apt https://review.openstack.org/276970 | 10:20 |
*** gildub has quit IRC | 10:21 | |
*** gildub has joined #puppet-openstack | 10:22 | |
*** sid14726 has joined #puppet-openstack | 10:24 | |
*** fedexo_ has quit IRC | 10:24 | |
*** priteau has quit IRC | 10:27 | |
*** priteau has joined #puppet-openstack | 10:33 | |
*** paramite is now known as paramite|afk | 10:37 | |
*** chem has joined #puppet-openstack | 10:39 | |
*** morazi has joined #puppet-openstack | 10:41 | |
*** paramite|afk is now known as paramite | 10:50 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy. https://review.openstack.org/264951 | 10:50 |
*** morazi_ has joined #puppet-openstack | 10:55 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy. https://review.openstack.org/264951 | 10:56 |
*** yottatsa has joined #puppet-openstack | 10:57 | |
*** sid14726 has quit IRC | 10:57 | |
*** morazi has quit IRC | 10:58 | |
openstackgerrit | Merged openstack/puppet-nova: Allow to set max_concurrent_build for ironic compute https://review.openstack.org/276422 | 11:02 |
*** sid14726 has joined #puppet-openstack | 11:08 | |
*** morazi_ has quit IRC | 11:15 | |
*** _degorenko|afk is now known as degorenko | 11:19 | |
*** markvoelker has joined #puppet-openstack | 11:32 | |
*** markvoelker has quit IRC | 11:37 | |
*** sid14726 has quit IRC | 11:41 | |
*** paramite is now known as paramite|afk | 11:41 | |
*** sid14726 has joined #puppet-openstack | 11:45 | |
*** dprince has joined #puppet-openstack | 11:55 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function https://review.openstack.org/271260 | 11:58 |
openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Deprecate router[dhcp]_delete_namespaces parameter https://review.openstack.org/276674 | 12:09 |
*** gildub has quit IRC | 12:14 | |
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:15 |
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:16 |
chem | degorenko: ah, oki, cool :) | 12:17 |
chem | degorenko: sorry for the very late review ... | 12:17 |
*** sid14726 has quit IRC | 12:21 | |
openstackgerrit | narasimha18sv proposed openstack/puppet-neutron: Removing Keystone middleware configurations in neutron paste file https://review.openstack.org/279014 | 12:28 |
*** johnmilton has joined #puppet-openstack | 12:30 | |
*** markvoelker has joined #puppet-openstack | 12:33 | |
degorenko | chem, no problem :) | 12:35 |
chem | cool | 12:35 |
*** markvoelker has quit IRC | 12:37 | |
EmilienM | good morning! | 12:37 |
degorenko | chem, btw, thanks for regexp :D | 12:39 |
degorenko | Morning :) | 12:39 |
chem | degorenko: you're welcome, kinda like regexp :) | 12:39 |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets. https://review.openstack.org/264927 | 12:41 |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy. https://review.openstack.org/264951 | 12:42 |
*** sid14726 has joined #puppet-openstack | 12:45 | |
*** paramite|afk is now known as paramite | 12:47 | |
openstackgerrit | Merged openstack/puppet-tripleo: Make haproxy balancer default options configurable https://review.openstack.org/265298 | 12:50 |
openstackgerrit | Merged openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6 https://review.openstack.org/265309 | 12:51 |
EmilienM | dprince: can you look https://review.openstack.org/#/c/278795/ please ? | 12:57 |
dprince | EmilienM: +A | 12:59 |
EmilienM | thanks | 13:02 |
*** delattec has joined #puppet-openstack | 13:10 | |
*** ducttape_ has joined #puppet-openstack | 13:11 | |
*** jayg|g0n3 is now known as jayg | 13:12 | |
*** delatte has quit IRC | 13:13 | |
*** morazi has joined #puppet-openstack | 13:18 | |
*** jtomasek_ has joined #puppet-openstack | 13:20 | |
*** sid14726 has quit IRC | 13:21 | |
*** Guest57497 is now known as zeus | 13:22 | |
*** zeus has joined #puppet-openstack | 13:22 | |
*** markvoelker has joined #puppet-openstack | 13:23 | |
*** ducttape_ has quit IRC | 13:27 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-nova: Set default branch to stable/liberty in .gitreview https://review.openstack.org/279053 | 13:31 |
*** delatte has 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:32 |
*** duritong has quit IRC | 13:33 | |
*** duritong has joined #puppet-openstack | 13:34 | |
*** delattec has quit IRC | 13:35 | |
*** jtomasek_ has quit IRC | 13:37 | |
openstackgerrit | Radoslaw Smigielski proposed openstack/puppet-keystone: Update init.pp examples to something what really works https://review.openstack.org/279066 | 13:42 |
*** sid14726 has joined #puppet-openstack | 13:44 | |
*** jpena is now known as jpena|lunch | 13:48 | |
*** johnmilton has quit IRC | 13:59 | |
*** saneax is now known as saneax_AFK | 14:02 | |
*** richm has quit IRC | 14:03 | |
*** jaosorior has quit IRC | 14:05 | |
*** jaosorior has joined #puppet-openstack | 14:05 | |
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:09 |
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:10 |
*** rcarrillocruz has 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:12 |
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:14 |
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:15 |
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:16 |
rcarrillocruz | y | 14:17 |
yolanda | yep | 14:17 |
EmilienM | we'll try to fix it asap | 14:17 |
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 has joined #puppet-openstack | 14:18 | |
EmilienM | I suspect we need to patch openstacklib/lib/puppet/provider/openstack.rb | 14:19 |
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:21 |
EmilienM | yolanda: thanks | 14:22 |
yolanda | thanks for the quick support on that | 14:22 |
rcarrillocruz | ++ | 14:23 |
degorenko | EmilienM, we have neutron cli for neutron providers | 14:23 |
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:25 |
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:26 |
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:27 |
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:28 |
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:29 |
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:30 |
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:31 |
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:32 |
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:33 |
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:37 |
EmilienM | degorenko: oh nice, please send a PR :-P | 14:38 |
degorenko | EmilienM, :P http://paste.openstack.org/show/486699/ | 14:39 |
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:41 |
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:42 |
EmilienM | degorenko: I'm stealing your code :-P | 14:43 |
degorenko | no problem ;) | 14:43 |
rcarrillocruz | degorenko: that was quick, thx :-) | 14:44 |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat https://review.openstack.org/279110 | 14:48 |
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:49 |
*** sid14726 has quit IRC | 14:50 | |
yolanda | degorenko, i can test it in a sec | 14:51 |
*** xarses_ has quit IRC | 14:56 | |
*** johnmilton has joined #puppet-openstack | 14:56 | |
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_ has joined #puppet-openstack | 14:59 | |
*** jpena|lunch is now known as jpena | 15:05 | |
*** jaosorior has quit IRC | 15:06 | |
*** morazi has quit IRC | 15:08 | |
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:11 |
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:12 |
*** paramite is now known as paramite|afk | 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 |
*** paramite|afk is now known as paramite | 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:13 |
EmilienM | actually not in requests but urllib3 | 15:14 |
EmilienM | my bad | 15:14 |
*** paramite is now known as paramite|afk | 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:14 |
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:15 |
degorenko | but we don't have properties with name beginning with '/' | 15:16 |
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:20 |
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:21 |
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 has joined #puppet-openstack | 15:22 | |
degorenko | with value warning | 15:22 |
degorenko | we can use some regexp, based on pattern above for csv and shell | 15:23 |
*** ricardo_ has joined #puppet-openstack | 15:24 | |
*** ricardo_ has quit IRC | 15:25 | |
*** nlahouti has quit IRC | 15:27 | |
degorenko | yolanda, EmilienM, rcarrillocruz may be regexp: ^\/.*:\s\w+Warning:.*$ | 15:29 |
degorenko | http://rubular.com/r/F88PA0RNhF | 15:29 |
*** angdraug has joined #puppet-openstack | 15:29 | |
rcarrillocruz | hmm, yeah | 15:30 |
rcarrillocruz | that could work | 15:30 |
EmilienM | nice | 15:31 |
yolanda | ++ | 15:32 |
*** xarses_ has joined #puppet-openstack | 15:35 | |
*** dragomir has joined #puppet-openstack | 15:35 | |
*** dragomir has quit IRC | 15:37 | |
degorenko | no, it will not work again. if some warning will move message to next line | 15:37 |
*** dragomir has joined #puppet-openstack | 15:40 | |
*** dgurtner has quit IRC | 15:41 | |
*** dragomir has quit IRC | 15:42 | |
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 has 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:47 |
EmilienM | like, an URL so we can debug | 15:48 |
colonwq | one moment | 15:48 |
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:49 |
colonwq | had to get browser on the right computer | 15:50 |
*** xarses_ is now known as xarses | 15:51 | |
yolanda | degorenko, the two i pasted earlier, i don't have more | 15:51 |
degorenko | no, i mean :) from cmd directly with output | 15:52 |
*** rochaporto has 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:52 |
*** higgins has quit IRC | 15:53 | |
*** EmilienM has quit IRC | 15:53 | |
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:55 |
*** yottatsa has quit IRC | 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:56 |
colonwq | Thanks everyone. | 15:57 |
*** thumpba has joined #puppet-openstack | 15:57 | |
yolanda | yep, redeploying the test env now | 15:57 |
*** paramite|afk is now known as paramite | 15:58 | |
*** rcarrillocruz has left #puppet-openstack | 16:00 | |
*** thumpba has quit IRC | 16:01 | |
*** thumpba_ has joined #puppet-openstack | 16:02 | |
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:05 |
degorenko | yolanda, see this patch https://review.openstack.org/#/c/262809/ | 16:06 |
*** EmilienM has joined #puppet-openstack | 16:06 | |
*** thumpba_ has quit IRC | 16:06 | |
yolanda | oh | 16:06 |
*** higgins has joined #puppet-openstack | 16:09 | |
*** dgurtner has joined #puppet-openstack | 16:09 | |
degorenko | yolanda, so until we have openstack client with neutron features - we should have some workaround for neutron :( | 16:10 |
yolanda | ok running puppet code again, i had to redeploy | 16:11 |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-heat: Add support for RabbitMQ connection heartbeat https://review.openstack.org/279154 | 16:12 |
*** haplo37 has joined #puppet-openstack | 16:13 | |
*** darkxploit has joined #puppet-openstack | 16:14 | |
*** leanderthal is now known as leanderthal|afk | 16:15 | |
*** bnemec has quit IRC | 16:21 | |
*** mdorman has joined #puppet-openstack | 16:21 | |
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:25 |
chem | degorenko: the regex I propose won't work for this | 16:28 |
chem | degorenko: correct | 16:28 |
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:29 |
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:31 |
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:33 |
chem | degorenko: so value = "value1,value2", correct ? | 16:34 |
*** foxhunt7z is now known as foxhunt_away | 16:35 | |
degorenko | key="value1,value2" | 16:35 |
degorenko | that's why i have space between keys | 16:35 |
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:37 |
*** holser_ has quit IRC | 16:38 | |
degorenko | easier way always have space :) like key=v,1,2, key2=s,sd, key3=v3 | 16:38 |
*** mwhahaha has quit IRC | 16:39 | |
*** richm has quit IRC | 16:39 | |
*** holser_ has joined #puppet-openstack | 16:41 | |
*** mwhahaha has 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:43 |
degorenko | why? | 16:44 |
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:46 |
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:47 |
degorenko | chem, and we still can have validation on which item of array :) | 16:48 |
degorenko | on each* | 16:48 |
chem | degorenko: yes : check this https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_user_role.rb#L64-L70 | 16:49 |
degorenko | yep :) i remember about issync | 16:50 |
degorenko | insync | 16:50 |
chem | degorenko: yep, required to not depend on order | 16:50 |
degorenko | yes | 16:51 |
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:52 |
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:53 |
dmsimard | EmilienM: still working on packstack and tripleo .. :( | 16:54 |
*** holser_ is now known as holser | 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:54 |
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:55 |
*** rcernin has quit IRC | 16:56 | |
degorenko | but both of them provider methods | 16:56 |
degorenko | why we can't use them in same time? | 16:56 |
*** bnemec has joined #puppet-openstack | 16:56 | |
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:58 |
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 |
chem | degorenko:oki, i got it | 16:59 |
degorenko | sec | 16:59 |
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:00 |
*** samuelBartel has 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 |
*** dgurtner has quit IRC | 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:01 |
*** nlahouti has 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:02 |
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:03 |
colonwq | ah. That is easier and shorter. | 17:04 |
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:05 |
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:06 |
chem | degorenko: easier to read | 17:07 |
degorenko | chem, ack, thanks for help man o/ | 17:07 |
chem | degorenko: np :) | 17:08 |
*** baker has joined #puppet-openstack | 17:11 | |
*** derekh has quit IRC | 17:12 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer https://review.openstack.org/276141 | 17:12 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer https://review.openstack.org/276141 | 17:14 |
*** darkxploit has quit IRC | 17:18 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers https://review.openstack.org/273513 | 17:26 |
degorenko | chem, ^ should be good now | 17:28 |
chem | degorenko: looking | 17:30 |
openstackgerrit | Ben Nemec proposed openstack/puppet-tripleo: Handle redirects for Horizon https://review.openstack.org/268288 | 17:32 |
*** samuelBartel has quit IRC | 17:33 | |
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:37 |
*** sid14726 has joined #puppet-openstack | 17:39 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-neutron: Add clean up for shell format for neutron output https://review.openstack.org/279111 | 17:42 |
degorenko | yolanda, now it should work ^ | 17:43 |
degorenko | chem, we have problem with neutron output :) in case you have time ^ | 17:43 |
chem | degorenko: reviewed :) | 17:48 |
degorenko | chem, thanks :) i will update it tomorrow | 17:49 |
*** paramite has quit IRC | 17:50 | |
chem | degorenko: looking into the neutron stuff. | 17:51 |
chem | degorenko: have a good one and see you tomorrow | 17:51 |
degorenko | chem, i'm here for ~10-15 minutes yet | 17:52 |
chem | degorenko: same here :) | 17:52 |
degorenko | :D same what? | 17:53 |
*** rochaporto has quit IRC | 18:01 | |
chem | degorenko: I'm leaving as well :) I've reviewed the neutron patch. | 18:03 |
degorenko | chem, thanks again, good night :) see you tomorrow | 18:04 |
chem | degorenko:see ya | 18:04 |
*** degorenko is now known as _degorenko|afk | 18:04 | |
*** sid14726 has quit IRC | 18:07 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 18:09 |
openstackgerrit | Merged openstack/puppet-openstacklib: Utility to handle IPv6 address brackets. https://review.openstack.org/264927 | 18:13 |
*** jpena is now known as jpena|off | 18:15 | |
*** rcernin has joined #puppet-openstack | 18:16 | |
*** rmart04 has joined #puppet-openstack | 18:22 | |
*** rmart04 has quit IRC | 18:27 | |
*** IvanBerezovskiy has left #puppet-openstack | 18:27 | |
*** chem has quit IRC | 18:38 | |
*** angdraug has quit IRC | 18:39 | |
*** dgurtner has joined #puppet-openstack | 18:40 | |
*** dgurtner has quit IRC | 18:40 | |
*** dgurtner has joined #puppet-openstack | 18:40 | |
*** nlahouti has quit IRC | 18:51 | |
*** nlahouti has joined #puppet-openstack | 19:03 | |
openstackgerrit | David Gurtner proposed openstack/puppet-ceph: Update supported version information https://review.openstack.org/277562 | 19:04 |
*** nlahouti has quit IRC | 19:24 | |
*** nlahouti has joined #puppet-openstack | 19:29 | |
*** dkaigarodsev_ has quit IRC | 19:32 | |
*** nlahouti has quit IRC | 19:32 | |
*** dkaigarodsev has joined #puppet-openstack | 19:34 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class https://review.openstack.org/256248 | 19:43 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer https://review.openstack.org/276141 | 19:46 |
*** nlahouti has joined #puppet-openstack | 19:52 | |
*** vinsh has quit IRC | 20:00 | |
*** vinsh has joined #puppet-openstack | 20:00 | |
*** vinsh_ has joined #puppet-openstack | 20:07 | |
*** vinsh has quit IRC | 20:07 | |
*** dgurtner has quit IRC | 20:18 | |
*** rcernin has quit IRC | 20:24 | |
*** mewald has joined #puppet-openstack | 20:25 | |
mewald | Does puppet-ceph work with infernalis? My monitors are not starting and ceph.conf looks broken. Any experiences? | 20:27 |
*** gchamoul has quit IRC | 20:27 | |
*** rmart04 has joined #puppet-openstack | 20:28 | |
*** gchamoul has joined #puppet-openstack | 20:30 | |
*** jtomasek has quit IRC | 20:31 | |
*** rmart04 has quit IRC | 20:32 | |
*** nlahouti has quit IRC | 20:32 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option https://review.openstack.org/279055 | 20:36 |
*** gfidente has quit IRC | 20:38 | |
openstackgerrit | Merged openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone admin endpoint https://review.openstack.org/267417 | 20:40 |
*** sid14726 has joined #puppet-openstack | 20:57 | |
xarses | mewald: there was a note from EmilienM that it likely did not | 21:01 |
*** richm has joined #puppet-openstack | 21:01 | |
xarses | mewald: its most likely due to the change to support systemd init | 21:03 |
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:04 |
*** jpena|off has quit IRC | 21:14 | |
*** jfluhmann has joined #puppet-openstack | 21:20 | |
*** dprince has quit IRC | 21:31 | |
*** morazi has joined #puppet-openstack | 21:31 | |
*** sid14726 has quit IRC | 21:42 | |
*** richm has quit IRC | 21:42 | |
*** johnmilton has quit IRC | 21:42 | |
yolanda | degorenko, sorry, i finished with my testenv just now. I will test your patches tomorrow | 21:43 |
*** rahulait has joined #puppet-openstack | 21:44 | |
mewald | xarses: thx, that helped :) | 21:57 |
mewald | does it work to use LVM logical volumes as journal disks? | 21:57 |
*** delatte has quit IRC | 22:00 | |
*** _ducttape_ has joined #puppet-openstack | 22:00 | |
*** jayg is now known as jayg|g0n3 | 22:02 | |
*** ducttape_ has quit IRC | 22:03 | |
*** nlahouti has 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:06 |
*** johnmilton has joined #puppet-openstack | 22:09 | |
*** johnmilton has quit IRC | 22:10 | |
*** johnmilton has joined #puppet-openstack | 22:10 | |
openstackgerrit | Sam Morrison proposed openstack/puppet-neutron: Support midonet type driver for ML2 https://review.openstack.org/278207 | 22:29 |
*** sid14726 has joined #puppet-openstack | 22:32 | |
*** andybotting has joined #puppet-openstack | 22:35 | |
*** _ducttape_ has quit IRC | 22:37 | |
*** ducttape_ has joined #puppet-openstack | 22:38 | |
*** morazi has quit IRC | 22:42 | |
*** cdelatte has joined #puppet-openstack | 22:42 | |
*** jfluhmann has quit IRC | 22:51 | |
*** gildub has joined #puppet-openstack | 23:03 | |
*** xarses has quit IRC | 23:11 | |
*** johnmilton has quit IRC | 23:17 | |
*** sid14726 has quit IRC | 23:18 | |
*** saneax_AFK is now known as saneax | 23:19 | |
*** priteau has quit IRC | 23:25 | |
*** priteau_ has joined #puppet-openstack | 23:25 | |
*** mewald has quit IRC | 23:25 | |
*** priteau_ has quit IRC | 23:27 | |
*** baker has quit IRC | 23:31 | |
*** priteau has joined #puppet-openstack | 23:32 | |
*** priteau has quit IRC | 23:34 | |
*** xarses has joined #puppet-openstack | 23:36 | |
*** mewald has joined #puppet-openstack | 23:37 | |
*** mewald has quit IRC | 23:42 | |
*** haplo37 has quit IRC | 23:51 | |
*** mewald has joined #puppet-openstack | 23:53 | |
*** ducttape_ has quit IRC | 23:55 | |
*** mewald has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!