Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-12-05

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
23:00 bauzas joined #puppet-openstack
23:02 rmoe joined #puppet-openstack
23:14 wscheele joined #puppet-openstack
23:14 eglynn-afk joined #puppet-openstack
23:19 wscheele i keep running into: "Conflict between puppetlabs/apache/0.8.0 <git://github.com/puppetlabs/puppetlabs-apache#origin/0.x> and puppetlabs/apache (< 1.0.0, >= 0.9.0) <http://forge.puppetlabs.com>" when running librarian-puppet (on master as well as stable/havana). anyone recognize this or know how to resolve?
23:24 michchap joined #puppet-openstack
23:26 pabelanger joined #puppet-openstack
23:28 thumpba joined #puppet-openstack
23:47 eglynn-afk joined #puppet-openstack
23:52 mgagne wscheele: which Puppetfile are you using?
23:53 mgagne wscheele: this error means 2 (or more) modules have incompatible dependencies
23:56 rongze joined #puppet-openstack
00:01 wscheele hi mgagne, thanks. i came a bit further and managed to run through now. i think the problem is a combination of a bug in librarian-puppet with a missing dependency version requirement in the Modulefile of puppetlabs-git.
00:02 sputnik13net joined #puppet-openstack
00:02 tnoor1 joined #puppet-openstack
00:03 wscheele i ended using a master clone of librarian-puppet and patching that to add a default >=0 requirement when no requirement is specified, that allowed librarian-puppet to run through.
00:03 wscheele ow, i also had to remove the :ref from the apache definition in the Puppetfile
00:04 xarses joined #puppet-openstack
00:04 wscheele that was the original error i encountered (dependency on 0.9.0 of apache but the :ref provided only supplied 0.8.0)
00:04 wscheele this was all against stable/havana
00:05 tnoor2 joined #puppet-openstack
00:06 openstackgerrit joined #puppet-openstack
00:09 brdude joined #puppet-openstack
00:24 sputnik13net anyone familiar with setting up spice via puppet?
00:25 sputnik13net it seems like puppet-openstack doesn't support spice yet, and the nova::compute::spice module doesn't seem to do anything more than configuring nova with the right spice parameters
00:25 sputnik13net but there's nothing that manages installing spice nor the spice proxy
00:25 sputnik13net is this about accurate?
00:25 sputnik13net or am I missing something?
00:27 mgagne sputnik13net: TBH, I don't know how to should be setup. https://review.openstack.org/#/c/26852/
00:28 mgagne sputnik13net: EmilienM said it was working great. Better ask him how he tested it =)
00:28 mgagne sputnik13net: I guess this part is handled by nova::spicehtml5proxy
00:30 rharrison sputnik13net, is there a separate module, outside of packstack, that fills those requirements.  I haven't checked.
00:35 sputnik13net rharrison: packstack?  I don't know what that is :(
00:36 rharrison sputnik13net, another name for the openstack modules.  the puppetlabs-openstack modules are all pulled from the modules in the OpenStack project proper.
00:37 sputnik13net rharrison: not that I've found
00:38 rharrison sputnik13net, darn, shot in the dark...
00:38 sputnik13net rharrison: well, admittedly I haven't dug a lot :)
00:39 sputnik13net mgagne: yes that would seem to be the proxy module but on ubuntu there is no spicehtml5proxy, it's called nova-spiceproxy
00:39 sputnik13net mgagne: so I think there's still some work needed on the module
00:41 mgagne sputnik13net: with havana right?
00:41 mgagne sputnik13net: and I guess it will still be the same for icehouse
00:41 sputnik13net mgagne: I would assume so
00:41 mgagne sputnik13net: ok, I'll check into it
00:42 mgagne sputnik13net: it looks to be the same in grizzly...
00:42 mgagne it's nova-spiceproxy
00:43 rharrison mgagne, is that the module name?
00:43 mgagne rharrison: according to packages found in Ubuntu Cloud Archive, the package is named nova-spiceproxy
00:43 mgagne http://ubuntu-cloud.archive.canonical.com/ubuntu/pool/main/n/nova/
00:44 rharrison mgagne, ah gotcha
00:44 mgagne manifest should be renamed too while we are at it...
00:44 rharrison mgagne, I don't use Ubuntu for anything so I'll check the RHN channels.
00:45 mgagne nova::spicehtml5proxy -> nova::spiceproxy
00:46 rharrison mgagne, that name move seems to be a good idea. Is there another version of the spice proxy that isn't html5?  I wonder why they used such a convoluted name to begin with.
00:47 mgagne damn, this issue has been there since the beginning....
00:49 mgagne https://bugs.launchpad.net/puppet-nova/+bug/1257984
00:50 mgagne gotta have to update all projects to create icehouse series =)
00:51 sputnik13net well html5 is when websockets was added
00:53 mgagne config is still named spicehtml5proxy_host in nova
01:08 badiane_ka joined #puppet-openstack
01:17 xmir joined #puppet-openstack
01:22 mgagne working on the bug btw
01:24 dmsimard joined #puppet-openstack
01:28 rongze joined #puppet-openstack
01:33 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Fix package name for nova-spiceproxy on Ubuntu  https://review.openstack.org/60125
01:38 prad_ joined #puppet-openstack
01:52 prad_ joined #puppet-openstack
02:12 sputnik13net joined #puppet-openstack
02:20 mgagne EmilienM: ping
02:21 * mgagne has wishful thinking
02:24 openstackgerrit joined #puppet-openstack
02:33 xingchao joined #puppet-openstack
02:37 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Use database_ prefix for database parameters  https://review.openstack.org/60134
02:52 dmsimard joined #puppet-openstack
02:53 rongze joined #puppet-openstack
02:54 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Add Modulefile  https://review.openstack.org/60137
02:55 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Use database_ prefix for database parameters  https://review.openstack.org/60134
02:55 rongze_ joined #puppet-openstack
02:55 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Attempt to apply common rspec style  https://review.openstack.org/60139
03:10 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Ensure database backend package is installed  https://review.openstack.org/60142
03:39 tnoor1 joined #puppet-openstack
03:43 tnoor1 joined #puppet-openstack
03:46 tnoor1 joined #puppet-openstack
04:24 michchap_ joined #puppet-openstack
04:35 michchap joined #puppet-openstack
04:51 badiane_ka joined #puppet-openstack
04:52 otherwiseguy joined #puppet-openstack
05:03 marun joined #puppet-openstack
05:15 mgagne1 joined #puppet-openstack
05:15 mgagne1 joined #puppet-openstack
05:22 xarses joined #puppet-openstack
05:29 tnoor1 joined #puppet-openstack
05:49 xingchao joined #puppet-openstack
05:57 xingchao joined #puppet-openstack
06:44 jkyle joined #puppet-openstack
06:54 rongze joined #puppet-openstack
07:20 EmilienM good morning
07:20 EmilienM mgagne1: pong
07:21 EmilienM wow, puppet-ironic is on stackforge \o/
07:55 sputnik13net joined #puppet-openstack
08:14 openstackgerrit A change was merged to stackforge/puppet-cinder: Fix default Swift URL in Backup service  https://review.openstack.org/60013
08:27 openstackgerrit A change was merged to stackforge/puppet-keystone: Create memcache_servers option  https://review.openstack.org/57361
08:28 mattymo mgagne1, are you there?
08:30 bauzas joined #puppet-openstack
08:53 mmagr joined #puppet-openstack
09:03 eglynn-afk joined #puppet-openstack
10:13 derekh joined #puppet-openstack
10:31 michchap joined #puppet-openstack
10:31 fc__ joined #puppet-openstack
10:55 EmilienM mattymo: too early :) he is on the east coast
11:09 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ironic: Delete travis QA service  https://review.openstack.org/60204
11:15 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: TTL mechanism support with expirer service  https://review.openstack.org/60029
11:19 EmilienM i wondering why https://review.openstack.org/60204 fails
11:19 EmilienM sbadia: fc__ ^ any idea ?
11:20 fc__ EmilienM: http://logs.openstack.org/04/60204/1/check/gate-puppet-ironic-puppet-lint/dd1f260/console.html
11:21 fc__ EmilienM: ERROR: ironic::api not in autoload module layout on line 39
11:25 EmilienM fc__: yeah i know
11:25 EmilienM fc__: but why
11:25 EmilienM fc__: it fails only from this patch
11:25 EmilienM fc__: mgagne1 did a patch already and it did not fail afik
11:38 sbadia EmilienM: hum it's me or, git hash in the log correspond to… nothing in stackforge/puppet-ironic ?
11:39 sbadia EmilienM: HEAD is now at c6825ac Use database_ prefix for database parameters
11:39 sbadia EmilienM: we don't have this commit in the tree
11:40 sbadia EmilienM: error on gerrit config ?
11:48 morazi joined #puppet-openstack
11:57 rongze joined #puppet-openstack
12:06 morazi joined #puppet-openstack
12:32 LAnthony joined #puppet-openstack
12:33 xingchao joined #puppet-openstack
12:35 rongze joined #puppet-openstack
12:35 openstackgerrit Anthony Leprêtre proposed a change to stackforge/puppet-ceph: Update "README.md" and "USECASES.md" (I want to operate a production cluster)  https://review.openstack.org/60219
12:39 Jean-Roger Hi dachary, we would need a review https://review.openstack.org/60219 when you have time
12:40 * dachary checking
12:41 rongze_ joined #puppet-openstack
12:43 dachary Jean-Roger: you should have rebased against https://review.openstack.org/#/c/59091/
12:44 dachary can you abandon https://review.openstack.org/#/c/59091/ to clarify that it no longer is worked on ?
12:44 LAnthony we had a lot of problems to commit
12:44 Jean-Roger LAnthony screwed up with git, we're notifying the abandon
12:45 dachary ok
12:45 LAnthony Done !
12:45 Jean-Roger Done, can you confirm ?
12:46 dachary i confirm
12:47 dachary you will need to be approved by another core member ( dalgaaf_ mgagne bodepd dmsimard ) but it looks good to me
12:47 dachary Jean-Roger: ^
12:48 Jean-Roger Good, waiting for approval
12:49 LAnthony :)
12:53 openstackgerrit Sandro Mathys proposed a change to stackforge/puppet-nova: Configure libvirt migration on the RedHat and Debian  https://review.openstack.org/52364
13:09 openstackgerrit A change was merged to stackforge/puppet-nova: Fix package name for nova-spiceproxy on Ubuntu  https://review.openstack.org/60125
13:10 EmilienM need backport ^
13:13 prad joined #puppet-openstack
13:26 dachary dalgaaf_: sorry for nagging you about https://review.openstack.org/#/c/56841/ : it's perfectly fine if you don't have time for it. I'm just making sure it is not overlooked. It went thru so many revisions that it may be unclear it is ready for review, for good :-)
13:35 EmilienM thanks mmagr :)
13:36 mmagr no problem
13:39 openstackgerrit A change was merged to stackforge/puppet-swift: Fixing remaining typo in the README  https://review.openstack.org/59844
13:59 dprince joined #puppet-openstack
14:06 morazi joined #puppet-openstack
14:23 prad joined #puppet-openstack
14:31 otherwiseguy joined #puppet-openstack
15:03 dmsimard joined #puppet-openstack
15:30 mjblack joined #puppet-openstack
15:53 mattymo maybe mgagne1 will appear soon?
16:01 dmsimard joined #puppet-openstack
16:18 tnoor1 joined #puppet-openstack
16:18 mgagne mattymo: I'm here
16:19 mgagne mattymo: not anymore =) afk for 15m
16:19 mattymo noooo
16:19 mattymo mgagne, come back soon :)
16:20 tnoor2 joined #puppet-openstack
16:42 mattymo mgagne, ?
16:45 mgagne mattymo: back
16:47 mattymo mgagne, logging in cinder
16:47 mgagne yes
16:47 mgagne ?
16:47 mattymo you dislike our approach a lot, but bogdando and I have been working on a better approach
16:47 mgagne mattymo: go on
16:48 mattymo https://github.com/mattymo/puppet-cinder/tree/logging2
16:48 brdude joined #puppet-openstack
16:48 mattymo just a sec
16:49 mattymo we moved more variables to class params, made cinder_log_conf param, and discovered there's a log_config_append option now in Havana
16:49 mattymo it existed in Grizzly for a couple components but it's in all for Havana
16:49 mattymo append only overrides defaults and the default log_config file
16:50 mattymo so it should be compatible with any distro customizations and the template is completely optional
16:51 mgagne How can someone configure its own log_config_append?
16:51 mattymo log_config_append would point to a file /etc/cinder/logging.conf
16:51 mattymo this file is specified in params.pp
16:52 mgagne mattymo: by default right?
16:52 mattymo and the file would be filled by a tempalte
16:52 mattymo no by default this value is absent (as it is in ubuntu and rdo) and no template gets used to populate such a file
16:52 mgagne mattymo: is there an implicit file location used?
16:53 mattymo yes /etc/cinder/logging.conf is specified in params.pp for $::cinder::params::cinder_log_conf
16:53 mgagne mattymo: is /etc/cinder/logging.conf becoming some sort of "example" file until log_config_append is configured?
16:53 mattymo yes
16:53 mgagne mattymo: implicit file location from the POV of cinder
16:53 mattymo it can be changed to any other name of course
16:53 mattymo I'm sorry, but can you reword that?
16:54 mgagne mattymo: (lets exclude puppet from the question) if no log_config_append is provided, is /etc/cinder/logging.conf used by cinder?
16:54 mattymo no
16:54 mgagne mattymo: so until log_config_append is configured in cinder.conf, logging.conf is some sort of example file of what could be done/configured.
16:55 mattymo yes
16:55 mattymo https://github.com/openstack/oslo-incubator/blob/master/openstack/common/log.py
16:55 mattymo in here there are config options and defaults prescribed that get used
16:55 mattymo and those match what's in the default logging.conf
16:55 mgagne mattymo: with this version of cinder::logging, how can someone configure his own logging.conf using log_config_append? (and not use the template)
16:56 mattymo just a sec
16:58 bogdando mgagne: log_config_append is only a file name referenced. The file name is a standard python logging config
16:58 mattymo in https://github.com/mattymo/puppet-cinder/blob/d5d8573f11f92ef2c40ea9941bcd47356652bc62/manifests/logging.pp $log_config is a variable passed to cinder (and then to cinder::logging) that matches a template file in cinder/templates
16:58 mattymo and we have a template that can be used https://github.com/mattymo/puppet-cinder/blob/d5d8573f11f92ef2c40ea9941bcd47356652bc62/templates/logging_syslog.conf.erb
16:59 bogdando if user wants to use some files as logging config he can tell puppet to use template
16:59 bogdando s/files/file
16:59 mgagne mattymo, bogdando: how can I inject MY own template without having to patch puppet-cinder? The location of the template is hardcoded to the cinder module.
17:00 mgagne template or file
17:00 mattymo good point mgagne. it should be the entire variable space and we shouldn't force it
17:00 mattymo I feel this should be a template and not an API provider because this file is provided in ubuntu, but not RDO, and to do an INI provider for this file would require at least 100 lines of puppet code to popular this file
17:01 mgagne mattymo: therefore, the use case is: I want to be able to disable the puppet-cinder logging template and use my own in puppet-whatever_internal.
17:01 bogdando mgagne: I believe the steps should be: 1) create a foo.erb - a template file for logging conf, 2) put it to puppet-cinder/templates, 3) call cinder class usign log_config => foo.erb
17:01 LAnthony joined #puppet-openstack
17:01 bogdando nothing is hardcoded
17:01 mgagne bogdando: this is not something easily automated. puppet-cinder/templates is owned by the module and shouldn't contain local (to your setup) files.
17:02 mattymo log_config => 'my-module/cinder_logging.erb'
17:02 mgagne mattymo: try it and see the result
17:02 bogdando mattymo, actually user should not care about path, but only filename
17:02 mgagne mattymo: I would prefer this approach: https://github.com/stackforge/puppet-horizon/blob/master/manifests/init.pp#L89
17:03 mgagne mattymo: this approach allow me to use my own template if I feel the one provided by puppet-horizon isn't flexible enough to my taste.
17:03 LAnthony left #puppet-openstack
17:04 bogdando mgagne, hm, but we do use this approach, except the #{modulename} in the value? should we add it?
17:04 mgagne bogdando: why not?
17:04 mattymo mgagne, ok I definitely support this route too, but you seemed opposed to introducing default changes to logging settings for Cinder
17:08 mattymo mgagne, this file has similarities in that in future OpenStack releases the format and parameters of this file may change
17:08 mattymo just like with horizon settings you linked to
17:08 mgagne mattymo: this is a middle ground. People seems to expect puppet-cinder to be able to configure logging.conf. (although I feel this is a slipy slope) The 1st version replaced logging.conf by its own version without offering an option to opt-out to the end user. The 2st version kinds of allow the end user to opt out of it if $log_config is false. It however prevents him from using his own by explicitly setting log_config_append to absent. This
17:08 mattymo you got cut off... at "absent. This m"
17:09 mgagne to absent. This means the user can't configure this option using an other internal puppet module of his own.
17:10 mgagne mattymo: it removes flexibility to the end user.
17:10 mattymo mgagne, I appreciate your consideration. We don't have experience with users supplying separate templates/files for modules because we operate in a vacuum
17:11 mgagne mattymo: I operate in my own vacuum too =)
17:11 mattymo but that is no trouble for us and bogdando will update the request tomorrow
17:11 mattymo I just wanted to get an opinion from you before we try to propose more evil code
17:11 angdraug joined #puppet-openstack
17:12 mattymo mgagne, also I met your colleague in Hong Kong. He came from a recent acquisition for iWeb
17:12 mgagne mattymo: was it Raj?
17:12 mattymo he was french canadian, a bit tall, and bald
17:13 mgagne mattymo: oh, he is next time, he works with me ;)
17:13 mgagne mattymo: it's Boris and he is not Russian
17:13 mattymo yes!
17:13 mattymo I wanted to speak Russian to him after he said Boris
17:13 mgagne haha
17:14 mattymo I'm American but I live in Moscow
17:14 mgagne mattymo: templating config files is evil unless you can opt out or find a way to do it "the right way" (something I haven't found yet). Templating removes flexibility and adds the burden of this flexility to the (our) puppet module and this is something we should avoid.
17:15 mattymo no prob
17:15 mattymo mgagne, are you located in Montreal?
17:15 mgagne mattymo: yep
17:15 mattymo awesome. I was there only once but I need to go back some day
17:16 mgagne mattymo: hehe, hopefully not during a snowstorm ;)
17:16 mgagne mattymo: where you at the puppet meeting in HK?
17:16 mattymo yes. I sat in the back corner
17:16 mattymo but I had to leave a little early
17:20 mgagne mattymo: left back corner with a black t-shirt? https://twitter.com/mgagne/status/398375375895416833/photo/1/large
17:21 mattymo yes with the beard)
17:21 mgagne hehe
17:21 mattymo I work for Mirantis and we have Fuel, the irresponsible fork of puppet-openstack
17:21 mattymo and I'm trying to fix things
17:22 mgagne mattymo: hehe
17:22 mgagne mattymo: we do have our own puppet-openstack too ;)
17:23 mattymo does yours diverge from upstream by over 80,000 lines?
17:23 mgagne mattymo: 100%, it only has the same name =)
17:24 mattymo I mean puppet-*.. nova, cinder, neutron, glance, cinder, rabbitmq, qpid, mysql, etc
17:24 mgagne mattymo: and we do have an other internal module used to install controllers, nodes and other services.
17:24 mgagne mattymo: we use upstream for those.
17:24 mattymo we had to break upstream to do HA for a lot of components
17:25 mgagne mattymo: I'm sure of it. We put the HA part in an other internal module.
17:26 mattymo I understand. I have to go now. It's late here. I'll try to reach you tomorrow with our update
17:26 mgagne mattymo: cool, have a good one
17:26 mattymo bye
17:33 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Fix package name for nova-spiceproxy on Ubuntu  https://review.openstack.org/60295
17:34 LAnthony joined #puppet-openstack
17:37 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Add it_configures rspec helper  https://review.openstack.org/60297
17:42 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Fix package name for nova-spiceproxy on Ubuntu  https://review.openstack.org/60295
17:49 fvollero some of the cores when have a quantum of time could check https://review.openstack.org/#/c/60009/ ?
17:50 fvollero /win 31
17:56 dmsimard joined #puppet-openstack
17:59 rmoe joined #puppet-openstack
18:00 xarses joined #puppet-openstack
18:00 mgagne fvollero: only nitpick I found, triggered by my OCD, there is no space between the ERB variable and %>
18:02 fvollero mgagne: Sure thing, I will fix it. I am trying to avoid your OCD execution :)
18:02 mgagne lgtm otherwise
18:04 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-horizon: Added support for parametrized help_url path.  https://review.openstack.org/60009
18:04 fvollero mgagne: done!
18:21 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-horizon: Added support for parametrized help_url path.  https://review.openstack.org/60009
18:21 thumpba joined #puppet-openstack
18:44 Gamekiller77 joined #puppet-openstack
18:45 Gamekiller77 hey guys we using the puppetlab manfist but it not setting the repo for openstack at build time
18:45 Gamekiller77 we have file in place but it not getting called
18:49 pravka joined #puppet-openstack
18:51 blkperl fyi, releasing apache module 0.10.0 should be backwards compat but I havn't tested it with the openstack modules
18:51 blkperl if someone wants to do a quick test that would be awesome, https://github.com/puppetlabs/puppetlabs-apache/pull/507
18:52 dmsimard joined #puppet-openstack
18:55 mgagne Gamekiller77: which file?
18:55 Gamekiller77 pp
18:56 Gamekiller77 all.pp
18:56 Gamekiller77 what we just notice is the openstack install just fine
18:56 Gamekiller77 but when it tries to install OVS it fails
18:56 Gamekiller77 it not statically installed the repo
18:57 Gamekiller77 that what we are going to do
19:08 derekh joined #puppet-openstack
19:29 dmsimard1 joined #puppet-openstack
19:35 LAnthony left #puppet-openstack
19:36 bauzas joined #puppet-openstack
19:50 LAnthony joined #puppet-openstack
19:57 bkero Huh, I'm trying a fresh ::all install on a 12.04 machine and afaict openstack::repo is never instantiated as a resource
19:58 bkero so all the package install attempts are just failing
20:04 * bkero just calls a class { 'openstack::repo': before => Class['openstack::all'] }
20:15 dmsimard joined #puppet-openstack
20:20 eglynn joined #puppet-openstack
20:25 francois joined #puppet-openstack
20:53 JonnyNomad joined #puppet-openstack
21:14 eglynn joined #puppet-openstack
21:19 dtalton joined #puppet-openstack
21:21 LAnthony left #puppet-openstack
21:23 thumpba joined #puppet-openstack
21:56 pabelanger joined #puppet-openstack
22:08 sputnik13net joined #puppet-openstack
22:22 bodepd mgagne: pong
22:22 mgagne jesus, 3 days later
22:25 dmsimard ^ lol
22:25 dmsimard packet loss
22:25 mgagne bodepd: mainly for that one: https://review.openstack.org/#/c/47891/
22:32 dmsimard joined #puppet-openstack
22:48 openstackgerrit joined #puppet-openstack
22:58 bodepd mgagne: yeah, I've been pulled into some non-openstack related stuff
22:58 bodepd what's that thing called where people pay you to do stuff :)

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary