| *** imcsk8 has quit IRC | 00:03 | |
| *** imcsk8 has joined #puppet-openstack | 00:03 | |
| *** ducttape_ has quit IRC | 00:04 | |
| *** RichardRaseley has quit IRC | 00:04 | |
| social | EmilienM: haven't seen this, you don't happen to have keystone logs and debug output of puppet so we could see the error from command | 00:33 |
|---|---|---|
| *** xingchao has quit IRC | 00:33 | |
| *** ducttape_ has joined #puppet-openstack | 00:44 | |
| *** ducttape_ has quit IRC | 00:44 | |
| EmilienM | social: yeah, I should debug using logs, I was just wondering if that rang a bell to you :) | 00:59 |
| *** richm has quit IRC | 01:12 | |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-glance: Use OpenstackClient for glance_image auth https://review.openstack.org/172580 | 01:25 |
| *** xingchao has joined #puppet-openstack | 01:34 | |
| *** richm has joined #puppet-openstack | 01:39 | |
| *** xingchao has quit IRC | 01:39 | |
| EmilienM | crinkle, richm: I've re-read https://review.openstack.org/#/c/150108 and it looks good to me, +2 | 01:41 |
| EmilienM | congrats richm for fixing the bug in the blueprint \o/ | 01:41 |
| richm | serves me right for trying to be too fancy | 01:43 |
| EmilienM | ahah | 01:44 |
| *** stevemar has quit IRC | 01:53 | |
| *** gildub has quit IRC | 02:01 | |
| *** richm has quit IRC | 02:14 | |
| *** gildub has joined #puppet-openstack | 02:30 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-nova: move setting of novncproxy_base_url https://review.openstack.org/190464 | 02:31 |
| EmilienM | larsks: I backported your patch to Juno, iirc the CVE has been backported | 02:31 |
| *** stevemar has joined #puppet-openstack | 02:41 | |
| openstackgerrit | Merged stackforge/puppet-cinder: Don't add a new line if the rbd_user changes https://review.openstack.org/187448 | 02:44 |
| openstackgerrit | Matt Fischer proposed stackforge/puppet-keystone: Add support for Fernet Tokens https://review.openstack.org/185441 | 02:44 |
| openstackgerrit | Merged stackforge/puppet-swift: mount.pp: fix lint issue https://review.openstack.org/188952 | 02:52 |
| openstackgerrit | Merged stackforge/puppet-swift: Fix concat file mode https://review.openstack.org/188951 | 02:53 |
| openstackgerrit | Merged stackforge/puppet-swift: Handle both string and array for memcache param. https://review.openstack.org/188950 | 02:54 |
| openstackgerrit | Merged stackforge/puppet-glance: Fix is_public munge https://review.openstack.org/187318 | 02:56 |
| openstackgerrit | Merged stackforge/puppet-swift: Remove unused fragment_title variable https://review.openstack.org/188948 | 02:56 |
| openstackgerrit | Merged stackforge/puppet-swift: Add seed parameter to ringbuilder::rebalance. https://review.openstack.org/188942 | 02:57 |
| openstackgerrit | Merged stackforge/puppet-swift: Correct proxy::authtoken docs https://review.openstack.org/188941 | 02:57 |
| *** stevemar has quit IRC | 02:59 | |
| openstackgerrit | Merged stackforge/puppet-swift: read_affinity requires affinity sorting_method https://review.openstack.org/188949 | 03:00 |
| openstackgerrit | Merged stackforge/puppet-swift: doc spelling corrections https://review.openstack.org/188945 | 03:00 |
| EmilienM | boom | 03:01 |
| openstackgerrit | Merged stackforge/puppet-horizon: Add support for the configuration of OPENSTACK_CINDER_FEATURES https://review.openstack.org/188970 | 03:01 |
| openstackgerrit | Merged stackforge/puppet-swift: Add base `swift` class name to call https://review.openstack.org/188944 | 03:01 |
| openstackgerrit | Merged stackforge/puppet-swift: Swift proxy won't start if using proxy:ceilometer https://review.openstack.org/188943 | 03:01 |
| openstackgerrit | Merged stackforge/puppet-keystone: Add native types for keystone paste configuration https://review.openstack.org/187412 | 03:12 |
| mfisch | EmilienM: when beaker runs do you always get the latest os packages? | 03:14 |
| EmilienM | yeah | 03:14 |
| EmilienM | Kilo | 03:14 |
| EmilienM | from UCA & RDO | 03:14 |
| mfisch | what about cent packages? | 03:14 |
| mfisch | dailies? | 03:14 |
| mfisch | ok | 03:14 |
| EmilienM | not dailies | 03:14 |
| EmilienM | they are stable | 03:14 |
| EmilienM | except RDO which is not GA yet | 03:14 |
| EmilienM | but afik stable though | 03:15 |
| mfisch | I was wondering if the keystone packages caused that beaker issue with your keystone patch | 03:15 |
| EmilienM | mfisch: the centos issue with pki? | 03:44 |
| mfisch | yeah | 03:44 |
| EmilienM | no idea yet | 03:44 |
| EmilienM | I've not debugged | 03:44 |
| *** cwolferh_ has joined #puppet-openstack | 04:09 | |
| mfisch | EmilienM: do you know whats going on here about the concat? | 04:10 |
| mfisch | http://logs.openstack.org/51/187451/1/check/gate-puppet-designate-puppet-unit-3.4/814b9d1/console.html.gz | 04:10 |
| EmilienM | mfisch: no, only designate is affected? | 04:11 |
| mfisch | yeah so far | 04:11 |
| mfisch | its odd | 04:11 |
| *** cwolferh has quit IRC | 04:11 | |
| EmilienM | mhh | 04:13 |
| EmilienM | mfisch: I went thru *all* bugs | 04:23 |
| EmilienM | mfisch: and did triage for missing ones | 04:23 |
| EmilienM | I might have miss some | 04:23 |
| *** markvoelker has quit IRC | 04:26 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-neutron: Drop OVS & LB monolitic plugins https://review.openstack.org/190395 | 04:29 |
| mfisch | EmilienM: thanks | 04:35 |
| mfisch | I did some more cleanup too | 04:35 |
| EmilienM | the next step will be release management | 04:38 |
| *** tshtilma has joined #puppet-openstack | 04:47 | |
| *** [1]tshtilma has joined #puppet-openstack | 04:53 | |
| *** tshtilma has quit IRC | 04:55 | |
| *** [1]tshtilma is now known as tshtilma | 04:55 | |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-nova: Enable nova service by default https://review.openstack.org/184656 | 05:11 |
| *** xingchao has joined #puppet-openstack | 05:15 | |
| *** xingchao has quit IRC | 05:22 | |
| *** markvoelker has joined #puppet-openstack | 05:27 | |
| *** dgurtner has joined #puppet-openstack | 05:30 | |
| *** markvoelker has quit IRC | 05:32 | |
| *** rushiagr_away is now known as rushiagr | 05:41 | |
| *** stevemar has joined #puppet-openstack | 05:42 | |
| *** tshtilma has quit IRC | 06:03 | |
| *** stevemar2 has joined #puppet-openstack | 06:04 | |
| *** stevemar has quit IRC | 06:06 | |
| *** jpena has joined #puppet-openstack | 06:12 | |
| *** jtomasek has joined #puppet-openstack | 06:26 | |
| *** markvoelker has joined #puppet-openstack | 06:28 | |
| *** dgurtner has quit IRC | 06:29 | |
| *** markvoelker has quit IRC | 06:32 | |
| *** tfz has quit IRC | 06:43 | |
| *** rushiagr is now known as rushiagr_away | 06:50 | |
| *** alex_bh has joined #puppet-openstack | 07:27 | |
| gildub | I've seen some files with headers containing fields as copyright, author and Licence. I would like to know what's the convention? Because I see it inconsistently and I'm assuming we don't need it at all because this is covered by the whole project as an OSS community | 07:27 |
| gildub | EmilienM, ^ | 07:27 |
| *** tfz has joined #puppet-openstack | 07:28 | |
| *** imcsk8 has quit IRC | 07:30 | |
| *** jistr has joined #puppet-openstack | 07:33 | |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: support for keystone v3 api - v3 domain utility code https://review.openstack.org/178385 | 07:40 |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: support for keystone v3 api - the domain resource https://review.openstack.org/174439 | 07:40 |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: support for keystone v3 api - keystone_role resource https://review.openstack.org/189764 | 07:42 |
| *** arnaud_orange has joined #puppet-openstack | 07:44 | |
| *** arnaud_orange1 has joined #puppet-openstack | 07:45 | |
| *** arnaud_orange has quit IRC | 07:48 | |
| *** imcsk8 has joined #puppet-openstack | 07:48 | |
| *** michchap_ has joined #puppet-openstack | 07:56 | |
| *** stevemar2 has quit IRC | 08:09 | |
| *** xingchao has joined #puppet-openstack | 08:11 | |
| openstackgerrit | Gilles Dubreuil proposed stackforge/puppet-keystone: support for keystone v3 api - the project resource https://review.openstack.org/174468 | 08:12 |
| *** markvoelker has joined #puppet-openstack | 08:16 | |
| *** xingchao has quit IRC | 08:20 | |
| *** jpena is now known as jpena|mtg | 08:21 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** jistr has quit IRC | 08:27 | |
| *** michchap_ has quit IRC | 08:28 | |
| *** cdent has joined #puppet-openstack | 08:40 | |
| gildub | crinkle, EmilienM, please review https://review.openstack.org/#/c/187654 - Functional tests don't pass but it seems not related to patch | 08:43 |
| *** jistr has joined #puppet-openstack | 08:43 | |
| gildub | crinkle, EmilienM, the patch has no impact it only adds CredentialsV3 class that all bp/api-v3-support reviews depends-on. Thanks | 08:44 |
| openstackgerrit | vsaienko proposed stackforge/puppet-horizon: Add cache_backend and cache_options params https://review.openstack.org/190548 | 08:45 |
| openstackgerrit | vsaienko proposed stackforge/puppet-horizon: Add cache_backend and cache_options params https://review.openstack.org/190548 | 08:50 |
| rlaurika | EmilienM: regarding https://review.openstack.org/#/c/190308/: co-authored is fine. it's a trivial fix anyway. | 08:55 |
| *** jpena|mtg is now known as jpena | 08:58 | |
| *** tshtilma has joined #puppet-openstack | 09:01 | |
| *** gildub has quit IRC | 09:23 | |
| *** arnaud_orange1 has quit IRC | 09:48 | |
| *** michchap_ has joined #puppet-openstack | 09:52 | |
| *** michchap_ has quit IRC | 09:56 | |
| *** michchap_ has joined #puppet-openstack | 10:05 | |
| *** markvoelker has joined #puppet-openstack | 10:05 | |
| *** markvoelker has quit IRC | 10:10 | |
| *** gildub has joined #puppet-openstack | 10:28 | |
| *** alex_bh has quit IRC | 10:32 | |
| *** michchap_ has quit IRC | 10:33 | |
| *** rcallawa_ has joined #puppet-openstack | 10:52 | |
| *** jpena has quit IRC | 10:53 | |
| *** rcallawa has quit IRC | 10:55 | |
| *** cooldharma06 has joined #puppet-openstack | 10:57 | |
| *** jtomasek has quit IRC | 10:58 | |
| cooldharma06 | hi anybody any there | 10:59 |
| cooldharma06 | what is the minimum requirement to install puppet-opentack (allinone) in vm | 11:00 |
| cooldharma06 | disk and ram specification? | 11:00 |
| *** kbyrne has joined #puppet-openstack | 11:03 | |
| *** jtomasek has joined #puppet-openstack | 11:08 | |
| *** michchap_ has joined #puppet-openstack | 11:11 | |
| *** openstackgerrit has quit IRC | 11:13 | |
| *** openstackgerrit has joined #puppet-openstack | 11:14 | |
| *** markvoelker has joined #puppet-openstack | 11:21 | |
| *** markvoelker has quit IRC | 11:25 | |
| *** michchap_ has quit IRC | 11:26 | |
| *** ducttape_ has joined #puppet-openstack | 11:27 | |
| *** xingchao has joined #puppet-openstack | 11:39 | |
| *** alex_bh has joined #puppet-openstack | 11:41 | |
| *** rcallawa has joined #puppet-openstack | 11:43 | |
| *** ducttape_ has quit IRC | 11:45 | |
| *** rcallawa_ has quit IRC | 11:46 | |
| *** rcallawa has quit IRC | 11:47 | |
| *** xingchao has quit IRC | 11:48 | |
| *** derekh has joined #puppet-openstack | 11:49 | |
| EmilienM | good morning | 11:50 |
| EmilienM | rlaurika: ack, thx | 11:50 |
| EmilienM | gildub: will look today | 11:50 |
| gildub | EmilienM, G'Day mate! Great, thanks, | 11:52 |
| *** arnaud_orange has joined #puppet-openstack | 11:56 | |
| *** markvoelker has joined #puppet-openstack | 11:56 | |
| *** ferest has joined #puppet-openstack | 12:04 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 12:15 |
| EmilienM | gildub: your tests fail because of issues in rabbitmq module, I already tried to fix it yesterday ^ | 12:16 |
| EmilienM | and I continue today, hopefully I can make it so we can test your patches. | 12:16 |
| *** arnaud_orange has quit IRC | 12:21 | |
| *** michchap_ has joined #puppet-openstack | 12:24 | |
| gildub | EmilienM, I saw that too, but that was already there, not in the patch | 12:25 |
| EmilienM | gildub: I'll double check! don't worry about beaker, I'll give a hand today | 12:25 |
| gildub | EmilienM, yeah that's fine, so as beacker got in place there is a backlog of issues, correct? | 12:26 |
| EmilienM | I think we should have a backlog (trello card) of issues in Beaker | 12:26 |
| EmilienM | because I expect in future beaker break sometimes | 12:26 |
| EmilienM | and we need to track all work | 12:26 |
| EmilienM | I can't spend my days working on that, we need to scale this part | 12:27 |
| gildub | EmilienM, I undertand | 12:27 |
| *** cooldharma06 has quit IRC | 12:28 | |
| *** jayg|g0n3 is now known as jayg | 12:29 | |
| *** arnaud_orange has joined #puppet-openstack | 12:37 | |
| *** michchap_ has quit IRC | 12:39 | |
| *** dprince has joined #puppet-openstack | 12:39 | |
| *** derekh has quit IRC | 12:40 | |
| EmilienM | gildub: I confirm your patch and my fix are coherent. We hit the same thing | 12:40 |
| gildub | EmilienM, ok | 12:41 |
| *** arnaud_orange has quit IRC | 12:42 | |
| *** derekh has joined #puppet-openstack | 12:42 | |
| *** ferest has quit IRC | 12:43 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 12:46 |
| *** rcallawa has joined #puppet-openstack | 12:47 | |
| *** gildub has quit IRC | 13:00 | |
| *** michchap_ has joined #puppet-openstack | 13:06 | |
| *** ducttape_ has joined #puppet-openstack | 13:07 | |
| *** ducttape_ has quit IRC | 13:08 | |
| *** ducttape_ has joined #puppet-openstack | 13:09 | |
| *** ducttape_ has quit IRC | 13:09 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 13:11 |
| *** xingchao has joined #puppet-openstack | 13:12 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-neutron: Fix support for auth_uri setting in neutron provider https://review.openstack.org/190634 | 13:18 |
| *** tshtilma has quit IRC | 13:20 | |
| *** tshtilma has joined #puppet-openstack | 13:23 | |
| *** richm has joined #puppet-openstack | 13:25 | |
| *** michchap_ has quit IRC | 13:32 | |
| *** social has quit IRC | 13:40 | |
| *** social has joined #puppet-openstack | 13:44 | |
| *** michchap_ has joined #puppet-openstack | 13:52 | |
| *** RichardRaseley has joined #puppet-openstack | 13:58 | |
| RichardRaseley | EmilienM: Thanks for your response to Flavio. I had spoken to him at the summit about this, but haven't had the cycles yet. In your view, would the initial commit to the stackforge repository just be the boilerplate code from cookie cutter? | 14:00 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 14:00 |
| EmilienM | RichardRaseley: exactly | 14:00 |
| EmilienM | and then start to push bits from their module, using our conventions. | 14:00 |
| *** dgurtner has joined #puppet-openstack | 14:01 | |
| RichardRaseley | EmilienM: I think that sounds excellent. Would you like me to take on that initial bit to start? I am not sure if you need super-special permissions to generate a new stackforge repository... | 14:02 |
| EmilienM | RichardRaseley: yeah you can definitely start the work! | 14:02 |
| *** xingchao has quit IRC | 14:02 | |
| EmilienM | for stackforge I don't know, I think we might need to ask on #openstack-infra | 14:02 |
| RichardRaseley | EmilienM: Great, it will be a good learning experience. I will ask there this afternoon when I am beginning the work. | 14:03 |
| RichardRaseley | EmilienM: Thanks for the guidance. | 14:03 |
| EmilienM | cool | 14:03 |
| *** ducttape_ has joined #puppet-openstack | 14:06 | |
| *** RichardRaseley has quit IRC | 14:08 | |
| *** dgurtner has quit IRC | 14:10 | |
| *** michchap_ has quit IRC | 14:13 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed stackforge/puppet-openstack-specs: Enabling-k2k-federation https://review.openstack.org/190653 | 14:14 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 14:14 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed stackforge/puppet-openstack-specs: Enabling-k2k-federation https://review.openstack.org/190361 | 14:19 |
| *** ericpeterson has joined #puppet-openstack | 14:21 | |
| *** xingchao has joined #puppet-openstack | 14:22 | |
| *** ducttape_ has quit IRC | 14:24 | |
| rhefner | requesting a +A again on https://review.openstack.org/#/c/189771/ if any core has 2 minutes to review | 14:33 |
| *** kkirkpatrick has joined #puppet-openstack | 14:38 | |
| *** khappone has quit IRC | 14:39 | |
| *** khappone has joined #puppet-openstack | 14:40 | |
| rhefner | crinkle: mfisch: sbadia: ^ | 14:55 |
| *** tshtilma has quit IRC | 14:58 | |
| *** tshtilma has joined #puppet-openstack | 15:03 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 15:10 |
| *** michchap_ has joined #puppet-openstack | 15:11 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 15:14 |
| *** rlaurika has quit IRC | 15:14 | |
| *** rlaurika has joined #puppet-openstack | 15:14 | |
| mfisch | rhefner: why would I specify a service image but not want it loaded into glance? | 15:16 |
| *** jistr is now known as jistr|mtg | 15:17 | |
| *** alex_bh has quit IRC | 15:17 | |
| rhefner | mfisch: in rhel-osp 7, the glance endpoints aren't created until puppet is actually finished running. invoking service_instance causes a fatal error | 15:18 |
| rhefner | https://review.openstack.org/#/c/188137/7/puppet/manifests/overcloud_controller.pp | 15:18 |
| EmilienM | there is some orchestration to fix then :) | 15:19 |
| EmilienM | our modules should be install agnostic | 15:19 |
| rhefner | EmilienM: they've acknowledged that they need to work on that, but it is what it is for now | 15:19 |
| mfisch | well, I'm not going to block it but I won't +2 either because the use case makes no sense to me | 15:20 |
| mfisch | its more of someone else's bug | 15:20 |
| rhefner | hmm, so EmilienM should I abandon this patch and put the manila_config lines back into the overcloud_controller.pp above? (that you asked me to take out) | 15:25 |
| *** michchap_ has quit IRC | 15:26 | |
| *** mdorman has joined #puppet-openstack | 15:28 | |
| *** ducttape_ has joined #puppet-openstack | 15:32 | |
| EmilienM | crinkle, Hunner, nibalizer: I might need some help on https://review.openstack.org/#/c/190242 where beaker is failing and I don't understand the reason it can't find mysql binary. I have the feeling it runs the command before running Puppet. Any thoughts? | 15:33 |
| *** ericpeterson has quit IRC | 15:33 | |
| *** tshtilma has quit IRC | 15:35 | |
| *** jistr|mtg is now known as jistr | 15:35 | |
| *** fedexo has joined #puppet-openstack | 15:37 | |
| *** xingchao has quit IRC | 15:37 | |
| Hunner | EmilienM: commented | 15:40 |
| Hunner | Basically stuff needs to be inside `it` blocks | 15:40 |
| Hunner | So yeah, it's running during the first stage when it counts the rspec tests, but before it starts running the rspec tests | 15:41 |
| *** michchap_ has joined #puppet-openstack | 15:41 | |
| Hunner | (rspec runs in two stages: the first stage counts the tests and saves the before/after/it blocks to be run in the second stage, then the second stage runs the blocks and collects the returns) | 15:41 |
| *** ericpeterson has joined #puppet-openstack | 15:42 | |
| *** ducttape_ has quit IRC | 15:45 | |
| *** michchap_ has quit IRC | 15:46 | |
| EmilienM | Hunner: I learn one thing today, thx | 15:50 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 15:55 |
| *** RichardRaseley has joined #puppet-openstack | 16:03 | |
| EmilienM | crinkle: re- infra: even if we move everything under openstack namespace, though we might need to keep the compliancy thing | 16:20 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 16:24 |
| EmilienM | crinkle: but I think puppet-vswitch has not his place in openstack namespace | 16:24 |
| *** fedexo has quit IRC | 16:26 | |
| *** xingchao has joined #puppet-openstack | 16:38 | |
| crinkle | EmilienM: we should still keep the compliancy thing, it just might mean a note in the readme instead of a different namespace | 16:40 |
| crinkle | EmilienM: i don't care about puppet-vswitch, if jeblair thinks it makes sense to add it to openstack/ then i'm on board | 16:41 |
| *** michchap_ has joined #puppet-openstack | 16:42 | |
| *** tfz has quit IRC | 16:42 | |
| *** xingchao has quit IRC | 16:42 | |
| *** michchap_ has quit IRC | 16:46 | |
| *** aimon has quit IRC | 16:48 | |
| *** jpena has joined #puppet-openstack | 16:48 | |
| EmilienM | crinkle: I'm not really in favor of having it. Same for puppet-ceph | 16:49 |
| EmilienM | crinkle: though we have to move gnocchi, monasca and tuskar. The other things stay in stackforge | 16:50 |
| *** derekh has quit IRC | 16:51 | |
| EmilienM | crinkle: oh and there is puppet-murano... which is something we don't manage *at all* | 16:52 |
| *** stevemar has joined #puppet-openstack | 16:54 | |
| *** dprince has quit IRC | 16:56 | |
| EmilienM | I updated https://review.openstack.org/176326, please review | 16:57 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 17:03 |
| RichardRaseley | EmilienM: I tend to agree regarding inclusion in the OpenStack namespace - if the project is part of OpenStack than the corresponding module should, otherwise it should be in StackForge. Seems like the cleanest model to me. | 17:04 |
| _ody | Any core around that can review https://review.openstack.org/#/c/189873. This needs to merge before https://review.openstack.org/#/c/184646 will. | 17:22 |
| *** aimon has joined #puppet-openstack | 17:31 | |
| *** jistr has quit IRC | 17:35 | |
| *** jtomasek has quit IRC | 17:38 | |
| openstackgerrit | Merged stackforge/puppet-swift: Fix ipv6 support https://review.openstack.org/188947 | 17:38 |
| openstackgerrit | Merged stackforge/puppet-swift: Allow setting reseller_prefix for keystone filter https://review.openstack.org/188940 | 17:39 |
| *** dprince has joined #puppet-openstack | 17:41 | |
| openstackgerrit | Merged stackforge/puppet-keystone: Add support for Fernet Tokens https://review.openstack.org/185441 | 17:45 |
| EmilienM | _ody: the only one blocker to me are the changes in https://review.openstack.org/#/c/189873/ - they are not consistent with other modules, but I guess they will be | 17:48 |
| EmilienM | _ody: and nodepool.yml does not exist | 17:50 |
| *** jtomasek has joined #puppet-openstack | 17:51 | |
| *** degorenko has quit IRC | 18:01 | |
| RichardRaseley | EmilienM: In your opinion, are all of the caveats listed under the 'WORK IN PROGRESS' heading in the README for puppet-designate (https://github.com/stackforge/puppet-designate) still valid? | 18:05 |
| EmilienM | RichardRaseley: we can drop it imho | 18:08 |
| RichardRaseley | EmilienM: OK, thank you. | 18:17 |
| *** cdent has quit IRC | 18:20 | |
| openstackgerrit | Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit. https://review.openstack.org/189873 | 18:22 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: pin rabbitmq module to 5.2.1 https://review.openstack.org/190242 | 18:26 |
| *** tfz has joined #puppet-openstack | 18:27 | |
| *** jpena has quit IRC | 18:29 | |
| *** michchap_ has joined #puppet-openstack | 18:31 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-ceilometer: Deprecate old public, internal and admin parameters https://review.openstack.org/69006 | 18:33 |
| *** dprince has quit IRC | 18:34 | |
| *** michchap_ has quit IRC | 18:35 | |
| *** jpena has joined #puppet-openstack | 18:42 | |
| EmilienM | any core around can help me in unblocking openstacklib/beaker CI with +2 https://review.openstack.org/#/c/190242/ ? | 18:44 |
| mfisch | looking | 18:47 |
| mfisch | interesting response from Matthew | 18:48 |
| mfisch | EmilienM: lgtm but thats not my area of expertise | 18:48 |
| mfisch | I'm debating how to respond to Matthew | 18:50 |
| mfisch | I'm not too happy with the response | 18:50 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-designate: Introduce public_url, internal_url and admin_url https://review.openstack.org/70393 | 18:51 |
| *** rcallawa_ has joined #puppet-openstack | 19:02 | |
| *** bkero has quit IRC | 19:04 | |
| *** rcallawa has quit IRC | 19:05 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-glance: Introduce public_url, internal_url and admin_url https://review.openstack.org/70011 | 19:05 |
| *** bkero has joined #puppet-openstack | 19:05 | |
| *** jpena has quit IRC | 19:07 | |
| EmilienM | crinkle: for the serverspec change it's because I was hitting http://logs.openstack.org/42/190242/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty/09e766e/console.html#_2015-06-10_18_51_43_957 | 19:10 |
| mgagne | EmilienM: have you ever dreamed of having a way to auto-generate puppetdoc from the actual class parameter definitions? | 19:10 |
| RichardRaseley | EmilienM: Quick question on the Designate module, if I am reading this correctly it looks like there is some more places where the old 'dns' class name is hanging out in the module, e.g. https://github.com/stackforge/puppet-designate/blob/master/manifests/backend/bind9.pp:30 | 19:11 |
| mgagne | EmilienM: I'm not here to fullfill the dream, just inception the idea | 19:11 |
| RichardRaseley | (correction: https://github.com/stackforge/puppet-designate/blob/master/manifests/backend/bind9.pp line 30) | 19:11 |
| EmilienM | mgagne: using openstack real doc? | 19:11 |
| mgagne | https://github.com/stackforge/puppet-designate/blob/master/manifests/backend/bind9.pp#L30 | 19:11 |
| EmilienM | mgagne: I think we talked about that already | 19:11 |
| mgagne | EmilienM: yes (and no) | 19:12 |
| EmilienM | and had the same dream | 19:12 |
| *** stevemar has quit IRC | 19:12 | |
| mgagne | EmilienM: I'm reworking some patches and found that I have to copy/paste the default value here and there | 19:12 |
| RichardRaseley | mgagne: Ah, that's the syntax. Thx | 19:12 |
| mgagne | EmilienM: which doesn't make sense to me since it's already there in the manifest | 19:12 |
| mgagne | RichardRaseley: and that one works too if you ever need it: https://github.com/stackforge/puppet-designate/blob/master/manifests/backend/bind9.pp#L32-L37 | 19:12 |
| RichardRaseley | mgagne: +1 | 19:13 |
| crinkle | EmilienM: but that's completely unrelated to anything to do with rabbitmq and the commit says the patch is about rabbitmq | 19:13 |
| crinkle | EmilienM: if there's a problem with beaker that should be fixed in its own patch | 19:13 |
| EmilienM | crinkle: I should rename the title to 'fix beaker test' | 19:13 |
| crinkle | EmilienM: but then that excludes the part where you change how rabbitmq is installed and update the metadata | 19:14 |
| EmilienM | ok | 19:14 |
| EmilienM | crinkle: I guess I should split the patches | 19:14 |
| EmilienM | and have deps | 19:14 |
| crinkle | EmilienM: is this problem happening in other places? it seems like something we should have run into already | 19:15 |
| EmilienM | not afik | 19:15 |
| EmilienM | crinkle: but we don't have these kind of tests | 19:15 |
| mgagne | EmilienM: I'm not sure if it's the right form but: https://gist.github.com/mgagne/d526f176ff40c5b934e5 | 19:17 |
| EmilienM | mgagne: it's not right | 19:17 |
| mgagne | go on | 19:18 |
| EmilienM | crinkle: this is my origin problem: http://logs.openstack.org/42/190242/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty/a68ecfc/console.html#_2015-06-10_17_15_57_160 | 19:20 |
| EmilienM | crinkle: and I thought about your change in beaker version | 19:23 |
| crinkle | aha | 19:23 |
| crinkle | well | 19:23 |
| EmilienM | crinkle: am I wrong? | 19:23 |
| crinkle | that should have appeared in the patch that unpinned beaker | 19:23 |
| crinkle | why didn't it come up then? | 19:23 |
| EmilienM | hmm indeed | 19:23 |
| EmilienM | let me try again... | 19:23 |
| crinkle | or did they just make a new release | 19:24 |
| crinkle | it doesn't look like a new major release got installed | 19:24 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: test beaker - do not merge https://review.openstack.org/190770 | 19:24 |
| EmilienM | it | 19:25 |
| EmilienM | https://jenkins02.openstack.org/job/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7/17/console | 19:25 |
| crinkle | >.< | 19:25 |
| crinkle | https://review.openstack.org/#/c/189440/ | 19:25 |
| crinkle | it didn't pass | 19:25 |
| crinkle | and was merged anyway | 19:25 |
| * crinkle sigh | 19:26 | |
| EmilienM | damn | 19:26 |
| EmilienM | we will end up by making beaker voting | 19:26 |
| crinkle | yep | 19:26 |
| nibalizer | :( that sucks | 19:29 |
| nibalizer | but yay voting | 19:29 |
| EmilienM | crinkle: the error: https://jenkins02.openstack.org/job/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7/17/console | 19:31 |
| *** stevemar has joined #puppet-openstack | 19:40 | |
| EmilienM | crinkle: bingo | 19:41 |
| EmilienM | crinkle: with the unpin, serverspec is upgraded from 1.16.0 to 2.18 | 19:42 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: update serverspec syntax https://review.openstack.org/190770 | 19:42 |
| crinkle | EmilienM: commented | 19:43 |
| EmilienM | crinkle: thx | 19:44 |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-openstacklib: acceptance: update serverspec syntax https://review.openstack.org/190770 | 19:44 |
| EmilienM | crinkle: wdyt about https://review.openstack.org/#/c/189873/ ? | 19:45 |
| crinkle | EmilienM: i would ask spredzy | 19:46 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-heat: Introduce public_url, internal_url and admin_url https://review.openstack.org/70024 | 19:46 |
| RichardRaseley | mgagne: Is there still work that needs to be done preventing https://review.openstack.org/#/c/70458/ from being merged? I had planned to emulate that for the other patches, but wanted to wait until it was 'set in stone'. | 19:57 |
| mgagne | RichardRaseley: mostly waiting for other cores to approve | 19:57 |
| mgagne | RichardRaseley: I'm reviving the abandoned changes atm | 19:57 |
| *** rcallawa_ has quit IRC | 19:58 | |
| mgagne | btw, I think it didn't get the attention I wanted it to get but https://review.openstack.org/#/c/71158/ "Let me know if anyone feels they deserve their names as co-author in the commit message." | 19:59 |
| mgagne | hmm I'm having trouble running rspec tests for ironic. Getting undefined method `environmentpath=' | 20:00 |
| EmilienM | I've never seen that before | 20:00 |
| crinkle | mgagne: reinstall the gems with PUPPET_GEM_VERSION="~>3.0" bundle install | 20:00 |
| mgagne | thanks! | 20:01 |
| crinkle | or if you're trying to use puppet 4 then bundle update rspec-puppet | 20:01 |
| mgagne | I'm definitely not :D | 20:01 |
| crinkle | :) | 20:01 |
| EmilienM | mgagne: can you +A https://review.openstack.org/#/c/190770/ please? | 20:01 |
| EmilienM | crinkle: tests pass | 20:01 |
| crinkle | woot | 20:02 |
| mgagne | done | 20:02 |
| EmilienM | ok now tests are fixed, I can recheck the v3 thing | 20:03 |
| EmilienM | thx crinkle mgagne BTW | 20:03 |
| mgagne | The weather sure is weird here in Montreal. I thought it was about to rain (again) and now it's mostly sunny | 20:03 |
| EmilienM | I can predict we will have troubles sometimes with beaker jobs voting, but I can also predict we won't have much regression with beaker jobs voting | 20:03 |
| EmilienM | mgagne: I'm still wondering why I moved here sometimes :) | 20:04 |
| mgagne | haha | 20:04 |
| mgagne | I bought a bike and haven't found time to try it yet, been raining for days (when I'm not at work) | 20:05 |
| mgagne | bike is still in the car :-/ | 20:05 |
| crinkle | you guys can come to portland, it's been bright and sunny for weeks | 20:06 |
| EmilienM | crinkle: hum. | 20:06 |
| *** rcallawa has joined #puppet-openstack | 20:06 | |
| EmilienM | crinkle: at the summit it was raining all the time, are you kidding? | 20:06 |
| EmilienM | (portland, april 2013) | 20:06 |
| crinkle | not this year | 20:06 |
| crinkle | it's painfully sunny | 20:07 |
| EmilienM | 'painfully' | 20:07 |
| EmilienM | crinkle: define 'painfully' | 20:08 |
| crinkle | EmilienM: 90 degrees F with no air conditioning | 20:08 |
| openstackgerrit | Merged stackforge/puppet-openstacklib: acceptance: update serverspec syntax https://review.openstack.org/190770 | 20:09 |
| EmilienM | crinkle: inside? | 20:11 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url https://review.openstack.org/70160 | 20:12 |
| *** rcallawa has quit IRC | 20:13 | |
| EmilienM | mgagne: what you do is a good candidate for https://github.com/enovance/puppet-openstack-cookiecutter/blob/master/puppet-%7B%7Bcookiecutter.project_name%7D%7D/manifests/keystone/auth.pp | 20:13 |
| mgagne | EmilienM: true | 20:13 |
| EmilienM | crinkle: https://review.openstack.org/#/c/187654 is passing tests now (still in zuul, but beaker is green now). The code lgtm, maybe you can also have a look | 20:17 |
| *** chem has quit IRC | 20:18 | |
| *** michchap_ has joined #puppet-openstack | 20:19 | |
| *** xingchao has joined #puppet-openstack | 20:21 | |
| *** chem has joined #puppet-openstack | 20:22 | |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - v3 domain utility code https://review.openstack.org/178385 | 20:22 |
| nibalizer | EmilienM: so gating | 20:23 |
| nibalizer | i dunno if you want those in 'gate' or not | 20:24 |
| nibalizer | i think you should ask infra | 20:24 |
| *** michchap_ has quit IRC | 20:24 | |
| nibalizer | every time i think I know how it works I turn out to be wrong | 20:24 |
| EmilienM | nibalizer: well, iirc, it just running the job again before merging | 20:24 |
| nibalizer | kindof | 20:24 |
| nibalizer | its good to do it in the gate regardless | 20:25 |
| nibalizer | there is work https://review.openstack.org/#/c/184905/15/spec/spec_helper_acceptance.rb here | 20:25 |
| nibalizer | to let zuul take a more active role | 20:25 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-manila: Introduce public_url, internal_url and admin_url https://review.openstack.org/190802 | 20:26 |
| nibalizer | but I think it can't do that unless all the jobs have the same name | 20:26 |
| nibalizer | and are one test not many different tests | 20:26 |
| nibalizer | anyways, short answer is I think you should put those in the 'gate' as well | 20:26 |
| EmilienM | kk | 20:27 |
| mgagne | wth monasca::keystone::auth | 20:27 |
| *** xingchao has quit IRC | 20:27 | |
| EmilienM | mgagne: keep calm ! | 20:28 |
| EmilienM | this module is WIP I think :) | 20:28 |
| mgagne | it sure didn't used cookiecutter | 20:28 |
| EmilienM | sure thing, but future module will I hope! | 20:28 |
| EmilienM | nibalizer: done | 20:30 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the domain resource https://review.openstack.org/174439 | 20:30 |
| *** stevemar has quit IRC | 20:30 | |
| *** jtomasek has quit IRC | 20:33 | |
| EmilienM | richm: tests fail on https://review.openstack.org/#/c/178385/, do you need a hand? | 20:34 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the project resource https://review.openstack.org/174468 | 20:34 |
| *** jayg is now known as jayg|g0n3 | 20:34 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-neutron: Introduce public_url, internal_url and admin_url https://review.openstack.org/70411 | 20:47 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-ceilometer: Deprecate old public, internal and admin parameters https://review.openstack.org/69006 | 20:47 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-ceilometer: Deprecate old public, internal and admin parameters https://review.openstack.org/69006 | 20:48 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-designate: Introduce public_url, internal_url and admin_url https://review.openstack.org/70393 | 20:48 |
| EmilienM | richm: it's weird, tests are failing because the depends-on does not seem to work here, you don't have the right version of openstacklib | 20:48 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-glance: Introduce public_url, internal_url and admin_url https://review.openstack.org/70011 | 20:49 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-heat: Introduce public_url, internal_url and admin_url https://review.openstack.org/70024 | 20:49 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url https://review.openstack.org/70160 | 20:50 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-manila: Introduce public_url, internal_url and admin_url https://review.openstack.org/190802 | 20:50 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-neutron: Introduce public_url, internal_url and admin_url https://review.openstack.org/70411 | 20:50 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-nova: Introduce public_url, internal_url and admin_url (and v3/ec2) https://review.openstack.org/70458 | 20:51 |
| *** EmilienM sets mode: +o mgagne | 20:51 | |
| *** chem has quit IRC | 20:55 | |
| *** chem has joined #puppet-openstack | 20:55 | |
| *** chem has quit IRC | 20:58 | |
| *** chem has joined #puppet-openstack | 20:58 | |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the project resource https://review.openstack.org/174468 | 21:00 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-sahara: Introduce public_url, internal_url and admin_url https://review.openstack.org/190816 | 21:01 |
| EmilienM | crinkle: do you mind if I abandon https://review.openstack.org/#/c/188567/ ? | 21:07 |
| crinkle | EmilienM: abandoned | 21:09 |
| EmilienM | crinkle: thx | 21:09 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user resource https://review.openstack.org/174976 | 21:10 |
| EmilienM | there are some patches that we need for kilo/juno releases, please have a look :-) | 21:11 |
| *** RichardRaseley has quit IRC | 21:11 | |
| *** vinsh has joined #puppet-openstack | 21:16 | |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user_role resource https://review.openstack.org/174999 | 21:18 |
| *** chem has quit IRC | 21:19 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-trove: Introduce public_url, internal_url and admin_url https://review.openstack.org/190822 | 21:26 |
| *** chem has joined #puppet-openstack | 21:28 | |
| *** chem has quit IRC | 21:29 | |
| *** chem has joined #puppet-openstack | 21:29 | |
| dmsimard | mgagne is on fire | 21:29 |
| mgagne | my keyboard is on fire | 21:29 |
| dmsimard | :D | 21:29 |
| EmilienM | oh dear, again | 21:39 |
| *** chem has quit IRC | 21:44 | |
| *** chem has joined #puppet-openstack | 21:47 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-swift: Introduce public_url(_s3), internal_url(_s3) and admin_url(_s3) https://review.openstack.org/71108 | 22:01 |
| mgagne | alright, I think it's the last time I had to restore | 22:02 |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-swift: Introduce public_url(_s3), internal_url(_s3) and admin_url(_s3) https://review.openstack.org/71108 | 22:03 |
| *** aimon has quit IRC | 22:03 | |
| mgagne | what's up puppet-designate, breaking by yourself? :O | 22:04 |
| *** michchap_ has joined #puppet-openstack | 22:08 | |
| EmilienM | "Invalid resource type concat" | 22:10 |
| EmilienM | it used to work | 22:11 |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-keystone: Do not merge - investigating zuul references https://review.openstack.org/190839 | 22:11 |
| EmilienM | before May 15th, maybe a release somewhere | 22:11 |
| *** michchap_ has quit IRC | 22:12 | |
| _ody | Keep feeling like tests are failing randomly this week | 22:16 |
| *** kkirkpatrick has quit IRC | 22:16 | |
| openstackgerrit | Mathieu Gagné proposed stackforge/puppet-swift: Introduce public_url(_s3), internal_url(_s3) and admin_url(_s3) https://review.openstack.org/71108 | 22:17 |
| *** RichardRaseley has joined #puppet-openstack | 22:20 | |
| *** RichardRaseley has quit IRC | 22:32 | |
| *** guimaluf has quit IRC | 22:33 | |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-keystone: Do not merge - investigating zuul references https://review.openstack.org/190839 | 22:35 |
| *** ericpeterson has quit IRC | 22:36 | |
| *** michchap_ has joined #puppet-openstack | 23:09 | |
| *** markvoelker has quit IRC | 23:11 | |
| *** michchap_ has quit IRC | 23:13 | |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-keystone: Make beaker clone from zuul if possible https://review.openstack.org/190839 | 23:14 |
| openstackgerrit | Sam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default https://review.openstack.org/180378 | 23:23 |
| openstackgerrit | Colleen Murphy proposed stackforge/puppet-keystone: Make beaker clone from zuul if possible https://review.openstack.org/190839 | 23:25 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - v3 domain utility code https://review.openstack.org/178385 | 23:35 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the domain resource https://review.openstack.org/174439 | 23:37 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the project resource https://review.openstack.org/174468 | 23:39 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user resource https://review.openstack.org/174976 | 23:40 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user_role resource https://review.openstack.org/174999 | 23:42 |
| richm | I'm all about that rebase, 'bout that rebase, 'bout that rebase | 23:46 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 23:47 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - add domains to service_identity https://review.openstack.org/180586 | 23:48 |
| openstackgerrit | Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 23:50 |
| richm | no treble | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!