Thursday, 2015-09-10

*** markvoelker has quit IRC00:01
*** yogesh-pc has joined #puppet-openstack00:28
*** yogesh-pc_ has joined #puppet-openstack00:30
*** yogesh-pc has quit IRC00:33
*** gildub has joined #puppet-openstack00:38
*** linkedinyou has quit IRC00:39
*** xarses has quit IRC00:46
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces  https://review.openstack.org/22199100:51
*** ducttape_ has joined #puppet-openstack00:55
*** markvoelker has joined #puppet-openstack00:58
*** linkedinyou has joined #puppet-openstack00:58
*** ducttape_ has quit IRC01:01
*** markvoelker has quit IRC01:02
*** ducttape_ has joined #puppet-openstack01:02
*** tiswanso has joined #puppet-openstack01:04
*** xarses has joined #puppet-openstack01:05
*** tiswanso has quit IRC01:06
*** tiswanso has joined #puppet-openstack01:07
*** yogesh-pc_ has quit IRC01:14
*** ducttape_ has quit IRC01:16
*** ducttape_ has joined #puppet-openstack01:21
*** ducttape_ has quit IRC01:23
*** xingchao has joined #puppet-openstack01:24
gildubmichchap, would you mind reviewing the following ones? https://review.openstack.org/221119 https://review.openstack.org/221120 and https://review.openstack.org/22112001:30
michchapgildub: sure.01:30
michchapgildub: last two are the same link01:31
gildubmichchap, sorry https://review.openstack.org/22112101:31
*** linkedinyou has quit IRC01:31
gildubmichchap, hi - thanks!01:31
gildubmichchap, those are to address flaws with managing domains in keystone V301:32
gildubmichchap, and they might land in stable/kilo too, ultimately01:33
EmilienMgildub: why?01:33
EmilienMgildub: which bug are we fixing?01:33
gildubEmilienM, the domains01:33
EmilienMregarding https://review.openstack.org/22156701:33
EmilienMI don't see what is solving https://review.openstack.org/221567 in stable/kilo01:33
EmilienMmichchap: o/01:34
gildubEmilienM, I didn't mention 22156701:34
EmilienMbut I do :)01:34
EmilienMmichchap: I'm not conservative in backports, I just want to make sure we're backward compatible01:35
EmilienMjust saying01:35
gildubEmilienM, ah sorry I thought you're asking about the other ones01:35
EmilienMeverything that fix bugs needs to be backported01:35
gildubEmilienM, if  221567 is not backported then we won't be able to backport anything else01:35
EmilienMonly if it's backward compat01:35
richmwe will be able to backport without 221567, but it will be much harder01:36
richm== error/bug prone01:36
gildubEmilienM, yeah ^01:36
richmso there is risk either way01:37
EmilienMrichm: if it's harder, let's backport what we need01:37
EmilienMto make it easier (ie error prone)01:37
gildubEmilienM, understood the theory, but I think we're in a grey area here, the domain was still experimental when we moved from kilo to liberty, now unless we don't want to have better domain stuff in kilo then...01:37
gildubEmilienM, well and it's still experimental.01:38
EmilienMwe added domains support in stable/kilo. If there are some critical bugs, let's fix them01:38
EmilienMlike "not able to change the default domain id" is to me critical01:38
*** linkedinyou has joined #puppet-openstack01:38
EmilienMso I vote for backporting the domain bugfix to stable/kilo01:38
EmilienMbut not the new features01:39
gildubEmilienM, ok, btw the initial idea of 221567 was to help with v2 issues too01:39
EmilienMgildub: good, I'll +2 it01:40
EmilienMgildub: once chem will update his -101:40
richmgildub: The issue that chem found - do we need to fix that in master too?01:40
richmor was that some artifact of the cherry-pick?01:41
EmilienMa backport is a cherry pic01:41
EmilienMso do not modify it01:41
EmilienMexcept if code conflict01:42
EmilienMif something's wrong in master, let's fix master01:42
gildubrichm, we can definitely add two more cases for testing #ssl? but that can be done later, not blocker01:44
*** ilbot3 has quit IRC01:44
gildubrichm, same for using get_section instead of legacy code01:44
michchapEmilienM: yeah that's our policy since last year I think,right?01:45
*** mmoo2 has joined #puppet-openstack01:46
richmgildub: sure, that's stuff is just nice-to-have01:46
EmilienMmichchap: yeah01:46
richmgildub: but what about https://review.openstack.org/#/c/221567/2/lib/puppet/provider/keystone.rb - line 89 ?01:46
*** ilbot3 has joined #puppet-openstack01:47
*** mesamoo has quit IRC01:48
gildubrichm, yes I missed that one but same thing, not a blocker, in master later01:49
richmIt is ok because the only places where get_section is actually used, it always uses 'DEFAULT' as the group/section name.01:50
richmbut if someone tries to use get_section with a different group/section name, they will have to fix get_section first01:51
*** xingchao has quit IRC01:51
gildubrichm, oh! When I said I missed that one, I meant line 4001:52
gildubrichm, no I'm looking at line 89...01:53
gildubrichm, yes that's a bug but we're not hitting it because all the calls to get_section are made using 'DEFAULT' for group.01:55
richmright - so I'm ok with it01:55
*** mesamoo has joined #puppet-openstack01:55
gildubrichm, ok. It needs a fix for future use, I'll make a fix in master01:56
richmso you just need to get chem to +101:57
*** mmoo2 has quit IRC01:58
gildubrichm, yes thanks01:59
*** mesamoo has quit IRC02:02
gildubrichm, Do you think we've to keep both --domain and '<name>::<domain>' approaches since they are redundant?02:12
gildubrichm, would-it make sense to keep --domain only?02:13
gildubrichm, I'm thinking about future providers too, well besides the trust since we don't have a choice with this one02:14
richmgildub: having a domain parameter to keystone resources will only make it confusing02:15
richmgildub: we will not allow specifying a name without a domain in the future02:16
richmgildub: so the domain parameter will be useless02:16
gildubrichm, I know that, what I'm talking about is between using --domain or ::02:16
richmusing --domain how?02:16
richmwe have to use --domain in openstack command lines?02:17
richmOSC doesn't understand "user::domain"02:17
gildubrichm, I mean as a resource parameter (forget the --)02:17
richmgildub: Right, that's what I mean - we should get rid of the domain resource parameter02:17
richmas soon as possible02:18
gildubrichm, yes, that's what I thought02:18
gildubrichm, but when possible (unlike trust) if a domain parameter can be used why not use it, it makes things more natural02:19
gildubrichm, for instance  keystone_tenant { 'service': domain => 'domain', ... }02:20
gildubrichm, the domain being mandatory02:20
richmgildub: What will Keystone_tenant.self.instances name that resource?02:21
gildubrichm, ah yeah, forgot about that one.02:22
gildubrichm, ok, never mind then02:23
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swiftinit service provider  https://review.openstack.org/20322002:31
*** xingchao has joined #puppet-openstack02:33
*** yogesh-pc_ has joined #puppet-openstack02:35
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Providers tenant/user/user_role: domain check/name  https://review.openstack.org/21805902:40
*** llua has quit IRC02:54
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Fixes get_section for future use  https://review.openstack.org/22201702:54
*** markvoelker has joined #puppet-openstack02:55
*** llua^_^AznSmile- has joined #puppet-openstack02:57
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swiftinit service provider  https://review.openstack.org/20322002:58
Vinshgates broken again on centos03:09
Vinsh2015-09-10 03:08:37.965 | Error: Execution of '/usr/bin/yum -d 0 -e 0 -y list xinetd' returned 1: One of the configured repositories failed (RDO trunk),03:09
EmilienMVinsh: ok03:09
EmilienMI'm working on it03:09
VinshYou rock :)03:09
EmilienMVinsh: logs?03:10
Vinshhttps://jenkins06.openstack.org/job/gate-puppet-swift-puppet-beaker-rspec-dsvm-centos7/46/console03:10
EmilienMok03:10
Vinshmaybe it works through it this time...03:11
EmilienMVinsh: it's random issue03:11
Vinshfail before it was: http://logs.openstack.org/20/203220/32/check/gate-puppet-swift-puppet-beaker-rspec-upgrade-dsvm-centos7/ccf1963/console.html03:11
EmilienMsometimes rdo repo are down03:11
EmilienMI was about to change the repo03:11
EmilienMwe are going to to stop using delorean03:11
EmilienMI got an email from RDO guys, told me we should switch03:12
Vinshwhat is it you add to a review to re-trigger? is it "recheck nobug" ?03:12
Vinshah good to do then03:12
EmilienMrecheck is enough03:12
EmilienMVinsh: see https://etherpad.openstack.org/p/puppet-liberty-blocker03:12
EmilienMI have to update repos03:13
EmilienMI do it now03:13
VinshI had added a "recheck" to this https://review.openstack.org/#/c/203220/  but didn't notice the old results drop off and trigger a new check03:14
Vinshmaybe it does so without dropping existing results first03:14
EmilienMVinsh: zuul has not finished03:14
Vinshah that one tripleo job that takes forever. :)03:14
EmilienMno03:14
EmilienMit's experimental03:14
EmilienMso they don't count like others03:14
VinshOk, well I'll try the "recheck" comment again some time and see if it triggers.03:15
EmilienMyeah03:16
gildubEmilienM, were are going to get the RPMs if not from delorean?03:16
EmilienMSECRET03:16
EmilienMsorry I'm tired03:16
EmilienMhttp://cbs.centos.org/repos/cloud7-openstack-liberty-testing/x86_64/os/03:16
EmilienMhttp://cbs.centos.org/repos/cloud7-openstack-common-testing/x86_64/os/03:16
gildubEmilienM, no worries mate!03:17
EmilienMthe "new repos" lol03:17
VinshYeah, sleep time.  TTYL.03:17
EmilienMVinsh: o/03:17
gildubEmilienM, cool03:17
EmilienMI might have missed something03:26
EmilienMhttp://trunk.rdoproject.org/centos7/current/03:26
EmilienMit's empty03:26
gildubInteresting...03:27
EmilienMoh I got it03:30
EmilienMso we still need delorean03:30
EmilienMbut we can drop epel & rdo kilo03:30
gildubEmilienM, looks like something went wrong during the build in  http://trunk.rdoproject.org/centos7/current/03:31
gildubEmilienM, not following you, what about all the epel dependent packages then. And what's the point of cbs.centos.org...liberty-testing if we need delorean which contains the lastest builds anyway?03:33
EmilienMI don't know03:33
EmilienMI get an emal from alan03:34
EmilienMapevec03:34
gildubAh03:34
gildubEmilienM, Would it be possible to forward it to the list?03:35
EmilienMgildub: yeah03:35
gildubEmilienM, great, thanks.03:35
EmilienMgildub: done03:36
*** tiswanso has quit IRC03:36
*** yogesh-pc_ has quit IRC03:37
*** richm has quit IRC03:40
gildubEmilienM, I can see it, merci!03:40
EmilienM:)03:41
gildubThis one should actually be in RDO list too03:42
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202503:43
EmilienMlet' see if it works03:43
EmilienMhttp://lists.openstack.org/pipermail/openstack-dev/2015-September/074040.html03:48
EmilienMit's the most hilarious mail I've never read.03:48
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202503:53
EmilienMgildub: look http://trunk.rdoproject.org/centos7/current/rpmbuild.log04:03
EmilienMit seems like python-oslo-policy can't be installed04:04
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: DNM - CI test  https://review.openstack.org/22202804:05
EmilienMI'm afraid our CI is broken04:05
EmilienMgood night04:11
gildubEmilienM, yeah I saw that :( - Bonne nuit04:16
*** mflobo has quit IRC04:20
*** mflobo has joined #puppet-openstack04:32
*** xingchao has quit IRC04:35
*** markvoelker has quit IRC04:55
*** stamak has joined #puppet-openstack05:31
*** chem has quit IRC05:44
*** zhangjn has joined #puppet-openstack05:51
*** zhangjn has quit IRC05:51
*** zhangjn has joined #puppet-openstack05:52
*** stamak has quit IRC05:53
*** jerrygb has joined #puppet-openstack05:53
*** markvoelker has joined #puppet-openstack05:56
*** markvoelker has quit IRC06:01
*** jerrygb has quit IRC06:19
*** cgfbee has quit IRC06:27
*** cgfbee has joined #puppet-openstack06:29
*** arnaud_orange has joined #puppet-openstack06:40
*** jtomasek has joined #puppet-openstack06:50
*** Oszkar has joined #puppet-openstack07:01
openstackgerritGilles Dubreuil proposed openstack/puppet-openstack-specs: Adds deprecation for using a name without a domain  https://review.openstack.org/21912707:11
*** gildub has quit IRC07:11
*** Oszkar has quit IRC07:18
*** chandankumar has joined #puppet-openstack07:23
*** jpena has joined #puppet-openstack07:32
*** stamak has joined #puppet-openstack07:35
*** arnaud_orange has quit IRC07:37
*** sac has quit IRC07:47
*** arnaud_orange has joined #puppet-openstack07:48
*** markvoelker has joined #puppet-openstack07:57
*** bapalm has quit IRC07:58
*** bapalm has joined #puppet-openstack08:00
*** jistr has joined #puppet-openstack08:01
*** markvoelker has quit IRC08:02
*** gfidente is now known as gfidente|afk08:11
*** jerrygb has joined #puppet-openstack08:19
*** Oszkar has joined #puppet-openstack08:23
*** jerrygb has quit IRC08:24
*** IBerezovskiy has joined #puppet-openstack08:27
*** xingchao has joined #puppet-openstack08:40
*** Oszkar has quit IRC08:41
*** karume has joined #puppet-openstack08:45
openstackgerritLukas Bezdicka proposed openstack/puppet-trove: Remove trove ubuntu package hack  https://review.openstack.org/22127308:47
*** derekh has joined #puppet-openstack08:51
*** karume has quit IRC08:54
*** karume has joined #puppet-openstack08:54
openstackgerritGael Chamoulaud proposed openstack/puppet-designate: Fix rspec 3.x syntax  https://review.openstack.org/22207708:56
*** markvoelker has joined #puppet-openstack08:58
*** markvoelker has quit IRC09:02
openstackgerritGael Chamoulaud proposed openstack/puppet-glance: Fix rspec 3.x syntax  https://review.openstack.org/22208109:03
*** chem has joined #puppet-openstack09:08
*** Oszkar has joined #puppet-openstack09:09
*** jerrygb has joined #puppet-openstack09:20
*** paramite has joined #puppet-openstack09:23
*** jerrygb has quit IRC09:26
*** zhangjn has quit IRC09:56
openstackgerritGael Chamoulaud proposed openstack/puppet-horizon: Remove duplicate key for ':servername'  https://review.openstack.org/22210010:01
*** Oszkar has quit IRC10:04
openstackgerritGael Chamoulaud proposed openstack/puppet-ironic: Fix rspec 3.x syntax  https://review.openstack.org/22210310:06
*** Oszkar has joined #puppet-openstack10:10
*** Oszkar1 has joined #puppet-openstack10:13
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087710:13
openstackgerritGael Chamoulaud proposed openstack/puppet-manila: Fix rspec 3.x syntax  https://review.openstack.org/22210910:15
*** Oszkar has quit IRC10:15
openstackgerritGael Chamoulaud proposed openstack/puppet-monasca: Fix rspec 3.x syntax  https://review.openstack.org/22211210:19
openstackgerritMartin Mágr proposed openstack/puppet-nova: Rename default endpoint names  https://review.openstack.org/22212010:27
*** cgfbee has quit IRC10:29
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Fix rspec3 syntax  https://review.openstack.org/22212310:33
openstackgerritGael Chamoulaud proposed openstack/puppet-neutron: Fix rspec 3.x syntax  https://review.openstack.org/22212310:34
*** arnaud_orange has quit IRC10:41
*** cgfbee has joined #puppet-openstack10:42
openstackgerritGael Chamoulaud proposed openstack/puppet-swift: Fix rspec 3.x syntax  https://review.openstack.org/22212810:44
*** markvoelker has joined #puppet-openstack10:59
*** karume has quit IRC11:02
*** markvoelker has quit IRC11:04
openstackgerritLukas Bezdicka proposed openstack/puppet-nova: Remove installation of pm-utils  https://review.openstack.org/22214111:12
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087711:15
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Revert "Fixed /etc/sahara handling"  https://review.openstack.org/22214211:20
*** jerrygb has joined #puppet-openstack11:21
*** jerrygb has quit IRC11:26
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Revert "Fix Sahara installation for Ubuntu"  https://review.openstack.org/22214411:26
*** TC01 has quit IRC11:26
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Revert "Fix Sahara installation for Ubuntu"  https://review.openstack.org/22214411:27
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Revert "Fix Sahara installation for Ubuntu"  https://review.openstack.org/22214411:28
*** Oszkar1 has quit IRC11:29
*** TC01 has joined #puppet-openstack11:31
*** yogesh-pc_ has joined #puppet-openstack11:32
*** morazi has joined #puppet-openstack11:32
EmilienMgood morning folks11:35
degorenkoo/ hi11:35
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Remove POSIX users, groups, and file modes.  https://review.openstack.org/22214611:36
*** arnaud_orange has joined #puppet-openstack11:39
*** yogesh-pc_ has quit IRC11:40
*** jpena is now known as jpena|lunch11:41
*** xingchao has quit IRC11:43
*** tiswanso has joined #puppet-openstack11:50
*** xingchao has joined #puppet-openstack11:50
openstackgerritMerged openstack/puppet-keystone: domain name from id lookups return empty  https://review.openstack.org/22111911:51
*** markvoelker has joined #puppet-openstack11:55
*** xingchao has quit IRC11:56
*** jayg|g0n3 is now known as jayg11:57
*** karume has joined #puppet-openstack11:59
*** xingchao has joined #puppet-openstack12:02
*** delattec has joined #puppet-openstack12:02
*** jistr is now known as jistr|mtg12:03
*** leanderthal has joined #puppet-openstack12:03
*** karume has quit IRC12:04
*** Oszkar has joined #puppet-openstack12:08
*** Oszkar1 has joined #puppet-openstack12:09
*** krain_ has joined #puppet-openstack12:10
*** Oszkar has quit IRC12:12
*** zhangjn has joined #puppet-openstack12:12
*** zhangjn has quit IRC12:13
*** zhangjn has joined #puppet-openstack12:13
*** tiswanso has quit IRC12:14
*** zhangjn has quit IRC12:16
*** dprince has joined #puppet-openstack12:19
*** dprince has quit IRC12:20
*** dprince has joined #puppet-openstack12:20
*** ducttape_ has joined #puppet-openstack12:23
*** xingchao has quit IRC12:30
*** bklei has joined #puppet-openstack12:31
*** karume has joined #puppet-openstack12:32
*** bklei has quit IRC12:35
*** gfidente has joined #puppet-openstack12:37
*** ducttape_ has quit IRC12:40
*** bklei has joined #puppet-openstack12:40
*** jerrygb has joined #puppet-openstack12:40
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202512:40
*** bklei_ has joined #puppet-openstack12:43
*** bklei_ has quit IRC12:45
*** jerrygb has quit IRC12:45
*** bklei_ has joined #puppet-openstack12:45
*** bklei has quit IRC12:46
*** jpena|lunch is now known as jpena12:47
*** jistr|mtg is now known as jistr12:52
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202512:57
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202512:58
openstackgerritMerged openstack/puppet-keystone: able to use wrong value for default_domain_id  https://review.openstack.org/22112012:59
*** richm has joined #puppet-openstack13:00
openstackgerritMerged openstack/puppet-keystone: able to use wrong resource name  https://review.openstack.org/22112113:02
*** karume has quit IRC13:04
*** karume has joined #puppet-openstack13:05
*** bklei has joined #puppet-openstack13:06
*** bklei has quit IRC13:07
*** bklei_ has quit IRC13:10
VinshStarting CI... like an old lawnmower ;)13:10
*** leanderthal has quit IRC13:15
*** krain_ has quit IRC13:18
*** pradk has joined #puppet-openstack13:19
*** tiswanso has joined #puppet-openstack13:19
*** chandankumar has quit IRC13:26
EmilienMVinsh: today I'm working on new repos13:36
EmilienMsee https://etherpad.openstack.org/p/puppet-liberty-blocker13:36
Vinshcbs +113:37
*** jerrygb has joined #puppet-openstack13:41
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202513:41
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Revert "Fixed /etc/sahara handling"  https://review.openstack.org/22214213:42
*** zhangjn has joined #puppet-openstack13:43
*** zhangjn has quit IRC13:43
*** jerrygb has quit IRC13:45
openstackgerritJavier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service  https://review.openstack.org/21154913:52
*** xingchao has joined #puppet-openstack13:53
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swiftinit service provider  https://review.openstack.org/20322013:55
*** paramite has quit IRC13:55
*** dfisher has joined #puppet-openstack13:56
*** zhangjn has joined #puppet-openstack13:58
*** paramite has joined #puppet-openstack14:00
*** tiswanso has quit IRC14:00
*** ducttape_ has joined #puppet-openstack14:01
*** tiswanso has joined #puppet-openstack14:02
openstackgerritMerged openstack/puppet-designate: Fix rspec 3.x syntax  https://review.openstack.org/22207714:07
degorenkospredzy, hey o/ i want ask you about https://review.openstack.org/#/c/219275/ and https://review.openstack.org/#/c/221005/ Should i rebase on your patch? Or until we didn't receive any positive feedback for your patch, i can use old style? :)14:13
openstackgerritMerged openstack/puppet-glance: Fix rspec 3.x syntax  https://review.openstack.org/22208114:14
*** jerrygb has joined #puppet-openstack14:15
spredzydegorenko, I'd say yes if you don't mind :) it already got a +2 hopefully it won't last before getting merged14:15
openstackgerritMerged openstack/puppet-monasca: Fix rspec 3.x syntax  https://review.openstack.org/22211214:16
degorenkospredzy, i must upset you - your patch has merge conflict14:18
openstackgerritMerged openstack/puppet-ironic: Fix rspec 3.x syntax  https://review.openstack.org/22210314:18
*** chandankumar has joined #puppet-openstack14:18
EmilienMdegorenko: I never saw spredzy upset :P14:18
EmilienMI tried... :)14:18
spredzydegorenko, hahah no worries, it happens quite often lately across projects :)14:18
spredzyEmilienM, ahaha :D14:18
degorenkoEmilienM, spredzy :D14:19
spredzydegorenko, will rebase later today :)14:19
openstackgerritMerged openstack/puppet-manila: Fix rspec 3.x syntax  https://review.openstack.org/22210914:19
degorenkospredzy, so, i will wait until your patch will be merged, and then i'll update mine :)14:19
spredzydegorenko, deal ;)14:20
openstackgerritMerged openstack/puppet-neutron: Fix rspec 3.x syntax  https://review.openstack.org/22212314:25
*** IBerezovskiy has quit IRC14:25
*** degorenko has quit IRC14:26
openstackgerritMerged openstack/puppet-swift: Fix rspec 3.x syntax  https://review.openstack.org/22212814:27
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202514:34
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202514:39
*** guimaluf has joined #puppet-openstack14:44
*** zhangjn has quit IRC14:52
*** IBerezovskiy has joined #puppet-openstack14:53
*** degorenko has joined #puppet-openstack14:53
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces  https://review.openstack.org/22199114:58
*** paramite has quit IRC15:00
*** paramite has joined #puppet-openstack15:12
*** yogesh-pc has joined #puppet-openstack15:14
*** jistr is now known as jistr|mtg15:14
*** xarses has quit IRC15:16
yogesh-pcI would like to replicate the an openstack environment using 2 VM: 1 Compute and 1 Controller Node. If I am successful I will be implementing the same on actual HW nodes. Could someone let me know how I can go about it.15:19
*** jistr|mtg is now known as jistr15:26
*** IBerezovskiy has quit IRC15:46
*** degorenko has quit IRC15:46
*** tiswanso has quit IRC15:48
*** yogesh-pc_ has joined #puppet-openstack15:48
*** tiswanso has joined #puppet-openstack15:49
openstackgerritEmilien Macchi proposed openstack/puppet-nova: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22202515:49
*** yogesh-pc has quit IRC15:51
*** IBerezovskiy has joined #puppet-openstack15:52
*** stamak has quit IRC15:54
*** Fin1te has joined #puppet-openstack15:55
*** xarses has joined #puppet-openstack15:56
*** angdraug has joined #puppet-openstack15:58
*** degorenko has joined #puppet-openstack16:09
*** paramite is now known as paramite|afk16:13
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty  https://review.openstack.org/22226516:15
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Add new parameters for Sahara  https://review.openstack.org/22009016:15
*** paramite|afk is now known as paramite16:15
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22226616:15
openstackgerritEmilien Macchi proposed openstack/puppet-designate: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22226916:16
openstackgerritEmilien Macchi proposed openstack/puppet-glance: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227016:16
openstackgerritEmilien Macchi proposed openstack/puppet-heat: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227116:16
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: acceptance: bump to Liberty  https://review.openstack.org/22227216:16
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty  https://review.openstack.org/22227316:16
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227416:16
openstackgerritEmilien Macchi proposed openstack/puppet-manila: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227516:17
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227616:17
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227716:17
openstackgerritEmilien Macchi proposed openstack/puppet-swift: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227816:17
openstackgerritEmilien Macchi proposed openstack/puppet-trove: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227916:18
*** chem has quit IRC16:28
*** chem has joined #puppet-openstack16:30
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Remove POSIX file modes.  https://review.openstack.org/22214616:31
*** chandankumar has quit IRC16:38
*** jistr has quit IRC16:40
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: wsgi/ubuntu: do not manage keystone.py file  https://review.openstack.org/22228616:41
*** Fin1te has quit IRC16:45
*** LimorStotland has quit IRC16:48
*** jpena has quit IRC16:56
*** karume has quit IRC16:56
*** Oszkar1 has quit IRC16:57
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Manage swift with swiftinit service provider  https://review.openstack.org/20322016:57
mwhahahaEmilienM: https://review.openstack.org/#/c/219793/ i had already done that keystone fix :D16:58
EmilienMoh nice16:59
EmilienMbut it fails16:59
EmilienMno16:59
EmilienMit fails in integ err16:59
EmilienMmwhahaha: sorry, for dup, abandoning mine right now16:59
mwhahahait's all good16:59
EmilienMnice catch :P17:00
*** paramite has quit IRC17:05
*** derekh has quit IRC17:06
*** IBerezovskiy has quit IRC17:08
*** dtalmage_ has joined #puppet-openstack17:20
*** chandankumar has joined #puppet-openstack17:45
richmDoes it matter if the output of `puppet resource` does not match what's in a manifest?17:50
richmI think not17:50
richmI would like to not have to support that they match17:50
richmThe issue, once again, is the '::domain' in the resource name17:50
richmWhat if someone decides they don't want to do that, and just create a resource like keystone_tenant { 'projectname': domain => 'domain1' } ?17:51
richmbecause, why not?  It is completely unambiguous17:52
richmdoes it matter if `puppet resource` outputs keystone_tenant { 'projectname::domain1': domain => 'domain1' } ?17:52
richmI have two problems 1) I don't know how people actually use this stuff 2) Whichever way we do it, it will piss off someone17:53
*** angdraug has quit IRC17:53
EmilienMrichm: I guess people will start complaining or giving feedback if something's wrong17:54
richmFor example, if we require that the name is always "name::domain", do we even need a domain property for resources?  Is that even useful anymore?17:56
richmIf I always have to say keystone_tenant { 'projectname::domain1': ...}17:56
richmWhat is the domain property useful for?17:57
richmWhen I did the initial design, I was thinking that Puppet has facilities to easily change or provide default values for properties specified as parameters to resource or class declarations17:58
richmFor example, node based defaults, hiera integration, etc.17:58
richmBut based on the limited feedback I have had, it seems that people who deploy this stuff don't really care, because the names are parameterized too, so it is just as easy to generate a name with a domain or without17:59
richmso if we are going to require 'name::domain' everywhere, it is just as easy for deployers to do that as provide domain parameters18:00
richmwhich means we don't need the domain properties18:00
richmwhich means we should get rid of them asap18:00
richmwhich means officially deprecating them from a lot of places18:01
richmat the same time as we begin the deprecation process for names without '::domain'18:01
richmand just wait until you see how ugly this is going to make keystone_trust resource naming . . .18:02
richmor specifying a keystone_group with a lot of users . . .18:03
imcsk8richm: from my point of view the domain should be explicitly set18:05
EmilienMrichm: I suggest to ask operators mailing list18:05
EmilienMand openstack-dev/puppet18:05
EmilienMso we get attention from people on the field18:05
richmimcsk8: explicitly set in the name?18:06
*** kindjal has joined #puppet-openstack18:07
richmimcsk8: for example, how would this affect packstack, if you have to change everywhere a keystone resource is passed in, to make sure it is passed in with "::domain" added?18:07
kindjalSetting up keystone with puppet-keystone (kilo) yeilds this error, “openstack service list: error: unrecognized arguments: --long”...18:07
imcsk8richm: as parameter: domain  => 'domain1'18:07
kindjalappears that —long is hard coded, and I think only supported by openstackclient with API version 2… and not version 3… but I can’t figure out how to specify which API version to use.18:08
kindjalMy environment is empty (bootstrapping)18:08
kindjalanyone else see this?18:08
*** admin0 has joined #puppet-openstack18:08
imcsk8richm: if we have to do that in packstack we could concatenate the name in the python code18:09
richmimcsk8: packstack relies on using the default names for users and tenants in many places - there isn't even a way in packstack to specify a name for some users and tenants18:10
imcsk8richm: exactly we would have to change that. we are actually going to make packstack use keystone v318:11
imcsk8richm: so we'll have to make a lot of changes18:11
*** yogesh-pc_ has quit IRC18:13
*** aimon__ has quit IRC18:13
richmimcsk8: many changes either way - add 'domain', 'user_domain', 'project_domain', etc. parameters in many, many places, or change the way names are created18:15
*** severion has quit IRC18:15
imcsk8richm: yes it's gonna be fun and we'll be asking you a lot of questions ;)18:15
richmIf you just have to change the names, then you don't have to add domain parameters to many, many classes and resources18:15
*** severion has joined #puppet-openstack18:15
richmhowever, you have to make sure that every class understands what 'name::domain' means18:16
richmall of the puppet-keystone code will18:16
richmbut who knows where that name will be used throughout all of the puppet modules?18:16
richmfor example - lets say I have an admin user - let's say this user exists in corp LDAP server, so I create a domain 'admin_domain' for this user18:18
richmso I pass in 'admin::admin_domain' as the user name in all of my puppet classes/resources18:19
richmincluding nagios, let's say18:19
richmnow, nagios isn't technically an openstack project and technically doesn't have to know about Keystone v3 auth and domain scoped tokens18:20
richmnagios only knows about the corp LDAP and can use 'admin' without a domain.  Who is going to be responsible for stripping off the '::admin_domain' part in order to pass to the puppet nagios classes/resources?18:20
richmAnother question: In puppet, is it the 'name' property that must be unique, or is it the 'title'?18:22
richmfor example, is it legal to have both keystone_tenant { 'one tenant': name => 'tenant', domain => 'domain1'} and keystone_tenant { 'another tenant': name => 'tenant', domain => 'domain2'}18:23
*** Fin1te has joined #puppet-openstack18:25
*** xingchao has quit IRC18:28
VinshDang, zuul sure is backed up today.18:30
*** iurygregory has quit IRC18:30
*** mflobo has quit IRC18:31
*** linkedinyou has quit IRC18:31
*** linkedinyou has joined #puppet-openstack18:32
kindjalAnyone have a sec to help me run puppet-keystone? I get one error if I use API v2.0 and another error if I try API v3… not sure what it wants.18:34
kindjalv3 doesn’t like —long, and v2 doesn’t like —format18:34
*** iurygregory has joined #puppet-openstack18:34
*** mflobo has joined #puppet-openstack18:35
openstackgerritMerged openstack/puppet-keystone: Use Ubuntu provided wsgi.py for keystone.wsgi  https://review.openstack.org/21979318:41
*** chandankumar has quit IRC18:41
imcsk8richm:  title has to be unique https://docs.puppetlabs.com/puppet/latest/reference/lang_defined_types.html#resource-uniqueness18:42
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Identity Provider  https://review.openstack.org/20805418:43
*** EmilienM sets mode: +o spredzy18:45
*** EmilienM sets mode: +o mfisch18:45
*** EmilienM sets mode: +o clayton18:45
*** EmilienM sets mode: +o crinkle18:46
*** EmilienM sets mode: +o dprince18:46
*** EmilienM sets mode: +o mgagne18:46
*** EmilienM sets mode: +o bodepd18:46
*** aimon has joined #puppet-openstack18:52
richmkindjal: I'm not sure what you mean by "use API 2.0" and "use API 3" - the puppet-keystone keystone_service provider is hardcoded to use API 318:53
*** angdraug has joined #puppet-openstack18:55
kindjalrichm: Execution of '/usr/bin/openstack service list --quiet --format csv --long' returned 218:56
kindjalopenstack service list: error: unrecognized arguments: --long18:57
kindjalthat happens with an empty environment… playing around on the command line seems to reveal that the —long option is only available in API v2 in openstackclient18:57
richmkindjal: /usr/bin/openstack --os-identity-api-version 3 service list --quiet --format csv --long18:57
kindjalERROR: cliff.app You must provide a username via either --os-username or env[OS_USERNAME]18:58
richmyou have to tell openstack which identity api version you are using - the arguments are different depending on the version18:58
richmkindjal: env|grep ^OS_18:58
iurygregoryEmilienM, when you have time can you take a look at the acceptance in https://review.openstack.org/#/c/208054/ ^^18:58
kindjalrichm: No results18:59
kindjalnothing with OS_18:59
richmkindjal: did you expect /usr/bin/openstack service list --quiet --format csv --long to return a result with no authentication provided?18:59
richmkindjal: or were you just looking for it to provide a usage error message?18:59
kindjalwell, I’ve tried with OS_TOKEN and user creds18:59
kindjaland it fails for different reasons...19:00
kindjalthen I found this: https://bugs.launchpad.net/puppet-keystone/+bug/148801819:00
openstackLaunchpad bug 1488018 in puppet-keystone "Does not work if v3 API variables are set in the environment" [Undecided,Incomplete]19:00
kindjalwithin there is a comment, “To be able to bootstrap Keystone installation properly, I would not recommend to have an openrc or any openstack variables preset."19:00
kindjalI’m expecting it to read auth_token from /etc/keystone/keystone.conf, I think.19:00
richmkindjal: no, it won't do that19:00
*** derekh has joined #puppet-openstack19:00
richmyou have to specify OS_TOKEN= or openstack --os-token "value"19:01
kindjaland OS_URL too then?19:01
richmyes19:01
kindjalbut not OS_IDENTITY_API_VERSION?19:02
*** ducttape_ has quit IRC19:02
richm# OS_IDENTITY_API_VERSION=3 OS_TOKEN="value" OS_URL=http://localhost:35357/v3 openstack service list --long19:02
kindjalwith those set:19:02
kindjalopenstack service list: error: unrecognized arguments: --long19:03
richmkindjal: openstack --version19:03
kindjalopenstack 0.3.019:03
kindjalhmm19:03
kindjalpython-openstackclient | 1.0.3-0ubuntu1~cloud0 | http://ubuntu-cloud.archive.canonical.com/ubuntu/ trusty-updates/kilo/main amd64 Packages19:03
kindjalnot sure how I got the old version...19:03
kindjalapt-cache policy shows 1.0.3 available...19:04
richmyeah - 0.3.0 is not going to work for many reasons, this being only one of them19:04
kindjalpython-openstackclient is already the newest version.19:04
kindjaldespite 1.0.3 being a candidate19:04
kindjalhmm19:04
kindjalok19:04
kindjalthanks!19:04
*** imcsk8 is now known as imcsk8|afk19:06
*** ducttape_ has joined #puppet-openstack19:12
*** Fin1te has quit IRC19:20
*** Fin1te has joined #puppet-openstack19:26
*** xingchao has joined #puppet-openstack19:29
*** derekh has quit IRC19:29
*** xingchao has quit IRC19:33
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty  https://review.openstack.org/22226519:47
openstackgerritEmilien Macchi proposed openstack/puppet-heat: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227119:49
*** yogesh-pc has joined #puppet-openstack19:51
kindjalrichm: I got openstackclient 1.0.3 installed, and there’s still an error...19:54
kindjalwith OS_ variables set...19:54
kindjal /usr/bin/openstack domain list --quiet --format csv19:54
kindjalERROR: openstack Invalid command 'domain list --format'19:54
*** jerrygb has quit IRC19:58
kindjalif I set API v3 via OS_ then I get: openstack endpoint list: error: unrecognized arguments: --long20:00
kindjalif I unset all OS_ then I think it’s working!20:01
*** linkedinyou|2 has joined #puppet-openstack20:05
*** linkedinyou|2 has quit IRC20:06
*** linkedinyou has quit IRC20:07
*** kindjal has quit IRC20:08
iurygregoryhey richm, when configure keystone with mellon , i need to modify another file beyond keystone.conf and keystone-apache.conf (apache directory) ?20:15
openstackgerritMerged openstack/puppet-neutron: Configure Big Switch ML2 plugin  https://review.openstack.org/21468620:16
*** yogesh-pc has quit IRC20:18
*** jayg is now known as jayg|g0n320:20
*** delattec has quit IRC20:20
richmiurygregory: looking20:21
iurygregoryok20:22
richmiurygregory: not directly - mellon provides a script /usr/libexec/mod_auth_mellon/mellon_create_metadata.sh which creates the SP metadata file, and the cert and key files20:24
richmhttps://github.com/richm/rdo-vm-factory/blob/master/rdo-federation-setup/vm-post-cloud-init-rdo.sh#L12920:24
richmthen it uses those files in the keystone-apache.conf20:25
richmhttps://github.com/richm/rdo-vm-factory/blob/master/rdo-federation-setup/vm-post-cloud-init-rdo.sh#L12920:25
iurygregoryhummm, for shibboleth we need to update the shibolleth2.xml file and attribute-map.xml20:26
*** gfidente has quit IRC20:30
*** jerrygb has joined #puppet-openstack20:30
*** yogesh-pc has joined #puppet-openstack20:33
*** aimon has quit IRC20:34
richmiurygregory: ok - that's one of those things that will be different in each saml implementation20:36
iurygregoryyeah, i'm just wondering which puppet-module can i use to modify shibboleth file, since puppet-keystone will only change keystone conf files20:38
*** dtalmage_ has quit IRC20:43
*** imcsk8|afk is now known as imcsk820:46
richmiurygregory: hmm - don't know - did web search find anything?20:49
richmiurygregory: you might be the first :P20:49
iurygregoryrichm, i've found this (https://github.com/aethylred/puppet-shibboleth) XD, so the user who wants keystone as SP using shibboleth will need to use keystone::federation::saml::shibboleth to configure the keystone and use shibboleth from this repo..20:54
iurygregoryto apply changes to shibboleth files20:55
richmiurygregory: ok - not sure how puppet-shibboleth should be handled - it's not really a puppet-openstack module - sounds like something that should be on puppet forge20:56
EmilienMclayton: where do you run containers? on a VM?20:57
EmilienMdprince ^20:57
claytonon baremetal20:57
claytonfor deploying openstack itself20:57
dprinceclayton: where are your Dockerfiles?20:57
*** tiswanso has quit IRC20:57
dprinceclayton: for say... openstack service containers?20:57
claytonright, we run heat and designate in containers right now20:57
iurygregorythanks richm20:58
EmilienMclayton: have you investigated magnum?20:58
*** rhallisey has joined #puppet-openstack20:58
EmilienMor using Puppet to manage containers is good for you?20:58
*** Slower has joined #puppet-openstack20:58
claytonI'm guessing magnum wouldn't make sense unless we were doing tripleo20:59
EmilienMlol20:59
claytonbut no, I haven't looked into it hardly at all20:59
EmilienMit's funny because I'm proposing to use Magnum in TripleO20:59
EmilienMbut until now, we don't think it's the right approach for us20:59
claytonThe little I've read, Magnum sounds interesting21:00
EmilienMclayton: how do you manage configuration files with containers?21:00
claytonwe mount /etc/<service> and /var/log/<service> into the container for now21:01
claytonthe former being a read only mount21:01
EmilienMclayton: so for you, it's not possible to run 2 heat-api containers if they have different configurations21:01
claytonno, I still don't really understand why anyone would want to do that21:02
EmilienMclayton: I'm was just wondering21:02
*** gfidente has joined #puppet-openstack21:02
EmilienMdprince, rhallisey, Slower: this approach is interested, we might consider it too21:02
EmilienMinteresting*21:03
claytonthe glue code we use between the openstack puppet modules and our containers is here - https://github.com/twc-openstack/puppet-os_docker21:03
dprinceEmilienM: that is very similar to what we are doing21:03
dprinceEmilienM: we create our config files w/ puppet (in an /etc tree)21:03
claytonour dockerfiles aren't public right now, but they're not anything fancy.  we basically just build a venv inside the container with a frozen requirementx.txt21:03
EmilienMdprince: puppet-tripleo is tricking a lot of stuff though21:04
EmilienMwe might need to get rid of this21:04
rhalliseysounds a lot like what we do21:04
claytonWe've looked at kolla, but right now there are some features missing that we need (like being able to specify a frozen requirements.txt)21:04
dprinceEmilienM: I'm open to ideas21:04
rhalliseyclayton, if you need anything implemented make a bp21:04
rhalliseyand I'll make it happen21:05
EmilienMAlleluia21:05
rhalliseyfor kolla that is21:05
claytonI know, I talked to sdake about it already, I just don't have time right now21:05
rhalliseyclayton, kk21:05
dprinceEmilienM, clayton : this rhallisey work on the external config interface (that we plan on using with the initial TripleO work) https://review.openstack.org/#/c/216874/21:06
EmilienMdprince: we could investigate the 'tag' thing21:06
dprinceEmilienM: by all means, yes :)21:06
claytonand honestly, I think kolla looks like a good project, but it has a lot of flexibility we don't need right now, so switching from our home grown stuff isnt' very high priority.21:06
dprinceEmilienM: I planned on doing it to get you off my case :)21:06
EmilienMlol21:06
dprinceEmilienM: but if you beat me to it I wouldn't be unhappy21:06
EmilienMahah21:07
EmilienMoh dear21:07
EmilienMclayton: during the midcycle, could you push more patches for tags?21:07
clayton@EmilienM: that's the plan, given time21:08
EmilienMfor now, dprince is noop'ing the resources21:09
EmilienMsee https://review.openstack.org/#/c/209030/21:09
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Fixes get_section for future use  https://review.openstack.org/22201721:09
EmilienMdprince: how do you create DB then?21:10
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Keystone V3 support with domain checking  https://review.openstack.org/21804421:10
dprinceEmilienM: the nooping is fine grained21:10
dprinceEmilienM: you could simply allow Exec resources21:10
dprinceEmilienM: but for the compute role we don't run DB syncs21:10
dprinceEmilienM: so we are good to go without Exec's21:10
rhalliseydprince, kolla as a build in flag to run db_sync21:11
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Providers tenant/user/user_role: domain check/name  https://review.openstack.org/21805921:11
rhalliseyI think it's bootstrap or something21:11
dprincerhallisey: right, and we haven't decided where/what will run the db_syncs yet21:11
EmilienMdprince: I'm more interested by controllers, more challenging21:11
rhalliseyright21:11
*** admin0 has quit IRC21:11
dprinceEmilienM: agree21:11
EmilienMand well, do roles still mean something?21:11
EmilienMsince we are going to run containers21:11
*** gildub has joined #puppet-openstack21:12
dprinceEmilienM: we want better "composability" for roles21:12
*** kindjal has joined #puppet-openstack21:12
dprinceEmilienM: but for now they are still fairly rigid. And they may be meaningful because there could be different hardware underneath21:12
dprinceEmilienM: i.e. you've got attached storage in some places, but not others21:12
dprinceEmilienM: better nics on some nodes, etc21:13
gildubEmilienM, richm, now that we got that far, can we please finish cleaning the domain, the domains bugs got merged and I need to have the domain default in so we can:21:13
EmilienMdprince: yeah, containers would help to break roles model21:13
gildubEmilienM, richm, 1.backport 2.have a strong base for now (trust) and later (group)21:13
dprinceEmilienM: moving the stateless containers around would likely be easy. The others... not so much21:14
EmilienMdprince: right.. pacemaker, etc :/21:14
*** kindjal_ has joined #puppet-openstack21:15
*** kindjal has quit IRC21:16
*** kindjal_ is now known as kindjal21:16
*** dprince has quit IRC21:18
*** chem has quit IRC21:23
*** Fin1te has quit IRC21:25
*** aimon has joined #puppet-openstack21:31
*** chem has joined #puppet-openstack21:41
openstackgerritEmilien Macchi proposed openstack/puppet-swift: acceptance: switch RDO to CBS repositories  https://review.openstack.org/22227821:44
EmilienMgildub: ping https://review.openstack.org/#/c/218059/40/spec/acceptance/keystone_wsgi_apache_spec.rb,cm21:51
EmilienMgildub: you're dropping 'domain' parameter resource21:51
EmilienMI wonder why21:51
*** derekh has joined #puppet-openstack21:51
*** rhallisey has quit IRC21:54
EmilienMrichm, gildub: see my comment on https://review.openstack.org/#/c/21805921:57
EmilienMyogesh-pc: you should look puppet-nova21:58
yogesh-pcEmilienM: thanks.21:59
yogesh-pcEmilienM: can we place neutron within the controller host machine?22:00
EmilienMyou can place whatever you like22:00
EmilienMour modules are flexible enough22:00
*** ducttape_ has quit IRC22:07
*** morazi has quit IRC22:08
*** ducttape_ has joined #puppet-openstack22:11
yogesh-pcSo is there a manifest for the controller and compute node setups that I can use ?22:11
yogesh-pcI want to setup a quick openstack cloud as a proof of concept,.. pretty new to openstack22:13
EmilienMyogesh-pc: I can give you a single node manifest22:17
EmilienMyogesh-pc: we're currently working to have more documentation22:17
EmilienMand this is something we will provide very soon, probably next week or so22:17
yogesh-pcAwesome.. Can I get a single node to start with... that would be really useful22:18
EmilienMif you want a single node manifest, look: https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L17-L23722:18
yogesh-pcI am using vagrant to test the setup for now22:18
EmilienMtested on ubuntu trusty & centos22:18
EmilienMyogesh-pc: me too22:18
EmilienMthe manifest works fine, out of the box with master branch of modules22:18
yogesh-pcfor my final setup, I am thinking of deploying 2 nodes: one controller with all the services and a compute node that will host the VMs22:19
yogesh-pcso for one node how does it install the hypervisor?22:20
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: switch RDO to CBS repositories  https://review.openstack.org/22238522:20
EmilienMfor the compute you would only need https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L173-L17722:21
EmilienMhttps://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L206-L21422:21
EmilienMhttps://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L151-L16022:21
EmilienMand https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L226-L23122:21
EmilienMotherwise it's on the controller22:21
yogesh-pcSo other than the 3 classes rest all will be on the controller?22:24
yogesh-pcsorry 5 total22:25
yogesh-pcEmilienM: thanks for that... I will start with the single node. If someone could write a manifest for a two node setup, it will be great!22:27
*** xarses has quit IRC22:32
*** kindjal_ has joined #puppet-openstack22:42
*** kindjal has quit IRC22:43
*** kindjal_ is now known as kindjal22:43
*** aimon_ has joined #puppet-openstack22:43
*** cdelatte has joined #puppet-openstack22:44
*** aimon has quit IRC22:45
gildubEmilienM, re 218059, looking22:48
*** xarses has joined #puppet-openstack22:54
openstackgerritMerged openstack/puppet-openstack-integration: integration tests: use dstat to monitor system  https://review.openstack.org/21858722:58
*** dfisher has quit IRC22:58
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Providers tenant/user/user_role: domain check/name  https://review.openstack.org/21805923:07
gildubEmilienM, domain parameter should go away, because they're bad, as discussed with richm. We probably want to do this sooner than later, in a new patch, while domains are still experimental, otherwise we would have to do a deprecation phase23:07
EmilienMwe introduced it in stable/kilo23:07
EmilienMAFIK23:08
gildubEmilienM, yes but experimental23:08
EmilienMbut if we want to drop it OK, but not before M cycle23:08
EmilienMgildub: we can't just drop it23:08
EmilienMit's not our policy23:08
EmilienMgildub: we need to deprecate it23:08
*** cdelatte has quit IRC23:08
gildubEmilienM, ok, that's not a blocker anyway23:08
EmilienMrichm: ^ fyi23:09
EmilienMwe don't drop it, we deprecate it23:09
EmilienMand I want to see acceptance working like before at least for some resources23:09
EmilienMto prove it works like in stable/kilo23:09
gildubEmilienM, agreed, patch 218059 doesn't remove them23:09
EmilienMgildub: ack23:10
EmilienMplease continue to test it23:10
EmilienMat least for one resource23:10
EmilienMand add some comment maybe in acceptance to explain the plans23:10
gildubEmilienM, it's in, waiting for CI result....23:10
EmilienMit's very useful23:10
EmilienMgildub: ok :)23:10
gildubEmilienM, I will add comments23:10
*** TC01 has quit IRC23:13
*** TC01 has joined #puppet-openstack23:17
*** delattec has joined #puppet-openstack23:18
gildubEmilienM, also, could you +2 https://review.openstack.org/221567 ? chem has done +1 - That would clear the path (I think for starting process to back-port the domain bugs fixes23:18
gildubEmilienM, please :)23:18
EmilienMlooking23:18
*** delattec has quit IRC23:19
EmilienMgildub: can you explain that change? maybe I'm missing something23:20
EmilienMgildub: nevermind23:21
gildubEmilienM, the initial driver was to help with Keystone v2/v3 issues (fuel was the first to discover them, among others), now things have evolved, if we don't do that the backport of the domains bugs are going to be much harder23:21
gildubEmilienM, no worries :)23:21
EmilienMgildub: +223:22
gildubEmilienM, thanks23:22
*** ducttape_ has quit IRC23:23
gildubmichchap, G'Day - Could you please review https://review.openstack.org/221567 ?23:25
richmEmilienM: gildub: The more I think about it, the more I realize that having domain parameters is a good thing23:26
richm1) everywhere else Keystone is used, there are explicit domain parameters23:26
richm2) we can still have unique resource titles, even if the names are not unique - so keystone_tenant { 'my tenant': name => 'project', domain => 'domain1'} and keystone_tenant { 'another tenant': name => 'project', domain => 'domain2' } are distinguished by the domain23:27
richm3) who cares if puppet resource doesn't return things with the right name/title?  It doesn't now anyway23:28
michchapgildub: ack23:28
richmso even having "::" in the name is not necessary23:28
richmThat is - perhaps it is better to just get rid of the "name::domain" thing altogether, and just have people pass in domain parameters everywhere23:29
gildubrichm, I disagree with 3) what about idempotency ?23:30
gildubrichm, unless using description field (when available) for storing an id (or equivalent) to uniquely identify the resource, which is probably bad anyway23:32
gildubrichm, ... as you know,  there is no way to identify uniquely the resource otherwise, unless using domain parameter all times, which is not possible when doing discovery (self.instances)23:34
gildubrichm, and if we choose that road, it means adding extra logic into the providers where they shouldn't have to do so because trying to work around things to simulate some sort of idem-potency.23:35
gildubrichm, therefore I think this would end up in the bad design column23:36
gildubrichm, so actually I disagree with 2 and 3. I don't understand 1, unless you mean outside of puppet (like openstackclient), well they don't have to be idempotent, they have unique IDs available23:38
gildubrichm, also forgot to say, the recommendation from Keystone folks are to always explicitly use the domain, we should not forget that because we had to offer a compatibility bridge for PUppet v2 API23:41
gildubBTW, for everyone else here, the latter ^ was submitted via list, even though we didn't get lots of feedback, the recommendation, especially from Keystone side, seems pretty clear23:45
gildubmichchap, thanks23:47
gildubrichm, ... because we had to offer compatibilty we ended up with no mandatory domain23:49
*** angdraug has quit IRC23:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: switch RDO to CBS repositories  https://review.openstack.org/22238523:52
*** yogesh-pc has quit IRC23:53

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