Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-03-30

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

All times shown according to UTC.

Time Nick Message
00:24 ferest joined #puppet-openstack
00:35 xingchao joined #puppet-openstack
00:43 openstackgerrit Merged stackforge/puppet-ironic: Tag all Ironic packages  https://review.openstack.org/168610
00:47 openstackgerrit Merged stackforge/puppet-neutron: Add MidoNet plugin manifest  https://review.openstack.org/165772
01:00 imcsk8 joined #puppet-openstack
01:45 xingchao joined #puppet-openstack
01:45 xingchao_ joined #puppet-openstack
01:46 stevemar joined #puppet-openstack
02:12 xingchao joined #puppet-openstack
02:20 BrianShang_ joined #puppet-openstack
03:21 cwolferh_ joined #puppet-openstack
04:11 gildub joined #puppet-openstack
04:52 zerda joined #puppet-openstack
05:28 xingchao joined #puppet-openstack
05:38 xarses joined #puppet-openstack
05:41 saneax joined #puppet-openstack
06:03 cwolferh joined #puppet-openstack
06:04 imcsk8 /n
06:19 xingchao joined #puppet-openstack
06:22 markvoelker joined #puppet-openstack
06:26 tfz joined #puppet-openstack
06:27 stevemar joined #puppet-openstack
06:32 xingchao joined #puppet-openstack
06:34 stevemar joined #puppet-openstack
06:39 xingchao_ joined #puppet-openstack
07:23 markvoelker joined #puppet-openstack
07:26 dgurtner joined #puppet-openstack
07:26 dgurtner joined #puppet-openstack
07:28 arnaud_orange joined #puppet-openstack
07:36 stamak joined #puppet-openstack
08:04 mmagr joined #puppet-openstack
08:09 jistr joined #puppet-openstack
08:54 BrianShang joined #puppet-openstack
09:08 mattymo joined #puppet-openstack
09:24 markvoelker joined #puppet-openstack
09:36 derekh joined #puppet-openstack
09:57 ddmitriev1 joined #puppet-openstack
09:57 IBerezovskiy joined #puppet-openstack
09:58 ddmitriev joined #puppet-openstack
10:25 markvoelker joined #puppet-openstack
10:35 BrianShang_ joined #puppet-openstack
11:08 openstackgerrit Sergey Kolekonov proposed stackforge/puppet-neutron: Do not create tun and int bridges manually  https://review.openstack.org/168848
11:24 rcallawa joined #puppet-openstack
11:26 markvoelker joined #puppet-openstack
11:38 nosleep77 joined #puppet-openstack
11:43 xingchao joined #puppet-openstack
11:58 EmilienM good morning!
12:01 morazi joined #puppet-openstack
12:02 ferest joined #puppet-openstack
12:04 sbadia mgagne: yep thanks for the work! \o/
12:04 sbadia hello here
12:08 jistr joined #puppet-openstack
12:08 cdent joined #puppet-openstack
12:10 markvoelker joined #puppet-openstack
12:24 ferest joined #puppet-openstack
12:34 _matthias_ Hi! first of all everybody who contributed to the excellent puppet-openstack packages a big thank you …
12:35 _matthias_ anyway .. we now migrated from a completely unmanaged manual setup to a puppet managed setup
12:35 _matthias_ so far everything went quite well out of the box
12:35 _matthias_ however we now face tow funny things
12:37 _matthias_ contrary to the error and info messages we get the databases for cinder and glance are not setup correctly
12:38 _matthias_ they are empty
12:38 _matthias_ anyone had similar problems before?
12:39 gchamoul sbadia: https://review.openstack.org/#/q/topic:bug/1391209+status:open+owner:%22Gael+Chamoulaud+%253Cgchamoul%2540redhat.com%253E%22,n,z
12:41 EmilienM gchamoul: why are you adding "Signed-off-by" to all your patches ?
12:41 EmilienM it's usually done by kernel developers
12:42 gchamoul EmilienM: just for fun!
12:42 gchamoul :-p
12:43 gchamoul EmilienM: it's done automatically in my config ...
12:44 gchamoul EmilienM: It's useless ... I know!
12:54 xingchao joined #puppet-openstack
13:17 rcallawa joined #puppet-openstack
13:18 dprince joined #puppet-openstack
13:26 openstackgerrit Merged stackforge/puppet-heat: Tag all Heat packages  https://review.openstack.org/168605
13:31 saneax joined #puppet-openstack
13:37 _matthias_ nvmd my questions … got it
13:37 rhefner joined #puppet-openstack
13:37 _matthias_ we now just manually called the db sync jobs
13:39 britthouser joined #puppet-openstack
13:43 mdorman joined #puppet-openstack
13:47 britthouser joined #puppet-openstack
13:50 britthouser joined #puppet-openstack
13:52 stevemar joined #puppet-openstack
13:55 dvorak _matthias_: that should happen automatically if the packages are installed or updated
13:55 _matthias_ dvorak: unfortunately it did not
13:55 _matthias_ basically we manually fixed the database for neutron, cinder and glance
13:55 _matthias_ nevertheless now everything is working
13:55 dvorak well, or it did and you had an error when it ran
13:56 dvorak unfortunately neutron is the outlyer here, the db sync behavior has to be turned on
13:56 dvorak outlier even
13:57 dvorak for example, if the database isn't configured yet, or isn't up when the initial run occurs, you'll run into the problem you're seeing
13:58 _matthias_ exactly
13:58 _matthias_ neutron perhaps would have fixed itself magically on the next run .. the logs said something about checking database model
13:59 _matthias_ now we only run into minor troubles with noVNC proxy which we also had on our manual setup
14:00 _matthias_ where the nova.conf file is actually setup correctly but horizon refuses to put the correct url into the rendered HTML code
14:00 _matthias_ such that always the private IP address is used
14:00 dvorak I'd run neutron manage and make sure it thinks the database is stamped.  You can run into issues later with upgrades if the original db migrations don't run
14:01 dvorak because neutron can create the tables as needed, but you end up lacking the versioning information
14:02 _matthias_ neutron-db-manage --config-file=/etc/neutron/neutron.conf upgrade head
14:02 _matthias_ This is what we did
14:02 _matthias_ i can do this multiple times it then always tells me that everything seems perfect
14:03 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Fix behaviour of 'set-secret-value virsh' exec  https://review.openstack.org/168919
14:03 dvorak what do you get back if you run that with 'current' instead of 'upgade head' ?
14:03 _matthias_ INFO  [alembic.migration] Context impl MySQLImpl.
14:03 _matthias_ INFO  [alembic.migration] Will assume non-transactional DDL.
14:03 _matthias_ Current revision for mysql://neutron:XXXXX@192.168.100.10/neutron: 544673ac99ab -> juno (head), juno
14:03 _matthias_ looks OK doesn’t it?
14:03 dvorak you're good then
14:03 _matthias_ thanks so much for looking into that
14:03 _matthias_ now that I alredy got you hooked
14:04 _matthias_ any clues on the novnc issue?
14:04 dvorak nope :)
14:04 _matthias_ we put the following in nova.conf
14:04 _matthias_ :(
14:04 _matthias_ OK
14:05 dvorak well, can ask,
14:05 _matthias_ but isn’t there a hiera key that is pulled when writing the novnc config
14:05 _matthias_ our puppet setup obviously keeps reverting to our setup where the novncproxy_base_url is setup like this
14:05 _matthias_ http://192.168.100.10:6080/vnc_auto.html
14:06 _matthias_ whereas we want our public ip address filled
14:09 dvorak are you setting nova::compute::vncproxy_host ?
14:16 larsks EmilienM: howdy.
14:16 EmilienM larsks: hey
14:16 EmilienM larsks: https://github.com/stackforge/puppet-nova/blob/master/manifests/compute/spice.pp
14:17 EmilienM I was wondering why you don't want to follow this class for compute nodes
14:17 EmilienM something you need to know is, the novnc code in nova::compute is a design error from beginning
14:17 EmilienM and I truly think the fix would be to follow spice code
14:17 larsks Because the setting of novncproxy_base_url needs to be the same both on compute and controller hosts. I was trying to avoid having it set in two different places.
14:18 larsks So, with nova::compute::spice...
14:18 larsks ...hold on, let me see what packstack is doing with that.
14:18 ericpeterson joined #puppet-openstack
14:18 larsks Huh, it's not.
14:18 EmilienM larsks: I see !
14:19 larsks Which is probably why I didn't look at that :).
14:19 EmilienM larsks: so novncproxy_base_url is needed by controllers & compute ?
14:19 tfz joined #puppet-openstack
14:19 larsks Yes. Because of changes in the proxy service .
14:19 _matthias_ ok … actually it shouloud only be used on the controller
14:19 _matthias_ at least this was sufficient on our last seutp
14:19 larsks _matthias_: nooooope.
14:20 larsks _matthias_: previously, it was *only* required on compute hosts.
14:20 EmilienM larsks: if so, look at database parameters
14:20 larsks But with the recent changes, it is now used by nova-novncproxy, but *only* to determine the protocol (http vs https).
14:20 larsks I think the changes made in nova were poorly thought out, but that's what we've got.
14:20 EmilienM larsks: https://github.com/stackforge/puppet-nova/commit/30b6a9c4c7d9b662eb8e70b58a1ad63d1c4e89aa
14:20 larsks EmilienM: looking...
14:21 EmilienM larsks: it's an example, but I think you will understand what I mean
14:22 _matthias_ ok but setting openstack::nova::compute::vncproxy_* parameters in hiera should do the trick then for now?
14:22 larsks _matthias_: the existing nova::vncproxy module does not set the novncproxy_base_url setting, so no, unless I am misunderstanding your question...
14:24 _matthias_ ok … but does anyone know where it is set … because appearantly it is set somewhere … and I would like to fix it
14:24 larsks _matthias_: it is currently set by nova::compute
14:24 _matthias_ because I would really appreceate if I could just configure it from puppet
14:24 _matthias_ OK
14:24 _matthias_ so this in our hiera file will help
14:24 _matthias_ openstack::nova::compute::vncproxy_host: '0.0.0.0'
14:24 _matthias_ openstack::nova::compute::vncproxy_base_url: 'http://management.c3lab.tk.jku.at:6080/vnc_auto.html'
14:24 larsks EmilienM: If I understand your suggestion: move the setting of novncproxy_base_url into a separate class, and then 'include' that from both nova::compute and nova::vncproxy?
14:25 _matthias_ or do i need to really fix it in our site.pp (which is currently completely free form any site specific information)
14:25 EmilienM larsks: yes Sir
14:26 larsks EmilienM: On a system that includes both nova::compute and nova::vncproxy in the same puppet run, that should work as long as the parameters are identical, right?
14:26 EmilienM larsks: it's a suggestion, that is good because: you maintain backward compat, you have consistency with DB implementation and you split the novnc code somewhere else
14:28 _matthias_ ah ok .. compute.pp setups everything
14:29 _matthias_ so basicylly only setting nova::compute::vncproxy_host will hopefully build the correct URL in nova.conf
14:29 skippy joined #puppet-openstack
14:30 skippy are there instructions for the puppetlabs-openstack examples using Virtuablox instead of VMware Fusion?
14:36 openstackgerrit Lars Kellogg-Stedman proposed stackforge/puppet-nova: move setting of novncproxy_base_url  https://review.openstack.org/168545
14:36 larsks EmilienM: like that, maybe? ^^^
14:36 EmilienM larsks: looking in a few
14:39 EmilienM dmsimard: you may want to look https://review.openstack.org/#/c/168919
14:41 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Fix behaviour of 'set-secret-value virsh' exec  https://review.openstack.org/168919
14:53 _matthias_ OK the vnc proxy drives me nuts  .. will go home and give it another try tomorrow ;)
14:53 _matthias_ thanks everybody for listening to our problems!
15:01 rwsu joined #puppet-openstack
15:04 tfz joined #puppet-openstack
15:11 tfz joined #puppet-openstack
15:11 britthouser joined #puppet-openstack
15:12 dmsimard EmilienM: Will look
15:13 EmilienM dmsimard: thx
15:20 dmsimard EmilienM: I'm not sure I understand, it's like changing 1$ for 4 0.25$ for me ?
15:22 dmsimard Need to test, will check later today
15:22 EmilienM dmsimard: looking too, I asked the author to join us and discuss about it
15:24 gfidente joined #puppet-openstack
15:24 gfidente dmsimard, hi
15:26 dmsimard gfidente: hey, will have some time to check your review later today - this will be a busy day
15:26 gfidente was talking with EmilienM about https://review.openstack.org/#/c/168919/ to see if that could have caused issues
15:26 tfz joined #puppet-openstack
15:28 gfidente dmsimard, that is fine, ping me over irc when you have time, I just wanted to make sure there wasn't something relying on the old behaviour
15:28 gfidente which isn't working for me today btw
15:33 skippy are there instructions for the puppetlabs-openstack examples using Virtuablox instead of VMware Fusion?   The scripts and Vagrantfiles appear hard-coded for VMware Fusion.
15:42 mschmitt joined #puppet-openstack
15:45 tfz joined #puppet-openstack
15:49 EmilienM larsks: looking at your patch on https://review.openstack.org/#/c/168545
15:49 EmilienM larsks: you did not follow what I did for database (include, etc)
15:49 larsks EmilienM: well, it uses class {...} instead of include...
15:49 larsks But otherwise it is very similar, yes?
15:50 larsks Maybe I am missing something, but in order to provide backwards compatability we have to pass on the parameters from , e.g., compute.pp.
15:50 EmilienM larsks: no
15:50 EmilienM larsks: the way you grab parameters needs to be like in https://github.com/stackforge/puppet-nova/blob/30b6a9c4c7d9b662eb8e70b58a1ad63d1c4e89aa/manifests/db.pp
15:50 EmilienM i think it's saner
15:51 larsks EmilienM: So, in that manifest, your getting parameters from the main nova class (nova::database_connection), right?
15:52 EmilienM larsks: yeah
15:52 larsks From which class would we pick the vncproxy_base_url?
15:52 EmilienM you would do the same but from nova::compute ?
15:52 larsks Because you said it should *not* be part of the base nova class.
15:52 larsks And why nova::compute would not be instantiaated on a controller.
15:52 larsks And on a compute-only host, nova::vncproxy would not be instantiated.
15:53 EmilienM mhh, interesting, indeed
15:54 EmilienM well, larsks you can still follow database thing, and use ::nova for it. If really needed... We just need consistency and backward compatibility (at least for one release)
15:54 larsks Okay.  250107
15:54 larsks Whoops.
15:56 larsks So, what we've got with the current patch gets you that (backwards compatability with the existing module, and consistency by ensuring that there is only place that sets the value).
15:56 larsks But I will take another look...
16:03 _matthias_ now I am still hooked on my noVNC setup … Is there a list somewhere with all parameters one can set through hiera
16:03 _matthias_ … the reason I am asking
16:04 _matthias_ our site.pp file only contains include ::openstacke::role::XXXX lines for each host
16:09 vinsh joined #puppet-openstack
16:10 cwolferh joined #puppet-openstack
16:11 skippy left #puppet-openstack
16:16 dmsimard Is there any talks in the wider puppet-openstack community about revamping puppet-swift ? The template-based config is starting to hurt me and leading to commits to add params just to be able to support a config parameter.
16:22 dmsimard (perhaps this is best left as a mailing list post)
16:41 gfidente dmsimard, FWIW I left a comment in your change; I think they go well together
16:42 gfidente (eg. works for me, and thanks for adding support for the ephemeral on local disks while volumes only are on ceph)
16:46 dmsimard gfidente: Yeah I need to finish that one too ..
16:47 gfidente yeah so knowing we'll get that, I think mine is good in its scope as it really only fixes the underlying issue, your adds the feature
16:50 mmagr joined #puppet-openstack
16:50 aimon joined #puppet-openstack
16:53 xingchao joined #puppet-openstack
17:05 stamak joined #puppet-openstack
17:05 crinkle there hasn't been much discussion on http://lists.openstack.org/pipermail/openstack-dev/2015-March/059996.html
17:06 crinkle anyone want to add something to it?
17:29 dgurtner joined #puppet-openstack
17:29 dgurtner joined #puppet-openstack
17:38 tfz joined #puppet-openstack
17:43 xarses joined #puppet-openstack
17:57 EmilienM crinkle: I think this is a good start - though I don't understand the 'changing every week' thing
18:00 EmilienM sbadia: ^
18:25 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: Move rbd related options into glance_store section  https://review.openstack.org/169040
18:25 EmilienM crinkle: can you check that backport please?
18:35 bodepd_ crinkle: I have a question about this line: https://github.com/JioCloud/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L56
18:35 openstackgerrit Merged stackforge/puppet-horizon: Tag all Horizon packages  https://review.openstack.org/168608
18:37 bodepd_ crinkle: we are trying to use that parent provider for an implementation we are doing of nova_flavor, and every resource retries for 2 minutes before failing. In my case, the service just isn't ready, so every flavor resource timesouts on the first run, which can add 10-15 minutes to a run
18:38 bodepd_ I was just curious if other people had run into issues where that timeout length per resource was s bit large
18:40 crinkle bodepd_: we changed that to be 10 seconds https://github.com/stackforge/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L48
18:41 bodepd_ crinkle: sorry! I sent you a link to our fork when I meant to read the stackforge version :)
18:41 EmilienM mfisch: about https://review.openstack.org/#/c/158299/
18:41 crinkle bodepd_: :)
18:41 EmilienM mfisch: I'm not sure if we have to respect backward compatibility
18:42 openstackgerrit Merged stackforge/puppet-trove: Adds ability to override service name for service catalog  https://review.openstack.org/166670
18:42 EmilienM I don't know much people except us using this module
18:42 mfisch will look after this call
18:42 bodepd_ crinkle: yes, that makes sense that you would change that :)
18:42 crinkle bodepd_: it was originally copied from the neutron providers which were waiting for 120 seconds
18:43 crinkle and i think still are
18:43 mfisch EmilienM: whats the compat issue here?
18:43 EmilienM mfisch: the one you're mentioning in your comment
18:44 EmilienM mfisch: before +A, I wanted to double check with you
18:44 crinkle bodepd_: if you have time could you help review https://review.openstack.org/#/c/146926/ ?
18:44 mfisch EmilienM: let me look
18:44 crinkle would like some more t&p expertise to look at that
18:44 mfisch EmilienM: +2 he added what I wanted
18:45 mfisch EmilienM: did you see the notes that Mike & I added to the presentation? Can you add some more details too? Same etherpad
18:45 EmilienM mfisch: I'm catching up my emails/reviews things today
18:45 EmilienM I'll see asap
18:45 mfisch EmilienM: okay when you have time we added 2nd level detail. We should chat again later this week after you update it
18:46 openstackgerrit Merged stackforge/puppet-heat: Fix ipv6 support  https://review.openstack.org/164154
18:49 openstackgerrit Merged stackforge/puppet-ironic: Fix rabbitmq support  https://review.openstack.org/158299
18:49 EmilienM mfisch: crinkle: what do you think about https://review.openstack.org/#/c/166970/ ? it seems undef is not enough here, we really need an empty string
18:51 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Fix behaviour of 'set-secret-value virsh' exec  https://review.openstack.org/168919
18:53 crinkle EmilienM: sounds reasonable to me
18:53 crinkle it's documented and validated
18:54 openstackgerrit Merged stackforge/puppet-neutron: Fix ipv6 support  https://review.openstack.org/164153
18:54 openstackgerrit Lars Kellogg-Stedman proposed stackforge/puppet-nova: move setting of novncproxy_base_url  https://review.openstack.org/168545
18:55 openstackgerrit Merged stackforge/puppet-designate: Adds ability to override service name for service catalog  https://review.openstack.org/166675
18:55 openstackgerrit Lars Kellogg-Stedman proposed stackforge/puppet-nova: move setting of novncproxy_base_url  https://review.openstack.org/168545
18:56 larsks EmilienM: how about that one?
19:01 social joined #puppet-openstack
19:07 EmilienM larsks: will look asap :)
19:07 larsks No rush :)
19:08 rcallawa_ joined #puppet-openstack
19:23 tfz joined #puppet-openstack
19:23 openstackgerrit Merged stackforge/puppet-nova: Fix beaker tests  https://review.openstack.org/155186
19:23 openstackgerrit Merged stackforge/puppet-nova: Fix beaker tests  https://review.openstack.org/155186
19:33 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: Move rbd related options into glance_store section  https://review.openstack.org/169040
19:40 larsks EmilienM: Eh, never mind, that won't work.
19:48 tfz joined #puppet-openstack
19:59 openstackgerrit Crag Wolfe proposed stackforge/puppet-cinder: two additional rbd config options  https://review.openstack.org/169066
20:01 tfz joined #puppet-openstack
20:08 rcallawa joined #puppet-openstack
20:09 openstackgerrit Lars Kellogg-Stedman proposed stackforge/puppet-nova: move setting of novncproxy_base_url  https://review.openstack.org/168545
20:14 openstackgerrit Emilien Macchi proposed stackforge/puppet-heat: rename keystone_v2_authenticate method  https://review.openstack.org/162292
20:21 tfz joined #puppet-openstack
20:21 openstackgerrit Adam Vinsh proposed stackforge/puppet-swift: Added rsyslog logging support to object-server  https://review.openstack.org/169071
20:23 openstackgerrit Brad P. Crochet proposed stackforge/puppet-tuskar: Change puppet to match tuskar packaging and code  https://review.openstack.org/169072
20:23 openstackgerrit Adam Vinsh proposed stackforge/puppet-swift: Added rsyslog logging support to object-server  https://review.openstack.org/169071
20:24 openstackgerrit Adam Vinsh proposed stackforge/puppet-swift: Added rsyslog logging support to object-server  https://review.openstack.org/169071
20:28 ericpete_ joined #puppet-openstack
20:40 openstackgerrit Lars Kellogg-Stedman proposed stackforge/puppet-nova: move setting of novncproxy_base_url  https://review.openstack.org/168545
20:54 openstackgerrit Crag Wolfe proposed stackforge/puppet-cinder: two additional rbd config options  https://review.openstack.org/169066
20:56 xingchao joined #puppet-openstack
21:02 aimon joined #puppet-openstack
21:05 EmilienM larsks: quick review on https://review.openstack.org/#/c/168545 - I would use nova::vnc instead ?
21:05 EmilienM or vncproxy
21:06 larsks There is already a nova::vncproxy...hence the name.
21:06 EmilienM mhh
21:06 EmilienM so we could re-use it
21:06 EmilienM if you think it's okay
21:06 larsks Well, no, beccause nova::novncproxy is for setting up the *service*.  So it's not used on, say, a compute host.
21:07 EmilienM mhhh I see
21:07 larsks Unless we stuck a giant boolean around the package installs and service config, but that sounds ugly.
21:07 EmilienM it is!
21:07 EmilienM larsks: won't you kill me if I tell you it mays be a candidate to live in ::nova ?
21:07 EmilienM but I would like mfisch mgagne crinkle feedbacks
21:07 larsks Hey, that sounds familiar.
21:08 EmilienM larsks: :)
21:08 larsks I will leave it for now until other folks have a chance to weight in, I guess.
21:08 larsks Gotta go pick up kids and such.
21:08 EmilienM larsks: I'll make sure it happens.
21:10 imcsk8 larsks: i'm testing the patch
21:11 larsks imcsk8: it worked for me with packstack multi node and all in one.
21:12 imcsk8 larsks: cool! i'm testing with packstack also
21:12 larsks Using rdo Juno and packstack master w/the patch.
21:12 EmilienM larsks: imcsk8: the problem is not "it works or not"
21:12 EmilienM the problem is code quality and consistency here.
21:14 imcsk8 EmilienM: well, the code looks good. i just have the feeling that i should test it :)
21:15 hogepodge davidlenwell timing for design sessions is happening at the cross project meeting tomorrow
21:16 EmilienM crinkle: ^
21:16 EmilienM hogepodge: that's an interesting news
21:16 tfz joined #puppet-openstack
21:16 aimon joined #puppet-openstack
21:19 larsks EmilienM: ... But "works" is still critical, and thank you for the comment on Tue quality.
21:19 larsks *the
21:19 EmilienM larsks: the problem is not your code quality
21:20 EmilienM again, it's a problem in consistency with other way we manage it
21:20 EmilienM and also we try to run some conventions for class names
21:20 EmilienM hogepodge: was it for us?
21:22 larsks And that is why I added you to the review. I think we have largely achieved those goals. I am happy to iterate, although I think I agree with your original assessment that this should not be in the nova class itself.
21:26 EmilienM larsks: I dropped a review - now, the best we can do is to get feedback from other devs
21:40 bodepd_ crinkle: my goodness, that's a big patch. I might be missing logs of context to be helpful without tons of effort
21:41 bodepd_ crinkle: if you want, I can meet you for lunch and talk through it with you (but it would have to be tomorrow)
21:41 crinkle bodepd_: it's not my patch, so I wouldn't be that helpful in talking through it
21:42 crinkle bodepd_: there is a blueprint that might help give context though https://review.openstack.org/#/c/150108/
22:14 mdorman_ joined #puppet-openstack
22:17 mdorman joined #puppet-openstack
22:19 imcsk8 EmilienM: another question, in here https://github.com/redhat-openstack/puppet-pacemaker/blob/master/manifests/corosync.pp the pacemaker::corosync class inherits from the pacemaker class, this means that the pacemaker::corosync class should have all the pacemaker class params right?
22:28 rcallawa joined #puppet-openstack
22:29 tfz joined #puppet-openstack
22:41 hogepodge haha wrong room EmilienM :-P
22:47 mfisch Hey RedHat users and employees. Do you guys have a python-heat package in RHEL/Fedora/Etc?
22:47 mfisch I'd like to add it to the heat manifest
22:49 rcallawa joined #puppet-openstack
22:51 xarses joined #puppet-openstack
23:03 openstackgerrit Matt Fischer proposed stackforge/puppet-heat: Add a reference to python-heat on Debian platforms  https://review.openstack.org/169110
23:03 mfisch can some RedHat family guys please look at that? I did not find a python-heat package in Fedora but I might not be in the right place. ^^
23:05 openstackgerrit Matt Fischer proposed stackforge/puppet-heat: Add a reference to python-heat on Debian platforms  https://review.openstack.org/169110
23:30 mfisch mgagne: good questions
23:31 mgagne mfisch: using UCA packages or yours?
23:31 mfisch uca
23:31 mfisch python-heat is refusing to upgrade even with dist-upgrade
23:31 mfisch -> dinner
23:33 logan2 joined #puppet-openstack
23:49 aimon_ joined #puppet-openstack
23:53 markvoelker joined #puppet-openstack

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