| *** markvoelker has joined #puppet-openstack | 00:14 | |
| *** xarses has quit IRC | 00:27 | |
| mwhahaha | _ody: yea it'd probably be ok, we still test the other platforms in unit tests because of https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/defaults.rb#L8-L20 | 00:46 |
|---|---|---|
| *** xarses has joined #puppet-openstack | 00:59 | |
| *** saneax_AFK is now known as saneax | 01:02 | |
| *** iurygregory_ has joined #puppet-openstack | 01:24 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** ilbot3 has joined #puppet-openstack | 01:48 | |
| openstackgerrit | Colleen Murphy proposed openstack/puppet-nova: Fix default keystone auth_plugin for nova/neutron https://review.openstack.org/312300 | 01:52 |
| *** jaosorior has joined #puppet-openstack | 02:09 | |
| *** morazi has quit IRC | 02:12 | |
| *** cdelatte has quit IRC | 02:15 | |
| *** ducttape_ has joined #puppet-openstack | 02:30 | |
| EmilienM | _ody: +1 for doc | 02:31 |
| EmilienM | _ody: right, EL7 & 14.04 are reference distros since kilo I think | 02:31 |
| EmilienM | bkero: excellent work on doc, thanks | 02:32 |
| EmilienM | bkero: failures doc aren't necessary atm, we can drop them from wiki | 02:32 |
| *** ducttape_ has quit IRC | 02:45 | |
| *** markvoelker has quit IRC | 02:52 | |
| *** ducttape_ has joined #puppet-openstack | 02:54 | |
| bkero | ok | 02:59 |
| *** ducttape_ has quit IRC | 03:00 | |
| *** saneax is now known as saneax_AFK | 03:02 | |
| *** ducttape_ has joined #puppet-openstack | 03:03 | |
| *** cwolferh has quit IRC | 03:05 | |
| *** xingchao has joined #puppet-openstack | 03:08 | |
| *** saneax_AFK is now known as saneax | 03:10 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 03:21 |
| *** chandankumar has joined #puppet-openstack | 03:23 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 03:27 |
| *** cwolferh has joined #puppet-openstack | 03:28 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 03:29 |
| *** xingchao has quit IRC | 03:29 | |
| *** ducttape_ has quit IRC | 03:46 | |
| *** jaosorior has quit IRC | 03:51 | |
| *** markvoelker has joined #puppet-openstack | 03:52 | |
| *** fedexo has joined #puppet-openstack | 03:54 | |
| *** chandankumar has quit IRC | 03:55 | |
| *** markvoelker has quit IRC | 03:57 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate unit testing wiki https://review.openstack.org/311161 | 04:06 |
| *** jaosorior has joined #puppet-openstack | 04:06 | |
| *** richm has quit IRC | 04:06 | |
| *** fedexo has quit IRC | 04:20 | |
| *** fedexo has joined #puppet-openstack | 04:31 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 04:31 |
| *** iurygregory_ has quit IRC | 04:54 | |
| *** saneax is now known as saneax_AFK | 05:01 | |
| 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:08 |
| *** saneax_AFK is now known as saneax | 05:13 | |
| *** yolanda has joined #puppet-openstack | 05:16 | |
| *** bnemec has quit IRC | 05:17 | |
| *** ayoung has quit IRC | 05:31 | |
| *** mnaser has quit IRC | 05:41 | |
| *** mnaser has joined #puppet-openstack | 05:46 | |
| *** cwolferh has quit IRC | 05:55 | |
| *** jaosorior_ has joined #puppet-openstack | 06:07 | |
| *** jaosorior has quit IRC | 06:07 | |
| *** jaosorior_ has quit IRC | 06:13 | |
| *** jaosorior_ has joined #puppet-openstack | 06:17 | |
| *** openstackgerrit has quit IRC | 06:17 | |
| *** openstackgerrit has joined #puppet-openstack | 06:18 | |
| *** jaosorior_ is now known as jaosorior | 06:20 | |
| *** jmikolajczak has joined #puppet-openstack | 06:23 | |
| *** chandankumar has joined #puppet-openstack | 06:32 | |
| *** paramite has joined #puppet-openstack | 06:52 | |
| *** markvoelker has joined #puppet-openstack | 06:54 | |
| *** markvoelker has quit IRC | 07:00 | |
| *** tesseract has joined #puppet-openstack | 07:01 | |
| *** tesseract is now known as Guest23319 | 07:01 | |
| *** foxhunt_away is now known as foxhunt72 | 07:02 | |
| *** leanderthal|afk is now known as leanderthal | 07:04 | |
| *** imcsk8 is now known as imcsk8|zZz | 07:10 | |
| *** amoralej|off is now known as amoralej | 07:15 | |
| *** yolanda has quit IRC | 07:23 | |
| *** jpena|off is now known as jpena | 07:23 | |
| *** yolanda has joined #puppet-openstack | 07:32 | |
| *** jaosorior has quit IRC | 07:37 | |
| *** jaosorior has joined #puppet-openstack | 07:38 | |
| *** karimb has joined #puppet-openstack | 07:41 | |
| *** ayoung has joined #puppet-openstack | 07:49 | |
| *** dmk0202 has joined #puppet-openstack | 07:54 | |
| *** evidex has joined #puppet-openstack | 07:56 | |
| *** shardy has joined #puppet-openstack | 07:56 | |
| *** jaosorior_ has joined #puppet-openstack | 08:04 | |
| *** jaosorior has quit IRC | 08:07 | |
| *** xarses has quit IRC | 08:11 | |
| *** jaosorior_ is now known as jaosorior | 08:11 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Default keystone manage_endpoint to false https://review.openstack.org/312373 | 08:19 |
| *** jistr has joined #puppet-openstack | 08:24 | |
| *** fedexo has quit IRC | 08:26 | |
| openstackgerrit | Merged openstack/puppet-vswitch: set dkms_ensure to False by default https://review.openstack.org/311779 | 08:28 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 08:29 |
| *** gfidente has joined #puppet-openstack | 08:33 | |
| *** gfidente has quit IRC | 08:33 | |
| *** gfidente has joined #puppet-openstack | 08:33 | |
| *** mgould has joined #puppet-openstack | 08:40 | |
| *** chem has joined #puppet-openstack | 08:41 | |
| *** xarses has joined #puppet-openstack | 08:46 | |
| *** markvoelker has joined #puppet-openstack | 08:50 | |
| *** saneax is now known as saneax_AFK | 08:54 | |
| *** markvoelker has quit IRC | 08:55 | |
| *** derekh has joined #puppet-openstack | 08:56 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: WIP: Puppet manifest to configure NTP as a composable service https://review.openstack.org/310725 | 09:19 |
| *** saneax_AFK is now known as saneax | 09:30 | |
| *** yolanda has quit IRC | 09:36 | |
| *** yolanda has joined #puppet-openstack | 09:41 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Move databases creation and sync with the role https://review.openstack.org/309450 | 09:50 |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Noop start/stop/restart action for Glance and Keystone in the roles https://review.openstack.org/310935 | 09:53 |
| *** dmk0202 has quit IRC | 09:59 | |
| *** jpena is now known as jpena|brb | 10:01 | |
| *** artem_panchenko_ has quit IRC | 10:26 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Noop start/stop/restart action for Glance and Keystone in the roles https://review.openstack.org/310935 | 10:30 |
| *** yolanda has quit IRC | 10:33 | |
| openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 10:35 |
| *** yolanda has joined #puppet-openstack | 10:38 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add logging class https://review.openstack.org/312446 | 10:51 |
| *** yottatsa has joined #puppet-openstack | 10:54 | |
| *** xb_ has joined #puppet-openstack | 11:00 | |
| *** jaosorior has quit IRC | 11:10 | |
| *** jaosorior has joined #puppet-openstack | 11:10 | |
| openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 11:16 |
| *** johnmilton has joined #puppet-openstack | 11:18 | |
| 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:24 |
| *** xb_ has quit IRC | 11:26 | |
| *** yottatsa_ has joined #puppet-openstack | 11:28 | |
| *** jpena|brb is now known as jpena | 11:29 | |
| *** yottatsa has quit IRC | 11:29 | |
| *** xb_ has joined #puppet-openstack | 11:30 | |
| *** xingchao has joined #puppet-openstack | 11:34 | |
| *** xingchao has quit IRC | 11:39 | |
| *** karimb has quit IRC | 11:40 | |
| *** ducttape_ has joined #puppet-openstack | 11:46 | |
| *** ducttape_ has quit IRC | 11:50 | |
| *** cgfbee has quit IRC | 11:50 | |
| iurygregory | bkero, i'm finishing today =) | 12:01 |
| *** cdelatte has joined #puppet-openstack | 12:02 | |
| iurygregory | bkero, i agree that we can move without the index, EmilienM what you think? | 12:02 |
| *** cgfbee has joined #puppet-openstack | 12:03 | |
| *** markvoelker has joined #puppet-openstack | 12:04 | |
| *** morazi has joined #puppet-openstack | 12:05 | |
| 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:07 |
| jaosorior | EmilienM: The index is not necessarily needed. But then the document is not accessible unless you explicitly access that specific document | 12:09 |
| 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:10 |
| 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:11 |
| 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:12 |
| *** amoralej is now known as amoralej|off | 12:13 | |
| jaosorior | index it is then | 12:13 |
| 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:15 |
| *** jayg|g0n3 is now known as jayg | 12:15 | |
| 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:16 |
| *** dprince has joined #puppet-openstack | 12:17 | |
| 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:18 |
| EmilienM | iurygregory: http://docs.openstack.org/developer/puppet-openstack-guide/community.html#ptl-duty | 12:19 |
| iurygregory | Tks man | 12:19 |
| iurygregory | i will refer to this =) | 12:19 |
| *** jpena is now known as jpena|lunch | 12:34 | |
| *** ansmith has joined #puppet-openstack | 12:38 | |
| *** trown|outtypewww is now known as trown | 12:46 | |
| *** karimb has joined #puppet-openstack | 12:54 | |
| *** richm has joined #puppet-openstack | 12:56 | |
| *** imcsk8|zZz is now known as imcsk8 | 13:00 | |
| *** astupnikov has joined #puppet-openstack | 13:03 | |
| *** astupnikov has quit IRC | 13:05 | |
| *** gfidente has quit IRC | 13:05 | |
| *** gfidente has joined #puppet-openstack | 13:06 | |
| *** gfidente has quit IRC | 13:06 | |
| *** gfidente has joined #puppet-openstack | 13:06 | |
| *** astupnikov has joined #puppet-openstack | 13:09 | |
| *** astupnikov has quit IRC | 13:13 | |
| *** astupnikov has joined #puppet-openstack | 13:17 | |
| 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:20 |
| *** astupnikov has quit IRC | 13:21 | |
| *** astupnikov has joined #puppet-openstack | 13:25 | |
| mjblack | EmilienM: with openstack_extras repo | 13:27 |
| 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:28 |
| EmilienM | that's why I'm not deploying UCA when I test xenial | 13:29 |
| mjblack | there go, thats the solution :D | 13:29 |
| *** astupnikov has quit IRC | 13:29 | |
| 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:30 |
| 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:31 |
| *** astupnikov has joined #puppet-openstack | 13:33 | |
| mjblack | just a test that I'm doing in my environment | 13:33 |
| *** astupnikov has quit IRC | 13:34 | |
| *** fwdit has joined #puppet-openstack | 13:38 | |
| *** dfisher has joined #puppet-openstack | 13:40 | |
| *** amoralej|off is now known as amoralej | 13:55 | |
| *** jpena|lunch is now known as jpena | 13:55 | |
| mwhahaha | EmilienM: when I ran the local nova beaker tests I get fork/memory failures with 1g. 2g is fine | 14:01 |
| EmilienM | mwhahaha: :( what do we run more than integration jobs? | 14:02 |
| *** jtomasek has quit IRC | 14:02 | |
| mwhahaha | i'm not sure what sizes get used on infra | 14:04 |
| *** jtomasek has joined #puppet-openstack | 14:04 | |
| *** xarses has quit IRC | 14:05 | |
| EmilienM | I don't understand why our integration jobs don't have troubles and our beaker jobs for nova have issues | 14:05 |
| 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:07 |
| 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:08 |
| 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:09 |
| mwhahaha | yay | 14:10 |
| *** chandankumar has quit IRC | 14:10 | |
| *** dhollinger has joined #puppet-openstack | 14:12 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test in stable/mitaka https://review.openstack.org/312523 | 14:12 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: CI test in stable/mitaka https://review.openstack.org/312524 | 14:13 |
| *** dmk0202 has 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:14 |
| *** jmikolajczak has quit IRC | 14:16 | |
| *** twm2016 has joined #puppet-openstack | 14:18 | |
| *** mgould has quit IRC | 14:19 | |
| *** jmikolajczak has joined #puppet-openstack | 14:19 | |
| *** bnemec has joined #puppet-openstack | 14:21 | |
| *** mgould has joined #puppet-openstack | 14:23 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 14:24 |
| *** fwdit has quit IRC | 14:25 | |
| *** jaosorior has quit IRC | 14:25 | |
| *** jaosorior has joined #puppet-openstack | 14:25 | |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs https://review.openstack.org/310022 | 14:26 |
| dfisher | EmilienM: are release tarballs for each module still in the works? | 14:28 |
| 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:29 |
| *** jaosorior has quit IRC | 14:30 | |
| *** dmk0202 has quit IRC | 14:33 | |
| *** v1k0d3n has quit IRC | 14:36 | |
| *** v1k0d3n has joined #puppet-openstack | 14:36 | |
| 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:38 |
| *** dprince has quit IRC | 14:39 | |
| mwhahaha | http://logs.openstack.org/63/305463/11/check/gate-puppet-nova-puppet-beaker-rspec-centos-7/aa2383a/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:39 |
| 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:40 |
| 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:41 |
| mwhahaha | anyway i'll try that next | 14:42 |
| mwhahaha | brb | 14:42 |
| openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs https://review.openstack.org/310022 | 14:44 |
| *** jmikolajczak has quit IRC | 14:46 | |
| chem | mwhahaha: for the error a Nova_flavor<||> -> Keystone_endpoint<||> would do | 14:46 |
| *** vinsh_ has quit IRC | 14:47 | |
| EmilienM | chem: we might want to fix it in the provider | 14:47 |
| *** vinsh has joined #puppet-openstack | 14:47 | |
| EmilienM | chem: so it works out of the box for our users. | 14:47 |
| *** chandankumar has joined #puppet-openstack | 14:49 | |
| chem | EmilienM: mwhahaha if you puts this somewhere where it's loaded by an include glance, then the provider will work | 14:50 |
| chem | EmilienM: mwhahaha I don't think autorequire support glob pattern in the type | 14:51 |
| chem | EmilienM: mwhahaha oups, I meant "include nova" | 14:54 |
| mwhahaha | Yea i think i might need to do a Keystone_endpoint<| tag == 'nova' |> -> Nova_flavor<||> | 14:55 |
| mwhahaha | i just wanted to know if there was a better way :D | 14:56 |
| *** cwolferh has joined #puppet-openstack | 14:57 | |
| mwhahaha | Actually that should have already happened because the keystone endpoints for nova api should be before service nova-api | 14:58 |
| *** haplo37 has joined #puppet-openstack | 14:58 | |
| mwhahaha | https://github.com/openstack/puppet-nova/blob/master/manifests/keystone/auth.pp#L280 | 14:58 |
| mwhahaha | i'll figure it out | 14:58 |
| *** yolanda has quit IRC | 15:00 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add oslo.messaging notitication related options https://review.openstack.org/312583 | 15:00 |
| mwhahaha | that's odd it was run between the service v3 and service endpoint creation | 15:02 |
| *** yolanda has joined #puppet-openstack | 15:03 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add logging class https://review.openstack.org/312446 | 15:04 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 15:07 |
| 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:09 |
| 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:11 |
| 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:12 |
| 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:13 |
| EmilienM | partner: see https://github.com/openstack/puppet-cinder/blob/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:14 |
| *** yolanda has quit IRC | 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:15 |
| EmilienM | partner: nothing hard, just include this class https://github.com/openstack/puppet-cinder/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:16 |
| *** yolanda has 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:17 |
| EmilienM | partner: https://github.com/openstack/puppet-cinder/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:18 |
| partner | i've tried probably all the wrong ways there are, just one of many errors :) | 15:19 |
| *** ddmitriev has joined #puppet-openstack | 15:20 | |
| EmilienM | we fail here: https://github.com/puppetlabs/puppetlabs-postgresql/blob/4.6.x/manifests/globals.pp#L118 | 15:23 |
| EmilienM | (maybe we can just bump postgres module in stable/mitaka) | 15:23 |
| *** xarses has joined #puppet-openstack | 15:24 | |
| EmilienM | mwhahaha: I think we're hitting that: https://github.com/puppetlabs/puppetlabs-postgresql/blob/4.6.x/manifests/globals.pp#L65 | 15:28 |
| EmilienM | omg that module is a mess | 15:28 |
| 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 |
| *** yolanda has quit IRC | 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:30 |
| *** yolanda has joined #puppet-openstack | 15:32 | |
| *** foxhunt72 is now known as foxhunt_away | 15:33 | |
| partner | i think our hieradata usage differs enough so that this approach probably won't work.. | 15:36 |
| *** jistr has quit IRC | 15:37 | |
| partner | i guess i have no option but to search and replace the desired content by other means | 15:41 |
| *** ddmitriev has quit IRC | 15:44 | |
| *** mdorman has joined #puppet-openstack | 15:46 | |
| *** Guest23319 has quit IRC | 15:46 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate deprecated parameters wiki https://review.openstack.org/312609 | 15:50 |
| *** jistr has joined #puppet-openstack | 15:58 | |
| *** saneax is now known as saneax_AFK | 16:01 | |
| 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:09 |
| mwhahaha | yay | 16:10 |
| *** karimb has quit IRC | 16:13 | |
| *** dfisher has quit IRC | 16:16 | |
| *** dprince has joined #puppet-openstack | 16:16 | |
| *** dfisher has joined #puppet-openstack | 16:16 | |
| *** dtantsur is now known as dtantsur|afk | 16:17 | |
| partner | well, can't figure out how to do this with the policy.pp, fails no matter how i glue it up :( | 16:19 |
| partner | EmilienM: thanks for your assistance, i'll buy you a beer on next summit | 16:20 |
| 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-integration/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:21 |
| mwhahaha | i just wanted to get it working-ish :D | 16:22 |
| *** vinsh has quit IRC | 16:26 | |
| *** vinsh_ has joined #puppet-openstack | 16:26 | |
| 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:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 16:35 |
| iurygregory | bkero, are you around? | 16:38 |
| bkero | iurygregory: i am | 16:42 |
| iurygregory | bkero, do you think we can have the ci doc, functional tesst doc, unit test doc in just one doc with sections? | 16:44 |
| 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:45 |
| 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:46 |
| 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/1rJZxP52LyrFhFTy8w4J_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:47 |
| 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:48 |
| 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:49 |
| 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:50 |
| iurygregory | and we will have an overview in Testing that can go to CI etc.. | 16:51 |
| 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:52 |
| iurygregory | tonight i will place every thing o/ | 16:53 |
| *** jistr has quit IRC | 16:54 | |
| 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:55 |
| iurygregory | bkero, i will ping you tonight ok? | 16:56 |
| bkero | ok | 16:56 |
| EmilienM | omg https://wiki.openstack.org/wiki/Puppet/Deprecated_Parameters | 16:56 |
| EmilienM | should we maintain that list? | 16:56 |
| *** trown is now known as trown|lunch | 16:57 | |
| dmsimard | EmilienM: I can't see the responses from that form, can you share a paste or something ? | 16:57 |
| *** derekh has quit IRC | 16:57 | |
| EmilienM | dmsimard: it's public, no? | 16:58 |
| EmilienM | chem: were you able to read it? | 16:58 |
| 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:00 |
| 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:01 |
| EmilienM | why do you need edit? | 17:02 |
| EmilienM | anyway | 17:02 |
| EmilienM | I shared edit with everybody I don't care | 17:02 |
| 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:06 |
| chem | crinkle: ah, thanks for the insight. Can use named regexp then \o/ | 17:07 |
| 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:09 |
| crinkle | EmilienM: don't we want to keep 3.4 since that's what trusty packages? or do we care | 17:10 |
| EmilienM | oh right | 17:11 |
| EmilienM | I'll wait a bit to see the polls | 17:11 |
| *** dfisher has quit IRC | 17:17 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 17:19 |
| *** dfisher has joined #puppet-openstack | 17:20 | |
| *** paramite is now known as paramite|afk | 17:22 | |
| *** twm2016 has quit IRC | 17:24 | |
| dmsimard | EmilienM: wouldn't let me see the results without edit permissions | 17:25 |
| *** jpena is now known as jpena|off | 17:28 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 17:29 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: cinder: deploy cinderv3 on ipv6/ssl https://review.openstack.org/312652 | 17:33 |
| 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:38 |
| 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:39 |
| *** dfisher has quit IRC | 17:42 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-trove: Allow guestagent to use different rabbit params https://review.openstack.org/312661 | 17:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 18:00 |
| *** mgould has quit IRC | 18:02 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: bump postgresql module to 4.7.1 https://review.openstack.org/312668 | 18:09 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: CI test with newest pgsql module https://review.openstack.org/312669 | 18:10 |
| *** paramite|afk is now known as paramite | 18:14 | |
| *** chandankumar has quit IRC | 18:25 | |
| *** yolanda has quit IRC | 18:25 | |
| *** gfidente has quit IRC | 18:26 | |
| 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:27 |
| 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:28 |
| *** trown|lunch is now known as trown | 18:31 | |
| *** shardy has quit IRC | 18:34 | |
| mwhahaha | EmilienM: fuel ci is green on https://review.openstack.org/#/c/305463/, is tripleo ci broken at the moment? | 18:36 |
| *** vinsh_ has quit IRC | 18:36 | |
| *** vinsh has joined #puppet-openstack | 18:37 | |
| EmilienM | mwhahaha: should work | 18:39 |
| *** amoralej is now known as amoralej|off | 18:39 | |
| EmilienM | but this patch is not breaking OOO CI | 18:39 |
| *** yottatsa_ has quit IRC | 18:39 | |
| EmilienM | mwhahaha: go ahead if you want to +A | 18:39 |
| mwhahaha | k | 18:39 |
| *** dfisher has joined #puppet-openstack | 18:41 | |
| *** evidex has quit IRC | 18:47 | |
| _ody | mjblack: yeah sorry about the spam... | 18:49 |
| _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:51 |
| EmilienM | _ody: they might be valid | 18:57 |
| EmilienM | _ody, mwhahaha: can we land https://review.openstack.org/#/c/312668/1 please ? | 18:58 |
| EmilienM | it will unblock mitaka CI for aodh | 18:58 |
| *** vinsh_ has joined #puppet-openstack | 19:00 | |
| * _ody is having horrible git clone issues with git.openstack.org again | 19:01 | |
| *** xb_ has quit IRC | 19:01 | |
| nibalizer | _ody: old git client? bad ipv6 routing? | 19:02 |
| _ody | nibalizer: It was me this time. Host outbound firewall. | 19:03 |
| *** vinsh has quit IRC | 19:04 | |
| _ody | I should probably just turn that off while I am remoted into my laptop... | 19:04 |
| nibalizer | _ody: haha awesome | 19:04 |
| *** jmikolajczak has joined #puppet-openstack | 19:08 | |
| *** dhollinger has quit IRC | 19:10 | |
| 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:12 |
| 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:13 |
| 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:14 |
| xarses | EmilienM: fuel-ci cant vote in the gate | 19:15 |
| xarses | sorry, I mean to add it's vote on check | 19:15 |
| 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:17 |
| 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:18 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-trove: Allow guestagent to use different rabbit params https://review.openstack.org/312661 | 19:22 |
| 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:24 |
| openstackgerrit | Merged openstack/puppet-nova: Add nova flavor provider https://review.openstack.org/305463 | 19:25 |
| xarses | Kvisle: Nope, A quick Google implies its either not installing the libs in the puppet master, or timesync problem | 19:33 |
| 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:34 |
| *** paramite has quit IRC | 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:35 |
| *** pepijn has joined #puppet-openstack | 19:41 | |
| *** rahulait has joined #puppet-openstack | 19:46 | |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** openstackgerrit has joined #puppet-openstack | 19:48 | |
| *** dprince has quit IRC | 19:58 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Move flavor provisioning to new nova flavor provider https://review.openstack.org/312627 | 20:07 |
| *** cdelatte has quit IRC | 20:12 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments https://review.openstack.org/308530 | 20:13 |
| *** dmk0202 has 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:14 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Puppetfile: bump postgresql module to 4.7.1 https://review.openstack.org/312668 | 20:15 |
| *** foxhunt_away is now known as foxhunt72 | 20:23 | |
| *** jayg is now known as jayg|g0n3 | 20:24 | |
| 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:25 |
| *** fawadkhaliq has joined #puppet-openstack | 20:27 | |
| mfisch | do the puppet-trove spec tests work locally for anyone else? | 20:28 |
| mfisch | getting some odd errors from bundle | 20:28 |
| 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:29 |
| 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:30 |
| 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 has joined #puppet-openstack | 20:31 | |
| EmilienM | _ody: I found the issue | 20:32 |
| *** ansmith has quit IRC | 20:33 | |
| 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:34 |
| 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/gate-puppet-nova-puppet-unit-latest-centos-7/48676cf/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-cinder/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:35 |
| 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:36 |
| 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:37 |
| 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:38 |
| 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 |
| *** morazi has quit IRC | 20:39 | |
| degorenko | :D | 20:39 |
| partner | mfisch: ok, now the format is something that puppet can eat (that is not exactly hash as every example gives) | 20:44 |
| 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:45 |
| 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 has joined #puppet-openstack | 20:47 | |
| 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:48 |
| 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:49 |
| openstack | Launchpad bug 1403480 in puppet-vswitch "Catalog compilation failed sometimes" [High,Fix committed] - Assigned to Alex Schultz (alex-schultz) | 20:50 |
| *** fawadkhaliq has quit IRC | 20:50 | |
| 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:51 |
| 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:52 |
| ntaylor1781 | I know that feeling. | 20:53 |
| 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:54 |
| EmilienM | mfisch is nice except when he's on IRC | 20:55 |
| EmilienM | (ok this one was easy) | 20:55 |
| 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/puppet-openstack-guide/contribute.html | 20:56 |
| EmilienM | so much doc ! | 20:57 |
| *** jmikolajczak has quit IRC | 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/writing-a-new-openstack-puppet-module | 20:57 |
| 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:58 |
| 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..) | 20:59 |
| *** ducttape_ has joined #puppet-openstack | 21:00 | |
| *** richm has quit IRC | 21:00 | |
| partner | but not now, i will go and grab a beer and tap myself to back for "job well done".. | 21:01 |
| *** EmilienM sets mode: +o degorenko | 21:05 | |
| *** foxhunt72 is now known as foxhunt_away | 21:10 | |
| *** haplo37 has quit IRC | 21:12 | |
| *** mdorman has quit IRC | 21:17 | |
| *** trown is now known as trown|outtypewww | 21:17 | |
| ntaylor1781 | So am I doing something horribly wrong, or are the spec tests just failing for master puppet-keystone? | 21:19 |
| EmilienM | ntaylor1781: let me test | 21:20 |
| EmilienM | they work for me | 21:20 |
| ntaylor1781 | Hmm weird. Mine are failing for the puppet::type::keystone_domain_config::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:21 |
| 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.org/p/puppet-openstack-xenial | 21:22 |
| EmilienM | that's our blocker to run xenial | 21:22 |
| mwhahaha | oh i know that one | 21:22 |
| 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:23 |
| mwhahaha | cry in a corner? which ssl stuff :D | 21:24 |
| EmilienM | http://logs.openstack.org/30/308530/8/experimental/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/3f77b3f/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:24 |
| mwhahaha | https://github.com/puppetlabs/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:25 |
| 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:26 |
| mwhahaha | crap brb gotta pick up the kid | 21:27 |
| EmilienM | degorenko: we have other problems don't worry | 21:28 |
| degorenko | EmilienM, you think it sounds like: you can relax? :D | 21:29 |
| EmilienM | Hunner: could we have a release of puppetlabs-mysql ? | 21:30 |
| EmilienM | we need https://github.com/puppetlabs/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:30 |
| 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:31 |
| 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:32 |
| 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:33 |
| 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:34 |
| 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:35 |
| EmilienM | degorenko: http://goo.gl/Vw9aEy | 21:36 |
| degorenko | EmilienM, ok, we are :D | 21:36 |
| *** ducttape_ has quit IRC | 21:42 | |
| 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:44 |
| 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:46 |
| *** degorenko is now known as _degorenko|afk | 21:46 | |
| 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 | 21:47 |
| *** dmk0202 has quit IRC | 22:01 | |
| 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:04 |
| 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:07 |
| EmilienM | Hunner: did they pass for the last release? | 22:08 |
| *** jmikolajczak has joined #puppet-openstack | 22:09 | |
| 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:11 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion https://review.openstack.org/312768 | 22:14 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion https://review.openstack.org/312769 | 22:15 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-nova: Conditionally fall back to redhat service provider https://review.openstack.org/312284 | 22:16 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: DNM: stop hardcoding service_provider on upstart https://review.openstack.org/312770 | 22:17 |
| *** johnmilton has quit IRC | 22:18 | |
| _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:19 |
| mwhahaha | i'll take a quick look as well | 22:20 |
| 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:22 |
| EmilienM | _ody: do you have a gerrit topic handy? | 22:23 |
| *** ducttape_ has joined #puppet-openstack | 22:24 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 22:28 |
| EmilienM | ok found it | 22:29 |
| EmilienM | https://review.openstack.org/#/q/topic:prep_for_forge | 22:29 |
| *** pepijn has quit IRC | 22:30 | |
| 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:32 |
| 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:33 |
| partner | i no longer have any pressure, even the greatest seem to mess up.. ;) | 22:37 |
| 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:38 |
| 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:39 |
| EmilienM | mwhahaha: ok cool I'll use it | 22:40 |
| 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:41 |
| mwhahaha | yea for those kind of documentation or metadata.json mass updates, fuel-ci: disable :) | 22:42 |
| EmilienM | save the planet! | 22:42 |
| 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:46 |
| *** agrebennikov_ has quit IRC | 22:47 | |
| *** jmikolajczak has quit IRC | 22:47 | |
| _ody | Guess I'll work on my IRC tomorrow...the number of mentions I am missing is getting pretty bad. | 22:48 |
| openstackgerrit | Merged openstack/puppet-murano: Prepare module for publication to the forge https://review.openstack.org/312268 | 22:51 |
| openstackgerrit | Merged openstack/puppet-manila: Prepare module for publication to the forge https://review.openstack.org/312257 | 22:55 |
| *** dfisher has quit IRC | 22:56 | |
| 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 | 22:59 |
| *** ducttape_ has quit IRC | 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-openstack_spec_helper/blob/master/functions#L19-L21 | 23:00 |
| *** rchurch has quit IRC | 23:09 | |
| *** rchurch has joined #puppet-openstack | 23:10 | |
| 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:11 |
| 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:12 |
| EmilienM | great with my workarounds on xenial I found one more thing | 23:17 |
| EmilienM | libvirt does not start | 23:18 |
| *** xarses has quit IRC | 23:22 | |
| mwhahaha | wonder if they added yet another virt* service | 23:24 |
| 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/gate/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/4230a1a/console.html#_2016-05-04_23_12_09_227 | 23:25 |
| EmilienM | oh that sucks | 23:25 |
| EmilienM | mtreinish: ^ | 23:26 |
| EmilienM | weird, on my env it works | 23:26 |
| 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:27 |
| 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_ has joined #puppet-openstack | 23:29 | |
| mwhahaha | the first two already did | 23:29 |
| EmilienM | so our CI is broken now | 23:30 |
| EmilienM | and I was going to go out | 23:30 |
| *** bapalm_ has quit IRC | 23:30 | |
| mwhahaha | http://www.returnofkings.com/wp-content/uploads/2013/11/mig25eject.jpg | 23:30 |
| EmilienM | can I just run? | 23:31 |
| EmilienM | oh I reproduced | 23:31 |
| EmilienM | oh I have a fix | 23:31 |
| EmilienM | maybe | 23:34 |
| *** rahulait has quit IRC | 23:34 | |
| EmilienM | oh actually only stable is broken | 23:35 |
| EmilienM | I did that in master https://github.com/openstack/puppet-openstack-integration/commit/9c5c8689d34df7bf5853e2cc03078228c4ec007e | 23:36 |
| EmilienM | I need to partially backport it | 23:36 |
| mwhahaha | k | 23:37 |
| 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:40 |
| 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:42 |
| *** rahulait has joined #puppet-openstack | 23:45 | |
| *** fawadkhaliq has joined #puppet-openstack | 23:50 | |
| *** fawadk has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
