Tuesday, 2015-11-17

*** bklei has joined #puppet-openstack00:00
*** bklei_ has joined #puppet-openstack00:01
*** bklei has quit IRC00:04
*** bklei_ has quit IRC00:05
*** _bmjen is now known as bmjen00:13
*** sergmelikyan has joined #puppet-openstack00:13
*** aimon_ has quit IRC00:13
*** aimon has joined #puppet-openstack00:14
*** sergmelikyan has quit IRC00:18
*** reynir has quit IRC00:19
*** reynir has joined #puppet-openstack00:20
*** bogdando has quit IRC00:29
*** aimon has quit IRC00:29
*** bogdando has joined #puppet-openstack00:35
*** aimon has joined #puppet-openstack00:36
*** gildub_ has quit IRC00:38
*** jerrygb has quit IRC00:40
*** jerrygb has joined #puppet-openstack00:42
*** jerrygb has quit IRC00:42
*** bklei has joined #puppet-openstack00:47
*** bklei has quit IRC00:51
*** sanjayu has quit IRC00:55
*** aimon has quit IRC01:08
*** cwolferh has joined #puppet-openstack01:10
*** xarses has quit IRC01:24
*** xiaohua has joined #puppet-openstack01:30
*** nlahouti has quit IRC01:34
*** markvoelker has joined #puppet-openstack01:37
*** aimon has joined #puppet-openstack01:38
*** aimon_ has joined #puppet-openstack01:40
*** aimon has quit IRC01:43
*** bogdando has quit IRC01:43
*** bogdando has joined #puppet-openstack01:43
*** bogdando has quit IRC01:49
*** richm has quit IRC01:54
*** bogdando has joined #puppet-openstack02:04
*** xiaohua has quit IRC02:13
*** xiaohua has joined #puppet-openstack02:29
*** bklei has joined #puppet-openstack02:35
*** bklei has quit IRC02:40
*** fedexo has joined #puppet-openstack02:57
*** xarses has joined #puppet-openstack02:57
*** gildub_ has joined #puppet-openstack03:00
*** xiaohua has quit IRC03:00
*** cwolferh has quit IRC03:02
*** bapalm has quit IRC03:06
*** fedexo has quit IRC03:08
*** fvollero has quit IRC03:17
*** timrc has quit IRC03:17
*** fvollero has joined #puppet-openstack03:18
*** mkarpin has quit IRC03:18
*** mkarpin has joined #puppet-openstack03:18
*** morazi has quit IRC03:18
*** xiaohua has joined #puppet-openstack03:20
*** xiaohua has quit IRC03:24
*** timrc has joined #puppet-openstack03:24
*** rahulait has quit IRC03:27
*** ducttape_ has joined #puppet-openstack03:27
*** morazi has joined #puppet-openstack03:32
*** xiaohua has joined #puppet-openstack03:54
*** fedexo has joined #puppet-openstack03:57
*** ducttape_ has quit IRC04:08
*** ducttape_ has joined #puppet-openstack04:10
*** _ody has joined #puppet-openstack04:10
*** bklei has joined #puppet-openstack04:24
*** ducttape_ has quit IRC04:24
*** bklei has quit IRC04:29
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Remove occurrence of @@default_domain_id.  https://review.openstack.org/24128404:31
*** zhangjn has joined #puppet-openstack04:35
*** xiaohua has quit IRC04:38
*** zhangjn has quit IRC04:45
*** rahulait has joined #puppet-openstack04:54
*** imcsk8_ has joined #puppet-openstack04:56
*** joshsamuelson has joined #puppet-openstack04:57
*** __nick has joined #puppet-openstack04:57
*** xiaohua has joined #puppet-openstack04:59
*** jprs has quit IRC05:00
*** darkxploit has quit IRC05:00
*** _nick has quit IRC05:00
*** imcsk8 has quit IRC05:00
*** mgagne has quit IRC05:00
*** sbadia has quit IRC05:00
*** kbyrne has quit IRC05:00
*** bkero has quit IRC05:00
*** mwhahaha has quit IRC05:00
*** mgagne has joined #puppet-openstack05:00
*** jerrygb has joined #puppet-openstack05:01
*** kbyrne has joined #puppet-openstack05:01
*** mwhahaha has joined #puppet-openstack05:02
*** bkero has joined #puppet-openstack05:02
*** darkxploit has joined #puppet-openstack05:03
*** sbadia has joined #puppet-openstack05:05
*** xiaohua has quit IRC05:05
*** holser has quit IRC05:18
*** aglarendil has quit IRC05:18
*** bpiotrowski has quit IRC05:19
*** bnemec has joined #puppet-openstack05:19
*** xiaohua has joined #puppet-openstack05:23
*** spredzy has quit IRC05:25
*** beekneemech has quit IRC05:25
*** spredzy_ has joined #puppet-openstack05:25
*** aglarendil has joined #puppet-openstack05:28
*** holser has joined #puppet-openstack05:38
*** jerrygb has quit IRC05:38
*** bpiotrowski has joined #puppet-openstack05:39
*** xiaohua has quit IRC05:46
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/21928905:49
*** sanjayu has joined #puppet-openstack05:57
*** rahulait has quit IRC06:07
*** xiaohua has joined #puppet-openstack06:08
*** bklei has joined #puppet-openstack06:12
*** xiaohua has quit IRC06:13
*** bklei has quit IRC06:16
*** xiaohua has joined #puppet-openstack06:17
*** rcernin has joined #puppet-openstack06:22
*** jerrygb has joined #puppet-openstack06:39
*** jerrygb has quit IRC06:45
*** angdraug has joined #puppet-openstack06:53
*** gildub_ has quit IRC07:00
*** fedexo has quit IRC07:02
openstackgerritNikolay Starodubtsev proposed openstack/puppet-murano: Add ability to deploy Murano CloudFoundry service broker API  https://review.openstack.org/24504307:12
*** nlahouti has joined #puppet-openstack07:29
*** nlahouti has quit IRC07:33
*** spredzy_ is now known as spredzy07:36
*** bklei has joined #puppet-openstack08:01
*** dgurtner has joined #puppet-openstack08:03
*** dgurtner has quit IRC08:03
*** dgurtner has joined #puppet-openstack08:03
*** bklei has quit IRC08:05
*** xiaohua has quit IRC08:31
*** xiaohua has joined #puppet-openstack08:32
*** bkero has quit IRC08:34
*** bkero has joined #puppet-openstack08:34
*** dgurtner has quit IRC08:34
*** dgurtner has joined #puppet-openstack08:35
*** dgurtner has quit IRC08:35
*** dgurtner has joined #puppet-openstack08:35
*** gfidente has joined #puppet-openstack08:35
*** gfidente has quit IRC08:35
*** gfidente has joined #puppet-openstack08:35
*** jpena has joined #puppet-openstack08:36
*** jerrygb has joined #puppet-openstack08:40
*** jtomasek has quit IRC08:41
*** jerrygb has quit IRC08:46
*** openstackgerrit has quit IRC09:31
*** openstackgerrit has joined #puppet-openstack09:32
*** sleinen-AS559 has joined #puppet-openstack09:41
*** karimb has joined #puppet-openstack09:42
*** bklei has joined #puppet-openstack09:49
*** bklei has quit IRC09:54
*** markvoelker has quit IRC10:05
*** gfidente has quit IRC10:18
*** _degorenko is now known as degorenko10:23
*** ddmitriev has joined #puppet-openstack10:28
*** xiaohua has quit IRC10:30
*** xiaohua has joined #puppet-openstack10:30
*** openstackgerrit has quit IRC10:31
*** openstackgerrit has joined #puppet-openstack10:32
*** xiaohua has quit IRC10:32
*** xiaohua has joined #puppet-openstack10:32
*** jerrygb has joined #puppet-openstack10:43
*** jtomasek has joined #puppet-openstack10:46
*** jerrygb has quit IRC10:47
*** xiaohua has quit IRC10:53
*** xiaohua has joined #puppet-openstack10:54
*** marfx000 has joined #puppet-openstack10:54
*** marfx000_ has joined #puppet-openstack10:56
*** marfx000_ has quit IRC10:56
*** marfx000 has quit IRC10:56
*** marfx000 has joined #puppet-openstack10:59
*** markvoelker has joined #puppet-openstack11:06
*** markvoelker has quit IRC11:11
*** zhangjn has joined #puppet-openstack11:12
*** zhangjn has quit IRC11:12
*** zhangjn has joined #puppet-openstack11:13
*** zhangjn has quit IRC11:14
iurygregorygood morning people =D11:14
*** zhangjn has joined #puppet-openstack11:14
*** zhangjn has quit IRC11:15
iurygregoryHey just FYI - the link for the agenda for today meeting is https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20151117 and not https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-201511111:15
*** zhangjn has joined #puppet-openstack11:15
*** zhangjn has quit IRC11:15
*** gildub_ has joined #puppet-openstack11:16
*** zhangjn has joined #puppet-openstack11:16
*** zhangjn has quit IRC11:16
*** zhangjn has joined #puppet-openstack11:17
*** zhangjn has quit IRC11:18
*** zhangjn has joined #puppet-openstack11:18
*** zhangjn has quit IRC11:19
*** zhangjn has joined #puppet-openstack11:20
*** zhangjn has quit IRC11:20
*** zhangjn has joined #puppet-openstack11:21
*** rmart04 has joined #puppet-openstack11:23
*** gildub_ has quit IRC11:23
*** toMeloos has joined #puppet-openstack11:41
*** xiaohua has quit IRC11:41
*** xiaohua has joined #puppet-openstack11:42
*** xiaohua has quit IRC11:48
*** xiaohua has joined #puppet-openstack11:49
*** gfidente has joined #puppet-openstack11:51
*** xiaohua has quit IRC11:56
*** xiaohua has joined #puppet-openstack12:15
EmilienMgood morning folks12:17
xiaohuagood morning12:18
mkarpingood morning!12:19
*** morazi has quit IRC12:24
*** xiaohua has quit IRC12:25
*** xiaohua has joined #puppet-openstack12:26
*** xiaohua has quit IRC12:30
*** jerrygb has joined #puppet-openstack12:31
*** markvoelker has joined #puppet-openstack12:37
*** sleinen-AS5591 has joined #puppet-openstack12:41
*** sleinen-AS559 has quit IRC12:42
degorenkohey :) welcome back EmilienM12:42
*** markvoelker has quit IRC12:42
EmilienMthanks!12:42
*** jtomasek has quit IRC12:43
*** trown|outttypeww is now known as trown12:46
iurygregorymornig EmilienM  o/12:46
*** dprince has joined #puppet-openstack12:59
*** karimb has quit IRC13:02
openstackgerritClayton O'Neill proposed openstack/puppet-openstack-specs: Enhance inifile option deprecations spec  https://review.openstack.org/24051513:08
*** ducttape_ has joined #puppet-openstack13:14
*** __nick is now known as _nick13:17
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Make libvirt migration security configurable  https://review.openstack.org/24638413:22
*** jayg|g0n3 is now known as jayg13:24
EmilienMspredzy: have you looked https://bugs.launchpad.net/puppet-tuskar/+bug/1515273 ?13:24
openstackLaunchpad bug 1515273 in puppet-tuskar "Defaults for *::db classes are incorrect" [Undecided,New]13:24
*** morazi has joined #puppet-openstack13:26
*** bklei has joined #puppet-openstack13:27
*** markvoelker has joined #puppet-openstack13:27
*** bklei has quit IRC13:31
*** jpena is now known as jpena|lunch13:31
*** ducttape_ has quit IRC13:32
*** ken_barber has quit IRC13:42
*** jamespage has quit IRC13:42
*** ken_barber has joined #puppet-openstack13:46
*** jamespage has joined #puppet-openstack14:03
*** richm has joined #puppet-openstack14:05
spredzyEmilienM, I did, I answered on the ML http://lists.openstack.org/pipermail/openstack-dev/2015-November/079387.html14:08
spredzyHaven't done the code yet though14:09
EmilienMspredzy: oh, I did not read all emails yet :) thanks for pointing that out14:09
*** mdorman has joined #puppet-openstack14:09
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/24418414:09
*** bklei has joined #puppet-openstack14:11
*** xiaohua has joined #puppet-openstack14:11
*** bklei has quit IRC14:12
*** bklei has joined #puppet-openstack14:12
*** bklei has quit IRC14:13
*** bklei has joined #puppet-openstack14:16
*** xiaohua has quit IRC14:17
*** richm has quit IRC14:21
*** karimb has joined #puppet-openstack14:22
mkarpinEmilienM  spredzy crincle sbadia guys can you please take a look (when you have time) at  https://review.openstack.org/#/c/241614/, thank you!14:23
*** sanjayu has quit IRC14:26
*** karimb has quit IRC14:27
*** karimb_ has joined #puppet-openstack14:27
*** blentz_ is now known as blentz14:31
*** sergmelikyan has joined #puppet-openstack14:33
*** sergmelikyan has quit IRC14:34
*** sergmelikyan has joined #puppet-openstack14:34
*** sergmelikyan has quit IRC14:34
*** jpena|lunch is now known as jpena14:36
*** darkxploit has quit IRC14:39
*** sergmelikyan has joined #puppet-openstack14:41
*** jtomasek has joined #puppet-openstack14:42
*** jamespage has quit IRC14:45
*** xarses has quit IRC14:53
iurygregoryMeeting will start in 5 minutes in #openstack-meeting-4 =)14:54
*** sergmelikyan has quit IRC14:55
*** richm has joined #puppet-openstack14:58
*** zhangjn has quit IRC14:59
*** mlrn has joined #puppet-openstack15:02
*** spredzy is now known as spredzy|afk15:06
*** dfisher has joined #puppet-openstack15:06
*** ducttape_ has joined #puppet-openstack15:08
*** ducttape_ has quit IRC15:08
*** ducttape_ has joined #puppet-openstack15:09
*** xarses has joined #puppet-openstack15:09
*** Guest80875 is now known as mfisch15:16
*** mfisch is now known as Guest2059415:17
*** Guest20594 is now known as mfisch15:18
*** mfisch has quit IRC15:18
*** mfisch has joined #puppet-openstack15:18
*** samuelBartel has joined #puppet-openstack15:18
*** nlahouti has joined #puppet-openstack15:30
*** nlahouti has quit IRC15:36
*** nlahouti has joined #puppet-openstack15:37
*** nrandall has joined #puppet-openstack15:40
*** samuelBartel has quit IRC15:41
*** nrandall has quit IRC15:41
*** jamespage has joined #puppet-openstack15:47
*** bnemec has quit IRC15:48
*** nlahouti has quit IRC15:54
*** darkxploit has joined #puppet-openstack15:55
*** ntpttr has joined #puppet-openstack16:00
*** imcsk8_ is now known as imcsk816:02
*** bnemec has joined #puppet-openstack16:03
*** rcernin has quit IRC16:04
openstackgerritMerged openstack/puppet-keystone: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24213416:05
*** fedexo has joined #puppet-openstack16:06
*** sleinen-AS559 has joined #puppet-openstack16:14
*** sleinen-AS5591 has quit IRC16:14
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Application resource implementation  https://review.openstack.org/22023816:14
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/24648016:18
openstackgerritMerged openstack/puppet-neutron: Add workaround for v2 tenant names  https://review.openstack.org/24490016:24
*** _ducttape_ has joined #puppet-openstack16:28
*** bogdando has quit IRC16:29
*** morazi has quit IRC16:30
*** ducttape_ has quit IRC16:31
*** cwolferh has joined #puppet-openstack16:36
*** _ducttape_ has quit IRC16:37
EmilienMmwhahaha: https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L48116:39
mwhahahaso i think that's always been there right?16:39
openstackgerritMykyta Karpin proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/22686216:39
*** dgurtner has quit IRC16:40
mwhahahaI thought there was also an email about this where it only affected the people deploying via source and not packages as the packages provide a sane default16:40
mwhahahaalso that param is not actually used16:40
*** ducttape_ has joined #puppet-openstack16:41
mwhahahahttps://github.com/openstack/puppet-keystone/blob/master/manifests/logging.pp#L12616:41
mwhahahaso we want the init.pp log_dir to be undef and the logging.pp log_dir to be specified16:41
mwhahahato allow for backwards compatibility16:41
EmilienMmwhahaha: sounds like something is not consistent here16:45
*** rmart04 has quit IRC16:45
mwhahahathat's part of the original logging change16:45
mwhahahawhich is different then the os_service_default16:45
*** marfx000 has quit IRC16:46
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Switch Sahara to $::os_service_default  https://review.openstack.org/24517816:48
openstackgerritMerged openstack/puppet-nova: Use os_package_type for libvirt service name  https://review.openstack.org/24391216:48
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Keystone_endpoint match service by name/type.  https://review.openstack.org/23943416:49
*** sergmelikyan has joined #puppet-openstack16:50
*** sergmelikyan has quit IRC16:51
*** marfx000 has joined #puppet-openstack16:55
ntpttrEmilienM: Hi, when you have a moment would you mind checking out my initial commit of puppet-magnum :)? https://review.openstack.org/#/c/242164/16:56
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268916:57
*** ntpttr has left #puppet-openstack16:59
*** angdraug has quit IRC16:59
*** nlahouti has joined #puppet-openstack17:03
*** nlahouti has quit IRC17:04
*** nlahouti has joined #puppet-openstack17:04
*** imcsk8 is now known as ichavero|afk17:04
*** marfx000 has quit IRC17:04
*** nlahouti has quit IRC17:06
EmilienMclayton: can I assign https://bugs.launchpad.net/puppet-tuskar/+bug/1515273 + others to you ?17:13
openstackLaunchpad bug 1515273 in puppet-tuskar "Defaults for *::db classes are incorrect" [Undecided,New]17:13
chemmwhahaha: the patch has been merged, so I won't look into it.  But I think I know why you had this behaviour.  If the default domain is not defined in the catalog then it's not removed from the name.  As domain Default is by default, not defined in any puppet catalog, then "Default" is appended.17:14
*** nlahouti has joined #puppet-openstack17:14
*** ntpttr has joined #puppet-openstack17:15
mwhahahayes17:15
EmilienMchem: is it a problem to have this merged?17:15
chemEmilienM: no17:16
mwhahahathe problem (and their may be others) is that this code if executed on a different node than where the tenant is created it runs into that case. The patch was to at least temporarily continue to support the v2 keystone_tenant on neutron resources until we completely ditch the domain-less tenant names17:16
mwhahahaso yes, you're right but it was an issue created by our naming convention change and since it was in the neutron providers it was not found during the keystone tests17:17
chemmwhahaha: I agree :)17:17
chemmwhahaha: but the /problem/ is not that it's not executed on a different server, it's that there is not default domain definition in the catalog17:18
mwhahahaboth are true17:18
chemmwhahaha: to sumarize, adding "keystone_domain {'Default', is_default => true}"17:19
mwhahahabecause if i had keystone_tenant in my catalog it would have never done the lookup17:19
chemto the catalog would suffice17:19
mwhahahahow does that impact the keystone_tenant provider?17:19
chemmwhahaha: no, because the naming would be adjusted as Default would be recognized as default domain and then be removed from the naming17:19
mwhahahawhere?17:20
mwhahaha(in code)17:20
chemI show you 2s17:20
mwhahahasure17:20
chemmwhahaha: it's a bit involving :)17:23
chemmwhahaha: 1/ https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_tenant/openstack.rb#L7317:24
chemmwhahaha: 2/ https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L50-L5917:24
chemmwhahaha:  3/ https://github.com/openstack/puppet-keystone/blob/master/lib/puppet_x/keystone/type/default_domain.rb#L52-L5917:24
mwhahahaha yea that is totally not ovious17:24
mwhahahaobvious17:24
mwhahahawe might want to put a note in the keystone readme about the keystone_domain work around17:25
chemthe naming is checking default_domain which is set by the keystone_tenant/keystone_user/keystone_user_role *type* when they need it (at catalog compilation time)17:25
chemin the @@default_domain17:25
chemthen in 3/ you have the catalog searching for default_domain17:25
chemmwhahaha: I didn't though of this problem :)17:26
chembefore17:26
EmilienMdegorenko, clayton: one of you can take the bugs https://bugs.launchpad.net/puppet-tuskar/+bug/1515273 ? so we can track the work in progress17:27
openstackLaunchpad bug 1515273 in puppet-tuskar "Defaults for *::db classes are incorrect" [Undecided,New]17:27
chemmwhahaha: anyway that's why just to being coherent with people using the default_domain parameter in keystone::init manifest17:27
mwhahahabut that doesn't solve where i'm not pulling in keystone for my other resources on other catalogs17:28
mwhahaharight?17:28
mwhahahaso i hit this in neutron and if that's not on the same host as my keystone::init, i need to specify keystone_domain in the neutron catalog17:28
chemmwhahaha: if you're using different catalag, that doesn't work, you're right17:28
chemmwhahaha: ^^ that's correct17:29
mwhahahayea so we need to document that :D17:29
degorenkoEmilienM, as i understand - just need to move under service_default, right?17:29
dfishergot a dependency question for you all.  why is the puppetlabs-rabbitmq dependency pinned at such an old version?17:29
chemmwhahaha: there is this https://etherpad.openstack.org/p/keystone_no_domain17:30
dfisheror, it was for juno .. nevermind17:30
EmilienMdegorenko: yes, I think. Could be done in a separated patch17:30
chemand this  https://review.openstack.org/#/c/219127/ that need some love17:31
degorenkoEmilienM, ok, then i will fix it17:31
degorenkofor sahara too, of course :D17:31
degorenkobtw, for sahara it is on reivew17:31
EmilienMdfisher: we're using 5.2.3 and the latest is 5.3.117:31
mwhahahafor me etherpad != documentation (nor specs for that matter). something like this needs to be visible in the puppet module read me or something of the source17:31
chemI think putting this information in the blueprint would make sense17:31
chemmwhahaha: I agree17:32
EmilienMdfisher: please submit a patch against puppet-openstack-integration17:32
dfisherright.  Juno capped it at < 4.0.017:32
mwhahahabecause if i'm an end user i just see the readme and thats it17:32
dfishernono.  kilo is capped at < 6.0.017:32
EmilienMah, kilo17:32
dfisheryeah, we're fine.17:32
chemmwhahaha: ha, so the spec is no good, neither17:32
chem?17:32
mwhahahano17:32
mwhahahano operator is going to go looking in the specs for a given release17:32
mwhahahaif they are they have more time on their hands then they should17:33
mwhahahasomething like that is working with the module and needs to be packaged with the module17:33
mwhahahai would defer to others if they disagree, but when I'm consuming a puppet module i'm looking at the read me for important usage information17:33
chemmwhahaha: oki17:35
*** rcernin has joined #puppet-openstack17:36
*** ichavero|afk is now known as imcsk817:39
*** sergmelikyan has joined #puppet-openstack17:39
*** trown is now known as trown|outttypeww17:41
*** trown|outttypeww is now known as trown|lunch17:41
degorenkoEmilienM, for glance 237970 - i guess i can use https://review.openstack.org/237973 as dependency and just update my patch with os_service_default17:43
chemmwhahaha: reading the code again, the problem shouldn't have appeared as 'Default' is the fallback for default domain17:46
openstackgerritIvan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24121417:46
chemmwhahaha: and, as such, should have been removed from the name17:46
mwhahahahmm ok17:46
chemmwhahaha: if you got to see 'blah::Default' in the listing of the resource, then, it's a bug in the keystone module17:47
mwhahahayea that's what i got17:47
*** IvanBerezovskiy has left #puppet-openstack17:47
EmilienMdegorenko: ok17:48
chemmwhahaha: Oki. I check.17:48
*** aglarendil has quit IRC17:49
*** holser has quit IRC17:54
*** markvoelker_ has joined #puppet-openstack17:57
*** serg_melikyan has joined #puppet-openstack17:57
*** richm has quit IRC17:59
*** bnemec has quit IRC17:59
*** bpiotrowski has quit IRC18:00
*** sergmelikyan has quit IRC18:00
*** markvoelker has quit IRC18:00
*** sbadia has quit IRC18:00
*** timrc has quit IRC18:00
*** bnemec has joined #puppet-openstack18:00
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/24648018:02
*** serg_melikyan has quit IRC18:03
*** timrc has joined #puppet-openstack18:05
*** sergmelikyan has joined #puppet-openstack18:06
*** sbadia has joined #puppet-openstack18:06
*** crinkle sets mode: +o EmilienM18:08
*** richm has joined #puppet-openstack18:09
*** sanjayu has joined #puppet-openstack18:10
*** aimon_ has quit IRC18:10
openstackgerritMykyta Karpin proposed openstack/puppet-glance:  Add support of os_service_default fact in glance spec  https://review.openstack.org/23797318:11
*** sanjayu has quit IRC18:18
*** darkxploit has quit IRC18:20
chemmwhahaha: another question about your running environment.  Where you use the neutron module, you don't create any tenant in the catalog ?18:20
chemmwhahaha: is that correct ?18:20
mwhahahacorrect18:20
mwhahahafuel uses task based deployments so we only pull in the code required to do what we are working on18:21
mwhahahaso the tenants are setup elsewhere18:21
mwhahahaand neutron comes along to configure it's vips w/o the tenant/user definitions18:21
mwhahahathe equivalent would be setting up keystone on server A and neutron on server B18:21
mwhahahaor at least a similar condition18:22
mwhahahaspecifically the item i was running into was for the neutron_subnets resource18:22
chemmwhahaha: so no tenant/user/user_role creation in the catalog.  Oki, I pinpoint the bug, then.18:22
mwhahahacorrect18:22
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Introduce glance::db class  https://review.openstack.org/23797018:23
chemmwhahaha: it's only when one of those resource is present that the default domain is set18:23
mwhahahayea that's what i figured which is where documenting the keystone_domain { 'Default': is_default => true } would be imporant18:23
mwhahahaimportant18:23
chemmwhahaha: humm ... I think that the keystone module should be fixed for this case as well.18:24
mwhahahaprobably18:25
*** jpena is now known as jpena|away18:25
chemmwhahaha: once again the keystone_domain {is_default} is just a hack, you shouldn't need it to make it work18:25
openstackgerritDenis Egorenko proposed openstack/puppet-tuskar: Switch Tuskar to $::os_service_default  https://review.openstack.org/24654118:26
*** degorenko is now known as _degorenko18:27
*** nlahouti has quit IRC18:31
*** nlahouti has joined #puppet-openstack18:33
*** sergmelikyan has quit IRC18:36
*** aimon has joined #puppet-openstack18:41
*** trown|lunch is now known as trown18:41
*** aglarendil has joined #puppet-openstack18:48
*** ntpttr has quit IRC19:01
*** ntpttr has joined #puppet-openstack19:02
*** morazi has joined #puppet-openstack19:05
*** holser has joined #puppet-openstack19:10
*** karimb_ has quit IRC19:10
*** gfidente has quit IRC19:11
*** bpiotrowski has joined #puppet-openstack19:17
*** gildub_ has joined #puppet-openstack19:20
_odyI noticed things referencing $::os_service_default when I read today's meetings notes; are we now wanting to block commits using the old "<SERVICE DEFAULT>" pattern now?19:21
chemrichm: EmilienM mwhahaha gildub_ : I have created this bug report for capturing the problem they had with the neutron module: https://bugs.launchpad.net/puppet-keystone/+bug/151718719:36
openstackLaunchpad bug 1517187 in puppet-keystone "Default domain not calculated when no keystone_user/keystone_tenant/keystone_user_role is defined." [Undecided,New]19:36
EmilienMchem: interesting19:37
EmilienMI'm happy we see that bug before the release19:37
EmilienMchem: have you seen https://bugs.launchpad.net/puppet-heat/+bug/1516299 ?19:40
openstackLaunchpad bug 1516299 in puppet-heat "heat::keystone::auth configuration failing" [Undecided,New]19:40
EmilienM_ody, Hunner: big highlight on https://review.openstack.org/#/c/245874/19:41
chemEmilienM: richm gildub_ do we agree on this  "When not set, Keystone default domain id is 'default'"19:43
chemEmilienM: richm gildub_ from https://review.openstack.org/#/c/219127/6/specs/kilo/api-v3-support.rst19:43
EmilienMchem: yes or 'Default', no?19:43
chemEmilienM: ?19:44
EmilienMdefault = Default ?19:44
EmilienMor we don't care?19:44
iurygregoryno!19:44
iurygregoryit's different19:44
_odysbadia has a review open about that issue he was looking for feeback on.19:44
iurygregoryDefault is name19:44
iurygregorydefault is id19:44
iurygregoryfor the default domain =)19:44
chemEmilienM: iurygregory that's correct, and that's what is said in the review19:45
EmilienMgreat19:45
chemdomain_id = default means domain_name 'Default'19:45
iurygregoryyep ^^19:45
chemEmilienM: richm gildub_ because the corner raised by mwhahaha cannot be fixed by any other way19:46
chemEmilienM: richm gildub_ the catalog cannot be searched at self.instances time19:46
iurygregoryi think is valid consider when not set domain_id is default19:47
*** angdraug has joined #puppet-openstack19:48
_odyEmilienM: Looking into a good recommendation on the version string matching.  The referenced change might only be valid for that 3.8.x release of PE.  PE versioning changes after that and heavily relies on the version numbers assocaited with the all-in-one agent.19:48
chemiurygregory: EmilienM richm gildub_ when I say "not set", I mean that the keystone_user/keystone_tenant/keystone_user_role has no ::domain part19:48
chemiurygregory: EmilienM richm gildub_ in this case, then we consider that it's 'Default'19:49
chemiurygregory: EmilienM richm gildub_ so, is that ok to implement the behavior in this review ?19:50
chemiurygregory: EmilienM richm gildub_ relative to what we have now that means: no catalog searching at all19:51
chemiurygregory: EmilienM richm gildub_ relative to what we have now that means: and simply using "Default"19:51
iurygregorycan you point the review?19:51
iurygregoryi think this approach is valid19:52
chemiurygregory: https://review.openstack.org/#/c/219127/6/specs/kilo/api-v3-support.rst19:52
chemEmilienM: richm gildub_ iurygregory I have leave now.  Please comment in the bugzilla or somewhere I can find it tomorrow.19:53
iurygregorytonigh i'll take a look19:53
EmilienMchem: will look19:53
chemEmilienM: I think the puppet-heat bug is related, have to look.  Tomorrow..19:54
EmilienMchem: thanks and good night!19:54
chemEmilienM: the good news is that "Default" will certainly fix all those bug19:54
iurygregorynice to know chem, good night o/19:55
chemthanks, bye19:56
EmilienMchem: thanks a lot19:56
*** EmilienM sets mode: +o sbadia19:59
*** EmilienM sets mode: +o richm19:59
*** EmilienM sets mode: +o spredzy|afk19:59
*** EmilienM sets mode: +o mfisch19:59
*** EmilienM sets mode: +o clayton19:59
*** EmilienM sets mode: +o mgagne19:59
*** EmilienM sets mode: +o _degorenko19:59
*** EmilienM sets mode: +o dprince19:59
*** cgfbee has quit IRC20:01
*** cgfbee has joined #puppet-openstack20:04
*** fedexo has quit IRC20:11
chemEmilienM: iurygregory richm gildub_ : I have sum up my though on the matter there https://review.openstack.org/#/c/219127/ (the new comments, not the old one)20:11
*** cwolferh has quit IRC20:12
*** karimb has joined #puppet-openstack20:12
iurygregoryok chem =)20:13
*** cwolferh has joined #puppet-openstack20:13
*** nlahouti has quit IRC20:19
*** jpena|away has quit IRC20:43
*** rmart04 has joined #puppet-openstack20:50
*** rmart04 has left #puppet-openstack20:52
*** dprince has quit IRC20:52
*** rahulait has joined #puppet-openstack20:53
mfischanyone had issues with the nova providers not handling regions?20:58
mfischonce I put host aggregates into multinode I see an issue, its only hitting our east endpoint20:58
zigoEmilienM: Hi man!21:06
zigoHow was your holidays?21:06
zigoGuys, I really need this one to be merged: https://review.openstack.org/#/c/241218/21:07
zigoIt has a huge impact on so many things for me.21:08
zigomwhahaha: ^21:09
mwhahahaoh the pesky collectstatic21:10
mwhahahayou know i'm not sure that works correctly for ubuntu either21:11
zigomwhahaha: This one is a distraction, what I need is all the rest.21:12
zigoIe, correct .conf name, etc.21:12
mwhahahayea21:12
mwhahahaso did you ever see my response about the openstack-dashboard pulling in openstack-dashboard-apache?21:13
zigomwhahaha: It's IMO a package's decision to run collectstatic, and puppet should never deal with it.21:13
mwhahahabecause it's a recommended package or whatever21:13
zigoI'm not sure I did.21:13
mwhahahaso when i apt-get install openstack-dashboard it automatically is install openstack-dashboard-apache21:13
zigoWell, you don't have to run with recommended turned on...21:14
dfisherzigo: i agree with you (about collectstatic).  for solaris, we collectstatic/compress before packaging.21:14
zigoapt-get install --no-install-recommends also would work.21:14
mwhahahatrue, it's just the default21:14
zigoI run *everything* with recommended turned off.21:14
mwhahahawe should probably update fuel to set that21:15
zigoIf this is a blocker, I can turn the recommended into a suggest, I don't mind much.21:15
zigoThough really, if this patch is merged, everything is solved either ways, because we get the correct configuration file modified by puppet.21:16
mwhahahabut anyway so openstack-dashboard-apache conflicts with the horizon::wsgi::apache class21:16
mwhahahai think suggests also get automagically installed as well21:16
zigoBecause of the double-defined WSGI definition?21:16
mwhahahayea21:16
zigoWell, that patch fixes that !!!21:16
zigoThat's the whole point of it.21:16
mwhahahanope21:16
mwhahahait doesn't it makes it worse :D21:16
zigo-            $httpd_config_file     = '/etc/apache2/sites-available/openstack-dashboard.conf'21:16
zigo+            $httpd_config_file     = '/etc/apache2/sites-available/openstack-dashboard-alias-only.conf'21:16
zigoSo, where's that thing, so that I get it to learn what file to use? :)21:17
zigo(and add it to this patch...)21:17
mwhahahaso down the puppet rabbit hole we go21:17
mwhahahahttps://github.com/openstack/puppet-horizon/blob/master/manifests/wsgi/apache.pp21:17
zigoThanks.21:17
mwhahahaspecifically, https://github.com/openstack/puppet-horizon/blob/master/manifests/wsgi/apache.pp#L10621:17
mwhahahaer wait21:18
mwhahahayou might be right21:18
zigoLOL ! :)21:18
mwhahahait's totally different in fuel so maybe we're doing it elsewhere21:18
mwhahahaneed to try out a clean version of the module and package at some point21:18
mwhahahanope here it is21:19
mwhahahahttps://github.com/openstack/puppet-horizon/blob/master/manifests/wsgi/apache.pp#L8521:19
zigoclean package != the one from Ubuntu ... :P21:19
zigo?!?21:19
zigoThere's no vhost involved here ...21:19
zigoIt's only an Alias.21:20
mwhahaharight but that's where the wsgi stuff gets duped21:20
*** sergmelikyan has joined #puppet-openstack21:20
mwhahahaso i guess you're right, by fixing the http_config_file out puppet should clear that out21:20
mwhahahahttps://github.com/openstack/puppet-horizon/blob/master/manifests/wsgi/apache.pp#L106-L11621:20
mwhahahaok let me help you out with tests on that review21:20
mwhahahaand then i think it should be good21:20
zigo:)21:20
zigoCool, you're my hero, really.21:21
zigoIt was so far a nightmare to deal with, you're the only light I see at the end of the tunnel.21:21
mfischclayton: near as I can tell none of the nova providers can handle regions21:22
*** jayg is now known as jayg|g0n321:24
*** rahulait has quit IRC21:39
mwhahahazigo: how does the debian version of the package handle COMPRESS_OFFLINE in the local_settings.py?21:51
zigomwhahaha: It's added by the package.21:52
mwhahahaso we set it in puppet21:52
zigoecho "COMPRESS_OFFLINE=True" >>$(CURDIR)/debian/openstack-dashboard/usr/share/openstack-dashboard/local_settings.py21:52
zigoThat's what I do in the debian/rules.21:52
zigoI guess it's not so much of a trouble to have this defined twice though ...21:53
zigomwhahaha: Or is it?21:53
mwhahahawell no so i was looking at the change and techincally the collectstatic is only run if compress_offline is true21:53
zigoYou mean false?21:54
mwhahahaso normally it's true (and would be set) and your appending may be overridden21:54
mwhahahano it appears that if it's false it doesn't get added21:54
zigoI'm not sure I get it...21:54
zigocollectstatic is in fact not needed when you have COMPRESS_OFFLINE=True.21:54
zigoBut it is if it is False ...21:54
zigoHum...21:55
zigoI'm mistaking with no compression at all! :)21:55
mwhahahahttps://github.com/openstack/puppet-horizon/blob/master/manifests/init.pp#L39021:55
zigoNow I'm confused.21:55
zigoWhat's the logic behind this?21:55
mwhahahaso that's the line that actually triggers the collectstatic21:55
mwhahahano idea21:55
mwhahahawas like this when i got here :D21:55
mwhahaharefreshonly => true on the exec means it only runs when it's notified, and we only notify it when the config file is changed and if compress_offline is true21:56
mwhahahahttps://github.com/openstack/puppet-horizon/commit/720ce5cf845d57ce04d9d602cbd58b45f314412121:57
mwhahahawas the original change that put this thing in there21:57
mwhahahamy assumption as that whole collectstatic thing needs to get pulled21:59
EmilienMzigo: hey21:59
zigoEmilienM: \o21:59
zigomwhahaha: Agreed.22:01
EmilienMzigo: the beach was awesome, I already miss it22:01
zigoEmilienM: Where did you go?22:01
EmilienMzigo: Saint Martin22:01
bkeroOoo, nice part of france22:01
zigoNice.22:02
EmilienMan Island in the middle of nowhere, without anything but beach and sun. Exactly what I needed before canadian winter ;-)22:02
zigoWell, assuming he was on the French half ...22:02
EmilienMI was22:02
EmilienMzigo: what do you need from now, how can I help?22:02
EmilienMI'm about boarding to my flight to Paris22:03
zigoI need this fix to be merged: https://review.openstack.org/#/c/241218/22:03
EmilienMlooking22:03
bkeroSo much for the canadian winter22:03
mwhahahai'm writing tests for -^22:03
zigoOtherwise, I'm in a big shit, with the sky falling down on me, customers unhappy, and colleagues pointing fingers at me ... :D22:03
mwhahahaalso trying to fix this logic for this collectstatic crap22:03
zigoI'm only half joking here...22:04
mwhahahado we know if redhat needs the collecstatic?22:04
*** angdraug has quit IRC22:04
mwhahahait was specifically put in to work around debian last year22:04
EmilienMmwhahaha: I don't think so but I'm not sure22:05
mwhahahaso i'm going to wrap the whole lot up in a only for redhat for now22:05
mwhahahaand if we determine it's not need for redhat we can kill it later22:05
zigo:)22:05
zigoSounds like a good plan.22:05
zigomwhahaha: BTW, in what timezone are you? Sunnyvale?22:06
mwhahahacentral until next week then mountain22:06
dfisherthe best time zone.22:06
dfisher(mountain)22:06
mwhahahamoving back to a proper state22:06
mwhahahaone with seasons other than hot and hotter22:07
dfisherwhere you moving to?22:07
mwhahahaaurora, co22:07
dfishernice.  I'm in Broomfield :)22:08
zigoThat's Denver, right?22:08
mwhahahayea22:08
* mwhahaha stalks dfisher22:08
* dfisher makes it easier22:08
mwhahahai moved to austin ~2 years ago from denver22:08
*** karimb has quit IRC22:08
dfisherwelcome home.22:08
zigoYou should have wait until next summit, so you could have been "local" ! :)22:09
zigo(rare enough...)22:09
mwhahahaah i'll come back for the summit22:09
zigoI hope so.22:09
mwhahahamy wife is dying from the allergens here22:09
mwhahahacan't stay22:09
dfishersnowed here last night.   everything died.22:09
zigoThey announced snow here for next weekend...22:10
zigoAbout time !!!22:10
zigoSkiiiiiiiiiiiiiiiiiii ! :)22:10
dfisherusually the snow isn't quite good enough for skiing until after the new year.22:10
dfisherbut, we're due to get a TON this season22:10
zigoOver here, it's enough if you go high enough. But this year, even very high, they had none.22:11
ntpttrskiing in Portland on mt hood has been suffering the last few years as well22:11
bkeroThis year's not going to be any better either.22:12
zigohttp://www.tignes.net/en/skiing-in-tignes/webcams-290.html <--- Nothing, even at 2100m ...22:12
zigo(and higher)22:12
ntpttr:(22:12
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian  https://review.openstack.org/24121822:19
mwhahahazigo: -^ now with tests22:19
zigomwhahaha: Thanks.22:21
*** rcernin has quit IRC22:21
zigoLet's wait for the gate ...22:21
* mwhahaha wanders off22:25
mwhahahai'll check later to make sure everything went ok, but the tests passed locally22:25
*** ntpttr has quit IRC22:26
*** rahulait has joined #puppet-openstack22:27
openstackgerritMatt Fischer proposed openstack/puppet-nova: Add region support to nova providers  https://review.openstack.org/24663222:31
mfischfix for nova22:32
mfischlack of region name comprehension22:32
openstackgerritMatt Fischer proposed openstack/puppet-nova: Add region support to nova providers  https://review.openstack.org/24663222:33
*** ntpttr has joined #puppet-openstack22:35
dmsimardmfisch: probably want to replace conf['neutron'] by conf['nova']22:37
*** ntpttr has quit IRC22:38
zigomwhahaha: I don't think we care much about it, but the current stable version for Debian is 8.2, not 6.23:05
*** xavpaice has joined #puppet-openstack23:08
*** sergmelikyan has quit IRC23:10
*** sergmelikyan has joined #puppet-openstack23:12
*** morazi has quit IRC23:13
* _ody found the OpenStack packaging for squeeze unusable23:20
*** nlahouti has joined #puppet-openstack23:21
*** ducttape_ has quit IRC23:25
*** ddmitriev has quit IRC23:29
*** sergmelikyan has quit IRC23:30
*** dfisher1 has joined #puppet-openstack23:43
*** dfisher has quit IRC23:43
*** dfisher1 has left #puppet-openstack23:43

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