Monday, 2016-04-04

*** johnmilton has joined #puppet-openstack00:33
*** markvoelker has joined #puppet-openstack00:47
*** beddari has quit IRC01:10
*** beddari has joined #puppet-openstack01:12
*** johnmilton has quit IRC01:32
*** morazi has quit IRC01:39
*** ilbot3 has quit IRC01:45
*** ilbot3 has joined #puppet-openstack01:47
*** bapalm has joined #puppet-openstack01:51
*** ducttape_ has joined #puppet-openstack01:54
*** ducttape_ has quit IRC01:55
*** ducttape_ has joined #puppet-openstack01:55
*** ducttape_ has quit IRC02:33
*** ducttape_ has joined #puppet-openstack02:34
*** ducttape_ has quit IRC02:34
*** ducttape_ has joined #puppet-openstack02:34
*** ducttape_ has quit IRC02:57
*** ducttape_ has joined #puppet-openstack02:58
*** ducttape_ has quit IRC02:58
*** markvoelker has quit IRC03:14
*** fedexo has joined #puppet-openstack03:30
*** beddari has quit IRC03:45
*** beddari has joined #puppet-openstack03:52
*** beddari has quit IRC04:00
*** beddari has joined #puppet-openstack04:02
*** markvoelker has joined #puppet-openstack04:15
*** markvoelker has quit IRC04:21
*** aimon has quit IRC04:37
*** chandankumar has joined #puppet-openstack04:49
*** sshnaidm has joined #puppet-openstack05:05
*** aimon has joined #puppet-openstack05:20
*** aimon has quit IRC05:25
*** saneax_AFK is now known as saneax06:01
*** jtomasek has joined #puppet-openstack06:20
*** jmikolajczak has joined #puppet-openstack06:40
*** magicboiz has joined #puppet-openstack06:44
*** rmallah has joined #puppet-openstack06:44
*** magicboiz has quit IRC06:45
*** Guest29628 has quit IRC06:45
rmallahthe "Begining with Nova" section of https://forge.puppetlabs.com/openstack/nova  does not document setting up of keystone user , services endpoint etc as documented  http://docs.openstack.org/liberty/install-guide-ubuntu/nova-controller-install.html06:47
rmallahare users of the module expected to do so on their own ?06:50
*** amoralej has joined #puppet-openstack06:51
*** foxhunt_away is now known as foxhunt7206:54
*** hjensas has joined #puppet-openstack07:07
*** fedexo has quit IRC07:08
*** aimon has joined #puppet-openstack07:08
*** aimon has quit IRC07:13
-openstackstatus- NOTICE: Gerrit is going to be restarted due to bad performance07:15
*** leanderthal|afk is now known as leanderthal07:21
*** jpena|off is now known as jpena07:24
*** paramite has joined #puppet-openstack07:32
openstackgerritIgor Zinovik proposed openstack/puppet-glance: Implement ability to pass CA bundle certificate for vCenter server  https://review.openstack.org/30049507:32
*** xingchao has joined #puppet-openstack07:33
openstackgerritIgor Zinovik proposed openstack/puppet-glance: Implement ability to pass CA bundle certificate for vCenter server  https://review.openstack.org/30049507:34
*** paramite is now known as paramite|afk07:34
*** dgurtner has joined #puppet-openstack07:35
*** rmallah has quit IRC07:39
*** xingchao has quit IRC07:41
*** jmikolajczak has quit IRC07:43
*** jmikolajczak has joined #puppet-openstack07:43
*** rcernin has joined #puppet-openstack07:43
*** paramite|afk is now known as paramite07:48
*** fawadkhaliq has joined #puppet-openstack07:54
*** fawadkhaliq has quit IRC08:10
*** jaosorior has joined #puppet-openstack08:14
openstackgerritMartin Mágr proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/23479808:15
*** gfidente has joined #puppet-openstack08:16
*** jistr has joined #puppet-openstack08:19
*** beddari has quit IRC08:25
*** rmallah has joined #puppet-openstack08:33
*** mdavidson has quit IRC08:39
*** derekh has joined #puppet-openstack08:41
*** beddari has joined #puppet-openstack08:42
*** dgurtner has quit IRC08:48
*** dgurtner has joined #puppet-openstack08:48
*** dgurtner has quit IRC08:48
*** dgurtner has joined #puppet-openstack08:48
*** dtantsur|afk is now known as dtantsur08:51
*** aimon has joined #puppet-openstack08:56
*** jaosorior has quit IRC08:57
*** _degorenko|afk is now known as degorenko08:59
*** aimon has quit IRC09:01
*** xingchao has joined #puppet-openstack09:03
*** rmallah has quit IRC09:04
*** jaosorior has joined #puppet-openstack09:05
*** skolekonov|away is now known as skolekonov09:15
*** chandankumar has quit IRC09:23
*** mdavidson has joined #puppet-openstack09:35
*** saneax is now known as saneax_AFK09:39
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/25043609:42
*** saneax_AFK is now known as saneax09:48
*** v1k0d3n has quit IRC10:05
*** v1k0d3n has joined #puppet-openstack10:05
*** f0f4r has joined #puppet-openstack10:08
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar  https://review.openstack.org/28151510:17
*** chandankumar has joined #puppet-openstack10:18
*** _matthias__ is now known as _matthias_10:31
*** saneax is now known as saneax_AFK10:35
*** saneax_AFK is now known as saneax10:41
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/25043610:43
*** aimon has joined #puppet-openstack10:45
*** dgurtner has quit IRC10:45
*** openstackgerrit has quit IRC10:48
*** openstackgerrit has joined #puppet-openstack10:48
*** aimon has quit IRC10:49
*** dgurtner has joined #puppet-openstack10:52
*** dgurtner has quit IRC10:52
*** dgurtner has joined #puppet-openstack10:52
*** dgurtner has quit IRC10:53
*** dgurtner has joined #puppet-openstack10:53
*** xingchao has quit IRC10:56
*** xingchao has joined #puppet-openstack10:57
*** chem has joined #puppet-openstack11:18
*** johnmilton has joined #puppet-openstack11:24
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::cache define  https://review.openstack.org/30105711:29
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::concurrency define  https://review.openstack.org/30105711:30
*** jtomasek_ has joined #puppet-openstack11:36
*** chandankumar has quit IRC11:40
*** jtomasek_ has quit IRC11:50
*** dgurtner has quit IRC11:55
*** dgurtner has joined #puppet-openstack11:57
*** dgurtner has quit IRC11:57
*** dgurtner has joined #puppet-openstack11:57
*** dgurtner_ has joined #puppet-openstack12:00
*** dgurtner_ has quit IRC12:00
*** dgurtner_ has joined #puppet-openstack12:00
*** dgurtner has quit IRC12:01
*** foxhunt72 is now known as foxhunt_away12:10
*** dprince has joined #puppet-openstack12:11
*** markvoelker has joined #puppet-openstack12:15
EmilienMhello12:17
*** jayg|g0n3 is now known as jayg12:17
EmilienMmfisch: on hypvervisors12:17
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/30107912:17
yottatsaplease review https://review.openstack.org/#/c/300609/ Use global_physnet_mtu to fix MTU settings in Mitaka12:21
*** amoralej is now known as amoralej|lunch12:22
EmilienMdegorenko: hey! I updated https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016040512:30
EmilienMdegorenko: and I already sent the email!12:30
*** aimon has joined #puppet-openstack12:33
EmilienMdprince: looking https://review.openstack.org/30064012:34
EmilienMdprince: don't they use oslo.db?12:34
EmilienMusing sql_connection is a really old thing12:34
*** jpena is now known as jpena|lunch12:36
dprinceEmilienM: agree it is old12:37
*** aimon has quit IRC12:38
EmilienMdprince: I commented but can drop -1 if something comes up in follow-up12:38
dprinceEmilienM: but that is functionally required to make barbican actually work fo me12:38
EmilienMdprince: cool no problem12:38
EmilienMas long as it works and there is no warning in logs.12:38
dprinceEmilienM: like without it db-sync fails, there is no Mysql database, etc.12:38
dprinceEmilienM: My guess is people have been using it w/ sqlite (the default)12:38
dprinceEmilienM: that would have worked with the broken setting12:39
iurygregorymorning o/12:40
EmilienMdprince: yeah; we want to test mysql.12:40
EmilienMdprince: I want to see beakers tests for that ^12:41
EmilienMthey are currently missing.12:41
mjblackso I'm seeing that lbaas v2 config is using multi line options with the same key, just like ceilometer is doing12:43
mjblacktrying to think of a good way to handle that scenario12:44
openstackgerritVladimir Eremin proposed openstack/puppet-neutron: Use global_physnet_mtu to fix MTU settings in Mitaka  https://review.openstack.org/30060112:53
yottatsaEmilienM I've added releasenotes ^^12:53
EmilienMyottatsa: thanks a ton12:54
mjblackEmilienM,mfisch: any thoughts of how to tackle multiple keys with same name in the ini settings?12:56
EmilienMmjblack: paramite did that for lbaasv2, maybe he knows out of the box12:57
mjblackthere is already an lbaasv2 class in puppet-neutron?12:59
paramitemjblack, there is a openstack_config resource provider in puppet-openstacklib which has this capability12:59
paramitemjblack, yes there is12:59
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/30110613:00
mjblackwhere is it? I didnt see it under the agents directory13:00
mjblackah ok, found it13:01
mjblackparamite: isnt service_authtoken required for the config?13:04
paramitemjblack, I don't remember tbh13:11
paramitemjblack, but if it is not in the class, I guess it was not when I was implementing it13:12
mjblackits not in the manifests but the file dropped by the package does have the sections for it13:12
mjblackplus load balancer pool scheduler13:12
*** dtantsur is now known as dtantsur|brb13:12
*** jaosorior has quit IRC13:14
degorenkoEmilienM, hey, ok, thanks :)13:14
*** jaosorior has joined #puppet-openstack13:14
EmilienMdegorenko: fuel CI seems passing on https://review.openstack.org/30044213:15
degorenkoEmilienM, noop tests - library unit tests failed13:16
EmilienMok13:17
*** jaosorior has quit IRC13:17
*** jaosorior has joined #puppet-openstack13:18
openstackgerritMerged openstack/puppet-glance: Implement ability to pass CA bundle certificate for vCenter server  https://review.openstack.org/30049513:18
*** foxhunt_away is now known as foxhunt7213:19
*** morazi has joined #puppet-openstack13:20
beddariEmilienM: has this been discussed more? https://github.com/twc-openstack/puppet-os_docker/issues/1  ... this is a topic I'm really interested in, thanks for bringing it up13:20
beddariEmilienM: the code does look rather specific to their workflow though :)13:21
EmilienMbeddari: we'll discuss about it at Summit https://etherpad.openstack.org/p/newton-design-puppet13:21
*** amoralej|lunch is now known as amoralej13:21
beddariEmilienM: great, thanks13:24
*** rmallah has joined #puppet-openstack13:34
*** magicboiz has joined #puppet-openstack13:35
*** magicboiz has quit IRC13:35
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Manage Neutron AAS test packages when Neutron is available  https://review.openstack.org/30078613:38
yottatsaEmilienM > you need to add |13:48
yottatsawhat do you mean?13:48
*** jpena|lunch is now known as jpena13:50
*** chandankumar has joined #puppet-openstack13:51
EmilienMyottatsa: err, yes13:51
*** magicboiz has joined #puppet-openstack13:51
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Add ml2 package requirement to neutron::plugins::ml2::mech_driver  https://review.openstack.org/30114213:53
openstackgerritVladimir Eremin proposed openstack/puppet-neutron: Use global_physnet_mtu to fix MTU settings in Mitaka  https://review.openstack.org/30060113:55
yottatsaEmilienM fixed13:55
EmilienMyottatsa: thanks13:55
yottatsatox passed13:56
*** dtantsur|brb is now known as dtantsur13:58
*** EmilienM changes topic to "CI status: ORANGE: openstack-infra/zuul is experiencing issues - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"14:02
*** xarses has quit IRC14:10
mattymoEmilienM, I hope it gets fixed soon14:10
mfischEmilienM: have you seen this with puppet-tempest?  Error: /Stage[main]/Tempest/Tempest_neutron_net_id_setter[public_network_id]: Could not evaluate: Command openstack_command is missing14:12
mfischError: /Stage[main]/Tempest/Tempest_glance_id_setter[image_ref]: Could not evaluate: Command openstack_command is missing14:12
mfischI'm missing a setting it seems but dont know which14:15
EmilienMmfisch: do you have osclient installed?14:16
mfischit appears no14:16
mfischI will add14:16
EmilienMmaybe we should add the dep14:17
mfischwe should make that a dep14:17
mfischjinx14:17
EmilienMmfisch:  include '::openstacklib::openstackclient'14:17
EmilienMmfisch: I can do it14:17
mfischthx14:17
*** Guest29628 has joined #puppet-openstack14:20
*** dhollinger has joined #puppet-openstack14:21
*** aimon has joined #puppet-openstack14:21
EmilienMmfisch: do you run tempest from an external node?14:24
EmilienMor from a controller?14:24
mfischEmilienM: I'm just playing around with it right now, will probably be the same node14:25
mfischjust an experiment14:25
EmilienMmfisch: I don't understand why you miss the osclient then14:25
mfischI haven't installed the rest of openstack yet14:25
EmilienMmhh ok14:25
*** aimon has quit IRC14:26
*** haplo37 has joined #puppet-openstack14:27
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: deploy openstackclient and make our providers requiring it  https://review.openstack.org/30116214:29
EmilienMmfisch: ^14:29
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: deploy openstackclient and make our providers requiring it  https://review.openstack.org/30116214:30
*** rmallah has quit IRC14:33
*** rmallah has joined #puppet-openstack14:34
mfischEmilienM: c'est bon merci14:37
*** ducttape_ has joined #puppet-openstack14:41
*** rmart04 has joined #puppet-openstack14:42
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Add ml2 package requirement to neutron::plugins::ml2::mech_driver  https://review.openstack.org/30114214:44
*** chandankumar has quit IRC14:48
*** chandankumar has joined #puppet-openstack14:48
*** xingchao has quit IRC14:48
*** EmilienM changes topic to "CI status: RED: openstack-infra/zuul is experiencing critical issues that prevents CI to run - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"14:49
*** emccormick has joined #puppet-openstack14:52
*** xarses has joined #puppet-openstack14:53
*** xingchao has joined #puppet-openstack14:54
*** chandankumar is now known as chandankumar|afk14:55
*** magicboiz has left #puppet-openstack15:01
openstackgerritValery Tschopp proposed openstack/puppet-vswitch: add missing dev keyword to ip link command  https://review.openstack.org/30117615:05
*** EmilienM changes topic to "CI status: GREEN - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"15:06
*** saneax is now known as saneax_AFK15:07
degorenkoEmilienM, hey, do you know anything about this commit: https://review.openstack.org/#/c/276103/ ?15:07
EmilienMdegorenko: I don't15:08
degorenko:(15:08
degorenkois this guy here in irc?15:08
openstackgerritPradeep Kilambi proposed openstack/puppet-gnocchi: Add sync_db param to gnocchi db class  https://review.openstack.org/30060415:08
*** alee has joined #puppet-openstack15:12
*** leanderthal is now known as leanderthal|afk15:15
*** aimon has joined #puppet-openstack15:15
EmilienMgood news: zuul is back15:18
EmilienMdo not recheck, it's useless15:18
degorenkomflobo, ping15:20
*** aimon has quit IRC15:20
*** jmikolajczak has quit IRC15:22
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: enable IPv6 on centos7  https://review.openstack.org/30118415:25
*** mdorman has joined #puppet-openstack15:26
*** aimon has joined #puppet-openstack15:29
*** fedexo has joined #puppet-openstack15:37
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/30107915:37
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::concurrency define  https://review.openstack.org/30105715:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: enable IPv6 on centos7  https://review.openstack.org/30118415:41
*** jistr has quit IRC15:41
EmilienMdegorenko: fyi, tempest plugins are going to land, so we open more testing now: murano, zaqar, barbican, etc etc...15:43
degorenkoEmilienM, i've updated today patch for murano15:44
EmilienMexcellent, I'll look15:44
EmilienMI'm working on zaqar15:44
EmilienMthanks for your help on zaqar btw15:44
degorenkono problem15:44
degorenkoi have a few questions15:44
degorenkofor prev patch set15:44
degorenkotake a look on them15:44
EmilienMoops zaqar fails, and it's something in zaqar it seems15:45
EmilienMI'm going on #openstack-zaqar15:45
mwhahahadegorenko: is there anything specific you'd like addressed in https://review.openstack.org/#/c/298718/ before it can be merged?15:46
* degorenko looking15:46
*** jaosorior has quit IRC15:46
openstackgerritMerged openstack/puppet-openstack-integration: scenario003: enable SSL by default  https://review.openstack.org/29960515:47
*** jaosorior has joined #puppet-openstack15:47
EmilienMdegorenko: I'll do the gnocchi approach, using a case15:47
degorenkomwhahaha, for https://review.openstack.org/#/c/298718/7/spec/acceptance/basic_glance_spec.rb agree for checking properties, only this. Otherwise lgtm15:47
EmilienMdegorenko: so we send a warning on ubuntu nodes to say zaqar is not deployed15:47
*** jaosorior has quit IRC15:48
mwhahahak i'll update that15:48
*** jaosorior has joined #puppet-openstack15:48
degorenkoEmilienM, we have not murano for rdo, but here is some problems on ubuntu, i will check15:48
EmilienMdegorenko: err, in fact no gnocchi does not have a case, it also use if else15:48
EmilienMdegorenko: no murano in rdo?15:49
degorenkoEmilienM, it is on review i guess (STILL)15:49
EmilienMdegorenko: link?15:49
EmilienMdegorenko: I can review it15:49
degorenkohttps://github.com/openstack-packages/murano15:49
degorenkofor review, let me find15:50
EmilienMdegorenko: link of review15:50
*** xingchao has quit IRC15:51
degorenkoEmilienM, i can't find it15:51
degorenkogoing to #rdo15:51
rmallahcan anyone point to some discussion or quickly tell the main reason behind deprecating   https://github.com/stackforge/puppet-openstack16:00
*** cwolferh has joined #puppet-openstack16:01
*** dgurtner_ has quit IRC16:04
*** f0f4r has quit IRC16:05
*** aimon has quit IRC16:07
*** aimon has joined #puppet-openstack16:08
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Fix comparison to control_virtual_ip  https://review.openstack.org/30121616:10
*** foxhunt72 is now known as foxhunt_away16:12
*** chandankumar|afk is now known as chandankumar16:12
*** foxhunt_away is now known as foxhunt7216:13
*** rmallah has quit IRC16:19
EmilienMdegorenko: I know why murano fails16:20
EmilienMyou're missing all the logic to have ssl / ipv616:20
degorenkoEmilienM, scenario3 is working only with ssl / ipv6?16:21
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Fix comparison to control_virtual_ip  https://review.openstack.org/30121616:21
EmilienMdegorenko: right, indeed16:21
EmilienMdegorenko: I'm looking at logs16:21
EmilienMdegorenko: btw, now scenario003 is on ssl and ipv6 is coming up16:21
degorenkoEmilienM, ok, i will update it then16:21
*** fedexo has quit IRC16:22
EmilienMdegorenko: and tempest is failing: http://logs.openstack.org/36/250436/25/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/036b944/console.html#_2016-04-04_11_00_17_11716:22
EmilienMit's probably a bug in murano16:22
EmilienMI see the same in Zaqar16:23
EmilienMdoes murano run tempest plugin tests in gate?16:23
degorenkoit should16:23
EmilienMdegorenko: ImportError: No module named murano_tempest_tests.plugin16:25
degorenkoEmilienM, patch for ssl for scenario003 was merged after my tests failed16:25
EmilienMdegorenko: right, the failure is : ImportError: No module named murano_tempest_tests.plugin16:25
degorenkoso probably bug in murano16:25
degorenkoi've asked our murano guys16:25
degorenkowaiting for reply16:25
EmilienMdegorenko: cool, I'm taking care of zaqar16:29
*** rmallah has joined #puppet-openstack16:31
*** rmallah has quit IRC16:33
*** v1k0d3n has quit IRC16:44
*** v1k0d3n has joined #puppet-openstack16:45
*** aimon has quit IRC16:46
*** aimon has joined #puppet-openstack16:47
*** rcernin has quit IRC16:48
openstackgerritIvan Chavero proposed openstack/puppet-neutron: Add ml2 package requirement to neutron::plugins::ml2::mech_driver  https://review.openstack.org/30114216:49
*** chandankumar has quit IRC16:53
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Do not raise error when ca_file is not passed  https://review.openstack.org/30124016:55
*** foxhunt72 is now known as foxhunt_away16:55
*** f0f4r has joined #puppet-openstack16:57
degorenkoEmilienM, i know why murano is failed16:58
*** dgurtner has joined #puppet-openstack16:59
*** chandankumar has joined #puppet-openstack16:59
*** rahulait has joined #puppet-openstack17:02
*** rmart04 has quit IRC17:03
*** paramite has quit IRC17:04
openstackgerritMykyta Karpin proposed openstack/puppet-glance: [WIP] Switch glance to os_service_default facts  https://review.openstack.org/29982217:09
openstackgerritMykyta Karpin proposed openstack/puppet-glance: [WIP] Switch glance to os_service_default facts  https://review.openstack.org/29982217:12
*** dgurtner has quit IRC17:16
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Add ability use ssl for Rabbitmq and for Murano RabbitMQ  https://review.openstack.org/30125317:17
*** trown is now known as trown|lunch17:20
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/25043617:20
EmilienMdegorenko, mwhahaha: infra is about to land https://review.openstack.org/#/c/301251/ this week. Since I'm afk the next 2 days, please report any issue related to centos7 jobs to pabelanger17:39
*** aimon has quit IRC17:39
mwhahahak17:39
degorenkoack17:39
*** aimon has joined #puppet-openstack17:39
degorenkoEmilienM, for murano: the problem is in this patch https://review.openstack.org/#/c/297647/ it is not in rc1, available only for rc317:39
EmilienMdegorenko: ok so ubuntu will update packages soon when they release17:41
EmilienMdegorenko: in the meantime, we can start testing on RDO17:41
degorenkoEmilienM, ack, then we will wait. Btw, i've updated patch for murano and we have 2 additional patches17:41
degorenkohttps://review.openstack.org/301240 & https://review.openstack.org/30125317:41
EmilienMexcellent17:42
mwhahahadegorenko: were you looking at the ec2api module init patch?17:43
degorenkomwhahaha, [openstack-dev] [puppet][ec2api] - Official EC2 puppet module repo and https://review.openstack.org/#/c/276103/17:43
mwhahahayea i know, i saw it last week and was thinking of picking it up since it had been sitting for a while17:44
mwhahahawe should probably split that patch up so we could land it :D17:44
degorenkomwhahaha, i've asked mflobo - he said that will update patch on this week17:44
mwhahahasince it used a really old cookiecutter it might be beneficial to something more like what baribican was doing17:45
degorenkomay be yes17:45
degorenkoyou can share your thoughts in mail tread btw17:45
*** jpena is now known as jpena|off17:46
mwhahahayea i'll send a note17:46
degorenkonice17:46
*** rahulait has quit IRC17:47
*** dgurtner has joined #puppet-openstack17:49
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Add ability use ssl for Rabbitmq and for Murano RabbitMQ  https://review.openstack.org/30125317:49
*** foxhunt_away is now known as foxhunt7217:52
EmilienMmjblack: are you sure this bug is valid? https://bugs.launchpad.net/puppet-neutron/+bug/156575417:53
openstackLaunchpad bug 1565754 in puppet-neutron "add support for lbaas v2" [Undecided,New]17:53
openstackgerritAlex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/29871817:56
*** angdraug has joined #puppet-openstack17:58
mjblackEmilienM: its not valid, I didnt know lbaasv2 was in a different location17:58
mjblackEmilienM: I'm going to be submitting a new bug report for cinder...looks like devs removed ssl from cinder-api wsgi process17:58
mjblackhttps://review.openstack.org/#/c/219671/17:59
EmilienMwhen I added SSL to our CI, cinder was not working.17:59
*** dtantsur is now known as dtantsur|afk18:01
mjblackEmilienM: for mitaka?18:01
*** aimon has quit IRC18:01
EmilienMyes18:01
mjblackThey removed the ssl options18:01
*** aimon has joined #puppet-openstack18:02
mjblackthey have in the config reference an ssl section but from what I can tell its not even being looked at on startup18:02
*** degorenko is now known as _degorenko|afk18:06
aleeEmilienM, _degorenko|afk hey - I just want to clarify the whole $os_service_default thing again.  When we set a parameter to this value, we are trying to use whatever upstream uses.18:09
_degorenko|afkalee, yes :)18:09
_degorenko|afkhey18:09
aleenow in terms of a config file like barbican.conf, for instance, that means that we end up commenting out or eliminating whatever value is there ?  or we do nothing at all to that setting in barbican.conf?18:10
*** dgurtner has quit IRC18:11
*** gfidente has quit IRC18:12
alee_degorenko|afk, EmilienM - what seems to be happening is that we are actually removing the parameter from barbican.conf - as if the only thing that is there is effectively overrides.18:12
aleeand then any upstream defaults are supposed to be hardcoded into code18:13
aleeis that the intention?18:13
EmilienMalee: os_service_default does not remove anything18:16
aleeEmilienM, ok - so the intention is that -- when the puppet modules apply -- if we see os_service_default, then that parameter is left alone?18:17
aleeand should maintain whatever value was delivered in the file by the rpm package?18:17
aleeEmilienM, I want to confirm that before I start hunting down what happened.18:19
*** Matum has joined #puppet-openstack18:21
EmilienMalee: https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack_config/ini_setting.rb#L21-L2318:21
EmilienMwe ensure the resource is absent in config file18:21
EmilienMso barbican.conf won't have the value and rely on what is set in code18:21
*** trown|lunch is now known as trown18:23
*** Matum has quit IRC18:23
aleeEmilienM, ok - so it does remove whatever value is in barbican.conf18:23
EmilienMalee: if a value is managed by puppet and set to os_service_default, yes18:24
aleeEmilienM, ok - so if there is a value which upstream barbican sets only with a default in barbican.conf, and not in code - then we cannot use os_service_default.18:25
*** chandankumar has quit IRC18:25
EmilienMalee: this is a barbican bug18:26
EmilienMall other projects set default values in their code18:26
aleeEmilienM, and theoretically that should be a bug in upstream barbican18:26
aleegotcha18:26
EmilienMit's a mistake to set default value in config file only18:26
EmilienMconfig files should be generated by oslo config generator18:26
EmilienMand if you don't do that, that's also a bug18:26
aleeEmilienM, ok thats good to know -- need to look into that.18:27
EmilienMalee: http://docs.openstack.org/developer/oslo.config/generator.html18:27
_odyEmilienM: Do you know when RDO and UCA will certify their mitaka repos as "released"?18:29
openstackgerritMatthew J Black proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796418:29
*** angdraug has quit IRC18:29
EmilienM_ody: no but soon18:29
_odyOk. I'll keep an eye on it.  Just don't desire releasing to the forge until then.18:30
EmilienM_ody: why that?18:30
EmilienM_ody: stable/mitaka already deploys by default the stable repos (they just don't exist yet)18:31
EmilienMour CI is pin on repos that work now18:31
_odyWell.  What happens if the modules are released and they can't find packages?18:32
EmilienMin my experience, at every summit we ask who is using Puppetlabs forge, and it's very rare to see someone raising hand18:38
EmilienMbut yeah, you can wait.18:38
_odyWho uses it at the summit is completely beside the point.18:40
EmilienM_ody: yeah, exactly. The point here is to wait for a release date. Our code has been released, and repos needs to be created, we don't handle that, so we move forward.18:42
EmilienMwe decided to not wait for stable repos to release our modules18:42
EmilienMbecause it's something we don't actually manage18:43
_odyDoesn't that result in a broken release?18:43
EmilienMthe code that setups the repo is in openstack_extras, so yes, you can say "8.0.0 release of openstack_extras is broken until packaging is fixed"18:44
EmilienMif you have a suggestion, I'm very open to it18:44
*** darkxploit has joined #puppet-openstack18:46
EmilienMmjblack: I'll also let you rebase https://review.openstack.org/#/c/288091/18:46
mjblackEmilienM: which beaker test did it fail with?18:47
EmilienMmjblack: I'll WIP https://review.openstack.org/#/c/287964/ until 288091 pass CI.18:47
EmilienMbeaker actually pass, because we don't test cinder API18:47
EmilienMbut it actually does not work, if you look at https://review.openstack.org/#/c/288091/18:47
EmilienMmjblack: where you able to run cinder in WSGI?18:51
*** f0f4r has quit IRC18:52
_odyEmilienM: Not realy.  Seems like a documentation issue since everyone pretty much has to rebuild a OpenStack package or two to go production.  I'd say highlight it in the guide repo we just launched.18:52
*** angdraug has joined #puppet-openstack18:57
mjblackEmilienM: I will test it out in a few in my environment18:58
EmilienMmjblack: the -1 is just to make sure we don't land it before it's tested.18:59
mjblackI'd run the beaker scenario but I'm having issues with my laptop here19:00
EmilienMmjblack: use cloud ;-)19:00
mjblacklol havent played with beaker, an easy way to point it to an openstack environment?19:03
EmilienMmjblack: http://my1.fr/blog/puppet-module-functional-testing-with-vagrant-openstack-and-beaker/19:08
*** emccormickva has joined #puppet-openstack19:11
*** emccormick has quit IRC19:14
*** imcsk8 is now known as imcsk8|lunch19:15
EmilienM_ody: I'm rebasing https://review.openstack.org/#/c/296557/ and updating list of dependencies19:20
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655719:23
EmilienM_ody: rebased & commented19:24
_odyEmilienM: Your last comment could be considered out of scope for that review.  Those requires are I just implementing the already present dependencies in a different way. I am more inclined to just remove those from the change.19:32
_odyThe only ones that really need to added are the ones on the execs.19:32
EmilienMexactly19:32
EmilienM_ody: I just want to drop the requires on glance & neutron providers19:33
EmilienMmhh19:33
*** derekh has quit IRC19:34
_odyEmilienM: yeah.  I'll remove the requires and we can work on removing the arrow chaing in a differen review.19:34
EmilienM_ody: well, it will probably fail19:34
EmilienMI guess you can keep it19:34
EmilienM_ody: what was the issue with: Keystone_user_role['admin@openstack'] -> Neutron_network<||> ?19:34
EmilienMit does not work anymore with puppet4?19:35
mwhahahadid they removed --quiet from the openstack client? http://logs.openstack.org/18/298718/8/check/gate-puppet-glance-puppet-beaker-rspec-ubuntu-trusty/73967e0/console.html#_2016-04-04_18_57_08_03119:36
mwhahahaor is it the trailing space or something? ic an't figure out why that throws an error now19:37
EmilienMmwhahaha: same result on centosè?19:38
mwhahahayea19:38
EmilienMcentos7?19:38
EmilienMmhhok19:38
mwhahahait was working and i updated the tests and now it's puking19:38
EmilienMcan you give a date when it worked?19:38
mwhahahahttp://logs.openstack.org/18/298718/8/check/gate-puppet-glance-puppet-beaker-rspec-devstack-centos7/1691928/console.html#_2016-04-04_18_09_15_44519:39
EmilienMso we can analize the repos19:39
mwhahahahttps://review.openstack.org/#/c/298718/ is the review19:39
mwhahahalast patch was on the 30th19:40
EmilienMI don't see anything related in osclient yet19:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: enable IPv6 on centos7  https://review.openstack.org/30118419:46
mwhahahai bet it's the space at the end19:49
mwhahahaor maybe there's a nonprintable char19:50
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: Set OS specific init service providers  https://review.openstack.org/29977319:52
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: Set OS specific init service providers  https://review.openstack.org/29977319:54
EmilienM_ody: on the puppet patch, what is weird is that it's only failing on beaker/centos7 but integration scenario is working fine19:54
*** jaosorior has quit IRC19:57
EmilienMjust by switching to the new provider19:59
EmilienM_ody: are you aware of any new requirement to run this service provider? a specific package?19:59
*** amoralej has quit IRC20:02
openstackgerritAlex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/29871820:07
*** Matum has joined #puppet-openstack20:17
_odyEmilienM: I also found it quite weird and why I had to pass it on.  There is nothing in puppet I know that explains what i happening.  The only other required package is rehdat-lsb-core but TBH that kinda needs to exist for puppet to work properly in the first place anyway.20:18
_odyPeople depend on quite a few facts that are also derived from that package.20:18
EmilienM_ody: yeah rehdat-lsb-core is by default in our nodes20:18
_odyAs for "Keystone_user_role['admin@openstack'] -> Neutron_network<||>".  I think it is better to not create dependencies on resource inside classes that are outside the current module.  It creates a reliance on implementation unnecessarily.  Depending on the class just simple abstracts away the need of knowing what the admin user is or that we even use a resource named keystone_user_role.20:22
_odyIt makes any unknown future deprecations easier.20:23
EmilienM_ody: right, I 100% agree on that, let's keep your patch like is it20:27
*** twm2016 has joined #puppet-openstack20:28
_odyEmilienM: Now that we've talked about, I'd like to split it up.  It isn't a Puppet 4 requirement.20:29
EmilienM_ody: sure20:30
twm2016furlongm: I made a comment on your reported bug, https://bugs.launchpad.net/puppet-swift/+bug/155483520:30
openstackLaunchpad bug 1554835 in puppet-swift "missing configuration options for EC / object-reconstructor" [Medium,Confirmed]20:30
twm2016I think this is related https://review.openstack.org/#/c/264966/ isn't it?20:30
*** jayg is now known as jayg|g0n320:31
EmilienMcrinkle: I addressed some comments on https://review.openstack.org/#/c/300696/ - do you have some? did you want to keep the Puppetfile in place?20:32
crinkleEmilienM: looking20:35
*** johnmilton has quit IRC20:36
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655720:36
EmilienMwe'll need to add support for stable branches20:36
EmilienMfor openstack modules20:36
openstackgerritMerged openstack/puppet-tempest: deploy openstackclient and make our providers requiring it  https://review.openstack.org/30116220:38
*** dgurtner has joined #puppet-openstack20:38
*** dgurtner has quit IRC20:38
*** dgurtner has joined #puppet-openstack20:38
crinkleEmilienM: hmm i feel like we could store the lists from the *.txt files in some kind of partial puppetfile20:41
crinkleEmilienM: but if we're moving towards having a job that updates a puppetfile then that will be even better20:42
crinkleEmilienM: i like it20:42
openstackgerritMerged openstack/puppet-tempest: Manage Neutron AAS test packages when Neutron is available  https://review.openstack.org/30078620:42
EmilienMcrinkle: yes, I'm currently working on a patch in openstack-infra that will run periodically and propose changes.20:44
EmilienMcrinkle: which means, Puppetfile stays in the openstack-integration but the script I wrote might move to project-config or something20:45
EmilienMdoes it make sense?20:45
crinkleEmilienM: yep20:45
EmilienMthx20:45
*** dprince has quit IRC20:46
openstackgerritAlex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/29871820:57
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: enable IPv6 on centos7  https://review.openstack.org/30118421:10
mwhahahalooks like our beaker tests don't like it when you have > 1 glance image21:16
mwhahahai assume it's a delay in the image processing in glance causing some sync issues when we run it immediately after21:16
*** trown is now known as trown|outtypewww21:16
*** Guest29628 has quit IRC21:18
EmilienMmwhahaha: don't we have a retry in the provider?21:20
mwhahahanot completely sure because it's not obvious21:20
EmilienMmwhahaha: our integration jobs have multiple images, cirros and cirros_alt21:21
EmilienMdo they fail too?21:21
mwhahahano21:21
mwhahahajust the beaker21:21
mwhahahalike i said i think it's because of the idempotency check21:22
twm2016Hi everyone, I'm new to puppet and am trying to deploy my own OpenStack cloud using puppet All-In-One.21:34
twm2016It seems I am getting errors related to my works proxy. With GET calls for ruby files.21:35
twm2016Should I look for help here or in openstack-infra?21:36
EmilienMtwm2016: which doc are you following?21:37
*** foxhunt72 is now known as foxhunt_away21:37
twm2016EmilienM: https://github.com/openstack/puppet-openstack-integration#all-in-one21:37
*** brandon has joined #puppet-openstack21:37
EmilienMtwm2016: ok, what kind of proxy errors? they should be related to your network21:38
*** richm has joined #puppet-openstack21:38
twm2016EmilienM: This is the output from my console: http://paste.openstack.org/show/492931/21:39
EmilienMGET http://rubygems.org/gems/bundler-1.11.2.gem21:39
EmilienM302 Not Allowed21:39
EmilienMit sounds like your network is not allowing to download gems from http://rubygems.org21:40
twm2016I looked for a work around online and I didn't find anything.21:40
EmilienMthere is actually one21:41
EmilienMopen Gemfile and change GEM_SOURCE, to find another mirror21:41
twm2016Okay, I will try that. Thank you.21:43
*** imcsk8|lunch is now known as imcsk821:45
*** haplo37 has quit IRC21:51
*** dgurtner has quit IRC21:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Implement a script to generate Puppetfile  https://review.openstack.org/30069621:57
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Implement a script to generate Puppetfile  https://review.openstack.org/30069622:00
mwhahahaEmilienM: https://review.openstack.org/#/c/298718 before you got on vacation :D22:04
EmilienMmwhahaha: ok I'll look this one  before leaving in far canadian north22:05
mwhahahadon't get eaten by a polar bear22:05
EmilienMmwhahaha: I'll eat them22:06
bkeroEmilienM is a polar bear hunter22:06
EmilienMhopefully not22:07
bkeroWild polar bears of Montreal22:07
bkero...are they French polar bears?22:07
EmilienMbkero: I don't live in MTL !!22:08
bkero...Québécois polar bears?22:08
EmilienMbkero: in mtl, it's more beers :P22:08
bkeroRight. Mmm Unibroue22:08
*** dhollinger has quit IRC22:11
*** ducttape_ has quit IRC22:12
twm2016EmilienM: I have tried modifying the Gemfile like you said but I haven't had any results. Something interesting I found was that I get the same error even if I delete the Gemfile entirely.22:14
EmilienMtwm2016: you need to run bundle update22:14
EmilienM(I think)22:14
EmilienMcrinkle: here we go! https://review.openstack.org/30137522:17
crinkleEmilienM: woot22:17
*** twm2016 has quit IRC22:25
*** morazi has quit IRC22:29
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: enable IPv6 on centos7  https://review.openstack.org/30118422:31
*** johneax has joined #puppet-openstack22:36
*** johneax has quit IRC22:37
*** markvoelker has quit IRC22:37
EmilienM_ody: lol it pass https://review.openstack.org/#/c/299773/22:44
EmilienMmwhahaha: can you +A this one?22:44
mwhahahayea22:44
EmilienMI don't know why it failed before though22:44
mwhahahawhat no tests?22:44
EmilienMlet's see ceph tests22:45
EmilienMno coverage for that22:45
mwhahahai know that's why i asked :D22:47
xarsesEmilienM: anything change with the centos image?22:50
EmilienMxarses: probably22:50
EmilienMbut go figure22:50
* xarses sighs22:50
*** mdorman has quit IRC22:53
xarsesI'm fine with the no coverage, the service providers need to all be re-done to systemd support anyways22:54
mwhahahak22:54
EmilienMprobably puppet-swift will need updates for puppet422:55
xarsesWhich was going to be my next step in making this work, but apparently EmilienM has to one up me with a rebase22:55
xarseswell played...22:55
_odyEmilienM: ...23:02
_odyWell then.23:03
_odySigh.23:03
EmilienM_ody: what was the last blockers for puppet4?23:03
EmilienM_ody: your last patchset is not passing: https://review.openstack.org/#/c/296557/23:03
_odyI am betting typo23:04
EmilienMtypo is my life23:04
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655723:05
_odySurprised my syntax checking didn't catch that.23:05
EmilienM_ody: so you took over my last patchset and missed a Depends-On the puppet-ceph- can you add it again?23:06
EmilienM_ody: if you want to checkout a patchset, just run "git review -d 296557" so you'll get the changes23:07
_odyyep.  Simply forgot you'd rebased and such.23:07
EmilienM_ody: you can drop all the others23:08
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/30138323:09
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655723:10
*** ducttape_ has joined #puppet-openstack23:26
*** ducttape_ has quit IRC23:28
EmilienM_ody: still syntax http://logs.openstack.org/57/296557/28/check/gate-puppet-openstack-integration-3-scenario001-tempest-devstack-centos7/24600aa/console.html#_2016-04-04_23_15_06_46223:29
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655723:36
*** markvoelker has joined #puppet-openstack23:37
*** xarses has quit IRC23:38
openstackgerritMerged openstack/puppet-ceph: Set OS specific init service providers  https://review.openstack.org/29977323:46
openstackgerritMerged openstack/puppet-openstack-integration: Run Tempest plugins  https://review.openstack.org/28378623:48
EmilienMmwhahaha, _ody: this one is I think, ready to go: https://review.openstack.org/#/c/301184/23:53
EmilienMcheers guys, see you on Thursday :-)23:53
mwhahahaLater23:54
*** angdraug has quit IRC23:58

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