Tuesday, 2016-12-13

« Monday, 2016-12-12 Index Wednesday, 2016-12-14 »
*** ducttape_ has joined #puppet-openstack00:01
EmilienMmwhahaha: yes, we might want to set it to false first. So we can backport it and then we switch poi to use it and then change to true eventually00:03
EmilienMwe'll also need tripleo & fuel with latest packages eventually00:04
mwhahahayea00:04
mwhahahait's going to be one of those weeks00:04
mwhahahaalso it's release week :D00:04
* mwhahaha wanders off00:05
EmilienMoh that00:05
openstackgerritzhongshengping proposed openstack/puppet-glare: Added retries for db_sync  https://review.openstack.org/40961100:45
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969300:59
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969301:11
*** ducttape_ has quit IRC01:16
*** ducttape_ has joined #puppet-openstack01:22
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969301:22
*** ayoung has quit IRC01:25
*** chem has quit IRC01:35
*** rajinir has quit IRC01:36
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969301:43
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Nova Placement API support  https://review.openstack.org/40630002:19
EmilienMmwhahaha: ready for review again ^02:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: nova: activate placement API service  https://review.openstack.org/40630102:19
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969302:23
*** ducttape_ has quit IRC02:26
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:48
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969302:56
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969303:23
*** agrebennikov_ has quit IRC03:26
*** ducttape_ has joined #puppet-openstack03:26
*** ducttape_ has quit IRC03:32
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969303:50
openstackgerritMerged openstack/puppet-nova: Add the enable_new_services option to nova::conductor  https://review.openstack.org/40995403:50
*** ducttape_ has joined #puppet-openstack04:42
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969304:51
*** ducttape_ has quit IRC04:59
*** snarwade has joined #puppet-openstack05:02
*** agrebennikov_ has joined #puppet-openstack05:08
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969305:17
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969305:39
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969305:45
*** nmathew has joined #puppet-openstack05:57
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969305:58
*** ducttape_ has joined #puppet-openstack06:00
*** ducttape_ has quit IRC06:05
*** aruns has joined #puppet-openstack06:07
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969306:19
*** zhongshengping has joined #puppet-openstack06:22
*** agrebennikov_ has quit IRC06:36
*** zhongshengping has quit IRC06:39
*** richm has quit IRC06:41
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969306:42
*** saneax-_-|AFK is now known as saneax07:02
*** tdb has quit IRC07:08
*** fxpester has joined #puppet-openstack07:09
*** imcsk8_ has quit IRC07:09
*** imcsk8_ has joined #puppet-openstack07:09
*** tdb has joined #puppet-openstack07:10
*** rasca has joined #puppet-openstack07:20
*** ducttape_ has joined #puppet-openstack07:31
*** ducttape_ has quit IRC07:35
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/41007007:37
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty: Add the acceptance test to deploy the cloudkitty service  https://review.openstack.org/40969307:50
*** mkoderer has joined #puppet-openstack07:54
*** ccamacho has joined #puppet-openstack07:59
*** jaosorior has joined #puppet-openstack07:59
*** basilAB has quit IRC08:04
*** rcernin has joined #puppet-openstack08:05
*** basilAB has joined #puppet-openstack08:09
*** jaosorior has quit IRC08:12
*** jaosorior has joined #puppet-openstack08:12
openstackgerritMerged openstack/puppet-murano: Generate separate db for murano service broker  https://review.openstack.org/40506508:13
*** aruns has quit IRC08:21
*** rcernin has quit IRC08:22
*** rcernin has joined #puppet-openstack08:25
*** jpena|off is now known as jpena08:25
*** aruns has joined #puppet-openstack08:33
*** jpena is now known as jpena|off08:37
*** jpena|off is now known as jpena08:50
*** astupnikov has joined #puppet-openstack08:55
*** jpich has joined #puppet-openstack08:57
*** rmart04 has joined #puppet-openstack08:57
*** chem has joined #puppet-openstack08:57
*** mkarpin has quit IRC09:00
*** ducttape_ has joined #puppet-openstack09:01
*** shardy has joined #puppet-openstack09:04
*** panda|afk is now known as panda09:05
*** ducttape_ has quit IRC09:06
*** tdb has quit IRC09:08
*** tdb has joined #puppet-openstack09:09
*** fxpester has quit IRC09:13
*** fxpester has joined #puppet-openstack09:14
*** chem has quit IRC09:21
*** chem has joined #puppet-openstack09:21
*** owalsh-afk is now known as owalsh09:27
*** mkarpin has joined #puppet-openstack09:29
*** lucas-afk is now known as lucasagomes09:32
*** gfidente has joined #puppet-openstack09:34
*** derekh has joined #puppet-openstack09:41
*** dtantsur|afk is now known as dtantsur09:47
*** paramite has joined #puppet-openstack09:59
openstackgerritMerged openstack/puppet-openstack-integration: Pin older openstack-selinux  https://review.openstack.org/40900810:10
*** ducttape_ has joined #puppet-openstack10:32
*** ducttape_ has quit IRC10:37
*** snarwade is now known as snarwade|afk10:49
*** pester has joined #puppet-openstack11:05
*** fxpester has quit IRC11:08
openstackgerritXing Zhou proposed openstack/puppet-openstacklib: Add support when creating multiple databases with same db username  with puppet-openstacklib  https://review.openstack.org/41016611:08
*** nmathew has quit IRC11:08
*** richm has joined #puppet-openstack11:14
*** amoralej|off is now known as amoralej11:25
*** ntpttr has quit IRC11:30
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Add manifests to deploy APIs over httpd  https://review.openstack.org/39483511:31
*** ntpttr has joined #puppet-openstack11:39
*** snarwade|afk is now known as snarwade11:47
*** aruns has quit IRC11:51
*** ducttape_ has joined #puppet-openstack12:02
*** ducttape_ has quit IRC12:07
*** nmathew has joined #puppet-openstack12:07
*** jaosorior is now known as jaosorior_brb12:10
*** dgurtner has joined #puppet-openstack12:21
*** dprince has joined #puppet-openstack12:23
*** shardy is now known as shardy_lunch12:29
*** ducttape_ has joined #puppet-openstack12:36
*** snarwade has quit IRC12:42
*** dprince has quit IRC12:52
*** shardy_lunch is now known as shardy12:57
*** lucasagomes is now known as lucas-hungry12:58
*** alee has quit IRC13:01
*** astupnikov has quit IRC13:01
*** jayg|g0n3 is now known as jayg13:09
*** johnmilton has joined #puppet-openstack13:11
*** ducttape_ has quit IRC13:22
*** pradk has joined #puppet-openstack13:33
*** trown|outtypewww is now known as trown13:35
*** jpena is now known as jpena|lunch13:41
*** amoralej is now known as amoralej|lunch13:44
openstackgerritFlavio Percoco proposed openstack/puppet-glance: Don't pass glance-registry.conf in every sync_db  https://review.openstack.org/40855413:47
*** aruns has joined #puppet-openstack13:55
iurygregorymwhahaha, when you update https://review.openstack.org/#/c/409476/ can you add Closes-bug: #1604463 ? =)13:56
openstackbug 1604463 in puppet-mistral "Modules should use keystone::resource::authtoken to configure keystone_authtoken settings" [Medium,In progress] https://launchpad.net/bugs/1604463 - Assigned to Iury Gregory Melo Ferreira (iurygregory)13:56
*** ayoung has joined #puppet-openstack13:58
*** dprince has joined #puppet-openstack14:02
*** morazi has joined #puppet-openstack14:06
*** ducttape_ has joined #puppet-openstack14:08
*** lucas-hungry is now known as lucasagomes14:08
*** jaosorior_brb is now known as jaosorior14:09
*** dgurtner has quit IRC14:11
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Improves support for ceph-mds  https://review.openstack.org/40238514:13
*** morazi has quit IRC14:14
*** ducttape_ has quit IRC14:15
*** paramite has quit IRC14:18
*** dmsimard has quit IRC14:21
*** dmsimard has joined #puppet-openstack14:21
*** agrebennikov_ has joined #puppet-openstack14:22
*** johnmilton has quit IRC14:22
*** johnmilton has joined #puppet-openstack14:23
mwhahahaiurygregory: k14:24
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Move Mistral to authtoken  https://review.openstack.org/40947614:26
*** pester has quit IRC14:26
*** pester has joined #puppet-openstack14:27
*** morazi has joined #puppet-openstack14:27
*** dmsimard has quit IRC14:28
*** dmsimard has joined #puppet-openstack14:28
*** dmsimard has quit IRC14:28
*** saneax is now known as saneax-_-|AFK14:29
*** dmsimard has joined #puppet-openstack14:30
*** amoralej|lunch is now known as amoralej14:36
*** paramite has joined #puppet-openstack14:42
*** jpena|lunch is now known as jpena14:46
*** ducttape_ has joined #puppet-openstack14:50
*** zhongshengping has joined #puppet-openstack14:58
*** alee has joined #puppet-openstack15:03
*** jayg is now known as jayg|g0n315:05
*** zhongshengping has quit IRC15:09
*** nmathew- has joined #puppet-openstack15:10
*** nmathew has quit IRC15:11
*** jayg|g0n3 is now known as jayg15:25
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Add meeting #101  https://review.openstack.org/41027015:25
*** sanagikoki has joined #puppet-openstack15:33
*** zhongshengping has joined #puppet-openstack15:34
*** zhongshengping has quit IRC15:35
*** zhongshengping has joined #puppet-openstack15:43
*** zhongshengping has quit IRC15:48
*** saneax-_-|AFK is now known as saneax15:48
*** aruns has quit IRC15:50
EmilienMiurygregory: do you have a script?15:52
openstackgerritMerged openstack/puppet-openstack-guide: Add meeting #101  https://review.openstack.org/41027015:54
mfischiurygregory: you around?15:57
*** rajinir has joined #puppet-openstack15:59
iurygregoryEmilienM, script for what?16:00
iurygregorymeetings?16:00
iurygregorymfisch, i'm16:00
*** _ducttape_ has joined #puppet-openstack16:00
iurygregorysorry i was in lunch time :D16:00
EmilienMiurygregory: yeah16:01
iurygregoryEmilienM, nope haha16:01
iurygregoryi do ctrl c + v  in the last and just update the dates haha16:01
*** ducttape_ has quit IRC16:03
*** rcernin has quit IRC16:04
mfischiurygregory: I cant seem to get into gerrit but wondering if you had time to look at authtoken for mistral16:05
EmilienMwe need a puppet module for that16:05
iurygregorymfisch, oh we had the patch already16:06
mwhahahahttps://review.openstack.org/#/c/409476/16:06
mfischthanks16:06
iurygregorybut now is failing in triple own16:06
mfischI dont know why I cant get in16:06
mfischit says the auth provider is not configured16:06
mwhahahaclear them cookies :D16:06
iurygregoryctrl + f5 haha16:07
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:12
*** jaosorior has quit IRC16:12
*** jaosorior has joined #puppet-openstack16:13
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:16
*** rmart04 has quit IRC16:17
mwhahahaiurygregory: https://review.openstack.org/#/c/409945/ if you have a second16:19
iurygregorymwhahaha, sure16:19
*** _ducttape_ has quit IRC16:22
*** ducttape_ has joined #puppet-openstack16:24
*** ducttape_ has quit IRC16:24
-openstackstatus- NOTICE: Launchpad SSO is not currently working, so logins to our services like review.openstack.org and wiki.openstack.org are failing; the admins at Canonical are looking into the issue but there is no estimated time for a fix yet.16:27
*** ducttape_ has joined #puppet-openstack16:27
*** ChanServ changes topic to "Launchpad SSO is not currently working, so logins to our services like review.openstack.org and wiki.openstack.org are failing; the admins at Canonical are looking into the issue but there is no estimated time for a fix yet."16:27
iurygregorymfisch, ^haha16:27
iurygregorymaybe this is your problem XD16:27
mfischyep16:27
mfischI got 99 problems and Canonical auth is one16:27
iurygregoryomg =(16:28
*** jaosorior has quit IRC16:28
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:28
*** panda is now known as panda|bbl16:37
*** mkoderer has quit IRC16:46
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:53
openstackgerritAlex Schultz proposed openstack/puppet-nova: Fix fix region name release note  https://review.openstack.org/41031716:54
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:58
*** rcernin has joined #puppet-openstack17:00
*** ducttape_ has quit IRC17:02
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994817:02
*** ducttape_ has joined #puppet-openstack17:02
*** ChanServ changes topic to "CI status: green - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/"17:03
-openstackstatus- NOTICE: Canonical admins have resolved the issue with login.launchpad.net, so authentication should be restored now.17:03
mwhahahaEmilienM: so the cell_v2 error only currently shows up on upgrades17:11
mwhahahaEmilienM: which is why packages are fine17:11
openstackgerritAlex Schultz proposed openstack/puppet-nova: Enable cell_v2 setup by default  https://review.openstack.org/41032417:13
*** jpich has quit IRC17:19
*** markvoelker has quit IRC17:20
dmsimardEmilienM: nova placement17:21
dmsimardyou're rechecking a lot and gloriously ignoring my comment ;(17:21
EmilienMdmsimard: I didn't see it17:21
*** markvoelker has joined #puppet-openstack17:22
EmilienMdmsimard: it's a bug in nova, that cdent is working on17:22
EmilienMplacement API works well17:22
dmsimardok, do we have any way of testingit ?17:22
EmilienMand there is no need of tempest test for now, compute works fine. But they are working on adding tempest tests17:22
dmsimardit*17:22
dmsimardokay, I saw some functional tests in the tree but it didn't seem like it was using tempest17:23
EmilienMnope17:23
EmilienMbut that's fine17:23
*** rcernin has quit IRC17:24
EmilienMdmsimard: https://bugs.launchpad.net/nova/+bug/164731617:24
openstackLaunchpad bug 1647316 in OpenStack Compute (nova) "scheduler report client sends allocations with value of zero, violating min_unit" [Medium,In progress] - Assigned to Ed Leafe (ed-leafe)17:24
EmilienMyou picked a trace which is a bug17:25
EmilienMbut it doesn't block you to spawn a VM17:25
EmilienMit's on nova side17:25
*** bogdando has quit IRC17:25
EmilienMdmsimard: I actually reported this trace to cdent :D17:26
mwhahahais scenario003 broken on centos?17:27
mwhahahahttps://review.openstack.org/#/c/409476/ keeps failing tempest on compute17:27
mwhahahafor a mistral change17:27
dmsimardmaybe another designate-ish issue ?17:34
dmsimardwhere in p-o-i it passes but beaker it doesn't17:34
* dmsimard looks17:34
mwhahahabut that's p-o-i17:34
mwhahahai'm rechecking a change in p-o-i to see if it's happening there as well17:35
*** ccamacho has quit IRC17:35
*** rcernin has joined #puppet-openstack17:35
*** rcernin has quit IRC17:35
dmsimardwhat do you mean that's p-o-i17:35
dmsimardoh, yeah it's p-o-i but in puppet-mistral17:36
dmsimardlet me see17:36
dmsimardmwhahaha: mistral isn't even enabled in p-o-i17:37
dmsimardhttps://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario003.pp#L17-L3417:37
mwhahahai think there's a latent bug in nova somewhere because the qa folks had reported the same error on a tempest change but that was scenario00117:37
dmsimardyeah it looks like a nova issue from looking at the tempest failures17:37
dmsimardhttp://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/nova/nova-compute.txt.gz#_2016-12-13_16_39_50_83317:38
*** guimaluf has quit IRC17:38
dmsimardhttp://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/libvirt/qemu/instance-00000001.txt.gz17:38
dmsimardI think that's the qemu host model issue thing17:39
dmsimardamoralej: ^ ?17:39
amoralejyeah17:39
mwhahahaso what's the fix?17:39
amoralejyou need qemu-kvm-ev 2.6.017:39
dmsimardamoralej: so that's fixed when kvm 2.6 lands in 7.3 ?17:39
amoralejyes17:39
mwhahahais this a new thing17:40
mwhahahalike why is this starting now?17:40
dmsimardwhich is signed and pushed so we're just waiting for CDN sync ?17:40
amoralejonly affects to amd, i'd say17:40
dmsimardmwhahaha: because 7.3 is officially out since yesterday17:40
dmsimardmwhahaha: and there's a yum update that was added somewhere recently ?17:40
amoralejnop17:40
mwhahahayea we merged that yum update17:40
amoraleji meant, yes, not only to AMD, :)17:41
amoralejany processor with arat extension17:41
dmsimardbasically qemu-kvm-ev 2.6.0 was meant to ship at the same time as 7.3 but it looks like it's lagging a bit behind17:41
amoralejyou can, whether wait to get it synced in centos official repos17:41
amoralejwhat should happen very soon17:41
amoralejor enable virtsig testing repo17:42
amoralejwhich is what we are doing in rdo-ci17:42
*** nmathew- has quit IRC17:43
*** derekh has quit IRC17:43
*** lucasagomes is now known as lucas-afk17:50
*** jpena is now known as jpena|off17:52
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Pass a generic cpu to qemu instead of trying to pass host-model  https://review.openstack.org/41034417:56
dmsimardamoralej, mwhahaha: ^17:57
dmsimardI'll need to add an override on weirdo to maintain nested virt capabilities17:57
amoralejdmsimard, maybe we could fix it in puppet-nova and set cpu_mode to none as soon as virt_type is qemu,18:00
dmsimardmaybe18:00
dmsimardmwhahaha: wdyt ? Should we change the puppet-nova default to match devstack config ?18:01
dmsimardi.e, when qemu, default to 'none' instead of 'host-model'18:01
dmsimardEmilienM: does tripleo jobs use qemu or kvm ?18:02
EmilienMdmsimard: qemu18:03
EmilienMfor multinode18:03
EmilienMfor ovb I'm not sure kvm maybe18:03
dmsimardwhat about libvirt_cpu_mode ?18:03
dmsimardEmilienM: for context, this can also break tripleo: https://review.openstack.org/#/c/410344/ + http://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/libvirt/qemu/instance-00000001.txt.gz18:04
EmilienMdmsimard: can you submit a patch in THT/ci/environments please?18:05
dmsimardSure but ihavenoideawhatimdoing.jpg18:06
EmilienMdmsimard: learn :)18:06
*** dtantsur is now known as dtantsur|afk18:10
*** pester has quit IRC18:20
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Nova Placement API support  https://review.openstack.org/40630018:26
EmilienMok cdent did a last review, should be fine now18:26
EmilienMdmsimard: so back to the qemu problem. Is it only when qemu is used and not kvm?18:27
EmilienMif yes, you might just need to patch https://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments/scenario001-multinode.yaml#L61-L6218:28
EmilienMand add the hiera params to configure that18:28
EmilienMyou'll need to do it for all environments in https://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments18:28
EmilienMand also in https://github.com/openstack-infra/tripleo-ci/blob/master/test-environments/multinode.yaml18:28
*** rasca has quit IRC18:37
EmilienMdmsimard: what are the symptoms again? I see a lot of tripleo failures, might be related18:37
dmsimardEmilienM: tempest/nova failing with http://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/nova/nova-compute.txt.gz#_2016-12-13_16_39_50_833 and http://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/libvirt/qemu/instance-00000001.txt.gz18:38
dmsimardwith centos 7.3, kvm 2.6, qemu and host-model18:38
EmilienMhttp://logs.openstack.org/92/409692/4/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/21f0f1e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2016-12-13_16_35_36_62218:38
EmilienMnevermind18:38
EmilienMmy problem is still the same as we had in tripleo CI and we fixed it but patch is still in gate18:38
dmsimardEmilienM: I'll send patches to modify those lines you sent me.18:38
EmilienMdmsimard: merci18:39
* EmilienM lunch18:39
*** shardy is now known as shardy_afk18:43
*** dprince has quit IRC18:54
dmsimardEmilienM: https://review.openstack.org/#/q/topic:kvm-2.618:55
EmilienMdmsimard: nice18:55
dmsimardA question amoralej asked is if it would make sense to change the default provided by puppet-nova18:55
EmilienMdmsimard: what is the default in Nova?18:55
dmsimardI don't have a strong opinion18:56
dmsimardhost-model when using qemu18:56
EmilienMok so default is broken, right?18:56
EmilienMas it should be 'none' iiuc18:56
dmsimardhttps://github.com/openstack/puppet-nova/blob/master/manifests/compute/libvirt.pp#L13818:56
dmsimardwell, this is a regression in 2.6.018:56
dmsimardat least how I understand the current state of things18:56
dmsimardThere's an internal red hat thread about the issue.18:57
amoralejthis is a bug in qemu 2.6.0, but it's probably uncovering a wrong default, IMHO18:57
EmilienMdoes it affect newton/mitaka CI too?18:57
amoralejyes18:57
dmsimardnewton yes, mitaka no18:57
amoralejwhy not dmsimard?18:58
dmsimardwe're not forcing qemu >= 2.3 in mitaka18:58
dmsimardthat merged in newton18:58
dmsimardI guess it would manifest in mitaka if you're using the virt sig, which was not bundled in rdo release in Mitaka.18:58
amoralejso, we are not enabling virsig repo in p-o-i for mitaka?18:59
dmsimardnot afaik18:59
amoralejit'd affect for no p-o-i anyway18:59
dmsimardwhat ?18:59
amoralejpackstack and tripleo if it uses qemu19:00
amoraleji mean19:00
amoralejif we fix it in puppet-nova, we'd fix to all19:00
amoralejor even in upstream nova19:01
dmsimardUnless people are actually putting host-model or until it's merged tested and released19:01
dmsimardreleased and people actually update their packages*19:01
dmsimardand in mitaka opm was monolithic19:01
dmsimardthis is a nightmare19:01
mwhahahahttps://github.com/openstack/puppet-nova/blob/master/manifests/compute/libvirt.pp#L140-L14219:01
mwhahahathat's the fix19:01
mwhahaharemove qemu from setting host-model19:01
EmilienMdmsimard: so we need to backport https://review.openstack.org/#/c/410344/ ?19:02
dmsimardEmilienM: like amoralej said, there's probably more value in doing the fix in both puppet-nova and elsewhere19:02
EmilienMmwhahaha: yes indeed, it sounds like it, though behavior might differ on ubuntu19:02
EmilienMdmsimard: I agree19:03
dmsimardEmilienM: yeah that's why I'm hesitating a bit, if it's a regression, it's bound to be fixed19:03
EmilienMcan we try https://github.com/openstack/puppet-nova/blob/master/manifests/compute/libvirt.pp#L140-L142 ?19:03
EmilienMand see how CI works19:03
mwhahahait might but this might be a case of where the modules are oppinionated and it's biting us19:03
dmsimardyeah but you already approved my patch :P19:03
dmsimardmwhahaha: we'd also be changing a default of stable release19:03
mwhahahadmsimard: well we should fix it for ocata forward at least19:04
mwhahahabrb19:04
EmilienMbrb too, third attempt to have lunch19:06
dmsimardmwhahaha, EmilienM: to be fair, puppet-nova is simply emulating what nova is already doing... https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3411-L341919:07
dmsimardSo one could argue that the proper fix would be in nova and then we could set libvirt_cpu_mode to $::os_service_default or something19:08
*** dprince has joined #puppet-openstack19:08
dmsimardjust thinking out loud19:08
*** rcernin has joined #puppet-openstack19:14
*** panda|bbl is now known as panda19:19
*** paramite has quit IRC19:21
mwhahahaYea it should be os service default19:26
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable cell_v2 setup for p-o-i  https://review.openstack.org/41037719:50
openstackgerritAlex Schultz proposed openstack/puppet-nova: Enable cell_v2 setup by default  https://review.openstack.org/41032419:51
mwhahahalining up those patch sets19:51
mwhahahahttp://logs.openstack.org/77/410377/1/check/gate-puppet-openstack-integration-puppet-syntax-3-centos-7/330baf0/console.html#_2016-12-13_19_52_43_01610820:01
mwhahaha‎(ﾉಥ益ಥ）ﾉ ┻━┻20:02
mwhahahathat is all.20:02
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Pin fast_gettext  https://review.openstack.org/41038020:06
mwhahahaEmilienM: iurygregory -^20:06
iurygregorymwhahaha, done :D20:06
iurygregoryand you have change your emoji lol20:06
iurygregory(ﾉಥ益ಥ）ﾉ ┻━ hahaha20:06
EmilienMmwhahaha: +A. Don't forget to backport20:06
mwhahaharuby 2.1 is now the bane of my existance and will probably be so for the next several years20:08
iurygregory#fail20:10
iurygregoryhttps://review.openstack.org/#/c/410380/20:10
mwhahahawat20:11
mwhahahaguess we need 1.1.020:11
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Pin fast_gettext  https://review.openstack.org/41038020:12
mwhahahathat's odd, wonder who we skipped from 1.1.0 1.3.020:12
mwhahahasomeone must have added 1.3.0 as a dep somewhere20:12
mwhahahai wish bundler/ruby took ruby version into consideration20:13
*** amoralej is now known as amoralej|off20:17
iurygregorygate-puppet-openstack-libraries-puppet-beaker-rspec-centos-7 #failure20:19
mwhahahabecause of libvirt?20:19
mwhahahanope couldn't install r10k20:20
mwhahahaof course20:20
mwhahahawhy not20:20
mwhahaha2016-12-13 20:17:32.434358 |   ERROR:  Error installing r10k:20:21
mwhahaha2016-12-13 20:17:32.434619 |   fast_gettext requires Ruby version >= 2.1.0.20:21
mwhahahaok so CI is completely hosed20:21
* mwhahaha gives up20:21
iurygregoryNOOOOOOOOOO <O>20:21
mwhahahaif you need me i'll be under my desk crying20:21
* iurygregory time to leave the office ha20:22
mwhahahagettext-setup released today20:24
mwhahahaso it's probably that20:24
mwhahahaHunner: is that a puppet thing?20:24
mwhahahaEmilienM: http://logs.openstack.org/80/410380/2/check/gate-puppet-openstack-libraries-puppet-beaker-rspec-centos-7/a7fa8bb/console.html#_2016-12-13_20_17_25_799715 where does this come from?20:32
mwhahahaoh it comes from puppet-openstack-integration20:33
* mwhahaha sighs20:33
*** dgurtner has joined #puppet-openstack20:33
*** dgurtner has quit IRC20:33
*** dgurtner has joined #puppet-openstack20:33
EmilienMmwhahaha: http://git.openstack.org/cgit/openstack/puppet-openstack-integration/tree/install_modules.sh#n2520:33
EmilienMhttp://codesearch.openstack.org/?q=gem%20install%20r10k&i=nope&files=&repos=20:33
mwhahahaso we can't fix spec_helper because p-o-i and we can't fix p-o-i because of spec_helper20:33
*** johnmilton has quit IRC20:33
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable cell_v2 setup for p-o-i  https://review.openstack.org/41037720:35
iurygregoryso we are in a loop LOL20:36
EmilienMin a mtg, i'll look in a bit20:37
EmilienMbut we can still hack something I think20:37
mwhahahayea looking at it20:38
mwhahahamight be able to force something in p-o-i20:38
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Pin fast_gettext  https://review.openstack.org/41038820:38
*** shardy_afk is now known as shardy20:40
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Pin fast_gettext  https://review.openstack.org/41038820:43
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Pin fast_gettext  https://review.openstack.org/41038020:44
mwhahahathat might do it20:44
iurygregoryok ^^20:44
mwhahahawhy does CI seem to always fall apart every week20:46
EmilienMback21:02
EmilienMthis TC meeting never ends :)21:02
EmilienMmwhahaha: +221:02
EmilienMlet's revert https://review.openstack.org/#/c/410388/ once it's merged in spec_helper21:02
mwhahahawe can't21:02
mwhahahabecause r10k can't install21:03
EmilienMah21:03
mwhahahawe can partial revert the Gemfile but the other is here to stay21:03
EmilienMmwhahaha: let's see how CI works and we'll merge it21:03
mwhahahayea it looked green so far21:03
mwhahahai'm sure we'll jsut hit the libvirt issue21:03
mwhahahaso i've pretty much given up on anything actually merging today21:03
EmilienMdo we need to align the stars?21:03
mwhahahaquite possibly, but lets see21:04
* EmilienM is invoking stars21:04
* EmilienM makes special dance21:04
iurygregoryhahahha21:04
mwhahahaand here i was hoping to actually get some work done today21:05
mwhahahaso far it's been queuing things up and maybe some day they'll merge21:05
* mwhahaha goes back to queuing things up while we wait21:05
EmilienMmwhahaha: if things are broken, squash commits21:08
mwhahahasure21:09
openstackgerritAlex Schultz proposed openstack/puppet-barbican: Add oslo_messaging_notification transport_url  https://review.openstack.org/41040021:19
iurygregorymwhahaha, i will review the patches after my dinner o/21:20
mwhahahaiurygregory: thanks21:20
iurygregorynp ^^21:20
*** gfidente has quit IRC21:21
EmilienMso iiuc we need notification_transport_url and default_transport_url21:21
EmilienMwhy don't we set notification_transport_url in poi yet?21:22
mwhahahai think we do for ceilometer maybe?21:23
mwhahahai see in cinder, cinder::ceilometer does the notification_transport_url which is odd21:23
mwhahahai have no idea what notification_transport_url is actually used for21:24
mwhahahamaybe neutron->nova or vice versa might use it21:24
EmilienMhttps://review.openstack.org/#/c/233258/21:25
*** shardy has quit IRC21:25
*** jayg is now known as jayg|g0n321:25
mwhahahaoh but it inherits from default_transport_url i assume so thats why we never change it21:25
mwhahahabecause generally people aren't deploying two instances for each21:25
EmilienMwhere do you see that?21:26
mwhahahawell i can only assume that's how it's still working :D let me see if i can validate in code21:26
*** rajinir has quit IRC21:26
*** sanagikoki has quit IRC21:26
mwhahahabecause i would assume 'DEFAULT/transport_url' would be the fall back for stuff21:26
*** imcsk8_ is now known as imcsk821:27
mwhahahahttps://github.com/openstack/oslo.messaging/blob/afaa4d935dd77760492f36dba622f6a01fe14fd9/oslo_messaging/notify/notifier.py#L142-L15421:27
mwhahahayea it falls back21:28
mwhahahawhich is why this is generally not a problem because tripleo/fuel/etc sets a single rabbitmq endpoint21:28
mwhahahawe aren't deploying multiple rabbit clusters at the moment21:28
EmilienMmwhahaha: ok so we are good with this param for now21:30
EmilienMmwhahaha: the only thing we need to solve is the ssl thing21:30
mwhahahaEmilienM: switch use_ssl to accept whatever then just hard code the true/false openstacklib21:30
mnaseris it possible that puppet-openstack_infra_spec_helper is broken under centos 7?  http://logs.openstack.org/02/410402/1/check/gate-openstackci-beaker-centos-7/77bd08e/console.html#_2016-12-13_21_27_28_99921821:36
*** amoralej|off is now known as amoralej21:36
mwhahahamnaser: probably21:37
mnasercan i update the gemspec to the older release?21:37
mwhahahayea sec we're trying to do that ourselves21:37
mnaserlike pin fast_gettext to 1.2021:37
mnaseroh okay21:37
mwhahahahttps://review.openstack.org/#/c/410380/21:37
mwhahaha1.1021:37
mwhahaha1.1.021:37
mwhahaha1.2.0 also requires ruby >= 2.121:37
mnaserso fast D:21:37
EmilienMmwhahaha: how can we see if ssl is enabled in the url?21:38
mwhahahaEmilienM: ssl can be enabled via a query param in the url21:38
mwhahahaEmilienM: so i'm not sure what you mean21:38
EmilienMI'll look in your function21:38
*** dprince has quit IRC21:39
EmilienMI want to patch your function to accept a boolean21:39
*** dprince has joined #puppet-openstack21:39
EmilienMdoes it make sense?21:39
mwhahahaEmilienM: Yea you can do that it should accept string/bool21:39
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/parser/functions/os_transport_url.rb#L7221:40
mwhahahamight have to some tricky logic there to do it hto21:40
EmilienMmwhahaha: wdyt about carlos comment about string in https://review.openstack.org/#/c/409911/6/manifests/profile/base/nova.pp ?21:41
EmilienMI think my code is ok21:41
mwhahahai guess it depends on how THT outputs it21:41
mwhahahanot sure if int or string21:41
mwhahahathe function requires a string21:41
EmilienMah right21:41
*** panda is now known as panda|Zz21:42
openstackgerritAlex Schultz proposed openstack/puppet-congress: Add oslo_messaging_notification transport_url  https://review.openstack.org/41040921:44
mwhahahastupid libvirt21:46
mwhahahawheres that libvrit p-o-i change21:46
dmsimardmwhahaha: mine ?21:46
dmsimardhttps://review.openstack.org/#/c/410344/21:46
mwhahahadmsimard: yea we need to merge our changes together21:46
mwhahahahttps://review.openstack.org/#/c/410388/21:46
dmsimardpff I don't want your patch21:46
mwhahahawell youre patch is red21:47
mwhahahaso if you want my patch,you must accept my patch :D21:47
dmsimardlol21:47
dmsimardsure, you want to squash it ?21:47
mwhahahaer first my = your21:47
mwhahahayea i can21:47
dmsimardI'm busy in bikeshedding in regards to the libvirt issue21:47
mwhahahasounds glorious21:47
*** johnmilton has joined #puppet-openstack21:47
*** johnmilton has quit IRC21:48
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Align stars to fix ci  https://review.openstack.org/41038821:49
*** dprince has quit IRC21:50
*** trown is now known as trown|outtypewww21:59
*** amoralej is now known as amoralej|off22:01
*** rmart04 has joined #puppet-openstack22:06
*** rajinir has joined #puppet-openstack22:08
dmsimardEmilienM: I don't understand your comments on my tripleo patches :(22:14
EmilienMdmsimard: you created a new parameter, just use it22:14
EmilienMdmsimard: like we do the Ceph FSID later in the file22:14
dmsimardbut libvirt_virt_type is also written as just "qemu" but there's a parameter for it22:15
dmsimardI looked at what existed for libvirt_virt_type and did the same thing for libvirt_cpu_mode ?22:15
EmilienMdmsimard: there isn't parameter because nobody created it22:15
dmsimardsure there is22:15
*** rmart04 has quit IRC22:15
EmilienMlet me look again22:15
EmilienMdmsimard: indeed, that's a bug22:16
dmsimardEmilienM: https://review.openstack.org/#/c/410358/1/puppet/services/nova-libvirt.yaml22:16
EmilienMwe should replace this hiera thing22:16
dmsimardit's right above what I added22:16
EmilienMand use the actual parameter :)22:16
EmilienMwe should use NovaComputeLibvirtType22:16
EmilienMi'll propose a patch that change that, or feel free to do it in your patch22:16
dmsimardok so in the scenarios, I put {get_param: NovaComputeLibvirtCpuMode} instead ?22:17
dmsimardbut then how do I make sure it's set to none ? we're defaulting to kvm/host-model in the parameters22:17
EmilienMdmsimard: no, NovaComputeLibvirtCpuMode: 'qemu'22:17
EmilienMerr22:17
dmsimardoh, okay I set the parameter, not the hiera value22:17
EmilienMNovaComputeLibvirtCpuMode: none22:17
EmilienMdmsimard: exactly!22:17
dmsimardeh, ok22:18
dmsimardI'll do the cpu_mode thing, just to touch as little as possible -- fixing the other non-parameterized stuff is out of scope22:18
EmilienMdmsimard: sure, you can also propose a separated patch. using parameters makes things easier to understand22:19
EmilienMusing hiera is kind of hack22:20
EmilienMmwhahaha: https://review.openstack.org/#/c/410388/ is passing CI jobs, i'm approving it22:29
EmilienM(looking at zuul)22:29
mwhahahak22:29
dmsimardI +2'd it so it doesn't look like you self approved it :p22:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Align stars to fix ci  https://review.openstack.org/41043822:35
EmilienMthe backport to newton ^22:35
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Align stars to fix ci  https://review.openstack.org/41043922:38
EmilienMthe backport to mitaka ^22:38
dmsimardmgariepy: FYI there's issues with CentOS 7.3 (released yesterday) and qemu-kvm-ev 2.6.0 (released today) when using virt_type=qemu and cpu_mode=host-model. You need to use cpu_mode=none (like devstack does) or you'll run into something like this:22:48
dmsimardhttp://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/libvirt/qemu/instance-00000001.txt.gz22:48
dmsimarddamnit wrong channel22:48
dmsimardmgariepy: FYI there's issues with CentOS 7.3 (released yesterday) and qemu-kvm-ev 2.6.0 (released today) when using virt_type=qemu and cpu_mode=host-model. You need to use cpu_mode=none (like devstack does) or you'll run into something like this:22:48
dmsimardhttp://logs.openstack.org/76/409476/4/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8881991/logs/libvirt/qemu/instance-00000001.txt.gz22:48
dmsimardwtf22:48
*** dgurtner has quit IRC22:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Remove testing for Glance Registry  https://review.openstack.org/41044823:05
EmilienMdmsimard: qemu version: 2.3.023:07
EmilienMis it expected?23:07
dmsimardwhat version of openstack23:07
dmsimardmitaka ?23:07
EmilienMin your logs23:08
EmilienMin https://review.openstack.org/#/c/409476/23:08
*** alee has quit IRC23:11
*** ducttape_ has quit IRC23:12
dmsimard2.3.0 doesn't work with 7.323:13
dmsimardand 2.6.0 doesn't work with 7.3 when cpu_mode=host-model23:13
dmsimard2.6.0 was release sometime today and is progressively being sync'd to all mirrors23:14
dmsimard7.3 was released yesterday23:14
EmilienMare we ok to land the backport ? https://review.openstack.org/#/c/410438/23:14
mwhahahaoh noes mitaka ci looks very unhappy23:14
EmilienMhttp://logs.openstack.org/39/410439/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/8ec0e86/console.html#_2016-12-13_22_54_22_19168423:15
EmilienMsounds like a mirror issue23:15
mwhahahaand all the centos is red for different reasons :D23:15
EmilienMceph.com is always down23:15
EmilienMlet me see if we can use the mirror for ceph in mitaka23:15
EmilienMI'll let you see the centos issue23:16
EmilienMhttp://mirror.regionone.osic-cloud1.openstack.org/ceph-deb-hammer/23:16
mwhahahashould be ok on newton if dmsimard wants to +A https://review.openstack.org/#/c/410438/23:16
EmilienMperfect23:16
dmsimardI sent an email to rdo-list, openstack-dev and openstack-operators about the issue23:16
EmilienMdmsimard: lol - please ping the world23:16
EmilienMall@all.com23:16
dmsimardhttp://lists.openstack.org/pipermail/openstack-dev/2016-December/108884.html23:17
dmsimardI prefer to pre-emptively communicate so that people don't lose too much time over this23:17
EmilienMnice, thanks23:17
mwhahahadmsimard: your pro-activeness is appreciated and will be abused later :D23:18
dmsimardI was an operator at $oldjob :(23:18
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/41045423:23
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: repos: allow to configure Debian mirror  https://review.openstack.org/41045523:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy ceph using OpenStack mirrors  https://review.openstack.org/41045923:29
EmilienMok that should fix the ceph mirror problem23:29
EmilienMdon't look at code please just +A :D23:30
EmilienMhttps://review.openstack.org/#/q/topic:ceph/mirror/hammer23:30
EmilienMnow the centos7 thing23:31
EmilienMscenario001 sounds broken because of ceph23:31
*** agrebennikov_ has quit IRC23:32
EmilienMscenario003 seems to fail on 3 tempest tests, probably something with nova/neutron23:32
EmilienMtest_server_basic_ops works, so I guess it's not the qemu thing23:32
EmilienMthe floating ip is not reachable23:32
EmilienMsame for scenario00223:33
EmilienMwhich means we have a general regression somewhere23:33
EmilienMhttps://review.openstack.org/#/q/project:%255Eopenstack/puppet-.*+status:merged+branch:stable/mitaka23:34
EmilienMeverything worked fine on December 9th, so 4 days ago23:34
EmilienMlet's diff a bit23:34
EmilienMhttps://www.diffchecker.com/Tr1gVPvy23:36
EmilienMok it's a mess23:36
EmilienMI don't see any OpenStack related thing23:37
EmilienMthough I see a lot of libvirt/qemu23:37
EmilienMdmsimard: in mitaka, I see qemu older23:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Mitaka CI test  https://review.openstack.org/41046023:40
mwhahahai'll be back later23:42
* mwhahaha wanders off23:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Remove testing for Glance Registry  https://review.openstack.org/41044823:42
EmilienMmwhahaha: happy meetup23:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Pin fast_gettext to 1.2.0  https://review.openstack.org/41046323:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy ceph using OpenStack mirrors  https://review.openstack.org/41045923:45
« Monday, 2016-12-12 Index Wednesday, 2016-12-14 »

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