Wednesday, 2016-08-17

« Tuesday, 2016-08-16 Index Thursday, 2016-08-18 »
*** Guest67611 is now known as saneax_AFK00:53
*** ducttape_ has quit IRC01:34
*** ducttape_ has joined #puppet-openstack01:35
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: re-enable trove  https://review.openstack.org/35195301:57
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing"  https://review.openstack.org/33981302:01
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: gnocchi: export db-sync output in logs  https://review.openstack.org/35617802:09
openstackgerritMichael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken  https://review.openstack.org/35198002:47
*** ducttape_ has quit IRC02:49
*** dhollinger has joined #puppet-openstack03:07
*** ducttape_ has joined #puppet-openstack03:13
*** ducttape_ has quit IRC03:36
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795603:39
*** dhollinger has quit IRC04:11
*** ducttape_ has joined #puppet-openstack04:36
*** ducttape_ has quit IRC04:41
openstackgerritMerged openstack/puppet-openstacklib: Add missing namevar method to openstack_config provider  https://review.openstack.org/35523504:52
*** jaosorior has joined #puppet-openstack05:13
*** ducttape_ has joined #puppet-openstack05:38
*** ducttape_ has quit IRC05:43
*** rcernin has joined #puppet-openstack05:46
openstackgerritKarthik S proposed openstack/puppet-neutron: Set firewall driver for SR-IOV ports  https://review.openstack.org/34350505:47
*** saneax_AFK is now known as saneax05:52
*** saneax is now known as Guest1805905:52
openstackgerritMerged openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings  https://review.openstack.org/35561506:06
openstackgerritMerged openstack/puppet-tripleo: Align hiera keys with service names  https://review.openstack.org/35358206:06
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/35624506:12
*** Guest18059 is now known as saneax_AFK06:15
*** gfidente has quit IRC06:18
*** ebalduf has quit IRC06:21
*** sofie has joined #puppet-openstack06:32
openstackgerritCharlie Llewellyn proposed openstack/puppet-cinder: Add variable to set availabilty zone fallback  https://review.openstack.org/35625506:35
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Enable aodh systemd services on non-bootstrap nodes  https://review.openstack.org/35611006:36
*** ducttape_ has joined #puppet-openstack06:39
*** sofie has quit IRC06:44
*** ducttape_ has quit IRC06:45
*** leanderthal|afk is now known as leanderthal06:54
*** markvoelker has quit IRC07:00
*** tesseract- has joined #puppet-openstack07:00
*** gfidente has joined #puppet-openstack07:00
*** foxhunt_away is now known as foxhunt7207:02
*** johnma has joined #puppet-openstack07:16
openstackgerritMerged openstack/puppet-tripleo: Add cinder-backup profiles  https://review.openstack.org/31121807:20
*** johnma has left #puppet-openstack07:21
*** amoralej|off is now known as amoralej07:26
*** dtantsur|afk is now known as dtantsur07:49
*** markvoelker has joined #puppet-openstack08:00
*** markvoelker has quit IRC08:05
*** ducttape_ has joined #puppet-openstack08:08
*** chem has joined #puppet-openstack08:09
*** ducttape_ has quit IRC08:12
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Fix deployment steps for AODH profiles  https://review.openstack.org/35237208:22
*** priteau has joined #puppet-openstack08:24
*** paramite has joined #puppet-openstack08:31
*** dgurtner has joined #puppet-openstack08:33
*** _degorenko|afk is now known as degorenko08:52
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles  https://review.openstack.org/35540908:55
*** xarses_ has joined #puppet-openstack08:55
*** derekh has joined #puppet-openstack08:57
*** xarses has quit IRC08:58
openstackgerritLukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class  https://review.openstack.org/35412508:59
openstackgerritErno Kuvaja proposed openstack/puppet-manila: Add CephFSNative driver logic  https://review.openstack.org/35392809:00
*** markvoelker has joined #puppet-openstack09:01
*** markvoelker has quit IRC09:06
*** gfidente has quit IRC09:07
*** ducttape_ has joined #puppet-openstack09:08
*** ducttape_ has quit IRC09:13
*** gfidente has joined #puppet-openstack09:26
*** maestropandy has joined #puppet-openstack09:29
*** maestropandy has left #puppet-openstack09:30
openstackgerritMerged openstack/puppet-tacker: Add authtoken class  https://review.openstack.org/35177909:32
*** openstack has joined #puppet-openstack10:17
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Add support for oslo privsep management  https://review.openstack.org/35600710:20
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: [WIP] Manage privsep helper for nova  https://review.openstack.org/35600510:21
*** hjensas has quit IRC10:45
openstackgerritErno Kuvaja proposed openstack/puppet-ceph: Expose osd crush update on start option  https://review.openstack.org/35637610:45
*** hjensas has joined #puppet-openstack10:46
*** hjensas has joined #puppet-openstack10:46
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles  https://review.openstack.org/35540910:50
*** paramite has quit IRC10:52
*** paramite has joined #puppet-openstack10:53
openstackgerritErno Kuvaja proposed openstack/puppet-ceph: Expose osd crush update on start option  https://review.openstack.org/35637610:59
openstackgerritSaravanan KR proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots  https://review.openstack.org/35249411:03
openstackgerritErno Kuvaja proposed openstack/puppet-ceph: Expose osd crush update on start option  https://review.openstack.org/35637611:07
*** ducttape_ has joined #puppet-openstack11:10
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Configure galera-monitor on all controller nodes  https://review.openstack.org/35639011:11
*** dtantsur is now known as dtantsur|bbl11:11
*** ducttape_ has quit IRC11:15
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668511:32
*** jaosorior has quit IRC11:35
*** jaosorior has joined #puppet-openstack11:36
*** amoralej is now known as amoralej|lunch11:40
*** ansmith has joined #puppet-openstack11:49
EmilienMhello11:53
*** morazi has joined #puppet-openstack11:59
*** jayg|g0n3 is now known as jayg12:01
*** johnmilton has quit IRC12:02
*** markvoelker has joined #puppet-openstack12:09
*** ducttape_ has joined #puppet-openstack12:10
openstackgerritLukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class  https://review.openstack.org/35412512:10
EmilienMiberezovskiy, degorenko: hey! any progress on https://review.openstack.org/#/c/355632/ ? :)12:12
*** jpena is now known as jpena|lunch12:15
*** cdelatte has joined #puppet-openstack12:16
*** dprince has joined #puppet-openstack12:16
EmilienMiberezovskiy, degorenko: tripleo is waiting for this patch, it's kind of urgent12:17
*** chem has quit IRC12:18
*** chem has joined #puppet-openstack12:19
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi  https://review.openstack.org/35247612:19
*** johnmilton has joined #puppet-openstack12:22
Jokke_iurygregory: thanks for all your help on the reviews. https://review.openstack.org/#/c/353928/ is passing the tests again!12:23
*** ducttape_ has quit IRC12:23
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles  https://review.openstack.org/35540912:27
*** johnmilton has quit IRC12:27
openstackgerritMerged openstack/puppet-tripleo: Fix deployment steps for AODH profiles  https://review.openstack.org/35237212:29
iurygregoryJokke_, you're welcome o/12:30
iurygregorymorning people12:30
*** guimaluf has joined #puppet-openstack12:30
*** johnmilton has joined #puppet-openstack12:41
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add manifests to inject and trust CA certificates  https://review.openstack.org/35302112:43
openstackgerritCharlie Llewellyn proposed openstack/puppet-cinder: Fixed alginment and added test  https://review.openstack.org/35635912:45
iberezovskiyEmilienM, unfortunately, Fuel Ci is broken. so we can't merge our adapt patch. I think we need to unblock you first. but let's please wait for mwhahaha. I want to notify him about this problem12:46
EmilienMok12:46
EmilienMwe don't want to break you more :)12:46
*** fultonj_ has joined #puppet-openstack12:46
degorenkoEmilienM, fuel ci is already broken by our internal problems, so, it doesnt matter if we will merge your patch12:47
degorenkobut agree, lets wait for Alex12:47
openstackgerritSven Anderson proposed openstack/puppet-openstack-integration: scenario003: Enable ec2api tempest tests  https://review.openstack.org/35644212:48
EmilienMok12:48
iberezovskiyEmilienM, jfyi https://review.openstack.org/#/c/356007/12:52
beaglesand the winning recheck goes to iurygregory !12:53
beaglesthanks man12:54
beaglesdid they disable osic or something?12:54
EmilienMiberezovskiy: oh nice12:55
EmilienMbeagles: your patch got merged?12:55
beaglesEmilienM, yup12:55
EmilienMbeagles: congrats \o/12:57
beaglesEmilienM, it was a team effort :)12:57
EmilienMand stars12:57
beaglesyup it looks like they disabled osic12:59
*** ducttape_ has joined #puppet-openstack13:00
*** paramite has quit IRC13:03
iurygregorybeagles, haha tks \o/13:03
*** mkarpin has quit IRC13:04
*** mkarpin has joined #puppet-openstack13:06
EmilienMchem: what should we do with https://review.openstack.org/#/c/354872/ ?13:08
EmilienMmerge it until next version of puppet?13:09
EmilienMit's blocking multiple patches13:09
chemEmilienM: they have release it master and backported it, but yes, in the meantime we could merge it.  I'm just afraid that we just forget to remove it.13:10
EmilienMchem: do they have released it?13:10
chemEmilienM: merged to stable: 05a7ced, and merged to master 2 days ago13:11
EmilienMah, so it's not released13:11
*** amoralej|lunch is now known as amoralej13:12
chemnope13:12
chemEmilienM: I've just added a comment asking for a quick release.13:14
*** fultonj_ has quit IRC13:15
*** fultonj_ has joined #puppet-openstack13:16
*** fultonj_ has quit IRC13:16
EmilienMnice!13:16
chemin the meantime, we can merge it and let an note on the next weekling meeting to revert it if released13:17
chemEmilienM: ^ ?13:17
EmilienMchem: yes13:17
EmilienMI did a recheck13:18
EmilienM+2 now13:18
chemack, I'm updating https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016082313:18
*** jpena|lunch is now known as jpena13:18
openstackgerritErno Kuvaja proposed openstack/puppet-ceph: Expose osd crush update on start option  https://review.openstack.org/35637613:19
EmilienMchem: thx13:20
*** fultonj has quit IRC13:20
iurygregoryEmilienM, can you +A https://review.openstack.org/#/c/356014/ ? ^^13:22
*** fultonj has joined #puppet-openstack13:22
EmilienMiurygregory: done!13:23
EmilienMalmost 100 meetings!13:23
iurygregoryyeah13:23
iurygregory:D13:23
iurygregorydid you have sucess in know the TripleO problem in nova authtoken?13:24
*** dtantsur|bbl is now known as dtantsur13:24
openstackgerritMerged openstack/puppet-openstack-guide: Update meeting #90  https://review.openstack.org/35601413:26
*** ducttape_ has quit IRC13:26
EmilienMiurygregory: not yet13:27
iurygregoryEmilienM, ok, i was about to try the recheck lol13:28
EmilienMiurygregory: I did it, because the tripleo error was not related13:28
iurygregoryEmilienM, awesome :D13:28
beaglesEmilienM++13:43
beagles"recheck this is insane" lol13:43
EmilienMlol13:43
*** xarses_ has quit IRC13:47
*** xarses_ has joined #puppet-openstack13:48
*** karthiks_afk is now known as karthiks13:50
*** danpawlik has joined #puppet-openstack13:54
*** ducttape_ has joined #puppet-openstack13:55
mwhahahaiberezovskiy: is it still broken?14:05
*** xarses_ has quit IRC14:06
*** dfisher has joined #puppet-openstack14:08
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: trove: add db_flavor_ref parameter  https://review.openstack.org/35649114:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: re-enable trove  https://review.openstack.org/35195314:12
EmilienMtripleo CI is currently blocked to promote to latest trunk and we might need https://review.openstack.org/#/c/355632/ asap14:13
EmilienMmwhahaha, iberezovskiy: would it be ok to land the puppet patch?14:14
mwhahahayea gimme a sec to try and catch up this morning, we can probably land it14:15
iberezovskiymwhahaha, as I can see it was fixed some minutes ago, but I didn't check14:15
mwhahahayea it looks like it's fixed now14:16
*** xarses_ has joined #puppet-openstack14:30
EmilienMmwhahaha: should we recheck fuel CI?14:38
iberezovskiynot yet14:41
EmilienMok14:41
EmilienMiberezovskiy: it would be great if we could merge it today, just fyi14:42
iberezovskiyyep, I understand14:42
iberezovskiyI'm on it14:42
karthiksEmilienM, regarding https://review.openstack.org/#/c/343505/3/manifests/agents/ml2/sriov.pp@78, if we need to set a fixed value in puppet, wouldn't it be appropriate to set it as default.14:45
EmilienMkarthiks: the default value in neutron doesn't work?14:46
karthiksEmilienM, As far as I understand from  http://docs.openstack.org/mitaka/networking-guide/adv-config-sriov.html, (section Enable neutron sriov-agent) this configuration needs to be given explicitly14:50
EmilienMkarthiks: ok then14:50
EmilienMkarthiks: +214:51
EmilienMand we're waiting for tripleo/fuel CI to be green14:51
karthiksThanks EmilienM :)14:51
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Clean up unused params from aodh profiles  https://review.openstack.org/35652314:51
*** derekh has quit IRC14:52
*** openstackgerrit has quit IRC15:03
*** openstackgerrit has joined #puppet-openstack15:04
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795615:10
*** dprince has quit IRC15:11
*** ducttape_ has quit IRC15:11
*** dhollinger has joined #puppet-openstack15:19
*** saneax_AFK is now known as saneax15:26
*** saneax is now known as Guest6646615:27
colonwqxarses_, If I hit the patch enough, it will work and I will have a flat spot on my head.15:27
*** leanderthal is now known as leanderthal|afk15:39
*** foxhunt72 is now known as foxhunt_away15:39
*** ducttape_ has joined #puppet-openstack15:40
beaglesEmilienM (and anyone else who wants to chime in) while I don't have much puppet experience - nor ruby for that matter - I do like to apply what I learn as I go. The reason I ask is I'm wondering how rigorous I should go on reviewing puppet patches. My feeling is be as rigorous as I know how and if I'm wrong somebody who knows better will correct me.15:51
*** jaosorior has quit IRC15:51
beaglesthoughts?15:52
EmilienMbeagles: I started the review process by looking how others used to review and what were common use15:52
EmilienMjust pick some patches under review, and try to see if the patch has the doc, is tested, is following conventions: http://docs.openstack.org/developer/puppet-openstack-guide/coding-style.html15:53
beaglesEmilienM, ack15:53
EmilienMif it's passing CI, if the commit message is good15:53
EmilienMfor the code itself, it comes by reading existing code and be aware about how to write good puppet manifests15:53
beaglesthe usual :)15:54
beaglesit's a bit interesting as each community has - even within the mundane things like commit messages - different standards15:57
beaglesso the reviewing the reviewers is the big thing I need to work at15:57
*** ebalduf has joined #puppet-openstack15:57
beaglesEmilienM, thanks15:58
*** rcernin has quit IRC15:58
*** bodepd has quit IRC15:59
*** dtantsur is now known as dtantsur|brb16:00
*** amoralej is now known as amoralej|off16:06
*** danpawlik has quit IRC16:13
EmilienMyeah ! https://review.openstack.org/#/c/356491/ really helps to run trove tests16:14
EmilienMif anyone can review it ^16:14
EmilienMthere is one more failure but I think it's a bug in trove/tempest tests, I'm on it16:14
EmilienMhttp://logs.openstack.org/53/351953/8/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/6bfc661/console.html#_2016-08-17_14_48_58_39263416:14
EmilienMbut the 2 other failures we had are gone with my puppet-tempest patch16:14
*** dprince has joined #puppet-openstack16:27
EmilienMnot sure I'm doing right: https://review.openstack.org/35658716:28
beaglesdegorenko, iberezovskiy, the reason I asked that question of EmilienM above is I was in the middle of reviewing https://review.openstack.org/#/c/344918/ and I got to wondering "what's too far?". Maybe you can advise me a bit here?16:36
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles  https://review.openstack.org/35540916:37
*** rwsu has quit IRC16:38
*** dtantsur|brb is now known as dtantsur16:49
*** tesseract- has quit IRC17:00
*** rcernin has joined #puppet-openstack17:26
iberezovskiybeagles, good comments I think17:29
iberezovskiybeagles, the main idea on review for me is to do not say 'this is wrong', the idea is to describe what's exactly wrong and suggest how to fix, attach links for additional information if possible17:31
beaglesiberezovskiy, ack17:35
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/35624517:36
*** priteau has quit IRC17:36
openstackgerritMerged openstack/puppet-tempest: trove: add db_flavor_ref parameter  https://review.openstack.org/35649117:36
*** dtantsur is now known as dtantsur|afk17:37
beaglesiberezovskiy, what are your thoughts on two resources conflicting with each other, ie 'eth0:4', 'eth0:1'17:37
iberezovskiyit won't conflict because of different title17:38
iberezovskiybut they will manage the same configuration file17:38
beaglesiberezovskiy, in puppet they will but they end up hitting the same file yeah17:38
iberezovskiyI think it's the user issue, because he provided incorrect input17:38
beaglesiberezovskiy, it just ends up being a user issue17:38
beaglesyeah what you said :)17:38
iberezovskiy:)17:38
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Move ceilometer api to run under apache wsgi  https://review.openstack.org/35247618:02
EmilienMiberezovskiy: thx a lot18:07
iberezovskiyEmilienM, thx for the patience. mwhahaha can land both patches then18:10
iberezovskiyI'm off for today18:10
*** iberezovskiy is now known as iberezovskiy|off18:10
*** xarses_ has quit IRC18:13
dmsimardEmilienM: hi18:16
dmsimardThis is sort of weird: https://github.com/openstack/puppet-openstack-integration/blob/master/Puppetfile#L8818:16
dmsimardSince the module includes itself (rather than copy itself?), it installs the master branch of puppet-openstack-integration in /etc/puppet/modules instead of whatever version I'm executing run_tests.sh from18:17
dmsimardso for instance I'm trying to reproduce the designate issues outside the gate to troubleshoot and it's unable to find openstack_integration::designate because the master branch doesn't have it18:18
dmsimardShouldn't p-o-i do a puppet module install of itself or something like that instead ?18:18
mwhahahazuul cloner overwrites it i think18:22
mwhahahain the gate18:23
*** gfidente has quit IRC18:24
*** laurosn2 has joined #puppet-openstack18:26
*** _ducttape_ has joined #puppet-openstack18:27
dmsimardI'm testing outside the gate, though :p18:27
EmilienMdmsimard: hi18:28
EmilienMI see your problem18:28
EmilienMyou'll need to hack something in functions18:29
*** ducttape_ has quit IRC18:30
dmsimardEmilienM: here? https://github.com/openstack/puppet-openstack-integration/blob/master/functions#L6718:32
EmilienMmaybe yeah18:33
dmsimardand the hack would be to delete the openstack_integration module from /etc/puppet/modules and copy itself to /etc/puppet/modules ?18:33
dmsimard:(18:33
*** degorenko is now known as _degorenko|afk18:37
openstackgerritLars Kellogg-Stedman proposed openstack/puppet-tripleo: [WIP] updates to fluentd support  https://review.openstack.org/35350718:48
EmilienMmwhahaha: we good for https://review.openstack.org/#/c/355632/ ?18:51
mwhahahanot yet, trying to land the fuel fix (waiting on ci)18:51
mwhahahaif it doesn't come through i'll just land that one anyway in a bit18:51
mwhahahahttps://review.openstack.org/#/c/356336/ if you're following along at home :D18:52
EmilienMsorry to push18:52
EmilienMthe reason is tripleo CI blocked for 3 days18:52
mwhahahano it's ok, i'm hoping in the next hour18:53
EmilienMkk18:53
iurygregoryEmilienM, can you ping when TripleO be unblocked? =)18:54
*** _ducttape_ has quit IRC18:56
*** ducttape_ has joined #puppet-openstack18:57
EmilienMiurygregory: when the puppet-ironic is merged :)18:57
iurygregoryEmilienM, ok ;D18:58
*** paramite has joined #puppet-openstack18:59
*** laurosn2 has quit IRC19:07
*** beagles is now known as beagles_biab19:08
*** paramite has quit IRC19:08
*** danpawlik has joined #puppet-openstack19:15
mwhahahalife in openstack, watching paint dry aka waiting for CI19:20
EmilienMlol19:21
iurygregorymwhahaha, go watch the olympic games XD19:21
danpawlikmwhahaha: xD19:21
iurygregorywhile you wait for CI19:21
EmilienMI don't have a TV :(19:22
iurygregoryEmilienM, internet LOL19:22
EmilienMwhat is that?19:22
mwhahahanbc has a lock on the coverage in the US (and it's awful)19:22
EmilienMI'm really old school19:22
EmilienMmy laptop crash when I run youtube video19:23
iurygregoryEmilienM, LOL19:23
EmilienMI have a tablet19:23
iurygregoryyou have a mac no?19:23
EmilienMno19:23
iurygregorysad =(19:23
EmilienMold dell xps :P19:23
mwhahahahe runs linux i think so that's probably the problem ;)19:23
iurygregorylinux is not the problem hahah we watch olympic games in the second monitor here at the lab :D19:24
EmilienMI'm watching zuul19:25
EmilienMwhich is more than olympic game19:25
iurygregoryYeah19:26
EmilienM"trying to make CI working" is like olympic sometimes19:26
EmilienMbut we'll survive19:26
EmilienMif mwhahaha merges my patch19:27
EmilienM:-P19:27
EmilienM /rent19:27
mwhahahasooon19:27
EmilienM /rant* even19:27
iurygregoryEmilienM, the sab pat is that you don't recive a gold medal for have the  CI working haha19:27
iurygregorysad*19:27
*** ayoung has quit IRC19:27
*** danpawlik has quit IRC19:28
EmilienMwe all receive a medal19:28
EmilienMthe medal will be Newton release19:28
*** _ducttape_ has joined #puppet-openstack19:28
*** xarses_ has joined #puppet-openstack19:29
* mwhahaha grumbles about memory utilization19:30
iurygregoryEmilienM, good one :D19:30
mwhahahaalmost there on fuel ci, deploy passed just waiting for health checks so i can merge that patch (hopefully in the next 5-10 mins)19:30
*** ducttape_ has quit IRC19:32
EmilienMok19:33
*** johnmilton has quit IRC19:34
*** xarses_ has quit IRC19:34
*** xarses_ has joined #puppet-openstack19:44
mwhahahayay it passed come on jenkins19:44
*** danpawlik has joined #puppet-openstack19:45
-openstackstatus- NOTICE: The volume for logs.openstack.org filled up rather suddenly, causing a number of jobs to fail with a POST_FAILURE result and no logs; we're manually expiring some logs now to buy breathing room, but any changes which hit that in the past few minutes will need to be rechecked and/or approved again19:46
mwhahahabooo logs.openstack.org19:46
EmilienMyay19:47
*** _ducttape_ has quit IRC19:49
mwhahahahmm19:49
*** ducttape_ has joined #puppet-openstack19:50
mwhahahai guess i was looking at the wrong tab, it failed let me double check it's  not related and i might just force it19:51
mwhahahaEmilienM: just merge your change to unblock tripleo and i'll work on getting the fuel patch landed soon19:52
*** dgurtner has quit IRC19:52
mwhahahapesky CI shenanigans today19:52
EmilienMmwhahaha: i don't want to break your CI too19:52
mwhahahait's already hosed for probably completely unrelated reasons19:53
mwhahahai've retriggered the fuel-ci for it so it'll be another 2 hours before it comes back19:54
EmilienMdang19:58
EmilienMwe really need to land this patch in puppet-ironic19:58
mwhahahaEmilienM: did you see https://review.openstack.org/#/c/355203/19:58
EmilienMmwhahaha: no19:58
EmilienMthanks19:58
EmilienMlooking19:59
EmilienMmwhahaha: -219:59
mwhahaha:o19:59
EmilienM-620:00
EmilienMI wish I could -6 sometimes20:00
EmilienMmwhahaha: do we wait 2 more hours? lol20:01
mwhahahajust land it20:02
*** rwsu has joined #puppet-openstack20:02
mwhahahait doesn't break noop or anything and we don't test ironic in CI20:02
mwhahahai'll get our fix landed later today20:02
mwhahahaor i will20:03
EmilienMok thx20:03
*** xarses_ has quit IRC20:12
*** xarses_ has joined #puppet-openstack20:12
*** danpawlik has quit IRC20:17
*** danpawlik has joined #puppet-openstack20:17
*** ayoung has joined #puppet-openstack20:18
*** jayg is now known as jayg|g0n320:18
*** dprince has quit IRC20:19
EmilienMmwhahaha: wdyt about https://review.openstack.org/#/c/354872/ until puppetlabs release > 4.6.0 ?20:21
mwhahahai guess as a temporary thing20:21
mwhahahabut the number of places we need to do that for various modules is quite annoying20:22
EmilienMpuppet-keystone is kind of prio20:22
mwhahahayea20:22
EmilienMas it breaks beaker jobs20:22
EmilienMother projects *only* have unit-latest broken20:23
EmilienMwhich is non voting20:23
mwhahahabetter to pin to puppet agent 1.5.2?20:23
mwhahahacause that's ultimately what's doing it20:23
mwhahahai think we'll end up with weird parse order issues if we have to go include everything20:23
*** cdelatte has quit IRC20:25
*** danpawlik has quit IRC20:25
mfischchem: ping20:27
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-guide: CI: updates about current jobs  https://review.openstack.org/35412220:27
EmilienMmfisch: what's up20:28
mfischworking on this weird domain Id bug with franza20:28
mfischwe can see the default domain Id changing after a resource is made20:28
mfischkinda strange20:28
mfischhttp://paste.openstack.org/show/559094/20:29
mfischafter we create the heat domain the default domain ID changes20:29
EmilienMyou running master?20:29
mfischwhich is wrong20:29
EmilienMfork?20:29
mfischno20:29
mfischyou think this is fixed?20:29
mfischwe're a bit behind but thise code looks the same from what I saw20:29
EmilienMno, just making sure we test the same code20:29
EmilienMwhat is your manifest?20:30
EmilienMcan we reproduce in CI? our scripts test idempotency20:30
mfischWe're from March 1620:30
mfischdo we create a domain in any of our tests?20:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: nova: switch libvirt_virt_type from qemu to kvm  https://review.openstack.org/35669520:31
EmilienMdmsimard: ^20:31
dmsimardEmilienM: http://logs.openstack.org/99/355599/1/check/gate-tempest-dsvm-full-ubuntu-xenial/fbe5f9b/logs/etc/nova/nova.conf.txt.gz20:31
EmilienMmfisch: lol20:31
EmilienMis it a real question ahah20:31
dmsimardvirt_type = qemu20:32
EmilienMdmsimard: ah so I can cancelt it20:32
mfischhah I dont even remember20:32
EmilienMmfisch: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/default_domain_spec.rb20:32
EmilienMmfisch: look in https://github.com/openstack/puppet-keystone/tree/master/spec/acceptance :)20:32
dmsimardEmilienM: well, no20:32
dmsimardEmilienM: feel free to let it go20:32
EmilienMmfisch: we also test here: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb20:33
EmilienMdmsimard: let's see how it works :)20:33
mfischEmilienM: I bet I could get this to fail by changing the intergration tests20:34
EmilienMsure, please try20:35
EmilienMmfisch: again, what is your manifest?20:35
*** Jokke_ has quit IRC20:35
mfischEmilienM: adding to the bug20:35
mfischLP 161414120:36
openstackLaunchpad bug 1614141 in puppet-keystone "Need to create heat domain without changing the default_domain_id" [Undecided,New] https://launchpad.net/bugs/161414120:36
mfischpretty basic just some ensure resources20:36
mfischon keystone_domain20:36
*** Jokke_ has joined #puppet-openstack20:37
openstackgerritMerged openstack/puppet-ironic: conductor: add api_url parameter  https://review.openstack.org/35563220:38
*** danpawlik has joined #puppet-openstack20:38
EmilienMmfisch: why don't use use https://github.com/openstack/puppet-heat/blob/master/manifests/keystone/domain.pp ?20:39
mfischlol because of the other bug I filed earlier that you commented on20:39
mfischheat and Keystone on different nodes20:39
mfischthats where the code came from20:39
EmilienMah20:39
EmilienMindeed20:40
*** rcernin has quit IRC20:40
EmilienMwhat I would do is :20:41
EmilienM1) patch heat::keystone::domain to add a new conditional if $manage_heat_config20:41
EmilienM2) use heat::keystone::domain on keystone server and set manage_domain, manage_user and manage_role to True20:41
EmilienM3) use heat::keystone::domain and set $manage_heat_config to True and the other to False20:42
EmilienMand honestly, I don't understand 100% the bug description20:43
EmilienM"Even if I set the identity/default_domain_id to default, the mistake persists."20:43
EmilienMfrom keystone.conf?20:43
EmilienMdid you configure something into heat.conf?20:44
EmilienMand why did you mention the idempotency problem, which isn't mentionned in the bug report20:44
*** ansmith has quit IRC20:47
mfischEmilienM: I think I found it20:47
mfischweird corner case20:47
mfischwill update the bug20:47
EmilienMbug set to incomplete20:47
EmilienMyeah and please make it a bit more clear20:48
EmilienMmfisch: and also please submit a patch into puppet-heat20:48
EmilienMdo not duplicate code like this in your manifests, this is really durty20:48
*** danpawlik has quit IRC20:48
mfischagreed on dirty20:48
EmilienMyour use case is really valid (containers)20:48
mfischEmilienM: this bug happens when the default domain is NOT in your catalog and a non-default domain IS20:48
EmilienMmhh i'm not sure to get it :)20:49
mfischconfirming now20:49
mfischyou can make a catalog that does not define the default domain20:49
mfischbut that does define a domain called "matt"20:49
mfischthen this code fails20:49
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Parameterize the libvirt_virt_type parameter so it can be overridden  https://review.openstack.org/35670820:51
EmilienMmfisch: so in our CI we do not define a default domain20:52
EmilienMhttps://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp#L5220:52
mfischopenning all your CI link spam again mon ami20:52
EmilienMyou say, if I add keystone_domain { 'mfish' } it will fail?20:52
mfischI can make it not fail if you set is_default to false on the mfish20:53
mfischthe failure is just a warning20:53
EmilienMmfisch: can you look https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp and try to submit a patch that test this corner case?20:53
EmilienMI'm really unsure about how to reproduce it20:54
EmilienMand you seem the best person to do it20:54
mfischEmilienM: can I just fork this test and push a review up?20:54
mfischwill that show it?20:54
iurygregorywhat error you get mfisch ?20:54
mfischiurygregory: a warning20:54
mfischlet me update the bug since nothing is clear right now20:55
mfischbut I could break this test easily20:55
EmilienMmfisch: fork what?20:55
EmilienMthis is not a test20:55
EmilienMthis is a manifest20:55
mfischsure20:55
mfischan integraton manifest20:55
mfischI can make one that breaks by changing about 8 lines20:55
EmilienMwe have 3 scenarios, right?20:55
EmilienMthey run keystone20:56
EmilienMso modify this keystone.pp to make it fail and we'll see what is the problem20:56
mfischyep ok20:57
EmilienMthanks!20:57
mfischiurygregory: updated20:59
mfischhttps://bugs.launchpad.net/puppet-keystone/+bug/161414120:59
openstackLaunchpad bug 1614141 in puppet-keystone "creating a keystone_domain can make it the default even though is_default is false" [Undecided,Incomplete]20:59
mfischsee if that makes sense to you ^20:59
mfischI'm going to work on the integration for slave driver EmilienM ;)20:59
openstackgerritCharlie Llewellyn proposed openstack/puppet-cinder: Fixed alginment and added test  https://review.openstack.org/35635921:00
iurygregorymfisch, tks I'll look at home =)21:03
EmilienMmfisch: lol21:03
mfischiurygregory: can I have my guy re-file the bug in portuguese?21:04
iurygregorymfisch, oh is the laurosn problem?21:05
iurygregoryO.o21:05
mfischiurygregory: this guy went to your school21:05
mfischhe lives here now21:05
*** danpawlik has joined #puppet-openstack21:05
iurygregorymfisch, oh awesome XD , there is no need to be in portugiese =)21:06
mfischI was j/k21:07
mfischjust cool that 2 guys fro the same school working on this21:07
iurygregoryI'll try to help be sure o/ just need to finish internal things for tomorrow deadline21:08
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Break tests for 1614141  https://review.openstack.org/35672621:09
mfischEmilienM: I think thats all you need for a corner case ^21:10
mfischlet me make a better commit msg21:10
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Break tests for 1614141  https://review.openstack.org/35672621:11
mfischiurygregory: so this was originally filed by a guy you met, Franza21:14
mfischFranciraldo21:14
iurygregorymfisch, yeah i saw the comments in bug :D21:14
EmilienMoh wow21:17
EmilienMhttps://review.openstack.org/#/c/356695/21:17
EmilienMdmsimard: ^ it pass CI21:17
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Parameterize the libvirt_virt_type parameter so it can be overridden  https://review.openstack.org/35670821:17
dmsimardEmilienM: lol wtf21:17
dmsimardEmilienM: well21:17
dmsimardEmilienM: I /think/ nested KVM works, even without kvm_nested=121:18
dmsimardIt'll just be slow because hardware extensions aren't exposed21:18
EmilienMvirt_type=kvm21:18
bkeroIt will fall back to qemu21:18
dmsimardbkero: you think ?21:18
bkeroThat's how I've seen it behave21:18
EmilienMcan you see in our logs? http://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/21:18
EmilienMif we actually run kvm?21:18
bkerodmsimard: Yeah, I've seen whole clouds accidentally not kvm_nested=1, and everything worked. VMs were just slow.21:19
EmilienMI see in http://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/dstat.txt.gz21:19
dmsimardEmilienM: we'd probably see it here if tempest wouldn't kill the VMs http://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/ps.txt.gz21:19
EmilienMqemu-kvm21:19
EmilienMyou can see it in dstat21:19
dmsimardEmilienM: oh, and otherwise you'd just have qemu ?21:19
EmilienMah no21:19
EmilienMqemu-kvm is also started on other jobs without this patch21:20
EmilienMhow can we check?21:20
bkeroThat's just the binary name I think.21:20
dmsimardyeah, you kinda need the whole process line and args to check what hypervisor it runs21:20
EmilienMin libvirt logs? http://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/libvirt/qemu/21:20
EmilienMI have one:21:20
bkeroThe option is -enable-kvm21:20
EmilienMhttp://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/libvirt/qemu/instance-0000000b.txt.gz21:20
dmsimardlike http://paste.openstack.org/show/559102/21:20
bkeroor I guess passing -machine=blah,accel=kvm21:20
dmsimardhttp://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/libvirt/qemu/instance-0000000b.txt.gz looks right to me21:21
EmilienMI have accel=kvm21:21
bkeroIf it didn't throw a "no kvm support" message, it is probably enabled21:21
EmilienMoh ben is right21:21
bkeroqemu throws that to stdout/stderr21:21
EmilienMlooks this is a job without kvm:21:21
EmilienMhttp://logs.openstack.org/45/356245/1/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/528a28f/logs/libvirt/qemu/instance-0000000a.txt.gz21:21
EmilienMand there is no accel=kvm21:22
bkeroaccel=tcg. Wonder what tcg is21:22
EmilienMlet's see in syslog21:22
EmilienMhttp://logs.openstack.org/95/356695/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1741c97/logs/syslog.txt.gz#_Aug_17_20_44_3221:22
dmsimardthat ran on centos-7-osic-cloud1-357389621:22
EmilienMHypervisor detected: KVM21:22
EmilienMah nevermind21:22
bkeroTCG = tiny code generator, probably some HLE21:22
EmilienMit's the VM itself21:22
dmsimardI'd be curious to check something off of Rackspace, which runs Xen21:22
bkeroDo you have the output of /proc/cpuinfo?21:23
bkeroCan check for 'vmx' flag21:23
dmsimardlol, all 6 jobs ran from osic21:23
dmsimardT_T21:23
bkerohttp://logs.openstack.org/45/356245/1/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/528a28f/logs/cpuinfo.txt.gz looks so21:23
dmsimardMaybe osic exposes nested and stuff21:23
dmsimardI would expect the job to fail on other providers21:23
EmilienMwe can do recheck to see if it works again21:24
dmsimardEmilienM: can we do a recheck until we hit something else than osic21:24
EmilienMmaybe we can use facter to determine if nested virt is enabled21:25
EmilienMand whether or not set qemu as value21:25
EmilienMbut yeah, it's weird kvm is not tested21:25
EmilienMdmsimard: osic is not like other rackspace clouds FYI21:26
EmilienMit's not rackspace cloud servers, etc21:26
EmilienMit's another project afik21:26
* dmsimard shrugs21:27
bkeroDifferent clouds pass different settings to their hypervisors. I wouldn't necessarily expect nested=1 everywhere21:27
dmsimardyeah, exactly21:27
*** gfidente has joined #puppet-openstack21:27
dmsimardespecially since it's not a nodepool provider requirement21:27
dmsimard(although I would've somehow expected otherwise)21:28
bkeroYou could add a new label to zuul and tell it which providers have nested kvm, then have the jobs run on that.21:28
dmsimardseriously, we're testing openstack on openstack clouds without nested virt as a requirement21:28
dmsimardthat sounds pretty awkward21:28
dmsimardbkero: not a bad idea21:28
*** Guest66466 is now known as saneax_AFK21:29
openstackgerritMerged openstack/puppet-openstack-guide: CI: updates about current jobs  https://review.openstack.org/35412221:31
bkerodmsimard: That might not be accepted as an undue burden on a single provider21:31
bkeroWhat about doing multinode testing with ironic?21:32
dmsimardyeah they're not interested in any way that would facilitate targetting nested virt deployment scenarios21:34
bkeroclarkb says it's unreliable21:34
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: (mitaka only) loadbalancer: manage ipv6 sysctl  https://review.openstack.org/35675521:42
EmilienMcouldn't we use an existing fact?21:45
EmilienMto find out if kvm is available?21:45
dmsimardEmilienM: but then you'd be sort of randomly testing nested or non-nested21:46
EmilienMyeah21:46
EmilienMI'll abandon my patch21:46
EmilienMit's risky21:47
EmilienMI'm sad21:47
EmilienMwe were the first one doing that :P21:47
dmsimardEmilienM: I submitted https://review.openstack.org/#/c/356708/21:47
dmsimardI'll leverage it with weirdo.21:47
EmilienMI saw21:47
EmilienM-621:47
dmsimard?21:47
EmilienMit's my new score21:48
EmilienMok I go running now, please don't break CI21:48
dmsimardSPACESHIP OPERATOR21:51
dmsimardI love the name of that operator.21:51
iurygregoryO.o21:51
iurygregoryspaceship?21:51
iurygregoryhahah21:51
dmsimardiurygregory: https://docs.puppet.com/puppet/latest/reference/lang_collectors.html21:52
dmsimardResource collectors (AKA the spaceship operator)21:52
iurygregorydmsimard, ohhhh21:52
dmsimard<| |> looks like a spaceship :)21:52
EmilienM<| |>      --  --  --  -- <| |>21:53
EmilienMit's a battle of spaceships21:53
EmilienMok I go now21:53
iurygregoryhahahah awesome name21:54
dmsimardIs it possible to check the value of another fact within a fact ?21:56
dmsimardi.e, a custom fact that'd check the value of another fact in order to return an appropriate value21:57
dmsimardor would I need to run a literal "facter" command within the custom fact to retrieve the value21:57
dmsimardAh, you can check with Facter.value(:fact)21:59
dmsimardCool.21:59
*** ayoung has quit IRC22:14
*** amoralej|off has quit IRC22:24
dmsimardEmilienM: fyi, hacky but it works :p https://review.openstack.org/#/c/356774/2/packstack/puppet/modules/packstack/lib/facter/default_hypervisor.rb22:26
dmsimardEmilienM: I can submit it to openstacklib in a more generic format if you like the approach22:27
dmsimardpuppet-nova defaults to kvm22:27
*** amoralej has joined #puppet-openstack22:30
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Change authtoken to class  https://review.openstack.org/35679222:30
*** dhollinger has quit IRC22:31
*** saneax_AFK is now known as saneax22:33
*** saneax is now known as Guest6566322:34
*** danpawlik has quit IRC22:37
EmilienMdmsimard: yes it would work22:42
EmilienMwell I wouldn't have written a fact for that22:42
EmilienMbut it works22:42
*** beagles_biab is now known as beagles22:42
dmsimardEmilienM: it's a required hack in the context of packstack22:42
EmilienMi met a spaceship when I ran22:42
EmilienMnot sure if it's /me tired22:43
EmilienMdmsimard: if $::is_virtual { $hypervisor = 'qemu' } else { $hypervisor = 'kvm' } would also work22:43
dmsimardThat's exactly what Packstack has and it doesn't allow for someone to run qemu on baremetal or kvm on virtual.22:44
dmsimardSo I made the hypervisor configurable while keeping the same default/backwards compat22:44
EmilienMexcellent22:44
EmilienM+622:44
*** xarses_ has quit IRC22:45
*** ducttape_ has quit IRC22:58
*** dfisher has quit IRC23:02
EmilienMmfisch: your patch is working well https://review.openstack.org/#/c/356726/23:10
EmilienMI don't see where could be the bug23:11
EmilienMat least the puppet runs are idempotent23:11
*** ayoung has joined #puppet-openstack23:18
*** fultonj has quit IRC23:37
*** rwsu has quit IRC23:51
« Tuesday, 2016-08-16 Index Thursday, 2016-08-18 »

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