Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-10-02

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

All times shown according to UTC.

Time Nick Message
00:02 openstackgerrit Merged openstack/puppet-openstack-integration: run_tests: drop dupplicated code  https://review.openstack.org/229426
00:02 openstackgerrit Merged openstack/puppet-openstack-integration: scenario001: use 2 workers for sahara-api  https://review.openstack.org/229470
00:06 openstackgerrit Merged openstack/puppet-tempest: Allow to enable Trove tests  https://review.openstack.org/229445
00:07 xarses joined #puppet-openstack
00:19 gildub joined #puppet-openstack
00:30 markvoelker joined #puppet-openstack
00:38 itsuugo joined #puppet-openstack
00:42 rahulait joined #puppet-openstack
00:55 xingchao_ joined #puppet-openstack
01:06 jfluhmann joined #puppet-openstack
01:59 rahulait joined #puppet-openstack
02:32 markvoelker joined #puppet-openstack
02:34 jfluhmann joined #puppet-openstack
02:55 ferest joined #puppet-openstack
03:02 jfluhmann joined #puppet-openstack
03:43 gildub joined #puppet-openstack
04:17 jfluhmann joined #puppet-openstack
04:33 markvoelker joined #puppet-openstack
04:39 jfluhmann joined #puppet-openstack
05:32 lavaboy joined #puppet-openstack
05:46 openstackgerrit Gilles Dubreuil proposed openstack/puppet-keystone: Replace indirection calls  https://review.openstack.org/226624
06:09 arnaud_orange joined #puppet-openstack
06:11 openstackgerrit Gilles Dubreuil proposed openstack/puppet-keystone: Deletes tenant parameter from keystone_user  https://review.openstack.org/228744
06:14 lavaboy joined #puppet-openstack
06:50 crinkle joined #puppet-openstack
06:51 itsuugo joined #puppet-openstack
06:54 ric` joined #puppet-openstack
06:55 ddmitriev joined #puppet-openstack
07:10 sergmelikyan joined #puppet-openstack
07:12 openstackgerrit Merged openstack/puppet-openstack-integration: Cleanup some legacy code  https://review.openstack.org/229071
07:14 openstackgerrit Merged openstack/puppet-openstack-integration: run_tests: allow to run another scenario  https://review.openstack.org/230013
07:22 paramite joined #puppet-openstack
07:31 dgurtner joined #puppet-openstack
07:34 gfidente joined #puppet-openstack
07:39 openstackgerrit Martin Mágr proposed openstack/puppet-cinder: Add ability to search nova in the service catalog  https://review.openstack.org/230297
07:42 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude  https://review.openstack.org/214093
07:43 samuelBartel joined #puppet-openstack
07:46 jpena joined #puppet-openstack
07:47 openstackgerrit joined #puppet-openstack
07:49 openstackgerrit Merged openstack/puppet-cinder: Fix RSpec 3.x syntax  https://review.openstack.org/229760
07:49 openstackgerrit Merged openstack/puppet-ironic: Fix RSpec 3.x syntax  https://review.openstack.org/229793
07:51 openstackgerrit Merged openstack/puppet-neutron: Simplify rpc_backend parameter  https://review.openstack.org/227994
07:57 sergmelikyan joined #puppet-openstack
07:58 openstackgerrit Merged openstack/puppet-glance: Add S3 backend configuration for glance  https://review.openstack.org/226015
08:03 openstackgerrit Merged openstack/puppet-neutron: Map allocation_pool property to set  https://review.openstack.org/226156
08:04 derekh joined #puppet-openstack
08:06 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Put all the logging related parameters to the logging class  https://review.openstack.org/227705
08:13 arnaud_orange joined #puppet-openstack
08:14 openstackgerrit Yanis Guenane proposed openstack/puppet-keystone: Put all the logging related parameters to the logging class  https://review.openstack.org/227843
08:24 jistr joined #puppet-openstack
08:31 openstackgerrit Derek Higgins proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/230314
08:32 derekh By the looks of it that commit ^^ brock the tripleo ceph job
08:32 derekh *broke
08:53 gildub derekh, hi - Re your comment on https://review.openstack.org/#/c/226624/14 , I think the tripleo-ci test set might need a change to add the domain name in the user_role provider call
08:56 derekh gildub: ahh ok, I'll take a look, looks like the dsvm test need an update also
08:57 gildub derekh, more precisely heat_admin::heat_domain@::heat_stack
08:57 gildub derekh, the dsvm?
08:57 gildub derekh, I mean, anything wrong? Looking...
08:58 derekh gildub: http://logs.openstack.org/24/226624/15/check/gate-puppet-openstack-integration-dsvm-centos7/983075a/console.html#_2015-10-02_06_07_40_505
08:58 derekh gildub: brb
08:58 gildub derekh, yeah, do mean the integration-dsvm ones? They are non voting
08:59 gildub derekh, no worries, I'll back later => kids (stories/bed)
09:00 derekh gildub: I know, I would have thought we would want to keep them passing if we can, do they normally pass ?
09:00 derekh gildub: no prob, thans for the pointers, I'll get back to looking at it in a bit
09:02 sergmelikyan joined #puppet-openstack
09:04 markvoelker joined #puppet-openstack
09:14 sergmelikyan joined #puppet-openstack
09:17 Vivek joined #puppet-openstack
09:17 dmsimard joined #puppet-openstack
09:20 _nick joined #puppet-openstack
09:21 mattymo joined #puppet-openstack
09:22 openstack joined #puppet-openstack
09:23 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
09:24 blentz joined #puppet-openstack
09:25 lavaboy joined #puppet-openstack
09:32 rahulait joined #puppet-openstack
09:32 sergmelikyan joined #puppet-openstack
09:36 paramite joined #puppet-openstack
09:41 gfidente derekh, so https://review.openstack.org/#/c/203909 is actually wrong in virsh secret-dumpxml --secret ${libvirt_rbd_secret_uuid} 2>/dev/null | grep -q -v -w \"${rbd_keyring}\""
09:42 gfidente that returns 0 even when there is no secret with that uuid
09:42 gfidente but looks fixable, so we probably don't need the revert, trying
09:43 derekh gfidente: cool
09:47 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: README: update Puppet wiki URL  https://review.openstack.org/229856
09:48 jtomasek joined #puppet-openstack
09:48 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Extend the logging class capabilities.  https://review.openstack.org/229899
09:48 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Updates on config provider  https://review.openstack.org/229439
09:49 aojea_ joined #puppet-openstack
09:49 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: update keystone/auth bits  https://review.openstack.org/229849
09:58 openstackgerrit Giulio Fidente proposed openstack/puppet-nova: Fix virsh secret cleanup script  https://review.openstack.org/230348
09:59 sergmelikyan joined #puppet-openstack
10:13 pcaruana joined #puppet-openstack
10:32 gildub derekh, where are those puppet values stored in the tripleo-ironic-overcloud-f21puppet-nonha?
10:35 derekh gildub: see ^ gfidente is taking a look at a fix , most our puppet values either come from hieradata http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/puppet/hieradata or from heat (and mapped to hieradata)
10:35 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Update the autobuild.asc key URL  https://review.openstack.org/230362
10:37 gfidente gildub, not sure if we're looking into the same issue, I'm checking the ceph job failures and trying this https://review.openstack.org/#/c/230348/
10:38 gildub gfidente, derekh, not the same issue, I'm talking about https://review.openstack.org/#/c/226624
10:38 gildub derekh, that said ^ thanks for the hieradata pointer :)
10:38 derekh gildub: gfidente sorry, got confused with what I was doing
10:39 derekh gildub: gfidente tw0 seperate issues
10:39 gildub derekh, no worries
10:39 derekh gildub: I've yet to look into it properly,
10:41 gfidente do you guys know if Kuo-Le.Mei is online and which nick he has?
10:41 gildub derekh, ok, I'm going to call it a week, just leave me a comment in the review
10:42 derekh gildub: cool, have a good weekend, I should have something for you by monday
10:42 derekh gfidente: dunno
10:42 gildub derekh, thanks, have a good one
10:54 leanderthal joined #puppet-openstack
11:01 dsutyagin joined #puppet-openstack
11:04 markvoelker joined #puppet-openstack
11:06 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
11:06 skolekonov joined #puppet-openstack
11:16 delatte joined #puppet-openstack
11:16 IvanBerezovskiy joined #puppet-openstack
11:21 openstackgerrit Giulio Fidente proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/230376
11:37 aojea_ joined #puppet-openstack
11:37 itsuugo joined #puppet-openstack
11:38 EmilienM good morning!
11:45 iurygregory good morning :D
11:56 openstackgerrit Merged openstack/puppet-heat: Make the role for heat_stack_user configurable  https://review.openstack.org/225533
12:03 pradk joined #puppet-openstack
12:05 markvoelker joined #puppet-openstack
12:07 guessi_ joined #puppet-openstack
12:07 EmilienM today is friday \o/
12:08 guessi_ hi gfidente,
12:09 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
12:09 markvoelker joined #puppet-openstack
12:10 guessi_ left #puppet-openstack
12:10 guessi_ joined #puppet-openstack
12:16 dprince joined #puppet-openstack
12:18 jpena|lunch joined #puppet-openstack
12:18 gfidente guessi_, hey
12:19 ducttape_ joined #puppet-openstack
12:19 guessi_ sorry for late reply, i'm not familiar with irc :(
12:20 gfidente guessi_, hey no worries, welcome
12:20 guessi_ i'm trying to solve the issue with grepping problem,
12:20 gfidente so I saw your comment, indeed I think we should compare the key contents but that is in libvirt_key not in rbd_keysring
12:20 guessi_ does "test -z XXX" look good to you?
12:20 gfidente so we should somehow parse the output of dumpxml to get the base64 encoded string corresponding to the key
12:21 gfidente looked non-trivial so I voted to revert until we come up with another implementation
12:22 EmilienM gfidente: should we merge the revert or wait for your patch instread ? cc spredzy
12:23 gfidente I think the revert is safer for now
12:23 EmilienM ok
12:23 EmilienM spredzy: ==
12:23 EmilienM let's revert it now
12:23 gfidente because the patch might make it pass, but it won't behave as intended, it's going to remove/recreate the key every time
12:23 guessi_ i'm okay for the revert
12:23 EmilienM guessi_: thanks
12:23 gfidente guessi_, the problem is parsing that xml output
12:24 guessi_ is this okay for grepping "libvirt_key" before define it?
12:24 guessi_ https://github.com/openstack/puppet-nova/blob/master/manifests/compute/rbd.pp#L100-L104
12:24 EmilienM gfidente, dprince: we might need to had the tripleo/ceph job in puppet-cinder & puppet-nova gate.
12:24 gfidente guessi_, the key is base64 encoded so libvirt_key won't contain the same string as found in the virsh dump
12:26 guessi_ gfidente, question: how could we tell if the content is changed with base64 encoded id?
12:27 gfidente yeah I think we should basically re-encode the new libvirt_key, parse the xmldump and compare those two
12:27 gfidente because the xmldump will emit it base64 encoded
12:27 gfidente then, if those differ it means the key has been updated and we should execute the cleanup
12:28 dprince EmilienM: I would suggest perhaps just having the core team no approve unless that job passes first
12:28 * gfidente back soon
12:29 EmilienM dprince: the problem is OOO CI is unstable to be fair, we can't wait everytime OOO CI is green to merge a patch
12:29 EmilienM but having ceph job for puppet-cinder & glance could help
12:29 dprince EmilienM: right, which is why I'm suggesting not gating on it
12:29 EmilienM ok
12:29 dprince EmilienM: just watch it closely, and if our stability improves then we can consider other ideas...
12:30 EmilienM dprince, gfidente, guessi_: revert patch is +2 +A
12:30 EmilienM dprince: cool
12:31 guessi_ gfidente, according to doc@libvirt, it is not necessary in UUID format, maybe we could give it an user-defined id? not sure... need verify :p
12:31 guessi_ @EmilienM: thanks !
12:32 guessi_ forget the reference link: https://libvirt.org/formatsecret.html#SecretAttributes
12:34 ducttape_ joined #puppet-openstack
12:37 delatte joined #puppet-openstack
12:43 guessi joined #puppet-openstack
12:44 aderyugin Hi, folks! Can you please take a look at my commit https://review.openstack.org/#/c/216682/ I hope it's ready
12:45 openstackgerrit Merged openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/230314
12:50 openstackgerrit Emilien Macchi proposed openstack/puppet-barbican: puppet-barbican: Initial commit  https://review.openstack.org/229871
12:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency  https://review.openstack.org/230413
12:52 richm joined #puppet-openstack
12:59 itsuugo joined #puppet-openstack
13:00 aojea_ joined #puppet-openstack
13:00 morazi joined #puppet-openstack
13:08 tiswanso joined #puppet-openstack
13:17 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
13:18 vinsh jpena: Good eye!! ^^
13:19 vinsh I had that commented out because It was giving me hell when trying to write a spec test for the provider... I couldn't figure out how to stub out a file in the spec.  Adding more to the provider spec will come in a later review once I consult with some ruby experts.
13:26 guessi joined #puppet-openstack
13:40 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/230431
13:42 mattymo_ Does anyone know about the state of puppet-openstack modules with Puppet 4.1.0?
13:44 mattymo_ dprince, EmilienM do you guys know?
13:45 dprince mattymo_: I've not used 4.1 yet myself, I don't think unit test coverage is there yet either
13:46 dprince mattymo_: I take that back, fedora 22 has 4.1
13:46 mdorman joined #puppet-openstack
13:46 EmilienM icehouse? we don't officially support it anymore.
13:46 dprince mattymo_: so I have used it, but not w/ all of the OPM modules
13:47 EmilienM mattymo_: https://wiki.openstack.org/wiki/Releases
13:47 EmilienM Icehouse is EOL
13:48 EmilienM I mean, you can still backport patches - it's just hard to make them pass CI, due to rspec updates.
13:48 mattymo_ EmilienM, I don't mean Icehouse modules. I mean puppet itself, the tool
13:48 mattymo_ dprince answered my question. Thanks!
13:48 EmilienM ah
13:49 EmilienM mattymo_: sorry, I'm in the middle of releasing Puppet modules
13:49 EmilienM so I thought you were wondering about the modules themselves
13:49 dprince mattymo_: I did try running puppet 4.1 to general configs only (in a docker container). It worked fine there w/ OPM
13:49 * EmilienM facepalm
13:49 dprince mattymo_: generate configs
13:49 EmilienM mattymo_, dprince: our modules are gated by Puppet 4.x
13:49 EmilienM I don't see any issue, and it should work pretty good.
13:50 dprince EmilienM: yeah, I think it is close.
13:50 dprince EmilienM: but we aren't gating on it yet
13:50 EmilienM dprince: what?
13:50 dprince EmilienM: 4.1
13:50 dprince EmilienM: I thought we just used 4.0
13:51 dprince EmilienM: 4.0 doesn't imply 4.x does it?
13:51 EmilienM http://logs.openstack.org/97/228597/2/check/gate-puppet-ceilometer-puppet-unit-latest-dsvm-centos7/4735ca4/console.html#_2015-10-01_21_30_46_810
13:51 EmilienM dprince: ^
13:51 EmilienM 4.2.2
13:51 EmilienM dprince, mattymo_ : please rtfm : https://wiki.openstack.org/wiki/Puppet/CI
13:51 EmilienM :)
13:51 dprince EmilienM: okay, so latest covers us here
13:51 EmilienM yup
13:51 dprince EmilienM: but are we skipping 4.1
13:52 EmilienM dprince: well... we can't test all 4.x & 3.x versions.
13:52 dprince EmilienM: I see 4.0, and then apparently latest is 4.2
13:52 EmilienM we test 4.0 as voting
13:52 EmilienM and latest as non voting
13:52 dprince EmilienM: right, so we actually aren't testing 4.1 specifically, but it should work fine we think
13:53 mattymo_ EmilienM, I found 4.0 covered but not 4.1. That's why I asked
13:53 EmilienM dprince: we could bump the 4.0 job to 4.1
13:53 ducttape_ joined #puppet-openstack
13:53 khushbu joined #puppet-openstack
13:53 EmilienM but I don't see huge value
13:54 EmilienM mattymo_: have you hit any issue?
13:54 mattymo_ EmilienM, not yet, thankfully
13:54 EmilienM mattymo_: I don't think you'll have any issue
13:54 mattymo_ we're just rebasing puppet version in fuel. We're very far behind the times
13:55 EmilienM our modules are really close to latest puppet version, rspec and upstream dependencies
13:56 sanjayu joined #puppet-openstack
14:01 mwhahaha mattymo_: thought we were updating to 3.8 not 4.x
14:01 mattymo_ mwhahaha, yeah I thought so too :D
14:01 sergmelikyan joined #puppet-openstack
14:01 mattymo_ mwhahaha, but as long as we're moving, just go to something even newer
14:03 mwhahaha true but i doubt fuel is 4.x compatible :D
14:06 mattymo_ mwhahaha, we won't know until we try
14:17 itsuugo joined #puppet-openstack
14:17 aojea_ joined #puppet-openstack
14:28 khushbu joined #puppet-openstack
14:40 sanjayu joined #puppet-openstack
14:40 khushbu joined #puppet-openstack
14:44 openstackgerrit Merged openstack/puppet-ceilometer: wsgi: make sure eventlet process is stopped before httpd  https://review.openstack.org/228597
14:53 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency  https://review.openstack.org/230413
14:57 dfisher joined #puppet-openstack
15:01 guessi left #puppet-openstack
15:02 openstackgerrit Merged openstack/puppet-barbican: puppet-barbican: Initial commit  https://review.openstack.org/229871
15:02 openstackgerrit Merged openstack/puppet-keystone: Release 5.2.0 - Juno  https://review.openstack.org/230173
15:03 openstackgerrit Merged openstack/puppet-glance: Release 5.2.0 - Juno  https://review.openstack.org/230171
15:04 openstackgerrit Merged openstack/puppet-cinder: Release 5.2.0 - Juno  https://review.openstack.org/230170
15:04 openstackgerrit Merged openstack/puppet-heat: Release 5.2.0 - Juno  https://review.openstack.org/230172
15:04 openstackgerrit Merged openstack/puppet-aodh: puppet-aodh: Initial commit  https://review.openstack.org/225015
15:05 openstackgerrit Merged openstack/puppet-ceilometer: Release 5.2.0 - Juno  https://review.openstack.org/230166
15:09 openstackgerrit Merged openstack/puppet-openstack_extras: Release 5.1.0 - Juno  https://review.openstack.org/230182
15:14 herewego joined #puppet-openstack
15:14 openstackgerrit Merged openstack/puppet-nova: Release 5.2.0 - Juno  https://review.openstack.org/230179
15:16 sanjayu joined #puppet-openstack
15:20 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
15:24 sanjayu joined #puppet-openstack
15:24 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Make sure service nova-network is running before create networks  https://review.openstack.org/229895
15:25 degorenko hello guys. EmilienM sbadia mfisch spredzy take a look please: https://review.openstack.org/#/c/229895
15:27 openstackgerrit Merged openstack/puppet-neutron: Release 5.2.0 - Juno  https://review.openstack.org/230174
15:30 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Rewrite nova_network provider with using only nova client  https://review.openstack.org/229548
15:31 linkedinyou joined #puppet-openstack
15:33 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/226862
15:56 jfluhmann joined #puppet-openstack
16:01 dprince joined #puppet-openstack
16:02 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: DNM - CI test  https://review.openstack.org/230524
16:03 IvanBerezovskiy left #puppet-openstack
16:04 EmilienM mfisch, mgagne: how do you release new versions in launchpad, do you have a script or manually?
16:04 mfisch I think mgagne has a tool
16:06 iurygregory any core around can review https://review.openstack.org/#/c/223777/ ? already have a +2 ^^
16:08 mfisch looking
16:10 iurygregory Thanks mfisch o/
16:13 EmilienM I just found out https://github.com/mgagne/openstack-puppet-release-tools
16:16 ducttape_ joined #puppet-openstack
16:16 mgagne that's the tool ^
16:21 EmilienM mgagne: well, I would be happy to move it in openstack
16:21 mgagne EmilienM: sure
16:21 EmilienM cool
16:21 EmilienM mgagne: I'll let you do the infra patch
16:22 mgagne EmilienM: all fine with me. let me know the review so I can +1
16:22 EmilienM mgagne: not sure everything is good for 5.2.0, I did it manually
16:22 mgagne EmilienM: want me to double-check?
16:22 EmilienM mgagne: you want me to do it? :)
16:22 EmilienM mgagne: it would be nice
16:23 lavaboy joined #puppet-openstack
16:25 mgagne no release for horizon or this has yet to happen?
16:26 openstackgerrit Merged openstack/puppet-openstack-specs: Changes in enabling federation spec  https://review.openstack.org/223777
16:26 EmilienM mgagne: no release
16:27 mgagne ok
16:28 EmilienM mgagne: https://docs.google.com/spreadsheets/d/1XVrmEiLrJSdxDo-S_vFB7ljxTdYg-pe8hiMUryRor5A/edit#gid=0
16:29 EmilienM mgagne: you can update only those in green, in Launchpad column
16:29 mgagne all done
16:30 mgagne https://github.com/mgagne/openstack-puppet-release-tools/commit/4dec8160bc2115213da247f9215b8eff3413e8c0
16:30 EmilienM mgagne: thx a lot
16:30 mgagne I already ran the script
16:31 openstackgerrit Clayton O'Neill proposed openstack/puppet-neutron: Set new ml2::ovs parameters to use SERVICE DEFAULT  https://review.openstack.org/230536
16:31 EmilienM clayton: we are not using this pattern ^
16:32 EmilienM clayton: in the last meeting, we talked about using the fact
16:32 clayton ah, right, I remember that from the ML
16:32 clayton are there modules using the new way I can crib from?
16:39 clayton ah, I see the fact review.
16:39 clayton I'll leave my review oustanding and fix it to use the fact when that one is merged.
16:56 EmilienM Ubuntu recently updated packaging to have RC1
16:56 EmilienM I need to run CI testing now, I'm sorry for the spams that is going to happen
16:56 mfisch is regsubst the best way to pre-pend a string to every element in an array?
16:56 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: DNM - CI test  https://review.openstack.org/230541
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: DNM - CI test  https://review.openstack.org/230542
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: DNM - CI test  https://review.openstack.org/230543
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: DNM - CI test  https://review.openstack.org/230544
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: DNM - CI test  https://review.openstack.org/230545
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: DNM - CI test  https://review.openstack.org/230546
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: DNM - CI test  https://review.openstack.org/230547
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: DNM - CI test  https://review.openstack.org/230548
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: DNM - CI test  https://review.openstack.org/230549
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: DNM - CI test  https://review.openstack.org/230550
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: DNM - CI test  https://review.openstack.org/230551
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: DNM - CI test  https://review.openstack.org/230552
16:57 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: DNM - CI test  https://review.openstack.org/230553
17:00 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: acceptance: include glance::backend::file  https://review.openstack.org/230556
17:01 jamespage joined #puppet-openstack
17:01 jamespage i/
17:01 EmilienM jamespage: thx for being here
17:01 EmilienM jamespage: I'm now checking everything is clean except glance
17:02 jamespage EmilienM, http://paste.ubuntu.com/12639363/
17:02 jamespage thats the diff in requirements.txt from b3->rc1 for glance
17:02 EmilienM we need to see the diff in glance-store
17:03 jamespage swiftclient gets dropped, and we auto-generate/align our runtime dependencies based on what's in the openstack project being packaged
17:03 jamespage EmilienM, glance-store has never had swiftclient as a dep
17:03 jamespage I check that already
17:03 jamespage EmilienM, the problem is that these are all soft-depends depending on which storage is being used
17:04 sanjayu joined #puppet-openstack
17:04 EmilienM jamespage: our CI is red, I suggest we consider it and adjust uca packaging.
17:04 EmilienM I guess this one: http://logs.openstack.org/24/230524/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/a16802a/logs/glance/glance-api.txt.gz#_2015-10-02_16_45_22_427
17:04 EmilienM is the one to take in consideration
17:04 EmilienM for http://logs.openstack.org/24/230524/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/a16802a/logs/glance/glance-api.txt.gz#_2015-10-02_16_45_22_431 I proposed https://review.openstack.org/230556
17:05 jamespage EmilienM, https://github.com/openstack/glance/commit/5c5888611337ac3d12277e70f368aa9c23ebaabd
17:05 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
17:05 EmilienM jamespage: so how do you explain ImportError: No module named swiftclient ?
17:05 jamespage EmilienM, because swiftclient is dropped from depends, it's not installed
17:06 jamespage EmilienM, the glance charm installs clients explicitly based on what backend is being used - ceph swift etc...
17:06 EmilienM jamespage: so I should patch puppet-glance to install it?
17:06 EmilienM it's really not backward compatible, but well..
17:07 jamespage EmilienM, I think that would be inline with the upstream glance thinking....
17:07 EmilienM we try to not manage package dependencies in Puppet
17:07 jamespage EmilienM, this is not a package dependency, its a feature dependency
17:07 jamespage think of it that way :-)
17:07 EmilienM jamespage: the fact is, our CI is not using Swift backend and it fails anyway
17:07 jamespage really?
17:07 EmilienM yeas
17:07 EmilienM http://logs.openstack.org/24/230524/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/a16802a/logs/glance/glance-api.txt.gz#_2015-10-02_16_45_22_431
17:07 jamespage does it enable the swift backend by default?
17:08 EmilienM we're using filesystem
17:08 EmilienM see http://logs.openstack.org/24/230524/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/a16802a/logs/etc/glance/glance-api.conf.txt.gz
17:08 guessi joined #puppet-openstack
17:08 EmilienM #stores = file,http
17:08 EmilienM #default_store = file
17:10 jamespage well that is odd
17:12 EmilienM jamespage: https://review.openstack.org/#/q/owner:%22Emilien+Macchi%22+status:open+topic:%22ci-test%22,n,z
17:13 EmilienM we will see what else is coming :)
17:14 jamespage well on monday
17:14 jamespage sorry have to eod
17:14 EmilienM jamespage: any other contact?
17:15 jamespage coreycb
17:15 EmilienM maybe coreycb could come here
17:15 jamespage he's in #ubuntu-server
17:15 trown joined #puppet-openstack
17:15 jamespage can't guarantee turnaround on fixes now anyway as we're in final freeze so all changes have to go through the release team in ubuntu
17:16 EmilienM jamespage: I'm just saying you, people will complain if you don't change it.
17:16 jamespage just saying we have process and checks that take time
17:16 jamespage and we don't resource 24.x7
17:16 jamespage sorry
17:17 EmilienM it's weird it works in RDO and not in UCA
17:17 EmilienM you both run liberty
17:18 xarses joined #puppet-openstack
17:19 EmilienM jamespage: people who use file backend will do "apt-get install glance-api glance-registry" and they will have the failure we're having now
17:19 EmilienM i'm giving you free feedback that says "your package will not make Glance working by default"
17:20 EmilienM if you don't fix it, yes we will have to fix it in Puppet but you'll also have complains from people not using Puppet.
17:21 EmilienM zigo: just fyi ^
17:21 dfisher puppet should not be in the business of managing packaging.  that's … for packaging to manage.
17:21 dfisher just set a freakin' dependency on it.
17:22 EmilienM jamespage is right in saying "glance drop the swiftclient dependency", but it still looks like it needs, logs do not lie, Glance is not starting.
17:23 EmilienM and no, we don't want to deal with packaging dependencies in our modules, that's dupplicated work with packaging folks.
17:24 dfisher at best it's a duplicate.  at worst it's a conflict.
17:24 dfisher puppet says install x y z, packaging says x y a b c
17:24 sanjayu joined #puppet-openstack
17:25 imcsk8 joined #puppet-openstack
17:27 jamespage EmilienM, look I'm not saying I won't fix it in the packaging; but it does point to a bug somewhere - swift is an optional backend - glance should not fail if its not installed
17:28 jamespage EmilienM, how about we start with a bug report please? ping me the link and I'll try find time later - have to step out todo some non-work stuff
17:32 EmilienM jamespage: no worries, ok I'll find a bug and let you know. see you on monday
17:33 jamespage EmilienM, you might see me in +3 hrs
17:33 jamespage ;)
17:48 sanjayu joined #puppet-openstack
17:59 ducttape_ joined #puppet-openstack
18:10 EmilienM jamespage: I have a good news, glance is the only issue I think
18:16 EmilienM crinkle: do you still -1 https://review.openstack.org/#/c/226830/ after the recent changes ? I addressed your comment.
18:20 mfisch EmilienM: this is why we manage a bunch of package deps ourselves in puppet
18:20 tiswanso Hi all,  I have a puppet manifest writing question.  I'm in a scenario where I need create resources from a 2nd level key of a hash, where the 2nd level keys are not unique across first level keys.
18:21 tiswanso This is a problem with using nested define types where the outer resource invokes create_resources to iterate the inner hash key.
18:21 EmilienM mfisch: it's not sane imho
18:21 mfisch I realize its not ideal but my deployment doesnt wait for ideal
18:22 tiswanso the stdlib .each method should be usable to get around this but it's "parser=future"
18:22 tiswanso any alternatives?
18:22 EmilienM mfisch: lol
18:22 EmilienM mfisch: I try to make sure our CI manifest do not do too much workarounds and try to fix it upstream
18:25 mfisch I agree thats the goal
18:27 sanjayu joined #puppet-openstack
18:28 crinkle EmilienM: still -1
18:29 EmilienM crinkle: I just found out the ceilometer patch has a dependency cycle issue.
18:29 crinkle EmilienM: :'(
18:29 EmilienM crinkle: yeah it's weird, I'm working on it :)
18:30 EmilienM crinkle: ack your comment, working on it too :-)
18:31 EmilienM crinkle: I'm linking because it's simplier than cloning, also, fixtures is also using links.
18:31 EmilienM crinkle: the rm is because the ln_s fails if the file already exist.
18:32 EmilienM crinkle: which is also something fixtures does
18:39 crinkle EmilienM: I would rather it worked about the same way as the real spec_prep does
18:39 EmilienM crinkle: I replied inline
18:43 trown hola puppeteers, https://review.openstack.org/223690 is ready for review despite the centos acceptance tests failing (trusty is broke afaict). The centos tests will pass on a recheck after an Ironic packaging change propagates, I tested with the "current" repo in PS24 and it passed.
18:44 EmilienM trown: trust works
18:44 EmilienM trusty*
18:44 sergmelikyan joined #puppet-openstack
18:46 trown EmilienM: ah you are right, I did not even look since it was broke yesterday...the conditional to only test inspector port on centos is not working
18:46 EmilienM mhh, it should iirc we already used it
18:47 trown trusty failed on port 5050 not listening
18:47 trown which it should not have run if the conditional evaluated to true
18:49 trown EmilienM: I am unconvinced that worked before... it would have failed silently
18:51 EmilienM crinkle: ok, so I need to 1/ drop the rm_rf and ln_s 2/ use zuul-cloner with the right map file option. Right ?
18:54 sergmelikyan joined #puppet-openstack
18:54 crinkle EmilienM: yes
18:55 EmilienM crinkle: so I need to create clonemap.yaml, etc ?
18:56 EmilienM crinkle: it won't when you will run tests on your laptop
18:56 EmilienM won't run*
18:57 EmilienM it does not do the rm_rf because there is Puppet::FileSystem::exist?(target in https://github.com/puppetlabs/puppetlabs_spec_helper/blob/master/lib/puppetlabs_spec_helper/rake_tasks.rb#L130-L137
18:57 EmilienM my last comment is false ^
18:59 crinkle EmilienM: the zuul-cloner part should actually just work if you give it the right puppetfile_dir because http://git.openstack.org/cgit/openstack/puppet-openstack-integration/tree/install_modules.sh#n20
18:59 gchamoul joined #puppet-openstack
19:00 EmilienM crinkle: apologizes, but I don't understand the symlink problem.
19:01 sanjayu joined #puppet-openstack
19:02 EmilienM crinkle: iiuc, I can drop the ln_s & rm_rf and that's it, because the module should already be cloned, right?
19:02 EmilienM I think I got it :)
19:03 EmilienM crinkle: the problem is the case where don't run zuul-cloner
19:03 EmilienM so I need to move my rm_rf & ln_s with a conditionnal, right?
19:04 crinkle EmilienM: oh i see - i think it's probably fine to rm and ln the current module ; what i don't like is rm_rf("#{Dir.pwd}/spec/fixtures/modules")
19:05 EmilienM crinkle: L82 ? well, it's just when tests are finished, you want to clean the dir, no?
19:05 crinkle no line 52
19:05 EmilienM oh indeed
19:06 EmilienM ok, trying
19:07 EmilienM it works, I update
19:08 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
19:08 EmilienM crinkle: this patchset just drop L52 ^
19:16 sergmelikyan joined #puppet-openstack
19:16 aojea_ joined #puppet-openstack
19:17 itsuugo joined #puppet-openstack
19:25 xarses joined #puppet-openstack
19:26 itsuugo joined #puppet-openstack
19:26 aojea_ joined #puppet-openstack
19:33 openstackgerrit Alex Schultz proposed openstack/puppet-openstacklib: Add os_service_default fact  https://review.openstack.org/229050
19:33 sanjayu joined #puppet-openstack
19:37 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/229095
19:49 sanjayu joined #puppet-openstack
19:53 zigo EmilienM: FYI, the Debian package does work with the file backend by default.
19:56 EmilienM zigo: do you install swiftclient by default?
19:56 zigo Looking ...
19:56 zigo EmilienM: Nop, but I can if you wish.
19:57 EmilienM zigo: it's weird, glance is not starting with default packaging. I'm investigating
19:57 EmilienM zigo: I'll let you know
19:57 zigo EmilienM: You mean in Debian?
19:57 zigo It does for me ...
19:58 zigo EmilienM: I'm currently trying to beat tempest. So far, it's working better and better.
19:58 EmilienM we shamelessly install it from source.
19:59 zigo EmilienM: Install what from source? Glance?
19:59 zigo git commit -a -m "Adding python-swiftclient as runtime dependency (request from puppet team)."
19:59 zigo :)
19:59 EmilienM tempest
19:59 zigo Ah...
19:59 zigo Well, I don't ! :)
20:00 EmilienM zigo: thx but I'm not sure for swiftclient, I noticed that this morning but I need time to investigate
20:00 EmilienM like usual I appreciate your responsiveness :)
20:01 dfisher see?  add a dependency.
20:01 dfisher it's so easy.
20:02 EmilienM dfisher: zigo & I have that special relationship you know ;-)
20:02 dfisher :)
20:02 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: wsgi: make sure keystone service is stopped before starting httpd  https://review.openstack.org/230626
20:03 zigo EmilienM: Could you look again in the thread which I opened: "[puppet] Running Debian packages on top of Trusty" ?
20:03 EmilienM zigo: sure, I was not much in e-mails today
20:03 zigo EmilienM: You missed my point. I was surprised of it, because we had the issue, and you already replied to me correctly on IRC about it...
20:04 skolekonov joined #puppet-openstack
20:05 zigo EmilienM: BTW, these days, do you do a 100% on tempest, or what?
20:05 EmilienM mwhahaha: I think you can override package name, because they are all tagged
20:06 EmilienM for example: https://github.com/openstack/puppet-ceilometer/blob/master/manifests/api.pp#L108
20:07 mwhahaha yea i know that, doesn't mean everyone else wants to :D
20:09 mwhahaha like i said in the email we're doing that already for some stuff. I think we need to take a deeper look into how much would need to be overridden in such a way
20:15 sanjayu joined #puppet-openstack
20:15 EmilienM crinkle: I found the ceilo/wsgi bug
20:23 openstackgerrit Sergey Kolekonov proposed openstack/puppet-keystone: Pass necessary options to Apache when using WSGI  https://review.openstack.org/230633
20:23 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: auth: drop service dependency for Keystone_user_role  https://review.openstack.org/230634
20:23 EmilienM crinkle: ^
20:25 mfisch is Maksym Yatsenko in this channel?
20:29 EmilienM myatsenko: ^
20:31 mfisch probably late there unless he's us
20:31 mfisch I have to fix his rbd/host change it broke our cinder
20:31 mfisch and is deprecated
20:53 angdraug joined #puppet-openstack
20:57 skolekonov joined #puppet-openstack
21:03 EmilienM I don't understand what's happenning here: https://review.openstack.org/#/c/230556/
21:03 EmilienM http://logs.openstack.org/56/230556/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/c62504e/console.html#_2015-10-02_20_42_33_739
21:04 EmilienM the parameter is passed, see https://review.openstack.org/#/c/230556/1/spec/acceptance/basic_glance_spec.rb,cm
21:09 crinkle EmilienM: glance::backend::file inherits glance::api
21:10 crinkle you can't inherit a class with parameters
21:10 EmilienM crinkle: we found a new bug :)
21:10 crinkle yay testing \o/
21:10 EmilienM crinkle: lol
21:11 EmilienM crinkle: git blame tells me who is paying beers at puppetconf
21:11 EmilienM bodepd: hey what's up? :)
21:11 iurygregory by people, have a good weekend o/
21:12 EmilienM iurygregory: you too :)
21:12 iurygregory ^^
21:12 dfisher beers?   sake*
21:12 dfisher oh.  puppetconf.
21:12 dfisher ignore me.
21:15 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: file backend: do not inherit from glance::api  https://review.openstack.org/230556
21:16 EmilienM so if we want integration jobs back alive, we need https://review.openstack.org/230634 and https://review.openstack.org/230556
21:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/230431
21:22 karmix joined #puppet-openstack
21:23 EmilienM _skolekonov: I'm reading https://bugs.launchpad.net/bugs/1502318 and I don't think "keystone is broken in WSGI" > we're gating it
21:23 openstack Launchpad bug 1502318 in puppet-openstack "keystone is set up incorrectly under Apache" [Undecided,New] - Assigned to Sergey Kolekonov (skolekonov)
21:25 zeus joined #puppet-openstack
21:27 richm joined #puppet-openstack
21:36 rahulait joined #puppet-openstack
21:40 EmilienM mtreinish: I found something weird in tempest/smoke: even if I disable nova/neutron/cinder/glance services -> tempest will try (and fail) to run scenarios
21:40 EmilienM they should be SKIPPED I think
21:41 mtreinish EmilienM: yeah they should raise a skip exception if the service is disabled they required
21:41 EmilienM yeah or just skip
21:41 mtreinish that's what the @test.services decorator is supposed to do
21:42 devnull0 joined #puppet-openstack
21:42 EmilienM mtreinish: I'm proposing a patch
21:42 mtreinish EmilienM: ok
21:43 EmilienM mtreinish: wait, they already have the python decorator
21:43 EmilienM @test.services('compute', 'network')
21:44 bnemec joined #puppet-openstack
21:44 mtreinish yeah, that's what I meant, that decorator should raise a skip exception if either compute or network is unavailable (well not network because n-net v neutron makes that hard to detect)
21:44 EmilienM I'll investigate logs
21:45 EmilienM mtreinish: quick update - the split happenned - I'm trying to balance services now: https://review.openstack.org/#/c/230431/2/README.md,cm
21:46 mtreinish EmilienM: well you need nova, glance, etc for heat, sahara, trove to work
21:46 EmilienM arg
21:46 EmilienM ok
21:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/230431
22:07 mtreinish EmilienM: ^^^ you dropped swift from the table?
22:07 EmilienM oops I would not
22:07 EmilienM mtreinish: no, it's in scenario002
22:08 mtreinish oh, yep sry missed it
22:08 mtreinish blended together with sahara in my head
22:08 EmilienM mtreinish: np, thx for helping
22:08 mtreinish too many s's
22:08 EmilienM yeah lol
22:14 EmilienM jamespage: I have issues with cinder & glance so far. I'll investigate later
22:33 zeus joined #puppet-openstack
22:36 sergmelikyan joined #puppet-openstack
22:42 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/230431
23:00 sanjayu joined #puppet-openstack
23:37 sergmelikyan joined #puppet-openstack

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