| *** baker has quit IRC | 00:00 | |
| *** mdorman has quit IRC | 00:01 | |
| *** baker has joined #puppet-openstack | 00:04 | |
| *** ayoung has quit IRC | 00:04 | |
| *** baker has quit IRC | 00:07 | |
| EmilienM | mwhahaha: omg | 00:11 |
|---|---|---|
| EmilienM | https://review.openstack.org/#/c/284900/ | 00:11 |
| EmilienM | from 45 min to 28 min | 00:12 |
| mwhahaha | :o | 00:12 |
| EmilienM | yeah | 00:12 |
| EmilienM | I increased it early this week to have RBD jobs successful but 60 was too much | 00:13 |
| * EmilienM feels like http://goo.gl/T2Kelq | 00:13 | |
| EmilienM | or maybe http://goo.gl/7BiA0g | 00:13 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: generate testrepository.subunit all the time https://review.openstack.org/283930 | 00:16 |
| EmilienM | mwhahaha: fwiw, scenario003 is still useful, it will help to scale our tests and also have more coverage | 00:17 |
| *** baker has joined #puppet-openstack | 00:21 | |
| *** rahulait has joined #puppet-openstack | 00:27 | |
| *** rahulait has quit IRC | 00:29 | |
| *** baker has quit IRC | 00:30 | |
| *** yolanda_ has joined #puppet-openstack | 00:30 | |
| *** yolanda_ is now known as yolanda | 00:31 | |
| *** baker has joined #puppet-openstack | 00:36 | |
| *** xingchao has joined #puppet-openstack | 00:37 | |
| openstackgerrit | Merged openstack/puppet-gnocchi: Fix incorrect class params https://review.openstack.org/281566 | 00:39 |
| openstackgerrit | Merged openstack/puppet-neutron: Set v3 auth related options by default https://review.openstack.org/282309 | 00:39 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Add ability to manually install puppet modules https://review.openstack.org/282020 | 00:39 |
| openstackgerrit | Merged openstack/puppet-trove: Add kombu_reconnect_delay option https://review.openstack.org/279809 | 00:39 |
| *** baker has quit IRC | 00:40 | |
| *** xingchao has quit IRC | 00:44 | |
| *** baker has joined #puppet-openstack | 00:50 | |
| *** ayoung has joined #puppet-openstack | 00:52 | |
| *** cwolferh has quit IRC | 00:55 | |
| *** RichardRaseley has quit IRC | 00:57 | |
| *** baker has quit IRC | 00:59 | |
| *** baker has joined #puppet-openstack | 01:02 | |
| *** baker has quit IRC | 01:06 | |
| *** ducttape_ has joined #puppet-openstack | 01:08 | |
| *** saneax is now known as saneax_AFK | 01:13 | |
| *** ducttape_ has quit IRC | 01:15 | |
| *** cwolferh has joined #puppet-openstack | 01:16 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-horizon: Making root_url configurable https://review.openstack.org/283827 | 01:20 |
| *** baker has joined #puppet-openstack | 01:24 | |
| *** baker has quit IRC | 01:30 | |
| *** baker has joined #puppet-openstack | 01:36 | |
| *** baker has quit IRC | 01:44 | |
| *** baker has joined #puppet-openstack | 01:47 | |
| EmilienM | ok Ubuntu looks better | 02:04 |
| EmilienM | mwhahaha: we'll need to land https://review.openstack.org/#/c/284978/ | 02:05 |
| *** xingchao has joined #puppet-openstack | 02:07 | |
| *** xarses has quit IRC | 02:14 | |
| openstackgerrit | Dan Prince proposed openstack/puppet-keystone: Run keystone-manage bootstrap https://review.openstack.org/284876 | 02:31 |
| *** brandon has joined #puppet-openstack | 02:34 | |
| *** brandon_ has quit IRC | 02:36 | |
| *** xarses has joined #puppet-openstack | 02:36 | |
| *** brandon has quit IRC | 02:36 | |
| *** ducttape_ has joined #puppet-openstack | 02:37 | |
| *** ducttape_ has quit IRC | 02:40 | |
| mwhahaha | I'll need to look into that further because that change appears to fail the fuel CI https://ci.fuel-infra.org/view/puppet-openstack/job/puppet-openstack.fuel-library.pkgs.ubuntu.neutron_vlan_ha/148/console | 02:42 |
| mwhahaha | So we might need to land something first so we don't have a repeat of the ml2 conf problem | 02:43 |
| *** baker has quit IRC | 02:44 | |
| *** ducttape_ has joined #puppet-openstack | 02:44 | |
| *** ilbot3 has quit IRC | 02:45 | |
| *** ducttape_ has quit IRC | 02:46 | |
| *** ilbot3 has joined #puppet-openstack | 02:47 | |
| *** xingchao has quit IRC | 02:49 | |
| *** xingchao has joined #puppet-openstack | 02:51 | |
| *** ducttape_ has joined #puppet-openstack | 02:53 | |
| *** delatte has quit IRC | 02:56 | |
| *** cwolferh has quit IRC | 03:05 | |
| *** baker has joined #puppet-openstack | 03:05 | |
| *** baker has quit IRC | 03:11 | |
| *** baker has joined #puppet-openstack | 03:12 | |
| mwhahaha | actually it might have failed because i think it started prior to our merge for the ml2 conf issue, rerunning to make sure it's ok | 03:35 |
| *** gchamoul has quit IRC | 03:38 | |
| *** gchamoul has joined #puppet-openstack | 03:43 | |
| *** fedexo has joined #puppet-openstack | 03:45 | |
| *** ducttape_ has quit IRC | 03:46 | |
| *** xingchao has quit IRC | 03:50 | |
| *** markvoelker has quit IRC | 03:51 | |
| *** brandon has joined #puppet-openstack | 04:01 | |
| *** baker has quit IRC | 04:07 | |
| *** baker has joined #puppet-openstack | 04:07 | |
| *** richm has quit IRC | 04:10 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: run_tests: generate testrepository.subunit all the time https://review.openstack.org/283930 | 04:10 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 04:15 |
| *** brad[] has quit IRC | 04:17 | |
| *** fedexo has quit IRC | 04:35 | |
| *** brandon has quit IRC | 04:46 | |
| *** brandon has joined #puppet-openstack | 04:47 | |
| *** markvoelker has joined #puppet-openstack | 04:51 | |
| *** markvoelker has quit IRC | 04:56 | |
| *** ducttape_ has joined #puppet-openstack | 05:05 | |
| *** brandon has quit IRC | 05:11 | |
| *** severion has joined #puppet-openstack | 05:12 | |
| *** v1k0d3n has quit IRC | 05:14 | |
| *** saneax_AFK is now known as saneax | 05:15 | |
| mwhahaha | yea we can land that patch, should be ok | 05:29 |
| *** ducttape_ has quit IRC | 05:37 | |
| *** khappone has joined #puppet-openstack | 05:38 | |
| *** khappone_ has quit IRC | 05:39 | |
| *** xingchao has joined #puppet-openstack | 05:58 | |
| *** rlaurika has quit IRC | 05:58 | |
| *** rlaurika has joined #puppet-openstack | 05:58 | |
| *** imcsk8 has quit IRC | 06:02 | |
| *** imcsk8 has joined #puppet-openstack | 06:02 | |
| *** baker has quit IRC | 06:11 | |
| *** markvoelker has joined #puppet-openstack | 06:53 | |
| *** markvoelker has quit IRC | 06:57 | |
| *** rcernin has joined #puppet-openstack | 07:05 | |
| *** magicboiz has joined #puppet-openstack | 07:09 | |
| *** vinsh_ has joined #puppet-openstack | 07:09 | |
| *** magicboiz has left #puppet-openstack | 07:12 | |
| *** vinsh has quit IRC | 07:12 | |
| *** jtomasek has joined #puppet-openstack | 07:52 | |
| *** furlongm_ has joined #puppet-openstack | 07:52 | |
| *** furlongm has quit IRC | 07:52 | |
| *** foxhunt_away is now known as foxhunt72 | 07:59 | |
| *** jtomasek_ has joined #puppet-openstack | 07:59 | |
| openstackgerrit | Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default https://review.openstack.org/279444 | 08:21 |
| *** dgurtner has joined #puppet-openstack | 08:29 | |
| *** jpena has joined #puppet-openstack | 08:31 | |
| *** hjensas has joined #puppet-openstack | 08:41 | |
| *** jtomasek_ has quit IRC | 08:48 | |
| *** markvoelker has joined #puppet-openstack | 08:53 | |
| *** markvoelker has quit IRC | 08:57 | |
| *** rcernin has quit IRC | 09:14 | |
| *** gfidente has joined #puppet-openstack | 09:22 | |
| *** yolanda has quit IRC | 09:23 | |
| *** jistr has joined #puppet-openstack | 09:27 | |
| *** rcernin has joined #puppet-openstack | 09:28 | |
| *** brandon has joined #puppet-openstack | 09:30 | |
| *** xarses_ has joined #puppet-openstack | 09:32 | |
| *** v1k0d3n has joined #puppet-openstack | 09:32 | |
| *** severion has quit IRC | 09:35 | |
| *** xarses has quit IRC | 09:35 | |
| *** davideagle_ has joined #puppet-openstack | 09:50 | |
| *** paramite has joined #puppet-openstack | 10:00 | |
| *** derekh has joined #puppet-openstack | 10:13 | |
| *** _degorenko|afk is now known as degorenko | 10:16 | |
| *** dalees has quit IRC | 10:25 | |
| *** ducttape_ has joined #puppet-openstack | 10:34 | |
| *** davideagle_ has quit IRC | 10:34 | |
| *** dalees has joined #puppet-openstack | 10:38 | |
| openstackgerrit | Merged openstack/puppet-neutron: change default value for lock_path https://review.openstack.org/284978 | 10:43 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: tempest: reduce compute/build_interval from 60 to 10 https://review.openstack.org/284900 | 10:43 |
| *** davideagle_ has joined #puppet-openstack | 10:47 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: ubuntu/nova: configure notifications using keystone v3 https://review.openstack.org/284503 | 10:48 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs https://review.openstack.org/284771 | 10:53 |
| *** markvoelker has joined #puppet-openstack | 10:54 | |
| *** saneax is now known as saneax_AFK | 10:55 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Add support of Block Device backend for Cinder https://review.openstack.org/284834 | 10:55 |
| *** ducttape_ has quit IRC | 10:57 | |
| *** markvoelker has quit IRC | 10:58 | |
| *** leanderthal|afk is now known as leanderthal | 11:17 | |
| openstackgerrit | Merged openstack/puppet-neutron: Link linuxbridge agent to rabbitmq changes https://review.openstack.org/284842 | 11:19 |
| *** davideagle_ has quit IRC | 11:48 | |
| *** xingchao has quit IRC | 11:49 | |
| *** xingchao has joined #puppet-openstack | 11:50 | |
| *** davideagle_ has joined #puppet-openstack | 11:51 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs https://review.openstack.org/284771 | 11:54 |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb https://review.openstack.org/254548 | 11:56 |
| *** xingchao has quit IRC | 11:56 | |
| EmilienM | good morning | 12:17 |
| *** davideagle_ has quit IRC | 12:18 | |
| *** cdelatte has joined #puppet-openstack | 12:18 | |
| EmilienM | degorenko: thx a lot for the reviews | 12:26 |
| EmilienM | so we have Ubuntu almost green now, except for scenario003, with Heat | 12:26 |
| EmilienM | but I'm working on it so we can re-enable voting maybe today | 12:26 |
| *** davideagle_ has joined #puppet-openstack | 12:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend https://review.openstack.org/285273 | 12:37 |
| *** markvoelker has joined #puppet-openstack | 12:40 | |
| *** markvoelker has quit IRC | 12:44 | |
| degorenko | EmilienM, no problem :) | 12:50 |
| degorenko | EmilienM, btw, https://review.openstack.org/#/c/284771/ green :) | 12:50 |
| degorenko | i forgot update store parameter in glance_store section | 12:50 |
| EmilienM | oh nice !! | 12:51 |
| EmilienM | Excellent | 12:51 |
| EmilienM | I think we can try to add Cinder on scenario002, using iscsi backend | 12:51 |
| degorenko | EmilienM, but wait with merging, i've found related issue in glance swift backend class | 12:52 |
| EmilienM | degorenko: which one? | 12:52 |
| degorenko | EmilienM, as you see here: https://review.openstack.org/#/c/284771/4/manifests/glance.pp - i passed auth_address for swift | 12:52 |
| degorenko | because default value without http protocol, without any protocol | 12:53 |
| degorenko | and it uses https | 12:53 |
| EmilienM | https://github.com/openstack/puppet-glance/blob/master/manifests/backend/swift.pp#L44 | 12:53 |
| degorenko | do we want leave it as it now | 12:53 |
| degorenko | ? | 12:53 |
| EmilienM | we need to fix that | 12:53 |
| degorenko | ok, i will update it to http protocol then | 12:53 |
| EmilienM | or? | 12:53 |
| EmilienM | what is the default in upstream? | 12:53 |
| degorenko | or update it :) | 12:53 |
| * degorenko looking | 12:53 | |
| degorenko | EmilienM, in liberty is none | 12:54 |
| degorenko | so, i guess we can use http here by default | 12:54 |
| EmilienM | ok | 12:54 |
| EmilienM | go ahead | 12:54 |
| EmilienM | and use Depends-On to test it in the integ patch | 12:54 |
| degorenko | EmilienM, ok :) | 12:54 |
| EmilienM | degorenko: I'm happy to see jobs back to ~30 min | 12:55 |
| EmilienM | instead of 1h or 50 min | 12:55 |
| degorenko | :D yes | 12:55 |
| *** gfidente has quit IRC | 12:56 | |
| *** dprince has joined #puppet-openstack | 12:57 | |
| *** foxhunt72 is now known as foxhunt_away | 12:57 | |
| colonwq | EmilienM, The checks are going much faster today. I guess I'll see less of the outside today. | 12:58 |
| EmilienM | dprince: I'm investigating the Ubuntu thing on https://review.openstack.org/#/c/284876 - you don't have to worry about it | 13:01 |
| *** jpena is now known as jpena|lunch | 13:02 | |
| dprince | EmilienM: okay, thanks | 13:03 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-openstack-integration: Fix rspec testing outside of the Openstack CI. https://review.openstack.org/285285 | 13:03 |
| EmilienM | nice shot ^ | 13:05 |
| EmilienM | mfisch: I think chem is fixing a bug you reported in a last meeting | 13:06 |
| chem | mfisch: EmilienM, kind of though that It would be easy to provide a how-to, but it wasn't :) | 13:07 |
| EmilienM | chem: https://bugs.launchpad.net/puppet-keystone/+bug/1548872 | 13:07 |
| openstack | Launchpad bug 1548872 in puppet-keystone "bundle exec rake spec fails to find r10k - tests wont run" [High,Confirmed] | 13:07 |
| EmilienM | he already created a bug I think, is it the same? | 13:08 |
| chem | mfisch: EmilienM had to patch the stuff, preparing a mail now | 13:08 |
| chem | EmilienM: yep, oups sorry | 13:08 |
| EmilienM | chem: no worries | 13:08 |
| EmilienM | I close his bug | 13:08 |
| chem | EmilienM: mfisch I knew that my working env was /luckily/ working, but it kind of bother me for a while ... I think the patch should do | 13:09 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Use http protocol by default in auth_address for Swift backend https://review.openstack.org/285290 | 13:10 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs https://review.openstack.org/284771 | 13:11 |
| *** ducttape_ has joined #puppet-openstack | 13:13 | |
| EmilienM | degorenko: can you -A and +A again please? | 13:15 |
| EmilienM | https://review.openstack.org/#/c/284863/ | 13:15 |
| EmilienM | to trigger zuul | 13:15 |
| degorenko | done :) | 13:15 |
| *** jayg|g0n3 is now known as jayg | 13:16 | |
| EmilienM | it's not triggered | 13:16 |
| EmilienM | I need to rebase it I think | 13:16 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: heat: manage heat user role before openrc https://review.openstack.org/284863 | 13:16 |
| EmilienM | degorenko: a last time? :) | 13:16 |
| degorenko | boom :) | 13:17 |
| *** markvoelker has joined #puppet-openstack | 13:20 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: Switch creating cinder types to providers from define classes https://review.openstack.org/275669 | 13:24 |
| *** brad[] has joined #puppet-openstack | 13:25 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service https://review.openstack.org/250436 | 13:26 |
| degorenko | EmilienM, do you want to add iscsi to cinder scenario002 or i can do it? :) | 13:27 |
| chem | EmilienM: degorenko, whoua, that was fast. Thanks, hope I didn't break all CI :) | 13:28 |
| degorenko | chem, can you +1 again me here: https://review.openstack.org/273513 please ? :D | 13:29 |
| EmilienM | degorenko: go ahead | 13:29 |
| degorenko | ack :) | 13:29 |
| chem | degorenko: basically you re-introduced the original type for a deprecation period, got it right ? | 13:32 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: Add testing Cinder+ISCSI backend for scenario002 jobs https://review.openstack.org/285300 | 13:32 |
| degorenko | chem, that's right. I totally forgot to go through deprecation cycle for this defines :D | 13:32 |
| chem | degorenko: not only you :) | 13:33 |
| degorenko | thanks mwhahaha for reminder :D | 13:33 |
| chem | degorenko: in the meantime, we don't care about the current error on centos ? | 13:33 |
| degorenko | chem, well, yes, it should go away after some count of rechecks :D | 13:34 |
| degorenko | because patch works before this update :) and now it still works for trusty, so it is ci issues | 13:35 |
| *** ducttape_ has quit IRC | 13:35 | |
| chem | degorenko: looks like Disable_admin_token_auth/ is not not working ok ... anyway, completly unrelated to your patch | 13:35 |
| chem | degorenko: +1 | 13:36 |
| degorenko | thanks :) | 13:36 |
| *** johnmilton has joined #puppet-openstack | 14:00 | |
| *** yolanda has joined #puppet-openstack | 14:00 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service https://review.openstack.org/250436 | 14:03 |
| *** richm has joined #puppet-openstack | 14:03 | |
| EmilienM | degorenko: you'll need https://review.openstack.org/283786 first | 14:07 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: enable fwaas https://review.openstack.org/283920 | 14:07 |
| *** vinsh_ has quit IRC | 14:08 | |
| *** xarses_ has quit IRC | 14:08 | |
| degorenko | EmilienM, i will rebase on your, just to make sure that tests are working | 14:09 |
| EmilienM | ok | 14:09 |
| EmilienM | but running tempest plugins require some work, in upstream projects | 14:09 |
| EmilienM | it won't be easy | 14:09 |
| *** jpena|lunch is now known as jpena | 14:16 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Fix rspec testing outside of the Openstack CI. https://review.openstack.org/285285 | 14:17 |
| *** vinsh has joined #puppet-openstack | 14:17 | |
| *** karimb has joined #puppet-openstack | 14:17 | |
| *** karimb has quit IRC | 14:17 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: heat: manage heat user role before openrc https://review.openstack.org/284863 | 14:30 |
| *** davideagle_ has quit IRC | 14:36 | |
| *** hjensas has quit IRC | 14:37 | |
| *** xarses_ has joined #puppet-openstack | 14:42 | |
| *** davideagle_ has joined #puppet-openstack | 14:44 | |
| *** dfisher has joined #puppet-openstack | 14:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging https://review.openstack.org/285333 | 14:48 |
| *** foxhunt_away is now known as foxhunt72 | 14:49 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service https://review.openstack.org/250436 | 14:53 |
| *** ducttape_ has joined #puppet-openstack | 14:56 | |
| *** xarses_ has quit IRC | 15:03 | |
| *** delattec has joined #puppet-openstack | 15:04 | |
| *** xingchao has joined #puppet-openstack | 15:06 | |
| *** cdelatte has quit IRC | 15:06 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer https://review.openstack.org/285344 | 15:07 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: use stevedore names for driver / backends https://review.openstack.org/285345 | 15:07 |
| EmilienM | richm, chem: I need your eyes on ^ | 15:07 |
| EmilienM | mfisch: also | 15:07 |
| chem | EmilienM: ack, I add it to my favorite patch :) | 15:08 |
| *** dhollinger has joined #puppet-openstack | 15:08 | |
| *** vinsh has quit IRC | 15:08 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Run keystone-manage bootstrap https://review.openstack.org/284876 | 15:09 |
| *** vinsh has joined #puppet-openstack | 15:14 | |
| *** baker has joined #puppet-openstack | 15:18 | |
| *** paramite is now known as paramite|afk | 15:20 | |
| *** baker has quit IRC | 15:22 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer https://review.openstack.org/285358 | 15:26 |
| *** delatte has joined #puppet-openstack | 15:28 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer https://review.openstack.org/285358 | 15:29 |
| *** baker has joined #puppet-openstack | 15:30 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer https://review.openstack.org/285358 | 15:30 |
| *** delattec has quit IRC | 15:30 | |
| *** davideagle_ has quit IRC | 15:31 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin https://review.openstack.org/283786 | 15:33 |
| *** davideagle_ has joined #puppet-openstack | 15:33 | |
| EmilienM | degorenko: you'll need to rebase https://review.openstack.org/#/c/250436/ on top of ^ | 15:34 |
| EmilienM | but I doubt 250436 will pass easily | 15:34 |
| EmilienM | I found many bugs upstream that prevent us to run tempest plugins | 15:34 |
| *** dprince has quit IRC | 15:35 | |
| degorenko | EmilienM, for 250436 we also don't have rdo murano packages anyway :D | 15:38 |
| *** xingchao has quit IRC | 15:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging https://review.openstack.org/285333 | 15:39 |
| *** xarses_ has joined #puppet-openstack | 15:40 | |
| EmilienM | degorenko: not yet? what is blocking? | 15:40 |
| degorenko | EmilienM, i don't know :( | 15:40 |
| *** alee has quit IRC | 15:54 | |
| *** alee has joined #puppet-openstack | 15:55 | |
| *** paramite|afk is now known as paramite | 16:04 | |
| *** davideagle_ has quit IRC | 16:06 | |
| *** foxhunt72 is now known as foxhunt_away | 16:06 | |
| *** haplo37 has joined #puppet-openstack | 16:06 | |
| *** alee has quit IRC | 16:07 | |
| *** baker has quit IRC | 16:08 | |
| EmilienM | chem: mhh now when I run integ jobs in my servers, it fails to run r10k | 16:13 |
| chem | EmilienM: your server on CI openstack ? | 16:13 |
| EmilienM | no | 16:13 |
| EmilienM | *my* servers | 16:13 |
| *** logan- has quit IRC | 16:14 | |
| EmilienM | I'm digging | 16:14 |
| *** logan- has joined #puppet-openstack | 16:14 | |
| chem | EmilienM: hum you must have hardcoded the GEM_PATH_BIN somewhere | 16:14 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules.sh: run gem install with --verbose https://review.openstack.org/283696 | 16:17 |
| *** alee has joined #puppet-openstack | 16:20 | |
| *** alee has quit IRC | 16:22 | |
| *** IvanBerezovskiy has left #puppet-openstack | 16:31 | |
| *** rcernin has quit IRC | 16:31 | |
| EmilienM | degorenko: can you +A https://review.openstack.org/#/c/284031/ please? | 16:34 |
| EmilienM | ayoung: chem wrote testing for ldap, maybe you can talk each other | 16:35 |
| EmilienM | chem: ayoung is confused because about puppet-keystone/ldap support | 16:36 |
| degorenko | EmilienM, so many lines, i believe that everything fine here :) +A | 16:36 |
| degorenko | oh, cherry pick | 16:36 |
| chem | ayoung: what is the problem ? | 16:36 |
| ayoung | chem, EmilienM showed me a log with a referencetop an LDAP Credential backend | 16:37 |
| ayoung | http://logs.openstack.org/76/284876/4/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/161cc03/console.html#_2016-02-26_11_27_31_950 | 16:37 |
| ayoung | and there is no LDAP credential backend, nor has there ever been. | 16:37 |
| chem | ayoung: I think I got it | 16:38 |
| ayoung | cool | 16:39 |
| chem | ayoung: I didn't have a look at the patch yet, but I know that I could not have keystone to run, in my test. I just test that files are created and clean it up, because if I try any command it would fails as there is no actual backend | 16:40 |
| chem | ayoung: it would require installing an actual ldap server and all | 16:40 |
| chem | ayoung: which at the time was considered good to have but non urgent | 16:40 |
| ayoung | chem, it looks like a typo, maybe copied over from the Identity backend. That is the only place LDAP should ever be used. It is deprecated or removed from all of Keystone but Identityt | 16:41 |
| chem | ayoung: oki, I just looked at the code and ... it's big change :) | 16:44 |
| chem | ayoung: the Keystone_config<||> ~> Exec<| title == 'keystone-manage bootstrap'|> | 16:44 |
| chem | ayoung: line is hard | 16:45 |
| ayoung | Yeah. It should actually be a net reduction in calls from Puppet | 16:45 |
| ayoung | I'm still getting my head around the bootstrap call. Jamie Lennox caught an interesting paradox there: | 16:45 |
| ayoung | we need to set up the first endpoint, which is the keystone Identity Service/endpoint itself. Or we can't do anything | 16:46 |
| chem | ayoung: so as I said the ldap test must not restart keystone, or try to run it as they will *fail*, with this catch all I think that trigger a keystone run that fail | 16:46 |
| ayoung | chem, So this came from the first time someone tried to run the LDAP tests? | 16:46 |
| chem | ayoung: I did a quick review of the associated change https://review.openstack.org/#/c/284876/5 | 16:48 |
| chem | ayoung: I think the problem would disapear if the unconditional trigger is removed | 16:48 |
| ayoung | chem, so let me show you another approach | 16:48 |
| ayoung | https://review.openstack.org/#/c/284778/ | 16:49 |
| *** fedexo has joined #puppet-openstack | 16:49 | |
| chem | ayoung: it's friday evening :) what is the link ? | 16:50 |
| ayoung | chem, it is a call to crete the default domain on demand | 16:50 |
| ayoung | taking it off your hands | 16:50 |
| *** davideagle_ has joined #puppet-openstack | 16:51 | |
| ayoung | "V2 operations create default domain on demand To support deployment utilities that are still using v2, create a default domain for them when they first use the v2 API. " | 16:51 |
| chem | ayoung: how it is related to ldap ? | 16:51 |
| ayoung | chem, that was the issue with the DB sync; it used to create the default domain. We stopped doing that. | 16:52 |
| ayoung | I am not quite sure the rationale, I didn;'t review the change, but it happened as part of the the every-release-compact-the-migrations process | 16:52 |
| ayoung | chem, the link you sent had this comment in the review: "Per upstream Keystone Mitaka commit 7b7fea7a3fe7677981fbf9bac5121bc15601163 keystone no longer creates the default domain during the db_sync. This patch updates the keystone class so that it adds a new option to enable_bootstrap (true by default) to re-add this functionality." | 16:53 |
| chem | ayoung: I send you this as I though it was linked to the error log you sent me :) | 16:54 |
| *** davideagle_ has quit IRC | 16:55 | |
| *** angdraug has joined #puppet-openstack | 16:56 | |
| chem | ayoung: do you want to bj, I think we misunderstand each other :) | 16:56 |
| EmilienM | ayoung: chem did not do this patch, dprince did | 17:00 |
| ayoung | chem, nah...I have something I am heads down in at the moment. Is this going to hold you up release wise? | 17:00 |
| EmilienM | ayoung: what chem helped, is ldap bits in puppet-keystone | 17:00 |
| ayoung | I need to finish a pretty significant patch, that other people are depending on | 17:00 |
| ayoung | chem, so, LDAP in Puppet keystone needs to be Identity only. Ignore all other backends | 17:01 |
| ayoung | you can remove those capability from puppet-keystone | 17:01 |
| ayoung | as Keysteon will no longer support anything but Identity, and that is headed toward read-only | 17:01 |
| *** darkxploit has joined #puppet-openstack | 17:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Make sure to use GEM_BIN_DIR everwhere r10k is run https://review.openstack.org/285438 | 17:05 |
| chem | ayoung: oki. Not sure what change that imply, but for the test included in the acceptance test, as I said, there are only there to test that the files get created, not to actualy works, as it would require a ldap backend https://trello.com/c/DuNmElKz/70-add-a-ldap-server-to-keystone-acceptance-test-for-testing-ldap-backend | 17:06 |
| *** cwolferh has joined #puppet-openstack | 17:06 | |
| ayoung | chem, there is code in Devstack to set up an LDAP server that you can use if you need | 17:06 |
| chem | ayoung: EmilienM for the error at hand, I'm quite sure that adding the conditional I suggest in the review should make this error go away | 17:07 |
| ayoung | chem, but I would check the puppet module to see why it is trying to set up a Credential LDAP backend, cuz that is wrong | 17:07 |
| *** dprince has joined #puppet-openstack | 17:07 | |
| *** jpena has quit IRC | 17:08 | |
| openstackgerrit | Merged openstack/puppet-trove: Remove trove ubuntu package hack https://review.openstack.org/284031 | 17:09 |
| chem | ayoung: it's just the "more realist" example I was given when I did the acceptance test, and they just test that "file are created with the right content", not that the data are actually correct: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L266-L474 | 17:10 |
| *** michchap_ has joined #puppet-openstack | 17:11 | |
| ayoung | chem, yeah that seems like a good tradeoff, but if you fire up the server afterwards, it is going to fail | 17:12 |
| chem | ayoung: yes, I know, that's why there is this https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L458 | 17:12 |
| *** asilenkov has joined #puppet-openstack | 17:12 | |
| *** m3ntat has joined #puppet-openstack | 17:12 | |
| chem | ayoung: which try to clean everything for other tests | 17:12 |
| *** xenolog13 has joined #puppet-openstack | 17:13 | |
| *** reynir1 has joined #puppet-openstack | 17:13 | |
| chem | ayoung: the problem with the error in the patch is the everytime the config is changed keystone bootstrap is rerun (when exactly depend on puppet) | 17:13 |
| chem | ayoung: this re-run fails at that time as the conf is updated, with "dummy" ldap parameters | 17:14 |
| *** dmburmistrov_ has joined #puppet-openstack | 17:14 | |
| *** pleia2_ has joined #puppet-openstack | 17:15 | |
| chem | ayoung: the solution here, is, as I put it in my comment of the review, to conditionaly add this trigger, only when enable_bootstrap is set to true | 17:15 |
| *** cfloare_ has joined #puppet-openstack | 17:15 | |
| *** francois1 has joined #puppet-openstack | 17:15 | |
| ayoung | chem, would the "skip the restart" ever be used in real world deployments, or just there for testing? | 17:16 |
| chem | ayoung: the server is actually restarted (I'm pretty sure), just not *used* (as it will fail as there is no backend that support the given configuration). | 17:17 |
| chem | ayoung: you cannot query the keystone server during this configuration change | 17:18 |
| chem | ayoung: but it's restarted, yes. | 17:18 |
| ayoung | chem, it sounds like you have it under control. | 17:18 |
| chem | ayoung: I remember the time I spend to have this working ... | 17:19 |
| *** fedexo has quit IRC | 17:19 | |
| *** ilbot3 has quit IRC | 17:20 | |
| *** michchap has quit IRC | 17:20 | |
| *** cfloare has quit IRC | 17:20 | |
| *** iurygregory has quit IRC | 17:20 | |
| *** francois has quit IRC | 17:20 | |
| *** llua has quit IRC | 17:20 | |
| *** dmburmistrov has quit IRC | 17:20 | |
| *** bodepd has quit IRC | 17:20 | |
| *** duritong has quit IRC | 17:20 | |
| *** mentat has quit IRC | 17:20 | |
| *** xenolog has quit IRC | 17:20 | |
| *** reynir has quit IRC | 17:20 | |
| *** asilenkov_ has quit IRC | 17:20 | |
| *** pleia2 has quit IRC | 17:20 | |
| *** pma has quit IRC | 17:20 | |
| *** bapalm has quit IRC | 17:20 | |
| *** _stowa has quit IRC | 17:20 | |
| chem | ayoung: but I may have missed something, and you seems pretty in control of keystone yourself. As for the ldap stuff, if you have any idea to have it more robust don't hesitate to add comment in this card: https://trello.com/c/DuNmElKz/70-add-a-ldap-server-to-keystone-acceptance-test-for-testing-ldap-backend | 17:21 |
| *** bapalm has joined #puppet-openstack | 17:21 | |
| *** llua has joined #puppet-openstack | 17:21 | |
| ayoung | chem, let me see if I can get rodrigo to own that | 17:21 |
| *** bodepd has joined #puppet-openstack | 17:23 | |
| chem | ayoung: ack. Cool if he can :) | 17:25 |
| *** _stowa_ has joined #puppet-openstack | 17:25 | |
| *** iurygregory has joined #puppet-openstack | 17:26 | |
| *** pma has joined #puppet-openstack | 17:26 | |
| *** duritong has joined #puppet-openstack | 17:27 | |
| *** ilbot3 has joined #puppet-openstack | 17:29 | |
| *** reynir1 is now known as reynir | 17:31 | |
| *** v1k0d3n has quit IRC | 17:33 | |
| *** _stowa_ is now known as _stowa | 17:35 | |
| *** dgurtner has quit IRC | 17:39 | |
| *** dhollinger has quit IRC | 17:48 | |
| *** jistr has quit IRC | 17:49 | |
| *** davideagle_ has joined #puppet-openstack | 17:50 | |
| *** xarses_ is now known as xarses | 17:54 | |
| *** derekh has quit IRC | 18:00 | |
| *** degorenko is now known as _degorenko|afk | 18:01 | |
| *** dhollinger has joined #puppet-openstack | 18:04 | |
| *** RichardRaseley has joined #puppet-openstack | 18:11 | |
| *** dhollinger has quit IRC | 18:12 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Add testing Cinder+ISCSI backend for scenario002 jobs https://review.openstack.org/285300 | 18:14 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend https://review.openstack.org/285273 | 18:15 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: neutron: enable fwaas https://review.openstack.org/283920 | 18:18 |
| *** ducttape_ has quit IRC | 18:34 | |
| colonwq | xarses, Can you give my patch (254548) a hopefully last look over? | 18:37 |
| *** imcsk8 has quit IRC | 18:39 | |
| *** paramite is now known as paramite|afk | 18:43 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: use stevedore names when possible and cleanup ldap testing https://review.openstack.org/285345 | 18:44 |
| EmilienM | crinkle, _ody: would you mind to confirm my statement on https://review.openstack.org/#/c/284876/5/manifests/init.pp ? | 18:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging https://review.openstack.org/285333 | 18:49 |
| *** dprince has quit IRC | 18:49 | |
| EmilienM | xarses: I tested https://review.openstack.org/#/c/276141 | 18:58 |
| EmilienM | and even without the patch, restarting a node does not restart ceph | 18:58 |
| * xarses slaps forhead | 18:58 | |
| EmilienM | you have to run puppet again. | 18:58 |
| EmilienM | I have a public VM if you want to try | 18:58 |
| *** imcsk8 has joined #puppet-openstack | 18:58 | |
| EmilienM | but I just tried with & without | 18:58 |
| xarses | thats fine, it's already broken. | 18:59 |
| EmilienM | puppet-ceph has a bug, we need to fix it | 18:59 |
| xarses | we can merge this one then | 18:59 |
| xarses | do you want to fix it here, or somewhere else? | 19:00 |
| EmilienM | xarses: somewhere else | 19:00 |
| EmilienM | I like atomic commits | 19:00 |
| xarses | I'm good with that | 19:00 |
| xarses | I'll give this another once over | 19:00 |
| EmilienM | I'm reporting the bug and working on it now. | 19:01 |
| EmilienM | xarses: and mwhahaha already reported it: https://bugs.launchpad.net/puppet-ceph/+bug/1540744 | 19:01 |
| openstack | Launchpad bug 1540744 in puppet-ceph "puppet-ceph does not ensure services start on boot" [Undecided,Confirmed] | 19:01 |
| xarses | sigh, I thought that only mucked up the MON's | 19:02 |
| EmilienM | and he abandonned his patch because rgw was fixed | 19:02 |
| EmilienM | but not mon & osd | 19:02 |
| EmilienM | so we need to fix mon & osd | 19:02 |
| colonwq | EmilienM, which OS does that happen on? | 19:03 |
| EmilienM | colonwq: centos7 | 19:03 |
| colonwq | EmilienM, could it be related to the 'systemctl daemon-relaod' nastiness of old init scripts? I have not looked at the logs yet. | 19:04 |
| xarses | thanks EmilienM, I've +2+A'd it | 19:05 |
| *** xingchao has joined #puppet-openstack | 19:05 | |
| *** RichardRaseley has quit IRC | 19:05 | |
| EmilienM | colonwq: maybe. I need to dig | 19:06 |
| EmilienM | xarses: thx | 19:06 |
| *** delattec has joined #puppet-openstack | 19:06 | |
| EmilienM | xarses: I find you a bit negative on the review. What is broken exactly? | 19:07 |
| EmilienM | it's an hack I agree, but it's not related to puppet-ceph directly, is it? | 19:07 |
| *** leanderthal is now known as leanderthal|afk | 19:08 | |
| *** delatte has quit IRC | 19:08 | |
| *** delatte has joined #puppet-openstack | 19:09 | |
| *** darkxploit has quit IRC | 19:09 | |
| *** delattec has quit IRC | 19:11 | |
| xarses | sorry, that reboots are already broken | 19:12 |
| EmilienM | I'm testing it now | 19:12 |
| EmilienM | honestly, a lot of things are broken after a reboot | 19:12 |
| EmilienM | specially when running OpenStack in production, with clustering, etc | 19:13 |
| xarses | ya, we run an entire suite of tests to try and ensure it's fine | 19:13 |
| xarses | and still miss things | 19:13 |
| xarses | I just had it in my mind, that disabeling udev would break reboots for OSD's | 19:13 |
| xarses | turns out, it was already broken | 19:14 |
| EmilienM | let's fix it | 19:14 |
| *** dprince has joined #puppet-openstack | 19:14 | |
| dmsimard | mtreinish: I know I've asked you this before (probably) | 19:16 |
| dmsimard | mtreinish: but where does the subunit file need to be for it to be picked up by the health thingie ? | 19:16 |
| EmilienM | dmsimard: https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L108 | 19:16 |
| dmsimard | mtreinish: at the root of /logs ? | 19:16 |
| dmsimard | EmilienM: you're putting it in /tmp ... | 19:17 |
| dmsimard | EmilienM: I guess copy puppet logs puts it somewhere useful | 19:17 |
| dmsimard | https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh#L101 | 19:17 |
| dmsimard | ok so root of /logs | 19:17 |
| *** xarses_ has joined #puppet-openstack | 19:17 | |
| *** xarses has quit IRC | 19:18 | |
| EmilienM | yes | 19:18 |
| *** dprince has quit IRC | 19:21 | |
| *** dprince has joined #puppet-openstack | 19:21 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Run keystone-manage bootstrap https://review.openstack.org/284876 | 19:21 |
| *** yolanda has quit IRC | 19:23 | |
| *** ducttape_ has joined #puppet-openstack | 19:23 | |
| *** dhollinger has joined #puppet-openstack | 19:23 | |
| *** paramite|afk is now known as paramite | 19:25 | |
| *** paramite has quit IRC | 19:32 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: fix collectors: Ceph_Config -> Ceph_config https://review.openstack.org/285510 | 19:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: debug https://review.openstack.org/#/c/280351 https://review.openstack.org/285517 | 19:44 |
| *** leanderthal|afk is now known as leanderthal | 19:44 | |
| openstackgerrit | Trevor McCasland proposed openstack/puppet-neutron: Requires allocation pools for creation of subnet https://review.openstack.org/285518 | 19:44 |
| *** xingchao has quit IRC | 19:48 | |
| openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Update the README about the third scenario https://review.openstack.org/285523 | 19:57 |
| openstackgerrit | Merged openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer https://review.openstack.org/276141 | 20:00 |
| dmsimard | EmilienM: all three scenarios are stable on centos now right ? | 20:00 |
| EmilienM | yes | 20:00 |
| EmilienM | and 001/002 on ubuntu too | 20:00 |
| dmsimard | ok adding the third scenario to our pipeline | 20:00 |
| dmsimard | thanks | 20:00 |
| EmilienM | 003 is broken though | 20:00 |
| EmilienM | cool np | 20:00 |
| *** yolanda has joined #puppet-openstack | 20:03 | |
| mtreinish | dmsimard: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/files/logstash/jenkins-log-client.yaml#L14 is where it looks for things after a run | 20:08 |
| dmsimard | mtreinish: ty, just fixed the packstack jobs to drop it there | 20:09 |
| EmilienM | mtreinish: I'm still confused by running tempest plugins does not work for us https://review.openstack.org/#/c/283786/ | 20:13 |
| mtreinish | EmilienM: so that looks like it's caused by a bad fwaas plugin tempest-lib is being used but never installed (it's not in requirements) http://logs.openstack.org/86/283786/5/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/4568e94/console.html#_2016-02-26_18_10_30_362 | 20:15 |
| EmilienM | mtreinish: so it's in neutron_fwaas upstream, isn't? | 20:16 |
| mtreinish | yeah, I think so | 20:17 |
| EmilienM | https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/tempest_plugin/tests/scenario/test_fwaas.py#L21 | 20:17 |
| EmilienM | https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/tempest_plugin/tests/scenario/base.py#L16-L17 | 20:18 |
| EmilienM | it's a bug in neutron_fwaas | 20:18 |
| mtreinish | yeah, they don't have tempest-lib in requirements.txt or test-requirements.txt | 20:19 |
| mtreinish | so nothing is installing tempest-lib and the plugin code imports it | 20:19 |
| EmilienM | do people actually test things? | 20:20 |
| mtreinish | heh, doesn't look like it. Although to be fair this wouldn't have been caught before because tempest used to depend on tempest-lib so it would install it for them | 20:20 |
| mtreinish | but that changed this week | 20:20 |
| EmilienM | mtreinish: should I patch https://github.com/openstack/neutron-fwaas/blob/master/test-requirements.txt ? | 20:22 |
| mtreinish | EmilienM: yeah, that or add a setuptools extras target for the plugin. Either will work | 20:22 |
| mtreinish | but if it's in test requirements you'll have to make sure to install those too | 20:23 |
| *** leanderthal is now known as leanderthal|afk | 20:29 | |
| *** angdraug has quit IRC | 20:30 | |
| EmilienM | mtreinish: I did like nova does: https://review.openstack.org/285534 | 20:32 |
| mtreinish | EmilienM: ok that works. Just note when you install fwaas, you'll have to install test-requirements manually, just doing pip install neutron-fwaas won't do that | 20:33 |
| EmilienM | ok | 20:33 |
| EmilienM | mtreinish: but in the puppet CI, we use packaging | 20:33 |
| EmilienM | so even with my patch, we'll still have the bug I think, isn't? | 20:34 |
| EmilienM | we deploy tempest-lib in a venv | 20:34 |
| mtreinish | right, I think that'll be the case. Since nothing is installing tempest-lib | 20:34 |
| EmilienM | so should I have to run pip install tempest-lib before running tests? | 20:34 |
| dmsimard | EmilienM: that stems from the fact that we don't typically install test-requirements for runtime | 20:35 |
| mtreinish | EmilienM: that would fix it | 20:35 |
| dmsimard | EmilienM: i.e, we might install sphinx (from test-requirements) at buildtime only for docs but sphinx won't get installed when you install the package | 20:35 |
| EmilienM | mtreinish: why installing tempest does not install tempest lib? | 20:35 |
| mtreinish | btw, this was part of my argument that tempest plugins should be self contained pythong projects | 20:35 |
| mtreinish | because packaging and installing gets much more complicated when they share a repo | 20:36 |
| EmilienM | yeah but they still rely on tempest-lib | 20:36 |
| EmilienM | ok | 20:36 |
| EmilienM | so let's install tempest-lib system wide | 20:36 |
| dmsimard | mtreinish: so does tempest-lib become a runtime dependency then ? | 20:36 |
| mtreinish | EmilienM: because of: https://review.openstack.org/#/c/283691/ | 20:36 |
| EmilienM | dmsimard: iiuc, yes | 20:36 |
| mtreinish | EmilienM: which is part of: http://specs.openstack.org/openstack/qa-specs/specs/tempest/reintegrate-tempest-lib.html | 20:36 |
| EmilienM | mtreinish: NEED IT ! | 20:37 |
| dmsimard | then it needs to be in requirements.txt, not test-requirements.txt | 20:37 |
| dmsimard | and we can pick it up in packaging if necessary | 20:37 |
| EmilienM | mtreinish: so people should update their tests right? | 20:37 |
| dmsimard | but I sort of agree with EmilienM that tempest-lib should come bundled with tempest | 20:37 |
| mtreinish | EmilienM: they should, but there is no requirement to do it. (the old tempest-lib isn't going away) | 20:37 |
| mtreinish | dmsimard: tempest-lib and tempest are seperate packages | 20:38 |
| dmsimard | fair enough | 20:38 |
| mtreinish | tempest isn't even on pypi yet | 20:38 |
| dmsimard | neither is nova | 20:39 |
| mtreinish | (although that'll change next week when I push a release) | 20:39 |
| dmsimard | :) | 20:39 |
| mtreinish | right, but for requirements in openstack projects that's one of the minimum requirements. Nova isn't supposed to be imported so it doesn't really matter | 20:40 |
| *** yolanda has quit IRC | 20:40 | |
| dmsimard | I see | 20:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin https://review.openstack.org/283786 | 20:42 |
| *** foxhunt_away is now known as foxhunt72 | 20:46 | |
| EmilienM | mwhahaha: have you some progress on lint/centos7? | 20:49 |
| mwhahaha | no not at the moment | 20:50 |
| mwhahaha | got side tracked on a bunch of other stuff | 20:50 |
| EmilienM | mwhahaha: need help? | 20:50 |
| mwhahaha | it seems fairly straight forward but did we want to just combine the syntax/lint checks? it seems a waste to have to spin up a vm just for lint | 20:50 |
| mwhahaha | i think i had asked about that the last time linting took hours | 20:51 |
| EmilienM | mwhahaha: we have this discussion in the past, let me find the history | 20:58 |
| EmilienM | https://review.openstack.org/#/c/220527/ | 20:58 |
| EmilienM | oh and look https://review.openstack.org/#/c/209147/ | 20:59 |
| openstackgerrit | Merged openstack/puppet-neutron: Support midonet type driver for ML2 https://review.openstack.org/278207 | 21:00 |
| *** brandon has joined #puppet-openstack | 21:02 | |
| *** davideagle_ has quit IRC | 21:03 | |
| EmilienM | mwhahaha: well, looks like they won't let it happen | 21:03 |
| EmilienM | on #openstack-infra | 21:03 |
| *** dgurtner has joined #puppet-openstack | 21:05 | |
| *** dgurtner has quit IRC | 21:05 | |
| *** dgurtner has joined #puppet-openstack | 21:05 | |
| mwhahaha | yea that was my other concern as i had a feeling they wouldn't be ok with the switch since there wasn't a really good technical reason | 21:06 |
| *** brandon has quit IRC | 21:10 | |
| *** delattec has joined #puppet-openstack | 21:15 | |
| EmilienM | mwhahaha: trying something with https://review.openstack.org/285542 | 21:16 |
| *** delatte has quit IRC | 21:17 | |
| *** delattec has quit IRC | 21:20 | |
| *** delattec has joined #puppet-openstack | 21:22 | |
| *** davideagle has joined #puppet-openstack | 21:22 | |
| *** johnmilton has quit IRC | 21:24 | |
| *** foxhunt72 is now known as foxhunt_away | 21:51 | |
| *** spredzy has quit IRC | 21:56 | |
| EmilienM | mfisch: can you try out if r10k works for you now? | 21:56 |
| *** spredzy has joined #puppet-openstack | 21:57 | |
| EmilienM | chem's patch actually broke me when I deploy integration jobs on my personal servers | 21:57 |
| EmilienM | (I run all-in-one) | 21:57 |
| *** dgurtner has quit IRC | 22:12 | |
| EmilienM | crinkle: where is cacert_content value in infra-cloud? | 22:13 |
| EmilienM | I guess in system-config | 22:15 |
| crinkle | EmilienM: here http://git.openstack.org/cgit/openstack-infra/system-config/tree/hiera/common.yaml#n361 | 22:17 |
| *** davideagle has quit IRC | 22:17 | |
| EmilienM | thanks | 22:17 |
| *** dhollinger has quit IRC | 22:23 | |
| *** davideagle_ has joined #puppet-openstack | 22:23 | |
| *** brandon has joined #puppet-openstack | 22:23 | |
| *** brandon has quit IRC | 22:32 | |
| *** davideagle_ has quit IRC | 22:32 | |
| *** brandon has joined #puppet-openstack | 22:34 | |
| EmilienM | crinkle: do you store ssl_key_file_contents content in hiera too? | 22:38 |
| crinkle | EmilienM: yes but it's private | 22:39 |
| EmilienM | ok I see | 22:39 |
| EmilienM | I wanted to re-use your files :-P | 22:39 |
| EmilienM | I'll generate it on my laptop for our CI | 22:40 |
| crinkle | EmilienM: just generate new ones :P | 22:40 |
| EmilienM | I'm lazy! | 22:40 |
| EmilienM | crinkle: just by curiosity, how many times do you run puppet on a fresh infra cloud node to deploy it? | 22:52 |
| crinkle | EmilienM: it works the first time :D | 22:53 |
| EmilienM | omg | 22:54 |
| *** jtomasek has quit IRC | 22:54 | |
| crinkle | EmilienM: we don't use swift | 22:54 |
| EmilienM | crinkle: I'm wondering because you don't set any orchestration for SSL certificates | 22:54 |
| EmilienM | and what if they are installed *after* deploying nova | 22:55 |
| crinkle | EmilienM: there is orchestration i think | 22:55 |
| EmilienM | indeed | 22:56 |
| EmilienM | I see notify / require | 22:56 |
| EmilienM | my bad | 22:56 |
| *** ducttape_ has quit IRC | 23:10 | |
| EmilienM | crinkle: I see you don't manage /etc/ssl/certs/openstack_infra_ca.pem with Puppet, right? | 23:14 |
| nibalizer | EmilienM: i think that is managed | 23:14 |
| nibalizer | certainly it exists in hiera | 23:15 |
| EmilienM | nibalizer: I don't find it in .pp files | 23:15 |
| EmilienM | maybe in system-config? | 23:15 |
| nibalizer | http://git.openstack.org/cgit/openstack-infra/system-config/tree/hiera/common.yaml#n389 | 23:15 |
| crinkle | EmilienM: the infracloud::cacert class has an exec that links /usr/share/local/ca-certificates/thing.crt to /etc/ssl/certs | 23:16 |
| crinkle | i think it's an ubuntuism | 23:16 |
| EmilienM | oh.. the execs does that | 23:16 |
| EmilienM | yeah there is something similar in centos | 23:16 |
| EmilienM | hence the refreshonly to make sure it's idempotent | 23:16 |
| *** haplo37 has quit IRC | 23:24 | |
| *** angdraug has joined #puppet-openstack | 23:29 | |
| *** johnmilton has joined #puppet-openstack | 23:30 | |
| *** jayg is now known as jayg|g0n3 | 23:33 | |
| *** haplo37 has joined #puppet-openstack | 23:40 | |
| *** yolanda has joined #puppet-openstack | 23:48 | |
| *** dfisher has quit IRC | 23:48 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Fix rabbitmq ssl logic https://review.openstack.org/280462 | 23:55 |
| EmilienM | just a rebase ^ | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!