Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-04-23

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

All times shown according to UTC.

Time Nick Message
00:01 markvoelker joined #puppet-openstack
00:12 xuhaiwei joined #puppet-openstack
00:19 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: V3 support for Service provider only  https://review.openstack.org/174695
00:37 rcallawa joined #puppet-openstack
00:38 ibravo joined #puppet-openstack
00:48 Vinsh joined #puppet-openstack
00:55 ericpeterson joined #puppet-openstack
01:21 rcallawa joined #puppet-openstack
01:36 xarses joined #puppet-openstack
01:49 rcallawa joined #puppet-openstack
02:17 xingchao joined #puppet-openstack
02:22 markvoelker joined #puppet-openstack
02:25 rcallawa joined #puppet-openstack
03:13 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Beaker tests  https://review.openstack.org/176098
03:25 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
03:31 gildub_ joined #puppet-openstack
03:32 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
03:32 gildub_ joined #puppet-openstack
03:32 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
03:36 gildub joined #puppet-openstack
03:43 openstackgerrit Emilien Macchi proposed stackforge/puppet-ceilometer: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176493
03:45 openstackgerrit Emilien Macchi proposed stackforge/puppet-cinder: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176591
03:47 openstackgerrit Emilien Macchi proposed stackforge/puppet-designate: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176593
03:48 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176594
03:49 openstackgerrit Emilien Macchi proposed stackforge/puppet-gnocchi: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176596
03:49 openstackgerrit Emilien Macchi proposed stackforge/puppet-heat: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176597
03:51 openstackgerrit Emilien Macchi proposed stackforge/puppet-horizon: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176599
03:52 openstackgerrit Emilien Macchi proposed stackforge/puppet-ironic: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176600
03:52 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176601
03:53 openstackgerrit Emilien Macchi proposed stackforge/puppet-manila: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176602
03:54 openstackgerrit Emilien Macchi proposed stackforge/puppet-neutron: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176604
03:54 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176605
03:55 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176606
03:56 openstackgerrit Emilien Macchi proposed stackforge/puppet-sahara: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176607
03:57 openstackgerrit Emilien Macchi proposed stackforge/puppet-swift: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176608
03:57 openstackgerrit Emilien Macchi proposed stackforge/puppet-tempest: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176609
03:58 openstackgerrit Emilien Macchi proposed stackforge/puppet-tripleo: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176610
03:58 openstackgerrit Emilien Macchi proposed stackforge/puppet-trove: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176611
04:01 openstackgerrit Emilien Macchi proposed stackforge/puppet-tuskar: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176613
04:02 openstackgerrit Emilien Macchi proposed stackforge/puppet-vswitch: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176614
04:03 rcallawa joined #puppet-openstack
04:05 rcallawa joined #puppet-openstack
04:06 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
04:07 openstackgerrit Merged stackforge/puppet-neutron: Make retry message more descriptive  https://review.openstack.org/176138
04:07 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Beaker tests  https://review.openstack.org/176098
04:08 openstackgerrit Merged stackforge/puppet-neutron: Revert "Move rabbit/kombu settings to oslo_messaging_rabbit section"  https://review.openstack.org/174964
04:19 rcallawa joined #puppet-openstack
04:22 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
04:24 EmilienM mfisch: for the bump patches, maybe I should fix the tests within the same patch for those which are failing
04:31 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Update Beaker tests  https://review.openstack.org/174182
04:32 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
04:34 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Update Beaker tests  https://review.openstack.org/174182
04:40 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
05:02 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
05:05 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Update Beaker tests  https://review.openstack.org/174182
05:14 saneax joined #puppet-openstack
05:21 markvoelker_ joined #puppet-openstack
05:41 guimaluf joined #puppet-openstack
05:45 rcallawa joined #puppet-openstack
05:47 stamak joined #puppet-openstack
06:00 rcallawa_ joined #puppet-openstack
06:02 tfz joined #puppet-openstack
06:04 rcallawa joined #puppet-openstack
06:46 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: V3 support for service provider only  https://review.openstack.org/174695
06:50 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: V3 support for endpoint provider  https://review.openstack.org/174702
06:55 ddmitriev joined #puppet-openstack
06:58 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Using V3 by default for token credentials  https://review.openstack.org/174692
07:04 tfz joined #puppet-openstack
07:16 xingchao joined #puppet-openstack
07:26 jpena joined #puppet-openstack
07:28 dgurtner joined #puppet-openstack
07:30 stamak joined #puppet-openstack
07:38 jistr joined #puppet-openstack
07:45 morazi joined #puppet-openstack
08:03 gfidente joined #puppet-openstack
08:03 gfidente joined #puppet-openstack
08:07 openstackgerrit Javier Peña proposed stackforge/puppet-swift: Removal of SSH Components  https://review.openstack.org/176432
08:58 ddmitriev joined #puppet-openstack
09:37 openstackgerrit Arnoud de Jonge proposed stackforge/puppet-nova: Moved spice configuration options from DEFAULT to spice section.  https://review.openstack.org/135600
09:41 derekh joined #puppet-openstack
09:43 rcallawa joined #puppet-openstack
09:47 rcallawa joined #puppet-openstack
10:17 gildub joined #puppet-openstack
10:32 cdent joined #puppet-openstack
10:42 dgurtner joined #puppet-openstack
10:42 dgurtner joined #puppet-openstack
10:43 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
10:46 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
10:53 dalgaaf joined #puppet-openstack
11:04 dgurtner joined #puppet-openstack
11:09 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
11:09 gfidente dmsimard, ^^ I updated this as we want to use it in tripleo! :)
11:58 mmagr joined #puppet-openstack
12:00 dgurtner joined #puppet-openstack
12:10 ericpeterson joined #puppet-openstack
12:14 cdent joined #puppet-openstack
12:16 richm joined #puppet-openstack
12:23 dprince joined #puppet-openstack
12:29 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
12:36 rcallawa joined #puppet-openstack
12:40 openstackgerrit Alexandre Maumené proposed stackforge/puppet-cinder: Add nfs_mount_options variable when backend is NetApp  https://review.openstack.org/176721
12:42 dgurtner joined #puppet-openstack
13:00 openstackgerrit Alexandre Maumené proposed stackforge/puppet-cinder: Add nfs_mount_options variable when backend is NetApp  https://review.openstack.org/176721
13:01 ferest joined #puppet-openstack
13:03 robklg joined #puppet-openstack
13:19 xingchao joined #puppet-openstack
13:21 robklg left #puppet-openstack
13:34 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Bump rspec-puppet to 2.1.0 & fix tests  https://review.openstack.org/176601
13:36 xingchao joined #puppet-openstack
13:45 ibravo joined #puppet-openstack
13:45 ibravo_ joined #puppet-openstack
13:46 ibravo joined #puppet-openstack
13:51 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176601
13:57 ericpeterson joined #puppet-openstack
13:57 ericpeterson joined #puppet-openstack
13:59 ericpete_ joined #puppet-openstack
14:07 rcallawa joined #puppet-openstack
14:07 dmellado joined #puppet-openstack
14:10 rcallawa joined #puppet-openstack
14:11 EmilienM I reported a bug for tracking rspec 2.1.0 failing tests: https://bugs.launchpad.net/p​uppet-keystone/+bug/1447620
14:11 EmilienM it affects most of our modules.
14:16 ericpeterson joined #puppet-openstack
14:19 rcallawa joined #puppet-openstack
14:24 rwsu joined #puppet-openstack
14:29 mschmitt joined #puppet-openstack
14:31 Vinsh joined #puppet-openstack
14:32 alex_bh joined #puppet-openstack
14:33 rcallawa_ joined #puppet-openstack
14:35 openstackgerrit Merged stackforge/puppet-cinder: Add nfs_mount_options variable when backend is NetApp  https://review.openstack.org/176721
14:35 mfisch EmilienM: makes sense wrt bump patches
14:35 rcallawa_ joined #puppet-openstack
14:35 EmilienM mfisch: actually, no
14:36 EmilienM mfisch: I give up to fix the tests in the same patchset
14:36 mfisch ok
14:36 EmilienM I think we can merge the bump patches
14:36 EmilienM and then use https://bugs.launchpad.net/p​uppet-keystone/+bug/1447620
14:36 EmilienM to track the patches that fix the tests
14:36 EmilienM different chunks == move faster
14:36 EmilienM sbadia, mfisch: if you can look at rspec 2.1.0 patches, and maybe merge them. Would be awesome
14:37 sbadia https://review.openstack.org/​#/q/status:open+topic:puppet4,n,z ?
14:37 sbadia oki
14:37 sbadia (hi here)
14:38 mfisch lookng
14:39 mfisch its changes like these that make we wish we had something lke a puppet openstack core module that stuff inherited from, I know that doesn't match really how puppet code works, but its on my wishlist
14:40 EmilienM mfisch: like msync?
14:40 mfisch whats mysync?
14:40 mfisch msync
14:41 sbadia module_sync
14:41 sbadia t https://github.com/puppetlabs/modulesync
14:41 EmilienM mfisch: https://review.openstack.org/#/c/164125/
14:41 EmilienM spredzy: is working on it
14:42 EmilienM mfisch: we talked about it in the last meeting BTW
14:42 mfisch I can tell from the title in github that this is what I mean
14:42 mfisch yeah unfortunately the meeting overlap my team meeting so sometimes I have to focus on that for a few mins and I can miss stuff :(
14:42 openstackgerrit Merged stackforge/puppet-modulesync-configs: puppet-modulesync-configs: Initial commit  https://review.openstack.org/164125
14:43 EmilienM lol
14:43 EmilienM who merged that
14:43 EmilienM ahah sbadia
14:43 sbadia 0:-)
14:43 mfisch was gonna say probably some French guy
14:43 mfisch just based on odds
14:43 EmilienM mfisch: ahah
14:44 * mfisch has to go to the dentist... back later
14:45 EmilienM mfisch: :(
14:45 EmilienM good luck
14:47 BroadcastStorm joined #puppet-openstack
14:47 bcstorm joined #puppet-openstack
14:50 nibalizer EmilienM: https://bugs.launchpad.net/p​uppet-keystone/+bug/1447649
14:52 EmilienM nibalizer: wow.
14:52 EmilienM nibalizer: until now, I was integrating keystone in nova tests
14:53 EmilienM ie: puppet-nova/beaker installs keystone and also nova so we can validate nova actually works. But I'm missing Glance & Neutron
14:54 nibalizer EmilienM: ya i think what you're doing is good
14:54 nibalizer but that i think is the next step
14:54 nibalizer if you want to get more ambitious
14:54 EmilienM defintly
14:54 EmilienM so
14:54 EmilienM instead of having gate-puppet-ceilometer-puppet-beaker-rspec
14:54 EmilienM we would have gate-puppet-openstack-beaker-rspec
14:54 nibalizer well i think you'd want to keep that test around tbh
14:54 EmilienM that would test everything
14:54 nibalizer since its feedback is valuable
14:54 EmilienM in the same job
14:55 nibalizer ya maybe
14:55 nibalizer there is some stuff to work on there
14:55 nibalizer and we should figure out where the value is
14:55 EmilienM nibalizer: so my question is: where would you see the tests reside ?
14:55 EmilienM in each module?
14:55 EmilienM if not, how to enforce a patch to include beaker tests?
14:55 nibalizer because its kinda easy to make a super big long integration test that doesnt actually provide much value
14:56 EmilienM the value is high, even more if we run tempest after
14:56 nibalizer ill be back later and we can bang itout
14:56 EmilienM nibalizer: ack
14:56 bcstorm left #puppet-openstack
14:56 openstackgerrit Merged stackforge/puppet-heat: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176597
14:56 BroadcastStorm left #puppet-openstack
14:57 openstackgerrit Merged stackforge/puppet-designate: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176593
14:58 openstackgerrit Merged stackforge/puppet-tempest: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176609
15:00 EmilienM Hunner: ping
15:02 openstackgerrit Merged stackforge/puppet-swift: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176608
15:03 nibalizer EmilienM: back a bit
15:03 nibalizer EmilienM: ya if we could run a striped down tempest that would be amazing
15:04 openstackgerrit Merged stackforge/puppet-horizon: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176599
15:04 nibalizer EmilienM: so i think maybe the integration test could go in its own repo
15:04 openstackgerrit Merged stackforge/puppet-vswitch: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176614
15:05 gfidente EmilienM, help! :)
15:05 nibalizer that might be easier than maintaining the same test accross all repositories
15:05 EmilienM nibalizer: :(
15:05 EmilienM nibalizer: I don't like having the tests outside
15:05 EmilienM because it will lead to new features without beaker tests
15:05 EmilienM gfidente: o/
15:05 nibalizer EmilienM: devstack is its own repo is my point
15:05 nibalizer thats what we're really talking about here
15:05 nibalizer it doesnt apply to just puppet-nova or just puppet-glance it applys to everything
15:06 gfidente EmilienM, I have no clue why the tests https://review.openstack.org/#/c/119093/ fail
15:06 gfidente nor how to run the tests locally
15:06 gfidente RTFM is a perfectly fine response, but I need at least a pointer to the M !
15:06 EmilienM gfidente: looking
15:06 openstackgerrit Merged stackforge/puppet-trove: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176611
15:06 gfidente EmilienM, it's just me who doesn't know why
15:06 openstackgerrit Merged stackforge/puppet-neutron: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176604
15:07 EmilienM oh
15:07 EmilienM gfidente: I think I got it, wait a bit
15:07 gfidente okay I wait to check the diffs
15:07 gfidente thanks :)
15:08 EmilienM nibalizer: a sec
15:08 EmilienM nibalizer: I'm still concerned about tests and where they should live
15:09 EmilienM to me, each patch should contains doc, manifest, rspec-unit,classes,acceptance
15:09 nibalizer hrm okay
15:09 EmilienM so if you pull out the tests somewhere else
15:10 EmilienM you bring a new feature that does not update acceptance tests
15:11 openstackgerrit Merged stackforge/puppet-ironic: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176600
15:12 nibalizer yea i think that would go in the nova tests that live in nova
15:12 nibalizer er puppet-nova
15:12 openstackgerrit Merged stackforge/puppet-glance: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176594
15:13 openstackgerrit Merged stackforge/puppet-nova: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176605
15:14 openstackgerrit Merged stackforge/puppet-keystone: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176601
15:16 openstackgerrit Merged stackforge/puppet-cinder: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176591
15:17 openstackgerrit Merged stackforge/puppet-ceilometer: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176493
15:32 jpena joined #puppet-openstack
15:37 Hunner EmilienM: pong
15:37 EmilienM Hunner: hey. So I bumbed all our modules to 2.1.0
15:37 EmilienM Hunner: and I have weird things for Puppet 4.0 jobs
15:38 EmilienM I've not found time to fix it yet, but I track the bugs here: https://bugs.launchpad.net/​puppet-tuskar/+bug/1447620
15:48 dfisher joined #puppet-openstack
15:49 Hunner EmilienM: Did you update puppetlabs_spec_helper to 0.10.2 also?
15:49 EmilienM Hunner: no
15:49 Hunner Ah. That is needed also :)
15:49 EmilienM cool!
15:49 EmilienM I should have known that. Next time I'll check :P
15:50 Hunner Unfortunately psh and r-p can't depend on each other because it creates a cycle
15:53 EmilienM Hunner: well, the Gemfile is not pulling a version so I guess it pulls the last version which should be 0.10.2
15:53 alex_bh joined #puppet-openstack
15:54 xarses joined #puppet-openstack
15:54 jpena joined #puppet-openstack
15:54 EmilienM Hunner: http://logs.openstack.org/96/176596/1/check/gate-p​uppet-gnocchi-puppet-unit-4.0/16ec773/console.html
15:54 EmilienM using puppetlabs_spec_helper 0.10.2
15:55 EmilienM so we're good
15:55 EmilienM and the tests fail for another reason then?
15:56 alex_bh joined #puppet-openstack
15:58 alex_bh joined #puppet-openstack
16:04 maximov joined #puppet-openstack
16:04 EmilienM Hunner: any clue is welcome, I tried to fix the test, but it broke older versions
16:07 Hunner Yeah. I don't know why this is passing on 3.7 either
16:07 Hunner Usually a property with :array_matching => :all should always return its value as an array
16:07 Hunner But in 3.7 it doesn't
16:08 EmilienM Hunner: I tracked all modules affected on https://bugs.launchpad.net​/puppet-nova/+bug/1447620
16:09 Hunner EmilienM: I was looking at that, but am I blind or does it not actually link to bug output?
16:09 Hunner err, unit test output
16:10 EmilienM Hunner: I did not put the link for all modules
16:11 Hunner Hmm. I wonder if it's a change in the behavior of ensure_resource or something
16:15 aimon joined #puppet-openstack
16:20 Vinsh_ joined #puppet-openstack
16:20 tfz joined #puppet-openstack
16:21 Hunner oh, no... ["admin"] is the default, and nothing is being passed in
16:22 Vinsh joined #puppet-openstack
16:22 EmilienM Hunner: so I tried that one
16:22 EmilienM to use the array
16:22 EmilienM it fix 4.0 but breaks 3.X
16:22 EmilienM :)
16:32 Hunner It's still an array at the point that https://github.com/puppetlabs/puppet/blob/3.7.5/li​b/puppet/parser/functions/create_resources.rb#L73 is called
16:32 Hunner Which means the AST construction hasn't changed between versions
16:34 mfisch Do we have any modules with providers that can handle multiple entries with the same name like neutron needs for service_providers:  http://docs.openstack.org/juno/config-ref​erence/content/section_neutron.conf.html
16:35 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
16:35 mfisch its basically like:
16:35 mfisch service_provider=LOADBALANCER:Haproxy:neutro​n.services.loadbalancer.drivers.haproxy.plug​in_driver.HaproxyOnHostPluginDriver:default
16:35 mfisch service_provider=VPN:openswan:neutron.services.v​pn.service_drivers.ipsec.IPsecVPNDriver:default
16:35 mfisch and most providers wouldnt be happy with repeated names
16:36 * mfisch shakes fist at neutron, again
16:38 dfisher GA
16:38 dfisher whoops
16:48 EmilienM mfisch: I backported the MySQL/Collate thing in stable/juno, you may want to +A if it's good for you
16:48 EmilienM https://review.openstack.org/#/q/own​er:%22Emilien+Macchi%22+status:open,n,z
16:49 mfisch ok
16:50 mfisch will look in a bit
16:51 EmilienM mfisch: should I ask to core-reviewers to consider check-tripleo-ironic-overcloud-f20puppet-nonha new job?
16:52 mfisch EmilienM: can you also look at this one from clayton?  https://review.openstack.org/#/c/148693/
16:52 EmilienM mfisch: right now
16:52 mfisch I need to catch up on the mails about that tripleo stuff just been busy
16:53 EmilienM mfisch: the gael's remark makes sense.
16:53 mfisch he didn't -1 it so maybe clayton didnt see
16:53 mfisch I end up ignoring all the emails and just look at my dashboard, too many emails
16:53 EmilienM I'm wondering why this change in Gemfile
16:53 EmilienM mfisch: same.
16:54 mfisch yeah unsure about that
16:54 mfisch clayton is at lunch I'm sure he can update
16:54 EmilienM I'm going to lunch too now, bbl
17:02 openstackgerrit Merged stackforge/puppet-cinder: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176516
17:02 openstackgerrit Merged stackforge/puppet-openstacklib: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176606
17:06 maximov left #puppet-openstack
17:06 openstackgerrit Merged stackforge/puppet-glance: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176509
17:06 openstackgerrit Merged stackforge/puppet-ceilometer: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176510
17:08 openstackgerrit Merged stackforge/puppet-openstacklib: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176506
17:09 openstackgerrit Merged stackforge/puppet-neutron: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176505
17:11 openstackgerrit Merged stackforge/puppet-nova: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/176507
17:13 ericpete_ joined #puppet-openstack
17:16 Hunner EmilienM: Unit tests aren't broken due to rspec-puppet... they're due to puppet 4 :P
17:16 Hunner I just noticed that in the bug title
17:25 xingchao joined #puppet-openstack
17:36 EmilienM Hunner: I can change it
17:36 EmilienM Hunner: https://bugs.launchpad.net​/puppet-nova/+bug/1447620
17:42 clayton EmilienM: it was a mistake that that got into that change, I'll remove it
17:42 clayton I had added it locally to make testing easier
17:47 openstackgerrit Clayton O'Neill proposed stackforge/puppet-heat: Make package_ensure consistent across classes  https://review.openstack.org/148693
17:49 aimon_ joined #puppet-openstack
17:50 openstackgerrit Denis Egorenko proposed stackforge/puppet-cinder: Add cinder::scheduler::filter for managing scheduler.filter  https://review.openstack.org/176924
17:52 EmilienM clayton: thx, will look asap
17:52 EmilienM can someone approve https://review.openstack.org/​#/q/status:open+topic:puppet4,n,z ?
17:53 dprince joined #puppet-openstack
17:58 nibalizer EmilienM: okay i reviewed https://review.openstack.org/#/c/176098/
17:59 nibalizer and a lot of that (all of that?) applies to the othre patches as well
17:59 nibalizer i gotta go grab dinner
18:01 EmilienM nibalizer: thx for that
18:05 morazi joined #puppet-openstack
18:08 EmilienM clayton: +A
18:08 EmilienM oh wait
18:08 EmilienM no test for API ?
18:08 EmilienM clayton: ^
18:09 clayton the existing tests already test for present.  The engine test had to be updated because it had used 'installed'
18:10 clayton assuming we're talking about my heat patch
18:11 EmilienM clayton: I don't see package tests in api tests
18:11 EmilienM I may be blind
18:12 clayton ah, perhaps they were never there.  I can go back and look at it
18:13 clayton it's been a while since I seriously looked at that patch, I think I originally submitted it as a big batch back in febuary
18:16 hogepodge does anyone know what happened to the rdo openstack-neutron-lbaas packages?
18:16 richm hogepodge: you might try #rdo
18:17 derekh joined #puppet-openstack
18:19 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Beaker tests  https://review.openstack.org/176098
18:19 ibravo joined #puppet-openstack
18:22 openstackgerrit Merged stackforge/puppet-gnocchi: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176596
18:22 openstackgerrit Merged stackforge/puppet-manila: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176602
18:23 openstackgerrit Merged stackforge/puppet-tripleo: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176610
18:23 openstackgerrit Merged stackforge/puppet-sahara: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176607
18:24 openstackgerrit Merged stackforge/puppet-tuskar: Bump rspec-puppet to 2.1.0  https://review.openstack.org/176613
18:24 openstackgerrit Sebastien Badia proposed stackforge/puppet-keystone: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/176936
18:25 EmilienM sbadia: I tried https://review.openstack.org/#/c/176936/1/spec/​defines/keystone_resource_service_identity_spec.rb,cm
18:25 EmilienM and it did not work on 3.X
18:25 EmilienM sbadia: and https://review.openstack.org/#/c/1769​36/1/spec/classes/keystone_ldap_spec.rb,cm is not really a solution :
18:25 EmilienM :P
18:26 sbadia hum erf, I didn't see your patch…
18:26 sbadia for the array it's work on my local env
18:26 sbadia 3.2 3.7 4.0
18:26 sbadia and for the value, I think it's ok, we test the presence of the key, not the value…
18:26 sbadia the value is undef by the way…
18:27 sbadia EmilienM: no?
18:28 EmilienM sbadia: it's okay
18:28 EmilienM sbadia: thanks for pushing that
18:28 sbadia np :-) I'll continue the other modules
18:28 EmilienM \o/
18:32 ericpeterson joined #puppet-openstack
18:35 EmilienM clayton: it's okay, I can approve it
18:35 Hunner Ah, found it. ["admin"] is converted to "admin" because of https://github.com/puppetlabs/puppet/blo​b/3.7.5/lib/puppet/resource.rb#L446-L453
18:35 Hunner EmilienM: ^
18:36 Hunner So anyway, you'll have to conditionalize your unit tests. I usually use `if Puppet.version.to_f >= 4.0`
18:36 EmilienM sbadia: you see ? :P
18:36 EmilienM sbadia: you own me a beer now.
18:36 EmilienM Hunner: thanks :) (I'll transfer the beer to you)
18:36 sbadia Hunner: oh good catch, thanks!
18:37 Hunner That's something I never knew about puppet 4. Kind of lame that 3 ever did that anyway
18:37 Hunner But keeping an array an array is the better behavior
18:37 sbadia +1 :)
18:39 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Update Beaker tests  https://review.openstack.org/174182
18:41 ddmitriev1 joined #puppet-openstack
18:42 rcallawa joined #puppet-openstack
18:43 hogepodge I see the problem, puppet-neutron is trying to install packages that don't exist
18:45 EmilienM hogepodge: which ones?
18:46 hogepodge neutron-lbass-agent and neutron-fwaas-agent
18:46 hogepodge I'm writing up a patch right now
18:48 EmilienM hogepodge: which OS ?
18:50 hogepodge redhat
18:50 EmilienM oops
18:50 EmilienM I remember a patch
18:50 EmilienM from jpena
18:50 EmilienM let me find it
18:50 crinkle https://review.openstack.org/#/c/167226/ and https://review.openstack.org/#/c/161349/
18:50 ibravo joined #puppet-openstack
18:51 crinkle there is a discussion on backwards-incompatible changes http://lists.openstack.org/pipermail/​openstack-dev/2015-April/061640.html
18:51 hogepodge EmilienM: I think it's incorrect
18:52 hogepodge I don't think kilo is adding those packages. Am I wrong?
18:52 dgurtner joined #puppet-openstack
18:52 dgurtner joined #puppet-openstack
18:52 hogepodge Ah, I guess kilo is adding them
18:52 EmilienM crinkle: thx
18:53 hogepodge That's what I get for pulling from master
18:54 crinkle iirc glance did something similar and then someone argued and they decided against it
18:54 crinkle it would be cool if they did the same for fwaas and lbaas
18:58 ericpete_ joined #puppet-openstack
19:00 derekh joined #puppet-openstack
19:01 stamak joined #puppet-openstack
19:23 openstackgerrit Merged stackforge/puppet-heat: Make package_ensure consistent across classes  https://review.openstack.org/148693
19:31 openstackgerrit Sergey Kolekonov proposed stackforge/puppet-neutron: Do not create tun and int bridges manually  https://review.openstack.org/168848
19:36 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
19:41 ibravo joined #puppet-openstack
19:44 EmilienM richm, crinkle: I rebased by glance/beaker patch on top of your provider refacto (just FYI: https://review.openstack.org/#/c/176521/ ) - the result is, I can't create the glance image, but I don't think it's related to your patch.
19:45 crinkle EmilienM: could you target your patch against master to see if that works? then we know whether our changes are breaking it
19:46 EmilienM crinkle: I did that before :P - same result.
19:46 crinkle okay
19:46 EmilienM crinkle: something's wrong in keystone (logs)
19:46 EmilienM crinkle: I'll take time to reproduce & fix it later
19:46 EmilienM I want openstacklib & keystone merged first
19:50 maximov joined #puppet-openstack
19:50 maximov left #puppet-openstack
19:55 ericpeterson joined #puppet-openstack
20:01 maximov joined #puppet-openstack
20:01 jpena joined #puppet-openstack
20:02 jpena left #puppet-openstack
20:06 openstackgerrit Ryan Bak proposed stackforge/puppet-monasca: Allow monasca-agent install from wheels  https://review.openstack.org/174599
20:07 openstackgerrit Ryan Bak proposed stackforge/puppet-monasca: Allow monasca-agent install from wheels  https://review.openstack.org/174599
20:12 rcallawa joined #puppet-openstack
20:18 cdent joined #puppet-openstack
20:20 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
20:20 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Beaker tests  https://review.openstack.org/176098
20:42 Vinsh Anyone here seen this sort of thing "Error: Failed to apply catalog: Could not find filebucket  specified in backup"
20:43 Vinsh I'm hitting this when using the health_check filter from puppet-swift
20:43 Vinsh If I include that concat part.. puppet just fails to run.
20:43 Vinsh Remove that filter.. and its fine.
20:43 Vinsh Can't figure out what caused this.. but it started for me a day ago and I don't see any changes I made to trigger it.
20:46 Vinsh This sounds similar: https://tickets.puppetlabs.com/browse/MODULES-1933
20:46 Vinsh Just not sure what I did to deserve it :)
20:54 Vinsh Found it.. I would expect other users of puppet-swift or any other module that uses concat in a defined resource to hit this.
20:54 Vinsh In the concat module.
20:54 Vinsh commit e34eac0996dd1b6d376c0a974a0ca090d285888f
20:54 Vinsh Author: Bill ONeill <woneill@pobox.com>
20:54 Vinsh Date:   Thu Apr 2 13:18:08 2015 -0400
20:54 Vinsh (MODULES-1700) Change the filebucketing behavior so static_compiler can handle them
20:55 Vinsh will bust things.
21:05 mfisch crinkle: is someone from puppet looking at this issue that Adam found? ^
21:05 mfisch BTW - Vinsh is good for free beers in Vancouver!
21:06 Vinsh beer??? you know my style.. Champagne!!!
21:06 mfisch Poppin' bottles and exporting resources is Adam's way of life
21:06 mfisch I bet the France crew will be all over the champagne
21:06 Vinsh So, they added stuff in MODULE-1700.. we pulled in that change to puppet-concat recently here.
21:07 Vinsh and in MODULE-1933 filed today.. its showing it breaks things
21:09 crinkle mfisch: Vinsh: bmjen is working on it
21:09 Vinsh ORLY?
21:09 Vinsh Spent about 6 hours banging my head on this one :)
21:10 Vinsh Do you think the fix is in the puppet-openstack modules? set some correct backup form? Or maybe they will merge something like what is in 1933 into concat.
21:11 mfisch thanks crinkle
21:11 crinkle Vinsh: hrm? it seems like a concat issue, not sure how it's related to puppet-openstack (or which puppet-openstack module you're talking about)
21:12 Vinsh What I mean to say is:  Maybe there are some settings we need to set in the modules that use concat to work properly.
21:12 Vinsh OR maybe its just bad for concat to do this and they will fix concat.
21:12 Vinsh filebuckets and such.
21:12 xingchao joined #puppet-openstack
21:13 stamak joined #puppet-openstack
21:14 Vinsh If anyone knows how to reach bmjen or what ticket its being worked in.. I can confirm that the last comment in https://tickets.puppetlabs.com/browse/MODULES-1933 fixes this.
21:14 crinkle Vinsh: bmjen is assigned to it in that ticket and i just confirmed with him that he's aware and working on it
21:15 Vinsh oh hah!
21:15 crinkle he's bmjen on irc and he's around in #puppet-dev
21:15 Vinsh Perfect.  Thanks for helping me connect those dots.  Now to go get the last 6 hours back :)
21:15 crinkle :)
21:16 crinkle Vinsh: if you want to submit a pull request and ping him it might make things go faster
21:16 Vinsh OK, reaching out in irc right now. TY
21:17 crinkle yw
21:37 openstackgerrit Merged stackforge/puppet-monasca: Allow monasca-agent install from wheels  https://review.openstack.org/174599
21:43 chem joined #puppet-openstack
21:44 hogepodge EmilienM: I think that this line should be DEFAULT, and not glance_store (for region) https://github.com/stackforge/puppet-glance/b​lob/master/lib/puppet/provider/glance.rb#L19
21:44 EmilienM hogepodge: it changed in Kilo AFIK
21:44 EmilienM I remember a revert about that, that broke Juno
21:45 hogepodge Ok, shouldn't it be backported to juno then?
21:45 hogepodge is master tracking kilo?
21:45 EmilienM hogepodge: well... that's a discussion in progress
21:46 EmilienM https://etherpad.openstack.org/p​/puppet-openstack-master-policy
21:46 hogepodge I see
21:47 chem hi guys, I did a patch for the tempest module that need some love, if someone is interested in reviewing it https://review.openstack.org/#/c/174638/
21:47 hogepodge I guess it's the same story for the neutron providers then? They break on kilo too.
21:47 EmilienM chem: welcome here :-P
21:47 * EmilienM waves on yet another french guy
21:47 chem EmilienM: thanks :)
21:51 derekh joined #puppet-openstack
21:54 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Beaker tests  https://review.openstack.org/155873
21:55 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstacklib: Beaker tests  https://review.openstack.org/176098
21:56 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: V3 support for service provider only  https://review.openstack.org/174695
22:12 EmilienM I would like to start a matrix of our modules jobs status
22:26 EmilienM https://docs.google.com/spreadsheets/d/1i2z5Qyv​ukHCWU_JjkWrTpn-PexPBnt3bWPlQfiDGsj8/edit#gid=0
22:44 aimon joined #puppet-openstack
22:51 openstackgerrit Emilien Macchi proposed stackforge/puppet-glance: beaker: test glance_image provider  https://review.openstack.org/176521
23:02 derekh joined #puppet-openstack
23:03 xingchao joined #puppet-openstack
23:27 xarses joined #puppet-openstack
23:30 panda joined #puppet-openstack
23:44 openstackgerrit Adam Vinsh proposed stackforge/puppet-swift: Add support for dedicated replication network.  https://review.openstack.org/177037

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