Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:14 markvoelker joined #puppet-openstack
00:46 mwhahaha _ody: yea it'd probably be ok, we still test the other platforms in unit tests because of https://github.com/openstack/puppet-opens​tack_spec_helper/blob/master/lib/puppet-o​penstack_spec_helper/defaults.rb#L8-L20
00:59 xarses joined #puppet-openstack
01:24 iurygregory_ joined #puppet-openstack
01:48 ilbot3 joined #puppet-openstack
01:48 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
01:52 openstackgerrit Colleen Murphy proposed openstack/puppet-nova: Fix default keystone auth_plugin for nova/neutron  https://review.openstack.org/312300
02:09 jaosorior joined #puppet-openstack
02:30 ducttape_ joined #puppet-openstack
02:31 EmilienM _ody: +1 for doc
02:31 EmilienM _ody: right, EL7 & 14.04 are reference distros since kilo I think
02:32 EmilienM bkero: excellent work on doc, thanks
02:32 EmilienM bkero: failures doc aren't necessary atm, we can drop them from wiki
02:54 ducttape_ joined #puppet-openstack
02:59 bkero ok
03:03 ducttape_ joined #puppet-openstack
03:08 xingchao joined #puppet-openstack
03:20 chandankumar joined #puppet-openstack
03:23 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
03:27 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
03:28 cwolferh joined #puppet-openstack
03:29 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
03:52 markvoelker joined #puppet-openstack
03:54 fedexo joined #puppet-openstack
04:06 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate unit testing wiki  https://review.openstack.org/311161
04:06 jaosorior joined #puppet-openstack
04:31 fedexo joined #puppet-openstack
04:31 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
05:08 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
05:08 mwhahaha nova acceptance tests no longer fit in 1g vms :/
05:16 yolanda joined #puppet-openstack
05:46 mnaser joined #puppet-openstack
06:07 jaosorior_ joined #puppet-openstack
06:17 jaosorior_ joined #puppet-openstack
06:18 openstackgerrit joined #puppet-openstack
06:23 jmikolajczak joined #puppet-openstack
06:32 chandankumar joined #puppet-openstack
06:52 paramite joined #puppet-openstack
06:54 markvoelker joined #puppet-openstack
07:01 tesseract joined #puppet-openstack
07:32 yolanda joined #puppet-openstack
07:38 jaosorior joined #puppet-openstack
07:41 karimb joined #puppet-openstack
07:49 ayoung joined #puppet-openstack
07:54 dmk0202 joined #puppet-openstack
07:56 evidex joined #puppet-openstack
07:56 shardy joined #puppet-openstack
08:04 jaosorior_ joined #puppet-openstack
08:19 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Default keystone manage_endpoint to false  https://review.openstack.org/312373
08:24 jistr joined #puppet-openstack
08:28 openstackgerrit Merged openstack/puppet-vswitch: set dkms_ensure to False by default  https://review.openstack.org/311779
08:29 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/304125
08:33 gfidente joined #puppet-openstack
08:33 gfidente joined #puppet-openstack
08:40 mgould joined #puppet-openstack
08:41 chem joined #puppet-openstack
08:46 xarses joined #puppet-openstack
08:50 markvoelker joined #puppet-openstack
08:56 derekh joined #puppet-openstack
09:19 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: WIP: Puppet manifest to configure NTP as a composable service  https://review.openstack.org/310725
09:41 yolanda joined #puppet-openstack
09:50 openstackgerrit Merged openstack/puppet-tripleo: Move databases creation and sync with the role  https://review.openstack.org/309450
09:53 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Noop start/stop/restart action for Glance and Keystone in the roles  https://review.openstack.org/310935
10:30 openstackgerrit Merged openstack/puppet-tripleo: Noop start/stop/restart action for Glance and Keystone in the roles  https://review.openstack.org/310935
10:35 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
10:38 yolanda joined #puppet-openstack
10:51 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add logging class  https://review.openstack.org/312446
10:54 yottatsa joined #puppet-openstack
11:00 xb_ joined #puppet-openstack
11:10 jaosorior joined #puppet-openstack
11:16 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
11:18 johnmilton joined #puppet-openstack
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Default keystone manage_endpoint to false  https://review.openstack.org/312373
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/304125
11:28 yottatsa_ joined #puppet-openstack
11:30 xb_ joined #puppet-openstack
11:34 xingchao joined #puppet-openstack
11:46 ducttape_ joined #puppet-openstack
12:01 iurygregory bkero, i'm finishing today =)
12:02 cdelatte joined #puppet-openstack
12:02 iurygregory bkero, i agree that we can move without the index, EmilienM what you think?
12:03 cgfbee joined #puppet-openstack
12:04 markvoelker joined #puppet-openstack
12:05 morazi joined #puppet-openstack
12:07 EmilienM mwhahaha: why?
12:07 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: WIP: Puppet manifest to configure NTP as a composable service  https://review.openstack.org/310725
12:07 EmilienM iurygregory: without the index? how's that?
12:09 jaosorior EmilienM: The index is not necessarily needed. But then the document is not accessible unless you explicitly access that specific document
12:10 jaosorior EmilienM: What bkero had initially proposed (in the summit), is to upload the files without the references in the index to avoid merge conflicts, and afterwards push them to the index, once they're done.
12:10 EmilienM jaosorior: why that?
12:11 EmilienM it's an early stage document, let's get the more feedback as possible
12:11 iurygregory EmilienM, if every commit have the index people will have merge conflict
12:11 EmilienM so keep indexes
12:11 EmilienM iurygregory: that's not a problem, I solve merged conflicts every day
12:12 jaosorior EmilienM: It just makes it slightly faster. But either way, for the commits I proposed I ended up solving the merge conflicts. They are pretty trivial. So it's just a matter of making the merging of the docs slightly faster
12:12 iurygregory EmilienM, yeah it's easy soo i will ask to have the index
12:13 jaosorior index it is then
12:15 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
12:15 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate unit testing wiki  https://review.openstack.org/311161
12:16 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds  https://review.openstack.org/309542
12:16 EmilienM mwhahaha: ok this done we're good ^ ovn can be deployed out of the box on ubuntu/redhat
12:16 EmilienM this time*
12:17 dprince joined #puppet-openstack
12:18 iurygregory EmilienM, https://wiki.openstack.org/wiki/PTL_Guide is deprecated but i think http://docs.openstack.org/project-team-guide does not cover everything that is in the wiki, should i just point the docs?
12:19 EmilienM iurygregory: http://docs.openstack.org/developer/puppe​t-openstack-guide/community.html#ptl-duty
12:19 iurygregory Tks man
12:19 iurygregory i will refer to this =)
12:38 ansmith joined #puppet-openstack
12:54 karimb joined #puppet-openstack
12:56 richm joined #puppet-openstack
13:02 astupnikov joined #puppet-openstack
13:06 gfidente joined #puppet-openstack
13:06 gfidente joined #puppet-openstack
13:09 astupnikov joined #puppet-openstack
13:17 astupnikov joined #puppet-openstack
13:20 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Create dbs in step 3 for the roles  https://review.openstack.org/312496
13:20 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Create dbs in step 3 for the roles  https://review.openstack.org/312496
13:25 astupnikov joined #puppet-openstack
13:27 mjblack EmilienM: with openstack_extras repo
13:28 mjblack I believe xenial comes with mitaka by default
13:28 EmilienM right
13:28 mjblack so creating the repo will be a problem since there is no uca repo for xenial/mitaka
13:28 EmilienM https://review.openstack.org/​#/c/308530/6/manifests/repos.pp
13:29 EmilienM that's why I'm not deploying UCA when I test xenial
13:29 mjblack there go, thats the solution :D
13:30 mjblack EmilienM: it looks like if you try to use lxd with xenial host and trusty64 container, the kernel version is too high for trusty ovs 2.5
13:30 EmilienM our CI does not run containers
13:30 EmilienM honestly, I just want to make our CI working on xenial
13:31 mjblack yeah
13:31 mjblack I'm testing out lxd cause it might solve some solutions for us in CI
13:31 mjblack like simulating multihosts
13:31 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
13:33 astupnikov joined #puppet-openstack
13:33 mjblack just a test that I'm doing in my environment
13:38 fwdit joined #puppet-openstack
13:40 dfisher joined #puppet-openstack
14:01 mwhahaha EmilienM: when I ran the local nova beaker tests I get fork/memory failures with 1g. 2g is fine
14:02 EmilienM mwhahaha: :( what do we run more than integration jobs?
14:04 mwhahaha i'm not sure what sizes get used on infra
14:04 jtomasek joined #puppet-openstack
14:05 EmilienM I don't understand why our integration jobs don't have troubles and our beaker jobs for nova have issues
14:07 mwhahaha no this is when i run locally on my laptop, the vagrant image is a 1g vm because that's the default in default.yml
14:08 mwhahaha at some point i'll look into it more, just thought i'd mention it
14:08 EmilienM ah
14:08 EmilienM iberezovskiy, degorenko: ping
14:09 EmilienM mwhahaha: can we get https://review.openstack.org/#/c/309542/ please?
14:09 EmilienM they fixed packaging in fedora, so everything works out of the box now
14:10 mwhahaha yay
14:12 dhollinger joined #puppet-openstack
14:12 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test in stable/mitaka  https://review.openstack.org/312523
14:13 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: CI test in stable/mitaka  https://review.openstack.org/312524
14:14 dmk0202 joined #puppet-openstack
14:14 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: CI test in stable/mitaka  https://review.openstack.org/312527
14:14 EmilienM our unit tests for stable might be broken, I'm investigating
14:14 EmilienM I saw https://review.openstack.org/#/c/312266/
14:18 twm2016 joined #puppet-openstack
14:19 jmikolajczak joined #puppet-openstack
14:21 bnemec joined #puppet-openstack
14:23 mgould joined #puppet-openstack
14:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/304125
14:25 jaosorior joined #puppet-openstack
14:26 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs  https://review.openstack.org/310022
14:28 dfisher EmilienM: are release tarballs for each module still in the works?
14:29 EmilienM dfisher: pabelanger is still working on https://review.openstack.org/#/c/311754/, it should land today
14:29 dfisher nice.
14:29 EmilienM when we have that, we'll be able to create tarballs with tags
14:36 v1k0d3n joined #puppet-openstack
14:38 mwhahaha question for folks, so in the nova_flavor provider work it appears that it's being run before the keystone endpoint is created. Any thoughts on the correct way to make sure the provider doesn't run until after the nova service and the associated keystone items are finished?
14:38 mwhahaha on centos it's failing (works on ubuntu)
14:39 mwhahaha http://logs.openstack.org/63/305463/11/check/g​ate-puppet-nova-puppet-beaker-rspec-centos-7/a​a2383a/console.html#_2016-05-04_05_22_44_742
14:39 EmilienM mwhahaha: maybe autorequire on something?
14:39 mwhahaha i auto required the nova-api
14:40 mwhahaha but it's the keystone items that i'd need to autorequire i guess?
14:40 EmilienM not enough imho
14:40 EmilienM maybe we need to autorequire on keystone_<something>
14:40 mwhahaha probably
14:40 EmilienM I remember the same issue with glance_image
14:40 mwhahaha keystone_service i guess
14:40 mwhahaha yea i tried to model glance_image
14:41 EmilienM mwhahaha: no, you need the user_role I guess
14:41 EmilienM because puppet could try to create flavors right after keystone_service and it would fail
14:41 EmilienM since you also need keystone_user and keystone_user_role
14:41 mwhahaha no keystone_service is the endpoing
14:41 mwhahaha it's currently dying because the publicURL isn't there yet
14:42 mwhahaha anyway i'll try that next
14:42 mwhahaha brb
14:44 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs  https://review.openstack.org/310022
14:46 chem mwhahaha: for the error a Nova_flavor<||> -> Keystone_endpoint<||> would do
14:47 EmilienM chem: we might want to fix it in the provider
14:47 vinsh joined #puppet-openstack
14:47 EmilienM chem: so it works out of the box for our users.
14:49 chandankumar joined #puppet-openstack
14:50 chem EmilienM: mwhahaha if you puts this somewhere where it's loaded by an include glance, then the provider will work
14:51 chem EmilienM: mwhahaha I don't think autorequire support glob pattern in the type
14:54 chem EmilienM: mwhahaha oups,  I meant "include nova"
14:55 mwhahaha Yea i think i might need to do a Keystone_endpoint<| tag == 'nova' |> -> Nova_flavor<||>
14:56 mwhahaha i just wanted to know if there was a better way :D
14:57 cwolferh joined #puppet-openstack
14:58 mwhahaha Actually that should have already happened because the keystone endpoints for nova api should be before service nova-api
14:58 haplo37 joined #puppet-openstack
14:58 mwhahaha https://github.com/openstack/puppet-nova/b​lob/master/manifests/keystone/auth.pp#L280
14:58 mwhahaha i'll figure it out
15:00 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add oslo.messaging notitication related options  https://review.openstack.org/312583
15:02 mwhahaha that's odd it was run between the service v3 and service endpoint creation
15:03 yolanda joined #puppet-openstack
15:04 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add logging class  https://review.openstack.org/312446
15:07 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
15:09 partner feel stupid for even asking but how am i supposed to call policy changes from manifests? say as an example i want to change cinder policy: "admin_api": "is_admin:True",
15:09 partner there is some example included in every policy.pp but i have no idea where to put my stuff in that format..
15:09 EmilienM there is no stupid question, are you using Hiera or not?
15:11 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add oslo.messaging notitication related options  https://review.openstack.org/312583
15:11 EmilienM so unit tests are broken on stable/mitaka for puppet-aodh https://review.openstack.org/#/c/312527/
15:11 EmilienM need to figure that out after #rdo meeting
15:11 partner i do have various things defined on hieradata
15:12 partner i cannot find a single example of using these or at least my googling fails there big time..
15:12 mwhahaha the aodh failures look like it might be postgresql related
15:12 mwhahaha wonder if we're missing a fact or something
15:13 EmilienM mwhahaha: right, but it's only on puppet-aodh afik
15:13 EmilienM partner: if you use hiera, you can manage policies in hieradata, it's hashes
15:14 EmilienM partner: see https://github.com/openstack/puppet-cinder/bl​ob/master/spec/classes/cinder_policy_spec.rb
15:14 EmilienM https://github.com/openstack/puppet-cinder/blob/​master/spec/classes/cinder_policy_spec.rb#L9-L12
15:14 EmilienM this is an hash
15:15 partner yeah i can make my hash but i have no idea where to put it or how to refer to it to actually get the policy class to kick in :(
15:15 mwhahaha aodh was the first migrated to rspec_puppet_facts and it still has postgres, others might not
15:16 EmilienM partner: nothing hard, just include this class https://github.com/openstack/puppet-c​inder/blob/master/manifests/policy.pp
15:16 EmilienM and add your policy in hieradata, like I said, with an hash: cinder::policy::policies: <hash>
15:16 EmilienM mwhahaha: other have
15:17 yolanda joined #puppet-openstack
15:17 mwhahaha k i can take a look in a bit
15:17 EmilienM currently in meeting, will look later but feel free to look also
15:17 partner i get Duplicate declaration: Class[Glance::Policy] is already declared; (glance, cinder, they are all the same)
15:17 EmilienM maybe you don't need to declared, I haven't read all code
15:18 EmilienM partner: https://github.com/openstack/puppet-ci​nder/blob/master/manifests/api.pp#L189
15:18 * mjblack bangs head against desk
15:18 EmilienM you don't need to do anything, except adding hieradata
15:18 partner hmm
15:18 EmilienM do not include the class, it's done for you
15:18 partner o
15:19 partner i've tried probably all the wrong ways there are, just one of many errors :)
15:20 ddmitriev joined #puppet-openstack
15:23 EmilienM we fail here: https://github.com/puppetlabs/puppetlabs-pos​tgresql/blob/4.6.x/manifests/globals.pp#L118
15:23 EmilienM (maybe we can just bump postgres module in stable/mitaka)
15:24 xarses joined #puppet-openstack
15:28 EmilienM mwhahaha: I think we're hitting that: https://github.com/puppetlabs/puppetlabs-pos​tgresql/blob/4.6.x/manifests/globals.pp#L65
15:28 EmilienM omg that module is a mess
15:29 dmsimard mwhahaha: thanks for coming to the rescue of the nova flavor provider
15:29 mwhahaha so they don't support fedora? so we need to drop the fedora checking
15:29 mwhahaha wonder if that's new
15:29 EmilienM mwhahaha: or bump to recent tag
15:29 mwhahaha dmsimard: np, hopefully i can get it in a usable state this week
15:29 mwhahaha bump :D
15:29 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: CI test - never merge  https://review.openstack.org/312598
15:30 EmilienM let's see if it works on master
15:30 EmilienM if yes, we'll bump the module in stable
15:32 yolanda joined #puppet-openstack
15:36 partner i think our hieradata usage differs enough so that this approach probably won't work..
15:41 partner i guess i have no option but to search and replace the desired content by other means
15:46 mdorman joined #puppet-openstack
15:50 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate deprecated parameters wiki  https://review.openstack.org/312609
15:58 jistr joined #puppet-openstack
16:09 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate reviews wiki  https://review.openstack.org/312618
16:09 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
16:09 dmsimard mwhahaha: yay that passed
16:10 mwhahaha yay
16:16 dprince joined #puppet-openstack
16:16 dfisher joined #puppet-openstack
16:19 partner well, can't figure out how to do this with the policy.pp, fails no matter how i glue it up :(
16:20 partner EmilienM: thanks for your assistance, i'll buy you a beer on next summit
16:21 dmsimard mwhahaha: I think a good test would be to base a patch on top of that (depends-on) and replace that bit https://github.com/openstack/puppet-openstack-inte​gration/blob/master/manifests/provision.pp#L9-L23
16:21 dmsimard mwhahaha: I can submit it if you'd like
16:21 mwhahaha sure go for it
16:21 EmilienM ++ on that ^
16:21 mwhahaha i still need to improve the testing around properties (and updates)
16:22 mwhahaha i just wanted to get it working-ish :D
16:26 vinsh_ joined #puppet-openstack
16:27 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Move flavor provisioning to new nova flavor provider  https://review.openstack.org/312627
16:27 dmsimard mwhahaha, EmilienM ^
16:27 EmilienM omg this is happenning
16:35 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
16:38 iurygregory bkero, are you around?
16:42 bkero iurygregory: i am
16:44 iurygregory bkero, do you think we can have the ci doc, functional tesst doc,  unit test doc in just one doc with sections?
16:45 iurygregory https://review.openstack.org/#/c/311161/ (unit testing) i make a structure that we can add everything about test in puppet
16:45 bkero iurygregory: what would you propose to call the page? "Testing"?
16:45 iurygregory Yeah something like this sounds good
16:45 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Move flavor provisioning to new nova flavor provider  https://review.openstack.org/312627
16:46 iurygregory we can add the informations about scenarios too i think
16:46 bkero okay
16:46 iurygregory i can go ahead and add your code to my if you don't have time =)
16:46 bkero iurygregory: I'd be careful about making the page too big
16:47 chem are we supporting ruby 1.8 ?
16:47 iurygregory i think we can have one page to each and just put the tree =)
16:47 EmilienM https://docs.google.com/forms/d/1rJZxP52LyrFhF​Ty8w4J_5tnA7-A5g32YVhHSaCd7-F8/edit#responses
16:47 EmilienM people are running puppet 3.8 and ruby 1.9 a lot
16:47 bkero iurygregory: first it was one doc, now one page for each? Which is it?
16:48 chem oki, but 1.8 :)  Or is there a official ruby support ?
16:48 EmilienM not sure
16:48 chem EmilienM: oki, i see the form
16:49 chem EmilienM: that may be a good thing to specify in any case
16:49 chem EmilienM: is it too late to update the form and ask if ruby 1.8 is needed ?
16:49 iurygregory bkero, sorry for the confussion.. i think we can have "Testing" and have some topics like CI  / Rspec+ Beaker
16:50 EmilienM chem: feel free to ask on ML
16:50 iurygregory bkero,  ci will have one rst and rspec + beaker another
16:50 chem ack
16:51 iurygregory and we will have an overview in Testing that can go to CI etc..
16:52 bkero iurygregory: it sounds like you have opinions about this. I'll let you use that enthusiasm :) Do whatever you like
16:52 iurygregory bkero, ahushasuhas it happens sometimes i think
16:53 iurygregory tonight i will place every thing  o/
16:55 iurygregory EmilienM, reviews and deprecated parameters are ready for review when you have time , i know you are in a busy day o/  ( https://review.openstack.org/#/c/312618/  https://review.openstack.org/#/c/312609/ )
16:55 EmilienM cool will look
16:55 bkero iurygregory: let me know if you need me to do anything with my reviews
16:56 iurygregory bkero, i will ping you tonight ok?
16:56 bkero ok
16:56 EmilienM omg https://wiki.openstack.org/wik​i/Puppet/Deprecated_Parameters
16:56 EmilienM should we maintain that list?
16:57 dmsimard EmilienM: I can't see the responses from that form, can you share a paste or something ?
16:58 EmilienM dmsimard: it's public, no?
16:58 EmilienM chem: were you able to read it?
16:59 chem EmilienM: yep
16:59 chem EmilienM: but I'm logged in so that may not be relevent
16:59 EmilienM ah, maybe you need a google account
17:00 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs  https://review.openstack.org/310022
17:01 openstackgerrit Andy Smith proposed openstack/puppet-aodh: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/312636
17:01 dmsimard EmilienM: there's a button to request edit, sec
17:01 dmsimard there, requested edit
17:02 EmilienM why do you need edit?
17:02 EmilienM anyway
17:02 EmilienM I shared edit with everybody I don't care
17:06 crinkle chem: i don't think ruby 1.8 is needed because we don't support any operatingsystems where 1.8 is the default, the lowest is 1.9.3 on trusty
17:07 chem crinkle: ah, thanks for the insight.  Can use named regexp then \o/
17:08 EmilienM crinkle: regarding the poll, I'll probably propose we stop testing puppet < 3.6 from current master and beyong
17:08 EmilienM beyond*
17:10 crinkle EmilienM: don't we want to keep 3.4 since that's what trusty packages? or do we care
17:11 EmilienM oh right
17:11 EmilienM I'll wait a bit to see the polls
17:19 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
17:20 dfisher joined #puppet-openstack
17:25 dmsimard EmilienM: wouldn't let me see the results without edit permissions
17:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: cinder: deploy cinderv3 on ipv6/ssl  https://review.openstack.org/312652
17:38 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Move flavor provisioning to new nova flavor provider  https://review.openstack.org/312627
17:38 dmsimard mwhahaha, EmilienM: p-o-i passes with new provider.. it just failed due to lint
17:38 EmilienM w00t W00t
17:38 EmilienM mwhahaha: failing on fuel CI: https://review.openstack.org/#/c/305463/
17:38 dmsimard I will try something funny in packstack now
17:39 EmilienM mwhahaha: I +2-ed, I'll let you +A
17:39 openstackgerrit Paul Belanger proposed openstack/puppet-aodh: [DNM] Test tripleo-centos-7 jenkins slave  https://review.openstack.org/312656
17:54 openstackgerrit Matt Fischer proposed openstack/puppet-trove: Allow guestagent to use different rabbit params  https://review.openstack.org/312661
18:00 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
18:09 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump postgresql module to 4.7.1  https://review.openstack.org/312668
18:10 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: CI test with newest pgsql module  https://review.openstack.org/312669
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-openstacklib: Prepare module for publication to the forge  https://review.openstack.org/312133
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-keystone: Prepare module for publication to the forge  https://review.openstack.org/312141
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Prepare module for publication to the forge  https://review.openstack.org/312202
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-nova: Prepare module for publication to the forge  https://review.openstack.org/312204
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-glance: Prepare module for publication to the forge  https://review.openstack.org/312205
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-neutron: Prepare module for publication to the forge  https://review.openstack.org/312213
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-heat: Prepare module for publication to the forge  https://review.openstack.org/312220
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-horizon: Prepare module for publication to the forge  https://review.openstack.org/312223
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-sahara: Prepare module for publication to the forge  https://review.openstack.org/312228
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-swift: Prepare module for publication to the forge  https://review.openstack.org/312232
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-ceilometer: Prepare module for publication to the forge  https://review.openstack.org/312236
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-ironic: Prepare module for publication to the forge  https://review.openstack.org/312242
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_extras: Prepare module for publication to the forge  https://review.openstack.org/312243
18:27 mjblack _ody: arrggghghghg
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-tempest: Prepare module for publication to the forge  https://review.openstack.org/312246
18:27 openstackgerrit Cody Herriges proposed openstack/puppet-trove: Prepare module for publication to the forge  https://review.openstack.org/312252
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-manila: Prepare module for publication to the forge  https://review.openstack.org/312257
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-designate: Prepare module for publication to the forge  https://review.openstack.org/312258
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-gnocchi: Prepare module for publication to the forge  https://review.openstack.org/312264
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-aodh: Prepare module for publication to the forge  https://review.openstack.org/312266
18:28 EmilienM _ody: I found issue in https://review.openstack.org/#/c/312284/
18:28 openstackgerrit Cody Herriges proposed openstack/puppet-murano: Prepare module for publication to the forge  https://review.openstack.org/312268
18:36 mwhahaha EmilienM: fuel ci is green on https://review.openstack.org/#/c/305463/, is tripleo ci broken at the moment?
18:37 vinsh joined #puppet-openstack
18:39 EmilienM mwhahaha: should work
18:39 EmilienM but this patch is not breaking OOO CI
18:39 EmilienM mwhahaha: go ahead if you want to +A
18:39 mwhahaha k
18:41 dfisher joined #puppet-openstack
18:49 _ody mjblack: yeah sorry about the spam...
18:51 _ody EmilienM: I didn't see the failed unit tests in my pre-rspec run and the failure in CI was marked because it took 60 minutes for some reason.
18:51 _ody I'll run tests again and see what I missed.
18:57 EmilienM _ody: they might be valid
18:58 EmilienM _ody, mwhahaha: can we land https://review.openstack.org/#/c/312668/1 please ?
18:58 EmilienM it will unblock mitaka CI for aodh
19:00 vinsh_ joined #puppet-openstack
19:01 * _ody is having horrible git clone issues with git.openstack.org again
19:02 nibalizer _ody: old git client? bad ipv6 routing?
19:03 _ody nibalizer: It was me this time.  Host outbound firewall.
19:04 _ody I should probably just turn that off while I am remoted into my laptop...
19:04 nibalizer _ody: haha awesome
19:08 jmikolajczak joined #puppet-openstack
19:12 xarses EmilienM: mwhahaha we should update fuel-ci to vote on puppet-ceph now since we switched finally
19:12 mwhahaha we need a puppet-ceph job for it
19:12 mwhahaha since normal deploys don't leverage ceph
19:12 mwhahaha but yes, we should
19:12 xarses eh, I thought one of the bvt's was by default
19:13 mwhahaha the bvt yea
19:13 mwhahaha we just need a simple ceph job
19:13 mwhahaha there might be one, would need to check with qa folks
19:14 EmilienM not sure fuel ci should vote
19:14 EmilienM it's external ci for us, it provides good feedback, but not in gate.
19:15 xarses EmilienM: fuel-ci cant vote in the gate
19:15 xarses sorry, I mean to add it's vote on check
19:17 xarses mwhahaha: hrm, I thought we had one of the smoke_neutron|neutron_vlan_ha running with ceph, but neither does, this is a problem in general
19:17 mwhahaha yea it is
19:18 mwhahaha ceph changes in fuel usually go straight to breaking bvt
19:18 xarses I don't know why it was removed, it was there
19:18 * mwhahaha shakes fist
19:18 * xarses shakes his head
19:22 openstackgerrit Matt Fischer proposed openstack/puppet-trove: Allow guestagent to use different rabbit params  https://review.openstack.org/312661
19:24 Kvisle we have some problems with puppet-cinder on the mitaka branch, possibly related to https://projects.puppetlabs.com/issues/4248 ... about 50% of our puppet runs fail with "Error 400 on SERVER: Could not autoload puppet/type/cinder_type"
19:24 Kvisle have anyone else experienced this?
19:25 openstackgerrit Merged openstack/puppet-nova: Add nova flavor provider  https://review.openstack.org/305463
19:33 xarses Kvisle: Nope, A quick Google implies its either not installing the libs in the puppet master, or timesync problem
19:34 mwhahaha i think we had something similar in the vswitch module previously, https://bugs.launchpad.net/​puppet-vswitch/+bug/1403480
19:34 openstack Launchpad bug 1403480 in puppet-vswitch "Catalog compilation failed sometimes" [High,Fix committed] - Assigned to Alex Schultz (alex-schultz)
19:35 mwhahaha i think it can be fixed by adding a requires in the correct provider
19:35 mwhahaha Kvisle: if you file a bug we can track it and hopefully get it addressed
19:41 pepijn joined #puppet-openstack
19:46 rahulait joined #puppet-openstack
19:48 openstackgerrit joined #puppet-openstack
20:07 openstackgerrit Merged openstack/puppet-openstack-integration: Move flavor provisioning to new nova flavor provider  https://review.openstack.org/312627
20:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
20:14 dmk0202 joined #puppet-openstack
20:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
20:15 openstackgerrit Merged openstack/puppet-openstack-integration: Puppetfile: bump postgresql module to 4.7.1  https://review.openstack.org/312668
20:25 openstackgerrit Merged openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds  https://review.openstack.org/309542
20:25 openstackgerrit Andy Smith proposed openstack/puppet-aodh: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/312636
20:27 fawadkhaliq joined #puppet-openstack
20:28 mfisch do the puppet-trove spec tests work locally for anyone else?
20:28 mfisch getting some odd errors from bundle
20:29 EmilienM _ody: re the puppet4 issue on https://review.openstack.org/#/c/312284/
20:29 EmilienM Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Undef at /home/jenkins/workspace/gate-puppet-nova-​puppet-unit-latest-centos-7/spec/fixtures​/modules/nova/manifests/params.pp:60:17 on node centos-7-rax-ord-558242
20:29 EmilienM mfisch: logs would help
20:29 partner sorry to get back to the topic but can someone please perhaps provide a working valid hieradata hash for those policies, i fail to find one that would satisfy the validator AND puppet and both ways fails :(
20:30 mfisch EmilienM: I just blew away all the old fixtures and Gemfile.lock and will try clean
20:30 EmilienM mfisch: did you deploy policies ? partner is asking for an example
20:30 mfisch policy.json no
20:30 mfisch puppet murders the format
20:30 EmilienM I thought you did
20:30 EmilienM mfisch: it's not puppet
20:30 EmilienM mfisch: it's ruby-json
20:31 partner i think i am on to something but i cannot figure it out.. i was looking into wrong place earlier
20:31 mfisch I think I said 'yes' to you once on that question because I thought you meant api-paste
20:31 EmilienM ah ok
20:31 cdelatte joined #puppet-openstack
20:32 EmilienM _ody: I found the issue
20:34 mfisch partner: whats the issue?
20:34 EmilienM _ody: we don't set the fact operatingsystemmajrelease in the unit tests
20:34 EmilienM neither puppetversion
20:34 EmilienM but it does not seem to fail because of puppetversion
20:35 openstackgerrit Paul Belanger proposed openstack/puppet-aodh: [DNM] Test tripleo-centos-7 jenkins slave  https://review.openstack.org/312656
20:35 EmilienM mwhahaha: should we add operatingsystemmajrelease fact in lib/puppet-openstack_spec_helper/defaults.rb ?
20:35 partner mfisch: issue is i am trying to use policy.pp delivered along many components but so far failed to figure out how to actually make it happen..
20:35 mwhahaha that should get populated via rspec-puppet-facts no?
20:35 EmilienM see http://logs.openstack.org/84/312284/1/check/g​ate-puppet-nova-puppet-unit-latest-centos-7/4​8676cf/console.html#_2016-05-03_23_43_02_578
20:35 partner mfisch: EmilienM pasted this along other links earlier today which is the same example i can find from my files: https://github.com/openstack/puppet-cin​der/blob/master/manifests/policy.pp#L9
20:35 openstackgerrit Matt Fischer proposed openstack/puppet-trove: Allow guestagent to use different rabbit params  https://review.openstack.org/312661
20:36 partner however that format, while perhaps a valid hash there, does not work from yaml as is
20:36 EmilienM mwhahaha: oh in fact it might be puppetversion
20:36 mfisch partner: I did this once in YAML but this is old code that I never merged
20:36 mfisch let me get a pastebin
20:36 EmilienM nova/manifests/params.pp:60
20:36 EmilienM https://review.openstack.org/​#/c/312284/1/manifests/params.pp
20:37 mfisch partner: http://paste.openstack.org/show/496151/
20:37 degorenko EmilienM, hey :)
20:37 EmilienM yeah, puppetversion fact is not populated
20:37 partner i got some help from #puppet and at least i now got to the point where the data is actually processed already by the policy thingy
20:37 EmilienM degorenko: GO ON PTO or I kick you
20:38 EmilienM well, except if it's raining and you don't know what to do :Pp
20:38 degorenko :D oooh, common, is here something important for me?
20:38 EmilienM degorenko: need help on Ubuntu
20:38 EmilienM I sent https://review.openstack.org/#/c/308530/ again
20:38 EmilienM tests are running, but major issue is SSL and python-requests, not sure what is it
20:38 EmilienM otherwise xenial testing is almost ready (some issues with ceph though)
20:39 mjblack degorenko: take the kick...
20:39 mjblack EmilienM: make up your mind :D
20:39 degorenko mjblack, i will try to take a look
20:39 degorenko :D
20:44 partner mfisch: ok, now the format is something that puppet can eat (that is not exactly hash as every example gives)
20:45 partner looks like it did the trick, only thing is that it removed all the linefeeds from previously readable file :o
20:45 EmilienM _ody: it sounds like $::puppetversion fact returns nothing on recent version of puppet
20:45 openstackgerrit Nathan Taylor proposed openstack/puppet-nova: Adding a purge_config option for nova_config  https://review.openstack.org/312734
20:47 Kvisle mwhahaha: do you have bug posting guidelines for the puppet modules? not sure what to include that would be relevant to ease troubleshooting
20:47 fawadk joined #puppet-openstack
20:48 partner great, thank you mfisch and EmilienM, finally its working. should the example perhaps be fixed as if it assumes hiera entries why its given in such has format? maybe its just me and don't understand to convert it and get confused.. :o
20:49 partner diff here for anybody interested (not that there's anything fancy): http://paste.openstack.org/show/496152/
20:49 mfisch partner: I'll +2 if you fix the comments ;)
20:49 partner (added an image maintainer role for publicize_image rule)
20:49 mwhahaha Kvisle: I'm not sure we have any hard requirements for what needs to be in it, https://bugs.launchpad.net/​puppet-vswitch/+bug/1403480 was the bug I had seen previously for a similar error. if you can post version of puppet master and perhaps a snipet of puppet manifests that might trigger the issue that'd be helpful
20:50 openstack Launchpad bug 1403480 in puppet-vswitch "Catalog compilation failed sometimes" [High,Fix committed] - Assigned to Alex Schultz (alex-schultz)
20:51 partner mfisch: i can try, need to get familiar with the community processes
20:51 Kvisle mwhahaha: thanks, I'll see what we can cook up :)
20:51 mfisch partner: it would be a good first commit, let me know if I can help
20:52 mfisch everyone is nice here except EmilienM he rules with an iron fist
20:52 mfisch j/k EmilienM
20:52 partner :)
20:52 EmilienM lol
20:52 ntaylor1781 lol
20:52 mwhahaha -2 mfisch
20:52 * crinkle is not nice
20:52 mwhahaha DENIED
20:52 partner mfisch: yeah, back from long trip to Austin so i'm kind of full of power towards openstack
20:53 ntaylor1781 I know that feeling.
20:54 partner i ALMOST ruined my long weekend today with this stuff, was supposed to leave early (like 3pm) and its now midnight...
20:54 mfisch crinkle is nice except when she plays Battlefront then she's a stone cold killer
20:54 crinkle heh
20:55 EmilienM mfisch is nice except when he's on IRC
20:55 EmilienM (ok this one was easy)
20:56 partner i just feel so stupid when i'm stuck with this kind of stuff.. we have 5000+ lines of manifests that set up the whole cloud and yet i fail on simplest things :/
20:56 mfisch partner: did we meet in Austin?
20:56 EmilienM mfisch: ♥
20:56 mfisch mon petite ami as usual
20:56 EmilienM look how is it easy to contribute in our project
20:56 partner mfisch: probably not, i don't know anybody.. :)
20:56 EmilienM http://docs.openstack.org/developer/p​uppet-openstack-guide/contribute.html
20:57 EmilienM so much doc !
20:57 partner nice, thanks EmilienM :D
20:57 EmilienM open the link before thanking me
20:57 partner i did...
20:57 partner found this as well earlier: https://blogs.rdoproject.org/7351/wr​iting-a-new-openstack-puppet-module
20:58 EmilienM omg I wrote that one year ago
20:58 partner hard to say, my browser denies to open it up due to invalid certificate.. :)
20:58 mfisch a comment change is a great start
20:58 mfisch partner: thats just Emilien's MITM stuff you can ignore it
20:59 partner ah, i think i've heard about that MITM stuff, kind of helper of sort or something ;)
20:59 EmilienM yeah don't worry
20:59 partner i'd be happy to help, please kick me if go silent (got all i needed, kthxbye..)
21:00 ducttape_ joined #puppet-openstack
21:01 partner but not now, i will go and grab a beer and tap myself to back for "job well done"..
21:19 ntaylor1781 So am I doing something horribly wrong, or are the spec tests just failing for master puppet-keystone?
21:20 EmilienM ntaylor1781: let me test
21:20 EmilienM they work for me
21:21 ntaylor1781 Hmm weird. Mine are failing for the puppet::type::keystone_domain_co​nfig::provideropenstack#create. That is on master, with no changes made yet.
21:21 EmilienM any patch?
21:21 ntaylor1781 http://paste.openstack.org/show/496157/
21:22 ntaylor1781 Nope, just straight master
21:22 ntaylor1781 I had just pulled it down to start working on something, but decided to run tests first.
21:22 ntaylor1781 just to make sure.
21:22 EmilienM degorenko: see https://etherpad.openstack.o​rg/p/puppet-openstack-xenial
21:22 EmilienM that's our blocker to run xenial
21:22 mwhahaha oh i know that one
21:23 mwhahaha there's a flag to disable that
21:23 mwhahaha in mysql
21:23 EmilienM I love you
21:23 EmilienM sorry
21:23 mwhahaha i ran into that trying to do something else
21:23 EmilienM ah yeah?
21:23 mwhahaha i think there's a patch in master mysql
21:23 mwhahaha since xenial doesn't have that
21:23 EmilienM have you thoughts about ssl stuff?
21:24 mwhahaha cry in a corner? which ssl stuff :D
21:24 EmilienM http://logs.openstack.org/30/308530/8/expe​rimental/gate-puppet-openstack-integration​-3-scenario002-tempest-ubuntu-xenial/3f77b​3f/console.html#_2016-05-04_20_29_48_735
21:24 degorenko oh, nice, i just open irc and link - and i see that's problem potentially solved :D
21:25 mwhahaha https://github.com/puppetlab​s/puppetlabs-mysql/pull/824
21:25 mwhahaha that solves it
21:25 mwhahaha (i think)
21:25 EmilienM I remember a guy called vinsh_
21:25 EmilienM he told us on a public channel he would help on puppet-swift stuffs
21:26 mwhahaha not sure on the ssl thing, will need to figure out what exactly it's complaining about
21:26 mwhahaha do we need better certs? or add the CA?
21:26 EmilienM puppet-swift is also broken
21:26 EmilienM mwhahaha: better certs?
21:26 EmilienM it's autosign stuff
21:27 mwhahaha crap brb gotta pick up the kid
21:28 EmilienM degorenko: we have other problems don't worry
21:29 degorenko EmilienM, you think it sounds like: you can relax? :D
21:30 EmilienM Hunner: could we have a release of puppetlabs-mysql ?
21:30 EmilienM we need https://github.com/puppetlab​s/puppetlabs-mysql/pull/824 to work on xenial
21:30 degorenko EmilienM, btw problem with ssl for keystone can be because of problem with mysql
21:31 EmilienM degorenko: uh. why that?
21:31 degorenko EmilienM, it's just a guess
21:31 EmilienM I don't see how it can be related
21:32 Hunner EmilienM: I was still having trouble with the mysqlbackup.sh stuff on xenial :/
21:32 degorenko unfortunately i can't check it now :( because i don't have any xenial VMs
21:33 EmilienM degorenko: so we have 4 blockers in the etherpad
21:33 degorenko EmilienM, yeah, i see
21:33 EmilienM (maybe more will come)
21:34 EmilienM degorenko: do you have time to help on it when you're back from PTO?
21:34 degorenko ...after solving these 4 problems :D
21:34 EmilienM ntaylor1781: I ran test on my machine, all works.
21:34 degorenko EmilienM, yeah,of course!
21:34 EmilienM degorenko: thx man
21:34 degorenko after PTO i will help you
21:34 EmilienM degorenko: I'm trying to keep some time to make progress on newton, we're almost there
21:34 ntaylor1781 Cool I am actually clearing out my vendor folder, and starting fresh.
21:34 degorenko i don't hve any urgent tasks for now
21:35 EmilienM ok I think I can solve the swift issue
21:35 degorenko so i think get xenial working will be one of my tasks for next week
21:35 EmilienM degorenko: you rocks man
21:35 degorenko EmilienM, not me
21:35 degorenko EmilienM, you :)
21:35 EmilienM no you!
21:36 EmilienM degorenko: http://goo.gl/Vw9aEy
21:36 degorenko EmilienM, ok, we are :D
21:44 mwhahaha I bet for the ssl one we need to add the CA to the host or something
21:44 mwhahaha that seems to be the generic error for when python can't verify certs
21:46 degorenko i'm off for today, i will look tomorrow on all this stuff
21:46 EmilienM mwhahaha: well, it works in trusty
21:46 EmilienM degorenko: thx man, good night
21:47 mwhahaha yea maybe they changed the location python looks for the ca certs or something, just a thought.
21:47 EmilienM right, it's worth to investigate
21:47 EmilienM I pinged ubuntu people and they were unable to tell me if something changed
22:04 EmilienM Hunner: I don't think mysqlbackup.sh is required to run the module
22:04 EmilienM Hunner: is it blocking to have a release soon?
22:07 Hunner It's not required, but our tests for it don't pass on xenial so yeah it blocks a release until it's fixed :/
22:07 mwhahaha time for the yak shaving
22:08 EmilienM Hunner: did they pass for the last release?
22:09 jmikolajczak joined #puppet-openstack
22:11 Hunner EmilienM: We didn't have xenial in the last release
22:11 EmilienM i'll ping our CI to 5695cef4a66364d34ede90a08fe07e4b277ff8ba beuuurk
22:11 Hunner so yes
22:11 EmilienM Hunner: ah. sad
22:11 Hunner EmilienM: Ooo, xenial is only in our staging CI, not production
22:11 Hunner So we can still do a release
22:11 Hunner Just not officially with 16.04 added
22:11 EmilienM cool
22:11 EmilienM works for me, I just want a tag
22:14 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion  https://review.openstack.org/312768
22:15 openstackgerrit Cody Herriges proposed openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion  https://review.openstack.org/312769
22:16 openstackgerrit Cody Herriges proposed openstack/puppet-nova: Conditionally fall back to redhat service provider  https://review.openstack.org/312284
22:17 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: DNM: stop hardcoding service_provider on upstart  https://review.openstack.org/312770
22:19 _ody Any cores up for helping me get all my forge prep changes merged?
22:19 EmilienM _ody: me
22:19 EmilienM let me finish my task
22:19 EmilienM and I'll review them all
22:19 _ody EmilienM: Thanks!
22:20 mwhahaha i'll take a quick look as well
22:22 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: WIP: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
22:22 EmilienM ok this patch contains all workarounds so we can run xenial lol
22:22 mwhahaha magic
22:22 EmilienM well, I disabled a lot of things
22:23 EmilienM _ody: do you have a gerrit topic handy?
22:24 ducttape_ joined #puppet-openstack
22:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
22:29 EmilienM ok found it
22:29 EmilienM https://review.openstack.or​g/#/q/topic:prep_for_forge
22:32 EmilienM mwhahaha: why are we using 8.0.1 everywhere?
22:32 EmilienM _ody: ^
22:32 mwhahaha cause 8.0.0 is already released?
22:32 EmilienM ah ok
22:32 mwhahaha i assume that's why it's 8.0.2 because you accidently tagged an 8.0.1 for aodh
22:33 mwhahaha if i recall correctly
22:33 EmilienM yeah
22:33 EmilienM I messed up
22:33 EmilienM you should not let me doing release, seriously
22:33 mwhahaha so it's a new version bump to include the file :D
22:37 partner i no longer have any pressure, even the greatest seem to mess up.. ;)
22:38 mwhahaha we probably should have used fuel-ci: disable for those patches
22:38 EmilienM mwhahaha:  in commit message?
22:38 mwhahaha if you want to skip fuel-ci, that'll keep it from running. other wise we just backup the queue
22:38 mwhahaha yea
22:38 EmilienM I did not know that
22:38 EmilienM it's a new feature?
22:38 mwhahaha nope
22:38 EmilienM TIL
22:39 mwhahaha it'll still run the noop test but skip the deployment test
22:39 EmilienM mwhahaha: can you get https://review.openstack.org/#/c/312768/1 also please please please ?
22:39 mwhahaha we use it when we're just doing thinks like readme updates
22:40 EmilienM mwhahaha: ok cool I'll use it
22:41 EmilienM mwhahaha: it reminds me the README stuff
22:41 EmilienM tomorrow or later I'll update all README to have a simple text, that redirect to official doc
22:41 mwhahaha sounds good
22:42 mwhahaha yea for those kind of documentation or metadata.json mass updates, fuel-ci: disable :)
22:42 EmilienM save the planet!
22:46 openstackgerrit Merged openstack/puppet-designate: Prepare module for publication to the forge  https://review.openstack.org/312258
22:46 _ody EmilienM: yeah.  What mwhahaha said about version numbers.
22:46 _ody And the topic is prep_for_forge but it looks like you found it already while is was away.
22:48 _ody Guess I'll work on my IRC tomorrow...the number of mentions I am missing is getting pretty bad.
22:51 openstackgerrit Merged openstack/puppet-murano: Prepare module for publication to the forge  https://review.openstack.org/312268
22:55 openstackgerrit Merged openstack/puppet-manila: Prepare module for publication to the forge  https://review.openstack.org/312257
22:59 openstackgerrit Merged openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion  https://review.openstack.org/312768
22:59 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Conditionally fall back to redhat service provider  https://review.openstack.org/312284
23:00 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Acceptence test: rgw/civetweb with keystone  https://review.openstack.org/311188
23:00 EmilienM _ody: rebased ^ the Depends-On was useless because it only work in the other way, the TL;DR is https://github.com/openstack/puppet-openstac​k_spec_helper/blob/master/functions#L19-L21
23:10 rchurch joined #puppet-openstack
23:11 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Simplify README to redirect to the main documentation  https://review.openstack.org/312779
23:11 EmilienM crinkle, mwhahaha: do you like it? or is it not enough?
23:11 EmilienM it's a bit rough, for me it's ok but I'm not sure
23:12 mwhahaha could probably be worded a little differently
23:12 mwhahaha let me think on it
23:12 EmilienM mwhahaha: feel free to push over it
23:12 EmilienM my english sucks
23:12 mwhahaha :D
23:17 EmilienM great with my workarounds on xenial I found one more thing
23:18 EmilienM libvirt does not start
23:24 mwhahaha wonder if they added yet another virt* service
23:25 EmilienM _ody, mwhahaha: can we get https://review.openstack.org/#/c/312652/ and https://review.openstack.org/#/c/311786/ please ?
23:25 mwhahaha http://logs.openstack.org/02/312202/3/ga​te/gate-puppet-openstack-integration-3-s​cenario001-tempest-ubuntu-trusty/4230a1a​/console.html#_2016-05-04_23_12_09_227
23:25 EmilienM oh that sucks
23:26 EmilienM mtreinish: ^
23:26 EmilienM weird, on my env it works
23:27 EmilienM mwhahaha: which patch?
23:27 mwhahaha the gate for ody's stuff
23:27 mwhahaha review 312202 (you can steal it from the url)
23:28 EmilienM yea
23:28 EmilienM 0e25a90793a17728ab235d21ee282db31add3d74
23:28 mwhahaha https://review.openstack.org/#/c/312202/
23:28 EmilienM I think mtreinish broke our CI :Pp
23:28 mwhahaha boo mtreinish
23:28 EmilienM see https://review.openstack.org/#/c/311256/
23:28 * mwhahaha throws things
23:28 EmilienM wait I'm not sure
23:28 mwhahaha well all of those forge changes are going to fail
23:28 ducttape_ joined #puppet-openstack
23:29 mwhahaha the first two already did
23:30 EmilienM so our CI is broken now
23:30 EmilienM and I was going to go out
23:30 mwhahaha http://www.returnofkings.com/wp-con​tent/uploads/2013/11/mig25eject.jpg
23:31 EmilienM can I just run?
23:31 EmilienM oh I reproduced
23:31 EmilienM oh I have a fix
23:34 EmilienM maybe
23:35 EmilienM oh actually only stable is broken
23:36 EmilienM I did that in master https://github.com/openstack/puppet​-openstack-integration/commit/9c5c8​689d34df7bf5853e2cc03078228c4ec007e
23:36 EmilienM I need to partially backport it
23:37 mwhahaha k
23:40 EmilienM ok let's try
23:40 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests.sh: run testr outside .tox  https://review.openstack.org/312787
23:40 EmilienM my commit message has probably no sense
23:40 EmilienM but it's beer o'clock
23:41 mwhahaha for all the time we spend on commit messages i'm pretty sure no one reads them anyway :D
23:41 EmilienM mwhahaha: nice catch btw
23:41 EmilienM ok now I'm out, if you see me online, it's my ghost
23:42 mwhahaha later
23:45 rahulait joined #puppet-openstack
23:50 fawadkhaliq joined #puppet-openstack

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