Tuesday, 2016-02-16

*** _ducttape_ has quit IRC00:03
*** gildub has joined #puppet-openstack00:12
*** dfisher has quit IRC00:16
*** markvoelker has joined #puppet-openstack00:35
*** xingchao has joined #puppet-openstack00:35
*** markvoelker has quit IRC00:39
*** gildub has quit IRC00:40
*** ducttape_ has joined #puppet-openstack00:40
*** xingchao has quit IRC00:44
*** ducttape_ has quit IRC00:45
*** xingchao has joined #puppet-openstack00:47
*** mdorman has quit IRC00:48
*** nibalizer is now known as nibz_irssi00:49
crinklewhy are these different? http://git.openstack.org/cgit/openstack/puppet-keystone/tree/manifests/init.pp#n622 and http://git.openstack.org/cgit/openstack/puppet-neutron/tree/manifests/init.pp#n35600:52
*** rahulait has quit IRC00:52
*** nibz_irssi has quit IRC00:53
*** gildub has joined #puppet-openstack00:56
openstackgerritzhongshengping proposed openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/27655101:02
*** xingchao has quit IRC01:05
*** nlahouti has quit IRC01:08
*** bkero has joined #puppet-openstack01:10
*** xingchao has joined #puppet-openstack01:11
bkeroHi there. I'm trying to run puppet-openstack-integration's all-in-one (or just run_tests.sh scenario003). I'm getting failure with "Error: nova --os-username admin --os-password a_big_secret --os-tenant-name openstack --os-auth-url http://127.0.0.1:5000/v2.0 flavor-create m1.micro 84 128 0 1 returned 1 instead of one of [0]"01:11
bkeroCould it be because I don't have cpu virtualization inside my kvm VM?01:12
*** brandon has joined #puppet-openstack01:17
crinklebkero: if you run the command manually you can get more information beyond 'returned 1'01:18
bkerocrinkle: Ah, good idea!01:18
*** xingchao has quit IRC01:19
openstackgerritzhongshengping proposed openstack/puppet-ironic: Add api_paste type/provider for Ironic  https://review.openstack.org/27656501:23
* bkero adds kvm_intel.nested=1 just in case01:23
*** xingchao has joined #puppet-openstack01:27
*** ducttape_ has joined #puppet-openstack01:29
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Fix rabbitmq ssl logic  https://review.openstack.org/28046201:32
bkeroGuh, seems to be something in cinder-volumes. Stderr: u'  Volume group "cinder-volumes" not found\n  Cannot process volume group cinder-volumes\n'01:33
*** johnmilton has joined #puppet-openstack01:37
*** xingchao has quit IRC01:40
*** ducttape_ has quit IRC01:44
*** ducttape_ has joined #puppet-openstack01:47
bkeroAlso ImportError: No module named oslo.middleware01:55
bkero:|01:56
* bkero wondering if there's something different between my VM and the nodepool VMs01:56
*** xingchao has joined #puppet-openstack01:59
openstackgerritzhongshengping proposed openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/27655102:14
*** xingchao has quit IRC02:14
bkeroUhh, I don't think it's a request_id compatibility shim, but that's the only occurrence of 'oslo.middleware' I can find.02:15
*** brandon has quit IRC02:24
*** ducttape_ has quit IRC02:30
*** markvoelker has joined #puppet-openstack02:36
*** markvoelker has quit IRC02:40
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:48
*** baker has joined #puppet-openstack02:50
*** xingchao has joined #puppet-openstack02:56
*** gildub has quit IRC02:59
*** fedexo has joined #puppet-openstack03:08
*** johnmilton has quit IRC03:08
*** ducttape_ has joined #puppet-openstack03:09
*** ducttape_ has quit IRC03:23
*** llua^_^AznSmile- is now known as llua03:27
*** llua has joined #puppet-openstack03:27
*** gildub has joined #puppet-openstack03:28
*** ducttape_ has joined #puppet-openstack03:28
bkerocrinkle: Problem was my VM was unclean. Had delorean-kilo repo in it :(03:33
*** ducttape_ has quit IRC03:57
*** xingchao has quit IRC04:07
*** gildub has quit IRC04:09
*** xingchao has joined #puppet-openstack04:14
*** gildub has joined #puppet-openstack04:34
*** markvoelker has joined #puppet-openstack04:36
*** markvoelker has quit IRC04:41
*** yottatsa has joined #puppet-openstack04:46
*** xingchao has quit IRC04:46
*** nlahouti has joined #puppet-openstack05:04
*** yottatsa has quit IRC05:06
*** nlahouti1 has joined #puppet-openstack05:07
*** nlahouti has quit IRC05:10
*** fedexo has quit IRC05:18
*** saneax_AFK is now known as saneax05:41
*** yottatsa has joined #puppet-openstack05:47
*** baker has quit IRC06:04
*** yottatsa has quit IRC06:06
*** yottatsa has joined #puppet-openstack06:19
*** gildub has quit IRC06:20
*** jaosorior has joined #puppet-openstack06:20
*** markvoelker has joined #puppet-openstack06:37
*** markvoelker has quit IRC06:42
*** baker has joined #puppet-openstack06:45
*** baker has quit IRC06:50
*** yottatsa has quit IRC07:02
*** leanderthal|afk is now known as leanderthal07:24
*** rcernin has joined #puppet-openstack07:37
*** sbadia has quit IRC07:42
*** IvanBerezovskiy has quit IRC07:44
*** jprs has quit IRC07:44
*** IvanBerezovskiy has joined #puppet-openstack07:45
*** Anticimex has quit IRC07:45
*** aarefiev has quit IRC07:45
*** aarefiev has joined #puppet-openstack07:46
*** jprs has joined #puppet-openstack07:46
*** Anticimex has joined #puppet-openstack07:46
*** sbadia has joined #puppet-openstack07:47
*** dgurtner has joined #puppet-openstack07:51
*** dgurtner has quit IRC07:51
*** dgurtner has joined #puppet-openstack07:51
*** nlahouti has joined #puppet-openstack07:52
*** nlahouti has quit IRC07:53
*** nlahouti1 has quit IRC07:54
*** samuelBartel has joined #puppet-openstack07:56
*** foxhunt_away is now known as foxhunt7208:11
*** rstarmer has joined #puppet-openstack08:13
*** rstarmer has quit IRC08:13
*** rstarmer has joined #puppet-openstack08:14
*** rstarmer has quit IRC08:16
*** mkarpin has left #puppet-openstack08:18
*** hjensas has joined #puppet-openstack08:20
*** mkarpin has joined #puppet-openstack08:21
*** yottatsa has joined #puppet-openstack08:21
*** jpena has joined #puppet-openstack08:22
*** rstarmer has joined #puppet-openstack09:12
*** paramite has joined #puppet-openstack09:24
*** rstarmer has quit IRC09:43
*** xingchao has joined #puppet-openstack09:45
*** rstarmer has joined #puppet-openstack09:47
*** Aegil has joined #puppet-openstack09:48
*** xingchao has quit IRC10:01
openstackgerritMerged openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/28025110:14
openstackgerritMerged openstack/puppet-neutron: Deprecate router[dhcp]_delete_namespaces parameter  https://review.openstack.org/27667410:14
*** rstarmer has quit IRC10:14
*** mkarpin has left #puppet-openstack11:00
*** andybotting has quit IRC11:13
*** xingchao has joined #puppet-openstack11:14
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Create a separate database configuration for ironic-inspector  https://review.openstack.org/27969511:18
openstackgerritIvan Berezovskiy proposed openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/28018111:20
openstackgerritIvan Berezovskiy proposed openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/28018111:34
*** _degorenko|afk is now known as degorenko11:39
*** rbabyuk has joined #puppet-openstack11:45
*** xingchao has quit IRC11:47
*** rstarmer has joined #puppet-openstack11:58
*** dprince has joined #puppet-openstack12:01
*** rbabyuk is now known as rbabyuk_12:01
*** rbabyuk_ is now known as rbabyuk12:03
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454812:06
*** baker has joined #puppet-openstack12:10
*** ericksonsantos has joined #puppet-openstack12:13
*** baker has quit IRC12:15
*** rstarmer has quit IRC12:21
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Allow setting of public_addr from ::profile::mon and ::mon  https://review.openstack.org/28035112:23
*** rbabyuk has quit IRC12:27
*** saneax is now known as saneax_AFK12:28
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow sending random kernel arguments to the IPA  https://review.openstack.org/28065212:33
EmilienMgood morning12:34
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/27969512:38
*** johnmilton has joined #puppet-openstack12:38
EmilienMbkero: our integration jobs do not run on kilo13:01
*** bdossant has joined #puppet-openstack13:07
*** jayg|g0n3 is now known as jayg13:08
*** ducttape_ has joined #puppet-openstack13:12
*** mflobo has left #puppet-openstack13:14
*** chem has quit IRC13:18
*** chem has joined #puppet-openstack13:21
*** ducttape_ has quit IRC13:25
*** rstarmer has joined #puppet-openstack13:32
chemmwhahaha: EmilienM could you review  https://review.openstack.org/#/c/272146/ when you've got 1 min ?  I'm going to request for the associated integration job to be merged as well13:38
*** v1k0d3n has quit IRC13:39
*** markvoelker has joined #puppet-openstack13:39
EmilienMchem: +213:40
chemEmilienM: cool13:40
EmilienMdegorenko: can you look https://review.openstack.org/#/c/277223/ please ,13:42
degorenkoEmilienM, sure13:43
degorenkoEmilienM, i have only one nit comment from 4 patch set13:44
*** hjensas has quit IRC13:44
degorenkowhat do you think about it?13:44
EmilienMdegorenko: it's a problem fixed last week13:46
EmilienMmistral package was broken13:46
degorenkoEmilienM, no, i'm about inherits + include for params class13:47
EmilienMhow the inherits stuff13:47
EmilienMok, I can drop it13:47
degorenkoEmilienM, wait13:47
degorenkoi'm looking for puppetlabs docs for this13:47
degorenkoit's just my opinion13:47
*** yottatsa has quit IRC13:48
*** jpena is now known as jpena|lunch13:52
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Allow to configure Murano Engine workers  https://review.openstack.org/28070513:53
EmilienMdegorenko: it's already like this in other modules13:54
EmilienMdegorenko: let's merge as it is, it does not break anything now13:54
EmilienMwe'll clean it later if we find that's not useful13:54
degorenkoEmilienM, ok13:54
EmilienMthanks13:54
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow sending random kernel arguments to the IPA  https://review.openstack.org/28065214:07
openstackgerritMerged openstack/puppet-mistral: Implement WSGI support for mistral-api  https://review.openstack.org/27722314:09
*** yottatsa has joined #puppet-openstack14:09
*** jaosorior has quit IRC14:16
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Enable to run boostrap without an upstream repo.  https://review.openstack.org/27214614:17
*** jaosorior has joined #puppet-openstack14:17
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/27969514:18
*** rstarmer has quit IRC14:18
openstackgerritMerged openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/27655114:21
*** rstarmer has joined #puppet-openstack14:23
openstackgerritMerged openstack/puppet-openstack_extras: auth file: default urls to keystone v3  https://review.openstack.org/28034414:25
*** saneax_AFK is now known as saneax14:30
*** baker has joined #puppet-openstack14:32
*** yottatsa has quit IRC14:32
openstackgerritMerged openstack/puppet-ironic: Add api_paste type/provider for Ironic  https://review.openstack.org/27656514:33
*** yottatsa has joined #puppet-openstack14:34
*** dfisher has joined #puppet-openstack14:35
*** yottatsa has quit IRC14:35
*** mflobo has joined #puppet-openstack14:36
*** paramite is now known as paramite|afk14:39
*** baker has quit IRC14:39
*** yottatsa has joined #puppet-openstack14:40
*** paramite|afk is now known as paramite14:40
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/27969514:42
*** baker has joined #puppet-openstack14:46
*** rstarmer has quit IRC14:47
EmilienMmeeting in 10 min, agenda is here: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016021614:50
*** baker has quit IRC14:51
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/28074214:51
*** aarefiev has quit IRC14:54
*** jpena|lunch is now known as jpena14:54
*** aarefiev has joined #puppet-openstack14:55
*** aarefiev has joined #puppet-openstack14:55
*** ducttape_ has joined #puppet-openstack14:57
*** ducttape_ has quit IRC14:57
*** hjensas has joined #puppet-openstack14:57
*** hjensas has joined #puppet-openstack14:57
*** ducttape_ has joined #puppet-openstack14:57
*** gchamoul has quit IRC14:58
*** morazi has joined #puppet-openstack14:59
*** gchamoul has joined #puppet-openstack15:02
*** baker has joined #puppet-openstack15:07
*** rstarmer has joined #puppet-openstack15:08
*** xarses_ has quit IRC15:11
*** baker has quit IRC15:12
*** baker has joined #puppet-openstack15:14
openstackgerritMerged openstack/puppet-openstack-integration: Use $default_domain for ::keystone::endpoint  https://review.openstack.org/28038315:17
*** rstarmer has quit IRC15:20
*** baker has quit IRC15:22
*** _ody_ is now known as _ody15:26
*** paramite is now known as paramite|afk15:28
*** mdorman has joined #puppet-openstack15:29
*** paramite|afk is now known as paramite15:31
*** morazi has quit IRC15:31
*** rochaporto has joined #puppet-openstack15:31
bkeroEmilienM: Yeah, was a mistake on my part. Unclean vm had an old repo that was pulling in packages.15:32
EmilienMno problem15:34
EmilienMbkero: we're in upstream meeting atm15:34
*** dprince has quit IRC15:35
*** xarses_ has joined #puppet-openstack15:35
bkeroAh ok. Don't have that one on my schedule.15:36
EmilienMbkero: https://wiki.openstack.org/wiki/Meetings/Puppet15:37
rochaportoEmilienM: i got late, but wanted to ask about https://review.openstack.org/#/c/275223/15:37
bkeroThanks15:37
*** rstarmer has joined #puppet-openstack15:37
rochaportomostly ask if someone could have a look :) and the magnum-api one from nate which seems to be waiting15:38
*** xarses_ is now known as xarses15:40
bkeroEmilienM: Thanks again, I'll try to attend in the future.15:41
openstackgerritDenis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers  https://review.openstack.org/27351315:47
*** paramite has quit IRC15:50
*** saneax is now known as saneax_AFK15:58
*** mflobo has quit IRC16:02
*** haplo37 has joined #puppet-openstack16:06
*** rstarmer has quit IRC16:26
*** bdossant has quit IRC16:29
*** hjensas has quit IRC16:31
*** rcernin has quit IRC16:37
*** leanderthal is now known as leanderthal|afk16:43
*** foxhunt72 is now known as foxhunt_away16:44
*** dfisher has quit IRC16:49
*** dfisher has joined #puppet-openstack16:49
*** mdorman_ has joined #puppet-openstack16:51
*** mdorman has quit IRC16:52
*** mdorman_ is now known as mdorman16:52
*** nlahouti has joined #puppet-openstack17:03
*** nlahouti has quit IRC17:03
*** nlahouti has joined #puppet-openstack17:03
*** dprince has joined #puppet-openstack17:04
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454817:05
*** dgurtner has quit IRC17:08
*** rcernin has joined #puppet-openstack17:09
*** samuelBartel has quit IRC17:13
*** rochaporto has quit IRC17:35
*** foxhunt_away is now known as foxhunt7217:36
*** saneax_AFK is now known as saneax17:39
EmilienMdegorenko: do you have progress on https://review.openstack.org/#/c/261326/ ?17:43
degorenkoEmilienM, it is in my activity list, but i don't have time for now, i will try to land it on this week17:43
EmilienMno problem17:43
EmilienMjust checking17:43
EmilienMif you need help let me know17:43
degorenkook, thanks17:44
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495117:44
degorenkohttps://review.openstack.org/226862 is also under refactoring17:44
*** rstarmer has joined #puppet-openstack17:44
*** degorenko is now known as _degorenko|afk17:45
*** dprince has quit IRC17:45
*** delattec has joined #puppet-openstack17:45
*** dgurtner has joined #puppet-openstack17:46
*** dprince has joined #puppet-openstack17:46
*** cdelatte has quit IRC17:48
*** rstarmer has quit IRC17:59
*** rahulait has joined #puppet-openstack18:00
*** baker has joined #puppet-openstack18:00
*** delattec has quit IRC18:01
*** delattec has joined #puppet-openstack18:03
*** xingchao has joined #puppet-openstack18:08
openstackgerritMerged openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/28018118:08
*** yottatsa has quit IRC18:09
*** rahulait has quit IRC18:09
*** v1k0d3n has joined #puppet-openstack18:10
*** yottatsa has joined #puppet-openstack18:10
*** rahulait has joined #puppet-openstack18:11
*** baker has quit IRC18:15
*** baker has joined #puppet-openstack18:17
*** yottatsa has quit IRC18:18
*** yottatsa has joined #puppet-openstack18:19
*** jpena has quit IRC18:21
bkeroHi #puppet-openstack people. I'm working on adding some RDO support for puppet-openstack-integration's scripts. Before I start a review for this, I thought I'd share this here. Is the upstream interested in taking changes that would facilitate this? http://paste.openstack.org/show/487154/18:22
*** baker has quit IRC18:24
dmsimardbkero: what's the objective ?18:29
*** gfidente has quit IRC18:29
bkerodmsimard: To be able to use puppet-openstack-integration and it's scenarios with delorean's openstack-puppet-modules.rpm18:29
dmsimardI'm not personally a fan of putting something like that upstream, it's kind of perverting it with downstream things18:37
*** baker has joined #puppet-openstack18:37
EmilienMright, things like this should be discussed on #rdo channel18:38
bkeroOkay18:39
EmilienMpuppet-openstack-integration is for upstream CI18:40
EmilienMif downstream projects want to use it, but we won't add specific bits in there.18:40
*** gfidente has joined #puppet-openstack18:41
EmilienMAFIK RDO is already using this repo as it is, and it works fine18:41
EmilienMdmsimard: ^18:41
EmilienMerr, bkero ^ ( dmsimard did the work )18:41
bkeroAh, got it18:42
EmilienMbkero: they use puppet-openstack-integration to gate all RDO packages18:42
*** baker has quit IRC18:42
EmilienMthanks for that, our upstream CI tend to never breaks18:42
EmilienM(because of RDO packaging)18:42
dmsimardIt was made to be generic and usable outside of the context of RDO or specific bits, Ubuntu/Canonical could use the tests with their own internal repositories if they wanted to18:43
EmilienMthat would be nice18:43
EmilienMbut I haven't seen them interested by doing that18:43
EmilienMAFIK they gate on their juju charms18:43
dmsimardyeah I don't know how they test things18:43
dmsimardoh ew18:43
bkerodmsimard: You mean by using the specific fixtures, but not the all-in-one/run-tests.sh?18:43
EmilienMwhich is very limited to a few services18:43
dmsimardbkero: this is what we re-use puppet-openstack-integration tests in RDO CI https://github.com/openstack/puppet-openstack-integration/commit/28007a665d8488b0c3d0a629b2423bcd5e26d42618:45
dmsimardSo anyone can configure their own repositories prior to running integration tests18:45
EmilienMI think that's what bkero is looking at.18:46
bkeroRight, that makes sense18:46
dmsimardAdding a feature upstream to essentially add support for installing forks of the upstream modules it's meant to test is a bit awkward18:46
bkerodmsimard: It's essentially adding support to disable the Puppetfile fetch18:47
dmsimardbkero: so add a switch to disable that, then18:47
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454818:48
dmsimardbut don't bake the fork installation right in18:48
dmsimardand do that in your job/test environment like RDO does18:48
bkeroOkay18:48
dmsimardalso, the manage_puppet_repo switch is probably not necessary, yum-plugin-priorities should make yum pick the right package just like when EPEL is enabled18:49
bkeroThat would also work, thank you18:50
*** foxhunt72 is now known as foxhunt_away18:51
*** baker has joined #puppet-openstack18:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/28035018:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test - do not merge  https://review.openstack.org/28088318:55
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Do not merge - CI test  https://review.openstack.org/28038518:56
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: CI test - do not merge  https://review.openstack.org/28088418:57
*** baker has quit IRC18:57
*** yottatsa has quit IRC18:58
EmilienMdgurtner: if you don't mind, I might need your guidance on a CI patch I'm working on, to deploy ceph: https://review.openstack.org/#/c/279826/3/manifests/ceph.pp18:58
*** yottatsa has joined #puppet-openstack18:58
*** nlahouti has quit IRC18:58
*** gfidente has quit IRC19:00
*** yottatsa has quit IRC19:00
*** nlahouti has joined #puppet-openstack19:01
*** baker has joined #puppet-openstack19:02
*** yottatsa has joined #puppet-openstack19:03
*** haplo37 has quit IRC19:03
*** yottatsa has quit IRC19:12
*** baker has quit IRC19:12
*** baker has joined #puppet-openstack19:13
*** rahulait has quit IRC19:18
*** rahulait has joined #puppet-openstack19:19
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/27934919:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::messaging::mongodb  https://review.openstack.org/27934819:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/27935119:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/27935019:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/27974719:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::management::mongodb  https://review.openstack.org/27934719:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/27974619:23
openstackgerritDan Prince proposed openstack/puppet-zaqar: Implement zaqar::init class  https://review.openstack.org/27934619:23
*** baker has quit IRC19:26
*** jtomasek_ has joined #puppet-openstack19:27
*** baker has joined #puppet-openstack19:29
EmilienMdprince: where is used python_redis_package_nam in https://review.openstack.org/279346 ?19:31
dprinceEmilienM: oh, I didn't submit that patch yet did I? Sorry.19:33
dprinceEmilienM: i can remove it too19:33
EmilienMthanks19:33
EmilienMlgtm otherwise19:33
dprinceEmilienM: I have redis patches... but they can come after this19:33
*** baker has quit IRC19:34
EmilienMdprince: wait then19:34
EmilienMwe can land it as it is, we are bootstraping the module anyway19:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/27934919:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::messaging::mongodb  https://review.openstack.org/27934819:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/27935119:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/27935019:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/27974719:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::management::mongodb  https://review.openstack.org/27934719:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/27974619:34
openstackgerritDan Prince proposed openstack/puppet-zaqar: Implement zaqar::init class  https://review.openstack.org/27934619:34
EmilienMtoo late19:34
EmilienMboom19:35
dprinceEmilienM: oh, sorry. Just caught your message19:35
EmilienM:) no problem19:35
dprinceEmilienM: it is fine. I can re-add it19:35
EmilienMlooking again, and I'll +219:35
EmilienMdprince: it's clean now!19:35
EmilienMdprince: +2 on init patch19:35
*** nlahouti has quit IRC19:36
EmilienMdprince: quick comment on https://review.openstack.org/#/c/279349/19:38
EmilienMdprince: and https://review.openstack.org/#/c/279350/19:40
EmilienMdprince: all reviewed19:43
*** haplo37 has joined #puppet-openstack19:49
*** guimaluf has joined #puppet-openstack19:49
*** dhollinger has joined #puppet-openstack19:56
*** baker has joined #puppet-openstack19:56
dprinceEmilienM: I used enherits because before I was defaulting parameters based on what was set in params.pp20:00
dprinceEmilienM: now that I removed that though the include you suggest is fine. I will switch to that20:00
EmilienMok20:00
*** saneax is now known as saneax_AFK20:01
*** saneax_AFK is now known as saneax20:02
*** jaosorior has quit IRC20:09
*** jfluhmann has joined #puppet-openstack20:11
*** saneax is now known as saneax_AFK20:16
*** guimaluf has quit IRC20:21
*** jfluhmann has quit IRC20:28
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/28035020:28
EmilienM_ody: we're still having a warning on puppet-vswitch http://logs.openstack.org/42/280742/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/8f52da4/console.html#_2016-02-16_15_38_27_214 -- any idea?20:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Update manifests to clean-up warnings  https://review.openstack.org/28091920:36
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/27934920:38
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/27935120:38
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/27935020:38
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/27974720:38
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/27974620:38
openstackgerritDan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/27935120:44
*** brandon has joined #puppet-openstack20:49
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: ceph: deploy keys & pools  https://review.openstack.org/27982620:53
*** rahulait has quit IRC20:55
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/28092921:05
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add project_name parameter for auth_file  https://review.openstack.org/28093021:06
*** rahulait has joined #puppet-openstack21:06
_odyEmilienM: Different than the one I fixed.  I'll take a quick look.21:10
EmilienMmaybe something we introduced recently, I'm aware we merged patches lately21:10
*** rahulait has quit IRC21:10
*** rahulait has joined #puppet-openstack21:13
_odyEmilienM: I see the issue, looks like I fixed something else but I am confused as to why I didn't see this issue when I fixed the last one.21:13
EmilienMcool :)21:14
_odyI'll go ahead and fix.21:15
EmilienMthanks21:15
*** brandon_ has joined #puppet-openstack21:20
openstackgerritMatt Fischer proposed openstack/puppet-keystone: Drop User/group resource dependency  https://review.openstack.org/19568421:21
*** brandon has quit IRC21:23
*** johnmilton has quit IRC21:23
*** dprince has quit IRC21:24
*** nlahouti has joined #puppet-openstack21:27
openstackgerritCody Herriges proposed openstack/puppet-vswitch: Remove duplicate loading of constants  https://review.openstack.org/28093721:29
_odyEmilienM: That should fix that other warning but...I just noticed, this module has near no tests.21:29
EmilienMwe have integration tests21:29
_odyYeah.  We'll have to rely on that.21:30
EmilienMthat's how I saw the WARNING21:31
*** nlahouti has quit IRC21:33
*** jtomasek_ has quit IRC21:39
*** rahulait has quit IRC21:42
EmilienMcrinkle: I might need your help on https://review.openstack.org/#/c/280350/ -- looking at zuul, it's still failing. Any idea of what I'm doing wrong?21:51
crinkleEmilienM: need to set project_name => 'openstack'21:55
EmilienMcrinkle: it's default, isn't?21:55
EmilienMcrinkle: https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/auth_file.pp#L9621:56
crinkleEmilienM: oh it wasn't before21:56
*** jtomasek has quit IRC21:58
EmilienMyeah I did it, you reviewed it I think21:58
crinkleEmilienM: i'm not sure then, we might need to set OS_IDENTITY_API_VERSION=3 but then i'm not sure why the keystone one was okay22:00
EmilienMcrinkle: identity_api_version is set to 3 by default already https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/auth_file.pp#L10822:00
crinklewelp22:00
* EmilienM goes on http://www.urbandictionary.com/define.php?term=Welp22:01
*** nibalizer has joined #puppet-openstack22:02
crinkleEmilienM: it's failing on glance_image which wasn't tested by the keystone module :(22:06
EmilienMit's also failing on neutron I think and also tempest22:07
EmilienMthat's why I'm adding your new class in our integ repo22:08
EmilienMto test it like you would have in prod22:08
EmilienMcrinkle: did you test it in infra cloud?22:08
crinkleEmilienM: nope22:09
EmilienMit is maybe an orchestration issue, I'm currently trying to reproduce the bug in my vm22:09
EmilienMcrinkle: the weird thing is, your new class has no effect on centos, because /etc/keystone/keystone-paste.ini does not exist22:12
EmilienMcrinkle: (it's in RDO master now, so it will exist soon)22:12
EmilienMso the problem is really in the openrc22:12
crinklei'm running glance func tests with the new keystone class to see if i can reproduce22:13
*** rstarmer has joined #puppet-openstack22:19
EmilienMcrinkle: so I can upload the image, bot not list it22:20
EmilienMoh wait, no image create also fails sorry22:20
EmilienMgot 40022:20
*** jayg is now known as jayg|g0n322:21
*** rahulait has joined #puppet-openstack22:21
EmilienMcrinkle: "Expecting to find domain in project"22:21
EmilienMcrinkle: I found it22:23
EmilienMI'm testing it before22:24
EmilienMyeah it works22:25
*** rahulait has quit IRC22:25
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: authfile: fix OS_IDENTITY_API_VERSION  https://review.openstack.org/28096022:25
EmilienMcrinkle: ^22:25
crinkleoh ha22:25
EmilienMsorry for that22:26
EmilienMI'll RTFM next time22:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/28035022:27
dfisherFYI: I just pushed v6.1.0 of cinder, glance, heat, horizon, ironic, keystone, neutron, nova, swift and openstacklib into Solaris 1222:29
openstackgerritBen Kero proposed openstack/puppet-openstack-integration: Add ability to pass puppet args in as env var  https://review.openstack.org/28096422:30
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Remove keystone_disable_admin_token_auth beaker test  https://review.openstack.org/28038522:33
openstackgerritBen Kero proposed openstack/puppet-openstack-integration: Add ability to manually install puppet modules  https://review.openstack.org/28096522:33
*** rahulait has joined #puppet-openstack22:35
dgurtnerEmilienM: ping re: https://review.openstack.org/#/c/279826/22:37
EmilienMdgurtner: hey22:37
EmilienMweird it fails, while it works on my laptp22:38
dgurtnerEmilienM: it fails due to lack of idempotency, no?22:38
dgurtnerEmilienM: I think this http://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bcc77e6/console.html#_2016-02-16_21_53_17_54022:38
EmilienMI'm checking22:38
EmilienMhttp://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/6f3b383/console.html#_2016-02-16_21_29_46_96222:39
dgurtnerEmilienM: and this http://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bcc77e6/console.html#_2016-02-16_21_53_17_72222:39
EmilienMlooking both links now22:39
dgurtnerEmilienM: the module actually lacks idempotency there22:40
EmilienMdgurtner: right we have a bug in puppet-ceph about idempotency22:40
EmilienMno prob, we'll fix it :-)22:40
dgurtnerEmilienM: when using cephx a file resource gets created and later the file gets removed via rm22:40
EmilienMthe good news is, it's only for one Exec22:40
EmilienMdgurtner: why do we remove it?22:41
dgurtnerit's a temporary file in /tmp/ which contains the key22:41
EmilienMwe need to fix the "unless"22:42
dgurtnerhttps://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L116-L11922:43
dgurtnerhttps://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L180-L18622:43
dgurtnerfirst adds the file, second one removes it22:43
EmilienMdgurtner: I'm more concerned about https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L18222:43
EmilienMwhy do we have /bin/true22:43
EmilienMthat's terrible no?22:44
EmilienMwe need to cleanup the file after the keyring creation, isn't?22:44
dgurtnerafter monitor bootstrap22:44
EmilienMdfisher: here https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L159 ?22:44
dgurtnerthe problem is that the "keyring_path" file shouldn't exist, but needs to be there22:45
dfisherEmilienM: pardon?22:45
dfisheryou might have meant 'dgurtner' :)22:45
EmilienMdfisher: sorry wrong nickname.22:45
dfisherno worries!22:45
EmilienMI'm so lazy, I just type "d" and tab22:45
dfisher:)22:46
EmilienMdgurtner: can we cleanup in another place?22:46
dgurtnerwe probably shouldn't have a file resource on https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L11622:46
EmilienMor we can do something else22:47
EmilienMin the rm-keyring-${id} Exec22:48
EmilienMadd a refreshonly => true22:48
dgurtnernot really, the file shouldn't exist22:48
EmilienMso this exec will be executed only if we have a change in the keyring22:48
dgurtnerthe problem is the file resource will just recreate it22:48
dgurtnerand that file will always contain the monitor key, which is a security issue22:49
*** haplo37 has quit IRC22:52
EmilienMdgurtner: we can still create an exec instead of FIle resource22:52
EmilienMso we can add conditionals to manage it22:52
EmilienMdgurtner: or stop using /tmp22:53
EmilienMmaybe use /etc/ceph/tmp ?22:53
EmilienMdgurtner: wdyt?22:57
dgurtnerthe solution is probably something ugly like this: http://paste.openstack.org/show/487200/22:57
dgurtnerEmilienM: ^22:57
EmilienMyes22:57
EmilienMit's ok22:58
EmilienMdgurtner: let's do it22:58
EmilienMdgurtner: you want to send a patch ? or you want me to do it?22:58
dgurtnerEmilienM: I can send it. the problem is if I send patches we need 2 of the other core-reviewers to approve ;)22:59
dgurtnerEmilienM: or in all fairness, given I already wrote this, we probably need 2 anyways22:59
EmilienMthis is not a critical bug, and not very urgent. So we can still ping xarses and dmsimard for a review tomorrow23:00
EmilienMdgurtner: I'll let you send it, and I'll review / test it in our CI23:00
dgurtnerEmilienM: works for me23:00
EmilienMif you can do it, otherwise I'll push it, I have a bit of time now23:00
xarses?23:02
xarseswhich now?23:02
EmilienMxarses: dgurtner is about to send a patch, to fix idempotency in ceph::profile::mon class23:02
xarsesok23:02
EmilienMwe found a bug23:02
xarsesYa, I'm not surprised =(23:03
*** rahulait has quit IRC23:03
*** cwolferh_ is now known as cwolferh23:03
EmilienMbut AFIK that's the last bug I've seen that is blocking our CI23:03
*** dhollinger has quit IRC23:04
EmilienMother than that, I can deploy puppet-ceph, create pools with idempotency. Which is cool23:04
xarsesyay!23:06
*** nlahouti has joined #puppet-openstack23:09
*** rcernin has quit IRC23:18
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Fix idempotency when bootstrapping ceph::mon with cephx  https://review.openstack.org/28098423:21
dgurtnerEmilienM: xarses ^ I want to add an acceptance test which uses cephx to test idempotency on this23:22
dgurtnerapart from that I think this should do the trick23:22
xarsesdgurtner: ok, cephx was only being tested integration in rgw / swift23:23
dgurtnerxarses: but probably not with idempotency23:23
EmilienMit sounds good to me, I'll test it23:23
xarsesthere was some acceptance tests that worked with it, but thats the only integration23:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: ceph: deploy keys & pools  https://review.openstack.org/27982623:24
EmilienMdgurtner: ^ testing in progress23:24
*** _ody_ has joined #puppet-openstack23:27
dgurtnermy hotel has random internet issues, so I'm currently having issues testing locally as it times out downloading gems :/23:27
*** rstarmer has quit IRC23:28
*** brandon has joined #puppet-openstack23:28
*** mdorman_ has joined #puppet-openstack23:28
*** saneax_AFK is now known as saneax23:29
*** mwhahaha_ has joined #puppet-openstack23:30
*** reynir1 has joined #puppet-openstack23:30
*** rhefner_ has joined #puppet-openstack23:30
*** ducttape_ has quit IRC23:31
*** Mosibi_ has joined #puppet-openstack23:32
*** kgalanov_ has joined #puppet-openstack23:33
*** itsuugo_ has joined #puppet-openstack23:35
*** EmilienM_ has joined #puppet-openstack23:36
*** clayton_ has joined #puppet-openstack23:36
*** brandon_ has quit IRC23:37
*** mdorman has quit IRC23:37
*** nibalizer has quit IRC23:37
*** itsuugo has quit IRC23:37
*** dalees has quit IRC23:37
*** clayton has quit IRC23:37
*** mwhahaha has quit IRC23:37
*** EmilienM has quit IRC23:37
*** _ody has quit IRC23:37
*** mdavidson has quit IRC23:37
*** Mosibi has quit IRC23:37
*** Hunner has quit IRC23:37
*** reynir has quit IRC23:37
*** rhefner has quit IRC23:37
*** kgalanov has quit IRC23:37
*** mdorman_ is now known as mdorman23:37
*** clayton_ is now known as clayton23:37
*** EmilienM_ is now known as EmilienM23:37
*** itsuugo_ is now known as itsuugo23:37
*** EmilienM is now known as Guest2883623:37
*** rhefner_ is now known as rhefner23:38
*** kgalanov_ is now known as kgalanov23:38
*** Hunner has joined #puppet-openstack23:38
*** Hunner has quit IRC23:38
*** Hunner has joined #puppet-openstack23:38
Guest28836crinkle: CI still does not pass on https://review.openstack.org/#/c/280350/ -- while it worked in my env... I'm a bit lost23:38
*** mwhahaha_ is now known as mwhahaha23:38
*** reynir1 is now known as reynir23:38
crinkleGuest28836: :'(23:39
Guest28836damn my nick is broken23:39
*** Guest28836 has quit IRC23:41
*** Guest28836 has joined #puppet-openstack23:41
*** Guest28836 is now known as EmilienM23:42
EmilienMcrinkle: it was me ! lol23:42
*** mdavidson has joined #puppet-openstack23:43
*** dalees has joined #puppet-openstack23:47
*** nibalizer has joined #puppet-openstack23:47
*** saneax is now known as saneax_AFK23:47
*** johnmilton has joined #puppet-openstack23:55
*** baker has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!