| *** johnmilton has quit IRC | 00:03 | |
| *** xarses has joined #puppet-openstack | 00:05 | |
| *** xarses_ has joined #puppet-openstack | 00:08 | |
| *** xarses has quit IRC | 00:10 | |
| *** xarses_ has quit IRC | 00:23 | |
| *** cdelatte has joined #puppet-openstack | 00:29 | |
| *** ducttape_ has joined #puppet-openstack | 00:34 | |
| *** rchurch_ has joined #puppet-openstack | 00:36 | |
| *** ntpttr_ has joined #puppet-openstack | 00:37 | |
| *** ntpttr has quit IRC | 00:39 | |
| *** jesusaur has quit IRC | 00:39 | |
| *** rchurch has quit IRC | 00:39 | |
| *** kylo has quit IRC | 00:39 | |
| *** iurygregory has quit IRC | 00:39 | |
| *** darkxploit has quit IRC | 00:39 | |
| *** ntpttr has joined #puppet-openstack | 00:44 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-trove: Remove the extra space in the conductor.pp https://review.openstack.org/308096 | 00:46 |
|---|---|---|
| *** darkxploit has joined #puppet-openstack | 00:53 | |
| *** jesusaur has joined #puppet-openstack | 00:53 | |
| *** iurygregory has joined #puppet-openstack | 00:55 | |
| *** johnmilton has joined #puppet-openstack | 01:20 | |
| *** ducttape_ has quit IRC | 01:23 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments https://review.openstack.org/308530 | 01:30 |
| EmilienM | mwhahaha: never give up https://review.openstack.org/#/c/306677/ | 01:31 |
| *** ntpttr- has joined #puppet-openstack | 01:32 | |
| *** kylo has joined #puppet-openstack | 01:36 | |
| *** johnmilton has quit IRC | 01:43 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton https://review.openstack.org/308573 | 01:46 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken https://review.openstack.org/266723 | 02:05 |
| colonwq | EmilienM, I ran a recheck after 308580 merged to fix the puppet issue. I still got the same error. Does it take a while for the fix to propagate after it is merged? | 02:16 |
| EmilienM | colonwq: which patch? | 02:16 |
| colonwq | EmilienM, https://review.openstack.org/#/c/299615/5 | 02:17 |
| colonwq | here is the error: http://logs.openstack.org/15/299615/5/check/gate-puppet-ceph-puppet-beaker-rspec-centos-7/9a85da2/console.html#_2016-04-20_21_46_10_413 | 02:17 |
| EmilienM | https://github.com/openstack/puppet-ceph/blob/master/Gemfile | 02:17 |
| EmilienM | puppet-ceph does not use our spec_helper | 02:18 |
| EmilienM | so beaker jbos are still broken on puppet-ceph | 02:18 |
| EmilienM | we should change puppet-ceph to use common gemspecs | 02:18 |
| colonwq | ah.... I seem to live in the odd of odd areas | 02:18 |
| mwhahaha | Stupid patch | 02:19 |
| mwhahaha | Some day it might land | 02:20 |
| *** ducttape_ has joined #puppet-openstack | 02:20 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again https://review.openstack.org/308454 | 02:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: install ipmitool on ubuntu if driver enabled https://review.openstack.org/308717 | 02:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again https://review.openstack.org/307977 | 02:29 |
| colonwq | EmilienM, I'm testing the same fix in puppet-ceph. | 02:32 |
| *** ducttape_ has quit IRC | 02:43 | |
| openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 02:44 |
| *** richm has quit IRC | 02:46 | |
| *** v1k0d3n has joined #puppet-openstack | 02:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 02:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Added params for debian packages https://review.openstack.org/307943 | 02:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 02:55 |
| EmilienM | mjblack: I inversed the order ^ | 02:55 |
| mjblack | EmilienM: ok | 02:55 |
| mjblack | btw | 02:55 |
| mjblack | I tested out that RUNLEVEL thing...it failed | 02:56 |
| EmilienM | I had to do it because the beaker tests are voting | 02:56 |
| mjblack | its not preventing keystone from starting | 02:56 |
| EmilienM | mjblack: imho, the keystone thing should be fixed with a sleep. | 02:56 |
| mjblack | I'm fine with that, but with regards to keystone we are seriously going to need to look at the update-rc.d | 02:56 |
| EmilienM | I don't think we should touch update-rc.d | 02:56 |
| mjblack | sorry policy-rc.d | 02:56 |
| EmilienM | it's too much opinionated, imho | 02:57 |
| mjblack | I know but the sleep wasnt working | 02:57 |
| mjblack | I can make it do it for a 5 minute max sleep | 02:57 |
| EmilienM | why wasn't it working? | 02:57 |
| mjblack | we tried 60 seconds, still ran into the issue | 02:57 |
| EmilienM | mjblack: you ok with the unit tests issues on puppert-ovn? | 02:58 |
| EmilienM | puppet* | 02:58 |
| mjblack | EmilienM: what if we made the policy-rc.d a flag and for beaker/scenario we have the flag turned on? | 02:58 |
| mjblack | I'm fine with it, I'll come back and fix my review with ovn if there are issues | 02:59 |
| EmilienM | mjblack: maybe you can steal the code from Fuel, add it in extras, and use it in puppet-keystone optionally? I don't know... it really sucks | 02:59 |
| EmilienM | why the sleep does not work?? | 03:00 |
| mjblack | thats what I'm suggesting | 03:00 |
| mjblack | have it in extras and then we toggle it on for CI | 03:00 |
| EmilienM | mwhahaha: fuel failure on https://review.openstack.org/#/c/308573/ | 03:00 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-oslo: Add oslo.middleware defines https://review.openstack.org/274422 | 03:00 |
| EmilienM | mwhahaha: I can -1 it if you want | 03:00 |
| mwhahaha | let me check it | 03:00 |
| EmilienM | mjblack: the problem is the problem is not only in our CI | 03:00 |
| EmilienM | it's everywhere on ubuntu | 03:01 |
| mjblack | sure | 03:01 |
| EmilienM | look at fuel, they have a workaround for it, I don't like it | 03:01 |
| mjblack | but so long as we provide an option, then that should be fine | 03:01 |
| mjblack | but we cant force the option for everyone | 03:01 |
| mjblack | so it has to be optional | 03:01 |
| mwhahaha | that fuel failure seems to be a server error, i'll recheck it | 03:01 |
| mjblack | we add something to README.md to point them to the option in extras | 03:01 |
| EmilienM | mwhahaha: ack | 03:01 |
| EmilienM | omg I tested our scenario against Xenial, so many failures | 03:02 |
| EmilienM | I'll need some help here | 03:02 |
| EmilienM | https://review.openstack.org/#/c/308530/ | 03:02 |
| EmilienM | mwhahaha: same failure on https://review.openstack.org/#/c/308452/ for fuel | 03:02 |
| mwhahaha | k retriggered it | 03:03 |
| mjblack | EmilienM: so are you ok with me going the route of putting policy-rc.d in extras as an option? | 03:03 |
| EmilienM | mhh | 03:03 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Relove puppetlabs-relase version conflict https://review.openstack.org/308722 | 03:03 |
| EmilienM | it's weird, we never used extras repo in our modules | 03:03 |
| EmilienM | it's supposed to be helpers | 03:03 |
| mjblack | well thats what it would be doing here :) | 03:03 |
| mjblack | helping us deal with ubuntu packaging :D | 03:04 |
| EmilienM | maybe in openstacklib then | 03:04 |
| EmilienM | which is a module we include in our other modules already | 03:04 |
| mwhahaha | we could reduce what we do in fuel to only in puppet-keystone and only in debian | 03:04 |
| mjblack | I can put it there | 03:04 |
| mwhahaha | 1) debian fact on if the package is installed | 03:04 |
| mwhahaha | 2) wrap install with don't start service workaround (thing from fuel) | 03:05 |
| EmilienM | yes ^ | 03:05 |
| mwhahaha | we use a puppet function which won't work for puppet-keystone | 03:05 |
| mjblack | mwhahaha: got a link handy where that is? | 03:05 |
| EmilienM | I go to bed, see you tomorrow guys | 03:05 |
| mjblack | good night | 03:06 |
| mwhahaha | later | 03:06 |
| mwhahaha | https://github.com/openstack/fuel-library/blob/master/deployment/puppet/tweaks/manifests/ubuntu_service_override.pp | 03:06 |
| mwhahaha | so is_pkg_installed would need to be converted to a fact only for debian | 03:06 |
| mwhahaha | and only for keystone | 03:06 |
| mjblack | mwhahaha: thats exactly what I'm suggesting :P | 03:06 |
| mwhahaha | so like if $::keystone_installed | 03:06 |
| mjblack | policy-rc.d | 03:06 |
| mwhahaha | the problem is that you'll run into a possible conflicts with fuel | 03:07 |
| mwhahaha | so you'll have to do your policy-rc.d carefully | 03:07 |
| mjblack | there is a package out there that helps manage that policy-rc.d | 03:07 |
| mwhahaha | i guess the ensure resource should work but the only danger would be for someone who already does that but with a different value or something | 03:07 |
| mjblack | so its not just a script that exits but you can define policies | 03:07 |
| mwhahaha | which one? | 03:08 |
| mjblack | http://manpages.ubuntu.com/manpages/wily/man8/zg-policy-rc.d.8.html | 03:08 |
| mwhahaha | that's be harder to convince to be added to puppet-keystone | 03:10 |
| mjblack | not suggesting to add it to puppet-keystone | 03:10 |
| mjblack | would be openstacklib or openstack_extras | 03:10 |
| mjblack | we could in theory use it to make ubuntu behave more like centos | 03:11 |
| mjblack | mwhahaha: so basically what it would do on install is behave normally but we can add exceptions where services arent started upon install | 03:12 |
| mwhahaha | i guess if we want to just do it for our unit tests | 03:12 |
| mwhahaha | and not actually integrate it into the module | 03:13 |
| mjblack | yeah it would have to be optional | 03:13 |
| mwhahaha | which i guess might be ideal | 03:13 |
| mwhahaha | and just kinda document the issue | 03:13 |
| mjblack | I was thinking of just adding an item to README.md in puppet-keystone pointing to it | 03:13 |
| mwhahaha | yup | 03:13 |
| *** colonwq is now known as colonwq_afk | 03:13 | |
| mwhahaha | or just do the hack in the beaker test :D | 03:14 |
| mjblack | honestly, I rather do it as an option for ubuntu | 03:15 |
| *** xarses_ has joined #puppet-openstack | 03:27 | |
| *** andreww has joined #puppet-openstack | 03:36 | |
| *** xarses_ has quit IRC | 03:39 | |
| openstackgerrit | Merged openstack/puppet-neutron: Don't declare the service when service_enable is false https://review.openstack.org/307879 | 03:49 |
| openstackgerrit | Merged openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb https://review.openstack.org/307465 | 03:50 |
| openstackgerrit | Merged openstack/puppet-barbican: Totally drop Qpid support https://review.openstack.org/307110 | 04:06 |
| *** bapalm has quit IRC | 04:10 | |
| openstackgerrit | Merged openstack/puppet-nova: Add other virt services https://review.openstack.org/306677 | 04:10 |
| *** bapalm has joined #puppet-openstack | 04:12 | |
| *** delattec has joined #puppet-openstack | 04:12 | |
| *** cdelatte has quit IRC | 04:15 | |
| *** fedexo has joined #puppet-openstack | 04:20 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Totally drop Qpid support https://review.openstack.org/307093 | 04:27 |
| *** saneax_AFK is now known as saneax | 04:34 | |
| *** ducttape_ has joined #puppet-openstack | 04:43 | |
| *** ducttape_ has quit IRC | 04:48 | |
| *** rcernin has quit IRC | 04:48 | |
| *** chandankumar has joined #puppet-openstack | 05:05 | |
| *** jaosorior has joined #puppet-openstack | 05:07 | |
| *** ayoung has quit IRC | 05:10 | |
| *** rcernin has joined #puppet-openstack | 05:17 | |
| *** ianw has quit IRC | 05:31 | |
| *** zerda has joined #puppet-openstack | 05:49 | |
| *** jmikolajczak has joined #puppet-openstack | 06:27 | |
| *** angdraug has joined #puppet-openstack | 06:32 | |
| *** aarefiev has quit IRC | 06:35 | |
| *** yolanda has quit IRC | 06:37 | |
| *** jtomasek has quit IRC | 06:38 | |
| *** angdraug has quit IRC | 06:39 | |
| openstackgerrit | Jerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents https://review.openstack.org/301106 | 06:45 |
| *** yolanda has joined #puppet-openstack | 06:47 | |
| *** leanderthal|afk is now known as leanderthal | 07:00 | |
| *** aarefiev has joined #puppet-openstack | 07:00 | |
| *** karimb has joined #puppet-openstack | 07:02 | |
| *** amoralej|off is now known as amoralej | 07:02 | |
| *** tesseract has joined #puppet-openstack | 07:03 | |
| *** tesseract is now known as Guest14509 | 07:04 | |
| *** paramite has joined #puppet-openstack | 07:11 | |
| *** dgurtner has joined #puppet-openstack | 07:15 | |
| *** fedexo has quit IRC | 07:18 | |
| *** permalac has quit IRC | 07:19 | |
| *** hjensas has quit IRC | 07:20 | |
| *** hjensas has joined #puppet-openstack | 07:21 | |
| *** shardy has joined #puppet-openstack | 07:32 | |
| *** leanderthal is now known as leanderthal|afk | 07:36 | |
| *** foxhunt_away is now known as foxhunt72 | 07:40 | |
| *** jpena|off is now known as jpena | 07:44 | |
| *** darkxploit has quit IRC | 07:51 | |
| *** hogepodge has quit IRC | 07:52 | |
| *** hogepodge has joined #puppet-openstack | 07:54 | |
| *** darkxploit has joined #puppet-openstack | 08:04 | |
| *** jistr has joined #puppet-openstack | 08:13 | |
| *** jaosorior has quit IRC | 08:22 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 08:23 |
| *** markvoelker has quit IRC | 08:27 | |
| *** jaosorior has joined #puppet-openstack | 08:30 | |
| *** jtomasek has joined #puppet-openstack | 08:31 | |
| *** derekh has joined #puppet-openstack | 08:34 | |
| *** dtantsur|afk is now known as dtantsur | 08:36 | |
| *** mgould|afk is now known as mgould | 08:37 | |
| *** ducttape_ has joined #puppet-openstack | 08:46 | |
| *** ducttape_ has quit IRC | 08:51 | |
| *** higgins has quit IRC | 08:58 | |
| *** imcsk8 is now known as imcsk8|zZz | 08:59 | |
| *** gfidente has joined #puppet-openstack | 09:01 | |
| *** higgins has joined #puppet-openstack | 09:05 | |
| *** karimb has quit IRC | 09:23 | |
| openstackgerrit | Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp https://review.openstack.org/308863 | 09:26 |
| *** markvoelker has joined #puppet-openstack | 09:27 | |
| *** markvoelker has quit IRC | 09:32 | |
| openstackgerrit | Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp https://review.openstack.org/308863 | 09:46 |
| *** _degorenko|afk is now known as degorenko | 09:49 | |
| *** johnmilton has joined #puppet-openstack | 09:53 | |
| *** johnmilton has quit IRC | 09:58 | |
| openstackgerrit | Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp https://review.openstack.org/308863 | 10:00 |
| *** Guest14509 has quit IRC | 10:22 | |
| *** Guest14509 has joined #puppet-openstack | 10:24 | |
| -openstackstatus- NOTICE: OVH servers are down, we are working to solve it. This will cause that jobs queue is processed slowly, please have patience. | 10:38 | |
| *** dgurtner has quit IRC | 10:47 | |
| *** dgurtner has joined #puppet-openstack | 10:47 | |
| *** ducttape_ has joined #puppet-openstack | 10:47 | |
| *** ducttape_ has quit IRC | 10:52 | |
| *** jaosorior has quit IRC | 10:52 | |
| *** jaosorior has joined #puppet-openstack | 10:53 | |
| *** imcsk8|z1z has joined #puppet-openstack | 10:55 | |
| *** imcsk8|zZz has quit IRC | 10:56 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Relove puppetlabs-relase version conflict https://review.openstack.org/308722 | 11:25 |
| *** colonwq_afk is now known as colonwq | 11:25 | |
| *** johnmilton has joined #puppet-openstack | 11:26 | |
| *** zerda has quit IRC | 11:31 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Change source for puppet-corosync https://review.openstack.org/308424 | 11:34 |
| *** paramite is now known as paramite|afk | 11:36 | |
| *** jpena is now known as jpena|lunch | 11:36 | |
| *** darkxploit has quit IRC | 11:36 | |
| *** paramite|afk is now known as paramite | 11:40 | |
| EmilienM | o/ | 11:44 |
| chem | yo | 11:44 |
| colonwq | EmilienM, what does OVH stand for? | 11:46 |
| EmilienM | colonwq: it's a public cloud, where OpenStack Infra is consummer | 11:47 |
| dmsimard | colonwq: https://www.ovh.com/us/ | 11:47 |
| dmsimard | They're a pretty big shop, probably >200k servers now ? Last I heard was 120k and that was a good while ago. | 11:48 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 11:48 |
| *** darkxploit has joined #puppet-openstack | 11:49 | |
| colonwq | EmilienM, dmsimard ok. thanks. Hopefully they are not in one of the parts of texas under water. | 11:49 |
| dmsimard | I think the capacity they provide to -infra is out of Montreal (Beauharnois) but I could be mistaken | 11:52 |
| *** amoralej is now known as amoralej|lunch | 11:53 | |
| dmsimard | Ah, so out of Canada and France. They don't have water problems :p | 11:54 |
| *** morazi has quit IRC | 11:55 | |
| colonwq | that is good. It explains the French which shows up sometimes. | 11:57 |
| *** ansmith has joined #puppet-openstack | 11:57 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 12:00 |
| *** dtantsur is now known as dtantsur|brb | 12:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 12:05 |
| *** saneax is now known as saneax_AFK | 12:09 | |
| *** ducttape_ has joined #puppet-openstack | 12:10 | |
| EmilienM | _ody: good news. Periodic jobs that test puppet4 are now working | 12:12 |
| EmilienM | http://logs.openstack.org/periodic/periodic-puppet-openstack-integration-4-scenario001-tempest-centos-7/2a1d3ea/console.html#_2016-04-21_06_40_48_596 | 12:12 |
| EmilienM | 0 failure | 12:13 |
| mattymo | EmilienM, others, have you ever thought about puppetizing nova service-delete $deleted_host_service_id? | 12:15 |
| EmilienM | what is the use case? | 12:15 |
| mattymo | It looks like this command exists in openstackclient, so no reason why we can't reuse our generic openstacklib provider | 12:15 |
| mattymo | nova service-list shows nodes that were deleted/powered off as down | 12:15 |
| mattymo | it will make health check tools think there's a problem | 12:15 |
| mattymo | without an automated way to clean it up, it raises concerns | 12:16 |
| EmilienM | what is the use case of having a provider in manifests? When would you use it? | 12:16 |
| mattymo | well, I'm going to add an array of deleted_node fqdns to my hiera config, and if there are any entries there, I should purge nova/cinder/neutron agent service entries related to those hosts | 12:16 |
| mattymo | the operator case is I have 3 controller nodes, 1 failed hardcore, I decommission it, and deploy a 4th one | 12:17 |
| *** markvoelker has joined #puppet-openstack | 12:17 | |
| EmilienM | sure, you can push this code | 12:17 |
| mattymo | I want to clean up that unused host and have all my status as green | 12:17 |
| mattymo | thanks for permission :) | 12:17 |
| EmilienM | heh | 12:17 |
| mattymo | EmilienM, just checking if there's anything already out there related to this task. It looks like nobody cares so much yet | 12:18 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Revert "Remove puppetlabs-release before installing puppet" https://review.openstack.org/308949 | 12:22 |
| EmilienM | mattymo: nope | 12:23 |
| EmilienM | degorenko, skolekonov: about neutron_auth_type | 12:30 |
| EmilienM | would it be possible to use $::os_service_default ? | 12:30 |
| EmilienM | in my puppet-nova patch | 12:30 |
| EmilienM | or do we have to set a value? | 12:30 |
| EmilienM | nevermind, i'll deprecate neutron_auth_plugin to avoid confusions | 12:32 |
| degorenko | EmilienM, we have some lags in internal network, sorry for delay | 12:33 |
| EmilienM | no worries | 12:33 |
| *** dtantsur|brb is now known as dtantsur | 12:33 | |
| EmilienM | I have some lag on morning, time I wake up | 12:33 |
| degorenko | :) | 12:34 |
| degorenko | EmilienM, i think you can set neutron_auth_type to same value as now neutron_auth_plugin | 12:34 |
| EmilienM | no, I'll deprecate it, so undef by default | 12:35 |
| EmilienM | so it's not confusing | 12:35 |
| degorenko | no, i'm set to current default | 12:35 |
| EmilienM | ok | 12:35 |
| degorenko | so it is v3password now | 12:35 |
| *** social has joined #puppet-openstack | 12:36 | |
| degorenko | and set a new neutron_auth_type to v3password | 12:36 |
| degorenko | probably :) | 12:36 |
| degorenko | may be skolekonov have any ideas? :D | 12:36 |
| *** ducttape_ has quit IRC | 12:38 | |
| EmilienM | I'm pushing a patch, you'll review | 12:39 |
| degorenko | sure, sir | 12:39 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration https://review.openstack.org/307907 | 12:42 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration https://review.openstack.org/308207 | 12:43 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton https://review.openstack.org/308573 | 12:43 |
| openstackgerrit | Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp https://review.openstack.org/308863 | 12:46 |
| openstackgerrit | Merged openstack/puppet-mistral: Totally drop Qpid support https://review.openstack.org/307111 | 12:47 |
| -openstackstatus- NOTICE: OVH provider is enabled again, please wait for the job queue to be processed | 12:49 | |
| *** degorenko is now known as _degorenko|afk | 12:53 | |
| openstackgerrit | Merged openstack/puppet-monasca: Add ability to include libvirt scale group metadata https://review.openstack.org/308589 | 12:56 |
| *** richm has joined #puppet-openstack | 12:56 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 12:58 |
| mjblack | EmilienM: ^ works really well at not letting keystone eventlet to start :D | 12:58 |
| *** chandankumar has quit IRC | 12:58 | |
| *** jpena|lunch is now known as jpena | 12:59 | |
| EmilienM | mjblack: excellent, I'll look after my breakfast :P | 13:01 |
| EmilienM | mjblack: where can I see an example of usage in puppet-keystone? | 13:01 |
| mjblack | EmilienM: cool but remember its not on by default, we'll have to include the class in the CI for it to take effect | 13:01 |
| *** morazi has joined #puppet-openstack | 13:01 | |
| EmilienM | we don't want to fix CI only | 13:01 |
| mjblack | I'm not suggesting we include it in puppet-keystone | 13:01 |
| EmilienM | we want to fix puppet-keystone | 13:01 |
| mjblack | we want to leave it up to the user to include it | 13:02 |
| EmilienM | yes, so add a boolean to disable it if needed | 13:02 |
| EmilienM | but enable it by default | 13:02 |
| EmilienM | deployments on ubuntu randomly fail, that's very critical | 13:02 |
| mjblack | I dont think thats a good idea | 13:02 |
| EmilienM | well. I'm afk 15 min | 13:02 |
| mjblack | EmilienM: when you get back I'll give you the reason why I dont want it turned on by default | 13:02 |
| *** chandankumar has joined #puppet-openstack | 13:08 | |
| mjblack | EmilienM: well I'll just sum it up, if we have it on by default it'll affect every package install on the system, preventing new package installs from starting services | 13:12 |
| EmilienM | ok back | 13:12 |
| mjblack | I dont think we want to make that decision for the user | 13:12 |
| EmilienM | croissant was good | 13:12 |
| EmilienM | why not doing that only for keystone? | 13:13 |
| mjblack | I guess I could see about making it only work for keystone | 13:15 |
| *** EmilienM sets mode: +o _ody | 13:15 | |
| *** EmilienM sets mode: +o clayton | 13:15 | |
| *** amoralej|lunch is now known as amoralej | 13:17 | |
| *** karimb has joined #puppet-openstack | 13:21 | |
| *** chandankumar has quit IRC | 13:22 | |
| mjblack | EmilienM: you ok with an inline template to create the shell script? | 13:24 |
| EmilienM | mjblack: I'm not the only reviewer here | 13:25 |
| EmilienM | we are a group, and the group will review your code | 13:25 |
| mjblack | I'm just thinking of making it a param class where we can list each service we dont want to auto start on install | 13:25 |
| *** trown|outtypewww is now known as trown | 13:29 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: add more debug when Puppet fails https://review.openstack.org/308991 | 13:53 |
| *** ducttape_ has joined #puppet-openstack | 13:54 | |
| *** ducttape_ has quit IRC | 13:59 | |
| *** _degorenko|afk is now known as degorenko | 14:06 | |
| * mjblack grumbles | 14:08 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 14:09 |
| mjblack | degorenko,EmilienM: can you look at ^? | 14:09 |
| mjblack | I'm getting an error with spec that shouldnt be happening | 14:09 |
| mjblack | this is the error http://paste.openstack.org/show/494990/ | 14:10 |
| colonwq | EmilienM, I am checking on the unit test answer from your review. | 14:11 |
| degorenko | mjblack, do you really think that this is good idea: https://review.openstack.org/#/c/308012/13/manifests/init.pp ? :) i'm not sure | 14:14 |
| EmilienM | degorenko: please drop +A on https://review.openstack.org/#/c/306644/ | 14:15 |
| degorenko | EmilienM, dropped | 14:15 |
| mjblack | degorenko: no, its a bad idea | 14:15 |
| mjblack | thats why we're doing... https://review.openstack.org/#/c/308833/ | 14:15 |
| mjblack | :D | 14:16 |
| degorenko | mjblack, so, what is the problem for failed tests for case with netstat? | 14:16 |
| degorenko | why we need complicate solution? :) | 14:16 |
| mjblack | degorenko: I think the policy-rc.d is the best solution | 14:16 |
| mjblack | if dropped before the package is installed, it'll prevent keystone from starting | 14:17 |
| *** ducttape_ has joined #puppet-openstack | 14:17 | |
| mjblack | degorenko: this is what happens when you install keystone with the policy-rc.d setup like in that review "invoke-rc.d: policy-rc.d denied execution of start." | 14:18 |
| degorenko | mjblack, hm, well, we have in fuel same solution https://github.com/openstack/fuel-library/blob/master/deployment/puppet/tweaks/manifests/ubuntu_service_override.pp | 14:18 |
| degorenko | mjblack, can you update your keystone patch with depends-on openstacklib patch? so, we can see will it work or not | 14:18 |
| mjblack | degorenko: sure but I need to update the keystone patch | 14:19 |
| *** ducttape_ has quit IRC | 14:19 | |
| degorenko | mjblack, that was i'm talking about :) | 14:19 |
| degorenko | mjblack, also i have one suggestion | 14:20 |
| mjblack | more awk? | 14:20 |
| degorenko | https://review.openstack.org/#/c/308833/3/manifests/policy/policyrcd.pp - may be we should not set to default keystone ? | 14:20 |
| degorenko | mjblack, nope, it was second idea | 14:20 |
| degorenko | :) | 14:20 |
| *** ducttape_ has joined #puppet-openstack | 14:20 | |
| mjblack | degorenko: default for keystone since its the only one with the issue we're facing right now | 14:20 |
| mjblack | but I also dont want to include in the puppet-keystone a hard coded value | 14:21 |
| degorenko | mjblack, well, i'd prefer just set services parameter as required | 14:21 |
| degorenko | so we can include this class from keystone | 14:21 |
| degorenko | mjblack, also you can do this class as define | 14:21 |
| degorenko | may be it more proper choice | 14:22 |
| degorenko | mwhahaha, ^^^ | 14:22 |
| degorenko | mjblack, so, in fuel we have exactly define | 14:22 |
| mjblack | degorenko: only way I can do a define is if I create a whole policy-rc.d system that reads directories and stuff | 14:22 |
| mwhahaha | ours removes the file after it's installed | 14:23 |
| mjblack | thats why a define works | 14:23 |
| mwhahaha | if you leave that inplace it changes the whole way the system works | 14:23 |
| mwhahaha | so i'm not sure that's ideal | 14:23 |
| mwhahaha | i'll look more in detail later | 14:23 |
| mjblack | mwhahaha: my change actually adds to the script conditionals to look for defined services to deny invoke-rc.d | 14:23 |
| mjblack | so if $1 == keystone it'll do exit 101 | 14:24 |
| mjblack | but if $1 == mysql it'll exit 0 | 14:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 14:29 |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper: Revert "Remove puppetlabs-release before installing puppet" https://review.openstack.org/308949 | 14:30 |
| *** ayoung has joined #puppet-openstack | 14:31 | |
| EmilienM | degorenko: we need to sqash https://review.openstack.org/#/c/308241/ and https://review.openstack.org/#/c/307853/ - otherwise CI will never pass. I'm on it. | 14:31 |
| openstackgerrit | Merged openstack/puppet-monasca: Gemfile: rely on puppet-openstack_spec_helper for dependencies https://review.openstack.org/307940 | 14:31 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start https://review.openstack.org/308012 | 14:34 |
| iurygregory | hey EmilienM, thanks for the +A ^^ | 14:35 |
| degorenko | EmilienM, your patch passed ci :) | 14:35 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 14:37 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 14:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: Update spec tests for Tempest https://review.openstack.org/307853 | 14:39 |
| * mjblack failing at gerrit | 14:39 | |
| EmilienM | degorenko: git rebase --interactive HEAD~2 && you replace the second pick by a "s" for squash and it works | 14:40 |
| EmilienM | degorenko: see commit ^ | 14:40 |
| degorenko | EmilienM, ok :) | 14:42 |
| *** dhollinger has joined #puppet-openstack | 14:42 | |
| openstackgerrit | Merged openstack/puppet-cinder: Add purge_config option for the cinder config https://review.openstack.org/307889 | 14:45 |
| openstackgerrit | Merged openstack/puppet-aodh: Manage policy configuration file https://review.openstack.org/305505 | 14:48 |
| openstackgerrit | Merged openstack/puppet-horizon: Move notification from File to Concat https://review.openstack.org/308294 | 14:48 |
| *** mdorman has joined #puppet-openstack | 14:48 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Add destination parameter to firewall rule https://review.openstack.org/308317 | 14:50 |
| *** chandankumar has joined #puppet-openstack | 14:52 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 14:53 |
| openstackgerrit | Merged openstack/puppet-tripleo: Enable HAProxy forwardfor option for Horizon. https://review.openstack.org/296497 | 14:53 |
| *** darkxploit has quit IRC | 14:54 | |
| *** jaosorior has quit IRC | 14:56 | |
| *** darkxploit has joined #puppet-openstack | 14:59 | |
| *** chandankumar has quit IRC | 15:01 | |
| *** foxhunt72 is now known as foxhunt_away | 15:03 | |
| EmilienM | could I have a review on https://review.openstack.org/#/c/308717/ and https://review.openstack.org/#/c/307977/ please? | 15:09 |
| *** chandankumar has joined #puppet-openstack | 15:09 | |
| *** haplo37 has joined #puppet-openstack | 15:10 | |
| *** kbyrne has quit IRC | 15:10 | |
| iurygregory | EmilienM, the bug is solved? | 15:10 |
| EmilienM | iurygregory: which bug? | 15:11 |
| iurygregory | " # ironic-conductor is broken for Ubuntu Trusty | 15:11 |
| iurygregory | # https://bugs.launchpad.net/cloud-archive/+bug/1530869" | 15:11 |
| openstack | Launchpad bug 1530869 in Ubuntu Cloud Archive "ironic-conductor does not load fake driver" [Undecided,New] | 15:11 |
| EmilienM | iurygregory: have you read the commit message in https://review.openstack.org/#/c/308717/ ? | 15:11 |
| iurygregory | oh i just got the second link | 15:12 |
| iurygregory | sorry =( | 15:12 |
| EmilienM | no worries :) | 15:12 |
| *** chandankumar has quit IRC | 15:15 | |
| *** jaosorior has joined #puppet-openstack | 15:16 | |
| *** jmikolajczak has quit IRC | 15:16 | |
| *** andreww has quit IRC | 15:27 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 15:33 |
| *** fedexo has joined #puppet-openstack | 15:38 | |
| openstackgerrit | Merged openstack/puppet-keystone: Add missing bit to the user prefetch removal. https://review.openstack.org/306313 | 15:44 |
| openstackgerrit | Merged openstack/puppet-mistral: Switch mistral to use puppet-oslo module https://review.openstack.org/306308 | 15:44 |
| openstackgerrit | Merged openstack/puppet-tempest: Add support for Gnocchi tests https://review.openstack.org/308469 | 15:44 |
| openstackgerrit | Merged openstack/puppet-neutron: Add ml2 package requirement to neutron::plugins::ml2::mech_driver https://review.openstack.org/306001 | 15:44 |
| *** gomarivera has joined #puppet-openstack | 15:45 | |
| *** dgurtner has quit IRC | 15:46 | |
| *** openstackgerrit has quit IRC | 15:48 | |
| *** jaosorior has quit IRC | 15:48 | |
| *** openstackgerrit has joined #puppet-openstack | 15:48 | |
| openstackgerrit | Merged openstack/puppet-trove: Remove the extra space in the conductor.pp https://review.openstack.org/308096 | 15:48 |
| openstackgerrit | Merged openstack/puppet-neutron: Remove deprecated options from agents https://review.openstack.org/301106 | 15:49 |
| *** dfisher has joined #puppet-openstack | 15:51 | |
| *** grimlock86 has joined #puppet-openstack | 15:51 | |
| *** trown is now known as trown|lunch | 15:53 | |
| iurygregory | have a nice day people, see you guys in Austin o/ | 15:54 |
| *** andreww has joined #puppet-openstack | 15:55 | |
| EmilienM | iurygregory: take care | 15:57 |
| dfisher | EmilienM: i saw a few tarballs were up this morning | 15:57 |
| EmilienM | dfisher: yes! my script worked | 15:57 |
| EmilienM | I'm working on stable builds now | 15:57 |
| dfisher | nice :) | 15:57 |
| EmilienM | dfisher: i'm currently asking infra to generate all tarballs, might be up today if they accept | 16:01 |
| dfisher | cool. I need the mitaka ones, but baby steps :) | 16:01 |
| *** gomarivera has quit IRC | 16:02 | |
| *** andreww is now known as xarses | 16:03 | |
| *** imcsk8|z1z is now known as imcsk8 | 16:08 | |
| *** karimb has quit IRC | 16:16 | |
| *** jistr has quit IRC | 16:21 | |
| *** grimlock86 has quit IRC | 16:27 | |
| openstackgerrit | Merged openstack/puppet-nova: Change scheduler configurations to use entrypoints https://review.openstack.org/308452 | 16:30 |
| *** Guest14509 has quit IRC | 16:40 | |
| openstackgerrit | Michael Glaser proposed openstack/puppet-keystone: Add Unit Tests for keystone_config type https://review.openstack.org/306644 | 16:41 |
| *** ayoung has quit IRC | 16:48 | |
| openstackgerrit | Matthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts https://review.openstack.org/309103 | 16:52 |
| openstackgerrit | Merged openstack/puppet-manila: Gemfile: rely on puppet-openstack_spec_helper for dependencies https://review.openstack.org/307942 | 16:52 |
| openstackgerrit | Matthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts https://review.openstack.org/309103 | 16:53 |
| *** fedexo has quit IRC | 16:56 | |
| *** leanderthal|afk has quit IRC | 16:58 | |
| *** jpena is now known as jpena|off | 16:58 | |
| *** delatte has joined #puppet-openstack | 17:01 | |
| *** trown|lunch is now known as trown | 17:02 | |
| *** delattec has quit IRC | 17:04 | |
| *** bookwar has quit IRC | 17:04 | |
| *** derekh has quit IRC | 17:05 | |
| *** Matum has joined #puppet-openstack | 17:07 | |
| *** bookwar has joined #puppet-openstack | 17:09 | |
| *** leanderthal|afk has joined #puppet-openstack | 17:10 | |
| *** tristanC_ has joined #puppet-openstack | 17:14 | |
| *** mkarpin1 has joined #puppet-openstack | 17:14 | |
| *** pleia2_ has joined #puppet-openstack | 17:15 | |
| *** shardy has quit IRC | 17:16 | |
| *** higgins` has joined #puppet-openstack | 17:18 | |
| *** johnmilton has quit IRC | 17:19 | |
| *** higgins has quit IRC | 17:19 | |
| *** gfidente has quit IRC | 17:19 | |
| *** pleia2 has quit IRC | 17:19 | |
| *** mkarpin has quit IRC | 17:19 | |
| *** dmsimard has quit IRC | 17:19 | |
| *** jasondotstar has quit IRC | 17:19 | |
| *** tristanC has quit IRC | 17:19 | |
| *** gfidente has joined #puppet-openstack | 17:19 | |
| *** gfidente has quit IRC | 17:19 | |
| *** gfidente has joined #puppet-openstack | 17:19 | |
| *** dmsimard has joined #puppet-openstack | 17:19 | |
| *** pleia2_ is now known as pleia2 | 17:20 | |
| *** johnmilton has joined #puppet-openstack | 17:22 | |
| *** jasondotstar has joined #puppet-openstack | 17:24 | |
| *** gfidente has quit IRC | 17:25 | |
| *** ayoung has joined #puppet-openstack | 17:26 | |
| *** ducttape_ has quit IRC | 17:27 | |
| *** fawadkhaliq has joined #puppet-openstack | 17:29 | |
| *** dtantsur is now known as dtantsur|afk | 17:30 | |
| *** mgould has quit IRC | 17:39 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Correct typo resulting in a hardcoded paramter used incorrectly https://review.openstack.org/299615 | 17:44 |
| *** ansmith has quit IRC | 17:45 | |
| *** ayoung has quit IRC | 17:47 | |
| *** ayoung has joined #puppet-openstack | 17:47 | |
| openstackgerrit | Merged openstack/puppet-heat: Cleanup rabbit ssl checks https://review.openstack.org/308487 | 17:57 |
| *** foxhunt_away is now known as foxhunt72 | 17:59 | |
| *** ansmith has joined #puppet-openstack | 18:01 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare https://review.openstack.org/309139 | 18:12 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare https://review.openstack.org/309139 | 18:14 |
| *** ansmith has quit IRC | 18:14 | |
| openstackgerrit | Mathieu Gagné proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/309141 | 18:14 |
| openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb https://review.openstack.org/309142 | 18:14 |
| *** degorenko is now known as _degorenko|afk | 18:14 | |
| mgagne | EmilienM: swift is partially broken in kilo without this patch: https://review.openstack.org/#/c/309141/1 | 18:14 |
| mgagne | EmilienM: can't add capacity due to the extra line | 18:15 |
| EmilienM | https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L143 | 18:17 |
| EmilienM | your backport is valid, +2 | 18:17 |
| *** darkxploit has quit IRC | 18:17 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 18:19 |
| openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb https://review.openstack.org/309144 | 18:20 |
| *** ducttape_ has joined #puppet-openstack | 18:20 | |
| *** ayoung has quit IRC | 18:21 | |
| *** ducttape_ has quit IRC | 18:21 | |
| *** ducttape_ has joined #puppet-openstack | 18:21 | |
| EmilienM | I'm starting an etherpad to track all errors that we have if we deploy Puppet OpenStack on Ubuntu Xenial | 18:22 |
| EmilienM | https://etherpad.openstack.org/p/puppet-openstack-xenial | 18:22 |
| mjblack | EmilienM: can we list ubuntu as a bug? | 18:22 |
| EmilienM | I would prefer some help | 18:23 |
| openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb https://review.openstack.org/309147 | 18:23 |
| mgagne | thanks | 18:24 |
| mjblack | EmilienM: we know puppetlabs-apache is good with Xenial? | 18:25 |
| EmilienM | why wouldn't it be? | 18:25 |
| mjblack | well with the "upstart" not functional, only guessing | 18:26 |
| crinkle | i doubt its ready for xenial yet https://github.com/puppetlabs/puppetlabs-apache#general | 18:29 |
| *** darkxploit has joined #puppet-openstack | 18:30 | |
| *** darkxploit has quit IRC | 18:32 | |
| *** darkxploit has joined #puppet-openstack | 18:33 | |
| colonwq | EmilienM, I added my unit tests and they passed (I think). I am waiting for the rest to finish. | 18:35 |
| mjblack | crinkle: doesnt mean it wont work, just that they dont test it against Xenial yet | 18:36 |
| EmilienM | crinkle: looking at apache/params.pp, does not seem we override the service provider, so it might work. From my early tests, I haven't seen Apache failing | 18:38 |
| EmilienM | actually it works http://logs.openstack.org/30/308530/3/experimental/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/4b5ced9/logs/apache/error.txt.gz | 18:39 |
| crinkle | oh neato | 18:39 |
| EmilienM | (apache starts) | 18:39 |
| EmilienM | I'm working on 2 blockers now, keystone & nova. Otherwise I think we can deploy Puppet OpenStack, or we're close. | 18:39 |
| EmilienM | the major thing in new LTS is systemd | 18:40 |
| mjblack | EmilienM: the issues you reported all look like easily solved | 18:46 |
| mjblack | well the two...mysql one have to look at | 18:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Deprecate service_provider https://review.openstack.org/309154 | 18:49 |
| EmilienM | this is the first fix ^ | 18:49 |
| openstackgerrit | Merged openstack/puppet-neutron: Add dnsmasq_dns_servers option support https://review.openstack.org/303933 | 18:53 |
| mgagne | EmilienM: I think puppet-swift broke by itself in stable/kilo due to changes in puppet-keystone | 18:59 |
| EmilienM | mgagne: what change? | 18:59 |
| EmilienM | log of failure? | 19:00 |
| mgagne | getting this error in spec: Invalid parameter: 'tenant' on Keystone_user[bar] at ./puppet-swift/spec/fixtures/modules/swift/manifests/keystone/dispersion.pp:39 | 19:00 |
| mgagne | EmilienM: currently in flight in gate and saw failure. I reproduced the failure on my laptop | 19:00 |
| EmilienM | I'll look at what logs say | 19:01 |
| mgagne | EmilienM: it looks like there was no change in puppet-swift stable/kilo since its initial release. So I guess changes to puppet-keystone were never tested against puppet-swift | 19:01 |
| mgagne | that's my first hypothesis | 19:01 |
| *** amoralej is now known as amoralej|off | 19:04 | |
| EmilienM | mgagne: I'll dig that, have a safe flight | 19:04 |
| mgagne | EmilienM: I'm not in flight, the change is :D | 19:05 |
| EmilienM | ah | 19:05 |
| EmilienM | mgagne: have a safe flight anyway, next time you take a flight | 19:05 |
| mgagne | :D | 19:05 |
| mgagne | next sunday ;) | 19:05 |
| EmilienM | cool. I take Saturday, from Quebec :P but through Montreal | 19:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Stop overriding service provider https://review.openstack.org/309156 | 19:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments https://review.openstack.org/308530 | 19:08 |
| EmilienM | ok let's try again | 19:08 |
| *** rcernin has quit IRC | 19:19 | |
| *** chandankumar has joined #puppet-openstack | 19:29 | |
| *** ansmith has joined #puppet-openstack | 19:30 | |
| *** chandankumar has quit IRC | 19:36 | |
| EmilienM | I'm seeing this error in our lint jobs http://logs.openstack.org/15/307215/2/check/gate-puppet-trove-puppet-lint/02a7d5a/console.html#_2016-04-21_17_02_23_006 | 19:37 |
| EmilienM | i'm wondering if anyone saw that one already in our CI, lately | 19:38 |
| EmilienM | mwhahaha: can you look https://review.openstack.org/#/c/308717/ please ? | 19:48 |
| EmilienM | mwhahaha: https://review.openstack.org/#/c/307977/ won't land before that | 19:48 |
| mwhahaha | k | 19:48 |
| *** morazi has quit IRC | 19:51 | |
| *** ayoung has joined #puppet-openstack | 19:55 | |
| mjblack | EmilienM: so that policy-rc.d looks like it might get that +1 from jenkins | 20:04 |
| mwhahaha | it's going to probably conflict with fuel | 20:04 |
| mjblack | inb4 jinxed | 20:04 |
| colonwq | EmilienM, Your fast. | 20:05 |
| EmilienM | I haven't followed what happened about keystone issue | 20:05 |
| EmilienM | what is the plan? | 20:05 |
| EmilienM | we have openstacklib patch and what? | 20:05 |
| mjblack | EmilienM: yeah | 20:05 |
| mjblack | the policy-rc.d that I'm implementing can be used for keystone or anything else | 20:06 |
| mwhahaha | as part of beaker tests right? not to be included by the class? | 20:06 |
| EmilienM | mwhahaha: it is: https://review.openstack.org/#/c/308012/ | 20:06 |
| EmilienM | IMHO we should not only fix beaker + integration jobs | 20:06 |
| EmilienM | we need to fix all deployments | 20:06 |
| mwhahaha | the problem is that it will conflict with fuel since we already do that | 20:06 |
| mjblack | mwhahaha: they asked me to include it in puppet-keystone but we can make it a flag that on fuel it doesnt do it | 20:07 |
| EmilienM | mwhahaha, mjblack: then, create a boolean in puppet-keystone that whether or not require it. | 20:07 |
| mjblack | sure I'll do that | 20:07 |
| *** fawadkhaliq has quit IRC | 20:11 | |
| *** rcernin has joined #puppet-openstack | 20:12 | |
| mfisch | Hunner: you have a sec for a weird error I get when I use an anchor in an ensure_resource? It's complaining about interns | 20:12 |
| *** fawadkhaliq has joined #puppet-openstack | 20:13 | |
| EmilienM | mwhahaha: also https://review.openstack.org/#/c/307968/ if possible | 20:17 |
| *** johnmilton has quit IRC | 20:17 | |
| *** fawadkhaliq has quit IRC | 20:19 | |
| *** fawadkhaliq has joined #puppet-openstack | 20:20 | |
| *** paramite has quit IRC | 20:24 | |
| *** fawadkhaliq has quit IRC | 20:24 | |
| *** delatte has quit IRC | 20:25 | |
| openstackgerrit | Merged openstack/puppet-ironic: install ipmitool on ubuntu if driver enabled https://review.openstack.org/308717 | 20:25 |
| *** fawadkhaliq has joined #puppet-openstack | 20:25 | |
| *** foxhunt72 is now known as foxhunt_away | 20:26 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again https://review.openstack.org/307977 | 20:31 |
| *** ducttape_ has quit IRC | 20:35 | |
| *** morazi has joined #puppet-openstack | 20:38 | |
| *** trown is now known as trown|outtypewww | 20:39 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 20:41 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 20:42 |
| *** morazi has quit IRC | 20:43 | |
| *** ansmith has quit IRC | 20:46 | |
| *** mdorman has quit IRC | 20:47 | |
| *** cdelatte has joined #puppet-openstack | 20:48 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor https://review.openstack.org/285847 | 20:50 |
| *** fawadk has joined #puppet-openstack | 20:54 | |
| *** fawadkhaliq has quit IRC | 20:54 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class. https://review.openstack.org/308833 | 20:54 |
| mjblack | think its time for sleep | 20:55 |
| *** morazi has joined #puppet-openstack | 20:56 | |
| *** ducttape_ has joined #puppet-openstack | 20:59 | |
| *** fawadk has quit IRC | 20:59 | |
| *** ducttape_ has quit IRC | 21:00 | |
| *** ducttape_ has joined #puppet-openstack | 21:03 | |
| openstackgerrit | Merged openstack/puppet-sahara: Enable Sahara testing on Ubuntu https://review.openstack.org/307968 | 21:04 |
| *** tristanC_ is now known as tristanC | 21:07 | |
| *** fawadkhaliq has joined #puppet-openstack | 21:07 | |
| *** fawadkhaliq has quit IRC | 21:08 | |
| *** fawadkhaliq has joined #puppet-openstack | 21:08 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 21:09 |
| *** haplo37 has quit IRC | 21:15 | |
| *** jtomasek has quit IRC | 21:27 | |
| mjblack | EmilienM: http://logs.openstack.org/33/308833/7/check/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-trusty/07c3a03/console.html#_2016-04-21_21_18_58_897 | 21:28 |
| mjblack | (ﾉಥ益ಥ）ﾉ ┻━┻ | 21:29 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add types, providers and unit tests https://review.openstack.org/309187 | 21:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Remove tenant parameter from keystone_user https://review.openstack.org/309189 | 21:36 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/309141 | 21:36 |
| EmilienM | mgagne: ^ | 21:36 |
| EmilienM | thanks for being our stable/kilo beta tester | 21:36 |
| mgagne | EmilienM: hehe, we are working on upgrading swift now =) | 21:36 |
| EmilienM | nice | 21:36 |
| mgagne | already running kilo everywhere else | 21:37 |
| EmilienM | mgagne: still using puppet openstack? | 21:37 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstack-integration: Filter debug messages. https://review.openstack.org/304797 | 21:37 |
| mgagne | puppet-openstack? we have a module with that name but it's our internal module | 21:37 |
| EmilienM | mgagne: I mean, our modules | 21:37 |
| mgagne | yes, for all services | 21:38 |
| mjblack | EmilienM: that still exists? | 21:38 |
| EmilienM | mgagne: cool | 21:39 |
| EmilienM | mjblack: what? | 21:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Deprecate service_provider https://review.openstack.org/309154 | 21:39 |
| mjblack | EmilienM: puppet-openstack module | 21:39 |
| *** rcernin has quit IRC | 21:39 | |
| EmilienM | mjblack: https://github.com/stackforge/puppet-openstack | 21:40 |
| EmilienM | no :) | 21:40 |
| mjblack | was going to say | 21:40 |
| mgagne | mjblack: we have a private module named openstack. it's not the same as the one upstream. it's our own implementation which tries to fix inconsistencies between modules and also make it a one-stop shop if you wish to install a service. | 21:42 |
| mjblack | EmilienM: can we just workflow the openstacklib change through? that error isnt from me... but a problem with git :D | 21:42 |
| EmilienM | got this SSL issue on xenial http://logs.openstack.org/30/308530/4/experimental/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/32df388/console.html#_2016-04-21_19_28_13_289 | 21:42 |
| mgagne | mjblack: it's greatly simplifier to support our own needs, use cases and drivers. so for example, you won't be able to install neutron with ovs with this module. | 21:42 |
| EmilienM | I'm going to run now and try to stay away from work a bit, but if I fail, ttyl | 21:43 |
| mjblack | mgagne: yeah I did the same with grizzly/havana/icehouse... for liberty/mitaka I went more pure hiera driven than custom manifests | 21:43 |
| openstackgerrit | Merged openstack/puppet-nova: Remove deprecated qpid config/classes https://review.openstack.org/303877 | 21:43 |
| openstackgerrit | Merged openstack/puppet-glance: Remove docs duplication for notification_driver https://review.openstack.org/308467 | 21:44 |
| mgagne | mjblack: advantage of our module is that all Hiera configs are under the openstack::<service> namespace. no need to read upstream modules source to find which configs are available. | 21:44 |
| mgagne | we tried with pure hiera and it's just madness | 21:45 |
| mjblack | hmmm I got it down pretty easy | 21:45 |
| mjblack | hiera lookups in hiera made it a lot easier | 21:45 |
| mjblack | I got an openstack region config down to about 15 lines of hiera that changes per region | 21:45 |
| mgagne | hehe | 21:46 |
| mjblack | not including the clientcert yaml files that include the classes of course | 21:46 |
| *** fawadkhaliq has joined #puppet-openstack | 21:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 21:55 |
| openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class. https://review.openstack.org/308833 | 22:00 |
| *** dhollinger has quit IRC | 22:02 | |
| EmilienM | dfisher: do you like it? http://tarballs.openstack.org/puppet-aodh/ | 22:10 |
| EmilienM | dfisher: next steps are: support tags | 22:10 |
| EmilienM | what do you prefer? | 22:10 |
| *** Matum has quit IRC | 22:11 | |
| *** MatUm has joined #puppet-openstack | 22:11 | |
| dfisher | that's outstanding. | 22:14 |
| dfisher | how often are those stable tarballs created? | 22:14 |
| *** fawadkhaliq has quit IRC | 22:16 | |
| EmilienM | dfisher: at every new commit | 22:18 |
| dfisher | hmm. | 22:18 |
| dfisher | it would be good to have an 8.0.0 tarball (and a 7.0.0 / 6.1.0 …) cut from that commit | 22:19 |
| dfisher | which doesn't change | 22:19 |
| dfisher | maybe | 22:19 |
| EmilienM | dfisher: I'm working on it later this week | 22:19 |
| EmilienM | damn it's thursday | 22:19 |
| EmilienM | well, sometimes | 22:19 |
| dfisher | sweet :) | 22:20 |
| dfisher | that would be all I need. | 22:20 |
| EmilienM | dfisher: I like IPA btw | 22:20 |
| dfisher | ha! | 22:22 |
| mjblack | when patch passes CI and someone does a -1... http://replygif.net/i/1166.gif | 22:22 |
| mjblack | especially this past week | 22:22 |
| dfisher | EmilienM: you're in Montreal, right? | 22:23 |
| EmilienM | dfisher: no, in Quebec City | 22:24 |
| dfisher | k :) | 22:24 |
| dfisher | i'm looking up how to send beer! | 22:24 |
| *** fawadkhaliq has joined #puppet-openstack | 22:25 | |
| EmilienM | no worries, we have enough beers here | 22:25 |
| dfisher | molson isn't beer :) | 22:25 |
| dfisher | ^ fighting words for a Canadian, btw | 22:26 |
| bkero | Unibroue++ | 22:26 |
| *** ducttape_ has quit IRC | 22:27 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor https://review.openstack.org/285847 | 22:28 |
| *** fawadkhaliq has quit IRC | 22:28 | |
| mjblack | dfisher: canadians dont fight though, they apologize for your slight | 22:28 |
| *** fawadkhaliq has joined #puppet-openstack | 22:30 | |
| dfisher | mjblack: if I start insulting Molson, Tim Hortons, Hockey, or the RCMP, I'm getting my sweater pulled over my head and hit in the stomach. | 22:30 |
| mjblack | dfisher: well everyone has their breaking point | 22:31 |
| dfisher | :) | 22:32 |
| *** fawadkhaliq has quit IRC | 22:40 | |
| *** morazi has quit IRC | 22:45 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: lbaas v2 service plugin support. https://review.openstack.org/301797 | 22:45 |
| *** dfisher has quit IRC | 22:51 | |
| Hunner | mfisch: interns with anchors in ensure_resource? Ick | 22:54 |
| *** MatUm has quit IRC | 22:56 | |
| *** MatUm has joined #puppet-openstack | 22:56 | |
| *** markvoelker has quit IRC | 23:20 | |
| *** haplo37 has joined #puppet-openstack | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
