| *** jerrygb has quit IRC | 00:15 | |
| *** gildub has joined #puppet-openstack | 00:24 | |
| *** ducttape_ has quit IRC | 00:25 | |
| *** ducttape_ has joined #puppet-openstack | 00:34 | |
| *** saneax is now known as saneax_AFK | 00:38 | |
| *** ducttape_ has quit IRC | 00:41 | |
| *** zhangjn has joined #puppet-openstack | 01:08 | |
| *** ducttape_ has joined #puppet-openstack | 01:08 | |
| openstackgerrit | David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts https://review.openstack.org/252664 | 01:16 |
|---|---|---|
| *** ducttape_ has quit IRC | 01:17 | |
| *** tchaypo_cloud has quit IRC | 01:21 | |
| *** ducttape_ has joined #puppet-openstack | 01:52 | |
| *** ducttape_ has quit IRC | 01:52 | |
| *** ducttape_ has joined #puppet-openstack | 01:56 | |
| *** zhangjn has quit IRC | 02:10 | |
| *** ducttape_ has quit IRC | 02:11 | |
| *** ducttape_ has joined #puppet-openstack | 02:23 | |
| *** ducttape_ has quit IRC | 02:32 | |
| *** ilbot3 has quit IRC | 02:45 | |
| *** ilbot3 has joined #puppet-openstack | 02:47 | |
| *** ducttape_ has joined #puppet-openstack | 02:53 | |
| *** zhangjn has joined #puppet-openstack | 03:03 | |
| *** zhangjn has quit IRC | 03:03 | |
| *** zhangjn has joined #puppet-openstack | 03:05 | |
| *** zhangjn has quit IRC | 03:15 | |
| *** ducttape_ has quit IRC | 03:20 | |
| *** ducttape_ has joined #puppet-openstack | 03:55 | |
| *** baker has joined #puppet-openstack | 04:06 | |
| *** ducttape_ has quit IRC | 04:10 | |
| *** zhangjn has joined #puppet-openstack | 04:32 | |
| *** ducttape_ has joined #puppet-openstack | 04:33 | |
| *** baker has quit IRC | 04:55 | |
| *** rahulait has joined #puppet-openstack | 05:00 | |
| *** saneax_AFK is now known as saneax | 05:02 | |
| *** ducttape_ has quit IRC | 05:10 | |
| *** ducttape_ has joined #puppet-openstack | 05:23 | |
| *** vishpat has joined #puppet-openstack | 05:24 | |
| vishpat | Hi, I have addressed all of the review comments for https://review.openstack.org/#/c/247239 How do I proceed further with this code review ? | 05:25 |
| *** ducttape_ has quit IRC | 05:28 | |
| *** zhangjn has quit IRC | 05:33 | |
| *** vishpat has quit IRC | 05:33 | |
| *** nlahouti has joined #puppet-openstack | 05:40 | |
| *** zhangjn has joined #puppet-openstack | 05:49 | |
| *** Mosibi_ has joined #puppet-openstack | 05:52 | |
| *** Mosibi_ has quit IRC | 05:53 | |
| *** Mosibi has quit IRC | 05:53 | |
| *** Mosibi has joined #puppet-openstack | 05:53 | |
| *** nlahouti has quit IRC | 06:00 | |
| *** zhangjn has quit IRC | 06:05 | |
| *** nlahouti has joined #puppet-openstack | 06:06 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-glance: Remove unused cluster_id parameter https://review.openstack.org/254019 | 06:06 |
| *** zhangjn has joined #puppet-openstack | 06:07 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-cinder: Deprecate cluster_id parameter https://review.openstack.org/254020 | 06:08 |
| *** ducttape_ has joined #puppet-openstack | 06:24 | |
| *** ducttape_ has quit IRC | 06:30 | |
| *** openstackgerrit_ has joined #puppet-openstack | 06:51 | |
| *** nlahouti has quit IRC | 06:55 | |
| *** leanderthal has joined #puppet-openstack | 07:06 | |
| *** nlahouti has joined #puppet-openstack | 07:10 | |
| *** jtomasek has quit IRC | 07:10 | |
| *** ducttape_ has joined #puppet-openstack | 07:27 | |
| *** foxhunt_away is now known as foxhunt72 | 07:29 | |
| *** ducttape_ has quit IRC | 07:31 | |
| *** gildub has quit IRC | 07:32 | |
| *** nlahouti has quit IRC | 07:45 | |
| *** zhangjn has quit IRC | 07:55 | |
| *** khappone has joined #puppet-openstack | 07:57 | |
| *** jpena has joined #puppet-openstack | 08:06 | |
| *** zhangjn has joined #puppet-openstack | 08:16 | |
| *** dgurtner has joined #puppet-openstack | 08:18 | |
| *** dgurtner has quit IRC | 08:18 | |
| *** dgurtner has joined #puppet-openstack | 08:18 | |
| *** ducttape_ has joined #puppet-openstack | 08:27 | |
| *** throwawaymonday has joined #puppet-openstack | 08:32 | |
| *** ducttape_ has quit IRC | 08:32 | |
| *** zhangjn has quit IRC | 08:33 | |
| *** gfidente has joined #puppet-openstack | 08:38 | |
| *** gfidente has quit IRC | 08:38 | |
| *** gfidente has joined #puppet-openstack | 08:38 | |
| *** zhangjn has joined #puppet-openstack | 08:46 | |
| *** zhangjn has quit IRC | 08:46 | |
| *** rochaporto has joined #puppet-openstack | 08:51 | |
| *** zhangjn has joined #puppet-openstack | 09:00 | |
| *** zhangjn has quit IRC | 09:04 | |
| *** zhangjn has joined #puppet-openstack | 09:09 | |
| *** zhangjn has quit IRC | 09:10 | |
| *** zhangjn has joined #puppet-openstack | 09:13 | |
| *** zhangjn has quit IRC | 09:13 | |
| *** paramite has joined #puppet-openstack | 09:27 | |
| *** ducttape_ has joined #puppet-openstack | 09:28 | |
| *** omartsyniuk has quit IRC | 09:29 | |
| *** zhangjn has joined #puppet-openstack | 09:32 | |
| *** ducttape_ has quit IRC | 09:32 | |
| *** jistr has joined #puppet-openstack | 09:36 | |
| *** jtomasek has joined #puppet-openstack | 09:37 | |
| *** chem has joined #puppet-openstack | 09:40 | |
| *** zhangjn has quit IRC | 09:40 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-ceph: Fix unit tests failing against Puppet 4.3.x https://review.openstack.org/254095 | 09:45 |
| *** zhangjn has joined #puppet-openstack | 09:47 | |
| *** zhangjn has quit IRC | 09:47 | |
| openstackgerrit | Matthew Mosesohn proposed openstack/puppet-cinder: Add keymgr/encryption_auth_url parameter to cinder API https://review.openstack.org/254099 | 09:49 |
| *** igajsin has joined #puppet-openstack | 09:53 | |
| *** igajsin has left #puppet-openstack | 09:53 | |
| *** _degorenko|afk is now known as degorenko | 09:58 | |
| degorenko | EmilienM, looking | 09:59 |
| *** _skolekonov is now known as skolekonov | 10:07 | |
| *** davideagle_ has joined #puppet-openstack | 10:10 | |
| *** rochaporto has quit IRC | 10:13 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2 https://review.openstack.org/249711 | 10:14 |
| *** ianw has quit IRC | 10:17 | |
| *** openstackgerrit_ has quit IRC | 10:23 | |
| mattymo | https://review.openstack.org/#/c/248150/ (puppet-swift) can I get some more reviewers on this? | 10:23 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2 https://review.openstack.org/253416 | 10:23 |
| mattymo | maybe skolekonov? | 10:24 |
| skolekonov | mattymo, looking | 10:24 |
| *** ducttape_ has joined #puppet-openstack | 10:29 | |
| *** jistr has quit IRC | 10:29 | |
| *** jistr has joined #puppet-openstack | 10:30 | |
| *** paramite is now known as paramite|afk | 10:30 | |
| *** ducttape_ has quit IRC | 10:33 | |
| *** karimb has joined #puppet-openstack | 10:37 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: remove POSIX users, groups and file modes https://review.openstack.org/195135 | 10:42 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: remove POSIX users, groups and file modes https://review.openstack.org/195135 | 10:48 |
| *** IvanBerezovskiy1 has joined #puppet-openstack | 10:49 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: remove POSIX users, groups and file modes https://review.openstack.org/195135 | 10:53 |
| *** rahulait has quit IRC | 10:58 | |
| *** davideagle_ has quit IRC | 10:59 | |
| *** yottatsa has joined #puppet-openstack | 11:03 | |
| *** davideagle_ has joined #puppet-openstack | 11:10 | |
| *** davideagle_ has quit IRC | 11:22 | |
| *** trown is now known as trown|flying | 11:26 | |
| *** paramite|afk is now known as paramite | 11:29 | |
| *** ducttape_ has joined #puppet-openstack | 11:30 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes. https://review.openstack.org/211593 | 11:30 |
| *** ducttape_ has quit IRC | 11:34 | |
| *** davideagle_ has joined #puppet-openstack | 11:34 | |
| *** bogdando has quit IRC | 11:35 | |
| *** jtomasek has quit IRC | 11:42 | |
| *** jtomasek has joined #puppet-openstack | 11:45 | |
| *** jtomasek has quit IRC | 11:46 | |
| *** jtomasek has joined #puppet-openstack | 11:48 | |
| *** bogdando has joined #puppet-openstack | 11:49 | |
| *** iurygregory has joined #puppet-openstack | 11:52 | |
| iurygregory | good morning people | 11:53 |
| skolekonov | good morning | 11:54 |
| EmilienM | good morning! | 12:04 |
| *** paramite is now known as paramite|afk | 12:07 | |
| *** davideagle_ has quit IRC | 12:07 | |
| *** paramite|afk is now known as paramite | 12:09 | |
| degorenko | morning o/ | 12:18 |
| iurygregory | =) | 12:19 |
| *** cdelatte has joined #puppet-openstack | 12:22 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2 https://review.openstack.org/249711 | 12:23 |
| *** jpena is now known as jpena|lunch | 12:29 | |
| *** davideagle_ has joined #puppet-openstack | 12:30 | |
| *** ducttape_ has joined #puppet-openstack | 12:30 | |
| *** ducttape_ has quit IRC | 12:35 | |
| EmilienM | degorenko: I have some patches you could maybe look | 12:36 |
| EmilienM | degorenko: https://goo.gl/os5PzS | 12:37 |
| degorenko | EmilienM, looking | 12:37 |
| degorenko | i have for you too :D a bunch of patches | 12:37 |
| EmilienM | degorenko: https://review.openstack.org/#/c/252562/ - https://review.openstack.org/252003 and https://review.openstack.org/252077 | 12:37 |
| openstackgerrit | Merged openstack/puppet-openstack-specs: Enhance inifile option deprecations spec https://review.openstack.org/240515 | 12:37 |
| EmilienM | clayton: ^ | 12:38 |
| clayton | yay! | 12:38 |
| EmilienM | degorenko: I'm trying to add puppet jobs on http://status.openstack.org/openstack-health/#/ | 12:38 |
| EmilienM | so we will have a nice monitoring of our CI | 12:38 |
| EmilienM | clayton: congrats! | 12:38 |
| iurygregory | nice =D | 12:38 |
| degorenko | EmilienM, clayton https://review.openstack.org/#/q/topic:deprecate_qpid,n,z and https://review.openstack.org/#/q/topic:remove_qpid,n,z | 12:39 |
| EmilienM | degorenko: ok - I'll look at them today | 12:39 |
| degorenko | EmilienM, wow, monitoring - nice | 12:39 |
| *** yottatsa has quit IRC | 12:45 | |
| clayton | degorenko: I didn't understand your comment here - https://review.openstack.org/#/c/252693/4/manifests/generic_service.pp | 12:45 |
| *** ducttape_ has joined #puppet-openstack | 12:45 | |
| degorenko | clayton, sec | 12:47 |
| degorenko | EmilienM, https://review.openstack.org/#/c/252562/ take a look here | 12:47 |
| degorenko | clayton, ugh | 12:47 |
| degorenko | sorry | 12:48 |
| degorenko | misread | 12:48 |
| clayton | np | 12:48 |
| degorenko | clayton, your patch looks good actually, some doubts about anchors and requires for some services | 12:49 |
| clayton | how so? | 12:49 |
| degorenko | it is just because i'm not familiar with anchors, i'm not use it before, and just hard to understand some things | 12:50 |
| clayton | they're basically just dummy resources. they propagate notifies if they receive them | 12:51 |
| degorenko | yeah, i know what it is | 12:51 |
| *** rahulait has joined #puppet-openstack | 12:52 | |
| degorenko | clayton, for example https://review.openstack.org/#/c/252693/4/manifests/init.pp you removed conf file for nova | 12:53 |
| degorenko | some services need this file | 12:53 |
| clayton | we've been removing that from all of the modules actually | 12:54 |
| degorenko | but it doesn't present in deps class | 12:54 |
| degorenko | oh | 12:54 |
| clayton | that declaration doesn't actually do anything | 12:54 |
| degorenko | just a some mix | 12:54 |
| degorenko | :) | 12:54 |
| degorenko | i was confused by this deletion | 12:55 |
| clayton | and when you start integrating with non-package mechanisms for installing software, having the modules managing the config files ends up making things a lot harder | 12:55 |
| clayton | degorenko: this is our module we're using with heat and designate right now, which already have a similar patch merged: https://github.com/twc-openstack/puppet-os_docker | 12:55 |
| clayton | this is the original module I did for supporting both docker and virtualenv with the anchors/hooks approach - https://github.com/twc-openstack/puppet-designate_ext | 12:56 |
| *** rahulait has quit IRC | 12:57 | |
| degorenko | clayton, ok, more less i understand. Thanks | 12:57 |
| clayton | I should probably write spec for this approach. With designate we were still kind of figuring out how it would work. I think with nova I have a pretty good idea of how to do it now | 12:58 |
| EmilienM | degorenko: ok i'll change it | 12:59 |
| degorenko | clayton, ok :) i answered you https://review.openstack.org/#/c/253042/1//COMMIT_MSG | 13:00 |
| clayton | gerrit upgrade is this week? | 13:00 |
| clayton | degorenko I'm fine with the patch, I just thought the commit message wasn't accurate :) | 13:01 |
| *** ducttape_ has quit IRC | 13:01 | |
| degorenko | clayton, no, on next week. 16th Dec | 13:01 |
| clayton | ah, too bad. I'm looking forward to it | 13:02 |
| degorenko | clayton, well :) probably your right. But we can't just remove all parameters :( right now | 13:02 |
| iurygregory | degorenko, can you take a look https://review.openstack.org/#/c/247269/ about qpid | 13:02 |
| degorenko | iurygregory, oh, nice catch | 13:03 |
| iurygregory | you have done some patches about qpid, have you done to mistral? | 13:03 |
| degorenko | iurygregory, yep :) take a look on my comment | 13:04 |
| iurygregory | tks o/ | 13:04 |
| degorenko | thank you actually :D | 13:05 |
| *** davideagle_ has quit IRC | 13:06 | |
| *** samuelBartel has joined #puppet-openstack | 13:09 | |
| *** davideagle_ has joined #puppet-openstack | 13:09 | |
| *** tlbr has quit IRC | 13:09 | |
| clayton | I'd rather go ahead and land dan's patch, he has other things that depend on it | 13:11 |
| EmilienM | yeah and puppet-mistral is really experimental | 13:12 |
| EmilienM | he basically rewrote it, which is good | 13:13 |
| iurygregory | i can remove the -1 no problem ^^, we can fix in another patch | 13:14 |
| *** jayg|g0n3 is now known as jayg | 13:15 | |
| degorenko | EmilienM, why we need this? https://review.openstack.org/#/c/253728/ | 13:16 |
| EmilienM | degorenko: to have puppet jobs in http://status.openstack.org/openstack-health/#/ | 13:17 |
| degorenko | oh | 13:18 |
| degorenko | :) | 13:19 |
| EmilienM | this file is going to be read | 13:19 |
| *** davideagle_ has quit IRC | 13:21 | |
| *** davideagle_ has joined #puppet-openstack | 13:22 | |
| mkarpin | @EmilienM i answered on the comment https://review.openstack.org/#/c/237035/, can you please take a look when have time? | 13:23 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: remove POSIX users, groups and file modes https://review.openstack.org/195135 | 13:24 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 13:24 |
| *** dprince has joined #puppet-openstack | 13:30 | |
| beddari | clayton: wow thanks for that virutalenv work, that would actually solve my use case managing stuff from source (e.g skipping packages as I was talking about at the summit) | 13:31 |
| *** yottatsa has joined #puppet-openstack | 13:32 | |
| *** samuelBartel has quit IRC | 13:37 | |
| *** jpena|lunch is now known as jpena | 13:40 | |
| *** samuelBartel has joined #puppet-openstack | 13:42 | |
| *** tlbr has joined #puppet-openstack | 13:50 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-designate: Remove POSIX users, groups, and file modes. https://review.openstack.org/197336 | 13:50 |
| *** ducttape_ has joined #puppet-openstack | 13:55 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2 https://review.openstack.org/249711 | 13:58 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2 https://review.openstack.org/249711 | 13:59 |
| *** jerrygb has joined #puppet-openstack | 14:04 | |
| *** davideagle_ has quit IRC | 14:07 | |
| *** saneax is now known as saneax_AFK | 14:11 | |
| *** morazi has joined #puppet-openstack | 14:12 | |
| *** ducttape_ has quit IRC | 14:14 | |
| *** ducttape_ has joined #puppet-openstack | 14:16 | |
| *** kylo has quit IRC | 14:16 | |
| *** davideagle_ has joined #puppet-openstack | 14:17 | |
| *** hogepodge has quit IRC | 14:17 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Remove references to puppet-openstack from README https://review.openstack.org/248389 | 14:17 |
| openstackgerrit | Merged openstack/puppet-neutron: Remove references to puppet-openstack from README https://review.openstack.org/248384 | 14:17 |
| *** ducttape_ has quit IRC | 14:18 | |
| openstackgerrit | Merged openstack/puppet-neutron: Add parameters for Neutron QoS support https://review.openstack.org/252077 | 14:18 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2 https://review.openstack.org/253416 | 14:20 |
| openstackgerrit | Merged openstack/puppet-nova: Remove references to puppet-openstack from README https://review.openstack.org/248385 | 14:21 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: Configure disable_non_metric_meters for notification agent https://review.openstack.org/253451 | 14:24 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 14:30 |
| IvanBerezovskiy1 | EmilienM: hi. can you prompt me, who is familiar with designate? I've updated https://review.openstack.org/#/c/197336 and want to ask for review | 14:30 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-tempest: Allow to activate Murano Service Broker tests https://review.openstack.org/253019 | 14:35 |
| *** mdorman has joined #puppet-openstack | 14:39 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 14:40 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ceilometer: Deprecate qpid messaging parameters https://review.openstack.org/253043 | 14:46 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 14:51 |
| *** ducttape_ has joined #puppet-openstack | 14:57 | |
| *** dfisher has joined #puppet-openstack | 15:01 | |
| *** xarses has quit IRC | 15:02 | |
| *** davideagle_ has quit IRC | 15:02 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 15:02 |
| *** davideagle_ has joined #puppet-openstack | 15:09 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-designate: Remove POSIX users, groups, and file modes. https://review.openstack.org/197336 | 15:12 |
| *** vishpat has joined #puppet-openstack | 15:13 | |
| *** hogepodge has joined #puppet-openstack | 15:14 | |
| *** baker has joined #puppet-openstack | 15:15 | |
| *** kylo has joined #puppet-openstack | 15:15 | |
| *** imcsk8_ is now known as imcsk8 | 15:17 | |
| *** yottatsa has quit IRC | 15:22 | |
| *** delattec has joined #puppet-openstack | 15:23 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Change default admin_user_name and admin_tenant_name https://review.openstack.org/254250 | 15:26 |
| *** cdelatte has quit IRC | 15:26 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Change default admin_user and admin_tenant_name https://review.openstack.org/254250 | 15:27 |
| *** davideagle_ has quit IRC | 15:28 | |
| openstackgerrit | Vishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration https://review.openstack.org/247239 | 15:36 |
| *** xarses has joined #puppet-openstack | 15:37 | |
| *** vpati011_ has joined #puppet-openstack | 15:39 | |
| *** dhollinger has joined #puppet-openstack | 15:39 | |
| vpati011_ | Denis, Sergey have addressed your review comments for https://review.openstack.org/#/c/247239/ | 15:43 |
| *** yottatsa has joined #puppet-openstack | 15:44 | |
| degorenko | vpati011_, hey, what about Sergey's comment on 10 patch set? | 15:44 |
| *** ddmitriev has quit IRC | 15:54 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 15:54 |
| *** iurygregory has left #puppet-openstack | 15:54 | |
| *** iurygregory has joined #puppet-openstack | 15:54 | |
| vpati011_ | Denis, I have addressed that as well | 15:57 |
| vpati011_ | degorenko patch set 11 addresses Sergey's comment | 15:58 |
| degorenko | oh, yeah, i see now. Sorry. vpati011_ but will be better when you also have answered on comment in place :) | 15:59 |
| degorenko | vpati011_, btw what about debian os. Do we need re db-sync when we changed /etc/default/neutron-server:NEUTRON_PLUGIN_CONFIG? | 16:00 |
| *** vpati011_ has quit IRC | 16:05 | |
| *** vishpat has quit IRC | 16:05 | |
| *** vishpat has joined #puppet-openstack | 16:05 | |
| *** vpati011_ has joined #puppet-openstack | 16:05 | |
| *** paramite is now known as paramite|afk | 16:10 | |
| *** vishpat_ has joined #puppet-openstack | 16:13 | |
| *** cdelatte has joined #puppet-openstack | 16:14 | |
| *** delattec has quit IRC | 16:15 | |
| openstackgerrit | Merged openstack/puppet-ironic: Remove references to puppet-openstack from README https://review.openstack.org/248381 | 16:15 |
| *** vpati011_ has quit IRC | 16:16 | |
| *** vishpat has quit IRC | 16:17 | |
| *** vishpat_ is now known as vishpat | 16:17 | |
| mkarpin | guys need one more core review https://review.openstack.org/#/c/240946/, thanks! | 16:17 |
| mkarpin | clayton, mfisch, sbadia, spredzy : ^^ | 16:18 |
| mattymo | https://review.openstack.org/#/c/254099/ ^ same :) | 16:21 |
| mattymo | it's a backport | 16:21 |
| mfisch | done | 16:31 |
| mfisch | mkarpin: works better if you dont put commas in your URLs | 16:31 |
| mfisch | degorenko: I have a few simple ones, could you look? | 16:32 |
| mfisch | degorenko: https://review.openstack.org/253309 | 16:32 |
| degorenko | mfisch, sure :) | 16:32 |
| mfisch | degorenko: https://review.openstack.org/254019 | 16:32 |
| mfisch | https://review.openstack.org/254020 | 16:32 |
| mfisch | mattymo: done | 16:33 |
| mfisch | mkarpin: done | 16:33 |
| degorenko | mfisch, for this - i would prefer to switch to ensure_resource other modules. Btw, what are modules use package for openstackclient? o_O | 16:33 |
| degorenko | https://review.openstack.org/#/c/253309/1 | 16:33 |
| degorenko | sorry) | 16:33 |
| *** foxhunt72 is now known as foxhunt_away | 16:34 | |
| mfisch | glance is | 16:34 |
| mkarpin | mfisch:thanks! and will try not to put commas :) | 16:34 |
| mfisch | degorenko: I have this change up | 16:34 |
| mfisch | https://review.openstack.org/#/c/253311/ | 16:34 |
| mfisch | but was still testing it | 16:34 |
| mfisch | I'd like to do both | 16:34 |
| degorenko | mfisch, for this https://review.openstack.org/#/c/254020/ - i guess you can backport it to liberty and remove at the end of Mitaka cycle | 16:36 |
| mfisch | clayton suggested just removing it now | 16:37 |
| mfisch | near as I can tell it wasnt ever used | 16:37 |
| mfisch | I looked back to grizzly | 16:37 |
| degorenko | mfisch, i don t understand, what's wrong with glance? I see also ensure_resource in your patch | 16:38 |
| degorenko | my question was - which modules is using package resource for openstackclient? | 16:38 |
| degorenko | i'm really not sure | 16:38 |
| degorenko | that we need to use ensure_resource instead of package in openstacklib for openstackclient package | 16:38 |
| mfisch | tbh I can't remember | 16:40 |
| mfisch | I can look later today | 16:41 |
| degorenko | ok, i will wait for two those patches for now | 16:45 |
| mfisch | hmm | 16:45 |
| mfisch | actually now I'm not reproducing it | 16:45 |
| mfisch | let me use a clean setup | 16:45 |
| degorenko | mfisch, i will be available for next 2 hours :) | 16:46 |
| *** jtomasek has quit IRC | 16:46 | |
| *** paramite|afk is now known as paramite | 16:46 | |
| mfisch | spasiba | 16:47 |
| *** igajsin has joined #puppet-openstack | 16:48 | |
| degorenko | mfisch, spasibO ;) | 16:48 |
| mfisch | 15 years since I took Russian class ;) | 16:48 |
| mfisch | too much Spanish gets mixed in | 16:48 |
| degorenko | wow, really? | 16:48 |
| mfisch | yeah I took it when in college | 16:48 |
| degorenko | in America? | 16:48 |
| mfisch | yes | 16:49 |
| degorenko | wow :) | 16:49 |
| mfisch | I think it was more common in the 1980s and 1990s | 16:49 |
| mfisch | for military purposes probably | 16:49 |
| degorenko | yea, i think so | 16:49 |
| mfisch | now probably Chinese and Arabic done more | 16:49 |
| degorenko | :D | 16:50 |
| degorenko | arabic especially | 16:50 |
| mfisch | probabloy over there too the way things are going | 16:50 |
| *** yottatsa has quit IRC | 16:51 | |
| *** saneax_AFK is now known as saneax | 16:53 | |
| degorenko | mfisch, can you review a few small changes for me: https://review.openstack.org/253422 https://review.openstack.org/253528 | 16:54 |
| degorenko | and this one https://review.openstack.org/253099 | 16:54 |
| mfisch | sure | 16:54 |
| degorenko | thanks :) | 16:54 |
| *** yottatsa has joined #puppet-openstack | 16:54 | |
| degorenko | mfisch, and merge this one :) https://review.openstack.org/#/c/246988/ | 16:55 |
| mfisch | how long have they been deprecated? | 16:55 |
| mfisch | for 253099 | 16:55 |
| degorenko | all liberty cycle :) | 16:56 |
| degorenko | sec | 16:56 |
| degorenko | https://github.com/openstack/puppet-sahara/tree/stable/liberty/manifests/notify | 16:56 |
| degorenko | https://github.com/openstack/puppet-sahara/blob/stable/liberty/manifests/init.pp#L288-L297 | 16:56 |
| *** jpena has quit IRC | 16:57 | |
| mfisch | degorenko: you missed some in https://review.openstack.org/#/c/253528/1 | 16:58 |
| openstackgerrit | Merged openstack/puppet-cinder: Add keymgr/encryption_auth_url parameter to cinder API https://review.openstack.org/254099 | 16:58 |
| degorenko | mfisch, oh no | 16:58 |
| degorenko | let me explain | 16:58 |
| mfisch | is it b/c we're removing qpid support? | 16:59 |
| degorenko | i have this: https://review.openstack.org/#/q/topic:remove_qpid,n,z and https://review.openstack.org/#/q/topic:deprecate_qpid,n,z | 16:59 |
| degorenko | yep :) | 16:59 |
| mfisch | yep ok | 16:59 |
| degorenko | mfisch, thanks o/ | 17:00 |
| mfisch | pas ne probleme | 17:03 |
| *** nlahouti has joined #puppet-openstack | 17:03 | |
| *** nlahouti has quit IRC | 17:04 | |
| openstackgerrit | Merged openstack/puppet-glance: Remove unused cluster_id parameter https://review.openstack.org/254019 | 17:04 |
| *** yottatsa has quit IRC | 17:07 | |
| *** nlahouti has joined #puppet-openstack | 17:09 | |
| *** saneax is now known as saneax_AFK | 17:10 | |
| openstackgerrit | Merged openstack/puppet-horizon: Fix doc defaults for some parameters. https://review.openstack.org/253422 | 17:11 |
| *** jistr has quit IRC | 17:11 | |
| *** karimb has quit IRC | 17:12 | |
| *** baker has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/puppet-keystone: Add policy driver option for keystone https://review.openstack.org/240946 | 17:16 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-aodh: Remove qpid messaging support https://review.openstack.org/253042 | 17:20 |
| *** davideagle has joined #puppet-openstack | 17:21 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ceilometer: Remove qpid messaging support https://review.openstack.org/253043 | 17:26 |
| openstackgerrit | Merged openstack/puppet-glance: Remove references to puppet-openstack from README https://review.openstack.org/248380 | 17:26 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Remove qpid messaging support https://review.openstack.org/253052 | 17:29 |
| openstackgerrit | Merged openstack/puppet-swift: Update ceilometer proxy template https://review.openstack.org/246988 | 17:29 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Remove qpid messaging support https://review.openstack.org/253057 | 17:30 |
| *** samuelBartel has quit IRC | 17:31 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Remove qpid messaging support https://review.openstack.org/253062 | 17:31 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ironic: Remove qpid messaging support https://review.openstack.org/253066 | 17:35 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-manila: Remove qpid messaging support https://review.openstack.org/253071 | 17:36 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-mistral: Remove qpid messaging support https://review.openstack.org/253085 | 17:37 |
| *** cwolferh has joined #puppet-openstack | 17:37 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-neutron: Remove qpid messaging support https://review.openstack.org/253090 | 17:38 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Remove qpid messaging support https://review.openstack.org/253097 | 17:40 |
| *** saneax_AFK is now known as saneax | 17:41 | |
| *** davideagle has quit IRC | 17:41 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Remove qpid messaging support https://review.openstack.org/253105 | 17:41 |
| *** dgurtner has quit IRC | 17:44 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-trove: Remove qpid messaging support https://review.openstack.org/253116 | 17:44 |
| *** paramite is now known as paramite|afk | 17:52 | |
| dmsimard | bye qpid | 17:55 |
| *** jistr has joined #puppet-openstack | 17:59 | |
| *** karimb has joined #puppet-openstack | 17:59 | |
| *** paramite|afk is now known as paramite | 18:02 | |
| *** baker has joined #puppet-openstack | 18:02 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API https://review.openstack.org/254181 | 18:02 |
| *** aimon_ has quit IRC | 18:03 | |
| *** nlahouti has quit IRC | 18:03 | |
| degorenko | dmsimard, #byeqpid :D | 18:09 |
| *** nlahouti has joined #puppet-openstack | 18:13 | |
| degorenko | spredzy, hey, can you +A https://review.openstack.org/#/c/237970/ ? | 18:13 |
| degorenko | it is glance::db class | 18:14 |
| degorenko | sbadia, ^^^ | 18:16 |
| *** RichardRaseley has joined #puppet-openstack | 18:18 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Switch Keystone to $::os_service_default https://review.openstack.org/250945 | 18:18 |
| iurygregory | degorenko, can you take a look tomorrow ? ^ | 18:19 |
| vinsh_ | Hey, I'm trying to run the acceptance test locally on a centos vm.. hitting something I havn't yet.. and can't figure out | 18:19 |
| vinsh_ | Error: Could not autoload puppet/provider/package/pkgdmg: cannot load such file -- plist | 18:19 |
| *** vinsh_ is now known as Vinsh | 18:19 | |
| Vinsh | This is when I first run puppet apply | 18:19 |
| Vinsh | Not sure what its missing | 18:20 |
| degorenko | iurygregory, will look :) | 18:20 |
| iurygregory | tks man o/ | 18:20 |
| degorenko | Vinsh, hey | 18:20 |
| degorenko | gem install plist | 18:20 |
| Vinsh | ah-hah.. ok cool. I must be missing basic deps for the tests. that fixed one problem now on to another.. but atleast I know the route to go. thanks! | 18:21 |
| degorenko | :) | 18:22 |
| iurygregory | bye people o/ | 18:24 |
| degorenko | \o | 18:24 |
| *** aimon has joined #puppet-openstack | 18:27 | |
| *** dhollinger has quit IRC | 18:28 | |
| *** degorenko is now known as _degorenko|afk | 18:30 | |
| openstackgerrit | Merged openstack/puppet-aodh: Support MongoDB as backend db https://review.openstack.org/250472 | 18:31 |
| *** trown|flying is now known as trown|BOS | 18:32 | |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Fix keystone configuration. https://review.openstack.org/247274 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::engine class https://review.openstack.org/248412 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::executor class https://review.openstack.org/248413 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::api class https://review.openstack.org/248410 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Allow distro packages to install dependencies, etc https://review.openstack.org/247277 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config. https://review.openstack.org/248406 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head' https://review.openstack.org/248404 | 18:34 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Default Rabbit port to 5672 https://review.openstack.org/247269 | 18:42 |
| *** karimb has quit IRC | 18:58 | |
| *** dhollinger has joined #puppet-openstack | 19:00 | |
| *** gfidente has quit IRC | 19:05 | |
| *** paramite has quit IRC | 19:15 | |
| *** saneax is now known as saneax_AFK | 19:16 | |
| *** IvanBerezovskiy1 has left #puppet-openstack | 19:21 | |
| *** saneax_AFK is now known as saneax | 19:23 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: tempest run: generate testrepository.subunit https://review.openstack.org/253728 | 19:25 |
| *** jistr has quit IRC | 19:26 | |
| *** jtomasek has joined #puppet-openstack | 19:26 | |
| *** paramite has joined #puppet-openstack | 19:27 | |
| *** yottatsa has joined #puppet-openstack | 19:30 | |
| *** dprince has quit IRC | 19:32 | |
| mfisch | EmilienM: thoughts on this? https://bugs.launchpad.net/puppet-keystone/+bug/1523643 | 19:33 |
| openstack | Launchpad bug 1523643 in puppet-keystone "glance & keystone fighting over openstack client" [Undecided,New] | 19:33 |
| EmilienM | mfisch: I never saw this | 19:34 |
| *** paramite has quit IRC | 19:34 | |
| mfisch | simple to repro, on liberty branch | 19:34 |
| mfisch | this fixes it but there was some discussion as to whether its right | 19:35 |
| mfisch | https://review.openstack.org/#/c/253309/ | 19:35 |
| *** yottatsa has quit IRC | 19:38 | |
| mfisch | I'll be afk for a bit will look to see if this gets discussed more | 19:39 |
| mfisch | clayton: You might also be interested since we're switching to L | 19:39 |
| *** bklei has joined #puppet-openstack | 19:52 | |
| *** jtomasek has quit IRC | 19:53 | |
| *** dprince has joined #puppet-openstack | 19:59 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: fwaas config needs to notify l3 agent service https://review.openstack.org/252562 | 19:59 |
| EmilienM | mwhahaha, _degorenko|afk ^ | 19:59 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 20:08 |
| *** vpati011_ has joined #puppet-openstack | 20:09 | |
| beagles | EmilienM, how does the process of incorporating the component specific puppet modules into the openstack-puppet-modules work? I'm wondering when the neutron QoS changes would make it into there | 20:09 |
| EmilienM | beagles: that kind of discussions go on #rdo-puppet (relating to OPM) | 20:10 |
| beagles | EmilienM, right thanks | 20:11 |
| EmilienM | beagles: your patch has been merged upstream, so you need to poke someone from rdo to update opm | 20:11 |
| beagles | EmilienM, cool thanks! | 20:12 |
| EmilienM | beagles: I backported it to stable/liberty and it's merged too | 20:12 |
| beagles | EmilienM, I noticed! Thanks +100 | 20:13 |
| *** davideagle has joined #puppet-openstack | 20:14 | |
| *** ntpttr has joined #puppet-openstack | 20:15 | |
| *** davideagle has quit IRC | 20:19 | |
| *** ianw has joined #puppet-openstack | 20:20 | |
| *** migi is now known as migi_afk | 20:25 | |
| *** vpati011_ has quit IRC | 20:35 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Avoid duplicate client resource https://review.openstack.org/253309 | 20:36 |
| *** nlahouti has quit IRC | 20:40 | |
| EmilienM | mfisch: your bug is weird | 20:50 |
| EmilienM | mfisch: we don't have it in our CI | 20:50 |
| EmilienM | and we run single node too | 20:50 |
| EmilienM | but +2 your patch, lgtm | 20:51 |
| openstackgerrit | Merged openstack/puppet-mistral: Fix keystone configuration. https://review.openstack.org/247274 | 20:51 |
| openstackgerrit | Merged openstack/puppet-mistral: Allow distro packages to install dependencies, etc https://review.openstack.org/247277 | 20:52 |
| mfisch | EmilienM: sent you an email on it too, I'd like to cleanup glance & keystone | 20:52 |
| *** davideagle has joined #puppet-openstack | 20:54 | |
| *** davideagle has quit IRC | 20:55 | |
| EmilienM | mfisch: an email? where | 20:55 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 21:00 |
| mfisch | EmilienM: to the ml | 21:02 |
| EmilienM | mfisch: just got it | 21:03 |
| EmilienM | mfisch: how do you explain our CI did not catch it? | 21:03 |
| EmilienM | we have functional jobs that install all-in-one | 21:03 |
| mfisch | EmilienM: I have no way to explain | 21:04 |
| *** throwawaymonday has quit IRC | 21:10 | |
| *** ducttape_ has quit IRC | 21:10 | |
| *** migi_afk is now known as migi | 21:11 | |
| *** jayg is now known as jayg|g0n3 | 21:23 | |
| EmilienM | mfisch: can you look https://review.openstack.org/#/c/252026/ ? | 21:24 |
| mfisch | oui | 21:25 |
| mfisch | mmmm gnocchi | 21:25 |
| EmilienM | mfisch: https://review.openstack.org/253620 and https://review.openstack.org/253626 too if you don't mind | 21:26 |
| openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Initial commit of api.pp https://review.openstack.org/254392 | 21:32 |
| mfisch | EmilienM: done | 21:36 |
| EmilienM | mfisch: thx man | 21:36 |
| *** nlahouti has joined #puppet-openstack | 21:38 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: all-in-one: export SCENARIO variable https://review.openstack.org/253620 | 21:39 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: scenario003: fix tempest path https://review.openstack.org/253626 | 21:39 |
| openstackgerrit | Merged openstack/puppet-sahara: Remove deprecated parameters since Liberty release https://review.openstack.org/253099 | 21:39 |
| *** saneax is now known as saneax_AFK | 21:50 | |
| *** vpati011_ has joined #puppet-openstack | 21:56 | |
| *** aimon has quit IRC | 21:58 | |
| xarses | dmsimard: I sent https://dmsimard.com/2015/12/07/thinking-outside-the-box-and-outside-the-gate-to-improve-openstack-and-rdo/ around internally, lets see if it peaks anyone's interest | 22:00 |
| xarses | nice write up | 22:00 |
| *** vpati011_ has quit IRC | 22:01 | |
| dmsimard | xarses: cool, thanks :) | 22:01 |
| *** gildub has joined #puppet-openstack | 22:02 | |
| *** karimb has joined #puppet-openstack | 22:03 | |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Remove qpid messaging support https://review.openstack.org/253085 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Default Rabbit port to 5672 https://review.openstack.org/247269 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::engine class https://review.openstack.org/248412 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::executor class https://review.openstack.org/248413 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Add mistral::api class https://review.openstack.org/248410 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config. https://review.openstack.org/248406 | 22:14 |
| openstackgerrit | Dan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head' https://review.openstack.org/248404 | 22:14 |
| *** ntpttr has quit IRC | 22:15 | |
| dmsimard | dprince is bringing puppet-mistral in 2015 | 22:17 |
| *** aimon has joined #puppet-openstack | 22:21 | |
| *** dprince has quit IRC | 22:27 | |
| *** morazi has quit IRC | 22:29 | |
| EmilienM | dmsimard: lol | 22:32 |
| EmilienM | dmsimard: mistral is really fun | 22:32 |
| dmsimard | not really familiar with it | 22:32 |
| *** gildub has quit IRC | 22:58 | |
| *** ducttape_ has joined #puppet-openstack | 22:59 | |
| *** dhollinger has quit IRC | 23:01 | |
| *** nlahouti has quit IRC | 23:05 | |
| *** davideagle has joined #puppet-openstack | 23:05 | |
| *** nlahouti has joined #puppet-openstack | 23:07 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: fwaas config needs to notify l3 agent service https://review.openstack.org/252562 | 23:08 |
| *** baker has quit IRC | 23:14 | |
| *** baker has joined #puppet-openstack | 23:14 | |
| *** davideagle has quit IRC | 23:19 | |
| *** baker has quit IRC | 23:19 | |
| *** dfisher has quit IRC | 23:28 | |
| *** bklei has quit IRC | 23:32 | |
| openstackgerrit | Merged openstack/puppet-glance: Introduce glance::db class https://review.openstack.org/237970 | 23:45 |
| openstackgerrit | Merged openstack/puppet-glance: Support of PyMySQL driver for MySQL backend https://review.openstack.org/246926 | 23:46 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-openstacklib: Avoid duplicate client resource https://review.openstack.org/253309 | 23:49 |
| *** mdorman has quit IRC | 23:51 | |
| openstackgerrit | Merged openstack/puppet-aodh: Remove qpid messaging support https://review.openstack.org/253042 | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!