Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-06-09

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

All times shown according to UTC.

Time Nick Message
00:04 beddari joined #puppet-openstack
00:11 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
00:20 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
00:28 rcallawa joined #puppet-openstack
00:29 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
00:34 rcallawa joined #puppet-openstack
00:35 openstackgerrit Merged stackforge/puppet-ironic: db: Add postgresql support  https://review.openstack.org/185091
00:41 openstackgerrit Merged stackforge/puppet-openstack_extras: metadata: Synchronize metadata file with module requirements  https://review.openstack.org/182740
01:08 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
01:27 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
01:39 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
01:47 gildub crinkle, hi - I fixed 181600 as far as I could but there is still an error, can you please have a look?
01:50 crinkle gildub: looking
01:50 gildub crinkle, thanks
01:52 crinkle gildub: need to add another section to doc/source/index.rst for kilo
01:53 gildub crinkle, initially auth-restructure.rst was under specs, I just added kilo because I thought that's the right way
01:54 crinkle gildub: it is, but it's the first one there so there needs to be an entry for it in the index
01:54 gildub crinkle, alright, I do that then, thanks
01:54 crinkle np
01:55 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
02:03 gildub crinkle, done! That's going to trigger a chain of merge...
02:04 gildub crinkle, with the BP, the syntax check is pretty tied, I had to remove code examples because that would break the tests
02:07 crinkle gildub: you shouldn't have to remove code examples, there are other examples in that repo that work that you could follow
02:07 crinkle richm helped me with that
02:10 gildub crinkle, ah ok, does the code need to be prefixed, wrapped. Because I got errors in test for patch set 9 for instance
02:11 crinkle gildub: it needs a :: then a blank line then the code block is indented 4 spaces
02:11 crinkle the :: could be following a line of text, e.g. "here is some code::" then blank line then code
02:12 gildub crinkle, ok I'll use that next time. BTW is this described somewhere?
02:13 EmilienM nibalizer: thanks
02:13 crinkle gildub: there are some guides on the internet, but i'm pretty new to it too
02:13 finchd gildub: in the rst spec, like https://getnikola.com/quickstart.html#preformatting-code-samples
02:14 gildub finchd, thanks
02:16 openstackgerrit Emilien Macchi proposed stackforge/puppet-tripleo: Implement Advanced Firewalling support  https://review.openstack.org/189484
02:18 gildub crinkle, ok, I think we're pretty [GREEN] on bp/auth_restructure
02:19 crinkle gildub: \o/
02:20 _morgan joined #puppet-openstack
02:20 Hunner joined #puppet-openstack
02:41 openstackgerrit Merged stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/181600
02:59 markvoelker joined #puppet-openstack
03:01 chem joined #puppet-openstack
03:29 xarses joined #puppet-openstack
04:12 saneax joined #puppet-openstack
04:21 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/178456
04:32 cwolferh joined #puppet-openstack
04:47 aimon joined #puppet-openstack
04:48 markvoelker joined #puppet-openstack
05:35 xarses joined #puppet-openstack
05:39 bkero joined #puppet-openstack
05:52 saneax joined #puppet-openstack
05:53 xarses joined #puppet-openstack
06:17 openstackgerrit Sam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/180378
06:27 simonmcc joined #puppet-openstack
06:29 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
06:32 openstackgerrit Sam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/180378
06:39 petems joined #puppet-openstack
07:04 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
07:22 paramite joined #puppet-openstack
07:26 tfz joined #puppet-openstack
07:26 rhefner joined #puppet-openstack
07:33 jpena joined #puppet-openstack
07:37 markvoelker joined #puppet-openstack
07:40 dgurtner joined #puppet-openstack
07:40 dgurtner joined #puppet-openstack
07:44 tchaypo_ joined #puppet-openstack
07:54 cdent joined #puppet-openstack
08:47 alex_bh joined #puppet-openstack
08:50 derekh joined #puppet-openstack
08:55 openstackgerrit Bogdan Dobrelya proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/174811
09:27 markvoelker joined #puppet-openstack
09:35 gildub joined #puppet-openstack
09:47 openstackgerrit Bogdan Dobrelya proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/174811
09:50 jtomasek joined #puppet-openstack
10:05 dgurtner joined #puppet-openstack
10:21 rcallawa joined #puppet-openstack
10:24 derekh joined #puppet-openstack
10:27 trash hm, https://github.com/stackforge/puppet-keystone/commit/c30afe5aef247cd35b110d9778bab91cd63b5f6a breaks puppet-keystone with apache wsgi, because this file does not exists and then the wsgi won't start.
10:28 trash and it's not optional because it has a default value
10:28 markvoelker joined #puppet-openstack
10:34 trash created a bug for that: https://bugs.launchpad.net/puppet-keystone/+bug/1463358
10:34 openstack Launchpad bug 1463358 in puppet-keystone "default value of paste_config breaks apache+wsgi on Ubuntu" [Undecided,New]
10:50 rcallawa joined #puppet-openstack
10:51 rcallawa joined #puppet-openstack
10:54 dgurtner joined #puppet-openstack
11:05 openstackgerrit Sergey Vasilenko proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/174811
11:28 markvoelker joined #puppet-openstack
11:30 ducttape_ joined #puppet-openstack
11:40 gildub joined #puppet-openstack
11:55 EmilienM good morning
11:55 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/178456
11:57 xingchao joined #puppet-openstack
12:05 dprince joined #puppet-openstack
12:06 ferest joined #puppet-openstack
12:14 prad_ joined #puppet-openstack
12:14 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document Compliant modules  https://review.openstack.org/189388
12:24 xingchao joined #puppet-openstack
12:26 rcallawa joined #puppet-openstack
12:34 openstackgerrit Merged stackforge/puppet-keystone: Rename resource instance variable  https://review.openstack.org/187418
12:35 openstackgerrit Merged stackforge/puppet-heat: Fix ipv6 support  https://review.openstack.org/187424
12:39 xingchao joined #puppet-openstack
12:48 dgurtner joined #puppet-openstack
12:48 dgurtner joined #puppet-openstack
12:49 iurygregory joined #puppet-openstack
12:50 morazi joined #puppet-openstack
12:52 EmilienM social: ping on https://review.openstack.org/#/c/182374 - don't we need unit tests ?
12:53 social EmilienM: I think we should rewrite it to use openstackclient
12:54 social the whole neutron module but probably this is usefull for juno? :/
12:54 social I'll have a look
12:54 EmilienM social: well, one step at a time
12:54 EmilienM social: the work on osclient is still WIP, and your patch is valuable to have it now
12:55 EmilienM social: crinkle +2 yours, but I was wondering if some tests would be great
12:55 social I'm not sure how to make tests for this :/
12:56 openstackgerrit Merged stackforge/puppet-manila: Add notification_driver and state_path option  https://review.openstack.org/177064
13:09 tshtilma joined #puppet-openstack
13:14 openstackgerrit Merged stackforge/puppet-heat: Fix Keystone domain class  https://review.openstack.org/187423
13:14 openstackgerrit Merged stackforge/puppet-glance: Add $notification_driver parameter to notify::rabbitmq  https://review.openstack.org/187340
13:19 openstackgerrit Merged stackforge/puppet-nova: Added parameters for availability zones configuration  https://review.openstack.org/187396
13:19 openstackgerrit Merged stackforge/puppet-neutron: Add MidoNet plugin manifest  https://review.openstack.org/187392
13:21 openstackgerrit Merged stackforge/puppet-nova: crontab: ensure nova-common is installed before  https://review.openstack.org/187401
13:24 openstackgerrit Merged stackforge/puppet-cinder: fixed behavior for rabbit_hosts variable  https://review.openstack.org/188593
13:24 openstackgerrit Merged stackforge/puppet-ceilometer: Clean up stray copy-paste error  https://review.openstack.org/188565
13:25 openstackgerrit Merged stackforge/puppet-glance: Bring Redhat support to acceptance tests  https://review.openstack.org/182673
13:26 richm joined #puppet-openstack
13:26 openstackgerrit Merged stackforge/puppet-glance: Rename class_spec.rb to basic_glance_spec.rb  https://review.openstack.org/182902
13:27 openstackgerrit Merged stackforge/puppet-glance: Correct password parameter for glance::api  https://review.openstack.org/189432
13:28 openstackgerrit Emilien Macchi proposed stackforge/puppet-openstack-specs: Document Compliant modules  https://review.openstack.org/189388
13:35 richm left #puppet-openstack
13:36 richm joined #puppet-openstack
13:50 openstackgerrit Drew Fisher proposed stackforge/puppet-keystone: remove POSIX users, groups and file modes  https://review.openstack.org/185663
13:53 _para joined #puppet-openstack
13:54 _para joined #puppet-openstack
14:00 bdemers joined #puppet-openstack
14:01 saju_m joined #puppet-openstack
14:02 openstackgerrit Merged stackforge/puppet-openstacklib: Adding augeas insertion check. Augeas change call will fail if key does not exists in policy.json. This fix will add the key/value if it doesn't exists or will change the value to match. Added unit test. Change-Id: Ibd97228c431fa6b23e8a15ce548f82f649131cc  https://review.openstack.org/188957
14:02 openstackgerrit Merged stackforge/puppet-cinder: Add configuration helpers for Quobyte  https://review.openstack.org/188592
14:05 openstackgerrit Merged stackforge/puppet-cinder: added HP 3par iscsi backend module  https://review.openstack.org/188595
14:05 openstackgerrit Merged stackforge/puppet-cinder: Add validate_array to NFS server list  https://review.openstack.org/188594
14:05 openstackgerrit Emilien Macchi proposed stackforge/puppet-swift: do not merge - beaker test  https://review.openstack.org/189730
14:20 openstackgerrit Merged stackforge/puppet-neutron: Fix fwaas package name for Ubuntu and Debian  https://review.openstack.org/186961
14:21 openstackgerrit Emilien Macchi proposed stackforge/puppet-neutron: LBaaS: update device_driver for Kilo  https://review.openstack.org/182724
14:24 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/178456
14:27 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Revert "Add paste_config optional parameter"  https://review.openstack.org/189742
14:27 [1]tshtilma joined #puppet-openstack
14:27 EmilienM crinkle, sbadia ^
14:34 EmilienM mgagne: can you justify your +1 on https://review.openstack.org/#/c/176924/ ? (and not +2) ?
14:37 kkirkpatrick joined #puppet-openstack
14:39 openstackgerrit Merged stackforge/puppet-ceilometer: Fix syntax error with postgresql database string  https://review.openstack.org/188566
14:39 openstackgerrit Merged stackforge/puppet-heat: Allow setting default config/signal transport  https://review.openstack.org/182906
14:39 openstackgerrit Merged stackforge/puppet-horizon: Update comment documentation  https://review.openstack.org/188971
14:40 ducttape_ joined #puppet-openstack
14:47 mgagne EmilienM: didn't check the code, just the idea
14:49 EmilienM mgagne: k
14:50 EmilienM reminder: our weekly meeting in 10 minutes
14:55 crinkle morning
14:55 EmilienM crinkle: o/
15:00 tshtilma joined #puppet-openstack
15:00 EmilienM meeting is about to start
15:00 xarses joined #puppet-openstack
15:03 ferest joined #puppet-openstack
15:06 mdorman joined #puppet-openstack
15:07 crinkle _ody: meeting in #openstack-meeting-4 if you're around
15:07 markvoelker joined #puppet-openstack
15:07 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - keystone_role resource  https://review.openstack.org/189764
15:09 ducttape_ joined #puppet-openstack
15:10 ducttape_ joined #puppet-openstack
15:10 _ody join openstack-meeting-4
15:14 rcallawa_ joined #puppet-openstack
15:15 openstackgerrit Rajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver  https://review.openstack.org/189372
15:22 openstackgerrit Ryan Hefner proposed stackforge/puppet-manila: Add option to push service image to Glance  https://review.openstack.org/189771
15:22 rhefner EmilienM: ^^ can you take a look at this when you get a chance? This adds a boolean to resolve the issue you raised here: https://review.openstack.org/#/c/188137/7/puppet/manifests/overcloud_controller.pp
15:22 EmilienM rhefner: in the middle of our weekly meeting now.
15:22 EmilienM will look after
15:22 rhefner EmilienM: no problem. thanks
15:25 openstackgerrit Merged stackforge/puppet-neutron: Allow l3_ha to be disabled  https://review.openstack.org/187389
15:28 openstackgerrit Merged stackforge/puppet-keystone: remove POSIX users, groups and file modes  https://review.openstack.org/185663
15:29 openstackgerrit Merged stackforge/puppet-neutron: Fix support for auth_uri setting in neutron provider  https://review.openstack.org/182374
15:29 arnaud_orange joined #puppet-openstack
15:31 ianw joined #puppet-openstack
15:31 openstackgerrit Merged stackforge/puppet-openstacklib: Unpin beaker-rspec  https://review.openstack.org/189440
15:32 openstackgerrit Merged stackforge/puppet-horizon: Unpin beaker-rspec  https://review.openstack.org/189438
15:34 dgurtner joined #puppet-openstack
15:34 dgurtner joined #puppet-openstack
15:40 tfz joined #puppet-openstack
15:41 tshtilma joined #puppet-openstack
15:43 tshtilma joined #puppet-openstack
15:44 xarses joined #puppet-openstack
15:51 Fin1te joined #puppet-openstack
15:51 tfz joined #puppet-openstack
15:54 dprince Hunner: in TripleO (using puppet) we've come upon a case where we'd like to dump (to a JSON or flat file) the list of packages managed by puppet
15:54 Hunner dprince: Just the packages from a catalog?
15:54 dprince Hunner: wondering if you had any thoughts on wiring up this sort of thing. I was thinking I could tie this into the custom package provider we already use (norpm) and create a JSON file from there
15:54 Hunner And at what time to you want to dump them? During a puppet apply? From a report?
15:54 dprince Hunner: hi, yes
15:55 dprince Hunner: I guess either a puppet apply was the initial thought
15:56 dprince Hunner: essentially we want a mechanism to easily exclude packages managed by puppet from other package based updates mechanisms...
15:56 rcallawa joined #puppet-openstack
15:56 dprince Hunner: stevebaker (one of the Heat guys) went and wrote a sed script to pull data out of /var/lib/puppet to do this... and that seemed potentially risky to me
15:57 Hunner dprince: `catalog.resources.collect { |r| r.title if r.type == :package }.compact` if you're in an apply run with a catalog
15:57 dprince Hunner: a clever idea, just wondering if we should tie this into puppet a bit more cleanly
15:58 Hunner You can make a resource with a provider that basically runs that and writes it out... as a lame first idea
15:58 EmilienM dprince: I use puppet partial for that
15:58 EmilienM it's awesome
15:58 EmilienM dprince: https://github.com/michaeltchapman/puppet-partial
15:58 Hunner so you declare like `count_packages { '/tmp/puppet-packages': }` in the catalog or something
15:58 Hunner ooo
15:58 Hunner oh, that's like tags but without the whole catalog?
15:59 EmilienM yeah
15:59 dprince EmilienM: cool, looking
15:59 EmilienM this is super cool
15:59 EmilienM and written by michael chapman (and me a bit)
15:59 EmilienM I use it to query the catalog and pull out any resource
15:59 Hunner Oh wait, /me re-reads his request
16:00 Hunner Yeah, my suggestion is just for getting a list of the packages to dump, but the puppet-partial sounds better
16:00 Hunner dprince: This is instead of norpm?
16:00 Hunner Oh oh, no the other way around
16:01 dprince Hunner: no, this is to export the list of packages to be used externally
16:01 dprince Hunner: so we can exclude them elsewhere
16:02 Hunner dprince: Where's the best place to have them excluded? On the master as a hiera .yaml file or something
16:03 Hunner you could have a report processor that writes out to a .yaml file the list of packages that you don't want to manage as a hiera value
16:03 dprince Hunner: the exclude lists would ideally be stored locally on all the nodes where we've run 'puppet apply'
16:03 Hunner chicken & egg though
16:03 Hunner Oh, apply, right
16:04 Hunner If you know what will consume it, I can give better workflow. Does it need to modify the catalog at runtime, or something?
16:04 Hunner Like, are you going to build an apt pin list from the packages in the catalog dynamically?
16:05 Hunner cus you could totally do that with a generate method in a type
16:05 Hunner But if you just really want to write out json to somewhere else... I guess you could still do that in generate, but it wouldn't muck with the catalog
16:06 dprince Exactly, we'll want to generate an exclude list for these packages. The idea here is we want to skip updating packages that puppet itself managages elsewhere
16:07 dprince I'm not sure we'd want this thing to edit Yum repo files directly... The update we are designing is script driven so any reasonable file format would do
16:07 Hunner I think no matter what you're wanting, creating a type with a enerate function with the snipped I pasted above and no provider to do whatever it is with the list of packages you want is the way to go
16:08 Hunner Then if you put `exclude_resources { 'package': path => '/tmp/exclude.json' }` or whatever in the catalog, the type's generate() method can read the catalog for all packages and write a json array out to that path
16:08 * _ody tries to figure out this msync thing...
16:09 Hunner _ody: Beware: it commits directly to master by default :D
16:09 crinkle spredzy: ^ set it up
16:09 Hunner I accidentally did that last time
16:09 crinkle Hunner: no that's not possible wit hgerrit
16:09 Hunner crinkle: Oh right! He means non-pl msync
16:09 Hunner github--
16:10 dprince Hunner, EmilienM: thanks for these ideas. I will tinker a bit and see what I can come up with and then relay it back to Steve Baker. :)
16:10 EmilienM dprince: puppet partial changed my life lol
16:11 Hunner dprince: gist it if you're having issues and ping me again :)
16:11 EmilienM dprince: I'm about to afk for lunch but I'll let you know why exactly here in a few
16:11 dprince EmilienM: :)
16:11 Hunner EmilienM: But he WANTS puppet to manage the packages
16:11 Hunner EmilienM: But NOT yum or whatever
16:11 Hunner well, not a script that uses yum to manage packages that isn't puppet
16:11 sbadia arf too late :-( (for the meeting)
16:12 David_Orange joined #puppet-openstack
16:12 Hunner sbadia: Hi! You had a ping at the beginning though :)
16:12 Hunner 08:07         EmilienM | #action sbadia to patch swift module for puppet4
16:12 sbadia yep but I was in the subway :)
16:13 sbadia np, I backlog the minutes
16:13 mfisch EmilienM: shouldnt we have made that paste config location a per OS param?
16:13 EmilienM yes
16:14 mfisch its always been in /etc for Ubuntu
16:14 mfisch should still be movable though...
16:14 mfisch the revert is not a full fix really
16:16 _ody sbadia: Never used it but...hacking through msync to move all the modules to rspec-puppet 2.2.0.  Needed for my puppet-neutron patch on 4.x.  I'll send something to the ML once I figure it out.
16:16 EmilienM mfisch: it's a first start.
16:16 EmilienM mfisch: revert + patch would be the idea
16:16 sbadia _ody: nice! thanks
16:17 sbadia I'll update my last patch on msync then
16:17 EmilienM dprince, Hunner: so I use puppet partial to determine which services/packages are managed by Puppet on one specific node so I can calculate and generate Ansible playbooks to upgrade a platform
16:17 EmilienM mfisch, crinkle: if you remember the tool I was mentioning during our Puppet diner (about upgrades): https://github.com/enovance/pitstop/blob/master/README.rst - it use puppet partial
16:18 mfisch EmilienM: agreed its a atsrt
16:18 mfisch a start!
16:19 dprince EmilienM: that is like exactly the same thing we need to do in TripleO
16:19 EmilienM heh
16:19 EmilienM dprince: this is how we do in SpinalStack
16:19 dprince EmilienM: cool
16:24 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: Manage default configs  https://review.openstack.org/177959
16:25 openstackgerrit Merged stackforge/puppet-keystone: Revert "Add paste_config optional parameter"  https://review.openstack.org/189742
16:32 bkero joined #puppet-openstack
16:48 David_Orange left #puppet-openstack
16:56 markvoelker joined #puppet-openstack
16:56 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: Synchronize nodeset (split trusty and centos)  https://review.openstack.org/184352
17:06 nibalizer huh
17:06 nibalizer so this is different than puppet apply --tags =things
17:06 nibalizer ?
17:07 openstackgerrit Merged stackforge/puppet-manila: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173872
17:09 openstackgerrit Merged stackforge/puppet-sahara: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173891
17:11 openstackgerrit Merged stackforge/puppet-ironic: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173495
17:11 openstackgerrit Merged stackforge/puppet-ceilometer: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173481
17:13 openstackgerrit Merged stackforge/puppet-heat: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173506
17:13 openstackgerrit Merged stackforge/puppet-heat: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173506
17:14 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: Add a readme for this repository  https://review.openstack.org/189819
17:18 openstackgerrit Merged stackforge/puppet-ceilometer: Unpin beaker-rspec  https://review.openstack.org/189421
17:18 openstackgerrit Merged stackforge/puppet-cinder: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/173491
17:19 rhefner EmilienM: just a reminder, can you take a look at https://review.openstack.org/#/c/189771/ when you get a chance?
17:21 ianw joined #puppet-openstack
17:41 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
17:48 crinkle hmm we decided to try out option #2 for the abandonment policy but didn't decide on an N for how long to leave it
17:48 crinkle anyone want to give a quick +1/-1 for 3 months?
17:57 xarses joined #puppet-openstack
18:00 barthalion +1 for month
18:00 xingchao joined #puppet-openstack
18:08 EmilienM crinkle: 3 months is good imho
18:17 morazi joined #puppet-openstack
18:19 EmilienM rhefner: looking now
18:24 EmilienM rhefner: I dropped a comment, not worth for -1 but still can you look
18:27 openstackgerrit Ryan Hefner proposed stackforge/puppet-manila: Add option to push service image to Glance  https://review.openstack.org/189771
18:27 rhefner EmilienM: ^
18:27 EmilienM fast!
18:28 rhefner EmilienM: thanks
18:37 iurygregory joined #puppet-openstack
18:45 markvoelker joined #puppet-openstack
18:49 openstackgerrit joined #puppet-openstack
18:53 saju_m joined #puppet-openstack
19:01 ianw joined #puppet-openstack
19:02 openstackgerrit Merged stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-ceilometer: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189862
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-cinder: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189863
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-designate: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189864
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-glance: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189865
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-gnocchi: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189866
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-heat: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189868
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-horizon: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189869
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-ironic: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189870
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-keystone: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189871
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-manila: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189872
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189873
19:20 _ody ...well that seems to be working
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-nova: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189874
19:20 openstackgerrit Cody Herriges proposed stackforge/puppet-openstacklib: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189875
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-openstack_extras: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189876
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-sahara: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189877
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-swift: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189878
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-tempest: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189879
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-tripleo: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189880
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-trove: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189881
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-tuskar: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189882
19:21 openstackgerrit Cody Herriges proposed stackforge/puppet-vswitch: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189883
19:21 EmilienM _ody: you're not kidding when you submit a patch!
19:21 dprince joined #puppet-openstack
19:22 crinkle :D
19:22 _ody ugh...except I did it wrong.
19:22 EmilienM lol
19:23 _ody Well they are in for review...I'll just get them fixed up from there.
19:23 _ody ...oh and email the ML.
19:27 openstackgerrit Cody Herriges proposed stackforge/puppet-modulesync-configs: Move to rspec-puppet 2.2.0  https://review.openstack.org/189886
19:41 jtomasek joined #puppet-openstack
19:42 EmilienM Hunner: ping
19:55 openstackgerrit Dan Prince proposed stackforge/puppet-keystone: Revert "remove POSIX users, groups and file modes"  https://review.openstack.org/189892
19:56 dprince EmilienM: ^^^ seeing all the puppet jobs fail now
19:56 EmilienM oh dear
19:57 mgagne looks like we are not properly testing dependencies
19:57 EmilienM clayton, mfisch ^
19:57 mgagne catalog compiles fine (in rspec tests) with missing deps...
19:57 EmilienM and the review was a bit fast imho
19:57 EmilienM it still had some User resources.
19:57 EmilienM I'm doing a patch now
19:58 mgagne "some" ?
19:58 EmilienM mgagne: ?
19:58 mgagne you mean references to now removed user resource?
20:00 mfisch revert looks good
20:00 EmilienM the question is how beaker could have worked
20:00 EmilienM mfisch: no.
20:00 EmilienM mfisch: I'm fixing it now
20:00 EmilienM I'm done almost
20:00 mfisch whats wrong with it?
20:00 EmilienM I don't want to revert something we can fix in 1 minute
20:01 dprince EmilienM: I don't mind rolling forward... but would point out a revert is always safe
20:01 openstackgerrit Emilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all  https://review.openstack.org/189908
20:01 EmilienM please review ^
20:01 dprince EmilienM: if it doesn't fix it I'll propose 2 ;)
20:01 * dprince doesn't mix reverts
20:01 EmilienM mfisch, clayton: how come you merge patches that do not pass beaker?
20:02 EmilienM mgagne: our CI detected it but jobs are non voting.
20:02 EmilienM http://logs.openstack.org/63/185663/3/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/2424588/console.html
20:02 EmilienM mgagne, mgagne, clayton ^
20:02 EmilienM it was quite explicit
20:02 clayton why can I merge reviews that don't pass beaker?
20:03 EmilienM because we decided for now there are non voting
20:03 EmilienM and I already asked reviewers to have a look during reviews
20:03 EmilienM it's non voting now because we did not want to slow down CI process while we were working on these jobs
20:03 EmilienM I proposed https://review.openstack.org/#/c/176343/ to make it vote
20:03 mgagne EmilienM: I think we should consider non-voting jobs as if no one look at them
20:04 EmilienM mgagne: well, I wish we could look at it
20:04 mgagne EmilienM: sure
20:04 clayton I will in the future
20:04 EmilienM if it was only me, they would vote.
20:04 mgagne EmilienM: so we don't have unrealistic expectations
20:04 EmilienM but if I do that
20:04 EmilienM people will complain a lot
20:05 EmilienM mgagne, clayton, crinkle, mfisch: do you agree to make them vote *now* ?
20:06 EmilienM looking at https://docs.google.com/spreadsheets/d/1i2z5QyvukHCWU_JjkWrTpn-PexPBnt3bWPlQfiDGsj8/edit#gid=0
20:06 EmilienM they are all green
20:06 clayton I'm willing to trust your judgement on that.  I'm also willing to make sure I look at them in the future.
20:06 EmilienM clayton: don't worry we all make mistakes, me first.
20:06 EmilienM hopefully our human-CI (called dprince) is here :P
20:21 kkirkpatrick joined #puppet-openstack
20:21 _ody sbadia: Has msync ever been ran before across all the modules?
20:23 rhefner joined #puppet-openstack
20:23 tchaypo joined #puppet-openstack
20:24 simonmcc joined #puppet-openstack
20:32 openstackgerrit Cody Herriges proposed stackforge/puppet-modulesync-configs: Manage default configs  https://review.openstack.org/177959
20:34 markvoelker joined #puppet-openstack
20:40 EmilienM crinkle: in beaker, why don't we see all puppet logs?
20:40 EmilienM I remember we could see them before
20:40 EmilienM keystone still fails and I don't see why, see https://review.openstack.org/177959
20:40 EmilienM https://jenkins04.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/38/console
20:43 EmilienM nibalizer: maybe I've missed a change while I was away?
20:44 nibalizer EmilienM: ?
20:44 nibalizer EmilienM: are you pinning concat?
20:44 EmilienM nibalizer: in beaker jobs I don't see puppet logs as before
20:44 nibalizer because if you use concat 2.x right now, you're gonna have a bad time
20:44 nibalizer EmilienM: i think crinkle moved us from beaker 2.x to 5.x
20:45 dprince EmilienM: I'm going to cherry pick my revert into TripleO CI for now....
20:45 EmilienM dprince: my fix does not work. I'm merging your patch
20:45 dprince EmilienM: if you look at the revert it does fix breaker too
20:45 EmilienM oh, looking
20:45 openstackgerrit Cody Herriges proposed stackforge/puppet-modulesync-configs: Move to rspec-puppet 2.2.0  https://review.openstack.org/189886
20:46 EmilienM dprince: oh yes, sure
20:46 EmilienM dprince: because you revert what breaks beaker :)
20:47 EmilienM mgagne, mgagne, clayton or crinkle: if you around, can you +2 https://review.openstack.org/#/c/189892 ?
20:47 clayton looking
20:47 EmilienM thanks
20:48 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189930
20:48 crinkle EmilienM: nibalizer oh yeah beaker 5 doesn't output logs by default :(
20:48 crinkle i can turn that on
20:48 EmilienM crinkle: that would be awesome
20:49 EmilienM crinkle: can you do that on keystone to start, and put my patch on top of this one ? ( https://review.openstack.org/#/c/189908 ) so we can see what's wrong without User/Group management
20:49 EmilienM crinkle: I can do 2nd action
20:50 dprince EmilienM: thanks, carry on as normal...
20:50 EmilienM dprince: the revert is merged very soon.
20:51 stevemar joined #puppet-openstack
20:51 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189873
20:56 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
20:57 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
20:57 openstackgerrit Merged stackforge/puppet-keystone: Revert "remove POSIX users, groups and file modes"  https://review.openstack.org/189892
20:57 crinkle EmilienM: https://github.com/puppetlabs/beaker/wiki/How-to-Write-a-Beaker-Test-for-a-Module#supported-env-variables
20:57 crinkle er
20:57 crinkle https://review.openstack.org/189937
20:57 crinkle is what i meant to paste
20:57 crinkle EmilienM: ^
20:57 EmilienM crinkle: thanks
20:58 crinkle Hunner: nibalizer could you review ^
20:59 nibalizer lgtm
21:06 mfisch are part of the gerrit project rename outage?  http://lists.openstack.org/pipermail/openstack-infra/2015-June/002815.html
21:07 crinkle if we get https://review.openstack.org/#/c/176326/ approved in time we can ask them to add us in
21:13 EmilienM crinkle: a Friday?
21:13 EmilienM well, before the week end
21:13 crinkle EmilienM: they need to take down the system so they have to do it on a low traffic day
21:26 EmilienM crinkle: ok done, I've updated https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting#Upcoming_Project_Renames
21:26 EmilienM crinkle: now I'm sending an email to our community
21:28 rcallawa joined #puppet-openstack
21:29 rcallawa joined #puppet-openstack
21:58 mdorman joined #puppet-openstack
22:20 cwolferh crinkle, i've added tests (and comment) for https://review.openstack.org/#/c/188200/ -- please take a look at your convenience
22:26 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/189873
22:27 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
22:34 crinkle cwolferh: so the package name is puppet-rdb on fedora?
22:37 xingchao joined #puppet-openstack
22:38 cwolferh crinkle, not sure
22:38 cwolferh python-rbd is the future though so if it isn't there yet, it will be
22:38 crinkle cwolferh: okay
22:40 cwolferh crinkle, thanks.  also for the record just verified it is in fedora
22:41 crinkle cwolferh: okay cool
22:52 wabu joined #puppet-openstack
23:01 kev joined #puppet-openstack
23:06 gildub joined #puppet-openstack
23:21 markvoelker joined #puppet-openstack
23:27 xingchao joined #puppet-openstack
23:33 stevemar joined #puppet-openstack

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