Thursday, 2016-08-25

*** michchap_ has joined #puppet-openstack00:11
*** michchap has quit IRC00:13
*** ducttape_ has quit IRC00:21
*** Ahharu has quit IRC00:32
*** ducttape_ has joined #puppet-openstack00:35
*** ducttape_ has quit IRC00:40
*** ducttape_ has joined #puppet-openstack00:50
*** ducttape_ has quit IRC00:51
EmilienM_ody: probably a packaging problem00:53
EmilienMprobably in https://github.com/rdo-packages/murano-distgit/blob/mitaka-rdo/openstack-murano.spec00:53
EmilienMI don't see a package for tempest00:54
EmilienMiurygregory: yes00:54
*** markvoelker has joined #puppet-openstack00:55
iurygregory_EmilienM, ack tks00:55
iurygregory_swift is the last one \o/00:55
iurygregory_all other patches are ready and upstream :D (heat, magnum, neutron, trove)00:56
EmilienMiurygregory_: excellent work here00:58
iurygregory_EmilienM, tks :D00:58
*** saneax-_-|AFK is now known as saneax01:16
*** ebalduf has quit IRC01:22
*** saneax is now known as saneax-_-|AFK01:32
*** markvoelker has quit IRC01:36
openstackgerritMerged openstack/puppet-swift: Add log directory to .gitignore  https://review.openstack.org/35933701:36
*** ilbot3 has quit IRC01:45
*** ilbot3 has joined #puppet-openstack01:47
openstackgerritMerged openstack/puppet-keystone: Add CORS settings  https://review.openstack.org/35408701:48
*** ducttape_ has joined #puppet-openstack01:52
*** ducttape_ has quit IRC01:57
*** saneax-_-|AFK is now known as saneax01:58
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004  https://review.openstack.org/33910602:02
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken  https://review.openstack.org/36014602:12
*** ducttape_ has joined #puppet-openstack02:15
*** saneax is now known as saneax-_-|AFK02:17
*** simonmcc has quit IRC02:26
*** honglei has joined #puppet-openstack02:27
*** simonmcc has joined #puppet-openstack02:28
*** honglei has quit IRC02:33
*** ducttape_ has quit IRC02:46
*** ducttape_ has joined #puppet-openstack02:53
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Allow setting of object name and namespace name max length  https://review.openstack.org/35802902:59
openstackgerritMerged openstack/puppet-nova: Remove manager option  https://review.openstack.org/35841303:05
*** saneax-_-|AFK is now known as saneax03:13
*** ducttape_ has quit IRC03:16
openstackgerritayoung proposed openstack/puppet-tripleo: Optionally include mod auth mellon for apache  https://review.openstack.org/35994203:20
*** ducttape_ has joined #puppet-openstack03:22
*** ducttape_ has quit IRC03:26
*** finchd has quit IRC03:39
*** morazi has quit IRC03:58
*** saneax is now known as saneax-_-|AFK04:03
openstackgerritMichael Liu proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken  https://review.openstack.org/35198004:20
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-swift: Change swift authtoken  https://review.openstack.org/36021604:23
*** ducttape_ has joined #puppet-openstack04:23
iurygregory_Vinsh, fyi https://review.openstack.org/360216, going to sleep now =)04:26
*** iurygregory_ has quit IRC04:28
*** ducttape_ has quit IRC04:28
*** morazi has joined #puppet-openstack04:55
*** ayoung has quit IRC05:03
*** johnmilton has quit IRC05:07
*** jaosorior has joined #puppet-openstack05:10
*** ducttape_ has joined #puppet-openstack05:25
*** johnmilton has joined #puppet-openstack05:27
*** ducttape_ has quit IRC05:29
*** richm has quit IRC05:40
*** morazi has quit IRC05:46
*** simonmcc has quit IRC05:51
*** simonmcc has joined #puppet-openstack05:52
*** Sanjay has quit IRC05:56
*** skramaja has joined #puppet-openstack05:59
*** johnmilton has quit IRC06:19
*** ducttape_ has joined #puppet-openstack06:26
*** johnmilton has joined #puppet-openstack06:27
*** rcernin has joined #puppet-openstack06:28
*** ducttape_ has quit IRC06:30
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/35997306:37
openstackgerritKarthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots  https://review.openstack.org/35249406:44
*** saneax-_-|AFK is now known as saneax06:55
*** johnmilton has quit IRC06:56
*** dgurtner has joined #puppet-openstack07:01
*** tesseract- has joined #puppet-openstack07:08
*** ifarkas_afk is now known as ifarkas07:09
*** dgurtner has quit IRC07:13
*** johnmilton has joined #puppet-openstack07:20
*** ducttape_ has joined #puppet-openstack07:26
*** davidmichaelkarr has quit IRC07:28
*** AndyWojo has quit IRC07:29
*** davidmichaelkarr has joined #puppet-openstack07:30
*** saneax is now known as saneax-_-|AFK07:30
*** ducttape_ has quit IRC07:31
*** AndyWojo has joined #puppet-openstack07:32
*** shardy has joined #puppet-openstack07:47
*** jmiu has quit IRC07:51
*** jmiu has joined #puppet-openstack07:57
*** jpena|away is now known as jpena08:09
*** skramaja has quit IRC08:15
*** chem has joined #puppet-openstack08:22
*** skramaja has joined #puppet-openstack08:23
*** ducttape_ has joined #puppet-openstack08:27
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/36032308:30
*** ducttape_ has quit IRC08:32
*** dtantsur|afk is now known as dtantsur08:32
*** saneax-_-|AFK is now known as saneax08:33
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Terminate Zaqar websocket endpoint in HAProxy  https://review.openstack.org/36032908:37
*** aarefiev_ has joined #puppet-openstack08:43
*** mkarpin1 has joined #puppet-openstack08:45
*** mkarpin has quit IRC08:46
*** ddmitriev has quit IRC08:46
*** aarefiev has quit IRC08:47
openstackgerritMartin André proposed openstack/puppet-swift: Trim useless blank lines in generated config file  https://review.openstack.org/35934208:47
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #puppet-openstack08:49
*** aarefiev_ is now known as aarefiev08:50
openstackgerritJavier Peña proposed openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/36035408:57
*** derekh has joined #puppet-openstack09:00
*** davideagle has joined #puppet-openstack09:01
openstackgerritMartin André proposed openstack/puppet-tripleo: [WIP] UI profile for tripleo  https://review.openstack.org/35463009:27
*** ducttape_ has joined #puppet-openstack09:28
*** ducttape_ has quit IRC09:33
openstackgerritKarthik S proposed openstack/puppet-neutron: SR-IOV numvfs config is made persistent across reboots  https://review.openstack.org/35249409:46
*** danpawlik has joined #puppet-openstack09:52
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Update  Midonet neutron plugin to 5.x  https://review.openstack.org/36039009:56
*** gfidente|afk is now known as gfidente09:56
*** xarses_ has joined #puppet-openstack10:02
*** xarses has quit IRC10:05
*** richm has joined #puppet-openstack10:07
*** ddmitriev has joined #puppet-openstack10:13
*** chem has quit IRC10:22
*** chem has joined #puppet-openstack10:22
socialchem: ping about https://review.openstack.org/#/c/354125/ what do you think about iurygregory point to use "Defaults to ::ironic::params::variable"?10:26
*** ducttape_ has joined #puppet-openstack10:29
chemiurygregory, regarding the social review ^, do you have other examples where a variable is used for a default ?  IMHO, it's more readable in a doc to have a something fixed as a default than a variable.  WDYT ?10:29
*** ducttape_ has quit IRC10:34
dtantsuriberezovskiy|off or somebody: this breaks Ironic: https://github.com/openstack/puppet-ironic/commit/eef5b11679e1c0e96a782f51f5e0c8aa6abfb3bd11:09
dtantsurI think we should revert it and then provide a complete set of credentials11:10
dtantsuror we should make sure we fill in all other credentials that used to fall back to the removed ones... which I think is a bit too optimistic right before the release11:13
* dtantsur looks what can be fixed on ironic side11:14
*** karthiks_afk is now known as karthiks11:23
*** _degorenko|afk is now known as degorenko11:24
*** jgr has joined #puppet-openstack11:26
*** jgr has quit IRC11:26
iberezovskiy|offdtantsur, what's actually wrong? could you please provide logs and steps to reproduce the issue11:34
dtantsurso, I've filed a bug, really not sure who to proceed: https://bugs.launchpad.net/puppet-ironic/+bug/161687311:34
openstackLaunchpad bug 1616873 in puppet-ironic "Accessing keystone catalog from conductor broken after recent refactoring" [Undecided,New]11:34
*** iberezovskiy|off is now known as iberezovskiy11:34
dtantsuriberezovskiy, see the bug ^^^ you can reproduce by installing ironic in overcloud, but it will take you some time11:34
dtantsurtripleo overrides several options that probably make this not reproducible for it11:34
dtantsurbut tl;dr is: you've removed some options that are used as fall back for (still missing) new options11:35
iberezovskiyhah, interesting11:40
dtantsuryeah, it's a bit hacky, I know... but it's tricky to keep backward compatibility when migrating to a completely new set of options that is not even controlled by us11:42
iberezovskiyfrom your log, it can't find username11:46
iberezovskiybut it will be set if you pass it11:46
iberezovskiydtantsur, I need help with reproducing the issue11:48
dtantsurwell, we no longer pass it, right?11:49
dtantsurI mean, we used to pass admin_username, which is something we expect11:50
dtantsurnow it passes just username, ironic does not know to fall back to it11:50
dtantsur(this is why I call it hacky)11:50
iberezovskiyI don't see admin_username option in default ironic conf11:51
*** jaosorior has quit IRC11:52
*** jaosorior has joined #puppet-openstack11:52
dtantsurme neither, but it might be due to deprecation11:53
iberezovskiyif so, it should be marked as deprecated11:53
* dtantsur has no clue, but it clearly no longer finds the username option11:54
*** dprince has joined #puppet-openstack11:55
iberezovskiythis is how we configure ironic auth in fuel https://github.com/openstack/fuel-library/blob/master/deployment/puppet/openstack_tasks/manifests/ironic/ironic.pp#L9411:56
dtantsuriberezovskiy, yeah, but tripleo got switched to new options12:00
dtantsuriberezovskiy, https://github.com/openstack/tripleo-heat-templates/commit/b5a54bf9852ab24c8c7e0501b258b9fa26e2ad0112:00
* iberezovskiy looking12:00
dtantsuriberezovskiy, I guess this bit gets broken: https://github.com/openstack/ironic/blob/master/ironic/common/keystone.py#L95-L9812:02
dtantsurit still relies on old names12:02
*** jpena is now known as jpena|lunch12:04
iberezovskiydamn12:04
dtantsurI can't even find where these names come from Oo neither in keystonemiddleware not in keystoneauth12:05
* dtantsur adds ironic to the bug12:06
*** dgurtner has joined #puppet-openstack12:06
*** dgurtner has quit IRC12:06
*** dgurtner has joined #puppet-openstack12:06
*** ducttape_ has joined #puppet-openstack12:11
iberezovskiyhttps://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token/_auth.py#L34-L4012:12
*** degorenko has quit IRC12:12
iberezovskiyhttps://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/auth_token/__init__.py#L191-L19812:13
*** degorenko has joined #puppet-openstack12:13
*** iberezovskiy has quit IRC12:14
*** skolekonov has quit IRC12:14
*** degorenko has quit IRC12:14
*** iberezovskiy has joined #puppet-openstack12:14
*** degorenko has joined #puppet-openstack12:14
iberezovskiydtantsur, ^12:14
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/35851012:14
* dtantsur looking12:14
*** skolekonov has joined #puppet-openstack12:15
*** colonwq has quit IRC12:19
*** colonwq has joined #puppet-openstack12:20
*** shardy has quit IRC12:21
*** mfisch has quit IRC12:21
*** shardy has joined #puppet-openstack12:22
*** mwhahaha_ has joined #puppet-openstack12:23
*** vern has quit IRC12:23
*** mwhahaha has quit IRC12:23
*** mwhahaha_ is now known as mwhahaha12:24
*** vern has joined #puppet-openstack12:24
dtantsuriberezovskiy, hmm, I'm trying to switch ironic to newer options and I get "no such option in group keystone_authtoken: username"12:24
*** ansmith has joined #puppet-openstack12:24
dtantsurwut12:24
*** mfisch has joined #puppet-openstack12:25
iberezovskiyomg12:25
*** mfisch has quit IRC12:25
*** mfisch has joined #puppet-openstack12:25
*** jaosorior is now known as jaosorior_away12:25
dtantsuriberezovskiy, I suspect the easiest thing we can do on our side (while I'm figuring our ironic side) is to bring back setting admin_* stuff to the same values12:26
*** mtreinish has quit IRC12:27
iberezovskiyso I don't understand only one thing. how did it work on CI12:27
dtantsurwhich CI you mean?12:27
iberezovskiyopenstack-integration checks12:27
dtantsurdo we launch ironic instances there?12:28
*** ducttape_ has quit IRC12:28
iberezovskiywe have scenario with ironic12:28
iberezovskiyand we run tempest for it12:28
* dtantsur checks what happened there12:29
iberezovskiyhttp://logs.openstack.org/28/346028/14/gate/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/59c18ce/console.html12:29
*** mtreinish has joined #puppet-openstack12:30
iberezovskiyi don't see the sest for ironic instances12:31
iberezovskiytest*12:31
dtantsurright, we don't boot anything http://logs.openstack.org/28/346028/14/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/7dd01e0/logs/ironic/ironic-conductor.txt.gz?level=INFO12:32
dtantsurthis is how it passed12:32
iberezovskiy:(12:33
dtantsurnow, what to do...12:33
openstackgerritIvan Berezovskiy proposed openstack/puppet-ironic: Revert "Configure keystone authtoken options"  https://review.openstack.org/36049912:34
iberezovskiydtantsur, but we still need to update ironic12:35
dtantsuriberezovskiy, wait a second with the revert, I think we can do it easier12:36
iberezovskiyok12:36
EmilienMmorning12:37
*** morazi has joined #puppet-openstack12:37
iberezovskiyhi12:37
dtantsurmorning EmilienM, a lot of fun today12:37
*** iberezovskiy has quit IRC12:37
*** skolekonov has quit IRC12:37
EmilienMdtantsur: excellent !12:38
*** iberezovskiy has joined #puppet-openstack12:39
*** skolekonov has joined #puppet-openstack12:39
* iberezovskiy have some troubles with connectivity12:40
EmilienMreview.openstack.org is unreachable for me12:40
chemhi12:41
*** degorenko has quit IRC12:41
*** iberezovskiy has quit IRC12:41
dtantsurwonderful12:42
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/36050612:42
*** cdelatte has joined #puppet-openstack12:42
EmilienMdtantsur: thx for helping12:42
EmilienM(as usual)12:42
dtantsur:)12:43
*** derekh has quit IRC12:44
*** ddmitriev has quit IRC12:44
*** degorenko has joined #puppet-openstack12:44
*** degorenko_ has joined #puppet-openstack12:44
* dtantsur writes tests for his patc12:45
*** iberezovskiy has joined #puppet-openstack12:45
*** cdelatte has quit IRC12:46
*** derekh has joined #puppet-openstack12:47
*** fxpester has joined #puppet-openstack12:47
*** markvoelker has joined #puppet-openstack12:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing"  https://review.openstack.org/33981312:50
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/36050612:52
dtantsurEmilienM, iberezovskiy ^^^12:52
*** AJaeger has joined #puppet-openstack12:52
EmilienMdtantsur: looking12:53
AJaegerEmilienM, could you +W https://review.openstack.org/#/c/355001/ , please? It has two +2s already12:53
dtantsurthis is much less invasive than revert the whole thing12:53
iberezovskiydtantsur, thanks12:53
EmilienMdtantsur: commented12:53
dtantsuroops, gimme a second to fix12:54
dtantsurEmilienM, actually it's missing because it's missing in the place where I copy-pasted it from12:54
*** mgagne_ is now known as mgagne12:54
EmilienMoops12:54
dtantsurI'll fix both12:55
*** mkarpin1 has quit IRC12:55
*** mkarpin has joined #puppet-openstack12:56
*** jpena|lunch is now known as jpena12:56
*** sanjayu|AFK is now known as sanjayu_12:57
*** morazi has quit IRC12:57
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Set fallback authentication options for Ironic  https://review.openstack.org/36050613:01
dtantsurEmilienM, iberezovskiy, updated, please check ^^^13:01
openstackgerritJavier Peña proposed openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/36035413:01
EmilienMok I want to see tripleo CI13:02
iurygregorymorning13:02
EmilienMbut yeah +213:02
socialiurygregory: ping about the ironic::pxe review and options13:03
iurygregorysocial, go ahead13:03
iberezovskiyonce all CI are green, I'll approve13:04
socialiurygregory: I commented yesterday also on review. Do you really prefer if I specify variable ironic::params::var instead of what the pick function will default to?13:04
dtantsurthanks (I wonder how much OoO CI will take though...)13:05
iurygregorysocial, in my mind i think the pick should be at other classes that will use.. that's why I put 'Not sure this is the best way' hahah =) I would appreciate a review from any core about this ^^13:06
*** alee has quit IRC13:07
*** morazi has joined #puppet-openstack13:07
*** paramite has joined #puppet-openstack13:09
*** skramaja has quit IRC13:10
*** skramaja has joined #puppet-openstack13:11
*** johnmilton has quit IRC13:14
*** johnmilton has joined #puppet-openstack13:15
socialiurygregory: thanks, well if I use it in the other classes the code would look much uglyer, it actually makes it nice if I use params class fro this13:15
iurygregorysocial, humm ok o/13:20
iurygregorylet's wait for someelse before you send the patch, i can remove the -1 so people will review I think haha13:20
socialchem: ^^13:22
* dtantsur does not have enough understanding of puppet to review this aspect :)13:22
*** ducttape_ has joined #puppet-openstack13:22
chemsocial: ack, will do later, buzy right now13:24
*** Ahharu has joined #puppet-openstack13:28
AhharuI am having trouble with this patch https://review.openstack.org/#/c/359973/3 because I keep being told to do a cherry-pick from master but that's not possible since the code is not exactly the same for the 2 branches, on MASTER the command(neutron) is defined at the "neutron" top scope provider and on stable/mitaka it is defined on each type13:29
AhharuWhat should I do?13:29
*** ducttape_ has quit IRC13:33
iurygregoryAhharu, the same fix from master will not work in stable/mitaka? well this is strange because we should make patches backward compatible .-.13:33
iurygregorydegorenko, i think heat broken fuel, did you check? i saw your +2 =)13:34
Ahharunope, it will break it13:35
Ahharubecause it will say that command neutron is not found13:35
degorenkoiurygregory, no, job failed not because of your patch13:35
iurygregoryAhharu, lol O.o13:35
iurygregorydegorenko, ack, tks man :D13:35
Ahharuthe thing is that is the command neutron declaration was moved into the "neutron.rb" provider13:36
Ahharuon which other types inherit13:36
Ahharumakes sense13:36
*** fultonj has joined #puppet-openstack13:39
*** ansmith_ has joined #puppet-openstack13:42
*** jaosorior_away is now known as jaosorior13:48
AhharuI think I can do it this way..13:48
Ahharucherry pick this 8a85bb0860e334bced12c86771e4ef23c6731dc2 into stable/mitaka13:49
Ahharuand backport from master13:49
iurygregorywhat?13:49
iurygregorybackport to master? O.o13:49
*** haaj has joined #puppet-openstack13:49
Ahharufrom master13:49
*** dfisher has joined #puppet-openstack13:50
Ahharuthis change 8a85bb0860e334bced12c86771e4ef23c6731dc2 takes the command neutron to top-scope13:50
iurygregoryAhharu, sorry still sleeping hahah13:50
*** haaj has quit IRC13:50
Ahharu:D :D it's already afternoon here13:50
openstackgerritLukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::pxe class  https://review.openstack.org/35412513:51
*** dsavineau has joined #puppet-openstack13:51
*** alee has joined #puppet-openstack13:52
*** ebalduf has joined #puppet-openstack13:57
*** NachoDuck has quit IRC14:01
*** NachoDuck has joined #puppet-openstack14:04
*** xarses_ has quit IRC14:06
AJaegerEmilienM, while I have your attention on reviewing ;), could you +W https://review.openstack.org/#/c/355001/ , please? It has two +2s already14:07
iurygregorydegorenko, ^14:08
*** jaosorior has quit IRC14:08
EmilienMAJaeger: done14:08
AJaegerthanks14:09
iurygregoryEmilienM, since magnum only have release 9.0.0 and 9.1.0 the authtoken should be backward compatible?14:10
EmilienMiurygregory: no14:10
EmilienMbreak magnum14:10
iurygregoryEmilienM, ok, degorenko you're right :D14:10
EmilienMI'm not sure someone is using it14:10
EmilienMand we have no real functional testing yet on it14:11
*** AJaeger has left #puppet-openstack14:11
iurygregoryEmilienM, yeah, we have two patches for authtoken i'll abandon my patch since the other is older14:11
EmilienMok14:11
*** ducttape_ has joined #puppet-openstack14:12
*** AJaeger has joined #puppet-openstack14:19
AJaegerEmilienM: my change is stacked on top of https://review.openstack.org/#/c/355202 ;(14:20
AJaegerAny cores around to review that one, please? It's needed to pass the testsuite...14:20
openstackgerritMatt Fischer proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446414:21
EmilienMiberezovskiy, degorenko: ^ can you review it please?14:22
iurygregorymwhahaha, you around?14:23
mwhahahafor a few14:23
mwhahahawhat's up14:23
iurygregorymwhahaha, auth_strategy for new modules make sense? we don't need to include the class in the api.pp since we don't have release...14:24
iurygregorylike: ec2api, watcher, magnum...14:25
mwhahahamy concern around not having is that we won't notify the end user that they need to provide the other class for keystone enablement14:25
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/36058414:25
mwhahahaif you include it, then automatically they get an error about having to configure the keystone options14:25
mwhahahaotherwise you'll end up with services with possibly no auth by default14:26
iurygregorymwhahaha, ack ;)14:26
mwhahahatime to bus children to school, back in 3014:27
*** karimb has joined #puppet-openstack14:33
Ahharuok, so after this is backported, I can backport the neutron port fix thing from master14:36
Ahharu:)14:36
iurygregoryAhharu, after the patch is merged, yes :D14:37
Ahharusure, will wait for it14:38
*** bogdando has quit IRC14:39
*** bogdando has joined #puppet-openstack14:43
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/36060814:48
*** openstackgerrit has quit IRC14:48
degorenkoEmilienM, i'm ready to +2 it :D waiting for ci14:49
*** openstackgerrit has joined #puppet-openstack14:49
EmilienMdegorenko: thx14:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-magnum: Add itesm for keystone_authtoken using class keystone::resource::authtoken  https://review.openstack.org/35198014:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/36061014:50
EmilienMdmburmistrov: FYI  ^14:50
degorenkonot sure that Dima is person who knows about this :D14:51
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-magnum: Move magnum to authtoken  https://review.openstack.org/35198014:53
*** finchd has joined #puppet-openstack14:54
*** aviramb has joined #puppet-openstack14:54
*** saneax is now known as saneax-_-|AFK14:56
*** xarses_ has joined #puppet-openstack14:56
iurygregorydegorenko, can you review magnum? :D14:56
*** AJaeger has left #puppet-openstack14:57
*** finchd has left #puppet-openstack14:58
degorenkoiurygregory, is it fair that you updated magnum patch? :)14:59
iurygregorydegorenko, yeah because we need to finish authtoken i think XD15:00
degorenkoiurygregory, but Michael is working on that right now, last update from his side was today :)15:01
iurygregorydegorenko, oh D: i was fast sorry .-.15:01
degorenkoiurygregory, ok, don't worry :)15:01
iurygregorybut he can update if you see something wrong =)15:01
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/35851015:03
*** sanjayu_ is now known as sanjayu|AFK15:04
degorenkoiurygregory, i'm ok with patch, one nit inline, also probably we need to update commit message, to reflect real goal of this change15:04
iurygregorydegorenko, ok I'll let Michael do this =) put in the review about the commit message please =)15:05
degorenkoya15:06
*** dgurtner has quit IRC15:08
*** bnemec has quit IRC15:08
*** dprince has quit IRC15:09
*** bnemec has joined #puppet-openstack15:13
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Write restart flags to restart services only when necessary  https://review.openstack.org/35851015:17
*** furlongm_ has joined #puppet-openstack15:19
*** furlongm has quit IRC15:20
degorenkoEmilienM, iurygregory , mwhahaha how we missed that one https://github.com/openstack/puppet-nova/blob/master/manifests/keystone/authtoken.pp#L226 ? :(15:21
EmilienMoops15:22
EmilienMdegorenko: can you submit a patch?15:22
mwhahahacause 4.6.015:22
mwhahaha:(15:22
degorenkonot exactly15:22
degorenkothat's because our acceptance doesn't have include for main nova class15:23
degorenkoi don't know why :D15:23
degorenkook, i'll do patch15:23
mwhahahainteresting15:23
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Remove unneeded include for main class  https://review.openstack.org/36063215:26
degorenkoEmilienM, mwhahaha ^15:26
EmilienMdegorenko: +615:27
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/35996715:27
degorenko:D15:27
openstackgerritEmilien Macchi proposed openstack/puppet-panko: puppet-panko: Initial commit  https://review.openstack.org/36063915:30
Ahharuit is some kind of weird neutron cli thing15:30
mwhahahapuppet-panko wtf is that15:31
mwhahahadoes it takes delicious on chicken?15:31
*** derekh has quit IRC15:31
EmilienMlol15:31
Ahharuthe network name is defined by checking the first thing that is "alone" , I mean without -- or containing "=" sign15:31
EmilienMmwhahaha: https://github.com/openstack/panko15:31
EmilienMmwhahaha: a new telemetry service15:31
degorenkolol15:32
EmilienMmwhahaha: for event storage this time15:32
mwhahahaof course it is15:32
degorenkowe need more telemetry services!15:32
degorenkoah, it's just renaming15:33
EmilienMdegorenko: no, it's moving the code15:33
EmilienMmwhahaha: I actually like how telemetry split things, well it brings us more work but it help to scale the services15:33
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/35996715:33
EmilienMpradk from telemetry team is going to take over my patch15:33
degorenkoah, ok15:33
mwhahahamore things to eat ram </troll>15:34
EmilienMand will write the module15:34
EmilienMhe's already familiar with it15:34
*** derekh has joined #puppet-openstack15:34
*** pradk has joined #puppet-openstack15:36
*** xarses_ is now known as xarses15:37
*** aviramb has quit IRC15:56
*** derekh has quit IRC16:02
*** danpawlik has quit IRC16:09
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/36066016:10
EmilienMdtantsur: ^16:10
dtantsurEmilienM, I suspect the debian package name is wrong there16:12
EmilienMdtantsur: http://ubuntu-cloud.archive.canonical.com/ubuntu/pool/main/i/ironic/16:12
EmilienMit's python-ironic16:12
dtantsurno, I don't think so16:12
dtantsurhttps://packages.debian.org/experimental/python-ironic-lib16:12
iurygregorydegorenko, well the reason to include was because memcached was not evaluated16:13
dtantsurEmilienM, I think python-ironic is just python modules extracted from the main Ironic package16:13
dtantsurEmilienM, here you go http://ubuntu-cloud.archive.canonical.com/ubuntu/pool/main/p/python-ironic-lib/ :)16:13
EmilienMdtantsur: i'm not sure it's in ubuntu though16:13
EmilienMoh16:13
EmilienMyou got it16:13
EmilienMdamn !16:13
EmilienMyour the best16:13
dtantsur:)16:13
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/36066016:15
degorenkoiurygregory, then this class should be included before authtoken16:15
degorenkonot inside16:16
iurygregorydegorenko, oh sorry .-.16:16
degorenkook, no problem16:16
iurygregorysocial, EmilienM put the same comment =)16:17
*** dprince has joined #puppet-openstack16:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/36060816:21
iurygregoryEmilienM, why you depend on the same patch?16:22
iurygregory'-'16:22
iurygregoryDepends-On: I8f5b7f3d7392bfa4215fccd7defe953857cef3f616:22
iurygregoryChange-Id: I8f5b7f3d7392bfa4215fccd7defe953857cef3f616:22
*** skolekonov has quit IRC16:22
EmilienMah16:23
EmilienMI'm tired16:23
iurygregoryit happens16:23
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/36060816:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/36061016:25
*** dprince has quit IRC16:37
*** ifarkas is now known as ifarkas_afk16:42
*** rcernin has quit IRC16:45
cheminternet is slow today16:47
EmilienMchem: gerrit is slow today16:48
*** hjensas has quit IRC16:52
*** dprince has joined #puppet-openstack16:53
*** davideagle has quit IRC16:54
*** tesseract- has quit IRC16:56
*** _ducttape_ has joined #puppet-openstack16:57
*** dgurtner has joined #puppet-openstack16:57
*** ducttape_ has quit IRC17:01
*** skramaja has quit IRC17:06
*** iberezovskiy is now known as iberezovskiy|off17:07
*** skramaja has joined #puppet-openstack17:08
*** 7JTABL8FZ has joined #puppet-openstack17:10
*** ebalduf has quit IRC17:12
*** chem is now known as chem|off17:13
openstackgerritMerged openstack/puppet-swift: Add statsd configuration options  https://review.openstack.org/36003217:14
*** AJaeger has joined #puppet-openstack17:22
*** _ducttape_ has quit IRC17:25
*** AJaeger has left #puppet-openstack17:25
*** AnarchyAo has joined #puppet-openstack17:27
*** dsavineau has left #puppet-openstack17:31
*** dtantsur is now known as dtantsur|afk17:36
*** degorenko is now known as _degorenko|afk17:37
*** karimb has quit IRC17:41
*** ducttape_ has joined #puppet-openstack17:51
*** rcernin has joined #puppet-openstack17:54
*** dgurtner has quit IRC18:02
openstackgerritBen Swartzlander proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class  https://review.openstack.org/35401418:12
socialEmilienM: is there any other way how to satisfy dtantsur's requirement for having a class for global override of options of classes that are independant? Or you would just preffer to have the pick function in the classes and the classes would inherit the common one?18:23
*** paramite has quit IRC18:33
iurygregorygerrit is slow only here? .-.18:44
*** paramite has joined #puppet-openstack18:45
*** Epx998 has joined #puppet-openstack18:49
mwhahahano it's  pretty bad today18:49
*** Epx998 has left #puppet-openstack18:50
*** paramite has quit IRC18:57
openstackgerritMerged openstack/puppet-trove: Policy support for Trove  https://review.openstack.org/35994819:14
openstackgerritMerged openstack/puppet-nova: Add support for new vendordata plugins  https://review.openstack.org/35960419:17
openstackgerritMerged openstack/puppet-openstack-integration: Move all lbaas to v2  https://review.openstack.org/36035419:26
*** johnmilton has quit IRC19:32
*** foxhunt_away is now known as foxhunt7219:38
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Support ironic-lib package and orchestrate Ironic upgrade  https://review.openstack.org/36066019:43
*** foxhunt72 is now known as foxhunt_away19:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Configure authtoken with new method  https://review.openstack.org/36060819:46
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: disable TestEncryptedCinderVolumes tests"  https://review.openstack.org/36061019:46
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting to implement some performance tuning adjustments, and should return to working order momentarily.19:47
*** johnmilton has joined #puppet-openstack19:48
*** _stowa has joined #puppet-openstack19:50
*** skramaja has quit IRC19:50
*** johnmilton has quit IRC19:53
*** _stowa has quit IRC19:55
openstackgerritMerged openstack/puppet-cinder: Add support for allow_availability_zone_fallback parameter  https://review.openstack.org/35635920:02
*** dprince has quit IRC20:07
*** johnmilton has joined #puppet-openstack20:09
iurygregoryEmilienM, TripleO does not run in puppet-heat?20:10
openstackgerritBrent Eagles proposed openstack/puppet-neutron: Adding support for overlay_ip_version  https://review.openstack.org/36076620:10
EmilienMit should be20:10
EmilienMiurygregory: show me patch20:10
iurygregoryEmilienM, https://review.openstack.org/#/c/359543/20:11
*** ducttape_ has quit IRC20:11
iurygregoryauthtoken haha XD20:11
EmilienMiurygregory: gate-tripleo-ci-centos-7-nonha-multinode20:11
EmilienMit's green20:11
EmilienMonly fuel is red20:11
iurygregoryi've talked to denis it's not related20:12
EmilienMok I'll review it shortly20:13
iurygregoryEmilienM, tks take your time o/20:13
iurygregorytomorrow i'll take a closer look in all modules to fix some nits like metadata is ok, and make sure we are using auth_strategy =)20:14
*** ducttape_ has joined #puppet-openstack20:14
EmilienMexcellent20:15
EmilienMwe're getting close to release and feature freeze20:15
EmilienMwe might want to stabilize a bit our modules20:15
iurygregoryyeah, i'll make sure they are in the same shape for authtoken o/20:15
*** johnmilton has quit IRC20:15
iurygregoryif you need any reviews let me know20:16
*** alee has quit IRC20:18
EmilienMiurygregory: will do20:20
*** johnmilton has joined #puppet-openstack20:28
EmilienMiurygregory: heat reviewed20:41
*** ansmith has quit IRC20:42
iurygregoryEmilienM, ack fixing right now20:42
*** ansmith_ has quit IRC20:43
*** morazi has quit IRC20:46
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Introduce scenario004  https://review.openstack.org/33910620:59
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-heat: Move Heat to authtoken  https://review.openstack.org/35954321:08
openstackgerritMerged openstack/puppet-nova: Remove unneeded include for main class  https://review.openstack.org/36063221:18
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/35996721:27
openstackgerritAlex Ruiz Estradera proposed openstack/puppet-neutron: Fix the neutron-port resource with binding options  https://review.openstack.org/35996721:33
*** Ahharu_ has joined #puppet-openstack21:35
Ahharu_@EmilienM how should I proceed with https://review.openstack.org/#/c/360390/ ?21:35
Ahharu_code is different from master and mitaka21:36
Ahharu_in terms of Anchors, requires and stuff21:36
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Change authtoken to class  https://review.openstack.org/35679221:39
*** pradk has quit IRC21:41
*** shardy has quit IRC21:57
*** Ahharu_ has quit IRC21:59
openstackgerritMerged openstack/puppet-swift: Trim useless blank lines in generated config file  https://review.openstack.org/35934222:01
EmilienMAhharu: ah :( indeed22:03
EmilienMAhharu: please specify it in commit message then22:03
openstackgerritMerged openstack/puppet-swift: Change swift authtoken  https://review.openstack.org/36021622:07
openstackgerritMerged openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668522:07
*** chem|off has quit IRC22:16
*** chem|off has joined #puppet-openstack22:18
*** ducttape_ has quit IRC22:25
*** ducttape_ has joined #puppet-openstack22:26
*** ducttape_ has quit IRC22:30
*** ntpttr has quit IRC22:31
*** ntpttr has joined #puppet-openstack22:36
*** 7JTABL8FZ has quit IRC22:40
*** chem|off` has joined #puppet-openstack22:46
*** chem|off has quit IRC22:48
openstackgerritMerged openstack/puppet-tripleo: SR-IOV nic agent changes in line with composable roles.  https://review.openstack.org/33219122:53
*** ducttape_ has joined #puppet-openstack23:00
*** ducttape_ has quit IRC23:08
*** dfisher has quit IRC23:14
*** ebalduf has joined #puppet-openstack23:18
*** ducttape_ has joined #puppet-openstack23:23
*** chem|off` has quit IRC23:25
*** chem|off` has joined #puppet-openstack23:26
*** xarses has quit IRC23:29
*** beagles has left #puppet-openstack23:32
*** ducttape_ has quit IRC23:33
*** AnarchyAo has quit IRC23:41
*** ducttape_ has joined #puppet-openstack23:47
*** rcernin has quit IRC23:53
*** xarses has joined #puppet-openstack23:56

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