| *** stevemar has quit IRC | 00:05 | |
| *** ducttape_ has joined #puppet-openstack | 00:17 | |
| *** markvoelker has joined #puppet-openstack | 00:21 | |
| *** xingchao has quit IRC | 00:23 | |
| *** markvoelker has quit IRC | 00:26 | |
| *** ducttape_ has quit IRC | 00:28 | |
| *** rcallawa_ has joined #puppet-openstack | 00:39 | |
| *** rcallawa has quit IRC | 00:42 | |
| *** dprince has joined #puppet-openstack | 00:46 | |
| *** dprince has quit IRC | 00:48 | |
| *** ducttape_ has joined #puppet-openstack | 00:52 | |
| *** gildub has joined #puppet-openstack | 00:55 | |
| *** ducttape_ has quit IRC | 01:02 | |
| *** xingchao has joined #puppet-openstack | 01:19 | |
| *** markvoelker has joined #puppet-openstack | 01:22 | |
| *** xingchao has quit IRC | 01:24 | |
| *** markvoelker has quit IRC | 01:27 | |
| *** ducttape_ has joined #puppet-openstack | 02:35 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Restructures authentication for resource providers https://review.openstack.org/181299 | 02:37 |
|---|---|---|
| openstackgerrit | Merged stackforge/puppet-openstacklib: Restructures authentication for resource providers https://review.openstack.org/180407 | 02:44 |
| EmilienM | richm, gildub ^ | 02:45 |
| EmilienM | boom. | 02:45 |
| gildub | big badaboom | 02:45 |
| EmilienM | gildub: I think keystone is broken | 02:48 |
| EmilienM | https://jenkins03.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/19/console | 02:48 |
| gildub | EmilienM, looking | 02:49 |
| richm | 2015-06-10 02:47:40.608 | [1;31mError: /Stage[main]/Main/Keystone::Resource::Service_identity[beaker-ci]/Keystone_user[beaker-ci]: Could not evaluate: can't convert false into String[0m | 02:49 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: do not merge - beaker test https://review.openstack.org/189999 | 02:49 |
| EmilienM | gildub: Could not evaluate: can't convert false into String | 02:49 |
| EmilienM | richm: oops sorry (helllo) | 02:50 |
| EmilienM | I'm checking if keystone/master is broken | 02:51 |
| EmilienM | crinkle: is it me or beaker is really faster with your patches to have latest version? | 02:52 |
| crinkle | ¯\_(ツ)_/¯ | 02:53 |
| crinkle | that's cool if it is | 02:53 |
| EmilienM | gildub, richm, crinkle: ok keystone master is broken | 02:54 |
| EmilienM | on ubuntu/centos | 02:54 |
| *** ducttape_ has quit IRC | 02:55 | |
| EmilienM | see https://review.openstack.org/#/c/189999/ | 02:55 |
| crinkle | EmilienM: we need to merge gildub's puppet-keystone patch at the same time, the openstacklib patch removed the puppet/util/openstack | 02:56 |
| EmilienM | crinkle: yeah but his patch does not pass beaker | 02:56 |
| EmilienM | gildub, richm: have you tested the patches on keystone? | 02:57 |
| gildub | EmilienM, crinkle, richm, with openstacklib master/HEAD which now includes 180407, 181299 passes specs without any trouble, just did it | 02:57 |
| *** ducttape_ has joined #puppet-openstack | 02:57 | |
| crinkle | gildub: but not the beaker-rspec tests | 02:57 |
| EmilienM | gildub: we are talking about functional testing here | 02:58 |
| gildub | EmilienM, crinkle, yeah, not the functional ones | 02:58 |
| EmilienM | gildub: that's the issue now | 02:58 |
| *** ducttape_ has quit IRC | 02:59 | |
| EmilienM | gildub: do you test the patches? | 02:59 |
| crinkle | i tried them, they seemed to work | 03:00 |
| crinkle | running the functional tests now | 03:00 |
| EmilienM | crinkle: since keystone/master is broken now, so all beakers jobs are broken everywhere + tripleo CI - should we revert if we don't find a quick fix? | 03:00 |
| crinkle | EmilienM: let's give ourselves a couple minutes, i think it's probably a really simple issue | 03:01 |
| EmilienM | cool, I'm digging too | 03:01 |
| gildub | EmilienM, no I've never done functional test so far :( | 03:04 |
| *** rushiagr_away is now known as rushiagr | 03:05 | |
| *** markvoelker has joined #puppet-openstack | 03:11 | |
| EmilienM | gildub, crinkle: I'm looking at 'flush' function, are enable/disable options set correctly from 'enabled' param ? | 03:15 |
| *** markvoelker has quit IRC | 03:16 | |
| richm | I have been testing indirectly - that is, I've been rebasing the v3 patches on top of them, and testing them with the v3 patches with packstack on el7 | 03:18 |
| gildub | EmilienM, looking | 03:19 |
| richm | imcsk8 has also been doing packstack testing with these patches | 03:19 |
| richm | probably el7 too | 03:19 |
| EmilienM | richm: could you use Keystone_user like our beaker test is doing ? | 03:23 |
| richm | what is it doing? | 03:24 |
| *** rushiagr is now known as rushiagr_away | 03:25 | |
| EmilienM | hey | 03:28 |
| EmilienM | I don't see ignore_default_tenant | 03:28 |
| EmilienM | it's declared in keystone::resource::service_identity/User | 03:29 |
| richm | return resource[:tenant] if sym_to_bool(resource[:ignore_default_tenant]) | 03:29 |
| EmilienM | damn | 03:29 |
| crinkle | there is a problem with the keystone::resource::service_identity resource in basic_keystone_spec.rb | 03:30 |
| crinkle | it doesn't provide a password for the beaker-ci user | 03:31 |
| crinkle | so when the password method gets called on the second puppet run it is failing | 03:31 |
| crinkle | not sure why this isn't a problem with the current implementation | 03:31 |
| crinkle | oh it's not a problem because we're not using withenv | 03:32 |
| crinkle | withenv needs its argument to be strings | 03:32 |
| crinkle | so just adding password => 'something' to the keystone::resource::service_identity resource in basic_keystone_spec.rb fixes it | 03:33 |
| crinkle | and we should fix that resource to validate that a password was set | 03:33 |
| EmilienM | yes | 03:33 |
| crinkle | gildub: ^ | 03:33 |
| *** richm has quit IRC | 03:35 | |
| gildub | crinkle, ok | 03:36 |
| gildub | crinkle, EmilienM, I'm fixing it | 03:37 |
| EmilienM | cool | 03:38 |
| EmilienM | crinkle: do you mean, fix service_identity to check a password is set if we manage a user ? | 03:38 |
| EmilienM | crinkle: or you want to do that in the provider? | 03:39 |
| crinkle | EmilienM: no i mean keystone::resource::service_identity should be validating it | 03:39 |
| gildub | EmilienM, I understand it's the basic_keystone_spec.rb which needs it, because otherwise there is blank field down the road | 03:40 |
| EmilienM | ok so let's fix in the manifest | 03:40 |
| crinkle | in a different patch | 03:40 |
| EmilienM | crinkle, gildub: I'm doing the validation | 03:40 |
| crinkle | okay | 03:40 |
| gildub | crinkle, +1. | 03:40 |
| EmilienM | gildub: you can just do the patch in the beaker test to have the password | 03:40 |
| EmilienM | so we can merge this patch | 03:40 |
| crinkle | yes | 03:40 |
| EmilienM | and I'll base my patch on your patch | 03:40 |
| gildub | EmilienM, ok, pushing it few seconds | 03:40 |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: Restructures authentication for resource providers https://review.openstack.org/181299 | 03:43 |
| gildub | EmilienM, ^ | 03:43 |
| EmilienM | let's wait for beaker and I'll +2 | 03:43 |
| EmilienM | crinkle: this is weird | 03:49 |
| EmilienM | crinkle: https://github.com/stackforge/puppet-keystone/blob/master/spec/defines/keystone_resource_service_identity_spec.rb#L65-L70 | 03:49 |
| EmilienM | please don't blame this code | 03:50 |
| crinkle | EmilienM: hmm | 03:52 |
| EmilienM | crinkle: in fact I think it used to fail before | 03:52 |
| EmilienM | crinkle, gildub : I +2 | 03:56 |
| gildub | EmilienM, ok great | 03:57 |
| crinkle | +2 +A | 03:57 |
| EmilienM | w00t | 04:00 |
| EmilienM | gildub: next steps are v3 auth support in openstacklib + v3 patches in keystone (rspec is red now I think) | 04:02 |
| gildub | EmilienM, okie dokie | 04:02 |
| EmilienM | gildub: good luck | 04:02 |
| gildub | EmilienM, crinkle: thanks | 04:02 |
| EmilienM | thanks everyone, hopefully crinkle found out the root issue :-) | 04:03 |
| *** rushiagr_away is now known as rushiagr | 04:05 | |
| *** rushiagr is now known as rushiagr_away | 04:05 | |
| *** rushiagr_away is now known as rushiagr | 04:06 | |
| openstackgerrit | Merged stackforge/puppet-keystone: Restructures authentication for resource providers https://review.openstack.org/181299 | 04:06 |
| _ody | ...somehow all my neutron tests went red and I can't identify why. | 04:08 |
| * _ody ignores it for the evening | 04:08 | |
| *** xarses has joined #puppet-openstack | 04:08 | |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using API version 3 https://review.openstack.org/184195 | 04:14 |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3 https://review.openstack.org/178456 | 04:15 |
| *** cwolferh has quit IRC | 04:16 | |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-openstacklib: support for Keystone v3 authentication https://review.openstack.org/187654 | 04:17 |
| *** cwolferh has joined #puppet-openstack | 04:18 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Validate servive_identity resources https://review.openstack.org/190016 | 04:23 |
| EmilienM | crinkle, gildub ^( also good night ) | 04:23 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Validate service_identity resources https://review.openstack.org/190016 | 04:23 |
| *** rushiagr is now known as rushiagr_away | 04:55 | |
| *** markvoelker has joined #puppet-openstack | 05:00 | |
| *** markvoelker has quit IRC | 05:04 | |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 05:10 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x https://review.openstack.org/184646 | 05:11 |
| *** tfz has quit IRC | 05:20 | |
| *** rushiagr_away is now known as rushiagr | 05:25 | |
| *** rcallawa_ has quit IRC | 05:39 | |
| *** prad has quit IRC | 05:39 | |
| *** tfz has joined #puppet-openstack | 05:52 | |
| *** jtomasek has joined #puppet-openstack | 06:00 | |
| *** tshtilma has joined #puppet-openstack | 06:10 | |
| *** saju_m has joined #puppet-openstack | 06:34 | |
| *** rcallawa has joined #puppet-openstack | 06:35 | |
| *** alex_bh has joined #puppet-openstack | 06:35 | |
| *** prad_ has joined #puppet-openstack | 06:48 | |
| *** markvoelker has joined #puppet-openstack | 06:49 | |
| *** markvoelker has quit IRC | 06:53 | |
| *** jpena has joined #puppet-openstack | 07:15 | |
| *** paramite has joined #puppet-openstack | 07:20 | |
| *** saju_m has quit IRC | 07:25 | |
| *** dgurtner has joined #puppet-openstack | 07:29 | |
| *** jistr has joined #puppet-openstack | 07:37 | |
| *** saju_m has joined #puppet-openstack | 07:38 | |
| *** [1]tshtilma has joined #puppet-openstack | 07:58 | |
| *** tshtilma has quit IRC | 07:59 | |
| *** [1]tshtilma is now known as tshtilma | 07:59 | |
| *** gildub has quit IRC | 08:06 | |
| *** saju_m has quit IRC | 08:12 | |
| *** saju_m has joined #puppet-openstack | 08:12 | |
| *** saju_m has quit IRC | 08:13 | |
| *** saju_m has joined #puppet-openstack | 08:14 | |
| *** saju_m has quit IRC | 08:19 | |
| *** aarefiev has quit IRC | 08:28 | |
| *** aarefiev has joined #puppet-openstack | 08:28 | |
| *** paramite has quit IRC | 08:31 | |
| *** markvoelker has joined #puppet-openstack | 08:37 | |
| *** markvoelker has quit IRC | 08:42 | |
| *** saju_m has joined #puppet-openstack | 09:04 | |
| openstackgerrit | Jiri Stransky proposed stackforge/puppet-tripleo: Fencing configuration https://review.openstack.org/188412 | 09:15 |
| *** partner has joined #puppet-openstack | 09:28 | |
| *** arnaud_orange has joined #puppet-openstack | 09:30 | |
| *** alex_bh has quit IRC | 09:30 | |
| openstackgerrit | Jiri Stransky proposed stackforge/puppet-tripleo: Configure fencing devices https://review.openstack.org/188412 | 09:32 |
| *** alex_bh has joined #puppet-openstack | 09:35 | |
| *** rushiagr is now known as rushiagr_away | 09:49 | |
| *** tshtilma has quit IRC | 10:02 | |
| *** tshtilma has joined #puppet-openstack | 10:02 | |
| partner | should i be using stackforge puppet modules with kilo? they all seem to talk about juno. just tried to set up keystone with wsgi and i don't see much happening, most likely my fault but haven't found any working examples yet.. | 10:15 |
| partner | puppetlabs implementation looked juno-centric as well so i'm a bit puzzled should i rely on any of these existing ones or keep doing my own automation, having a bit of schedule pressure here so was hoping to gain some speed from using existing ones and possibly be able also to contribute | 10:21 |
| *** markvoelker has joined #puppet-openstack | 10:23 | |
| *** markvoelker has quit IRC | 10:27 | |
| social | partner: we use masters of puppet openstack modules in in rdo kilo and it works fine, what issues you had? | 10:28 |
| *** derekh has joined #puppet-openstack | 10:38 | |
| *** rcallawa has quit IRC | 10:40 | |
| *** rcallawa has joined #puppet-openstack | 10:41 | |
| *** michchap has joined #puppet-openstack | 10:43 | |
| *** xingchao has joined #puppet-openstack | 10:52 | |
| *** xingchao has quit IRC | 10:56 | |
| partner | social: well, i guess i just don't know how to use it properly :) i was looking at this for starters: https://github.com/stackforge/puppet-keystone/blob/master/manifests/wsgi/apache.pp | 11:08 |
| social | partner: we'll I'd suggest running packstack kilo or any other puppet based installer which works with kilo and looking on the manifests it used | 11:10 |
| partner | hmm i could have a look at the packstack manifests, i used it for initial single-server poc but now doing full-blown setup | 11:12 |
| partner | oh, from packstack found yet another repo "redhat-openstack" with openstack-puppet-modules, even more puzzled now on what to use.. for the record i have centos 7, kilo from rdo and puppet 3.8.1 | 11:14 |
| partner | i think they are the same.. | 11:17 |
| social | partner: opm is collection of upstream modules used by redhat installer, most of them are just snapshots of upstream at some point (when it passed CI for example) and some have patches applied | 11:17 |
| social | patches are usually fixes that are just waiting for review in upstream modules, it's safer to use opm on redhat based platforms but than you first have to check if it's isn't our patch that broke thing before going here | 11:18 |
| partner | rgr that, thanks | 11:19 |
| social | partner: for opm https://github.com/redhat-openstack/openstack-puppet-modules master is unpatched upstream modules master-patches is master + our patches, feel free to use that and use openstack-puppet-modules package if you want something that actually seen QA | 11:19 |
| partner | social: by latter you mean exactly what repo? | 11:22 |
| partner | oh, rpm package? | 11:22 |
| social | rpm package | 11:22 |
| partner | roger that, thanks for clarifying | 11:22 |
| *** markvoelker has joined #puppet-openstack | 11:24 | |
| *** markvoelker has quit IRC | 11:28 | |
| *** ducttape_ has joined #puppet-openstack | 11:32 | |
| partner | hmm, weird, its like its enabling the service and not the wsgi.. | 11:40 |
| *** ducttape_ has quit IRC | 11:45 | |
| partner | i think i am missing the service_name which defaults to keystone ie. standalone service, should be set to httpd instead | 11:53 |
| *** markvoelker has joined #puppet-openstack | 11:54 | |
| *** dprince has joined #puppet-openstack | 11:59 | |
| partner | ok it now tried to do something but less suprisingly fails again.. | 12:05 |
| *** rushiagr_away is now known as rushiagr | 12:06 | |
| *** tshtilma has quit IRC | 12:06 | |
| partner | there's token_provider but no token_driver? should point to memcache backend but from somewhere it gets the sql there | 12:07 |
| *** xarses has quit IRC | 12:11 | |
| EmilienM | good morning folks! | 12:19 |
| *** jayg|g0n3 is now known as jayg | 12:21 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Validate service_identity resources https://review.openstack.org/190016 | 12:27 |
| partner | so umm, no examples of functional keystone wsgi setup? puppet keeps fighting back :/ | 12:28 |
| *** jpena is now known as jpena|lunch | 12:28 | |
| *** ducttape_ has joined #puppet-openstack | 12:29 | |
| *** ducttape_ has quit IRC | 12:32 | |
| *** prad_ has quit IRC | 12:34 | |
| *** prad_ has joined #puppet-openstack | 12:35 | |
| *** tshtilma has joined #puppet-openstack | 12:38 | |
| *** iurygregory has joined #puppet-openstack | 12:44 | |
| openstackgerrit | Yanis Guenane proposed stackforge/puppet-neutron: Opencontrail: Create neutron::plugins::opencontrail https://review.openstack.org/187986 | 12:47 |
| partner | do i need to install some additional modules for keystone? i keep hitting to this error: Error 400 on SERVER: Could not find resource 'Service[httpd]' for relationship from 'Keystone_config[DEFAULT/admin_token]' | 12:52 |
| *** skolekonov has joined #puppet-openstack | 12:57 | |
| *** tshtilma has quit IRC | 13:01 | |
| skolekonov | Hi. Could you please help with rspec tests for puppet-neutron manifests? I cloned the repo and switched to 5.0.0 tag (Juno). Then I tried to execute rspec tests and got 48 failures. Is it expected or I did something wrong? | 13:01 |
| skolekonov | I also exported 'PUPPET_GEM_VERSION=~> 3.4.0' | 13:02 |
| skolekonov | I mean, are tests out of sync with manifests for 5.0.0 tag? | 13:06 |
| *** ferest has joined #puppet-openstack | 13:06 | |
| *** richm has joined #puppet-openstack | 13:11 | |
| *** rushiagr is now known as rushiagr_away | 13:20 | |
| *** arnaud_orange has quit IRC | 13:29 | |
| *** ferest has quit IRC | 13:36 | |
| rhefner | when someone gets a chance, can I get a final +2 on https://review.openstack.org/#/c/189771/ ? Thanks! | 13:40 |
| *** xingchao has joined #puppet-openstack | 13:41 | |
| trash | I don't see the point of the double if there, seems like cosmetics | 13:43 |
| *** morazi has joined #puppet-openstack | 13:43 | |
| *** jpena|lunch is now known as jpena | 13:45 | |
| *** xingchao has quit IRC | 13:45 | |
| *** saju_m has quit IRC | 13:51 | |
| *** cdent has joined #puppet-openstack | 13:54 | |
| *** tshtilma has joined #puppet-openstack | 13:55 | |
| skolekonov | Actually it was my fault, rspec-puppet ~> 1.0.1 must be used | 14:01 |
| skolekonov | Though I had 2.2.0 | 14:02 |
| *** arnaud_orange has joined #puppet-openstack | 14:02 | |
| *** xingchao has joined #puppet-openstack | 14:09 | |
| *** britthouser has joined #puppet-openstack | 14:10 | |
| *** tshtilma has quit IRC | 14:12 | |
| *** degorenko has quit IRC | 14:12 | |
| *** britthou_ has quit IRC | 14:12 | |
| richm | question about the v3 work - we need to get https://review.openstack.org/187654 in puppet-openstacklib merged first | 14:15 |
| *** tfz has quit IRC | 14:15 | |
| richm | Is this ready to be merged? | 14:15 |
| richm | If not, what needs to be done? | 14:15 |
| *** degorenko has joined #puppet-openstack | 14:19 | |
| EmilienM | richm: beaker tests seem to fail but it's not related to your patch | 14:19 |
| richm | crinkle really wants to get the bp in first, before any of the patches land | 14:22 |
| richm | What do I need to do to get the bp merged? https://review.openstack.org/150108 | 14:23 |
| *** alex_bh has quit IRC | 14:23 | |
| *** alex_bh has joined #puppet-openstack | 14:24 | |
| EmilienM | richm: make it pass jenkins | 14:25 |
| EmilienM | I'm in logs now | 14:25 |
| EmilienM | richm: http://logs.openstack.org/08/150108/14/check/gate-puppet-openstack-specs-docs/8eb378f/console.html#_2015-06-08_21_19_58_453 | 14:25 |
| EmilienM | richm: can you fix it? | 14:26 |
| EmilienM | otherwise the BP is good to me | 14:26 |
| *** kkirkpatrick has joined #puppet-openstack | 14:35 | |
| *** bdemers has joined #puppet-openstack | 14:39 | |
| *** openstackgerrit has quit IRC | 14:41 | |
| *** openstackgerrit has joined #puppet-openstack | 14:41 | |
| *** alex_bh has quit IRC | 14:45 | |
| *** tfz has joined #puppet-openstack | 14:45 | |
| partner | http://fpaste.org/230746/33947850/ - any hints on how to resolve that, do i need to install some additional modules? i basically want kilo keystone to run as wsgi, the docs aren't too verbose on how to achieve that properly so i'm doing plenty of guess work here | 14:52 |
| partner | hmm i am probably lacking the openstacklib module.. | 15:00 |
| richm | EmilienM: How can I run that command on my local machine? | 15:00 |
| richm | EmilienM: I've tried to fix that, but I'm not sure why it's wrong - I use the same indentation elsewhere | 15:01 |
| EmilienM | richm: I'll check later | 15:08 |
| *** dgurtner has quit IRC | 15:09 | |
| *** britthouser has quit IRC | 15:18 | |
| *** britthouser has joined #puppet-openstack | 15:19 | |
| *** mdorman has joined #puppet-openstack | 15:20 | |
| *** tfz has quit IRC | 15:21 | |
| *** skolekonov has quit IRC | 15:30 | |
| *** fedexo has joined #puppet-openstack | 15:30 | |
| EmilienM | any core around could have a look at https://review.openstack.org/#/c/182724/ ? It's a kilo blocker | 15:51 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Revert "Revert "remove POSIX users, groups and file modes"" https://review.openstack.org/190237 | 15:52 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all https://review.openstack.org/189908 | 15:55 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: do not merge - beaker test https://review.openstack.org/190242 | 15:58 |
| *** rushiagr_away is now known as rushiagr | 16:06 | |
| *** jistr has quit IRC | 16:14 | |
| *** arnaud_orange has quit IRC | 16:15 | |
| richm | virtualenv .venv ; source .venv/bin/activate ; pip install -r requirements.txt ; tox -edocs | 16:29 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 16:30 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x https://review.openstack.org/184646 | 16:30 |
| *** stevemar has joined #puppet-openstack | 16:31 | |
| *** jpena has quit IRC | 16:37 | |
| *** xingchao has quit IRC | 16:41 | |
| *** prad_ is now known as prad | 16:42 | |
| *** xingchao has joined #puppet-openstack | 16:43 | |
| openstackgerrit | Merged stackforge/puppet-neutron: LBaaS: update device_driver for Kilo https://review.openstack.org/182724 | 16:43 |
| EmilienM | crinkle: I was wondering about backporting identity_url or not ( https://review.openstack.org/#/c/188570/ ) | 16:49 |
| EmilienM | crinkle: afik, it's only required in Kilo | 16:49 |
| EmilienM | dprince & mgagne can confirm maybe | 16:49 |
| *** fedexo has quit IRC | 16:52 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-glance: Use OpenstackClient for glance_image auth https://review.openstack.org/172580 | 17:00 |
| EmilienM | just a rebase ^ | 17:00 |
| mgagne | EmilienM: required in kilo? is it deprecated in juno? is it still available in kilo? | 17:02 |
| *** derekh has quit IRC | 17:04 | |
| crinkle | EmilienM: i thought auth_host etc was deprecated in juno so backporting would make sense, but i'm not sure | 17:05 |
| mgagne | looks like it's deprecated in kilo, the patch has deprecation warnings | 17:07 |
| mgagne | the point is that we can't ask our user to move to identity_uri because "auth_host" is deprecated when in fact, it's not (in juno) | 17:07 |
| crinkle | okay, i will drop those backports | 17:08 |
| crinkle | thanks mgagne | 17:08 |
| *** xingchao has quit IRC | 17:25 | |
| *** dgurtner has joined #puppet-openstack | 17:26 | |
| *** RichardRaseley has joined #puppet-openstack | 17:28 | |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-openstack-specs: Support Keystone v3 API in openstack puppet modules https://review.openstack.org/150108 | 17:49 |
| EmilienM | cool, thanks crinkle and mgagne ! | 17:49 |
| richm | ^^^ - v3 bp fixed and update | 17:50 |
| EmilienM | richm: +2 | 17:59 |
| richm | EmilienM: Thanks | 18:02 |
| *** ducttape_ has joined #puppet-openstack | 18:04 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Revert "Revert "remove POSIX users, groups and file modes"" https://review.openstack.org/190237 | 18:04 |
| *** ericpeterson has joined #puppet-openstack | 18:05 | |
| *** ducttape_ has quit IRC | 18:08 | |
| crinkle | looks like it's still failing | 18:09 |
| richm | ok - I guess I can't have an additional section - geez, debugging the docs is harder than debugging the code . . . | 18:09 |
| crinkle | haha | 18:10 |
| *** stevemar has quit IRC | 18:10 | |
| crinkle | i think it looks fine to me, mfisch could you have a look at https://review.openstack.org/#/c/150108 since you know keystone pretty well? | 18:10 |
| *** dgurtner has quit IRC | 18:10 | |
| crinkle | and anyone else ^ it's going to be a big change, lots of input would be good | 18:13 |
| *** tfz has joined #puppet-openstack | 18:14 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all https://review.openstack.org/189908 | 18:15 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all https://review.openstack.org/189908 | 18:15 |
| *** rushiagr is now known as rushiagr_away | 18:23 | |
| *** cdent has quit IRC | 18:25 | |
| EmilienM | I think some beakers tests might be broken because rabbitmq/master is now using puppetlabs-apt 2.0 | 18:31 |
| *** dprince has quit IRC | 18:31 | |
| crinkle | EmilienM: we can probably go back to installing rabbitmq from the forge, right? | 18:33 |
| EmilienM | I'm doing the patch | 18:33 |
| crinkle | cool | 18:33 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 18:34 |
| *** RichardRaseley has quit IRC | 18:36 | |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-openstack-specs: Support Keystone v3 API in openstack puppet modules https://review.openstack.org/150108 | 18:48 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 18:51 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x https://review.openstack.org/184646 | 18:52 |
| _ody | sbadia: You mind if I submit another patch set for https://review.openstack.org/#/c/177959? I found another issue with the Rakefile. Or if someone could tell me if there is a way in Gerrit to propse patch sets to patch sets. | 18:54 |
| _ody | Nvm on this one actually...it is in the config_defaults.yml. I'll just leave a comment. | 18:55 |
| openstackgerrit | Merged stackforge/puppet-glance: The package python-ceph no longer exists in el7, use python-rbd. https://review.openstack.org/188200 | 18:56 |
| *** dgurtner has joined #puppet-openstack | 19:03 | |
| *** dgurtner has quit IRC | 19:03 | |
| *** dgurtner has joined #puppet-openstack | 19:03 | |
| *** xingchao has joined #puppet-openstack | 19:14 | |
| mfisch | crinkle: looking | 19:16 |
| mfisch | crinkle: ah thats not a 10 min, will look tonght | 19:18 |
| *** prad has quit IRC | 19:23 | |
| *** jtomasek has quit IRC | 19:24 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-trove: Fix catalog compilation when not configuring endpoint https://review.openstack.org/190308 | 19:25 |
| *** paramite has joined #puppet-openstack | 19:26 | |
| *** xingchao has quit IRC | 19:27 | |
| *** stevemar has joined #puppet-openstack | 19:30 | |
| *** paramite has quit IRC | 19:36 | |
| *** prad_ has joined #puppet-openstack | 19:53 | |
| *** prad_ is now known as prad | 19:54 | |
| *** jpena has joined #puppet-openstack | 19:55 | |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 20:05 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x https://review.openstack.org/184646 | 20:06 |
| openstackgerrit | Rajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver https://review.openstack.org/189372 | 20:20 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 20:25 |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x https://review.openstack.org/184646 | 20:27 |
| *** jpena has quit IRC | 20:31 | |
| *** stevemar has quit IRC | 20:38 | |
| *** jayg is now known as jayg|g0n3 | 20:45 | |
| *** stevemar has joined #puppet-openstack | 20:52 | |
| _ody | Could someone with familiarity of beaker failures take a look at https://review.openstack.org/#/c/184646? | 21:01 |
| _ody | I swore it wasn't failing as of this morning and I have changed any code... | 21:01 |
| crinkle | _ody: it looks like it failed to install a package, i would just recheck it | 21:05 |
| _ody | woohoo! All green, https://review.openstack.org/#/c/184646/17. | 21:24 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed stackforge/puppet-openstack-specs: Enabling-k2k-federation https://review.openstack.org/190361 | 21:26 |
| *** RichardRaseley has joined #puppet-openstack | 21:36 | |
| EmilienM | rlaurika: https://review.openstack.org/#/c/190308/ | 21:47 |
| EmilienM | rlaurika: let me know if Co-Authored-By is enough for you, I fixed the bug for trove using your code | 21:48 |
| EmilienM | and I also put gchamoul who was commiter | 21:48 |
| *** stevemar has quit IRC | 22:06 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all https://review.openstack.org/189908 | 22:07 |
| *** aimon has quit IRC | 22:18 | |
| *** aimon has joined #puppet-openstack | 22:21 | |
| EmilienM | crinkle: I was thinking at gating puppet-keystone in puppet-openstacklib jobs | 22:21 |
| EmilienM | crinkle: I don't know if you remember but our issue was the git commit ID that is passed thru zuul, something that is not possible now with our acceptance config | 22:22 |
| EmilienM | do you think it's worth to gate keystone in openstacklib gate? | 22:23 |
| crinkle | ¯\_(ツ)_/¯ | 22:26 |
| EmilienM | lol I love this thing ^ | 22:27 |
| EmilienM | I'm sure crinkle use an alias for that! | 22:27 |
| EmilienM | a keyboard binding I mean | 22:27 |
| crinkle | EmilienM: https://github.com/cmurphy/weechat-shrug | 22:27 |
| EmilienM | oh dear. | 22:27 |
| *** stevemar has joined #puppet-openstack | 22:29 | |
| EmilienM | RichardRaseley: thank you for your +1 but as you can see, the beaker jobs are not all green. There is some issue on trusty with the pki_setup | 22:31 |
| EmilienM | on https://review.openstack.org/#/c/189908/ | 22:32 |
| *** ericpeterson has quit IRC | 22:32 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all https://review.openstack.org/189908 | 22:34 |
| EmilienM | dmsimard: are you ok if we "won't fix" https://bugs.launchpad.net/puppet-nova/+bug/1451460 ? | 22:35 |
| openstack | Launchpad bug 1451460 in puppet-nova "Gate job puppet-syntax-future requires rspec-puppet 2.x and rspec 3.x" [High,New] | 22:35 |
| EmilienM | I think our focus is now on juno/kilo/future liberty | 22:36 |
| dmsimard | EmilienM: Well, didn't you guys backport that commit everywhere ? | 22:36 |
| EmilienM | dmsimard: I'm not sure everywhere | 22:36 |
| dmsimard | I remember sbadia commiting the fix in the previous branches | 22:36 |
| dmsimard | Would've expected a related-bug in the commit perhaps | 22:36 |
| EmilienM | dmsimard: not sure he did for all repos | 22:36 |
| RichardRaseley | EmilienM: I was just +1 the code, as it looked semantically valid. Should I always wait to add my feedback until tests pass / fail? I thought that would be taken into account by whoever uses their +2. | 22:37 |
| dmsimard | Regardless, you also made it non-voting so I guess it's sort of fixed | 22:37 |
| EmilienM | dmsimard: my question is, do you think it's worth to 1/ backport the patches 2/ enable voting? I don't think so. | 22:37 |
| *** dgurtner has quit IRC | 22:37 | |
| dmsimard | Neither do I | 22:38 |
| EmilienM | cool | 22:38 |
| EmilienM | I close the bug | 22:38 |
| dmsimard | I wouldn't have backported the patches | 22:38 |
| dmsimard | :P | 22:38 |
| dmsimard | But he did anyway | 22:38 |
| *** morazi has quit IRC | 22:39 | |
| EmilienM | there is *a lot* of untriaged bugs | 22:39 |
| EmilienM | richm: can I have a status on https://bugs.launchpad.net/puppet-openstacklib/+bug/1451963 ? IIRC we can close it | 22:40 |
| openstack | Launchpad bug 1451963 in puppet-openstacklib "need to support all of the keystone v3 auth options" [High,New] - Assigned to Emilien Macchi (emilienm) | 22:40 |
| EmilienM | I'm not sure this one is still up to date with all our recent design changes | 22:41 |
| richm | EmilienM: yes, we can - closed as wontfix or invalid since it has been superseded by the new auth restructure code | 22:41 |
| EmilienM | richm: ack | 22:41 |
| EmilienM | mfisch: hey, are you still facing this bug ? https://bugs.launchpad.net/puppet-horizon/+bug/1332635 - I've looked at the code and it might be fixed in master | 22:45 |
| openstack | Launchpad bug 1332635 in puppet-horizon "cannot disable ssl_redirect when listen_ssl is on" [Medium,New] | 22:45 |
| EmilienM | social, imcsk8: I'm experiencing some issues with pki_setup on puppet-keystone, only with centos7 | 22:52 |
| EmilienM | http://logs.openstack.org/08/189908/6/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/068dc5d/console.html | 22:52 |
| EmilienM | social, imcsk8: can you tell me if it works in packstack? | 22:52 |
| EmilienM | mgagne: stop me if I'm really wrong but I'm thinking about dropping monolitic plugins in puppet-neutron (ovs&lb) | 23:00 |
| *** kkirkpatrick has quit IRC | 23:01 | |
| mgagne | EmilienM: have they been officially deprecated in Neutron? What's the status on that one? | 23:01 |
| RichardRaseley | EmilienM: Is there a way for us to get the full output of the Puppet run on that box? | 23:01 |
| RichardRaseley | (RE: http://logs.openstack.org/08/189908/6/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/068dc5d/console.html) | 23:01 |
| EmilienM | RichardRaseley: like here ? http://logs.openstack.org/08/189908/6/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/068dc5d/console.html#_2015-06-10_22_39_36_966 | 23:02 |
| EmilienM | mgagne: https://github.com/openstack/neutron/blob/master/neutron/plugins/openvswitch/README | 23:03 |
| RichardRaseley | EmilienM: Derp. Yes, that - was just looking at "last 10 lines" | 23:03 |
| mgagne | haha, so it's gone | 23:03 |
| mgagne | or not? | 23:03 |
| EmilienM | I don't see many code in this dir | 23:04 |
| EmilienM | https://github.com/openstack/neutron/tree/master/neutron/plugins/openvswitch | 23:04 |
| mgagne | hehe | 23:04 |
| EmilienM | same here: https://github.com/openstack/neutron/tree/master/neutron/plugins/linuxbridge | 23:04 |
| EmilienM | The Neutron Linux Bridge plugin has removed from the tree in Juno. | 23:04 |
| EmilienM | it's pretty clear mgagne | 23:04 |
| mgagne | yep | 23:04 |
| EmilienM | mgagne: should we drop the code in puppet or deprecate? | 23:04 |
| EmilienM | hmm | 23:04 |
| mgagne | I think we should remove it then unless someone uses master to run icehouse with deprecated plugin | 23:05 |
| EmilienM | lol | 23:05 |
| EmilienM | he's brave. | 23:05 |
| EmilienM | he/she's brave | 23:05 |
| mgagne | we should ask the ML then | 23:07 |
| mgagne | just in case | 23:07 |
| EmilienM | mgagne: ok I'm asking | 23:08 |
| EmilienM | "is someone brave here?" | 23:08 |
| mgagne | haha | 23:09 |
| *** xingchao has joined #puppet-openstack | 23:14 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-neutron: Drop OVS & LB monolitic plugins https://review.openstack.org/190395 | 23:15 |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-glance: Use OpenstackClient for glance_image auth https://review.openstack.org/172580 | 23:16 |
| *** mdorman has quit IRC | 23:18 | |
| *** xingchao has quit IRC | 23:21 | |
| EmilienM | mgagne: I won't touch to cisco/nvp/plumgrid because AFIK they don't have all ML2 | 23:22 |
| mgagne | yep | 23:22 |
| *** gildub has joined #puppet-openstack | 23:25 | |
| *** ducttape_ has joined #puppet-openstack | 23:28 | |
| *** xingchao has joined #puppet-openstack | 23:35 | |
| *** ducttape_ has quit IRC | 23:38 | |
| *** ducttape_ has joined #puppet-openstack | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!