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:02 |
*** gildub has quit IRC | 00:03 | |
*** mdorman has quit IRC | 00:04 | |
openstackgerrit | Merged openstack/puppet-tempest: Allow to enable Trove tests https://review.openstack.org/229445 | 00:06 |
*** xarses has joined #puppet-openstack | 00:07 | |
*** gildub has joined #puppet-openstack | 00:19 | |
*** markvoelker has joined #puppet-openstack | 00:30 | |
*** markvoelker has quit IRC | 00:35 | |
*** itsuugo has joined #puppet-openstack | 00:38 | |
*** rahulait has joined #puppet-openstack | 00:42 | |
*** sanjayu has quit IRC | 00:44 | |
*** xingchao_ has joined #puppet-openstack | 00:55 | |
*** richm has quit IRC | 00:57 | |
*** xingchao has quit IRC | 00:57 | |
*** jfluhmann has joined #puppet-openstack | 01:06 | |
*** ibravo has quit IRC | 01:15 | |
*** rahulait has quit IRC | 01:24 | |
*** angdraug has quit IRC | 01:44 | |
*** rahulait has joined #puppet-openstack | 01:59 | |
*** jfluhmann has quit IRC | 01:59 | |
*** rahulait has quit IRC | 02:11 | |
*** markvoelker has joined #puppet-openstack | 02:32 | |
*** jfluhmann has joined #puppet-openstack | 02:34 | |
*** markvoelker has quit IRC | 02:36 | |
*** jfluhmann has quit IRC | 02:53 | |
*** ferest has joined #puppet-openstack | 02:55 | |
*** ferest has quit IRC | 03:00 | |
*** jfluhmann has joined #puppet-openstack | 03:02 | |
*** gildub has quit IRC | 03:27 | |
*** itsuugo has quit IRC | 03:34 | |
*** gildub has joined #puppet-openstack | 03:43 | |
*** bnemec has quit IRC | 04:02 | |
*** jfluhmann has quit IRC | 04:04 | |
*** xingchao_ has quit IRC | 04:11 | |
*** jfluhmann has joined #puppet-openstack | 04:17 | |
*** jfluhmann has quit IRC | 04:27 | |
*** markvoelker has joined #puppet-openstack | 04:33 | |
*** jfluhmann has joined #puppet-openstack | 04:39 | |
*** crinkle has quit IRC | 05:04 | |
*** markvoelker has quit IRC | 05:07 | |
*** lavaboy has joined #puppet-openstack | 05:32 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Replace indirection calls https://review.openstack.org/226624 | 05:46 |
*** lavaboy has quit IRC | 06:00 | |
*** arnaud_orange has joined #puppet-openstack | 06:09 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Deletes tenant parameter from keystone_user https://review.openstack.org/228744 | 06:11 |
*** lavaboy has joined #puppet-openstack | 06:14 | |
*** jfluhmann has quit IRC | 06:18 | |
*** lavaboy has quit IRC | 06:35 | |
*** crinkle has joined #puppet-openstack | 06:50 | |
*** itsuugo has joined #puppet-openstack | 06:51 | |
*** ric` has joined #puppet-openstack | 06:54 | |
*** ddmitriev has joined #puppet-openstack | 06:55 | |
*** sergmelikyan has joined #puppet-openstack | 07:10 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Cleanup some legacy code https://review.openstack.org/229071 | 07:12 |
openstackgerrit | Merged openstack/puppet-openstack-integration: run_tests: allow to run another scenario https://review.openstack.org/230013 | 07:14 |
*** paramite has joined #puppet-openstack | 07:22 | |
*** dgurtner has joined #puppet-openstack | 07:31 | |
*** gfidente has joined #puppet-openstack | 07:34 | |
openstackgerrit | Martin Mágr proposed openstack/puppet-cinder: Add ability to search nova in the service catalog https://review.openstack.org/230297 | 07:39 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude https://review.openstack.org/214093 | 07:42 |
*** samuelBartel has joined #puppet-openstack | 07:44 | |
*** openstackgerrit has quit IRC | 07:46 | |
*** jpena has joined #puppet-openstack | 07:47 | |
*** openstackgerrit has joined #puppet-openstack | 07:47 | |
*** sergmelikyan has quit IRC | 07:48 | |
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:50 |
*** paramite has quit IRC | 07:51 | |
openstackgerrit | Merged openstack/puppet-neutron: Simplify rpc_backend parameter https://review.openstack.org/227994 | 07:51 |
*** sergmelikyan has joined #puppet-openstack | 07:57 | |
openstackgerrit | Merged openstack/puppet-glance: Add S3 backend configuration for glance https://review.openstack.org/226015 | 07:58 |
*** arnaud_orange has quit IRC | 07:58 | |
openstackgerrit | Merged openstack/puppet-neutron: Map allocation_pool property to set https://review.openstack.org/226156 | 08:03 |
*** derekh has joined #puppet-openstack | 08:04 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-ceilometer: Put all the logging related parameters to the logging class https://review.openstack.org/227705 | 08:06 |
*** arnaud_orange has joined #puppet-openstack | 08:13 | |
openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Put all the logging related parameters to the logging class https://review.openstack.org/227843 | 08:14 |
*** jistr has joined #puppet-openstack | 08:24 | |
openstackgerrit | Derek Higgins proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update" https://review.openstack.org/230314 | 08:31 |
derekh | By the looks of it that commit ^^ brock the tripleo ceph job | 08:32 |
derekh | *broke | 08:32 |
*** sergmelikyan has quit IRC | 08:42 | |
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:53 |
derekh | gildub: ahh ok, I'll take a look, looks like the dsvm test need an update also | 08:56 |
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:57 |
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:58 |
gildub | derekh, no worries, I'll back later => kids (stories/bed) | 08:59 |
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:00 |
*** sergmelikyan has joined #puppet-openstack | 09:02 | |
*** markvoelker has joined #puppet-openstack | 09:04 | |
*** markvoelker has quit IRC | 09:09 | |
*** mattymo has quit IRC | 09:14 | |
*** dmsimard has quit IRC | 09:14 | |
*** openstack has joined #puppet-openstack | 09:22 | |
openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support https://review.openstack.org/216654 | 09:23 |
*** blentz has joined #puppet-openstack | 09:24 | |
*** lavaboy has joined #puppet-openstack | 09:25 | |
*** sergmelikyan has quit IRC | 09:28 | |
*** rahulait has joined #puppet-openstack | 09:32 | |
*** sergmelikyan has joined #puppet-openstack | 09:32 | |
*** sergmelikyan has quit IRC | 09:33 | |
*** paramite has joined #puppet-openstack | 09:37 | |
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:41 |
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:42 |
derekh | gfidente: cool | 09:43 |
*** jtomasek has quit IRC | 09:46 | |
openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: README: update Puppet wiki URL https://review.openstack.org/229856 | 09:47 |
*** jtomasek has 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:48 |
*** aojea_ has joined #puppet-openstack | 09:49 | |
openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: update keystone/auth bits https://review.openstack.org/229849 | 09:49 |
*** rahulait has quit IRC | 09:54 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-nova: Fix virsh secret cleanup script https://review.openstack.org/230348 | 09:58 |
*** sergmelikyan has joined #puppet-openstack | 09:59 | |
*** pcaruana has joined #puppet-openstack | 10:13 | |
*** arnaud_orange has quit IRC | 10:24 | |
gildub | derekh, where are those puppet values stored in the tripleo-ironic-overcloud-f21puppet-nonha? | 10:32 |
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:35 |
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:37 |
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:38 |
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:39 |
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:41 |
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:42 |
*** gildub has quit IRC | 10:43 | |
*** leanderthal has joined #puppet-openstack | 10:54 | |
*** dsutyagin has joined #puppet-openstack | 11:01 | |
*** markvoelker has joined #puppet-openstack | 11:05 | |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 11:06 |
*** skolekonov has joined #puppet-openstack | 11:06 | |
*** markvoelker has quit IRC | 11:09 | |
*** delatte has joined #puppet-openstack | 11:16 | |
*** IvanBerezovskiy has joined #puppet-openstack | 11:16 | |
*** jpena is now known as jpena|lunch | 11:17 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update" https://review.openstack.org/230376 | 11:21 |
*** aojea_ has quit IRC | 11:27 | |
*** itsuugo has quit IRC | 11:27 | |
*** aojea_ has joined #puppet-openstack | 11:37 | |
*** itsuugo has joined #puppet-openstack | 11:37 | |
EmilienM | good morning! | 11:38 |
*** lavaboy has quit IRC | 11:39 | |
iurygregory | good morning :D | 11:45 |
*** pcaruana has quit IRC | 11:47 | |
*** jayg|g0n3 is now known as jayg | 11:53 | |
openstackgerrit | Merged openstack/puppet-heat: Make the role for heat_stack_user configurable https://review.openstack.org/225533 | 11:56 |
*** pradk has joined #puppet-openstack | 12:03 | |
*** markvoelker has joined #puppet-openstack | 12:05 | |
*** guessi_ has joined #puppet-openstack | 12:07 | |
EmilienM | today is friday \o/ | 12:07 |
guessi_ | hi gfidente, | 12:08 |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 12:09 |
*** markvoelker has quit IRC | 12:09 | |
*** markvoelker has joined #puppet-openstack | 12:09 | |
*** guessi_ has left #puppet-openstack | 12:10 | |
*** guessi_ has joined #puppet-openstack | 12:11 | |
*** sergmelikyan has quit IRC | 12:13 | |
*** dprince has joined #puppet-openstack | 12:16 | |
*** jpena|lunch has quit IRC | 12:17 | |
*** jpena|lunch has joined #puppet-openstack | 12:18 | |
gfidente | guessi_, hey | 12:18 |
*** ducttape_ has joined #puppet-openstack | 12:19 | |
guessi_ | sorry for late reply, i'm not familiar with irc :( | 12:19 |
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:20 |
gfidente | looked non-trivial so I voted to revert until we come up with another implementation | 12:21 |
EmilienM | gfidente: should we merge the revert or wait for your patch instread ? cc spredzy | 12:22 |
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:23 |
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:24 |
guessi_ | gfidente, question: how could we tell if the content is changed with base64 encoded id? | 12:26 |
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:27 |
dprince | EmilienM: I would suggest perhaps just having the core team no approve unless that job passes first | 12:28 |
* gfidente back soon | 12:28 | |
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:29 |
EmilienM | dprince, gfidente, guessi_: revert patch is +2 +A | 12:30 |
EmilienM | dprince: cool | 12:30 |
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:31 |
guessi_ | forget the reference link: https://libvirt.org/formatsecret.html#SecretAttributes | 12:32 |
*** ducttape_ has quit IRC | 12:33 | |
*** ducttape_ has joined #puppet-openstack | 12:34 | |
*** delatte has quit IRC | 12:36 | |
*** delatte has joined #puppet-openstack | 12:37 | |
*** ducttape_ has quit IRC | 12:39 | |
*** jpena|lunch is now known as jpena | 12:40 | |
*** guessi_ has quit IRC | 12:42 | |
*** guessi has joined #puppet-openstack | 12:43 | |
aderyugin | Hi, folks! Can you please take a look at my commit https://review.openstack.org/#/c/216682/ I hope it's ready | 12:44 |
*** itsuugo has quit IRC | 12:44 | |
*** aojea_ has quit IRC | 12:44 | |
openstackgerrit | Merged openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update" https://review.openstack.org/230314 | 12:45 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-barbican: puppet-barbican: Initial commit https://review.openstack.org/229871 | 12:50 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency https://review.openstack.org/230413 | 12:52 |
*** richm has joined #puppet-openstack | 12:52 | |
*** morazi has quit IRC | 12:57 | |
*** itsuugo has joined #puppet-openstack | 12:59 | |
*** aojea_ has joined #puppet-openstack | 13:00 | |
*** morazi has joined #puppet-openstack | 13:00 | |
*** tiswanso has joined #puppet-openstack | 13:08 | |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 13:17 |
vinsh | jpena: Good eye!! ^^ | 13:18 |
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:19 |
*** skolekonov has quit IRC | 13:24 | |
*** guessi has quit IRC | 13:26 | |
*** guessi has joined #puppet-openstack | 13:27 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios https://review.openstack.org/230431 | 13:40 |
mattymo_ | Does anyone know about the state of puppet-openstack modules with Puppet 4.1.0? | 13:42 |
mattymo_ | dprince, EmilienM do you guys know? | 13:44 |
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 has 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:47 |
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:48 |
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:49 |
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:50 |
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:51 |
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:52 |
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_ has joined #puppet-openstack | 13:53 | |
*** khushbu has joined #puppet-openstack | 13:53 | |
EmilienM | but I don't see huge value | 13:53 |
EmilienM | mattymo_: have you hit any issue? | 13:54 |
mattymo_ | EmilienM, not yet, thankfully | 13:54 |
*** paramite is now known as paramite|afk | 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:55 |
*** sanjayu has joined #puppet-openstack | 13:56 | |
*** itsuugo has quit IRC | 13:59 | |
*** aojea_ has quit IRC | 13:59 | |
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 has joined #puppet-openstack | 14:01 | |
mattymo_ | mwhahaha, but as long as we're moving, just go to something even newer | 14:02 |
mwhahaha | true but i doubt fuel is 4.x compatible :D | 14:03 |
mattymo_ | mwhahaha, we won't know until we try | 14:06 |
*** v1k0d3n has quit IRC | 14:14 | |
*** itsuugo has joined #puppet-openstack | 14:17 | |
*** aojea_ has joined #puppet-openstack | 14:17 | |
*** v1k0d3n_ has quit IRC | 14:24 | |
*** sergmelikyan has quit IRC | 14:25 | |
*** khushbu has quit IRC | 14:26 | |
*** khushbu has joined #puppet-openstack | 14:28 | |
*** paramite|afk is now known as paramite | 14:32 | |
*** khushbu has quit IRC | 14:39 | |
*** sanjayu has quit IRC | 14:39 | |
*** sanjayu has joined #puppet-openstack | 14:40 | |
*** khushbu has joined #puppet-openstack | 14:40 | |
*** dprince has quit IRC | 14:41 | |
openstackgerrit | Merged openstack/puppet-ceilometer: wsgi: make sure eventlet process is stopped before httpd https://review.openstack.org/228597 | 14:44 |
*** paramite is now known as paramite|afk | 14:48 | |
*** khushbu has quit IRC | 14:49 | |
openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency https://review.openstack.org/230413 | 14:53 |
*** dfisher has joined #puppet-openstack | 14:57 | |
*** guessi has left #puppet-openstack | 15:01 | |
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:02 |
openstackgerrit | Merged openstack/puppet-glance: Release 5.2.0 - Juno https://review.openstack.org/230171 | 15:03 |
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:04 |
openstackgerrit | Merged openstack/puppet-ceilometer: Release 5.2.0 - Juno https://review.openstack.org/230166 | 15:05 |
*** paramite|afk is now known as paramite | 15:07 | |
openstackgerrit | Merged openstack/puppet-openstack_extras: Release 5.1.0 - Juno https://review.openstack.org/230182 | 15:09 |
*** dweaver has quit IRC | 15:10 | |
*** herewego has joined #puppet-openstack | 15:14 | |
openstackgerrit | Merged openstack/puppet-nova: Release 5.2.0 - Juno https://review.openstack.org/230179 | 15:14 |
*** sanjayu has quit IRC | 15:16 | |
*** sanjayu has joined #puppet-openstack | 15:16 | |
openstackgerrit | John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support https://review.openstack.org/223690 | 15:20 |
*** sanjayu has quit IRC | 15:22 | |
*** sanjayu has 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:24 |
degorenko | hello guys. EmilienM sbadia mfisch spredzy take a look please: https://review.openstack.org/#/c/229895 | 15:25 |
*** leanderthal has quit IRC | 15:26 | |
openstackgerrit | Merged openstack/puppet-neutron: Release 5.2.0 - Juno https://review.openstack.org/230174 | 15:27 |
*** sanjayu has quit IRC | 15:28 | |
*** linkedinyou has quit IRC | 15:30 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Rewrite nova_network provider with using only nova client https://review.openstack.org/229548 | 15:30 |
*** linkedinyou has joined #puppet-openstack | 15:31 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth https://review.openstack.org/226862 | 15:33 |
*** jfluhmann has joined #puppet-openstack | 15:56 | |
*** dprince has joined #puppet-openstack | 16:01 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: DNM - CI test https://review.openstack.org/230524 | 16:02 |
*** IvanBerezovskiy has left #puppet-openstack | 16:03 | |
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:04 |
iurygregory | any core around can review https://review.openstack.org/#/c/223777/ ? already have a +2 ^^ | 16:06 |
mfisch | looking | 16:08 |
iurygregory | Thanks mfisch o/ | 16:10 |
EmilienM | I just found out https://github.com/mgagne/openstack-puppet-release-tools | 16:13 |
*** ducttape_ has quit IRC | 16:13 | |
*** ducttape_ has joined #puppet-openstack | 16:16 | |
mgagne | that's the tool ^ | 16:16 |
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:21 |
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:22 |
*** lavaboy has joined #puppet-openstack | 16:23 | |
mgagne | no release for horizon or this has yet to happen? | 16:25 |
openstackgerrit | Merged openstack/puppet-openstack-specs: Changes in enabling federation spec https://review.openstack.org/223777 | 16:26 |
EmilienM | mgagne: no release | 16:26 |
mgagne | ok | 16:27 |
EmilienM | mgagne: https://docs.google.com/spreadsheets/d/1XVrmEiLrJSdxDo-S_vFB7ljxTdYg-pe8hiMUryRor5A/edit#gid=0 | 16:28 |
EmilienM | mgagne: you can update only those in green, in Launchpad column | 16:29 |
mgagne | all done | 16:29 |
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:30 |
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:31 |
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:32 |
*** paramite has quit IRC | 16:39 | |
clayton | ah, I see the fact review. | 16:39 |
*** degorenko has quit IRC | 16:39 | |
*** karmix has quit IRC | 16:39 | |
clayton | I'll leave my review oustanding and fix it to use the fact when that one is merged. | 16:39 |
*** dgurtner has quit IRC | 16:45 | |
*** jpena has quit IRC | 16:50 | |
*** dsutyagin has quit IRC | 16:51 | |
*** xarses has quit IRC | 16:53 | |
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 |
*** apuimedo has quit IRC | 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:56 |
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 | 16:57 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: include glance::backend::file https://review.openstack.org/230556 | 17:00 |
*** jamespage has 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:01 |
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:02 |
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:03 |
*** sanjayu has 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:04 |
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:05 |
*** herewego has quit IRC | 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:07 |
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 has joined #puppet-openstack | 17:08 | |
EmilienM | #stores = file,http | 17:08 |
EmilienM | #default_store = file | 17:08 |
jamespage | well that is odd | 17:10 |
*** jistr has quit IRC | 17:10 | |
EmilienM | jamespage: https://review.openstack.org/#/q/owner:%22Emilien+Macchi%22+status:open+topic:%22ci-test%22,n,z | 17:12 |
EmilienM | we will see what else is coming :) | 17:13 |
jamespage | well on monday | 17:14 |
jamespage | sorry have to eod | 17:14 |
EmilienM | jamespage: any other contact? | 17:14 |
jamespage | coreycb | 17:15 |
EmilienM | maybe coreycb could come here | 17:15 |
jamespage | he's in #ubuntu-server | 17:15 |
*** trown has 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:15 |
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:16 |
EmilienM | it's weird it works in RDO and not in UCA | 17:17 |
EmilienM | you both run liberty | 17:17 |
*** trown is now known as trown|lunch | 17:17 | |
*** xarses has joined #puppet-openstack | 17:18 | |
*** derekh has quit IRC | 17:18 | |
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:19 |
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:20 |
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:21 |
*** ducttape_ has quit IRC | 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:22 |
EmilienM | and no, we don't want to deal with packaging dependencies in our modules, that's dupplicated work with packaging folks. | 17:23 |
dfisher | at best it's a duplicate. at worst it's a conflict. | 17:24 |
*** sanjayu has quit IRC | 17:24 | |
dfisher | puppet says install x y z, packaging says x y a b c | 17:24 |
*** sanjayu has joined #puppet-openstack | 17:24 | |
*** imcsk8 has quit IRC | 17:25 | |
*** imcsk8 has joined #puppet-openstack | 17:25 | |
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:27 |
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:28 |
EmilienM | jamespage: no worries, ok I'll find a bug and let you know. see you on monday | 17:32 |
jamespage | EmilienM, you might see me in +3 hrs | 17:33 |
jamespage | ;) | 17:33 |
*** sanjayu has quit IRC | 17:46 | |
*** sanjayu has joined #puppet-openstack | 17:48 | |
*** jfluhmann has quit IRC | 17:55 | |
*** ducttape_ has joined #puppet-openstack | 17:59 | |
*** delatte has quit IRC | 18:06 | |
EmilienM | jamespage: I have a good news, glance is the only issue I think | 18:10 |
EmilienM | crinkle: do you still -1 https://review.openstack.org/#/c/226830/ after the recent changes ? I addressed your comment. | 18:16 |
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:20 |
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:21 |
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:22 |
*** trown|lunch is now known as trown | 18:23 | |
mfisch | I agree thats the goal | 18:25 |
*** sanjayu has quit IRC | 18:26 | |
*** sanjayu has joined #puppet-openstack | 18:27 | |
crinkle | EmilienM: still -1 | 18:28 |
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:29 |
EmilienM | crinkle: ack your comment, working on it too :-) | 18:30 |
EmilienM | crinkle: I'm linking because it's simplier than cloning, also, fixtures is also using links. | 18:31 |
*** aojea_ has quit IRC | 18:31 | |
*** itsuugo has quit IRC | 18:31 | |
EmilienM | crinkle: the rm is because the ln_s fails if the file already exist. | 18:31 |
EmilienM | crinkle: which is also something fixtures does | 18:32 |
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:39 |
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:43 |
EmilienM | trown: trust works | 18:44 |
EmilienM | trusty* | 18:44 |
*** sergmelikyan has joined #puppet-openstack | 18:44 | |
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:46 |
trown | trusty failed on port 5050 not listening | 18:47 |
trown | which it should not have run if the conditional evaluated to true | 18:47 |
trown | EmilienM: I am unconvinced that worked before... it would have failed silently | 18:49 |
*** sergmelikyan has quit IRC | 18:49 | |
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:51 |
*** gfidente has quit IRC | 18:52 | |
*** sergmelikyan has joined #puppet-openstack | 18:54 | |
crinkle | EmilienM: yes | 18:54 |
EmilienM | crinkle: so I need to create clonemap.yaml, etc ? | 18:55 |
EmilienM | crinkle: it won't when you will run tests on your laptop | 18:56 |
EmilienM | won't run* | 18:56 |
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:57 |
*** gchamoul has quit IRC | 18:58 | |
*** sergmelikyan has quit IRC | 18:58 | |
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 has joined #puppet-openstack | 18:59 | |
EmilienM | crinkle: apologizes, but I don't understand the symlink problem. | 19:00 |
*** sanjayu has quit IRC | 19:00 | |
*** sanjayu has joined #puppet-openstack | 19:01 | |
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:02 |
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:03 |
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:04 |
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:05 |
EmilienM | ok, trying | 19:06 |
EmilienM | it works, I update | 19:07 |
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:08 |
*** sergmelikyan has joined #puppet-openstack | 19:16 | |
*** aojea_ has joined #puppet-openstack | 19:16 | |
*** itsuugo has joined #puppet-openstack | 19:17 | |
*** itsuugo has quit IRC | 19:25 | |
*** aojea_ has quit IRC | 19:25 | |
*** xarses has quit IRC | 19:25 | |
*** xarses has joined #puppet-openstack | 19:25 | |
*** itsuugo has joined #puppet-openstack | 19:26 | |
*** aojea_ has joined #puppet-openstack | 19:26 | |
*** sanjayu has quit IRC | 19:32 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-openstacklib: Add os_service_default fact https://review.openstack.org/229050 | 19:33 |
*** sanjayu has joined #puppet-openstack | 19:33 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact https://review.openstack.org/229095 | 19:37 |
*** sanjayu has quit IRC | 19:49 | |
*** sanjayu has joined #puppet-openstack | 19:49 | |
zigo | EmilienM: FYI, the Debian package does work with the file backend by default. | 19:53 |
*** sanjayu has quit IRC | 19:55 | |
EmilienM | zigo: do you install swiftclient by default? | 19:56 |
zigo | Looking ... | 19:56 |
zigo | EmilienM: Nop, but I can if you wish. | 19:56 |
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:57 |
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:58 |
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 ! :) | 19:59 |
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:00 |
dfisher | see? add a dependency. | 20:01 |
dfisher | it's so easy. | 20:01 |
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:02 |
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:03 |
*** skolekonov has joined #puppet-openstack | 20:04 | |
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:05 |
EmilienM | for example: https://github.com/openstack/puppet-ceilometer/blob/master/manifests/api.pp#L108 | 20:06 |
mwhahaha | yea i know that, doesn't mean everyone else wants to :D | 20:07 |
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:09 |
*** sanjayu has joined #puppet-openstack | 20:15 | |
EmilienM | crinkle: I found the ceilo/wsgi bug | 20:15 |
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:23 |
mfisch | is Maksym Yatsenko in this channel? | 20:25 |
*** jayg is now known as jayg|g0n3 | 20:29 | |
EmilienM | myatsenko: ^ | 20:29 |
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:32 |
*** dprince has quit IRC | 20:39 | |
*** skolekonov has quit IRC | 20:48 | |
*** angdraug has joined #puppet-openstack | 20:53 | |
*** skolekonov has joined #puppet-openstack | 20:57 | |
*** ducttape_ has quit IRC | 21:00 | |
*** skolekonov has quit IRC | 21:01 | |
*** trown is now known as trown|outttypeww | 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:03 |
EmilienM | the parameter is passed, see https://review.openstack.org/#/c/230556/1/spec/acceptance/basic_glance_spec.rb,cm | 21:04 |
*** sergmelikyan has quit IRC | 21:05 | |
crinkle | EmilienM: glance::backend::file inherits glance::api | 21:09 |
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:10 |
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:11 |
EmilienM | iurygregory: you too :) | 21:12 |
iurygregory | ^^ | 21:12 |
dfisher | beers? sake* | 21:12 |
*** iurygregory has quit IRC | 21:12 | |
dfisher | oh. puppetconf. | 21:12 |
dfisher | ignore me. | 21:12 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: file backend: do not inherit from glance::api https://review.openstack.org/230556 | 21:15 |
EmilienM | so if we want integration jobs back alive, we need https://review.openstack.org/230634 and https://review.openstack.org/230556 | 21:16 |
*** tiswanso has quit IRC | 21:19 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios https://review.openstack.org/230431 | 21:20 |
*** karmix has joined #puppet-openstack | 21:22 | |
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:23 |
*** zeus has quit IRC | 21:23 | |
*** richm has quit IRC | 21:23 | |
*** zeus has joined #puppet-openstack | 21:25 | |
*** zeus is now known as Guest30442 | 21:25 | |
*** richm has joined #puppet-openstack | 21:27 | |
*** rahulait has joined #puppet-openstack | 21:36 | |
*** aojea_ has quit IRC | 21:39 | |
*** itsuugo has quit IRC | 21:39 | |
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:40 |
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:41 |
*** devnull0 has joined #puppet-openstack | 21:42 | |
EmilienM | mtreinish: I'm proposing a patch | 21:42 |
mtreinish | EmilienM: ok | 21:42 |
EmilienM | mtreinish: wait, they already have the python decorator | 21:43 |
EmilienM | @test.services('compute', 'network') | 21:43 |
*** bnemec has 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:44 |
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:45 |
mtreinish | EmilienM: well you need nova, glance, etc for heat, sahara, trove to work | 21:46 |
EmilienM | arg | 21:46 |
EmilienM | ok | 21:46 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios https://review.openstack.org/230431 | 21:54 |
*** bnemec has quit IRC | 22:06 | |
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:07 |
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:08 |
EmilienM | jamespage: I have issues with cinder & glance so far. I'll investigate later | 22:14 |
*** guessi has quit IRC | 22:25 | |
*** Guest30442 is now known as zeus | 22:33 | |
*** zeus has quit IRC | 22:33 | |
*** zeus has joined #puppet-openstack | 22:33 | |
*** sergmelikyan has joined #puppet-openstack | 22:36 | |
*** EmilienM sets mode: +o crinkle | 22:37 | |
*** EmilienM sets mode: +o clayton | 22:37 | |
*** EmilienM sets mode: +o michchap | 22:37 | |
*** sanjayu has quit IRC | 22:39 | |
*** sergmelikyan has quit IRC | 22:40 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios https://review.openstack.org/230431 | 22:42 |
*** jasondotstar is now known as jasondotstar|afk | 22:48 | |
*** devnull0 has quit IRC | 22:53 | |
*** dfisher has quit IRC | 22:57 | |
*** sanjayu has joined #puppet-openstack | 23:00 | |
*** mdorman has quit IRC | 23:01 | |
*** sergmelikyan has joined #puppet-openstack | 23:37 | |
*** sergmelikyan has quit IRC | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!