Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-04-21

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

All times shown according to UTC.

Time Nick Message
00:05 xarses joined #puppet-openstack
00:08 xarses_ joined #puppet-openstack
00:29 cdelatte joined #puppet-openstack
00:34 ducttape_ joined #puppet-openstack
00:36 rchurch_ joined #puppet-openstack
00:37 ntpttr_ joined #puppet-openstack
00:44 ntpttr joined #puppet-openstack
00:46 openstackgerrit zhongshengping proposed openstack/puppet-trove: Remove the extra space in the conductor.pp  https://review.openstack.org/308096
00:53 darkxploit joined #puppet-openstack
00:53 jesusaur joined #puppet-openstack
00:55 iurygregory joined #puppet-openstack
01:20 johnmilton joined #puppet-openstack
01:30 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
01:31 EmilienM mwhahaha: never give up https://review.openstack.org/#/c/306677/
01:32 ntpttr- joined #puppet-openstack
01:36 kylo joined #puppet-openstack
01:46 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton  https://review.openstack.org/308573
02:05 openstackgerrit Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken  https://review.openstack.org/266723
02:16 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:17 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:18 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:19 mwhahaha Stupid patch
02:20 mwhahaha Some day it might land
02:20 ducttape_ joined #puppet-openstack
02:28 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:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/307977
02:32 colonwq EmilienM, I'm testing the same fix in puppet-ceph.
02:43 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
02:49 v1k0d3n joined #puppet-openstack
02:52 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:55 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:56 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:57 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:58 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:59 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
03:00 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:01 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:02 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:03 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:04 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:05 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:06 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:07 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:08 mwhahaha which one?
03:08 mjblack http://manpages.ubuntu.com/manpages/wily/man8/zg-policy-rc.d.8.html
03:10 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:11 mjblack we could in theory use it to make ubuntu behave more like centos
03:12 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:13 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:14 mwhahaha or just do the hack in the beaker test :D
03:15 mjblack honestly, I rather do it as an option for ubuntu
03:27 xarses_ joined #puppet-openstack
03:36 andreww joined #puppet-openstack
03:49 openstackgerrit Merged openstack/puppet-neutron: Don't declare the service when service_enable is false  https://review.openstack.org/307879
03:50 openstackgerrit Merged openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb  https://review.openstack.org/307465
04:06 openstackgerrit Merged openstack/puppet-barbican: Totally drop Qpid support  https://review.openstack.org/307110
04:10 openstackgerrit Merged openstack/puppet-nova: Add other virt services  https://review.openstack.org/306677
04:12 bapalm joined #puppet-openstack
04:12 delattec joined #puppet-openstack
04:20 fedexo joined #puppet-openstack
04:27 openstackgerrit Merged openstack/puppet-ceilometer: Totally drop Qpid support  https://review.openstack.org/307093
04:43 ducttape_ joined #puppet-openstack
05:05 chandankumar joined #puppet-openstack
05:07 jaosorior joined #puppet-openstack
05:17 rcernin joined #puppet-openstack
05:49 zerda joined #puppet-openstack
06:27 jmikolajczak joined #puppet-openstack
06:32 angdraug joined #puppet-openstack
06:45 openstackgerrit Jerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/301106
06:47 yolanda joined #puppet-openstack
07:00 aarefiev joined #puppet-openstack
07:02 karimb joined #puppet-openstack
07:03 tesseract joined #puppet-openstack
07:11 paramite joined #puppet-openstack
07:15 dgurtner joined #puppet-openstack
07:21 hjensas joined #puppet-openstack
07:32 shardy joined #puppet-openstack
07:54 hogepodge joined #puppet-openstack
08:04 darkxploit joined #puppet-openstack
08:13 jistr joined #puppet-openstack
08:23 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class.  https://review.openstack.org/308833
08:30 jaosorior joined #puppet-openstack
08:31 jtomasek joined #puppet-openstack
08:34 derekh joined #puppet-openstack
08:46 ducttape_ joined #puppet-openstack
09:01 gfidente joined #puppet-openstack
09:05 higgins joined #puppet-openstack
09:26 openstackgerrit Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp  https://review.openstack.org/308863
09:27 markvoelker joined #puppet-openstack
09:46 openstackgerrit Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp  https://review.openstack.org/308863
09:53 johnmilton joined #puppet-openstack
10:00 openstackgerrit Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp  https://review.openstack.org/308863
10:24 Guest14509 joined #puppet-openstack
10:38 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:47 dgurtner joined #puppet-openstack
10:47 ducttape_ joined #puppet-openstack
10:53 jaosorior joined #puppet-openstack
10:54 imcsk8|z1z joined #puppet-openstack
11:25 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Relove puppetlabs-relase version conflict  https://review.openstack.org/308722
11:26 johnmilton joined #puppet-openstack
11:34 openstackgerrit Merged openstack/puppet-openstack-integration: Change source for puppet-corosync  https://review.openstack.org/308424
11:44 EmilienM o/
11:44 chem yo
11:46 colonwq EmilienM, what does OVH stand for?
11:47 EmilienM colonwq: it's a public cloud, where OpenStack Infra is consummer
11:47 dmsimard colonwq: https://www.ovh.com/us/
11:48 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:49 darkxploit joined #puppet-openstack
11:49 colonwq EmilienM, dmsimard  ok. thanks. Hopefully they are not in one of the parts of texas under water.
11:52 dmsimard I think the capacity they provide to -infra is out of Montreal (Beauharnois) but I could be mistaken
11:54 dmsimard Ah, so out of Canada and France. They don't have water problems :p
11:57 colonwq that is good. It explains the French which shows up sometimes.
11:57 ansmith joined #puppet-openstack
12:00 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/307920
12:05 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/307920
12:10 ducttape_ joined #puppet-openstack
12:12 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:13 EmilienM 0 failure
12:14 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:16 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:17 mattymo the operator case is I have 3 controller nodes, 1 failed hardcore, I decommission it, and deploy a 4th one
12:17 markvoelker 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:18 mattymo EmilienM, just checking if there's anything already out there related to this task. It looks like nobody cares so much yet
12:22 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Revert "Remove puppetlabs-release before installing puppet"  https://review.openstack.org/308949
12:23 EmilienM mattymo: nope
12:30 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:32 EmilienM nevermind, i'll deprecate neutron_auth_plugin to avoid confusions
12:33 degorenko EmilienM, we have some lags in internal network, sorry for delay
12:33 EmilienM no worries
12:33 EmilienM I have some lag on morning, time I wake up
12:34 degorenko :)
12:34 degorenko EmilienM, i think you can set neutron_auth_type to same value as now neutron_auth_plugin
12:35 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:36 social 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:39 EmilienM I'm pushing a patch, you'll review
12:39 degorenko sure, sir
12:42 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/307907
12:43 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:46 openstackgerrit Marcos Fermín Lobo proposed openstack/puppet-ec2api: Changing default params for db/* and init.pp  https://review.openstack.org/308863
12:47 openstackgerrit Merged openstack/puppet-mistral: Totally drop Qpid support  https://review.openstack.org/307111
12:49 openstackstatus NOTICE: OVH provider is enabled again, please wait for the job queue to be processed
12:56 openstackgerrit Merged openstack/puppet-monasca: Add ability to include libvirt scale group metadata  https://review.openstack.org/308589
12:56 richm joined #puppet-openstack
12:58 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
13:01 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 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:02 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:08 chandankumar joined #puppet-openstack
13:12 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:13 EmilienM why not doing that only for keystone?
13:15 mjblack I guess I could see about making it only work for keystone
13:21 karimb joined #puppet-openstack
13:24 mjblack EmilienM: you ok with an inline template to create the shell script?
13:25 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:53 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: add more debug when Puppet fails  https://review.openstack.org/308991
13:54 ducttape_ joined #puppet-openstack
14:08 * mjblack grumbles
14:09 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:10 mjblack this is the error http://paste.openstack.org/show/494990/
14:11 colonwq EmilienM, I am checking on the unit test answer from your review.
14:14 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:15 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:16 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:17 mjblack if dropped before the package is installed, it'll prevent keystone from starting
14:17 ducttape_ joined #puppet-openstack
14:18 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:19 mjblack degorenko: sure but I need to update the keystone patch
14:19 degorenko mjblack, that was i'm talking about :)
14:20 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_ joined #puppet-openstack
14:20 mjblack degorenko: default for keystone since its the only one with the issue we're facing right now
14:21 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:22 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:23 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:24 mjblack so if $1 == keystone it'll do exit 101
14:24 mjblack but if $1 == mysql it'll exit 0
14:29 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/307920
14:30 openstackgerrit Merged openstack/puppet-openstack_spec_helper: Revert "Remove puppetlabs-release before installing puppet"  https://review.openstack.org/308949
14:31 ayoung 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:34 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
14:35 iurygregory hey EmilienM, thanks for the +A ^^
14:35 degorenko EmilienM, your patch passed ci :)
14:37 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/308012
14:39 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:40 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:42 degorenko EmilienM, ok :)
14:42 dhollinger joined #puppet-openstack
14:45 openstackgerrit Merged openstack/puppet-cinder: Add purge_config option for the cinder config  https://review.openstack.org/307889
14:48 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 joined #puppet-openstack
14:50 openstackgerrit Merged openstack/puppet-tripleo: Add destination parameter to firewall rule  https://review.openstack.org/308317
14:52 chandankumar joined #puppet-openstack
14:53 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:59 darkxploit joined #puppet-openstack
15:09 EmilienM could I have a review on https://review.openstack.org/#/c/308717/ and https://review.openstack.org/#/c/307977/ please?
15:09 chandankumar joined #puppet-openstack
15:10 haplo37 joined #puppet-openstack
15:10 iurygregory EmilienM, the bug is solved?
15:11 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:12 iurygregory oh i just got the second link
15:12 iurygregory sorry =(
15:12 EmilienM no worries :)
15:16 jaosorior joined #puppet-openstack
15:33 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
15:38 fedexo joined #puppet-openstack
15:43 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:45 gomarivera joined #puppet-openstack
15:48 openstackgerrit joined #puppet-openstack
15:48 openstackgerrit Merged openstack/puppet-trove: Remove the extra space in the conductor.pp  https://review.openstack.org/308096
15:49 openstackgerrit Merged openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/301106
15:51 dfisher joined #puppet-openstack
15:51 grimlock86 joined #puppet-openstack
15:54 iurygregory have a nice day people, see you guys in Austin o/
15:55 andreww joined #puppet-openstack
15:57 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 :)
16:01 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:30 openstackgerrit Merged openstack/puppet-nova: Change scheduler configurations to use entrypoints  https://review.openstack.org/308452
16:41 openstackgerrit Michael Glaser proposed openstack/puppet-keystone: Add Unit Tests for keystone_config type  https://review.openstack.org/306644
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:52 openstackgerrit Merged openstack/puppet-manila: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/307942
16:53 openstackgerrit Matthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts  https://review.openstack.org/309103
17:01 delatte joined #puppet-openstack
17:07 Matum joined #puppet-openstack
17:09 bookwar joined #puppet-openstack
17:10 leanderthal|afk joined #puppet-openstack
17:14 tristanC_ joined #puppet-openstack
17:14 mkarpin1 joined #puppet-openstack
17:15 pleia2_ joined #puppet-openstack
17:18 higgins` joined #puppet-openstack
17:19 gfidente joined #puppet-openstack
17:19 gfidente joined #puppet-openstack
17:19 dmsimard joined #puppet-openstack
17:22 johnmilton joined #puppet-openstack
17:24 jasondotstar joined #puppet-openstack
17:26 ayoung joined #puppet-openstack
17:29 fawadkhaliq joined #puppet-openstack
17:44 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Correct typo resulting in a hardcoded paramter used incorrectly  https://review.openstack.org/299615
17:47 ayoung joined #puppet-openstack
17:57 openstackgerrit Merged openstack/puppet-heat: Cleanup rabbit ssl checks  https://review.openstack.org/308487
18:01 ansmith joined #puppet-openstack
18:12 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare  https://review.openstack.org/309139
18:14 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare  https://review.openstack.org/309139
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 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:17 EmilienM https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L143
18:17 EmilienM your backport is valid, +2
18:19 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class.  https://review.openstack.org/308833
18:20 openstackgerrit Shaik Apsar proposed openstack/puppet-trove: Remove nova_* options from trove-guestagent.conf.erb  https://review.openstack.org/309144
18:20 ducttape_ joined #puppet-openstack
18:21 ducttape_ joined #puppet-openstack
18:22 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:23 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:24 mgagne thanks
18:25 mjblack EmilienM: we know puppetlabs-apache is good with Xenial?
18:25 EmilienM why wouldn't it be?
18:26 mjblack well with the "upstart" not functional, only guessing
18:29 crinkle i doubt its ready for xenial yet https://github.com/puppetlabs/puppetlabs-apache#general
18:30 darkxploit joined #puppet-openstack
18:33 darkxploit joined #puppet-openstack
18:35 colonwq EmilienM, I added my unit tests and they passed (I think). I am waiting for the rest to finish.
18:36 mjblack crinkle: doesnt mean it wont work, just that they dont test it against Xenial yet
18:38 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:39 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:40 EmilienM the major thing in new LTS is systemd
18:46 mjblack EmilienM: the issues you reported all look like easily solved
18:47 mjblack well the two...mysql one have to look at
18:49 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Deprecate service_provider  https://review.openstack.org/309154
18:49 EmilienM this is the first fix ^
18:53 openstackgerrit Merged openstack/puppet-neutron: Add dnsmasq_dns_servers option support  https://review.openstack.org/303933
18:59 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:01 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:04 EmilienM mgagne: I'll dig that, have a safe flight
19:05 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:06 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:08 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:29 chandankumar joined #puppet-openstack
19:30 ansmith joined #puppet-openstack
19:37 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:38 EmilienM i'm wondering if anyone saw that one already in our CI, lately
19:48 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:55 ayoung joined #puppet-openstack
20:04 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:05 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:12 rcernin 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:13 fawadkhaliq joined #puppet-openstack
20:17 EmilienM mwhahaha: also https://review.openstack.org/#/c/307968/ if possible
20:20 fawadkhaliq joined #puppet-openstack
20:25 openstackgerrit Merged openstack/puppet-ironic: install ipmitool on ubuntu if driver enabled  https://review.openstack.org/308717
20:25 fawadkhaliq joined #puppet-openstack
20:31 openstackgerrit Merged openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/307977
20:38 morazi joined #puppet-openstack
20:41 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class.  https://review.openstack.org/308833
20:42 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class.  https://review.openstack.org/308833
20:48 cdelatte joined #puppet-openstack
20:50 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/285847
20:54 fawadk joined #puppet-openstack
20:54 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added update-rc.d class.  https://review.openstack.org/308833
20:55 mjblack think its time for sleep
20:56 morazi joined #puppet-openstack
20:59 ducttape_ joined #puppet-openstack
21:03 ducttape_ joined #puppet-openstack
21:04 openstackgerrit Merged openstack/puppet-sahara: Enable Sahara testing on Ubuntu  https://review.openstack.org/307968
21:07 fawadkhaliq joined #puppet-openstack
21:08 fawadkhaliq joined #puppet-openstack
21:09 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/308012
21:28 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:29 mjblack ‎(ノಥ益ಥ)ノ ┻━┻
21:34 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add types, providers and unit tests  https://review.openstack.org/309187
21:36 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:37 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:38 mgagne yes, for all services
21:38 mjblack EmilienM: that still exists?
21:39 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:40 EmilienM mjblack: https://github.com/stackforge/puppet-openstack
21:40 EmilienM no :)
21:40 mjblack was going to say
21:42 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:43 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:44 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:45 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:46 mgagne hehe
21:46 mjblack not including the clientcert yaml files that include the classes of course
21:52 fawadkhaliq joined #puppet-openstack
21:55 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/307920
22:00 openstackgerrit Matthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class.  https://review.openstack.org/308833
22:10 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:11 MatUm joined #puppet-openstack
22:14 dfisher that's outstanding.
22:14 dfisher how often are those stable tarballs created?
22:18 EmilienM dfisher: at every new commit
22:18 dfisher hmm.
22:19 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:20 dfisher sweet :)
22:20 dfisher that would be all I need.
22:20 EmilienM dfisher: I like IPA btw
22:22 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:23 dfisher EmilienM: you're in Montreal, right?
22:24 EmilienM dfisher: no, in Quebec City
22:24 dfisher k :)
22:24 dfisher i'm looking up how to send beer!
22:25 fawadkhaliq joined #puppet-openstack
22:25 EmilienM no worries, we have enough beers here
22:25 dfisher molson isn't beer :)
22:26 dfisher ^ fighting words for a Canadian, btw
22:26 bkero Unibroue++
22:27 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/285847
22:28 mjblack dfisher: canadians dont fight though, they apologize for your slight
22:30 fawadkhaliq 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:31 mjblack dfisher: well everyone has their breaking point
22:31 dfisher :)
22:45 openstackgerrit Matthew J Black proposed openstack/puppet-neutron: lbaas v2 service plugin support.  https://review.openstack.org/301797
22:54 Hunner mfisch: interns with anchors in ensure_resource? Ick
22:56 MatUm joined #puppet-openstack
23:36 haplo37 joined #puppet-openstack

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