Tuesday, 2015-06-09

*** beddari has quit IRC00:03
*** beddari has joined #puppet-openstack00:04
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160000:11
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160000:20
*** rcallawa has joined #puppet-openstack00:28
*** saneax has quit IRC00:28
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160000:29
*** rcallawa has quit IRC00:33
*** rcallawa has joined #puppet-openstack00:34
openstackgerritMerged stackforge/puppet-ironic: db: Add postgresql support  https://review.openstack.org/18509100:36
openstackgerritMerged stackforge/puppet-openstack_extras: metadata: Synchronize metadata file with module requirements  https://review.openstack.org/18274000:41
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160001:08
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160001:28
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160001:39
gildubcrinkle, hi - I fixed 181600 as far as I could but there is still an error, can you please have a look?01:47
crinklegildub: looking01:50
gildubcrinkle, thanks01:50
crinklegildub: need to add another section to doc/source/index.rst for kilo01:52
gildubcrinkle, initially auth-restructure.rst was under specs, I just added kilo because I thought that's the right way01:53
crinklegildub: it is, but it's the first one there so there needs to be an entry for it in the index01:54
gildubcrinkle, alright, I do that then, thanks01:54
crinklenp01:54
openstackgerritGilles Dubreuil proposed stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160001:55
*** xingchao has quit IRC01:58
gildubcrinkle, done! That's going to trigger a chain of merge...02:03
gildubcrinkle, with the BP, the syntax check is pretty tied, I had to remove code examples because that would break the tests02:05
crinklegildub: you shouldn't have to remove code examples, there are other examples in that repo that work that you could follow02:07
crinklerichm helped me with that02:07
gildubcrinkle, ah ok, does the code need to be prefixed, wrapped. Because I got errors in test for patch set 9 for instance02:10
crinklegildub: it needs a :: then a blank line then the code block is indented 4 spaces02:11
crinklethe :: could be following a line of text, e.g. "here is some code::" then blank line then code02:11
gildubcrinkle, ok I'll use that next time. BTW is this described somewhere?02:12
EmilienMnibalizer: thanks02:13
crinklegildub: there are some guides on the internet, but i'm pretty new to it too02:13
finchdgildub: in the rst spec, like https://getnikola.com/quickstart.html#preformatting-code-samples02:13
gildubfinchd, thanks02:14
*** Hunner has quit IRC02:16
openstackgerritEmilien Macchi proposed stackforge/puppet-tripleo: Implement Advanced Firewalling support  https://review.openstack.org/18948402:16
gildubcrinkle, ok, I think we're pretty [GREEN] on bp/auth_restructure02:18
crinklegildub: \o/02:19
*** _morgan has quit IRC02:19
*** _morgan has joined #puppet-openstack02:20
*** Hunner has joined #puppet-openstack02:20
openstackgerritMerged stackforge/puppet-openstack-specs: Restructure authentication for resource providers  https://review.openstack.org/18160002:41
*** chem has quit IRC02:52
*** rushiagr_away is now known as rushiagr02:54
*** markvoelker has joined #puppet-openstack02:59
*** chem has joined #puppet-openstack03:01
*** markvoelker has quit IRC03:04
*** rhefner has quit IRC03:06
*** simonmcc has quit IRC03:06
*** petems has quit IRC03:07
*** rushiagr is now known as rushiagr_away03:28
*** xarses has joined #puppet-openstack03:29
*** richm has quit IRC03:42
*** xarses has quit IRC03:55
*** rushiagr_away is now known as rushiagr04:07
*** saneax has joined #puppet-openstack04:12
*** cwolferh has quit IRC04:20
openstackgerritGilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/17845604:21
*** cwolferh has joined #puppet-openstack04:32
*** gildub has quit IRC04:44
*** aimon has joined #puppet-openstack04:47
*** markvoelker has joined #puppet-openstack04:48
*** markvoelker has quit IRC04:53
*** xarses has joined #puppet-openstack05:35
*** bkero has quit IRC05:38
*** saneax has quit IRC05:38
*** bkero has joined #puppet-openstack05:39
*** xarses has quit IRC05:49
*** saneax has joined #puppet-openstack05:52
*** xarses has joined #puppet-openstack05:53
openstackgerritSam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/18037806:18
*** simonmcc has joined #puppet-openstack06:27
openstackgerritsac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/18445206:29
openstackgerritSam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/18037806:32
*** petems has joined #puppet-openstack06:39
*** tfz has quit IRC06:52
*** tchaypo has quit IRC06:53
openstackgerritsac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/18445207:04
*** paramite has joined #puppet-openstack07:22
*** jtomasek has quit IRC07:23
*** tfz has joined #puppet-openstack07:26
*** rhefner has joined #puppet-openstack07:26
*** jpena has joined #puppet-openstack07:33
*** markvoelker has joined #puppet-openstack07:37
*** dgurtner has joined #puppet-openstack07:40
*** dgurtner has quit IRC07:40
*** dgurtner has joined #puppet-openstack07:40
*** markvoelker has quit IRC07:43
*** tchaypo_ has joined #puppet-openstack07:44
*** cdent has joined #puppet-openstack07:54
*** tchaypo_ is now known as tchaypo07:58
*** paramite is now known as paramite|afk08:20
*** paramite|afk is now known as paramite08:29
*** alex_bh has joined #puppet-openstack08:47
*** derekh has joined #puppet-openstack08:50
openstackgerritBogdan Dobrelya proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/17481108:55
*** rushiagr is now known as rushiagr_away08:58
*** rushiagr_away is now known as rushiagr09:00
*** markvoelker has joined #puppet-openstack09:27
*** markvoelker has quit IRC09:32
*** gildub has joined #puppet-openstack09:35
openstackgerritBogdan Dobrelya proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/17481109:47
*** jtomasek has joined #puppet-openstack09:50
*** dgurtner has quit IRC10:04
*** dgurtner has joined #puppet-openstack10:05
*** derekh has quit IRC10:11
*** rcallawa has quit IRC10:20
*** rcallawa has joined #puppet-openstack10:21
*** rcallawa has quit IRC10:23
*** derekh has joined #puppet-openstack10:24
trashhm, 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:27
trashand it's not optional because it has a default value10:28
*** markvoelker has joined #puppet-openstack10:28
*** gchamoul is now known as gchamoul|aFk10:33
*** markvoelker has quit IRC10:33
trashcreated a bug for that: https://bugs.launchpad.net/puppet-keystone/+bug/146335810:34
openstackLaunchpad bug 1463358 in puppet-keystone "default value of paste_config breaks apache+wsgi on Ubuntu" [Undecided,New]10:34
*** dgurtner has quit IRC10:42
*** rcallawa has joined #puppet-openstack10:51
*** rcallawa has quit IRC10:51
*** rcallawa has joined #puppet-openstack10:51
*** gildub has quit IRC10:52
*** dgurtner has joined #puppet-openstack10:55
*** rushiagr is now known as rushiagr_away11:00
*** rcallawa has quit IRC11:00
openstackgerritSergey Vasilenko proposed stackforge/puppet-nova: Make package 'bridge-utils' install optional  https://review.openstack.org/17481111:05
*** spredzy is now known as spredzy|afk11:09
*** markvoelker has joined #puppet-openstack11:29
*** ducttape_ has joined #puppet-openstack11:30
*** markvoelker has quit IRC11:34
*** gildub has joined #puppet-openstack11:40
*** gchamoul|aFk is now known as gchamoul11:45
*** ducttape_ has quit IRC11:49
EmilienMgood morning11:55
openstackgerritGilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/17845611:55
*** xingchao has joined #puppet-openstack11:57
*** dprince has joined #puppet-openstack12:05
*** bdemers has quit IRC12:05
*** ferest has joined #puppet-openstack12:06
*** gildub has quit IRC12:09
*** ferest has quit IRC12:10
*** prad_ has joined #puppet-openstack12:14
*** prad has quit IRC12:14
*** prad_ is now known as prad12:14
openstackgerritEmilien Macchi proposed stackforge/puppet-openstack-specs: Document Compliant modules  https://review.openstack.org/18938812:15
*** xingchao has quit IRC12:16
*** jayg|g0n3 is now known as jayg12:18
*** jpena is now known as jpena|lunch12:19
*** xingchao has joined #puppet-openstack12:24
*** rcallawa has joined #puppet-openstack12:26
*** saneax has quit IRC12:27
openstackgerritMerged stackforge/puppet-keystone: Rename resource instance variable  https://review.openstack.org/18741812:34
openstackgerritMerged stackforge/puppet-heat: Fix ipv6 support  https://review.openstack.org/18742412:35
*** xingchao has quit IRC12:39
*** xingchao has joined #puppet-openstack12:39
*** dgurtner has quit IRC12:47
*** dgurtner has joined #puppet-openstack12:49
*** dgurtner has joined #puppet-openstack12:49
*** iurygregory has joined #puppet-openstack12:49
*** morazi has joined #puppet-openstack12:50
EmilienMsocial: ping on https://review.openstack.org/#/c/182374 - don't we need unit tests ?12:52
socialEmilienM: I think we should rewrite it to use openstackclient12:53
socialthe whole neutron module but probably this is usefull for juno? :/12:54
socialI'll have a look12:54
EmilienMsocial: well, one step at a time12:54
EmilienMsocial: the work on osclient is still WIP, and your patch is valuable to have it now12:54
EmilienMsocial: crinkle +2 yours, but I was wondering if some tests would be great12:55
socialI'm not sure how to make tests for this :/12:55
openstackgerritMerged stackforge/puppet-manila: Add notification_driver and state_path option  https://review.openstack.org/17706412:56
*** tshtilma has joined #puppet-openstack13:09
openstackgerritMerged stackforge/puppet-heat: Fix Keystone domain class  https://review.openstack.org/18742313:14
*** spredzy|afk is now known as spredzy13:14
openstackgerritMerged stackforge/puppet-glance: Add $notification_driver parameter to notify::rabbitmq  https://review.openstack.org/18734013:14
openstackgerritMerged stackforge/puppet-nova: Added parameters for availability zones configuration  https://review.openstack.org/18739613:19
openstackgerritMerged stackforge/puppet-neutron: Add MidoNet plugin manifest  https://review.openstack.org/18739213:19
openstackgerritMerged stackforge/puppet-nova: crontab: ensure nova-common is installed before  https://review.openstack.org/18740113:21
openstackgerritMerged stackforge/puppet-cinder: fixed behavior for rabbit_hosts variable  https://review.openstack.org/18859313:24
openstackgerritMerged stackforge/puppet-ceilometer: Clean up stray copy-paste error  https://review.openstack.org/18856513:24
openstackgerritMerged stackforge/puppet-glance: Bring Redhat support to acceptance tests  https://review.openstack.org/18267313:25
*** richm has joined #puppet-openstack13:26
openstackgerritMerged stackforge/puppet-glance: Rename class_spec.rb to basic_glance_spec.rb  https://review.openstack.org/18290213:26
openstackgerritMerged stackforge/puppet-glance: Correct password parameter for glance::api  https://review.openstack.org/18943213:27
openstackgerritEmilien Macchi proposed stackforge/puppet-openstack-specs: Document Compliant modules  https://review.openstack.org/18938813:28
*** richm has left #puppet-openstack13:35
*** richm has joined #puppet-openstack13:36
*** jpena|lunch is now known as jpena13:39
openstackgerritDrew Fisher proposed stackforge/puppet-keystone: remove POSIX users, groups and file modes  https://review.openstack.org/18566313:50
*** _para has joined #puppet-openstack13:53
*** _para has quit IRC13:54
*** _para has joined #puppet-openstack13:54
*** _para has quit IRC13:59
*** bdemers has joined #puppet-openstack14:00
*** saju_m has joined #puppet-openstack14:01
*** alex_bh has quit IRC14:01
openstackgerritMerged 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/18895714:02
openstackgerritMerged stackforge/puppet-cinder: Add configuration helpers for Quobyte  https://review.openstack.org/18859214:02
openstackgerritMerged stackforge/puppet-cinder: added HP 3par iscsi backend module  https://review.openstack.org/18859514:05
openstackgerritMerged stackforge/puppet-cinder: Add validate_array to NFS server list  https://review.openstack.org/18859414:05
openstackgerritEmilien Macchi proposed stackforge/puppet-swift: do not merge - beaker test  https://review.openstack.org/18973014:05
*** gchamoul is now known as gchamoul|BRB14:13
openstackgerritMerged stackforge/puppet-neutron: Fix fwaas package name for Ubuntu and Debian  https://review.openstack.org/18696114:20
openstackgerritEmilien Macchi proposed stackforge/puppet-neutron: LBaaS: update device_driver for Kilo  https://review.openstack.org/18272414:21
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/17845614:24
openstackgerritEmilien Macchi proposed stackforge/puppet-keystone: Revert "Add paste_config optional parameter"  https://review.openstack.org/18974214:27
*** [1]tshtilma has joined #puppet-openstack14:27
EmilienMcrinkle, sbadia ^14:27
*** tshtilma has quit IRC14:28
*** [1]tshtilma is now known as tshtilma14:28
*** tshtilma has quit IRC14:33
EmilienMmgagne: can you justify your +1 on https://review.openstack.org/#/c/176924/ ? (and not +2) ?14:34
*** rushiagr_away is now known as rushiagr14:37
*** kkirkpatrick has joined #puppet-openstack14:37
openstackgerritMerged stackforge/puppet-ceilometer: Fix syntax error with postgresql database string  https://review.openstack.org/18856614:39
openstackgerritMerged stackforge/puppet-heat: Allow setting default config/signal transport  https://review.openstack.org/18290614:39
openstackgerritMerged stackforge/puppet-horizon: Update comment documentation  https://review.openstack.org/18897114:39
*** ducttape_ has joined #puppet-openstack14:40
*** tfz has quit IRC14:42
*** ducttape_ has quit IRC14:43
*** gchamoul|BRB is now known as gchamoul14:46
mgagneEmilienM: didn't check the code, just the idea14:47
*** saju_m has quit IRC14:49
*** xarses has quit IRC14:49
EmilienMmgagne: k14:49
EmilienMreminder: our weekly meeting in 10 minutes14:50
*** kbyrne has quit IRC14:54
crinklemorning14:55
EmilienMcrinkle: o/14:55
*** tshtilma has joined #puppet-openstack15:00
EmilienMmeeting is about to start15:00
*** xarses has joined #puppet-openstack15:00
*** ferest has joined #puppet-openstack15:03
*** bdemers has quit IRC15:04
*** mdorman has joined #puppet-openstack15:06
crinkle_ody: meeting in #openstack-meeting-4 if you're around15:07
*** markvoelker has joined #puppet-openstack15:07
*** ferest has quit IRC15:07
openstackgerritRichard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - keystone_role resource  https://review.openstack.org/18976415:07
*** ducttape_ has joined #puppet-openstack15:09
*** ducttape_ has quit IRC15:09
*** ducttape_ has joined #puppet-openstack15:10
_odyjoin openstack-meeting-415:10
*** markvoelker has quit IRC15:12
*** rcallawa_ has joined #puppet-openstack15:14
openstackgerritRajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver  https://review.openstack.org/18937215:15
*** rcallawa has quit IRC15:18
openstackgerritRyan Hefner proposed stackforge/puppet-manila: Add option to push service image to Glance  https://review.openstack.org/18977115:22
rhefnerEmilienM: ^^ 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.pp15:22
EmilienMrhefner: in the middle of our weekly meeting now.15:22
EmilienMwill look after15:22
rhefnerEmilienM: no problem. thanks15:22
openstackgerritMerged stackforge/puppet-neutron: Allow l3_ha to be disabled  https://review.openstack.org/18738915:25
openstackgerritMerged stackforge/puppet-keystone: remove POSIX users, groups and file modes  https://review.openstack.org/18566315:28
*** dgurtner has quit IRC15:28
openstackgerritMerged stackforge/puppet-neutron: Fix support for auth_uri setting in neutron provider  https://review.openstack.org/18237415:29
*** arnaud_orange has joined #puppet-openstack15:29
*** ianw has quit IRC15:30
*** ianw has joined #puppet-openstack15:31
openstackgerritMerged stackforge/puppet-openstacklib: Unpin beaker-rspec  https://review.openstack.org/18944015:31
*** xarses has quit IRC15:32
openstackgerritMerged stackforge/puppet-horizon: Unpin beaker-rspec  https://review.openstack.org/18943815:32
*** dgurtner has joined #puppet-openstack15:34
*** dgurtner has quit IRC15:34
*** dgurtner has joined #puppet-openstack15:34
*** paramite has quit IRC15:39
*** tfz has joined #puppet-openstack15:40
*** tshtilma has quit IRC15:40
*** tfz has quit IRC15:41
*** tshtilma has joined #puppet-openstack15:41
*** tshtilma has quit IRC15:42
*** tshtilma has joined #puppet-openstack15:43
*** xarses has joined #puppet-openstack15:44
*** Fin1te has joined #puppet-openstack15:51
*** tfz has joined #puppet-openstack15:51
*** xarses has quit IRC15:52
dprinceHunner: 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 puppet15:54
Hunnerdprince: Just the packages from a catalog?15:54
dprinceHunner: 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 there15:54
HunnerAnd at what time to you want to dump them? During a puppet apply? From a report?15:54
dprinceHunner: hi, yes15:54
dprinceHunner: I guess either a puppet apply was the initial thought15:55
*** rcallawa_ has quit IRC15:55
dprinceHunner: essentially we want a mechanism to easily exclude packages managed by puppet from other package based updates mechanisms...15:56
*** rcallawa has joined #puppet-openstack15:56
dprinceHunner: 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 me15:56
Hunnerdprince: `catalog.resources.collect { |r| r.title if r.type == :package }.compact` if you're in an apply run with a catalog15:57
dprinceHunner: a clever idea, just wondering if we should tie this into puppet a bit more cleanly15:57
HunnerYou can make a resource with a provider that basically runs that and writes it out... as a lame first idea15:58
EmilienMdprince: I use puppet partial for that15:58
EmilienMit's awesome15:58
EmilienMdprince: https://github.com/michaeltchapman/puppet-partial15:58
Hunnerso you declare like `count_packages { '/tmp/puppet-packages': }` in the catalog or something15:58
Hunnerooo15:58
Hunneroh, that's like tags but without the whole catalog?15:58
EmilienMyeah15:59
dprinceEmilienM: cool, looking15:59
EmilienMthis is super cool15:59
EmilienMand written by michael chapman (and me a bit)15:59
EmilienMI use it to query the catalog and pull out any resource15:59
HunnerOh wait, /me re-reads his request15:59
HunnerYeah, my suggestion is just for getting a list of the packages to dump, but the puppet-partial sounds better16:00
Hunnerdprince: This is instead of norpm?16:00
HunnerOh oh, no the other way around16:00
dprinceHunner: no, this is to export the list of packages to be used externally16:01
dprinceHunner: so we can exclude them elsewhere16:01
*** cdent has quit IRC16:01
Hunnerdprince: Where's the best place to have them excluded? On the master as a hiera .yaml file or something16:02
Hunneryou 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 value16:03
dprinceHunner: the exclude lists would ideally be stored locally on all the nodes where we've run 'puppet apply'16:03
Hunnerchicken & egg though16:03
HunnerOh, apply, right16:03
HunnerIf you know what will consume it, I can give better workflow. Does it need to modify the catalog at runtime, or something?16:04
HunnerLike, are you going to build an apt pin list from the packages in the catalog dynamically?16:04
Hunnercus you could totally do that with a generate method in a type16:05
HunnerBut 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 catalog16:05
dprinceExactly, 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 elsewhere16:06
dprinceI'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 do16:07
HunnerI 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 go16:07
HunnerThen 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 path16:08
* _ody tries to figure out this msync thing...16:08
Hunner_ody: Beware: it commits directly to master by default :D16:09
crinklespredzy: ^ set it up16:09
HunnerI accidentally did that last time16:09
crinkleHunner: no that's not possible wit hgerrit16:09
Hunnercrinkle: Oh right! He means non-pl msync16:09
Hunnergithub--16:09
dprinceHunner, 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
EmilienMdprince: puppet partial changed my life lol16:10
Hunnerdprince: gist it if you're having issues and ping me again :)16:11
EmilienMdprince: I'm about to afk for lunch but I'll let you know why exactly here in a few16:11
dprinceEmilienM: :)16:11
HunnerEmilienM: But he WANTS puppet to manage the packages16:11
HunnerEmilienM: But NOT yum or whatever16:11
Hunnerwell, not a script that uses yum to manage packages that isn't puppet16:11
sbadiaarf too late :-( (for the meeting)16:11
*** David_Orange has joined #puppet-openstack16:12
Hunnersbadia: Hi! You had a ping at the beginning though :)16:12
Hunner08:07         EmilienM | #action sbadia to patch swift module for puppet416:12
sbadiayep but I was in the subway :)16:12
sbadianp, I backlog the minutes16:13
mfischEmilienM: shouldnt we have made that paste config location a per OS param?16:13
EmilienMyes16:13
mfischits always been in /etc for Ubuntu16:14
mfischshould still be movable though...16:14
mfischthe revert is not a full fix really16:14
_odysbadia: 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
EmilienMmfisch: it's a first start.16:16
EmilienMmfisch: revert + patch would be the idea16:16
sbadia_ody: nice! thanks16:16
sbadiaI'll update my last patch on msync then16:17
EmilienMdprince, 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 platform16:17
EmilienMmfisch, 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 partial16:17
mfischEmilienM: agreed its a atsrt16:18
mfischa start!16:18
dprinceEmilienM: that is like exactly the same thing we need to do in TripleO16:19
EmilienMheh16:19
EmilienMdprince: this is how we do in SpinalStack16:19
dprinceEmilienM: cool16:19
*** derekh has quit IRC16:23
openstackgerritSebastien Badia proposed stackforge/puppet-modulesync-configs: Manage default configs  https://review.openstack.org/17795916:24
*** tshtilma has quit IRC16:24
openstackgerritMerged stackforge/puppet-keystone: Revert "Add paste_config optional parameter"  https://review.openstack.org/18974216:25
*** jtomasek has quit IRC16:30
*** bkero has quit IRC16:32
*** bkero has joined #puppet-openstack16:32
*** Fin1te has quit IRC16:42
*** David_Orange has left #puppet-openstack16:48
*** markvoelker has joined #puppet-openstack16:56
openstackgerritSebastien Badia proposed stackforge/puppet-modulesync-configs: Synchronize nodeset (split trusty and centos)  https://review.openstack.org/18435216:56
*** xingchao has quit IRC17:00
*** markvoelker has quit IRC17:01
nibalizerhuh17:06
nibalizerso this is different than puppet apply --tags =things17:06
nibalizer?17:06
openstackgerritMerged stackforge/puppet-manila: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17387217:07
openstackgerritMerged stackforge/puppet-sahara: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17389117:09
*** IBerezovskiy has quit IRC17:11
openstackgerritMerged stackforge/puppet-ironic: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17349517:11
*** dgurtner has quit IRC17:11
openstackgerritMerged stackforge/puppet-ceilometer: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17348117:11
openstackgerritMerged stackforge/puppet-heat: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17350617:13
openstackgerritMerged stackforge/puppet-heat: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17350617:13
openstackgerritSebastien Badia proposed stackforge/puppet-modulesync-configs: Add a readme for this repository  https://review.openstack.org/18981917:14
openstackgerritMerged stackforge/puppet-ceilometer: Unpin beaker-rspec  https://review.openstack.org/18942117:18
openstackgerritMerged stackforge/puppet-cinder: Move rabbit/kombu settings to oslo_messaging_rabbit section  https://review.openstack.org/17349117:18
rhefnerEmilienM: just a reminder, can you take a look at https://review.openstack.org/#/c/189771/ when you get a chance?17:19
*** ianw has quit IRC17:21
*** ianw has joined #puppet-openstack17:21
openstackgerritSebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/18466817:41
*** jpena has quit IRC17:44
crinklehmm we decided to try out option #2 for the abandonment policy but didn't decide on an N for how long to leave it17:48
crinkleanyone want to give a quick +1/-1 for 3 months?17:48
*** xarses has joined #puppet-openstack17:57
barthalion+1 for month18:00
*** xingchao has joined #puppet-openstack18:00
*** xingchao has quit IRC18:05
*** morazi has quit IRC18:07
EmilienMcrinkle: 3 months is good imho18:08
*** morazi has joined #puppet-openstack18:17
EmilienMrhefner: looking now18:19
*** xarses has quit IRC18:21
EmilienMrhefner: I dropped a comment, not worth for -1 but still can you look18:24
openstackgerritRyan Hefner proposed stackforge/puppet-manila: Add option to push service image to Glance  https://review.openstack.org/18977118:27
rhefnerEmilienM: ^18:27
EmilienMfast!18:27
*** rushiagr is now known as rushiagr_away18:28
rhefnerEmilienM: thanks18:28
*** iurygregory has quit IRC18:37
*** iurygregory has joined #puppet-openstack18:37
*** dprince has quit IRC18:40
*** markvoelker has joined #puppet-openstack18:45
*** openstackgerrit has quit IRC18:48
*** openstackgerrit has joined #puppet-openstack18:49
*** markvoelker has quit IRC18:49
*** saju_m has joined #puppet-openstack18:53
*** ianw has quit IRC19:00
*** ianw has joined #puppet-openstack19:01
openstackgerritMerged stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/18466819:02
*** arnaud_orange has quit IRC19:12
openstackgerritCody Herriges proposed stackforge/puppet-ceilometer: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986219:20
openstackgerritCody Herriges proposed stackforge/puppet-cinder: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986319:20
openstackgerritCody Herriges proposed stackforge/puppet-designate: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986419:20
openstackgerritCody Herriges proposed stackforge/puppet-glance: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986519:20
openstackgerritCody Herriges proposed stackforge/puppet-gnocchi: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986619:20
openstackgerritCody Herriges proposed stackforge/puppet-heat: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986819:20
openstackgerritCody Herriges proposed stackforge/puppet-horizon: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18986919:20
openstackgerritCody Herriges proposed stackforge/puppet-ironic: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987019:20
openstackgerritCody Herriges proposed stackforge/puppet-keystone: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987119:20
openstackgerritCody Herriges proposed stackforge/puppet-manila: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987219:20
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987319:20
_ody...well that seems to be working19:20
openstackgerritCody Herriges proposed stackforge/puppet-nova: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987419:20
openstackgerritCody Herriges proposed stackforge/puppet-openstacklib: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987519:20
openstackgerritCody Herriges proposed stackforge/puppet-openstack_extras: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987619:21
openstackgerritCody Herriges proposed stackforge/puppet-sahara: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987719:21
openstackgerritCody Herriges proposed stackforge/puppet-swift: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987819:21
openstackgerritCody Herriges proposed stackforge/puppet-tempest: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987919:21
openstackgerritCody Herriges proposed stackforge/puppet-tripleo: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18988019:21
openstackgerritCody Herriges proposed stackforge/puppet-trove: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18988119:21
openstackgerritCody Herriges proposed stackforge/puppet-tuskar: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18988219:21
openstackgerritCody Herriges proposed stackforge/puppet-vswitch: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18988319:21
EmilienM_ody: you're not kidding when you submit a patch!19:21
*** dprince has joined #puppet-openstack19:21
crinkle:D19:22
_odyugh...except I did it wrong.19:22
EmilienMlol19:22
_odyWell they are in for review...I'll just get them fixed up from there.19:23
_ody...oh and email the ML.19:23
openstackgerritCody Herriges proposed stackforge/puppet-modulesync-configs: Move to rspec-puppet 2.2.0  https://review.openstack.org/18988619:27
*** jtomasek has joined #puppet-openstack19:41
*** saju_m has quit IRC19:41
EmilienMHunner: ping19:42
openstackgerritDan Prince proposed stackforge/puppet-keystone: Revert "remove POSIX users, groups and file modes"  https://review.openstack.org/18989219:55
dprinceEmilienM: ^^^ seeing all the puppet jobs fail now19:56
EmilienMoh dear19:56
mgagnelooks like we are not properly testing dependencies19:57
EmilienMclayton, mfisch ^19:57
mgagnecatalog compiles fine (in rspec tests) with missing deps...19:57
EmilienMand the review was a bit fast imho19:57
EmilienMit still had some User resources.19:57
EmilienMI'm doing a patch now19:57
mgagne"some" ?19:58
EmilienMmgagne: ?19:58
mgagneyou mean references to now removed user resource?19:58
mfischrevert looks good20:00
EmilienMthe question is how beaker could have worked20:00
EmilienMmfisch: no.20:00
EmilienMmfisch: I'm fixing it now20:00
EmilienMI'm done almost20:00
mfischwhats wrong with it?20:00
EmilienMI don't want to revert something we can fix in 1 minute20:00
dprinceEmilienM: I don't mind rolling forward... but would point out a revert is always safe20:01
openstackgerritEmilien Macchi proposed stackforge/puppet-keystone: Drop User resource dependency at all  https://review.openstack.org/18990820:01
EmilienMplease review ^20:01
dprinceEmilienM: if it doesn't fix it I'll propose 2 ;)20:01
* dprince doesn't mix reverts20:01
EmilienMmfisch, clayton: how come you merge patches that do not pass beaker?20:01
EmilienMmgagne: our CI detected it but jobs are non voting.20:02
EmilienMhttp://logs.openstack.org/63/185663/3/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/2424588/console.html20:02
EmilienMmgagne, mgagne, clayton ^20:02
EmilienMit was quite explicit20:02
claytonwhy can I merge reviews that don't pass beaker?20:02
EmilienMbecause we decided for now there are non voting20:03
EmilienMand I already asked reviewers to have a look during reviews20:03
EmilienMit's non voting now because we did not want to slow down CI process while we were working on these jobs20:03
EmilienMI proposed https://review.openstack.org/#/c/176343/ to make it vote20:03
mgagneEmilienM: I think we should consider non-voting jobs as if no one look at them20:03
EmilienMmgagne: well, I wish we could look at it20:04
mgagneEmilienM: sure20:04
claytonI will in the future20:04
EmilienMif it was only me, they would vote.20:04
mgagneEmilienM: so we don't have unrealistic expectations20:04
EmilienMbut if I do that20:04
EmilienMpeople will complain a lot20:04
EmilienMmgagne, clayton, crinkle, mfisch: do you agree to make them vote *now* ?20:05
EmilienMlooking at https://docs.google.com/spreadsheets/d/1i2z5QyvukHCWU_JjkWrTpn-PexPBnt3bWPlQfiDGsj8/edit#gid=020:06
EmilienMthey are all green20:06
claytonI'm willing to trust your judgement on that.  I'm also willing to make sure I look at them in the future.20:06
EmilienMclayton: don't worry we all make mistakes, me first.20:06
EmilienMhopefully our human-CI (called dprince) is here :P20:06
*** jayg is now known as jayg|g0n320:08
*** kkirkpatrick has quit IRC20:21
*** kkirkpatrick has joined #puppet-openstack20:21
*** rhefner has quit IRC20:21
*** simonmcc has quit IRC20:21
_odysbadia: Has msync ever been ran before across all the modules?20:21
*** tchaypo has quit IRC20:22
*** rhefner has joined #puppet-openstack20:23
*** tchaypo has joined #puppet-openstack20:23
*** simonmcc has joined #puppet-openstack20:24
openstackgerritCody Herriges proposed stackforge/puppet-modulesync-configs: Manage default configs  https://review.openstack.org/17795920:32
*** markvoelker has joined #puppet-openstack20:34
*** markvoelker has quit IRC20:39
EmilienMcrinkle: in beaker, why don't we see all puppet logs?20:40
EmilienMI remember we could see them before20:40
EmilienMkeystone still fails and I don't see why, see https://review.openstack.org/17795920:40
EmilienMhttps://jenkins04.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/38/console20:40
EmilienMnibalizer: maybe I've missed a change while I was away?20:43
nibalizerEmilienM: ?20:44
nibalizerEmilienM: are you pinning concat?20:44
EmilienMnibalizer: in beaker jobs I don't see puppet logs as before20:44
nibalizerbecause if you use concat 2.x right now, you're gonna have a bad time20:44
nibalizerEmilienM: i think crinkle moved us from beaker 2.x to 5.x20:45
dprinceEmilienM: I'm going to cherry pick my revert into TripleO CI for now....20:45
EmilienMdprince: my fix does not work. I'm merging your patch20:45
dprinceEmilienM: if you look at the revert it does fix breaker too20:45
EmilienMoh, looking20:45
openstackgerritCody Herriges proposed stackforge/puppet-modulesync-configs: Move to rspec-puppet 2.2.0  https://review.openstack.org/18988620:45
EmilienMdprince: oh yes, sure20:46
EmilienMdprince: because you revert what breaks beaker :)20:46
EmilienMmgagne, mgagne, clayton or crinkle: if you around, can you +2 https://review.openstack.org/#/c/189892 ?20:47
claytonlooking20:47
EmilienMthanks20:47
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18993020:48
crinkleEmilienM: nibalizer oh yeah beaker 5 doesn't output logs by default :(20:48
crinklei can turn that on20:48
EmilienMcrinkle: that would be awesome20:48
EmilienMcrinkle: 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 management20:49
EmilienMcrinkle: I can do 2nd action20:49
dprinceEmilienM: thanks, carry on as normal...20:50
EmilienMdprince: the revert is merged very soon.20:50
*** stevemar has joined #puppet-openstack20:51
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987320:51
*** iurygregory has quit IRC20:54
*** rcallawa has quit IRC20:55
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/18464620:56
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/18464620:57
openstackgerritMerged stackforge/puppet-keystone: Revert "remove POSIX users, groups and file modes"  https://review.openstack.org/18989220:57
crinkleEmilienM: https://github.com/puppetlabs/beaker/wiki/How-to-Write-a-Beaker-Test-for-a-Module#supported-env-variables20:57
crinkleer20:57
crinklehttps://review.openstack.org/18993720:57
crinkleis what i meant to paste20:57
crinkleEmilienM: ^20:57
EmilienMcrinkle: thanks20:57
crinkleHunner: nibalizer could you review ^20:58
nibalizerlgtm20:59
*** dprince has quit IRC21:00
mfischare part of the gerrit project rename outage?  http://lists.openstack.org/pipermail/openstack-infra/2015-June/002815.html21:06
crinkleif we get https://review.openstack.org/#/c/176326/ approved in time we can ask them to add us in21:07
*** ducttape_ has quit IRC21:12
EmilienMcrinkle: a Friday?21:13
EmilienMwell, before the week end21:13
*** jtomasek has quit IRC21:13
crinkleEmilienM: they need to take down the system so they have to do it on a low traffic day21:13
EmilienMcrinkle: ok done, I've updated https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting#Upcoming_Project_Renames21:26
EmilienMcrinkle: now I'm sending an email to our community21:26
*** rcallawa has joined #puppet-openstack21:28
*** rcallawa has quit IRC21:28
*** rcallawa has joined #puppet-openstack21:29
*** mdorman has quit IRC21:45
*** mdorman has joined #puppet-openstack21:58
*** morazi has quit IRC22:07
cwolferhcrinkle, i've added tests (and comment) for https://review.openstack.org/#/c/188200/ -- please take a look at your convenience22:20
*** wabu_ has quit IRC22:24
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Update all Puppet OpenStack Modules to rspec-puppet 2.2.0  https://review.openstack.org/18987322:26
openstackgerritCody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/18464622:27
crinklecwolferh: so the package name is puppet-rdb on fedora?22:34
*** stevemar has quit IRC22:35
*** xingchao has joined #puppet-openstack22:37
cwolferhcrinkle, not sure22:38
cwolferhpython-rbd is the future though so if it isn't there yet, it will be22:38
crinklecwolferh: okay22:38
cwolferhcrinkle, thanks.  also for the record just verified it is in fedora22:40
crinklecwolferh: okay cool22:41
*** wabu has joined #puppet-openstack22:52
*** xingchao has quit IRC22:53
*** kkirkpatrick has quit IRC23:01
*** kev has joined #puppet-openstack23:01
*** kev is now known as Guest9500223:01
*** gildub has joined #puppet-openstack23:06
*** Guest95002 has quit IRC23:16
*** markvoelker has joined #puppet-openstack23:21
*** markvoelker has quit IRC23:26
*** xingchao has joined #puppet-openstack23:27
*** stevemar has joined #puppet-openstack23:33
*** mdorman has quit IRC23:35
*** gildub has quit IRC23:47

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