Tuesday, 2015-08-25

« Monday, 2015-08-24 Index Wednesday, 2015-08-26 »
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - just run CI with current master  https://review.openstack.org/21646800:02
EmilienMgreat, now EPEL is down00:13
*** aimon has joined #puppet-openstack00:33
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/21646100:47
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/21646100:49
*** v1k0d3n has quit IRC00:54
*** Zhang has joined #puppet-openstack00:58
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/21646101:15
*** xarses_ has quit IRC01:16
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/21646101:19
*** dprince has quit IRC01:21
*** xingchao has quit IRC01:25
*** linkedinyou has quit IRC01:25
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/21646101:29
*** linkedinyou has joined #puppet-openstack01:30
*** tiswanso has joined #puppet-openstack01:35
*** asd112z has joined #puppet-openstack01:35
*** tiswanso has quit IRC01:39
*** tiswanso has joined #puppet-openstack01:39
*** xarses_ has joined #puppet-openstack01:41
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_service  https://review.openstack.org/21648801:58
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - CI test  https://review.openstack.org/21648901:59
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_service  https://review.openstack.org/21648802:00
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641802:10
*** richm has quit IRC02:16
*** tiswanso has quit IRC02:24
*** tiswanso has joined #puppet-openstack02:25
*** angdraug has quit IRC02:34
*** tiswanso has quit IRC02:36
*** tiswanso has joined #puppet-openstack02:37
*** v1k0d3n has joined #puppet-openstack02:55
*** tiswanso has quit IRC02:58
*** tiswanso has joined #puppet-openstack02:59
*** v1k0d3n has quit IRC03:00
*** v1k0d3n has joined #puppet-openstack03:05
*** tiswanso has quit IRC03:10
*** tiswanso has joined #puppet-openstack03:11
*** tiswanso has quit IRC03:19
*** aimon_ has joined #puppet-openstack03:19
*** aimon has quit IRC03:22
*** asd112z has quit IRC03:31
*** saneax has joined #puppet-openstack04:26
*** sergmelikyan has joined #puppet-openstack04:32
openstackgerritColleen Murphy proposed openstack/puppet-neutron: Fix neutron_agent_linuxbridge file path  https://review.openstack.org/21652304:37
openstackgerritliaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/21187204:44
*** Zhang has quit IRC04:47
*** Zhang has joined #puppet-openstack04:48
*** Zhang has quit IRC04:51
*** Zhang has joined #puppet-openstack04:51
*** sergmelikyan has quit IRC05:28
*** asd112z has joined #puppet-openstack05:32
*** asd112z has quit IRC05:51
openstackgerritliaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/21187206:01
*** rhefner has quit IRC06:12
*** simonmcc has quit IRC06:13
*** rhefner has joined #puppet-openstack06:16
*** simonmcc has joined #puppet-openstack06:16
openstackgerritliaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/21187206:39
*** v1k0d3n has quit IRC06:51
*** jpena has joined #puppet-openstack07:11
*** dgurtner has joined #puppet-openstack07:14
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497607:17
*** sergmelikyan has joined #puppet-openstack07:19
*** xingchao has joined #puppet-openstack07:19
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Switch back to port 80 in acceptance test  https://review.openstack.org/21657607:19
*** sergmelikyan has quit IRC07:22
*** linkedinyou has quit IRC07:28
*** stamak has joined #puppet-openstack07:40
*** jtomasek has joined #puppet-openstack07:42
*** paramite has joined #puppet-openstack07:54
*** cgfbee has quit IRC07:57
*** Zhang has quit IRC08:00
*** asd112z has joined #puppet-openstack08:03
*** asd112z has quit IRC08:08
*** cgfbee has joined #puppet-openstack08:12
*** jistr has joined #puppet-openstack08:12
*** Zhang has joined #puppet-openstack08:14
*** derekh has joined #puppet-openstack08:24
*** saneax has quit IRC08:34
*** saneax has joined #puppet-openstack08:36
*** IBerezovskiy has joined #puppet-openstack08:38
*** _nick has quit IRC08:47
*** _nick has joined #puppet-openstack08:49
*** v1k0d3n has joined #puppet-openstack08:52
*** v1k0d3n has quit IRC08:57
*** c0m0 has joined #puppet-openstack09:01
openstackgerritliaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/21187209:11
*** Zhang has quit IRC09:30
*** mjblack has joined #puppet-openstack09:32
openstackgerritMichael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/21664109:47
openstackgerritMichael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/21664109:54
*** Zhang has joined #puppet-openstack10:05
*** Zhang has quit IRC10:11
*** Zhang has joined #puppet-openstack10:13
*** Zhang has quit IRC10:15
*** saneax has quit IRC10:17
*** saneax has joined #puppet-openstack10:19
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/21104310:24
*** Zhang has joined #puppet-openstack10:26
*** Zhang has quit IRC10:27
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665410:33
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/21104310:33
*** chem has joined #puppet-openstack10:35
*** Zhang has joined #puppet-openstack10:38
*** Zhang has quit IRC10:39
*** chem has quit IRC10:42
*** Zhang has joined #puppet-openstack10:42
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/21104310:48
*** v1k0d3n has joined #puppet-openstack10:53
*** Zhang has quit IRC10:56
openstackgerritMichael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/21664110:57
*** v1k0d3n has quit IRC10:58
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/21104311:00
*** asd112z has joined #puppet-openstack11:03
*** Zhang has joined #puppet-openstack11:03
*** chem has joined #puppet-openstack11:03
*** asd112z has quit IRC11:08
*** Zhang has quit IRC11:09
*** Zhang has joined #puppet-openstack11:10
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/21104311:17
*** dprince has joined #puppet-openstack11:19
*** simon-AS559 has joined #puppet-openstack11:36
*** Zhang has quit IRC11:37
*** Zhang has joined #puppet-openstack11:39
*** mjblack_ has joined #puppet-openstack11:57
*** mjblack_ has quit IRC11:58
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087712:02
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/21668212:15
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/21668212:16
*** Zhang has quit IRC12:16
*** ducttape_ has joined #puppet-openstack12:17
*** Zhang has joined #puppet-openstack12:19
*** jpena is now known as jpena|lunch12:20
EmilienMgood morning12:21
*** prad_ has joined #puppet-openstack12:22
*** prad_ is now known as pradk12:24
*** ChanServ sets mode: +o EmilienM12:28
*** ducttape_ has quit IRC12:30
*** chandankumar has joined #puppet-openstack12:32
*** v1k0d3n has joined #puppet-openstack12:38
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087712:43
*** iurygregory has quit IRC12:45
*** iurygregory has joined #puppet-openstack12:48
*** dgurtner has quit IRC12:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: DNM - debug LP#1487645  https://review.openstack.org/21669412:57
*** dgurtner has joined #puppet-openstack13:00
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/21668213:01
*** iurygregory has quit IRC13:01
*** Zhang has quit IRC13:06
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268913:10
*** jayg|g0n3 is now known as jayg13:11
EmilienMchem: I'm looking https://review.openstack.org/#/c/202689/33/spec/acceptance/keystone_wsgi_apache_spec.rb,cm and I think you can drop 'kilo'13:12
chemEmilienM: oki, I'll have another look.  I just resolved all the conflicts with master and noticed that this have been dropped.13:13
EmilienMchem: which code has been dropped?13:13
chemEmilienM: relative the the version13:14
chemis the rspec13:14
*** richm has joined #puppet-openstack13:15
EmilienMchem: tell me if I'm wrong but "release = 'kilo'" was not in master, you introduced it13:15
*** saneax has quit IRC13:16
chemEmilienM: in master you had: release => 'kilo' for the class openstack_extras ....13:18
EmilienMah in openstack_extras yeah13:18
chemEmilienM: this has been dropped13:18
EmilienMwe manually configure the repo now, because openstack_extras does not support trunk repos13:19
EmilienMit could be a nice feature I presume13:19
*** Zhang has joined #puppet-openstack13:19
chemEmilienM: humm ... noted :)13:19
chemEmilienM: anyway, emacs tells me that the "release" variable is unused.  Dropping ... patch 34 coming13:21
*** Zhang has quit IRC13:22
*** dgurtner has quit IRC13:22
*** dgurtner has joined #puppet-openstack13:22
*** dgurtner has joined #puppet-openstack13:22
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268913:23
EmilienMchem: while you were in holidays, you might have seen we bumped our CI to liberty13:25
EmilienMexcept for heat and ironic, there are still broken ubuntu packages13:25
chemEmilienM: yes, I noticed, but it shouldn't matter for this patch13:25
chemEmilienM: but well done by the way.13:25
*** jpena|lunch is now known as jpena13:26
EmilienMchem: well, we still have issues13:26
chemEmilienM: If I understood well you have a way to test a upgrade from kilo to liberty, no ?13:26
*** ferest has joined #puppet-openstack13:26
EmilienMyes13:27
EmilienMchem: https://review.openstack.org/21594313:27
EmilienMchem: see the patch13:27
EmilienMchem: it will install kilo, run beaker, then update manifest to liberty and run beaker again13:28
EmilienMit's experimental now13:28
*** paramite is now known as paramite|afk13:28
chemI'll have a look13:29
*** ferest has quit IRC13:31
*** Zhang has joined #puppet-openstack13:31
jpenahi everyone, I'd like to get some feedback on a rspec issue I'm finding with https://review.openstack.org/211549 (see comment a) in patch set 6). Adam has been helping me through the review, and the only remaining issue is whether we should use autorequire in the custom puppet type (with the associated headache in the rspec test), or go back and use a explicit requirement just for that one.13:31
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/21668213:32
*** Zhang has quit IRC13:38
*** paramite|afk is now known as paramite13:42
*** Zhang has joined #puppet-openstack13:45
*** dfisher has joined #puppet-openstack13:45
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/21668213:47
*** bapalm_ is now known as bapalm13:51
*** ferest has joined #puppet-openstack13:53
*** ducttape_ has joined #puppet-openstack14:01
*** chandankumar has quit IRC14:06
*** iurygregory has joined #puppet-openstack14:08
*** ferest has quit IRC14:11
socialjpena: I'd go back and use explicit requirement as the code will be cleaner and just a bit less hacky14:11
socialjpena: about the issue of order these are present all around openstack puppet modules another one won't matter but just mention it in code14:12
*** Zhang has quit IRC14:12
jpenasocial: the ordering issue is now fixed, I'm just worried about the explicit/autorequirement. My personal preference would be to use a explicit requirement, but I'd rather have some feedback instead of getting -1's when I send the review ;)14:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: DNM - debug LP#1487645  https://review.openstack.org/21669414:16
*** linkedinyou has joined #puppet-openstack14:17
*** paramite is now known as paramite|afk14:21
*** saneax has joined #puppet-openstack14:23
*** paramite|afk is now known as paramite14:23
*** linkedinyou has quit IRC14:27
*** chem has quit IRC14:28
*** linkedinyou has joined #puppet-openstack14:29
*** asd112z has joined #puppet-openstack14:32
*** asd112z has quit IRC14:33
*** asd112z has joined #puppet-openstack14:33
*** asd112z has quit IRC14:34
*** asd112z has joined #puppet-openstack14:34
*** xingchao_ has joined #puppet-openstack14:35
EmilienMwe have our weekly meeting in ~20 min - please look at our agenda https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2015082514:39
jpenaEmillienM: do you mind if I add my question about the puppet-swift review to the agenda?14:39
*** aimon has joined #puppet-openstack14:39
EmilienMjpena: sure14:39
*** chem has joined #puppet-openstack14:40
*** aimon_ has quit IRC14:43
*** chem has quit IRC14:45
*** dgurtner has quit IRC14:48
*** xingchao_ has quit IRC14:50
*** xingchao_ has joined #puppet-openstack14:50
*** pabelanger has joined #puppet-openstack14:50
EmilienMcrinkle: I had a long discussion on #openstack-glance - so to summarize, your bug report is valid and only affects 0.19.0 - they are thinking at doing a release with the fix that is probably in master since RDO works. That might help ubuntu to build a new package from this newest version and fix puppet glance CI14:51
EmilienMcrinkle: in the meantime, I found https://launchpad.net/bugs/1488277 last night & proposed https://review.openstack.org/#/c/216488/14:52
openstackLaunchpad bug 1488277 in puppet-glance "glance_image provider: orchestration issue when running keystone wsgi" [Undecided,In progress] - Assigned to Emilien Macchi (emilienm)14:52
*** angdraug has joined #puppet-openstack14:54
crinkleEmilienM: jenkins is :( ?14:54
EmilienMcrinkle: for trusty14:55
EmilienMbecause of your bug14:55
EmilienMbecause of 0.19.014:55
EmilienMbut, I found a second issue last night, with is LP #148827714:55
openstackLaunchpad bug 1488277 in puppet-glance "glance_image provider: orchestration issue when running keystone wsgi" [Undecided,In progress] https://launchpad.net/bugs/1488277 - Assigned to Emilien Macchi (emilienm)14:55
crinkleoh :)14:56
EmilienMcrinkle: yeah14:56
EmilienMcrinkle: my fix is maybe not the right way? but I think it's ok. I was wondering if it's OK if there is no Glance_image resource in the catalog but I tested and it works even without14:56
*** chem has joined #puppet-openstack14:58
*** skolekonov has joined #puppet-openstack15:00
*** xarses_ is now known as xarses15:03
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Add dead_retry configuration option  https://review.openstack.org/21676715:23
*** Zhang has joined #puppet-openstack15:25
*** dgurtner has joined #puppet-openstack15:29
*** Zhang has quit IRC15:32
*** Zhang has joined #puppet-openstack15:36
*** chandankumar has joined #puppet-openstack15:37
*** _nick has quit IRC15:39
openstackgerritAleksandr Didenko proposed openstack/puppet-swift: Add support for DLO configuration.  https://review.openstack.org/17202115:40
*** _nick has joined #puppet-openstack15:44
*** samuelBartel has quit IRC15:48
jpenaok, we ran out of time in the meeting, so I'll fire my questions here :)16:01
jpenaIt's about https://review.openstack.org/211549 (see comment a) in patch set 6). In short, to get the new custom type to autorequire the package for object-expirer (different in Debian and RedHat), I had to use facter inside the custom type. So far so good, but it forces me to use http://www.rubydoc.info/gems/puppet/Puppet%2FMetaType%2FManager%3Armtype in the spec test, which I find a bit dirty16:01
EmilienMyou can also use something like chain arrows?16:02
jpenaSo the options are: 1- use this rmtype method, if it's ok, or 2- use an explicit requirement, unlike the rest of the module16:02
jpenaoriginally I was using an explicit requirement, but then https://review.openstack.org/#/c/211929/ was merged16:03
degorenkojust a minor issue: https://review.openstack.org/205988 :) this is a cherry-pick from master, which already merged.16:04
mfischEmilienM: looking at bugs for the midcycle, is this still an issue? I thought we switched to a different mirror: https://bugs.launchpad.net/puppet-cinder/+bug/146973016:05
openstackLaunchpad bug 1469730 in puppet-neutron "EPEL mirror issues" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)16:05
EmilienMmfisch: no16:05
EmilienMyou can close it16:05
mfischEmilienM: for all modules right?16:05
EmilienMyeah16:05
EmilienMit's a nofix16:05
mfischthx16:05
skolekonovGuys, from my side, please look at these two patches https://review.openstack.org/#/c/204152 https://review.openstack.org/#/c/207088/16:06
skolekonovThey have been on review for quite long16:06
*** Zhang has quit IRC16:09
EmilienMskolekonov: looking16:09
skolekonovEmilienM, thanks16:09
angdraugcinder/rbd is one more item we didn't have time for in the meeting: https://review.openstack.org/#/c/209412/6/manifests/backend/rbd.pp16:12
EmilienMjpena: is https://review.openstack.org/#/c/211929/ an issue for your use case?16:12
*** zacdev has joined #puppet-openstack16:13
*** zacdev has quit IRC16:14
*** zacdev has joined #puppet-openstack16:15
*** zacdev has quit IRC16:15
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Add additional memcache configuration options  https://review.openstack.org/21676716:15
*** aimon has quit IRC16:16
*** jistr has quit IRC16:17
EmilienMskolekonov: https://review.openstack.org/#/c/204152/ is not backward compatible16:17
EmilienMyou would need to send a warning to say it will check for True in the next release but now, it accepts both16:17
EmilienMI think16:17
EmilienMbecause it will make our users unhappy to see that kind of change16:17
skolekonovEmilienM, users can still set the value to true and false, why does it break compatibility?16:18
EmilienMoh really? let me read again16:19
skolekonovIt just adds stricter check16:19
EmilienMoh right16:19
EmilienMmy bad16:19
*** vsaienko has quit IRC16:20
EmilienMskolekonov: reviewed16:20
degorenkoEmilienM, mfisch can you review please https://review.openstack.org/198744 - has +1 from Sahara Core and Sahara PTL16:20
skolekonovEmilienM, it's ok, great, thanks again16:20
EmilienMskolekonov: you guys send a lot of patches - it's hard to catch-up :)16:21
EmilienMbut it's cool, please continue that way16:21
EmilienMangdraug: ^16:21
angdraug:)16:22
*** dgurtner has quit IRC16:22
mfischdegorenko: will look in a bit sure16:22
*** xarses has quit IRC16:22
mfischwhat's yanis's handle again here?16:23
degorenkomfisch, he suggested use hiera16:23
EmilienMmfisch: spredzy - but he's on PTO16:23
EmilienMmfisch: iirc he comes back in 2w16:24
mfischok I was wondering if he dropped his -1 and looks like he did16:24
mfischof course August in France nobody works ;)16:24
EmilienMlol16:24
degorenkoyes, he dropped his -1 :)16:24
EmilienMwhy people always say France does not work in summer? :P16:24
*** fedexo has joined #puppet-openstack16:24
angdraugbecause in US people typically only get 2 weeks of vacation per year, so we're jealous :)16:25
EmilienMtrue, french people are lucky16:25
mfischand we get guilted if we take time off16:25
jpenaEmilienM: well, yes and no. It doesn't interfere with what I'm doing, but if I wanted to keep consistent with what it does I had to do some nasty hacks16:25
EmilienMangdraug: I have less holidays now I live in canada btw :P16:25
EmilienMjpena: your issue is also maybe some packaging issue?16:26
jpenaEmilienM: yep, Debian has its own package for object-expirer, but RedHat puts it in the swift-proxy package16:26
EmilienMpackaging... :)16:27
vinsh_Hello jpena16:27
openstackgerritMerged openstack/puppet-sahara: Update parameters for Sahara  https://review.openstack.org/20598816:27
jpenaHi vinsh_16:28
EmilienMmwhahaha: thx for https://review.openstack.org/#/c/21676716:29
mwhahahanp16:29
mfischdegorenko: a few spacing issues in comments but no reason to hold it up16:30
degorenkomfisch, i think so :D16:30
mfischmerged16:30
degorenkomfisch, thank you so much16:30
vinsh_jpena: Depending on how your expire change progresses... I may end up pulling it into: https://review.openstack.org/#/c/203220/16:31
*** vinsh_ is now known as Vinsh16:31
*** stamak has quit IRC16:31
*** rochaporto has joined #puppet-openstack16:31
*** rochaporto has quit IRC16:31
Vinshjpena: what do you think about 203220 btw? as a swift guy16:31
degorenkoalso, sorry for flame, i want also discuss way of implementation for Sahara distributed mode https://review.openstack.org/192721 For now it is outdated, but i'm going to update it16:32
degorenkoand i want to know the best way for it16:32
*** paramite has quit IRC16:32
*** sergmelikyan has joined #puppet-openstack16:33
*** fedexo has quit IRC16:33
*** skolekonov is now known as _skolekonov16:36
jpenaVinsh: I'm not a swift expert, just know the basics :). But, are you trying to override the package-provided service files? I'm not sure that is a good idea to do by default in the module16:41
*** xarses has joined #puppet-openstack16:43
Vinshjpena: I'm managing swift using swift-init instead of distro service providers.16:44
Vinshjpena: Often times, the package-provided service files are not great.  Also consider the case I mention in the commit message.16:45
Vinshwhen you need to start using a dedicated replication network.  the package provided init files are invalid.16:45
VinshWhit this change, we have control over all of it.16:46
jpenayep, that makes sense. But I'd have it as an option, instead of the default16:46
VinshIn order for the services to start at boot under swift-init, they require the modified boot files.16:47
VinshOther wise they start under init or systemd, which don't know how to gracefully reload or stop them.16:47
openstackgerritMerged openstack/puppet-sahara: Configure rpc options separately from ceilometer notifications  https://review.openstack.org/19874416:48
EmilienMpabelanger, crinkle: integ repo running on puppet-keystone gate: https://jenkins06.openstack.org/job/gate-puppet-openstack-integration-dsvm-centos7/31/console16:48
EmilienM\o/16:48
*** aimon has joined #puppet-openstack16:49
jpenaVinsh: maybe you can try to create a custom provider for the service type, like the one here: https://github.com/openstack/puppet-openstack_extras/tree/master/lib/puppet/provider/service16:50
jpenabtw, gotta go now, family calling :)16:50
*** jpena is now known as jpena|away16:50
Vinshthat change IS the custom provider16:51
*** simon-AS559 has quit IRC16:55
*** c0m0 has quit IRC16:59
openstackgerritRichard Raseley proposed openstack/puppet-zaqar: puppet-zaqar: Initial commit  https://review.openstack.org/20422516:59
*** derekh has quit IRC16:59
xarsesmichchap: can I help you to understand the need behind https://review.openstack.org/#/c/209412/6/manifests/backend/rbd.pp ?17:03
*** chandankumar has quit IRC17:03
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641817:07
*** v1k0d3n has quit IRC17:11
*** _ody has quit IRC17:14
*** ferest has joined #puppet-openstack17:19
*** simon-AS559 has joined #puppet-openstack17:20
*** simon-AS5591 has joined #puppet-openstack17:21
Madkisswhen is the bug triage meeting?17:21
Madkissi'm really eager to hear some feedback on https://bugs.launchpad.net/puppet-keystone/+bug/148801817:22
openstackLaunchpad bug 1488018 in puppet-keystone "Does not work if v3 API variables are set in the environment" [Undecided,New]17:22
EmilienMMadkiss: it was earlier17:22
EmilienMrichm: ^17:22
crinkleMadkiss: I've been concerned about that too17:24
crinklethe problem is https://review.openstack.org/#/c/178456/ I think17:24
*** simon-AS559 has quit IRC17:24
Madkiss"problem"?17:25
mtreinishEmilienM: one thing I just thougt of with those tempest runs is you're using ncpu workers, that'll likely cause issues when you enable nova and cinder17:26
*** _ody has joined #puppet-openstack17:27
mtreinishbecause on a single node the cpu and memory pressure from running everything with 8 test workers tends to make things go funny17:27
mtreinishthat's why for the devstack tests we only use 4 workers17:27
*** ferest has quit IRC17:28
EmilienMmtreinish: http://git.openstack.org/cgit/openstack/puppet-nova/tree/manifests/api.pp#n16517:29
EmilienMso what you say is, I'll have to override the default puppet parameter to make sue we use 4 because AFIK dsvm have 8 CPU17:29
*** stamak has joined #puppet-openstack17:30
mtreinishEmilienM: heh, well I was talking about tempest's concurrency. But you'll likely have to do it there too17:30
EmilienMmtreinish: oh ok17:30
EmilienMmtreinish: look https://jenkins07.openstack.org/job/gate-puppet-openstack-integration-dsvm-centos7/27/console17:31
EmilienMI'm testing your patch17:31
EmilienMit looks good17:31
mtreinishI actually wrote a blog post about debugging the api workers eating all the memory: http://blog.kortar.org/?p=12717:31
mtreinishEmilienM: awesome17:31
EmilienMmtreinish: your tempest patch WORKS !17:31
EmilienMI got glance working & tempest happy17:31
pabelangerEmilienM: see my comments for run_tests.sh: https://review.openstack.org/#/c/216418/6/run_tests.sh we should not be hacking that code it. If Depends-On is now working, we need to fix it so zuul-cloner works as expected17:32
EmilienMpabelanger: I thought about that, but we use puppet to checkout the repo, I'm not sure Depends-On will work17:32
EmilienMI'm probably wrong17:33
pabelangerno, we have tempest local17:33
pabelangerin /opt17:33
pabelangerso, zuul-cloner should work17:33
EmilienMok cool !17:33
pabelangerif not, then it is a bug17:33
pabelangerand we need to fix it17:33
EmilienMpabelanger: thanks17:33
*** colby_ has left #puppet-openstack17:33
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641817:34
EmilienMpabelanger: done17:34
Vinshjpena|away: Damnit. You got me... I'll have to convert my change to a type/provider.  It's just better that way.17:35
EmilienMmtreinish: which tempest params are for concurrency?17:35
EmilienMmtreinish: I mean, which ones should I take care?17:35
EmilienMI continue to push on openstack-integration following http://governance.openstack.org/reference/tags/starter-kit_compute.html17:37
richmMadkiss: I think this is https://review.openstack.org/#/c/213957/17:37
EmilienMso next steps are nova & neutron17:37
Madkissrichm: ahum, okay17:38
pabelangerEmilienM: why is this needed? https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L1817:41
pabelangerEmilienM: also, sudo rm -rf is super dangourous IMO. https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules.sh#L4117:43
pabelangercan we change this?17:43
EmilienMpabelanger: I remember having ENV issues because ZUUL_ were not usable in sudo17:43
EmilienMand using sudo -E solved this17:44
EmilienMof course it's workaround17:44
pabelangerHmm17:44
pabelangerdidn't have that issue17:44
pabelangerwas this a beaker issue?17:44
EmilienMfor "rm -rf", the way we did it is safe17:44
EmilienMpabelanger: no, beaker does not have this issue because it does not use sudo17:44
EmilienMpabelanger: probably it's a mistake of me, you can submit a patch that drop the -E option and see how CI reacts17:45
EmilienMfor rm -rf, we have issues with inifile last week17:45
EmilienMand crinkle suggested to cleanup (like we also do in beaker jobs) /etc/puppet/modules/รน17:45
crinkle(for some reason we don't seem to need that anymore)17:45
pabelangerright, that was the main reason for sandboxing the modules into some new directory, so we didn't have to battle existing modules on the system from -infra POV.  I still think into .modules is the best way to avoid these issues17:46
EmilienMpabelanger: so you can submit a patch that cleanup both points17:46
EmilienMand see how CI works17:46
EmilienMcrinkle: why do we wanted modules in /etc/puppet/modules? I don't exactly recall...17:48
crinkleit's the normal place to install them17:48
pabelangerif beaker needs it, I will keep it17:48
crinkleif we don't put them there we either have to set MODULEPATH or muck with puppet.conf17:48
pabelangerbut want puppet-openstack-integration to sandbox them into something other then /etc/puppet/modules17:48
crinkleyeah I'm also not sure how beaker reacts to it or how to set it to a different path17:48
crinkleis there a specific problem that you are currently seeing or foresee with keeping them in /etc/puppet/modules?17:49
crinkleseems to me like pre-cleaning by rm'ing and then installing as normal is the simplest and most debuggable17:50
pabelanger1st issue, we'll always conflict with -infra since they dump modules there. We can work around it but adding sudo rm -rf, but I don't want that in a script. Dangerous, and no need to blindly delete things on a system.   2nd, puppet gives us the ability to set up the modulepath from command-line, we can just use it.  3rd, we shouldn't contaminate /etc/puppet folder outside of puppet runs. 4th, using17:53
pabelanger--noop mode, I don't want to require sudo17:53
crinklepabelanger: what if we had the script accept a customizeable path that defaults to /etc/puppet/modules?17:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/21682117:55
pabelangercrinkle: that is what I am doing now.  Adding the ability to pass the module path into install_modules.sh17:55
pabelangerthat is what we do today in -infra17:55
crinklepabelanger: okay +117:55
pabelangerif a folder needs to be deleted, then we should be adding it into jenkins jobs builder which I have a patch for17:57
EmilienMcool17:57
pabelangerhttps://review.openstack.org/#/c/208088/17:57
pabelangerso, this is my reason for sandboxing outside of /etc/puppet right now17:57
pabelangerbecause, -infra needs work17:57
pabelangerand, no need to add logic into our scripts to work around it, when we can just install into a different directory17:58
*** sergmelikyan has quit IRC18:05
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/20718018:05
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/20717918:05
*** sergmelikyan has joined #puppet-openstack18:08
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Add additional memcache configuration options  https://review.openstack.org/21676718:12
pabelangerEmilienM: why did you remove all the export logic from run_tests.sh?18:15
myatsenkomichchap: hello, Michael. There are additional comments in CR https://review.openstack.org/#/c/20941218:15
myatsenkoPlease take a look at it.18:15
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: glance_id_setter: execute after creating Glance image  https://review.openstack.org/21643218:15
pabelangerEmilienM: specifically, https://github.com/openstack/puppet-openstack-integration/commit/2921e36c790dd255b560f322f9444bd6d30be5f6#diff-c08082f308f76bbde9882b88503605beR1618:16
EmilienMpabelanger: because we don't use it18:16
pabelangersure we do18:16
pabelangerwell, I do18:16
pabelangerand our gate expects it18:17
EmilienMin the initial structure, we did not18:17
EmilienMwe're installing r10k from install_modules18:17
EmilienMcrinkle suggested me to drop that code for now18:17
EmilienMI guess we can add it in further iterations if you need it18:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ  https://review.openstack.org/21682818:23
*** xingchao_ has quit IRC18:29
*** xingchao_ has joined #puppet-openstack18:31
*** xingchao_ has quit IRC18:36
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron  https://review.openstack.org/21683118:38
openstackgerritPaul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/21683218:39
EmilienMpabelanger: Depends-On did not work: https://jenkins02.openstack.org/job/gate-puppet-openstack-integration-dsvm-centos7/28/console18:42
EmilienMpabelanger: for tempest patch18:42
pabelangerEmilienM: then we have a bug in our module18:42
EmilienMpabelanger: in puppet-tempest?18:42
pabelangerin how we clone modules18:43
EmilienMit's using vsc_repo provider18:43
EmilienMso it's not a bug I would say18:43
EmilienMit's a feature we want18:43
pabelangerno18:43
pabelangerwhat is the vcs_repo pointing to?18:43
EmilienMpabelanger: local dir18:43
EmilienMlocal dir is dynamic?18:43
pabelangerright, so you should not be changing the version of that directory. Is puppet-tempest checking out head?18:44
mtreinishEmilienM: it's a test runner param, so before the regexes you pass into tox you'd use --concurrency 4 or something like that18:44
mtreinishI'd check devstack-gate to see how it does it18:45
pabelangerpuppet-tempest should not be doing any git functions, that is what zuul-cloner is for.18:45
pabelangerneed to check puppet-tempest18:45
EmilienMI'm looking18:45
pabelangerbut we should just use a local directory for tempest, nothing related to git18:45
*** ricardo has joined #puppet-openstack18:46
*** ricardo is now known as Guest5874918:46
pabelangerunless you pass the zuul commit into tempest_repo_revision, it won't work18:47
pabelangerso, we have 2 options18:47
pabelangeradd zuul info into puppet-tempest via a params.  Or remove vcsrepo support from puppet-tempest and just a local directory18:47
EmilienMpabelanger: https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/master/lib/puppet/provider/vcsrepo/git.rb#L6118:47
EmilienMHEAD by default18:47
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/21682118:48
pabelangerRight, I say we you add logic into puppet-tempest not to manage git and use a local directory18:48
pabelangerthen zuul works as expected18:48
*** Guest58749 has quit IRC18:49
*** sergmelikyan has quit IRC18:49
*** ricardo_ has joined #puppet-openstack18:49
EmilienMpabelanger: I think I have another option much easier18:49
EmilienMI brought support for tempest is not installed from source18:49
EmilienMbecause I used to run rpms18:49
EmilienMI think we should enable this case for us18:49
pabelangerenable RPM?18:50
EmilienMno18:50
EmilienMsay "do not git clone, here is the tempest path'18:50
pabelangerright18:50
EmilienMwe can already do that iirc18:50
EmilienMlet me check the code again18:50
*** chandankumar has joined #puppet-openstack18:51
*** jpena|away is now known as jpena18:53
*** ricardo_ has quit IRC18:56
EmilienMpabelanger: who owns /opt/git/openstack/tempest ? root?19:02
pabelangerlikely19:03
pabelangeryou can check nodepool.o.o for the log files on how the images are built19:03
EmilienMoh right19:04
*** ducttape_ has quit IRC19:05
EmilienMpermissions are not really easy to find19:05
EmilienMjenkins:jenkins19:06
EmilienMsweet19:06
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: allow to optionally git clone tempest  https://review.openstack.org/21684119:09
*** paramite has joined #puppet-openstack19:10
openstackgerritPaul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/21683219:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/21684419:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641819:15
EmilienMpabelanger: done! it should work with Depends-On now :)19:15
EmilienMthanks for your suggestions!19:15
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ  https://review.openstack.org/21682819:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron  https://review.openstack.org/21683119:16
*** saneax has quit IRC19:17
*** jpena has quit IRC19:21
*** ducttape_ has joined #puppet-openstack19:23
EmilienMrichm, chem, iurygregory: I would like to highlight new jobs for puppet-keystone and puppet-openstacklib, non voting though but we might need you carefully check if green or not: gate-puppet-openstack-integration-dsvm-*19:24
EmilienMthey deploy keystone, etc.. and run tempest19:24
iurygregoryok EmilienM ^^19:24
*** paramite has quit IRC19:27
openstackgerritPaul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/21683219:32
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: allow to optionally git clone tempest  https://review.openstack.org/21684119:35
*** stamak has quit IRC19:36
*** ricardo has joined #puppet-openstack19:44
*** ricardo is now known as Guest8325219:44
*** Guest83252 has quit IRC19:49
*** ricardo_ has joined #puppet-openstack19:49
*** sergmelikyan has joined #puppet-openstack19:50
*** sergmelikyan has quit IRC19:54
*** chandankumar has quit IRC19:56
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swift_service defined type  https://review.openstack.org/20322019:59
*** paramite has joined #puppet-openstack20:02
*** paramite has quit IRC20:03
*** sergmelikyan has joined #puppet-openstack20:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/21684420:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641820:20
*** sergmelikyan has quit IRC20:22
*** jtomasek has quit IRC20:27
mfischEmilienM: would like to discuss the bug triage task for the midcycle20:51
EmilienMmfisch: sure - you just need to know I'm about to lead openstack cross project meeting on #openstack-meeting - i might be busy and slow to reply here20:51
mfischEmilienM: sure we can wait20:52
*** jayg is now known as jayg|g0n320:52
*** sergmelikyan has joined #puppet-openstack20:54
*** simon-AS5591 has quit IRC20:55
iurygregoryanyone who wants to review https://review.openstack.org/#/c/208054/  https://review.openstack.org/#/c/216821/  ^^20:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/21684420:59
*** ricardo has joined #puppet-openstack21:05
*** ricardo is now known as Guest5903121:05
*** Guest59031 has quit IRC21:06
*** rochaporto has joined #puppet-openstack21:07
*** rochaporto has quit IRC21:11
*** rochaporto has joined #puppet-openstack21:11
*** rochaporto has quit IRC21:18
*** rochaporto has joined #puppet-openstack21:18
*** dprince has quit IRC21:19
*** rochaporto has quit IRC21:19
*** rochaporto has joined #puppet-openstack21:19
*** rochaporto has quit IRC21:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/21684421:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641821:26
EmilienMheh! puppet-manila gate is back again21:28
* EmilienM updates https://etherpad.openstack.org/p/puppet-liberty-blocker21:28
crinkleEmilienM: is there a bug report for everything in that etherpad?21:30
crinklethe neutron one is :( for me21:31
*** xingchao_ has joined #puppet-openstack21:31
EmilienMcrinkle: no21:31
EmilienMshame on me.21:31
EmilienMthe fact is, where to report?21:32
EmilienMfor glanceclient, it's in LP, yes21:32
EmilienMbut heat and ironic no21:32
EmilienMand for neutron... no it's RDO21:32
EmilienMbut they are working on it I guess21:32
EmilienMcrinkle: are you running liberty?21:32
EmilienMoh yeah I've seen your patch about new LB config file path21:33
crinkleEmilienM: no just puppet-neutron patches can't get landed21:33
EmilienMyes :(21:33
crinklebecause db-sync is broken in rdo21:33
*** v1k0d3n has joined #puppet-openstack21:33
EmilienMcrinkle: yes, something with alembic21:35
EmilienMiirc21:35
mfischEmilienM: you back?21:36
*** sanjayu_ has joined #puppet-openstack21:38
EmilienMmfisch: more or less21:39
EmilienMmfisch: go ahead!21:40
mfischlol21:40
mfischok so we dont really have that many bugs that need to be triaged21:40
mfischwe could instead work on some of them21:40
mfischor a bit of both21:40
mfischor triage will be quick21:40
*** sanjayu has quit IRC21:41
EmilienMmfisch: afik, fuel folks have quite a good number of bugs in fuel that also affect puppet-*21:41
mfischthats true, I didnt think of that21:41
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/21684421:45
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641821:45
*** pradk has quit IRC21:53
*** xingchao_ has quit IRC21:58
*** angdraug has quit IRC22:09
*** aimon has quit IRC22:23
EmilienMpabelanger: there is a bug in nodepool image22:58
EmilienMpabelanger: the Depends-On does not work22:58
EmilienMand I'm using /opt/...22:58
*** asd112z has quit IRC22:59
*** dfisher has quit IRC23:00
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/21684423:28
*** bitblt has joined #puppet-openstack23:28
*** bitblt has quit IRC23:28
*** ducttape_ has quit IRC23:40
*** gildub has joined #puppet-openstack23:48
*** xingchao_ has joined #puppet-openstack23:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/21641823:58
« Monday, 2015-08-24 Index Wednesday, 2015-08-26 »

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