Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-08-25

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:02 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - just run CI with current master  https://review.openstack.org/216468
00:13 EmilienM great, now EPEL is down
00:33 aimon joined #puppet-openstack
00:47 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/216461
00:49 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/216461
00:58 Zhang joined #puppet-openstack
01:15 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/216461
01:19 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/216461
01:29 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - run acceptance with keystone/wsgi  https://review.openstack.org/216461
01:30 linkedinyou joined #puppet-openstack
01:35 tiswanso joined #puppet-openstack
01:35 asd112z joined #puppet-openstack
01:39 tiswanso joined #puppet-openstack
01:41 xarses_ joined #puppet-openstack
01:58 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_service  https://review.openstack.org/216488
01:59 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - CI test  https://review.openstack.org/216489
02:00 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_service  https://review.openstack.org/216488
02:10 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
02:25 tiswanso joined #puppet-openstack
02:37 tiswanso joined #puppet-openstack
02:54 v1k0d3n joined #puppet-openstack
02:59 tiswanso joined #puppet-openstack
03:05 v1k0d3n joined #puppet-openstack
03:11 tiswanso joined #puppet-openstack
03:19 aimon_ joined #puppet-openstack
04:26 saneax joined #puppet-openstack
04:32 sergmelikyan joined #puppet-openstack
04:37 openstackgerrit Colleen Murphy proposed openstack/puppet-neutron: Fix neutron_agent_linuxbridge file path  https://review.openstack.org/216523
04:44 openstackgerrit liaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/211872
04:48 Zhang joined #puppet-openstack
04:51 Zhang joined #puppet-openstack
05:32 asd112z joined #puppet-openstack
06:01 openstackgerrit liaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/211872
06:16 rhefner joined #puppet-openstack
06:16 simonmcc joined #puppet-openstack
06:39 openstackgerrit liaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/211872
07:11 jpena joined #puppet-openstack
07:14 dgurtner joined #puppet-openstack
07:17 openstackgerrit Xingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/214976
07:19 sergmelikyan joined #puppet-openstack
07:19 xingchao joined #puppet-openstack
07:19 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Switch back to port 80 in acceptance test  https://review.openstack.org/216576
07:40 stamak joined #puppet-openstack
07:42 jtomasek joined #puppet-openstack
07:54 paramite joined #puppet-openstack
08:03 asd112z joined #puppet-openstack
08:12 cgfbee joined #puppet-openstack
08:12 jistr joined #puppet-openstack
08:14 Zhang joined #puppet-openstack
08:24 derekh joined #puppet-openstack
08:36 saneax joined #puppet-openstack
08:38 IBerezovskiy joined #puppet-openstack
08:49 _nick joined #puppet-openstack
08:52 v1k0d3n joined #puppet-openstack
09:01 c0m0 joined #puppet-openstack
09:11 openstackgerrit liaonanhai proposed openstack/puppet-nova: Set reserved_host_memory_mb to 0 using ironic  https://review.openstack.org/211872
09:32 mjblack joined #puppet-openstack
09:47 openstackgerrit Michael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/216641
09:54 openstackgerrit Michael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/216641
10:05 Zhang joined #puppet-openstack
10:13 Zhang joined #puppet-openstack
10:19 saneax joined #puppet-openstack
10:24 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/211043
10:26 Zhang joined #puppet-openstack
10:33 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
10:33 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/211043
10:35 chem joined #puppet-openstack
10:38 Zhang joined #puppet-openstack
10:42 Zhang joined #puppet-openstack
10:48 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/211043
10:53 v1k0d3n joined #puppet-openstack
10:57 openstackgerrit Michael Polenchuk proposed openstack/puppet-keystone: Make sure admin token auth middleware is in place  https://review.openstack.org/216641
11:00 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/211043
11:03 asd112z joined #puppet-openstack
11:03 Zhang joined #puppet-openstack
11:03 chem joined #puppet-openstack
11:10 Zhang joined #puppet-openstack
11:17 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation  https://review.openstack.org/211043
11:19 dprince joined #puppet-openstack
11:36 simon-AS559 joined #puppet-openstack
11:39 Zhang joined #puppet-openstack
11:57 mjblack_ joined #puppet-openstack
12:02 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/200877
12:15 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/216682
12:16 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/216682
12:17 ducttape_ joined #puppet-openstack
12:19 Zhang joined #puppet-openstack
12:21 EmilienM good morning
12:22 prad_ joined #puppet-openstack
12:32 chandankumar joined #puppet-openstack
12:38 v1k0d3n joined #puppet-openstack
12:43 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/200877
12:48 iurygregory joined #puppet-openstack
12:57 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: DNM - debug LP#1487645  https://review.openstack.org/216694
13:00 dgurtner joined #puppet-openstack
13:01 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/216682
13:10 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
13:12 EmilienM chem: 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:13 chem EmilienM: oki, I'll have another look.  I just resolved all the conflicts with master and noticed that this have been dropped.
13:13 EmilienM chem: which code has been dropped?
13:14 chem EmilienM: relative the the version
13:14 chem is the rspec
13:15 richm joined #puppet-openstack
13:15 EmilienM chem: tell me if I'm wrong but "release = 'kilo'" was not in master, you introduced it
13:18 chem EmilienM: in master you had: release => 'kilo' for the class openstack_extras ....
13:18 EmilienM ah in openstack_extras yeah
13:18 chem EmilienM: this has been dropped
13:19 EmilienM we manually configure the repo now, because openstack_extras does not support trunk repos
13:19 EmilienM it could be a nice feature I presume
13:19 Zhang joined #puppet-openstack
13:19 chem EmilienM: humm ... noted :)
13:21 chem EmilienM: anyway, emacs tells me that the "release" variable is unused.  Dropping ... patch 34 coming
13:22 dgurtner joined #puppet-openstack
13:22 dgurtner joined #puppet-openstack
13:23 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
13:25 EmilienM chem: while you were in holidays, you might have seen we bumped our CI to liberty
13:25 EmilienM except for heat and ironic, there are still broken ubuntu packages
13:25 chem EmilienM: yes, I noticed, but it shouldn't matter for this patch
13:25 chem EmilienM: but well done by the way.
13:26 EmilienM chem: well, we still have issues
13:26 chem EmilienM: If I understood well you have a way to test a upgrade from kilo to liberty, no ?
13:26 ferest joined #puppet-openstack
13:27 EmilienM yes
13:27 EmilienM chem: https://review.openstack.org/215943
13:27 EmilienM chem: see the patch
13:28 EmilienM chem: it will install kilo, run beaker, then update manifest to liberty and run beaker again
13:28 EmilienM it's experimental now
13:29 chem I'll have a look
13:31 Zhang joined #puppet-openstack
13:31 jpena hi 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:32 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/216682
13:45 Zhang joined #puppet-openstack
13:45 dfisher joined #puppet-openstack
13:47 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation  https://review.openstack.org/216682
13:53 ferest joined #puppet-openstack
14:01 ducttape_ joined #puppet-openstack
14:08 iurygregory joined #puppet-openstack
14:11 social jpena: I'd go back and use explicit requirement as the code will be cleaner and just a bit less hacky
14:12 social jpena: about the issue of order these are present all around openstack puppet modules another one won't matter but just mention it in code
14:13 jpena social: 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:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: DNM - debug LP#1487645  https://review.openstack.org/216694
14:17 linkedinyou joined #puppet-openstack
14:23 saneax joined #puppet-openstack
14:29 linkedinyou joined #puppet-openstack
14:32 asd112z joined #puppet-openstack
14:33 asd112z joined #puppet-openstack
14:34 asd112z joined #puppet-openstack
14:35 xingchao_ joined #puppet-openstack
14:39 EmilienM we have our weekly meeting in ~20 min - please look at our agenda https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20150825
14:39 jpena EmillienM: do you mind if I add my question about the puppet-swift review to the agenda?
14:39 aimon joined #puppet-openstack
14:39 EmilienM jpena: sure
14:40 chem joined #puppet-openstack
14:50 xingchao_ joined #puppet-openstack
14:50 pabelanger joined #puppet-openstack
14:51 EmilienM crinkle: 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 CI
14:52 EmilienM crinkle: in the meantime, I found https://launchpad.net/bugs/1488277 last night & proposed https://review.openstack.org/#/c/216488/
14:52 openstack Launchpad bug 1488277 in puppet-glance "glance_image provider: orchestration issue when running keystone wsgi" [Undecided,In progress] - Assigned to Emilien Macchi (emilienm)
14:54 angdraug joined #puppet-openstack
14:54 crinkle EmilienM: jenkins is :( ?
14:55 EmilienM crinkle: for trusty
14:55 EmilienM because of your bug
14:55 EmilienM because of 0.19.0
14:55 EmilienM but, I found a second issue last night, with is LP #1488277
14:55 openstack Launchpad 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:56 crinkle oh :)
14:56 EmilienM crinkle: yeah
14:56 EmilienM crinkle: 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 without
14:58 chem joined #puppet-openstack
15:00 skolekonov joined #puppet-openstack
15:23 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Add dead_retry configuration option  https://review.openstack.org/216767
15:25 Zhang joined #puppet-openstack
15:29 dgurtner joined #puppet-openstack
15:36 Zhang joined #puppet-openstack
15:37 chandankumar joined #puppet-openstack
15:40 openstackgerrit Aleksandr Didenko proposed openstack/puppet-swift: Add support for DLO configuration.  https://review.openstack.org/172021
15:44 _nick joined #puppet-openstack
16:01 jpena ok, we ran out of time in the meeting, so I'll fire my questions here :)
16:01 jpena It'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 dirty
16:02 EmilienM you can also use something like chain arrows?
16:02 jpena So the options are: 1- use this rmtype method, if it's ok, or 2- use an explicit requirement, unlike the rest of the module
16:03 jpena originally I was using an explicit requirement, but then https://review.openstack.org/#/c/211929/ was merged
16:04 degorenko just a minor issue: https://review.openstack.org/205988 :) this is a cherry-pick from master, which already merged.
16:05 mfisch EmilienM: 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/1469730
16:05 openstack Launchpad bug 1469730 in puppet-neutron "EPEL mirror issues" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)
16:05 EmilienM mfisch: no
16:05 EmilienM you can close it
16:05 mfisch EmilienM: for all modules right?
16:05 EmilienM yeah
16:05 EmilienM it's a nofix
16:05 mfisch thx
16:06 skolekonov Guys, from my side, please look at these two patches https://review.openstack.org/#/c/204152 https://review.openstack.org/#/c/207088/
16:06 skolekonov They have been on review for quite long
16:09 EmilienM skolekonov: looking
16:09 skolekonov EmilienM, thanks
16:12 angdraug cinder/rbd is one more item we didn't have time for in the meeting: https://review.openstack.org/#/c​/209412/6/manifests/backend/rbd.pp
16:12 EmilienM jpena: is https://review.openstack.org/#/c/211929/ an issue for your use case?
16:13 zacdev joined #puppet-openstack
16:15 zacdev joined #puppet-openstack
16:15 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Add additional memcache configuration options  https://review.openstack.org/216767
16:17 EmilienM skolekonov: https://review.openstack.org/#/c/204152/ is not backward compatible
16:17 EmilienM you would need to send a warning to say it will check for True in the next release but now, it accepts both
16:17 EmilienM I think
16:17 EmilienM because it will make our users unhappy to see that kind of change
16:18 skolekonov EmilienM, users can still set the value to true and false, why does it break compatibility?
16:19 EmilienM oh really? let me read again
16:19 skolekonov It just adds stricter check
16:19 EmilienM oh right
16:19 EmilienM my bad
16:20 EmilienM skolekonov: reviewed
16:20 degorenko EmilienM, mfisch can you review please https://review.openstack.org/198744 - has +1 from Sahara Core and Sahara PTL
16:20 skolekonov EmilienM, it's ok, great, thanks again
16:21 EmilienM skolekonov: you guys send a lot of patches - it's hard to catch-up :)
16:21 EmilienM but it's cool, please continue that way
16:21 EmilienM angdraug: ^
16:22 angdraug :)
16:22 mfisch degorenko: will look in a bit sure
16:23 mfisch what's yanis's handle again here?
16:23 degorenko mfisch, he suggested use hiera
16:23 EmilienM mfisch: spredzy - but he's on PTO
16:24 EmilienM mfisch: iirc he comes back in 2w
16:24 mfisch ok I was wondering if he dropped his -1 and looks like he did
16:24 mfisch of course August in France nobody works ;)
16:24 EmilienM lol
16:24 degorenko yes, he dropped his -1 :)
16:24 EmilienM why people always say France does not work in summer? :P
16:24 fedexo joined #puppet-openstack
16:25 angdraug because in US people typically only get 2 weeks of vacation per year, so we're jealous :)
16:25 EmilienM true, french people are lucky
16:25 mfisch and we get guilted if we take time off
16:25 jpena EmilienM: 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 hacks
16:25 EmilienM angdraug: I have less holidays now I live in canada btw :P
16:26 EmilienM jpena: your issue is also maybe some packaging issue?
16:26 jpena EmilienM: yep, Debian has its own package for object-expirer, but RedHat puts it in the swift-proxy package
16:27 EmilienM packaging... :)
16:27 vinsh_ Hello jpena
16:27 openstackgerrit Merged openstack/puppet-sahara: Update parameters for Sahara  https://review.openstack.org/205988
16:28 jpena Hi vinsh_
16:29 EmilienM mwhahaha: thx for https://review.openstack.org/#/c/216767
16:29 mwhahaha np
16:30 mfisch degorenko: a few spacing issues in comments but no reason to hold it up
16:30 degorenko mfisch, i think so :D
16:30 mfisch merged
16:30 degorenko mfisch, thank you so much
16:31 vinsh_ jpena: Depending on how your expire change progresses... I may end up pulling it into: https://review.openstack.org/#/c/203220/
16:31 rochaporto joined #puppet-openstack
16:31 Vinsh jpena: what do you think about 203220 btw? as a swift guy
16:32 degorenko also, 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 it
16:32 degorenko and i want to know the best way for it
16:33 sergmelikyan joined #puppet-openstack
16:41 jpena Vinsh: 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 module
16:43 xarses joined #puppet-openstack
16:44 Vinsh jpena: I'm managing swift using swift-init instead of distro service providers.
16:45 Vinsh jpena: Often times, the package-provided service files are not great.  Also consider the case I mention in the commit message.
16:45 Vinsh when you need to start using a dedicated replication network.  the package provided init files are invalid.
16:46 Vinsh Whit this change, we have control over all of it.
16:46 jpena yep, that makes sense. But I'd have it as an option, instead of the default
16:47 Vinsh In order for the services to start at boot under swift-init, they require the modified boot files.
16:47 Vinsh Other wise they start under init or systemd, which don't know how to gracefully reload or stop them.
16:48 openstackgerrit Merged openstack/puppet-sahara: Configure rpc options separately from ceilometer notifications  https://review.openstack.org/198744
16:48 EmilienM pabelanger, crinkle: integ repo running on puppet-keystone gate: https://jenkins06.openstack.org/job/gate-puppet​-openstack-integration-dsvm-centos7/31/console
16:48 EmilienM \o/
16:49 aimon joined #puppet-openstack
16:50 jpena Vinsh: 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/service
16:50 jpena btw, gotta go now, family calling :)
16:51 Vinsh that change IS the custom provider
16:59 openstackgerrit Richard Raseley proposed openstack/puppet-zaqar: puppet-zaqar: Initial commit  https://review.openstack.org/204225
17:03 xarses michchap: can I help you to understand the need behind https://review.openstack.org/#/c​/209412/6/manifests/backend/rbd.pp ?
17:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
17:19 ferest joined #puppet-openstack
17:20 simon-AS559 joined #puppet-openstack
17:21 simon-AS5591 joined #puppet-openstack
17:21 Madkiss when is the bug triage meeting?
17:22 Madkiss i'm really eager to hear some feedback on https://bugs.launchpad.net/p​uppet-keystone/+bug/1488018
17:22 openstack Launchpad bug 1488018 in puppet-keystone "Does not work if v3 API variables are set in the environment" [Undecided,New]
17:22 EmilienM Madkiss: it was earlier
17:22 EmilienM richm: ^
17:24 crinkle Madkiss: I've been concerned about that too
17:24 crinkle the problem is https://review.openstack.org/#/c/178456/ I think
17:25 Madkiss "problem"?
17:26 mtreinish EmilienM: 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 cinder
17:27 _ody joined #puppet-openstack
17:27 mtreinish because on a single node the cpu and memory pressure from running everything with 8 test workers tends to make things go funny
17:27 mtreinish that's why for the devstack tests we only use 4 workers
17:29 EmilienM mtreinish: http://git.openstack.org/cgit/openstack​/puppet-nova/tree/manifests/api.pp#n165
17:29 EmilienM so what you say is, I'll have to override the default puppet parameter to make sue we use 4 because AFIK dsvm have 8 CPU
17:30 stamak joined #puppet-openstack
17:30 mtreinish EmilienM: heh, well I was talking about tempest's concurrency. But you'll likely have to do it there too
17:30 EmilienM mtreinish: oh ok
17:31 EmilienM mtreinish: look https://jenkins07.openstack.org/job/gate-puppet​-openstack-integration-dsvm-centos7/27/console
17:31 EmilienM I'm testing your patch
17:31 EmilienM it looks good
17:31 mtreinish I actually wrote a blog post about debugging the api workers eating all the memory: http://blog.kortar.org/?p=127
17:31 mtreinish EmilienM: awesome
17:31 EmilienM mtreinish: your tempest patch WORKS !
17:31 EmilienM I got glance working & tempest happy
17:32 pabelanger EmilienM: see my comments for run_tests.sh: https://review.openstack.o​rg/#/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 expected
17:32 EmilienM pabelanger: I thought about that, but we use puppet to checkout the repo, I'm not sure Depends-On will work
17:33 EmilienM I'm probably wrong
17:33 pabelanger no, we have tempest local
17:33 pabelanger in /opt
17:33 pabelanger so, zuul-cloner should work
17:33 EmilienM ok cool !
17:33 pabelanger if not, then it is a bug
17:33 pabelanger and we need to fix it
17:33 EmilienM pabelanger: thanks
17:33 colby_ left #puppet-openstack
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
17:34 EmilienM pabelanger: done
17:35 Vinsh jpena|away: Damnit. You got me... I'll have to convert my change to a type/provider.  It's just better that way.
17:35 EmilienM mtreinish: which tempest params are for concurrency?
17:35 EmilienM mtreinish: I mean, which ones should I take care?
17:37 EmilienM I continue to push on openstack-integration following http://governance.openstack.org/refe​rence/tags/starter-kit_compute.html
17:37 richm Madkiss: I think this is https://review.openstack.org/#/c/213957/
17:37 EmilienM so next steps are nova & neutron
17:38 Madkiss richm: ahum, okay
17:41 pabelanger EmilienM: why is this needed? https://github.com/openstack/puppet-opensta​ck-integration/blob/master/run_tests.sh#L18
17:43 pabelanger EmilienM: also, sudo rm -rf is super dangourous IMO. https://github.com/openstack/puppet-openstack-​integration/blob/master/install_modules.sh#L41
17:43 pabelanger can we change this?
17:43 EmilienM pabelanger: I remember having ENV issues because ZUUL_ were not usable in sudo
17:44 EmilienM and using sudo -E solved this
17:44 EmilienM of course it's workaround
17:44 pabelanger Hmm
17:44 pabelanger didn't have that issue
17:44 pabelanger was this a beaker issue?
17:44 EmilienM for "rm -rf", the way we did it is safe
17:44 EmilienM pabelanger: no, beaker does not have this issue because it does not use sudo
17:45 EmilienM pabelanger: probably it's a mistake of me, you can submit a patch that drop the -E option and see how CI reacts
17:45 EmilienM for rm -rf, we have issues with inifile last week
17:45 EmilienM and 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:46 pabelanger right, 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 issues
17:46 EmilienM pabelanger: so you can submit a patch that cleanup both points
17:46 EmilienM and see how CI works
17:48 EmilienM crinkle: why do we wanted modules in /etc/puppet/modules? I don't exactly recall...
17:48 crinkle it's the normal place to install them
17:48 pabelanger if beaker needs it, I will keep it
17:48 crinkle if we don't put them there we either have to set MODULEPATH or muck with puppet.conf
17:48 pabelanger but want puppet-openstack-integration to sandbox them into something other then /etc/puppet/modules
17:48 crinkle yeah I'm also not sure how beaker reacts to it or how to set it to a different path
17:49 crinkle is there a specific problem that you are currently seeing or foresee with keeping them in /etc/puppet/modules?
17:50 crinkle seems to me like pre-cleaning by rm'ing and then installing as normal is the simplest and most debuggable
17:53 pabelanger 1st 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, using
17:53 pabelanger --noop mode, I don't want to require sudo
17:54 crinkle pabelanger: what if we had the script accept a customizeable path that defaults to /etc/puppet/modules?
17:55 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/216821
17:55 pabelanger crinkle: that is what I am doing now.  Adding the ability to pass the module path into install_modules.sh
17:55 pabelanger that is what we do today in -infra
17:55 crinkle pabelanger: okay +1
17:57 pabelanger if a folder needs to be deleted, then we should be adding it into jenkins jobs builder which I have a patch for
17:57 EmilienM cool
17:57 pabelanger https://review.openstack.org/#/c/208088/
17:57 pabelanger so, this is my reason for sandboxing outside of /etc/puppet right now
17:57 pabelanger because, -infra needs work
17:58 pabelanger and, no need to add logic into our scripts to work around it, when we can just install into a different directory
18:05 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add acceptance tests for repo::* classes  https://review.openstack.org/207180
18:05 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add auth_file acceptance test  https://review.openstack.org/207179
18:08 sergmelikyan joined #puppet-openstack
18:12 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Add additional memcache configuration options  https://review.openstack.org/216767
18:15 pabelanger EmilienM: why did you remove all the export logic from run_tests.sh?
18:15 myatsenko michchap: hello, Michael. There are additional comments in CR https://review.openstack.org/#/c/209412
18:15 myatsenko Please take a look at it.
18:15 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: glance_id_setter: execute after creating Glance image  https://review.openstack.org/216432
18:16 pabelanger EmilienM: specifically, https://github.com/openstack/puppet-openstack-int​egration/commit/2921e36c790dd255b560f322f9444bd6d​30be5f6#diff-c08082f308f76bbde9882b88503605beR16
18:16 EmilienM pabelanger: because we don't use it
18:16 pabelanger sure we do
18:16 pabelanger well, I do
18:17 pabelanger and our gate expects it
18:17 EmilienM in the initial structure, we did not
18:17 EmilienM we're installing r10k from install_modules
18:17 EmilienM crinkle suggested me to drop that code for now
18:17 EmilienM I guess we can add it in further iterations if you need it
18:23 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ  https://review.openstack.org/216828
18:31 xingchao_ joined #puppet-openstack
18:38 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron  https://review.openstack.org/216831
18:39 openstackgerrit Paul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/216832
18:42 EmilienM pabelanger: Depends-On did not work: https://jenkins02.openstack.org/job/gate-puppet​-openstack-integration-dsvm-centos7/28/console
18:42 EmilienM pabelanger: for tempest patch
18:42 pabelanger EmilienM: then we have a bug in our module
18:42 EmilienM pabelanger: in puppet-tempest?
18:43 pabelanger in how we clone modules
18:43 EmilienM it's using vsc_repo provider
18:43 EmilienM so it's not a bug I would say
18:43 EmilienM it's a feature we want
18:43 pabelanger no
18:43 pabelanger what is the vcs_repo pointing to?
18:43 EmilienM pabelanger: local dir
18:43 EmilienM local dir is dynamic?
18:44 pabelanger right, so you should not be changing the version of that directory. Is puppet-tempest checking out head?
18:44 mtreinish EmilienM: it's a test runner param, so before the regexes you pass into tox you'd use --concurrency 4 or something like that
18:45 mtreinish I'd check devstack-gate to see how it does it
18:45 pabelanger puppet-tempest should not be doing any git functions, that is what zuul-cloner is for.
18:45 pabelanger need to check puppet-tempest
18:45 EmilienM I'm looking
18:45 pabelanger but we should just use a local directory for tempest, nothing related to git
18:46 ricardo joined #puppet-openstack
18:47 pabelanger unless you pass the zuul commit into tempest_repo_revision, it won't work
18:47 pabelanger so, we have 2 options
18:47 pabelanger add zuul info into puppet-tempest via a params.  Or remove vcsrepo support from puppet-tempest and just a local directory
18:47 EmilienM pabelanger: https://github.com/puppetlabs/puppetlabs-vcsrepo/b​lob/master/lib/puppet/provider/vcsrepo/git.rb#L61
18:47 EmilienM HEAD by default
18:48 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/216821
18:48 pabelanger Right, I say we you add logic into puppet-tempest not to manage git and use a local directory
18:48 pabelanger then zuul works as expected
18:49 ricardo_ joined #puppet-openstack
18:49 EmilienM pabelanger: I think I have another option much easier
18:49 EmilienM I brought support for tempest is not installed from source
18:49 EmilienM because I used to run rpms
18:49 EmilienM I think we should enable this case for us
18:50 pabelanger enable RPM?
18:50 EmilienM no
18:50 EmilienM say "do not git clone, here is the tempest path'
18:50 pabelanger right
18:50 EmilienM we can already do that iirc
18:50 EmilienM let me check the code again
18:51 chandankumar joined #puppet-openstack
19:02 EmilienM pabelanger: who owns /opt/git/openstack/tempest ? root?
19:03 pabelanger likely
19:03 pabelanger you can check nodepool.o.o for the log files on how the images are built
19:04 EmilienM oh right
19:05 EmilienM permissions are not really easy to find
19:06 EmilienM jenkins:jenkins
19:06 EmilienM sweet
19:09 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: allow to optionally git clone tempest  https://review.openstack.org/216841
19:10 paramite joined #puppet-openstack
19:11 openstackgerrit Paul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/216832
19:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/216844
19:15 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
19:15 EmilienM pabelanger: done! it should work with Depends-On now :)
19:15 EmilienM thanks for your suggestions!
19:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ  https://review.openstack.org/216828
19:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron  https://review.openstack.org/216831
19:23 ducttape_ joined #puppet-openstack
19:24 EmilienM richm, 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 EmilienM they deploy keystone, etc.. and run tempest
19:24 iurygregory ok EmilienM ^^
19:32 openstackgerrit Paul Belanger proposed openstack/puppet-openstack-integration: Rewrite sudo usage in run_tests.sh  https://review.openstack.org/216832
19:35 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: allow to optionally git clone tempest  https://review.openstack.org/216841
19:44 ricardo joined #puppet-openstack
19:49 ricardo_ joined #puppet-openstack
19:50 sergmelikyan joined #puppet-openstack
19:59 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swift_service defined type  https://review.openstack.org/203220
20:02 paramite joined #puppet-openstack
20:17 sergmelikyan joined #puppet-openstack
20:18 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/216844
20:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
20:51 mfisch EmilienM: would like to discuss the bug triage task for the midcycle
20:51 EmilienM mfisch: 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 here
20:52 mfisch EmilienM: sure we can wait
20:54 sergmelikyan joined #puppet-openstack
20:56 iurygregory anyone who wants to review https://review.openstack.org/#/c/208054/  https://review.openstack.org/#/c/216821/  ^^
20:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: do not git clone tempest  https://review.openstack.org/216844
21:05 ricardo joined #puppet-openstack
21:07 rochaporto joined #puppet-openstack
21:11 rochaporto joined #puppet-openstack
21:18 rochaporto joined #puppet-openstack
21:19 rochaporto joined #puppet-openstack
21:26 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/216844
21:26 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
21:28 EmilienM heh! puppet-manila gate is back again
21:28 * EmilienM updates https://etherpad.openstack.o​rg/p/puppet-liberty-blocker
21:30 crinkle EmilienM: is there a bug report for everything in that etherpad?
21:31 crinkle the neutron one is :( for me
21:31 xingchao_ joined #puppet-openstack
21:31 EmilienM crinkle: no
21:31 EmilienM shame on me.
21:32 EmilienM the fact is, where to report?
21:32 EmilienM for glanceclient, it's in LP, yes
21:32 EmilienM but heat and ironic no
21:32 EmilienM and for neutron... no it's RDO
21:32 EmilienM but they are working on it I guess
21:32 EmilienM crinkle: are you running liberty?
21:33 EmilienM oh yeah I've seen your patch about new LB config file path
21:33 crinkle EmilienM: no just puppet-neutron patches can't get landed
21:33 EmilienM yes :(
21:33 crinkle because db-sync is broken in rdo
21:33 v1k0d3n joined #puppet-openstack
21:35 EmilienM crinkle: yes, something with alembic
21:35 EmilienM iirc
21:36 mfisch EmilienM: you back?
21:38 sanjayu_ joined #puppet-openstack
21:39 EmilienM mfisch: more or less
21:40 EmilienM mfisch: go ahead!
21:40 mfisch lol
21:40 mfisch ok so we dont really have that many bugs that need to be triaged
21:40 mfisch we could instead work on some of them
21:40 mfisch or a bit of both
21:40 mfisch or triage will be quick
21:41 EmilienM mfisch: afik, fuel folks have quite a good number of bugs in fuel that also affect puppet-*
21:41 mfisch thats true, I didnt think of that
21:45 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/216844
21:45 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418
22:58 EmilienM pabelanger: there is a bug in nodepool image
22:58 EmilienM pabelanger: the Depends-On does not work
22:58 EmilienM and I'm using /opt/...
23:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes  https://review.openstack.org/216844
23:28 bitblt joined #puppet-openstack
23:48 gildub joined #puppet-openstack
23:56 xingchao_ joined #puppet-openstack
23:58 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance  https://review.openstack.org/216418

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary