Tuesday, 2015-06-02

openstackgerritColleen Murphy proposed stackforge/puppet-nova: Added parameters for availability zones configuration  https://review.openstack.org/18739600:02
*** xarses has joined #puppet-openstack00:08
openstackgerritColleen Murphy proposed stackforge/puppet-nova: Correct docs on format for nova::policy data  https://review.openstack.org/18739800:08
*** alexpilotti has joined #puppet-openstack00:09
openstackgerritColleen Murphy proposed stackforge/puppet-nova: Remove Python Package Declaration  https://review.openstack.org/18740000:09
*** rcallawa_ has joined #puppet-openstack00:09
alexpilottihi guys, does anybody know when puppet-nova Kilo is going to be ready?00:10
openstackgerritColleen Murphy proposed stackforge/puppet-nova: crontab: ensure nova-common is installed before  https://review.openstack.org/18740100:11
*** xarses has quit IRC00:12
*** mdorman has quit IRC00:12
*** rcallawa_ has quit IRC00:17
*** xarses has joined #puppet-openstack00:18
*** rcallawa_ has joined #puppet-openstack00:18
*** rcallaw__ has joined #puppet-openstack00:25
*** rcallawa_ has quit IRC00:28
*** ducttape_ has joined #puppet-openstack01:10
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Add native types for keystone paste configuration  https://review.openstack.org/18741201:16
alexpilottimgagne: hi01:20
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Add paste_config optional parameter  https://review.openstack.org/18741401:32
*** tchaypo has quit IRC01:35
*** simonmcc has quit IRC01:35
*** petems has quit IRC01:36
*** petems has joined #puppet-openstack01:40
*** tchaypo has joined #puppet-openstack01:40
*** simonmcc has joined #puppet-openstack01:42
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Fix deprecated LDAP config options  https://review.openstack.org/18741601:42
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Set WSGI process display-name  https://review.openstack.org/18741701:44
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Rename resource instance variable  https://review.openstack.org/18741801:48
openstackgerritColleen Murphy proposed stackforge/puppet-keystone: Switch the example URL to a fake address  https://review.openstack.org/18741901:51
openstackgerritColleen Murphy proposed stackforge/puppet-heat: added option to configure flavor in heat.conf  https://review.openstack.org/18742101:56
*** alexpilotti has quit IRC02:00
openstackgerritColleen Murphy proposed stackforge/puppet-heat: Fix Keystone domain class  https://review.openstack.org/18742302:05
openstackgerritColleen Murphy proposed stackforge/puppet-heat: Fix ipv6 support  https://review.openstack.org/18742402:09
*** ducttape_ has quit IRC02:18
*** gildub has quit IRC02:24
*** ducttape_ has joined #puppet-openstack02:32
*** richm has quit IRC02:47
*** gildub has joined #puppet-openstack03:10
*** ducttape_ has quit IRC03:22
openstackgerritMerged stackforge/puppet-nova: templates: Fix variable prepending  https://review.openstack.org/18013803:24
openstackgerritMerged stackforge/puppet-nova: spec: convert remaining deprecated keywords  https://review.openstack.org/18389703:25
*** ducttape_ has joined #puppet-openstack03:27
*** fedexo has joined #puppet-openstack03:33
*** chem has quit IRC03:53
*** ducttape_ has quit IRC03:54
*** ducttape_ has joined #puppet-openstack03:55
*** ducttape_ has quit IRC04:00
*** aimon has quit IRC04:02
*** aimon has joined #puppet-openstack04:02
*** britthou_ has joined #puppet-openstack04:03
*** britthouser has quit IRC04:05
openstackgerritMatt Fischer proposed stackforge/puppet-cinder: Don't add a new line if the rbd_user changes  https://review.openstack.org/18744804:08
*** chem has joined #puppet-openstack04:11
openstackgerritMatt Fischer proposed stackforge/puppet-designate: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/18745104:14
*** 16WAA4FX8 has quit IRC04:46
*** delattec has joined #puppet-openstack04:47
*** gildub has quit IRC04:48
*** ducttape_ has joined #puppet-openstack04:56
*** ducttape_ has quit IRC05:01
*** sks has joined #puppet-openstack05:03
*** cwolferh has quit IRC05:11
*** cwolferh has joined #puppet-openstack05:13
*** stevemar has quit IRC05:31
*** reed has quit IRC05:33
*** reed has joined #puppet-openstack05:47
*** tfz has quit IRC05:49
*** tshtilma has joined #puppet-openstack06:12
*** gfidente has quit IRC06:14
*** khappone has quit IRC06:26
*** rlaurika has quit IRC06:27
*** rlaurika has joined #puppet-openstack06:28
*** khappone has joined #puppet-openstack06:31
*** ducttape_ has joined #puppet-openstack06:45
*** tfz has joined #puppet-openstack06:46
*** ducttape_ has quit IRC06:50
*** dgurtner has joined #puppet-openstack07:16
*** dgurtner has quit IRC07:16
*** dgurtner has joined #puppet-openstack07:16
*** dalgaaf has joined #puppet-openstack07:17
*** panda|ko has quit IRC07:21
*** panda|ko has joined #puppet-openstack07:22
*** jpena has joined #puppet-openstack07:28
*** jistr has joined #puppet-openstack07:48
*** paramite has joined #puppet-openstack07:50
*** chem has quit IRC07:52
*** fedexo has quit IRC07:53
*** beddari1 has quit IRC07:56
*** beddari1 has joined #puppet-openstack07:57
*** ducttape_ has joined #puppet-openstack08:01
*** ducttape_ has quit IRC08:06
*** derekh has joined #puppet-openstack08:19
*** todin has joined #puppet-openstack08:31
todinhi08:31
todindoes the puppet-neutron support for service_providers entries like lbaas08:32
*** imcsk8 is now known as imcsk8|zZz08:40
*** paramite is now known as paramite|afk08:51
*** kbyrne has joined #puppet-openstack08:53
*** paramite|afk is now known as paramite09:05
*** paramite is now known as paramite|afk09:20
*** cdent has joined #puppet-openstack09:21
*** ducttape_ has joined #puppet-openstack09:50
*** ducttape_ has quit IRC09:55
*** dalgaaf has quit IRC09:58
openstackgerritsac proposed stackforge/puppet-manila: Add share driver options.  https://review.openstack.org/18669910:12
*** dgurtner has quit IRC10:22
*** rcallaw__ has quit IRC10:31
*** rcallawa_ has joined #puppet-openstack10:32
*** paramite|afk is now known as paramite10:33
openstackgerritsac proposed stackforge/puppet-manila: Add share driver options.  https://review.openstack.org/18669910:34
*** jpena has quit IRC10:36
*** alexpilotti has joined #puppet-openstack10:46
*** jpena has joined #puppet-openstack10:48
*** dgurtner has joined #puppet-openstack10:51
*** dgurtner has quit IRC10:51
*** dgurtner has joined #puppet-openstack10:51
*** ducttape_ has joined #puppet-openstack10:54
*** kevin has joined #puppet-openstack11:06
*** kevin is now known as Guest1640711:06
*** kkirkpatrick has quit IRC11:09
*** panda|ko has quit IRC11:21
*** rushiagr_away is now known as rushiagr11:22
*** igajsin has joined #puppet-openstack11:22
*** igajsin has quit IRC11:22
*** panda|ko has joined #puppet-openstack11:22
*** beddari1 has quit IRC11:26
*** Guest16407 has quit IRC11:30
*** Guest16407 has joined #puppet-openstack11:31
*** paramite is now known as paramite|afk11:35
*** paramite|afk is now known as paramite11:37
*** ducttape_ has quit IRC11:40
*** delattec has quit IRC11:42
*** dprince has joined #puppet-openstack11:48
*** paramite has quit IRC11:55
*** rcallawa_ has quit IRC12:03
*** ducttape_ has joined #puppet-openstack12:04
*** jayg|g0n3 is now known as jayg12:06
*** ducttape_ has quit IRC12:08
*** paramite has joined #puppet-openstack12:09
*** cdelatte has joined #puppet-openstack12:09
*** jistr is now known as jistr|class12:17
*** bogdando has quit IRC12:18
*** jpena is now known as jpena|lunch12:19
*** jistr|class is now known as jistr12:22
*** bogdando has joined #puppet-openstack12:23
*** paramite has quit IRC12:44
*** ahcorporto has joined #puppet-openstack12:49
*** rcallawa_ has joined #puppet-openstack12:50
*** ducttape_ has joined #puppet-openstack12:51
*** paramite has joined #puppet-openstack12:57
alexpilottimgagne: ping12:58
*** ducttape_ has quit IRC12:58
*** iurygregory has quit IRC12:59
*** tshtilma has quit IRC13:03
*** tfhartmann has joined #puppet-openstack13:03
*** sks has quit IRC13:03
*** ahcorporto1 has joined #puppet-openstack13:05
*** ahcorporto has quit IRC13:08
*** sks has joined #puppet-openstack13:17
*** richm has joined #puppet-openstack13:23
*** ducttape_ has joined #puppet-openstack13:27
*** ducttape_ has quit IRC13:27
*** jpena|lunch is now known as jpena13:28
*** tfhartmann has quit IRC13:40
*** rushiagr is now known as rushiagr_away13:41
*** tfhartmann has joined #puppet-openstack13:42
*** tfhartmann has quit IRC13:43
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Use node IP lists for HA Proxy ipaddresses  https://review.openstack.org/18677513:44
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Allow use of ssl for public api endpoints  https://review.openstack.org/17595813:44
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Configure virtual IPs for split out networks  https://review.openstack.org/18624013:44
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Add interface_for_ip function  https://review.openstack.org/18677613:44
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Stop keystone admin from listening on public interface  https://review.openstack.org/18737113:44
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Drop nova_meta and glance_registry on public vip  https://review.openstack.org/18758813:44
*** tfhartmann has joined #puppet-openstack13:47
*** tfhartmann has quit IRC13:48
*** tshtilma has joined #puppet-openstack13:53
openstackgerritDan Prince proposed stackforge/puppet-tripleo: Configure virtual IPs for split out networks  https://review.openstack.org/18624013:57
*** tfhartmann has joined #puppet-openstack14:02
*** tfhartmann has quit IRC14:03
*** chem has joined #puppet-openstack14:08
*** Guest16407 has quit IRC14:13
*** xingchao has joined #puppet-openstack14:13
*** ducttape_ has joined #puppet-openstack14:14
*** ToMiles has joined #puppet-openstack14:16
*** jistr is now known as jistr|mtg14:16
*** ericpeterson has joined #puppet-openstack14:18
*** stevemar has joined #puppet-openstack14:23
*** tfhartmann has joined #puppet-openstack14:23
*** ducttape_ has quit IRC14:23
*** tfz has quit IRC14:29
*** sks has quit IRC14:30
*** jistr|mtg is now known as jistr14:30
*** Guest16407 has joined #puppet-openstack14:46
*** guimaluf has joined #puppet-openstack14:48
openstackgerritMalik Bougacha proposed stackforge/puppet-glance: adding support for min ram and min disk  https://review.openstack.org/18761814:52
crinklemorning14:54
*** paramite is now known as paramite|afk14:54
*** dfisher has joined #puppet-openstack14:56
spredzyhello14:59
claytonmeeting today?15:00
crinkleyep15:00
*** paramite|afk is now known as paramite15:02
Hunner_ody: meeting?15:02
crinkle_ody: are you around? meeting is in #openstack-meeting-415:02
*** rcallaw__ has joined #puppet-openstack15:05
*** rcallawa_ has quit IRC15:08
*** tfz has joined #puppet-openstack15:09
*** mdorman has joined #puppet-openstack15:13
dfishergah.15:14
*** tfz has quit IRC15:19
*** panda|ko has quit IRC15:21
*** panda|ko has joined #puppet-openstack15:21
*** xingchao has quit IRC15:23
*** tfz has joined #puppet-openstack15:34
*** openstackgerrit has quit IRC15:42
*** openstackgerrit has joined #puppet-openstack15:43
*** tristanC has left #puppet-openstack15:46
_odyOk...must make coffee and food...15:54
* _ody takes laptop to kitchen for meeting15:54
HunnerThe beauty of an irc meeting :)16:00
*** tfhartmann has quit IRC16:00
*** rcallaw__ has quit IRC16:02
*** rcallawa_ has joined #puppet-openstack16:02
mdormancrap, totally missed the meeting16:04
mdormanyet another outlook calendar notificatoin fail16:04
*** ahcorporto1 has left #puppet-openstack16:04
*** jistr has quit IRC16:05
*** tfhartmann has joined #puppet-openstack16:08
*** ToMiles has quit IRC16:12
*** dgurtner has quit IRC16:13
_odymdorman: Everyone says they same about Google Calendar too16:14
mdormanyeah i don’t have good luck with taht either16:15
mdormannot sure why this is such a hard problem16:15
*** tfhartmann has quit IRC16:18
*** dprince has quit IRC16:18
*** paramite has quit IRC16:18
*** aimon has quit IRC16:21
*** tfhartmann has joined #puppet-openstack16:23
*** xingchao has joined #puppet-openstack16:24
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - add domains to service_identity  https://review.openstack.org/18058616:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: wip - support for keystone v3 api - the user_role resource  https://review.openstack.org/17499916:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - v3 domain utility code  https://review.openstack.org/17838516:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin  https://review.openstack.org/17506016:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: wip -support for keystone v3 api - the user resource  https://review.openstack.org/17497616:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: wip - support for keystone v3 api - the project resource  https://review.openstack.org/17446816:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: wip - support for keystone v3 api - beaker tests  https://review.openstack.org/17882816:25
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the domain resource  https://review.openstack.org/17443916:25
*** xingchao has quit IRC16:28
*** tfhartmann has quit IRC16:33
*** mattymo has joined #puppet-openstack16:35
*** dprince has joined #puppet-openstack16:36
*** jpena has quit IRC16:38
*** _ody has quit IRC16:39
*** _ody has joined #puppet-openstack16:40
*** xarses has quit IRC16:41
*** _ody has quit IRC16:42
dfisherwell, awesome.  keystone, glance, heat, and cinder (with a Solaris-specific cinder driver) all working on solaris.16:43
*** _ody has joined #puppet-openstack16:43
*** iurygregory has joined #puppet-openstack16:50
*** aimon has joined #puppet-openstack17:04
*** derekh has quit IRC17:06
dfisherquick puppet manifest question.  I need to run a command and save that value to use later to set things for nova.  Is there a magic incantation to do this?17:09
*** tshtilma has quit IRC17:09
Hunnerdfisher: On the agent side?17:14
HunnerAnd in nova for nova?17:14
HunnerCan it be a fact?17:14
dfisherhmm.17:15
*** tshtilma has joined #puppet-openstack17:15
dfishergood point.  will check facter17:15
openstackgerritMerged stackforge/puppet-glance: Correct docs on format for glance::policy data  https://review.openstack.org/18732017:15
dfisherif it's not in facter, what's the right way to do this?17:15
*** rushiagr_away is now known as rushiagr17:17
openstackgerritMerged stackforge/puppet-neutron: Correct docs on format for neutron::policy data  https://review.openstack.org/18738317:17
Hunnerdfisher: It depends on the use case. When does the command need to run? Are there any pre-steps that need to happen before the fact runs?17:17
HunnerWhat uses the value? How many places is it used?17:18
dfishernope.  just need the output from psrinfo -p17:18
Hunnerah17:18
dfisherused for all the workers configurations17:18
openstackgerritMerged stackforge/puppet-nova: Correct docs on format for nova::policy data  https://review.openstack.org/18739817:18
HunnerThen yeah, the module can add a custom fact17:18
dfisherok.17:18
HunnerOr you can make a facter PR :)17:18
dfisher:)17:18
dfisherok, will go from there.   thanks, Hunner17:18
HunnerThe puppet community hangout is going, if you want to join and ask about contributing to facter17:19
HunnerFrom #puppet-dev:17:19
Hunner09:55       peterhu | Puppet Community PR triage is starting in about 10 minutes: http://links.puppetlabs.com/pr-triage17:19
HunnerEvery week at this time17:19
dfisheroh.  maybe!17:20
*** xarses has joined #puppet-openstack17:20
dfisheri have so many things running right now.  can't make this one, sadly17:24
*** tshtilma has quit IRC17:28
*** tshtilma has joined #puppet-openstack17:28
*** cdent has quit IRC17:40
*** tshtilma has quit IRC17:49
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Use node IP lists for HA Proxy ipaddresses  https://review.openstack.org/18677517:54
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Allow use of ssl for public api endpoints  https://review.openstack.org/17595817:54
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Add interface_for_ip function  https://review.openstack.org/18677617:55
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Stop keystone admin from listening on public interface  https://review.openstack.org/18737117:55
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Drop nova_meta and glance_registry on public vip  https://review.openstack.org/18758817:55
openstackgerritRichard Megginson proposed stackforge/puppet-openstacklib: support for Keystone v3 authentication  https://review.openstack.org/18765417:55
openstackgerritMalik Bougacha proposed stackforge/puppet-glance: adding support for min ram and min disk  https://review.openstack.org/18770417:59
*** tfhartmann has joined #puppet-openstack18:09
openstackgerritMalik Bougacha proposed stackforge/puppet-glance: adding support for min ram and min disk  https://review.openstack.org/18770918:09
*** openstackgerrit has quit IRC18:09
*** openstackgerrit has joined #puppet-openstack18:10
*** imcsk8|zZz is now known as imcsk818:44
*** alexpilotti has quit IRC18:52
*** openstackgerrit has quit IRC18:56
*** openstackgerrit has joined #puppet-openstack18:56
*** tfhartmann has quit IRC18:58
*** tfhartmann has joined #puppet-openstack19:00
*** rushiagr is now known as rushiagr_away19:01
*** ToMiles has joined #puppet-openstack19:08
dfisherHunner:  turns out that facter already had the data I needed.  Didn't need a custom fact at all.19:12
dfisheron x86:19:13
dfisherroot@openstack-x52:/# facter physicalprocessorcount processorcount19:13
dfisherphysicalprocessorcount => 219:13
dfisherprocessorcount => 3219:13
dfisheron sparc:19:14
dfisher$ facter physicalprocessorcount processorcount19:14
dfisherphysicalprocessorcount => 119:14
dfisherprocessorcount => 12819:14
dfisherdo you like cores?  sparc has cores.19:14
*** cdent has joined #puppet-openstack19:21
*** aimon has quit IRC19:34
*** aimon_ has joined #puppet-openstack19:35
mgagneI can't believe we aren't using pick() more than that19:41
*** jtomasek has quit IRC19:48
*** cdent has quit IRC19:56
richmWhat's the status of the template for new projects?  One of the Barbican devs wants to create a puppet-barbican19:59
*** tfhartmann has quit IRC20:00
*** alee has joined #puppet-openstack20:00
dfishergot a bit of a manifest issue with Nova and Solaris here.  Anybody got a sec to look at http://paste.openstack.org/show/257183/ ?20:01
richmspredzy: I guess this would be the cookiecutter work?20:04
spredzyrichm, yes it is. https://github.com/enovance/cookiecutter-openstack-puppet-modules20:06
spredzythis + msync on top of it and you should have a well started modules for you20:07
*** nosleep77 has joined #puppet-openstack20:07
richmspredzy: thanks!20:08
spredzyrichm, yw20:08
*** openstackgerrit has quit IRC20:10
*** openstackgerrit has joined #puppet-openstack20:10
openstackgerritMatt Fischer proposed stackforge/puppet-nova: Remove comment for non-existant variable  https://review.openstack.org/18775220:15
mfischhere's a trivial if someone wants review points ^20:15
*** rcallaw__ has joined #puppet-openstack20:17
* crinkle noms points20:19
openstackgerritColleen Murphy proposed stackforge/puppet-neutron: Allow l3_ha to be disabled  https://review.openstack.org/18738920:19
*** jtomasek has joined #puppet-openstack20:19
mfischI like the essex reference20:19
*** rcallawa_ has quit IRC20:21
*** rcallaw__ has quit IRC20:21
*** cdelatte has quit IRC20:22
*** dprince has quit IRC20:22
openstackgerritColleen Murphy proposed stackforge/puppet-neutron: Add MidoNet plugin manifest  https://review.openstack.org/18739220:24
dfisheranybody got any ideas with my fun error above?20:26
mfischdfisher: my uninformed opinion is that you need to include ::nova::vncproxy before referencing any variables in it20:28
*** aimon_ has quit IRC20:28
dfisherignore those.20:28
dfisherthe last line.20:28
dfisherError: Could not find dependency Package[nova-compute] for Service[nova-compute] at /etc/puppet/modules/nova/manifests/generic_service.pp:6020:28
*** aimon has joined #puppet-openstack20:29
mfischlooks like params issue but yours seems ok20:29
mfischwonder if there is some code referencing that package direct by name20:29
*** aimon_ has joined #puppet-openstack20:30
dfisheri can paste a —verbose —debug dump too but I don't see much in there.20:30
mfischI found some code, but its inside if debian20:30
mfischI need to step away for a bit20:30
dfisherno worries.  i'm not going anywhere20:30
dfisherso if I change https://github.com/stackforge/puppet-nova/blob/master/manifests/generic_service.pp#L54 to be 'service { $service_name:   and L59 to be 'require => [Package['nova-common'], Package[$pacakge_name]] … everything seems to work20:34
*** aimon has quit IRC20:34
dfishernot really sure why $nova-title is used instead of the passed in $package_name / $service_names20:34
*** jayg is now known as jayg|g0n320:35
dfisherseems there was a cset which specifically changed it to what it is now though :/20:41
openstackgerritColleen Murphy proposed stackforge/puppet-nova: Added parameters for availability zones configuration  https://review.openstack.org/18739620:53
*** ToMiles has quit IRC20:54
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Use node IP lists for HA Proxy ipaddresses  https://review.openstack.org/18677520:56
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Allow use of ssl for public api endpoints  https://review.openstack.org/17595820:57
openstackgerritBen Nemec proposed stackforge/puppet-tripleo: Drop nova_meta and glance_registry on public vip  https://review.openstack.org/18758820:57
*** iurygregory has quit IRC21:01
*** ducttape_ has joined #puppet-openstack21:09
*** ericpeterson has quit IRC21:12
mgagnedfisher: I think $nova_title is used to get a predictable service and package resource title. Their names are overriden with $package_name and $service_name which are platform specific.21:14
dfisherbut line 23 in that file hardcodes … right?21:15
dfisher$nova_title = "nova-${name}"21:15
mgagnedfisher: it's dynamically built since the define name is the nova service name without the nova- prefix (for some reason)21:16
dfishermy head hurts.21:17
dfisher:(21:17
mgagnedfisher: with puppet, there is 2 different concepts: the resource title (a unique identifier) and the namevar (usually the name attribute)21:19
mgagnedfisher: both are unique but the later is usually platform specific.21:19
mgagnedfisher: since you need to still be able to refer those resources from elsewhere, you usually use a static/constant resource title which isn't platform specific.21:20
mgagnedfisher: with generic_service, the title ($nova_title) is always (for example) nova-compute, not the actual package name which is platform specific (nova-compute on Debian platforms and openstack-nova-compute on RedHat platforms)21:21
mgagnedfisher: this allows you to always be able to find the resource from other manifests using a "standard" resource title (nova-compute) instead of having to "guess" the actual package name the platform uses.21:21
dfisherok.  that all (somewhat) makes sense.21:22
dfisherwhat I'm not seeing is why the require isn't on the specific package name though :(21:23
mgagnedfisher: the package "name" found in the require is the title, not the name21:23
* dfisher nods21:24
dfisherok.21:24
mgagnedfisher: I guess you could use the name but don't =)21:24
* dfisher nods21:24
dfisherman, getting nova working with all the solaris-isms is … unfun.21:24
mgagnedfisher: better use a non-platform specific identifier so you don't have to include nova::params to fetch the actual package name to build your require21:24
mgagnedfisher: how is solaris a challenge? I'm not familiar with this plaform21:25
dfisher^ not sure what you mean on that one21:25
mgagnedfisher: for example: https://github.com/stackforge/puppet-nova/blob/master/manifests/compute/libvirt.pp#L11821:25
dfisheroh, lookie there.21:25
*** rcallawa_ has joined #puppet-openstack21:26
dfisherwell, mostly that in solaris we have a single package for all of Nova.21:26
dfisher(so far)21:26
dfisherlinux seems to like a crazy amount of granularity21:26
mgagnedfisher: without nova-compute as a resource *title*, you would have to include the nova::params class, fetch the package_name of the platform and dynamically building a resource name like Package[$::nova::params::compute_package_name].21:26
mgagnedfisher: oh god, I'm so sorry21:26
* dfisher ^H^H^H^H in api.pp21:27
dfisherno, it's really ok.  i mean, it's trying to balance how linux has 345236 nova packages and solaris has 1.21:27
*** rcallawa_ has quit IRC21:27
dfisheralso, s/nova/<any openstack component>21:27
mgagnesure21:27
mgagnetbh, I wouldn't know how to add support for solaris without refactoring the whole module21:28
dfisherso far it's not really been that bad.21:28
*** rcallawa_ has joined #puppet-openstack21:28
dfisherwe'll see at code review :)21:28
dfisheri need to slightly tweak a few small things and that's really it.21:28
dfisherand deliver a custom compute manifest, obv.21:28
mgagneyes, we have similar challenges with some platforms with glance which combines api and registry in the same package.21:28
dfisherglance causes me fits.  just in general.  not specifically with puppet21:29
dfisherstupid glance.21:29
dfisher"it's just a list of URLs, guys."21:29
mgagnehttps://github.com/stackforge/puppet-glance/blob/master/manifests/registry.pp#L165-L17221:29
dfisherayup :)21:29
dfisherheat patch:21:30
dfisher+  if ( $heat::params::engine_package_name != $heat::params::common_package_name ) {21:30
dfisher+    ensure_packages([$heat::params::engine_package_name],21:30
dfisher+      {21:30
dfisher+        ensure => $package_ensure,21:30
dfisher+        name   => $::heat::params::engine_package_name,21:30
dfisher+        tag    => ['openstack'],21:30
dfisher+      }21:30
dfisher+    )21:30
dfisher+  }21:30
dfisher+21:30
dfisherlook familar? :)21:30
mgagneyep21:30
* dfisher sad21:30
mgagneit's awful and I think we can do better21:30
dfisherat this point, i'm sort of just slamming code in to try to make this function21:30
dfisherhere's another fun one21:30
dfisherwe have an SMF service for doing foo-manage db sync21:31
dfisherwe don't call it directly21:31
dfisherso, the db_sync addition to openstacklib that's been proposed will impact us … i think?21:31
mgagneI think we should split all package installation in its own class so all the dirty logic is circumscribed in there21:31
dfishersolaris is easy-peasy:  pkg install nova21:31
dfisheraaaand done.21:31
mgagnedfisher: can you explain how SMF influence db sync? is it some kind of wrapper?21:31
dfisheryup21:32
dfisherthat's all it is21:32
dfisher    <exec_method timeout_seconds="60" type="method" name="start"21:32
dfisher      exec="/usr/bin/cinder-manage db sync">21:32
dfisher      <method_context>21:32
dfisher        <method_credential user='cinder' group='cinder' />21:32
dfisher      </method_context>21:32
dfisher    </exec_method>21:32
dfisherrun that command as that user/group21:32
dfisherand exit21:32
dfisherdone21:32
mgagnehow do you invoke it?21:32
mgagneecho | smf ? o_O21:33
dfisherboot :)21:33
mgagneyou sync at each boot?21:33
dfisherseems to be harmless21:33
dfisherso far :)21:34
mgagneI beg to differ21:34
dfisherohdear :)21:34
mgagnetry it with ceilometer for instance21:34
dfisherhey, we don't pkg ceilometer! :)21:34
* dfisher phew21:34
mgagneor neutron between grizzly -> havana21:35
mgagneunless solaris figured that one out21:35
dfisherhey, we only have one release at a time! :)21:35
dfisher:P21:35
mgagneor you tell me that solaris only support havana and up21:35
dfisherjuno!21:35
mgagnegod bless you21:36
dfisherwe added code to automatically upgrade people from havana to juno21:36
mgagne:O21:37
dfisher:D21:37
dfishersee, when you have have ZFS, you can get away with murder!21:37
dfisheroh, your entire OS is hosed?  *rolls back to previous environment*  *reboot*.  30 seconds later, we're back21:37
mgagneoh true, I forgot about that one21:37
dfisher:)21:37
mgagneok so the challenges so far are: unique package name (unlike linux distro) and SMF used to db sync at boot21:37
dfisheri'll have a blog post on this entire process "soon"21:37
dfisherso, the SMF thing, you shouldn't worry about21:38
dfisheri need to submit a PR against upstream puppet21:38
dfisheri can add a flag to puppet's invocation21:38
dfisherand everything gets fixed.21:38
dfisher--- puppet-3.6.2/lib/puppet/provider/service/smf.rb.orig        2015-06-02 10:28:10.870219833 -060021:38
dfisher+++ puppet-3.6.2/lib/puppet/provider/service/smf.rb      2015-06-02 10:28:19.325013006 -060021:38
dfisher@@ -70,7 +70,7 @@ Puppet::Type.type(:service).provide :smf21:38
dfisher     when :maintenance21:38
dfisher       [command(:adm), :clear, @resource[:name]]21:38
dfisher     else21:38
dfisher-      [command(:adm), :enable, "-s", @resource[:name]]21:38
dfisher+      [command(:adm), :enable, "-rs", @resource[:name]]21:38
dfisher     end21:38
dfisher   end21:38
dfisherSMF's dependencies will take care of the db sync service21:39
*** Guest16407 has quit IRC21:46
*** aimon_ has quit IRC21:49
*** aimon has joined #puppet-openstack21:49
*** jtomasek has quit IRC21:51
*** aimon_ has joined #puppet-openstack21:52
*** aimon has quit IRC21:54
mgagnewhy do we need keystone::python in nova again? https://github.com/stackforge/puppet-nova/blob/master/manifests/api.pp#L18821:55
stevemarmgagne, is that the keystoneclient python lib?21:56
mgagnestevemar: haven't been the case for 2 years I think21:57
mgagnestevemar: and I think nova-api package depends on it anyway21:57
stevemarmgagne, sorry, my question was simpler, i'm a ruby noob, what's require ::keystone::python mean ? :)21:58
stevemarmgagne, but yes, i can confirm that nova probably pulls it in, and should only need keystonemiddleware anyway21:58
*** alee has quit IRC21:58
mgagnestevemar: it means every resources found in the class will automatically requires this resource21:59
crinklestevemar: it includes this class https://github.com/stackforge/puppet-keystone/blob/master/manifests/python.pp which installs this package https://github.com/stackforge/puppet-keystone/blob/master/manifests/params.pp#L521:59
crinklei can't imagine why nova needs that...22:00
stevemarcrinkle, ahhh i get it22:00
stevemaryeah probably doesn't :\22:00
crinklemaybe it was to set up the nova keystone_user ?22:00
stevemarcrinkle, thats the only relationship it should have, but it doesn't look like https://github.com/stackforge/puppet-keystone/blob/master/manifests/params.pp#L5 does that22:03
*** petems has quit IRC22:06
dfisherok, what's22:22
dfisher  Nova_paste_api_ini<| |> ~> Service['nova-api']22:22
dfisherdo?22:22
crinkleit gathers every nova_paste_api_ini resource in the entire catalog, "realizes" it if necessary, and prompts the nova-api service to refresh if any of the nova_paste_api_ini resources are being updated22:25
crinklehttps://docs.puppetlabs.com/puppet/latest/reference/lang_collectors.html22:25
*** chem has quit IRC22:27
dfishergotcha.22:28
dfisherthanks22:28
*** petems has joined #puppet-openstack22:42
*** ducttape_ has quit IRC22:44
*** xingchao has joined #puppet-openstack22:58
dfisherugh.  package dependencies in puppet manifests makes for sad.23:04
*** dfisher has quit IRC23:09
*** stevemar has quit IRC23:12
*** stevemar has joined #puppet-openstack23:12
*** aimon_ has quit IRC23:15
*** markvoelker_ has quit IRC23:18
*** xarses has quit IRC23:19
*** panda|ko has quit IRC23:21
*** panda|ko has joined #puppet-openstack23:22
*** gildub has joined #puppet-openstack23:38
*** markvoelker has joined #puppet-openstack23:40
*** alee has joined #puppet-openstack23:40
*** aimon has joined #puppet-openstack23:43

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