Friday, 2015-10-02

openstackgerritMerged openstack/puppet-openstack-integration: run_tests: drop dupplicated code  https://review.openstack.org/22942600:02
openstackgerritMerged openstack/puppet-openstack-integration: scenario001: use 2 workers for sahara-api  https://review.openstack.org/22947000:02
*** gildub has quit IRC00:03
*** mdorman has quit IRC00:04
openstackgerritMerged openstack/puppet-tempest: Allow to enable Trove tests  https://review.openstack.org/22944500:06
*** xarses has joined #puppet-openstack00:07
*** gildub has joined #puppet-openstack00:19
*** markvoelker has joined #puppet-openstack00:30
*** markvoelker has quit IRC00:35
*** itsuugo has joined #puppet-openstack00:38
*** rahulait has joined #puppet-openstack00:42
*** sanjayu has quit IRC00:44
*** xingchao_ has joined #puppet-openstack00:55
*** richm has quit IRC00:57
*** xingchao has quit IRC00:57
*** jfluhmann has joined #puppet-openstack01:06
*** ibravo has quit IRC01:15
*** rahulait has quit IRC01:24
*** angdraug has quit IRC01:44
*** rahulait has joined #puppet-openstack01:59
*** jfluhmann has quit IRC01:59
*** rahulait has quit IRC02:11
*** markvoelker has joined #puppet-openstack02:32
*** jfluhmann has joined #puppet-openstack02:34
*** markvoelker has quit IRC02:36
*** jfluhmann has quit IRC02:53
*** ferest has joined #puppet-openstack02:55
*** ferest has quit IRC03:00
*** jfluhmann has joined #puppet-openstack03:02
*** gildub has quit IRC03:27
*** itsuugo has quit IRC03:34
*** gildub has joined #puppet-openstack03:43
*** bnemec has quit IRC04:02
*** jfluhmann has quit IRC04:04
*** xingchao_ has quit IRC04:11
*** jfluhmann has joined #puppet-openstack04:17
*** jfluhmann has quit IRC04:27
*** markvoelker has joined #puppet-openstack04:33
*** jfluhmann has joined #puppet-openstack04:39
*** crinkle has quit IRC05:04
*** markvoelker has quit IRC05:07
*** lavaboy has joined #puppet-openstack05:32
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Replace indirection calls  https://review.openstack.org/22662405:46
*** lavaboy has quit IRC06:00
*** arnaud_orange has joined #puppet-openstack06:09
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Deletes tenant parameter from keystone_user  https://review.openstack.org/22874406:11
*** lavaboy has joined #puppet-openstack06:14
*** jfluhmann has quit IRC06:18
*** lavaboy has quit IRC06:35
*** crinkle has joined #puppet-openstack06:50
*** itsuugo has joined #puppet-openstack06:51
*** ric` has joined #puppet-openstack06:54
*** ddmitriev has joined #puppet-openstack06:55
*** sergmelikyan has joined #puppet-openstack07:10
openstackgerritMerged openstack/puppet-openstack-integration: Cleanup some legacy code  https://review.openstack.org/22907107:12
openstackgerritMerged openstack/puppet-openstack-integration: run_tests: allow to run another scenario  https://review.openstack.org/23001307:14
*** paramite has joined #puppet-openstack07:22
*** dgurtner has joined #puppet-openstack07:31
*** gfidente has joined #puppet-openstack07:34
openstackgerritMartin Mágr proposed openstack/puppet-cinder: Add ability to search nova in the service catalog  https://review.openstack.org/23029707:39
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude  https://review.openstack.org/21409307:42
*** samuelBartel has joined #puppet-openstack07:44
*** openstackgerrit has quit IRC07:46
*** jpena has joined #puppet-openstack07:47
*** openstackgerrit has joined #puppet-openstack07:47
*** sergmelikyan has quit IRC07:48
openstackgerritMerged openstack/puppet-cinder: Fix RSpec 3.x syntax  https://review.openstack.org/22976007:49
openstackgerritMerged openstack/puppet-ironic: Fix RSpec 3.x syntax  https://review.openstack.org/22979307:50
*** paramite has quit IRC07:51
openstackgerritMerged openstack/puppet-neutron: Simplify rpc_backend parameter  https://review.openstack.org/22799407:51
*** sergmelikyan has joined #puppet-openstack07:57
openstackgerritMerged openstack/puppet-glance: Add S3 backend configuration for glance  https://review.openstack.org/22601507:58
*** arnaud_orange has quit IRC07:58
openstackgerritMerged openstack/puppet-neutron: Map allocation_pool property to set  https://review.openstack.org/22615608:03
*** derekh has joined #puppet-openstack08:04
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Put all the logging related parameters to the logging class  https://review.openstack.org/22770508:06
*** arnaud_orange has joined #puppet-openstack08:13
openstackgerritYanis Guenane proposed openstack/puppet-keystone: Put all the logging related parameters to the logging class  https://review.openstack.org/22784308:14
*** jistr has joined #puppet-openstack08:24
openstackgerritDerek Higgins proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/23031408:31
derekhBy the looks of it that commit ^^ brock the tripleo ceph job08:32
derekh*broke08:32
*** sergmelikyan has quit IRC08:42
gildubderekh, 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 call08:53
derekhgildub: ahh ok, I'll take a look, looks like the dsvm test need an update also08:56
gildubderekh, more precisely heat_admin::heat_domain@::heat_stack08:57
gildubderekh, the dsvm?08:57
gildubderekh, I mean, anything wrong? Looking...08:57
derekhgildub: http://logs.openstack.org/24/226624/15/check/gate-puppet-openstack-integration-dsvm-centos7/983075a/console.html#_2015-10-02_06_07_40_50508:58
derekhgildub: brb08:58
gildubderekh, yeah, do mean the integration-dsvm ones? They are non voting08:58
gildubderekh, no worries, I'll back later => kids (stories/bed)08:59
derekhgildub: I know, I would have thought we would want to keep them passing if we can, do they normally pass ?09:00
derekhgildub: no prob, thans for the pointers, I'll get back to looking at it in a bit09:00
*** sergmelikyan has joined #puppet-openstack09:02
*** markvoelker has joined #puppet-openstack09:04
*** markvoelker has quit IRC09:09
*** mattymo has quit IRC09:14
*** dmsimard has quit IRC09:14
*** openstack has joined #puppet-openstack09:22
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665409:23
*** blentz has joined #puppet-openstack09:24
*** lavaboy has joined #puppet-openstack09:25
*** sergmelikyan has quit IRC09:28
*** rahulait has joined #puppet-openstack09:32
*** sergmelikyan has joined #puppet-openstack09:32
*** sergmelikyan has quit IRC09:33
*** paramite has joined #puppet-openstack09:37
gfidentederekh, 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
gfidentethat returns 0 even when there is no secret with that uuid09:42
gfidentebut looks fixable, so we probably don't need the revert, trying09:42
derekhgfidente: cool09:43
*** jtomasek has quit IRC09:46
openstackgerritMerged openstack/puppet-openstack-cookiecutter: README: update Puppet wiki URL  https://review.openstack.org/22985609:47
*** jtomasek has joined #puppet-openstack09:48
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Extend the logging class capabilities.  https://review.openstack.org/22989909:48
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Updates on config provider  https://review.openstack.org/22943909:48
*** aojea_ has joined #puppet-openstack09:49
openstackgerritMerged openstack/puppet-openstack-cookiecutter: update keystone/auth bits  https://review.openstack.org/22984909:49
*** rahulait has quit IRC09:54
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Fix virsh secret cleanup script  https://review.openstack.org/23034809:58
*** sergmelikyan has joined #puppet-openstack09:59
*** pcaruana has joined #puppet-openstack10:13
*** arnaud_orange has quit IRC10:24
gildubderekh, where are those puppet values stored in the tripleo-ironic-overcloud-f21puppet-nonha?10:32
derekhgildub: 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
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Update the autobuild.asc key URL  https://review.openstack.org/23036210:35
gfidentegildub, 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
gildubgfidente, derekh, not the same issue, I'm talking about https://review.openstack.org/#/c/22662410:38
gildubderekh, that said ^ thanks for the hieradata pointer :)10:38
derekhgildub: gfidente sorry, got confused with what I was doing10:38
derekhgildub: gfidente tw0 seperate issues10:39
gildubderekh, no worries10:39
derekhgildub: I've yet to look into it properly,10:39
gfidentedo you guys know if Kuo-Le.Mei is online and which nick he has?10:41
gildubderekh, ok, I'm going to call it a week, just leave me a comment in the review10:41
derekhgildub: cool, have a good weekend, I should have something for you by monday10:42
derekhgfidente: dunno10:42
gildubderekh, thanks, have a good one10:42
*** gildub has quit IRC10:43
*** leanderthal has joined #puppet-openstack10:54
*** dsutyagin has joined #puppet-openstack11:01
*** markvoelker has joined #puppet-openstack11:05
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369011:06
*** skolekonov has joined #puppet-openstack11:06
*** markvoelker has quit IRC11:09
*** delatte has joined #puppet-openstack11:16
*** IvanBerezovskiy has joined #puppet-openstack11:16
*** jpena is now known as jpena|lunch11:17
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/23037611:21
*** aojea_ has quit IRC11:27
*** itsuugo has quit IRC11:27
*** aojea_ has joined #puppet-openstack11:37
*** itsuugo has joined #puppet-openstack11:37
EmilienMgood morning!11:38
*** lavaboy has quit IRC11:39
iurygregorygood morning :D11:45
*** pcaruana has quit IRC11:47
*** jayg|g0n3 is now known as jayg11:53
openstackgerritMerged openstack/puppet-heat: Make the role for heat_stack_user configurable  https://review.openstack.org/22553311:56
*** pradk has joined #puppet-openstack12:03
*** markvoelker has joined #puppet-openstack12:05
*** guessi_ has joined #puppet-openstack12:07
EmilienMtoday is friday \o/12:07
guessi_hi gfidente,12:08
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369012:09
*** markvoelker has quit IRC12:09
*** markvoelker has joined #puppet-openstack12:09
*** guessi_ has left #puppet-openstack12:10
*** guessi_ has joined #puppet-openstack12:11
*** sergmelikyan has quit IRC12:13
*** dprince has joined #puppet-openstack12:16
*** jpena|lunch has quit IRC12:17
*** jpena|lunch has joined #puppet-openstack12:18
gfidenteguessi_, hey12:18
*** ducttape_ has joined #puppet-openstack12:19
guessi_sorry for late reply, i'm not familiar with irc :(12:19
gfidenteguessi_, hey no worries, welcome12:20
guessi_i'm trying to solve the issue with grepping problem,12:20
gfidenteso I saw your comment, indeed I think we should compare the key contents but that is in libvirt_key not in rbd_keysring12:20
guessi_does "test -z XXX" look good to you?12:20
gfidenteso we should somehow parse the output of dumpxml to get the base64 encoded string corresponding to the key12:20
gfidentelooked non-trivial so I voted to revert until we come up with another implementation12:21
EmilienMgfidente: should we merge the revert or wait for your patch instread ? cc spredzy12:22
gfidenteI think the revert is safer for now12:23
EmilienMok12:23
EmilienMspredzy: ==12:23
EmilienMlet's revert it now12:23
gfidentebecause the patch might make it pass, but it won't behave as intended, it's going to remove/recreate the key every time12:23
guessi_i'm okay for the revert12:23
EmilienMguessi_: thanks12:23
gfidenteguessi_, the problem is parsing that xml output12: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-L10412:24
EmilienMgfidente, dprince: we might need to had the tripleo/ceph job in puppet-cinder & puppet-nova gate.12:24
gfidenteguessi_, the key is base64 encoded so libvirt_key won't contain the same string as found in the virsh dump12:24
guessi_gfidente, question: how could we tell if the content is changed with base64 encoded id?12:26
gfidenteyeah I think we should basically re-encode the new libvirt_key, parse the xmldump and compare those two12:27
gfidentebecause the xmldump will emit it base64 encoded12:27
gfidentethen, if those differ it means the key has been updated and we should execute the cleanup12:27
dprinceEmilienM: I would suggest perhaps just having the core team no approve unless that job passes first12:28
* gfidente back soon12:28
EmilienMdprince: the problem is OOO CI is unstable to be fair, we can't wait everytime OOO CI is green to merge a patch12:29
EmilienMbut having ceph job for puppet-cinder & glance could help12:29
dprinceEmilienM: right, which is why I'm suggesting not gating on it12:29
EmilienMok12:29
dprinceEmilienM: just watch it closely, and if our stability improves then we can consider other ideas...12:29
EmilienMdprince, gfidente, guessi_: revert patch is +2 +A12:30
EmilienMdprince: cool12: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 :p12:31
guessi_@EmilienM: thanks !12:31
guessi_forget the reference link: https://libvirt.org/formatsecret.html#SecretAttributes12:32
*** ducttape_ has quit IRC12:33
*** ducttape_ has joined #puppet-openstack12:34
*** delatte has quit IRC12:36
*** delatte has joined #puppet-openstack12:37
*** ducttape_ has quit IRC12:39
*** jpena|lunch is now known as jpena12:40
*** guessi_ has quit IRC12:42
*** guessi has joined #puppet-openstack12:43
aderyuginHi, folks! Can you please take a look at my commit https://review.openstack.org/#/c/216682/ I hope it's ready12:44
*** itsuugo has quit IRC12:44
*** aojea_ has quit IRC12:44
openstackgerritMerged openstack/puppet-nova: Revert "Fixed value `rbd.keyring` not update"  https://review.openstack.org/23031412:45
openstackgerritEmilien Macchi proposed openstack/puppet-barbican: puppet-barbican: Initial commit  https://review.openstack.org/22987112:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency  https://review.openstack.org/23041312:52
*** richm has joined #puppet-openstack12:52
*** morazi has quit IRC12:57
*** itsuugo has joined #puppet-openstack12:59
*** aojea_ has joined #puppet-openstack13:00
*** morazi has joined #puppet-openstack13:00
*** tiswanso has joined #puppet-openstack13:08
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/20322013:17
vinshjpena: Good eye!! ^^13:18
vinshI 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 IRC13:24
*** guessi has quit IRC13:26
*** guessi has joined #puppet-openstack13:27
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/23043113: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
dprincemattymo_: I've not used 4.1 yet myself, I don't think unit test coverage is there yet either13:46
dprincemattymo_: I take that back, fedora 22 has 4.113:46
*** mdorman has joined #puppet-openstack13:46
EmilienMicehouse? we don't officially support it anymore.13:46
dprincemattymo_: so I have used it, but not w/ all of the OPM modules13:47
EmilienMmattymo_: https://wiki.openstack.org/wiki/Releases13:47
EmilienMIcehouse is EOL13:47
EmilienMI 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 tool13:48
mattymo_dprince answered my question. Thanks!13:48
EmilienMah13:48
EmilienMmattymo_: sorry, I'm in the middle of releasing Puppet modules13:49
EmilienMso I thought you were wondering about the modules themselves13:49
dprincemattymo_: I did try running puppet 4.1 to general configs only (in a docker container). It worked fine there w/ OPM13:49
* EmilienM facepalm13:49
dprincemattymo_: generate configs13:49
EmilienMmattymo_, dprince: our modules are gated by Puppet 4.x13:49
EmilienMI don't see any issue, and it should work pretty good.13:49
dprinceEmilienM: yeah, I think it is close.13:50
dprinceEmilienM: but we aren't gating on it yet13:50
EmilienMdprince: what?13:50
dprinceEmilienM: 4.113:50
dprinceEmilienM: I thought we just used 4.013:50
dprinceEmilienM: 4.0 doesn't imply 4.x does it?13:51
EmilienMhttp://logs.openstack.org/97/228597/2/check/gate-puppet-ceilometer-puppet-unit-latest-dsvm-centos7/4735ca4/console.html#_2015-10-01_21_30_46_81013:51
EmilienMdprince: ^13:51
EmilienM4.2.213:51
EmilienMdprince, mattymo_ : please rtfm : https://wiki.openstack.org/wiki/Puppet/CI13:51
EmilienM:)13:51
dprinceEmilienM: okay, so latest covers us here13:51
EmilienMyup13:51
dprinceEmilienM: but are we skipping 4.113:51
EmilienMdprince: well... we can't test all 4.x & 3.x versions.13:52
dprinceEmilienM: I see 4.0, and then apparently latest is 4.213:52
EmilienMwe test 4.0 as voting13:52
EmilienMand latest as non voting13:52
dprinceEmilienM: right, so we actually aren't testing 4.1 specifically, but it should work fine we think13:52
mattymo_EmilienM, I found 4.0 covered but not 4.1. That's why I asked13:53
EmilienMdprince: we could bump the 4.0 job to 4.113:53
*** ducttape_ has joined #puppet-openstack13:53
*** khushbu has joined #puppet-openstack13:53
EmilienMbut I don't see huge value13:53
EmilienMmattymo_: have you hit any issue?13:54
mattymo_EmilienM, not yet, thankfully13:54
*** paramite is now known as paramite|afk13:54
EmilienMmattymo_: I don't think you'll have any issue13:54
mattymo_we're just rebasing puppet version in fuel. We're very far behind the times13:55
EmilienMour modules are really close to latest puppet version, rspec and upstream dependencies13:55
*** sanjayu has joined #puppet-openstack13:56
*** itsuugo has quit IRC13:59
*** aojea_ has quit IRC13:59
mwhahahamattymo_: thought we were updating to 3.8 not 4.x14:01
mattymo_mwhahaha, yeah I thought so too :D14:01
*** sergmelikyan has joined #puppet-openstack14:01
mattymo_mwhahaha, but as long as we're moving, just go to something even newer14:02
mwhahahatrue but i doubt fuel is 4.x compatible :D14:03
mattymo_mwhahaha, we won't know until we try14:06
*** v1k0d3n has quit IRC14:14
*** itsuugo has joined #puppet-openstack14:17
*** aojea_ has joined #puppet-openstack14:17
*** v1k0d3n_ has quit IRC14:24
*** sergmelikyan has quit IRC14:25
*** khushbu has quit IRC14:26
*** khushbu has joined #puppet-openstack14:28
*** paramite|afk is now known as paramite14:32
*** khushbu has quit IRC14:39
*** sanjayu has quit IRC14:39
*** sanjayu has joined #puppet-openstack14:40
*** khushbu has joined #puppet-openstack14:40
*** dprince has quit IRC14:41
openstackgerritMerged openstack/puppet-ceilometer: wsgi: make sure eventlet process is stopped before httpd  https://review.openstack.org/22859714:44
*** paramite is now known as paramite|afk14:48
*** khushbu has quit IRC14:49
openstackgerritMerged openstack/puppet-openstack-cookiecutter: db::sync: drop User dependency  https://review.openstack.org/23041314:53
*** dfisher has joined #puppet-openstack14:57
*** guessi has left #puppet-openstack15:01
openstackgerritMerged openstack/puppet-barbican: puppet-barbican: Initial commit  https://review.openstack.org/22987115:02
openstackgerritMerged openstack/puppet-keystone: Release 5.2.0 - Juno  https://review.openstack.org/23017315:02
openstackgerritMerged openstack/puppet-glance: Release 5.2.0 - Juno  https://review.openstack.org/23017115:03
openstackgerritMerged openstack/puppet-cinder: Release 5.2.0 - Juno  https://review.openstack.org/23017015:04
openstackgerritMerged openstack/puppet-heat: Release 5.2.0 - Juno  https://review.openstack.org/23017215:04
openstackgerritMerged openstack/puppet-aodh: puppet-aodh: Initial commit  https://review.openstack.org/22501515:04
openstackgerritMerged openstack/puppet-ceilometer: Release 5.2.0 - Juno  https://review.openstack.org/23016615:05
*** paramite|afk is now known as paramite15:07
openstackgerritMerged openstack/puppet-openstack_extras: Release 5.1.0 - Juno  https://review.openstack.org/23018215:09
*** dweaver has quit IRC15:10
*** herewego has joined #puppet-openstack15:14
openstackgerritMerged openstack/puppet-nova: Release 5.2.0 - Juno  https://review.openstack.org/23017915:14
*** sanjayu has quit IRC15:16
*** sanjayu has joined #puppet-openstack15:16
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369015:20
*** sanjayu has quit IRC15:22
*** sanjayu has joined #puppet-openstack15:24
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Make sure service nova-network is running before create networks  https://review.openstack.org/22989515:24
degorenkohello guys. EmilienM sbadia mfisch spredzy take a look please: https://review.openstack.org/#/c/22989515:25
*** leanderthal has quit IRC15:26
openstackgerritMerged openstack/puppet-neutron: Release 5.2.0 - Juno  https://review.openstack.org/23017415:27
*** sanjayu has quit IRC15:28
*** linkedinyou has quit IRC15:30
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Rewrite nova_network provider with using only nova client  https://review.openstack.org/22954815:30
*** linkedinyou has joined #puppet-openstack15:31
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/22686215:33
*** jfluhmann has joined #puppet-openstack15:56
*** dprince has joined #puppet-openstack16:01
openstackgerritEmilien Macchi proposed openstack/puppet-glance: DNM - CI test  https://review.openstack.org/23052416:02
*** IvanBerezovskiy has left #puppet-openstack16:03
EmilienMmfisch, mgagne: how do you release new versions in launchpad, do you have a script or manually?16:04
mfischI think mgagne has a tool16:04
iurygregoryany core around can review https://review.openstack.org/#/c/223777/ ? already have a +2 ^^16:06
mfischlooking16:08
iurygregoryThanks mfisch o/16:10
EmilienMI just found out https://github.com/mgagne/openstack-puppet-release-tools16:13
*** ducttape_ has quit IRC16:13
*** ducttape_ has joined #puppet-openstack16:16
mgagnethat's the tool ^16:16
EmilienMmgagne: well, I would be happy to move it in openstack16:21
mgagneEmilienM: sure16:21
EmilienMcool16:21
EmilienMmgagne: I'll let you do the infra patch16:21
mgagneEmilienM: all fine with me. let me know the review so I can +116:22
EmilienMmgagne: not sure everything is good for 5.2.0, I did it manually16:22
mgagneEmilienM: want me to double-check?16:22
EmilienMmgagne: you want me to do it? :)16:22
EmilienMmgagne: it would be nice16:22
*** lavaboy has joined #puppet-openstack16:23
mgagneno release for horizon or this has yet to happen?16:25
openstackgerritMerged openstack/puppet-openstack-specs: Changes in enabling federation spec  https://review.openstack.org/22377716:26
EmilienMmgagne: no release16:26
mgagneok16:27
EmilienMmgagne: https://docs.google.com/spreadsheets/d/1XVrmEiLrJSdxDo-S_vFB7ljxTdYg-pe8hiMUryRor5A/edit#gid=016:28
EmilienMmgagne: you can update only those in green, in Launchpad column16:29
mgagneall done16:29
mgagnehttps://github.com/mgagne/openstack-puppet-release-tools/commit/4dec8160bc2115213da247f9215b8eff3413e8c016:30
EmilienMmgagne: thx a lot16:30
mgagneI already ran the script16:30
openstackgerritClayton O'Neill proposed openstack/puppet-neutron: Set new ml2::ovs parameters to use SERVICE DEFAULT  https://review.openstack.org/23053616:31
EmilienMclayton: we are not using this pattern ^16:31
EmilienMclayton: in the last meeting, we talked about using the fact16:32
claytonah, right, I remember that from the ML16:32
claytonare there modules using the new way I can crib from?16:32
*** paramite has quit IRC16:39
claytonah, I see the fact review.16:39
*** degorenko has quit IRC16:39
*** karmix has quit IRC16:39
claytonI'll leave my review oustanding and fix it to use the fact when that one is merged.16:39
*** dgurtner has quit IRC16:45
*** jpena has quit IRC16:50
*** dsutyagin has quit IRC16:51
*** xarses has quit IRC16:53
EmilienMUbuntu recently updated packaging to have RC116:56
EmilienMI need to run CI testing now, I'm sorry for the spams that is going to happen16:56
*** apuimedo has quit IRC16:56
mfischis regsubst the best way to pre-pend a string to every element in an array?16:56
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: DNM - CI test  https://review.openstack.org/23054116:56
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: DNM - CI test  https://review.openstack.org/23054216:57
openstackgerritEmilien Macchi proposed openstack/puppet-designate: DNM - CI test  https://review.openstack.org/23054316:57
openstackgerritEmilien Macchi proposed openstack/puppet-heat: DNM - CI test  https://review.openstack.org/23054416:57
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: DNM - CI test  https://review.openstack.org/23054516:57
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: DNM - CI test  https://review.openstack.org/23054616:57
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: DNM - CI test  https://review.openstack.org/23054716:57
openstackgerritEmilien Macchi proposed openstack/puppet-manila: DNM - CI test  https://review.openstack.org/23054816:57
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: DNM - CI test  https://review.openstack.org/23054916:57
openstackgerritEmilien Macchi proposed openstack/puppet-nova: DNM - CI test  https://review.openstack.org/23055016:57
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: DNM - CI test  https://review.openstack.org/23055116:57
openstackgerritEmilien Macchi proposed openstack/puppet-swift: DNM - CI test  https://review.openstack.org/23055216:57
openstackgerritEmilien Macchi proposed openstack/puppet-trove: DNM - CI test  https://review.openstack.org/23055316:57
openstackgerritEmilien Macchi proposed openstack/puppet-glance: acceptance: include glance::backend::file  https://review.openstack.org/23055617:00
*** jamespage has joined #puppet-openstack17:01
jamespagei/17:01
EmilienMjamespage: thx for being here17:01
EmilienMjamespage: I'm now checking everything is clean except glance17:01
jamespageEmilienM, http://paste.ubuntu.com/12639363/17:02
jamespagethats the diff in requirements.txt from b3->rc1 for glance17:02
EmilienMwe need to see the diff in glance-store17:02
jamespageswiftclient gets dropped, and we auto-generate/align our runtime dependencies based on what's in the openstack project being packaged17:03
jamespageEmilienM, glance-store has never had swiftclient as a dep17:03
jamespageI check that already17:03
jamespageEmilienM, the problem is that these are all soft-depends depending on which storage is being used17:03
*** sanjayu has joined #puppet-openstack17:04
EmilienMjamespage: our CI is red, I suggest we consider it and adjust uca packaging.17:04
EmilienMI 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_42717:04
EmilienMis the one to take in consideration17:04
EmilienMfor 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/23055617:04
jamespageEmilienM, https://github.com/openstack/glance/commit/5c5888611337ac3d12277e70f368aa9c23ebaabd17:05
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369017:05
EmilienMjamespage: so how do you explain ImportError: No module named swiftclient ?17:05
jamespageEmilienM, because swiftclient is dropped from depends, it's not installed17:05
*** herewego has quit IRC17:06
jamespageEmilienM, the glance charm installs clients explicitly based on what backend is being used - ceph swift etc...17:06
EmilienMjamespage: so I should patch puppet-glance to install it?17:06
EmilienMit's really not backward compatible, but well..17:07
jamespageEmilienM, I think that would be inline with the upstream glance thinking....17:07
EmilienMwe try to not manage package dependencies in Puppet17:07
jamespageEmilienM, this is not a package dependency, its a feature dependency17:07
jamespagethink of it that way :-)17:07
EmilienMjamespage: the fact is, our CI is not using Swift backend and it fails anyway17:07
jamespagereally?17:07
EmilienMyeas17:07
EmilienMhttp://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_43117:07
jamespagedoes it enable the swift backend by default?17:07
EmilienMwe're using filesystem17:08
EmilienMsee http://logs.openstack.org/24/230524/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/a16802a/logs/etc/glance/glance-api.conf.txt.gz17:08
*** guessi has joined #puppet-openstack17:08
EmilienM#stores = file,http17:08
EmilienM#default_store = file17:08
jamespagewell that is odd17:10
*** jistr has quit IRC17:10
EmilienMjamespage: https://review.openstack.org/#/q/owner:%22Emilien+Macchi%22+status:open+topic:%22ci-test%22,n,z17:12
EmilienMwe will see what else is coming :)17:13
jamespagewell on monday17:14
jamespagesorry have to eod17:14
EmilienMjamespage: any other contact?17:14
jamespagecoreycb17:15
EmilienMmaybe coreycb could come here17:15
jamespagehe's in #ubuntu-server17:15
*** trown has joined #puppet-openstack17:15
jamespagecan't guarantee turnaround on fixes now anyway as we're in final freeze so all changes have to go through the release team in ubuntu17:15
EmilienMjamespage: I'm just saying you, people will complain if you don't change it.17:16
jamespagejust saying we have process and checks that take time17:16
jamespageand we don't resource 24.x717:16
jamespagesorry17:16
EmilienMit's weird it works in RDO and not in UCA17:17
EmilienMyou both run liberty17:17
*** trown is now known as trown|lunch17:17
*** xarses has joined #puppet-openstack17:18
*** derekh has quit IRC17:18
EmilienMjamespage: people who use file backend will do "apt-get install glance-api glance-registry" and they will have the failure we're having now17:19
EmilienMi'm giving you free feedback that says "your package will not make Glance working by default"17:19
EmilienMif 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
EmilienMzigo: just fyi ^17:21
dfisherpuppet should not be in the business of managing packaging.  that's … for packaging to manage.17:21
dfisherjust set a freakin' dependency on it.17:21
*** ducttape_ has quit IRC17:22
EmilienMjamespage 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
EmilienMand no, we don't want to deal with packaging dependencies in our modules, that's dupplicated work with packaging folks.17:23
dfisherat best it's a duplicate.  at worst it's a conflict.17:24
*** sanjayu has quit IRC17:24
dfisherpuppet says install x y z, packaging says x y a b c17:24
*** sanjayu has joined #puppet-openstack17:24
*** imcsk8 has quit IRC17:25
*** imcsk8 has joined #puppet-openstack17:25
jamespageEmilienM, 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 installed17:27
jamespageEmilienM, 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 stuff17:28
EmilienMjamespage: no worries, ok I'll find a bug and let you know. see you on monday17:32
jamespageEmilienM, you might see me in +3 hrs17:33
jamespage;)17:33
*** sanjayu has quit IRC17:46
*** sanjayu has joined #puppet-openstack17:48
*** jfluhmann has quit IRC17:55
*** ducttape_ has joined #puppet-openstack17:59
*** delatte has quit IRC18:06
EmilienMjamespage: I have a good news, glance is the only issue I think18:10
EmilienMcrinkle: do you still -1 https://review.openstack.org/#/c/226830/ after the recent changes ? I addressed your comment.18:16
mfischEmilienM: this is why we manage a bunch of package deps ourselves in puppet18:20
tiswansoHi 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
tiswansoThis is a problem with using nested define types where the outer resource invokes create_resources to iterate the inner hash key.18:21
EmilienMmfisch: it's not sane imho18:21
mfischI realize its not ideal but my deployment doesnt wait for ideal18:21
tiswansothe stdlib .each method should be usable to get around this but it's "parser=future"18:22
tiswansoany alternatives?18:22
EmilienMmfisch: lol18:22
EmilienMmfisch: I try to make sure our CI manifest do not do too much workarounds and try to fix it upstream18:22
*** trown|lunch is now known as trown18:23
mfischI agree thats the goal18:25
*** sanjayu has quit IRC18:26
*** sanjayu has joined #puppet-openstack18:27
crinkleEmilienM: still -118:28
EmilienMcrinkle: I just found out the ceilometer patch has a dependency cycle issue.18:29
crinkleEmilienM: :'(18:29
EmilienMcrinkle: yeah it's weird, I'm working on it :)18:29
EmilienMcrinkle: ack your comment, working on it too :-)18:30
EmilienMcrinkle: I'm linking because it's simplier than cloning, also, fixtures is also using links.18:31
*** aojea_ has quit IRC18:31
*** itsuugo has quit IRC18:31
EmilienMcrinkle: the rm is because the ln_s fails if the file already exist.18:31
EmilienMcrinkle: which is also something fixtures does18:32
crinkleEmilienM: I would rather it worked about the same way as the real spec_prep does18:39
EmilienMcrinkle: I replied inline18:39
trownhola 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
EmilienMtrown: trust works18:44
EmilienMtrusty*18:44
*** sergmelikyan has joined #puppet-openstack18:44
trownEmilienM: 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 working18:46
EmilienMmhh, it should iirc we already used it18:46
trowntrusty failed on port 5050 not listening18:47
trownwhich it should not have run if the conditional evaluated to true18:47
trownEmilienM: I am unconvinced that worked before... it would have failed silently18:49
*** sergmelikyan has quit IRC18:49
EmilienMcrinkle: 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 IRC18:52
*** sergmelikyan has joined #puppet-openstack18:54
crinkleEmilienM: yes18:54
EmilienMcrinkle: so I need to create clonemap.yaml, etc ?18:55
EmilienMcrinkle: it won't when you will run tests on your laptop18:56
EmilienMwon't run*18:56
EmilienMit 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-L13718:57
EmilienMmy last comment is false ^18:57
*** gchamoul has quit IRC18:58
*** sergmelikyan has quit IRC18:58
crinkleEmilienM: 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#n2018:59
*** gchamoul has joined #puppet-openstack18:59
EmilienMcrinkle: apologizes, but I don't understand the symlink problem.19:00
*** sanjayu has quit IRC19:00
*** sanjayu has joined #puppet-openstack19:01
EmilienMcrinkle: iiuc, I can drop the ln_s & rm_rf and that's it, because the module should already be cloned, right?19:02
EmilienMI think I got it :)19:02
EmilienMcrinkle: the problem is the case where don't run zuul-cloner19:03
EmilienMso I need to move my rm_rf & ln_s with a conditionnal, right?19:03
crinkleEmilienM: 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
EmilienMcrinkle: L82 ? well, it's just when tests are finished, you want to clean the dir, no?19:05
crinkleno line 5219:05
EmilienMoh indeed19:05
EmilienMok, trying19:06
EmilienMit works, I update19:07
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683019:08
EmilienMcrinkle: this patchset just drop L52 ^19:08
*** sergmelikyan has joined #puppet-openstack19:16
*** aojea_ has joined #puppet-openstack19:16
*** itsuugo has joined #puppet-openstack19:17
*** itsuugo has quit IRC19:25
*** aojea_ has quit IRC19:25
*** xarses has quit IRC19:25
*** xarses has joined #puppet-openstack19:25
*** itsuugo has joined #puppet-openstack19:26
*** aojea_ has joined #puppet-openstack19:26
*** sanjayu has quit IRC19:32
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib: Add os_service_default fact  https://review.openstack.org/22905019:33
*** sanjayu has joined #puppet-openstack19:33
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/22909519:37
*** sanjayu has quit IRC19:49
*** sanjayu has joined #puppet-openstack19:49
zigoEmilienM: FYI, the Debian package does work with the file backend by default.19:53
*** sanjayu has quit IRC19:55
EmilienMzigo: do you install swiftclient by default?19:56
zigoLooking ...19:56
zigoEmilienM: Nop, but I can if you wish.19:56
EmilienMzigo: it's weird, glance is not starting with default packaging. I'm investigating19:57
EmilienMzigo: I'll let you know19:57
zigoEmilienM: You mean in Debian?19:57
zigoIt does for me ...19:57
zigoEmilienM: I'm currently trying to beat tempest. So far, it's working better and better.19:58
EmilienMwe shamelessly install it from source.19:58
zigoEmilienM: Install what from source? Glance?19:59
zigogit commit -a -m "Adding python-swiftclient as runtime dependency (request from puppet team)."19:59
zigo:)19:59
EmilienMtempest19:59
zigoAh...19:59
zigoWell, I don't ! :)19:59
EmilienMzigo: thx but I'm not sure for swiftclient, I noticed that this morning but I need time to investigate20:00
EmilienMlike usual I appreciate your responsiveness :)20:00
dfishersee?  add a dependency.20:01
dfisherit's so easy.20:01
EmilienMdfisher: zigo & I have that special relationship you know ;-)20:02
dfisher:)20:02
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: wsgi: make sure keystone service is stopped before starting httpd  https://review.openstack.org/23062620:02
zigoEmilienM: Could you look again in the thread which I opened: "[puppet] Running Debian packages on top of Trusty" ?20:03
EmilienMzigo: sure, I was not much in e-mails today20:03
zigoEmilienM: 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-openstack20:04
zigoEmilienM: BTW, these days, do you do a 100% on tempest, or what?20:05
EmilienMmwhahaha: I think you can override package name, because they are all tagged20:05
EmilienMfor example: https://github.com/openstack/puppet-ceilometer/blob/master/manifests/api.pp#L10820:06
mwhahahayea i know that, doesn't mean everyone else wants to :D20:07
mwhahahalike 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 way20:09
*** sanjayu has joined #puppet-openstack20:15
EmilienMcrinkle: I found the ceilo/wsgi bug20:15
openstackgerritSergey Kolekonov proposed openstack/puppet-keystone: Pass necessary options to Apache when using WSGI  https://review.openstack.org/23063320:23
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: auth: drop service dependency for Keystone_user_role  https://review.openstack.org/23063420:23
EmilienMcrinkle: ^20:23
mfischis Maksym Yatsenko in this channel?20:25
*** jayg is now known as jayg|g0n320:29
EmilienMmyatsenko: ^20:29
mfischprobably late there unless he's us20:31
mfischI have to fix his rbd/host change it broke our cinder20:31
mfischand is deprecated20:32
*** dprince has quit IRC20:39
*** skolekonov has quit IRC20:48
*** angdraug has joined #puppet-openstack20:53
*** skolekonov has joined #puppet-openstack20:57
*** ducttape_ has quit IRC21:00
*** skolekonov has quit IRC21:01
*** trown is now known as trown|outttypeww21:03
EmilienMI don't understand what's happenning here: https://review.openstack.org/#/c/230556/21:03
EmilienMhttp://logs.openstack.org/56/230556/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/c62504e/console.html#_2015-10-02_20_42_33_73921:03
EmilienMthe parameter is passed, see https://review.openstack.org/#/c/230556/1/spec/acceptance/basic_glance_spec.rb,cm21:04
*** sergmelikyan has quit IRC21:05
crinkleEmilienM: glance::backend::file inherits glance::api21:09
crinkleyou can't inherit a class with parameters21:10
EmilienMcrinkle: we found a new bug :)21:10
crinkleyay testing \o/21:10
EmilienMcrinkle: lol21:10
EmilienMcrinkle: git blame tells me who is paying beers at puppetconf21:11
EmilienMbodepd: hey what's up? :)21:11
iurygregoryby people, have a good weekend o/21:11
EmilienMiurygregory: you too :)21:12
iurygregory^^21:12
dfisherbeers?   sake*21:12
*** iurygregory has quit IRC21:12
dfisheroh.  puppetconf.21:12
dfisherignore me.21:12
openstackgerritEmilien Macchi proposed openstack/puppet-glance: file backend: do not inherit from glance::api  https://review.openstack.org/23055621:15
EmilienMso if we want integration jobs back alive, we need https://review.openstack.org/230634 and https://review.openstack.org/23055621:16
*** tiswanso has quit IRC21:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/23043121:20
*** karmix has joined #puppet-openstack21: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 it21:23
openstackLaunchpad bug 1502318 in puppet-openstack "keystone is set up incorrectly under Apache" [Undecided,New] - Assigned to Sergey Kolekonov (skolekonov)21:23
*** zeus has quit IRC21:23
*** richm has quit IRC21:23
*** zeus has joined #puppet-openstack21:25
*** zeus is now known as Guest3044221:25
*** richm has joined #puppet-openstack21:27
*** rahulait has joined #puppet-openstack21:36
*** aojea_ has quit IRC21:39
*** itsuugo has quit IRC21:39
EmilienMmtreinish: I found something weird in tempest/smoke: even if I disable nova/neutron/cinder/glance services -> tempest will try (and fail) to run scenarios21:40
EmilienMthey should be SKIPPED I think21:40
mtreinishEmilienM: yeah they should raise a skip exception if the service is disabled they required21:41
EmilienMyeah or just skip21:41
mtreinishthat's what the @test.services decorator is supposed to do21:41
*** devnull0 has joined #puppet-openstack21:42
EmilienMmtreinish: I'm proposing a patch21:42
mtreinishEmilienM: ok21:42
EmilienMmtreinish: wait, they already have the python decorator21:43
EmilienM@test.services('compute', 'network')21:43
*** bnemec has joined #puppet-openstack21:44
mtreinishyeah, 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
EmilienMI'll investigate logs21:44
EmilienMmtreinish: quick update - the split happenned - I'm trying to balance services now: https://review.openstack.org/#/c/230431/2/README.md,cm21:45
mtreinishEmilienM: well you need nova, glance, etc for heat, sahara, trove to work21:46
EmilienMarg21:46
EmilienMok21:46
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/23043121:54
*** bnemec has quit IRC22:06
mtreinishEmilienM: ^^^ you dropped swift from the table?22:07
EmilienMoops I would not22:07
EmilienMmtreinish: no, it's in scenario00222:07
mtreinishoh, yep sry missed it22:08
mtreinishblended together with sahara in my head22:08
EmilienMmtreinish: np, thx for helping22:08
mtreinishtoo many s's22:08
EmilienMyeah lol22:08
EmilienMjamespage: I have issues with cinder & glance so far. I'll investigate later22:14
*** guessi has quit IRC22:25
*** Guest30442 is now known as zeus22:33
*** zeus has quit IRC22:33
*** zeus has joined #puppet-openstack22:33
*** sergmelikyan has joined #puppet-openstack22:36
*** EmilienM sets mode: +o crinkle22:37
*** EmilienM sets mode: +o clayton22:37
*** EmilienM sets mode: +o michchap22:37
*** sanjayu has quit IRC22:39
*** sergmelikyan has quit IRC22:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Split scenarios001 in two scenarios  https://review.openstack.org/23043122:42
*** jasondotstar is now known as jasondotstar|afk22:48
*** devnull0 has quit IRC22:53
*** dfisher has quit IRC22:57
*** sanjayu has joined #puppet-openstack23:00
*** mdorman has quit IRC23:01
*** sergmelikyan has joined #puppet-openstack23:37
*** sergmelikyan has quit IRC23:41

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