Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-10-04

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

All times shown according to UTC.

Time Nick Message
00:13 ducttape_ joined #puppet-openstack
00:25 johnmilton joined #puppet-openstack
00:29 markvoelker joined #puppet-openstack
00:46 xarses joined #puppet-openstack
00:52 ducttape_ joined #puppet-openstack
00:54 markvoelker joined #puppet-openstack
01:52 ducttape_ joined #puppet-openstack
01:56 openstackgerrit Merged openstack/puppet-openstacklib: Add region support to credentials  https://review.openstack.org/381288
02:04 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder backup to scenario002  https://review.openstack.org/381083
02:14 ducttape_ joined #puppet-openstack
02:18 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Clean out UI httpd configuration file  https://review.openstack.org/380152
02:47 EmilienM mwhahaha: I found a logstash query we might love
02:48 EmilienM build_name: *puppet* AND message: "is deprecated"
02:51 EmilienM build_name: *puppet* AND message: "deprecated" AND message: "Warning" AND message: "scope"
02:51 EmilienM even better
03:21 ducttape_ joined #puppet-openstack
03:57 openstackgerrit Sharat Sharma proposed openstack/puppet-aodh: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381385
03:58 openstackgerrit Sharat Sharma proposed openstack/puppet-barbican: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381386
03:59 openstackgerrit Sharat Sharma proposed openstack/puppet-ceilometer: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381387
03:59 dmsimard EmilienM: nice
03:59 openstackgerrit Sharat Sharma proposed openstack/puppet-cinder: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381388
04:00 openstackgerrit Sharat Sharma proposed openstack/puppet-ceph: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381390
04:03 openstackgerrit Sharat Sharma proposed openstack/puppet-ec2api: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381391
04:38 ducttape_ joined #puppet-openstack
04:56 openstackgerrit Sharat Sharma proposed openstack/puppet-glance: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381395
04:59 openstackgerrit Sharat Sharma proposed openstack/puppet-gnocchi: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381396
05:00 openstackgerrit Sharat Sharma proposed openstack/puppet-heat: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381397
05:02 openstackgerrit Sharat Sharma proposed openstack/puppet-horizon: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381398
05:04 openstackgerrit Sharat Sharma proposed openstack/puppet-ironic: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381399
05:10 openstackgerrit John Fulton proposed openstack/puppet-ceph: Split installed packages per role  https://review.openstack.org/381228
05:22 openstackgerrit Sharat Sharma proposed openstack/puppet-keystone: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381402
05:24 openstackgerrit Sharat Sharma proposed openstack/puppet-magnum: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381404
05:26 openstackgerrit Sharat Sharma proposed openstack/puppet-manila: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381405
05:27 openstackgerrit Sharat Sharma proposed openstack/puppet-mistral: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381406
05:31 openstackgerrit Sharat Sharma proposed openstack/puppet-murano: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381408
05:39 ducttape_ joined #puppet-openstack
05:46 jaosorior joined #puppet-openstack
05:48 openstackgerrit Merged openstack/puppet-tripleo: Clean out UI httpd configuration file  https://review.openstack.org/380152
05:49 openstackgerrit Sharat Sharma proposed openstack/puppet-neutron: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381413
05:51 openstackgerrit Sharat Sharma proposed openstack/puppet-nova: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381414
05:54 openstackgerrit Sharat Sharma proposed openstack/puppet-openstacklib: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381415
05:56 openstackgerrit Sharat Sharma proposed openstack/puppet-ovn: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381418
05:58 openstackgerrit Sharat Sharma proposed openstack/puppet-sahara: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381420
06:00 openstackgerrit Sharat Sharma proposed openstack/puppet-swift: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381421
06:03 openstackgerrit Sharat Sharma proposed openstack/puppet-tempest: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381423
06:09 openstackgerrit Sharat Sharma proposed openstack/puppet-trove: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381427
06:12 openstackgerrit Sharat Sharma proposed openstack/puppet-vswitch: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381428
06:15 openstackgerrit Sharat Sharma proposed openstack/puppet-zaqar: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381430
06:40 ducttape_ joined #puppet-openstack
06:43 rcernin joined #puppet-openstack
06:47 ccamacho joined #puppet-openstack
06:48 chem joined #puppet-openstack
06:52 tesseract- joined #puppet-openstack
07:01 dgurtner joined #puppet-openstack
07:35 hjensas joined #puppet-openstack
07:35 hjensas joined #puppet-openstack
07:41 ducttape_ joined #puppet-openstack
07:44 jpich joined #puppet-openstack
07:57 jistr joined #puppet-openstack
08:01 imcsk8 joined #puppet-openstack
08:02 dgurtner joined #puppet-openstack
08:02 dgurtner joined #puppet-openstack
08:03 rwsu joined #puppet-openstack
08:03 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/381492
08:24 markvoelker joined #puppet-openstack
08:33 danpawlik joined #puppet-openstack
08:47 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: normalize_ip_for_uri: allow to give an array of IP addresses  https://review.openstack.org/381545
09:03 openstackgerrit Julie Pichon proposed openstack/puppet-tripleo: Clean out UI httpd configuration file  https://review.openstack.org/381568
09:04 openstackgerrit Julie Pichon proposed openstack/puppet-tripleo: Use FallbackResource instead of Rewrite for UI  https://review.openstack.org/381276
09:04 derekh joined #puppet-openstack
09:04 paramite joined #puppet-openstack
09:11 gfidente joined #puppet-openstack
09:12 openstackgerrit Merged openstack/puppet-tripleo: Fix the timeout for pacemaker systemd resources  https://review.openstack.org/380665
09:30 openstackgerrit Sharat Sharma proposed openstack/puppet-openstack_extras: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381581
09:42 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Fix the timeout for pacemaker systemd resources  https://review.openstack.org/381584
09:47 shardy joined #puppet-openstack
09:47 openstackgerrit Fabian Zimmermann proposed openstack/puppet-cinder: fixed typo in ./manifests/backend/{netapp,iscsi,bdd}.pp  https://review.openstack.org/381585
09:48 openstackgerrit Fabian Zimmermann proposed openstack/puppet-cinder: fixed typo in ./manifests/backend/{netapp,iscsi,bdd}.pp  https://review.openstack.org/381585
09:49 openstackgerrit Fabian Zimmermann proposed openstack/puppet-cinder: fixed typo in ./manifests/backend/{netapp,iscsi,bdd}.pp  https://review.openstack.org/381585
09:53 gfidente joined #puppet-openstack
10:01 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role  https://review.openstack.org/381228
10:04 zigo joined #puppet-openstack
10:10 richm joined #puppet-openstack
10:16 zigo_ joined #puppet-openstack
10:26 markvoelker joined #puppet-openstack
10:31 ntpttr joined #puppet-openstack
10:31 openstackgerrit Martin Mágr proposed openstack/puppet-tripleo: Deploy monitoring/logging agents sooner  https://review.openstack.org/381604
10:45 ducttape_ joined #puppet-openstack
10:45 openstackgerrit Peter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management  https://review.openstack.org/380026
10:45 rain joined #puppet-openstack
10:56 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role  https://review.openstack.org/381228
10:58 rcernin joined #puppet-openstack
11:00 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role  https://review.openstack.org/381228
11:01 dprince joined #puppet-openstack
11:07 ccamacho guys mind to approve this patch? https://review.openstack.org/#/c/376254/ already +2ed and passing CI
11:20 dgurtner joined #puppet-openstack
11:20 dgurtner joined #puppet-openstack
11:23 openstackgerrit Peter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management  https://review.openstack.org/380026
11:25 openstackgerrit Peter Zhurba proposed openstack/puppet-glare: Add hooks for external install & svc management  https://review.openstack.org/380026
11:27 markvoelker joined #puppet-openstack
11:27 openstackgerrit Peter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/373805
11:38 EmilienM hi
11:38 dtantsur hi, has anyone seen: require': cannot load such file -- puppet-openstack_spec_helper/facts (LoadError)
11:38 dtantsur I can't run local tests due to that any more...
11:39 EmilienM you need to remove the Gemfile.lock
11:39 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
11:39 EmilienM and run bundle install again
11:39 dtantsur trying now, thanks
11:43 dtantsur btw, bundle install *always* fails for me in the end with "Could not find json-1.8.3.gem for installation". usually tests work after that anyway..
11:45 ducttape_ joined #puppet-openstack
11:46 openstackgerrit Sharat Sharma proposed openstack/puppet-trove: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381427
11:47 mkarpin Hello, i am locally have an error when running bundle exec rake spec on puppet-nova
11:47 mkarpin Gem::InstallError: docker-api requires Ruby version >= 2.0.0.
11:47 mkarpin i mean when running bundle install
11:48 mkarpin did anyone see something like this?
11:49 karthiks joined #puppet-openstack
11:50 dtantsur EmilienM, worked. though now instead of resetting HEAD to origin/master, it reverted my change locally, and tested it >_<
11:53 mkarpin nevermind) fixed)
11:54 johnmilton joined #puppet-openstack
11:55 johnmilton joined #puppet-openstack
11:57 EmilienM cool!
11:57 EmilienM dtantsur: sorry for that, I'm confused why it did that
11:58 dtantsur dunno, my environment is somehow special, so every after failed test run, the next run it does not test my change...
12:02 EmilienM dtantsur: I have a tips for you
12:02 EmilienM a sec
12:02 EmilienM dtantsur: https://paste.fedoraproject.org/443105/55825401/
12:02 EmilienM put this scripts into your ~/bin
12:02 EmilienM and run pt.sh (puppet tests) in the root of the module
12:02 EmilienM it will install gems and stuffs in /tmp
12:03 EmilienM all credits to degorenko
12:03 openstackgerrit Merged openstack/puppet-openstack_spec_helper: Remove add_custom_fact for puppet version  https://review.openstack.org/381328
12:03 dtantsur interesting, thanks!
12:04 degorenko credits? :D
12:05 degorenko ah, my script for running tests
12:05 EmilienM degorenko: yeah I modified it a bit
12:05 EmilienM degorenko: to use a dynamic directory
12:05 degorenko EmilienM, yeah, i improved it too, but much more :D
12:05 EmilienM dtantsur: the downside of it is that it takes longer if you want to re-run the tests
12:06 EmilienM I could improve it
12:06 EmilienM like adding a --continue option
12:06 EmilienM to skip the /tmp thing
12:06 EmilienM degorenko: dude! show me :P
12:06 fultonj joined #puppet-openstack
12:06 degorenko EmilienM, i've added ability to run only for one file, or for some stage - clean, prepare, standalone, give me a sec
12:07 degorenko also ability to run only units or classes or functions :)
12:08 EmilienM we have an expert here
12:09 degorenko EmilienM, enjoy http://paste.openstack.org/show/584178/
12:09 degorenko dtantsur, you too ^ please :)
12:09 dtantsur wow
12:09 ducttape_ joined #puppet-openstack
12:09 EmilienM ok cool
12:09 EmilienM I would add a feature, to run it in /tmp
12:10 degorenko EmilienM, -G /tmp ?
12:10 degorenko no? :D
12:10 dtantsur I wonder if we need to put it to our contributor docs..
12:10 EmilienM ah yeah
12:10 EmilienM but         echo "Directory ${2} does not exist."
12:10 EmilienM I would manage it
12:10 EmilienM like generating a uuid
12:11 EmilienM and create in /tmp/$uuid"
12:11 EmilienM let's create a new project !
12:11 dtantsur why not $(mktemp -d) ?
12:11 EmilienM yes !
12:11 degorenko i did it especially for me, because i forgot to create dir sometime :)
12:11 degorenko for me it's critical to see :D
12:11 degorenko you can improve it as you want :)
12:12 degorenko i'm not greedy
12:12 dtantsur free software ftw
12:12 degorenko we are open source software so lol
12:15 furlongm joined #puppet-openstack
12:19 karthiks joined #puppet-openstack
12:21 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Change section for setting package_service backend  https://review.openstack.org/381749
12:23 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder backup to scenario002  https://review.openstack.org/381083
12:32 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Change section for setting package_service backend  https://review.openstack.org/381749
12:33 cdelatte joined #puppet-openstack
12:33 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Change section for setting package_service backend  https://review.openstack.org/381749
12:33 markvoelker joined #puppet-openstack
12:46 iurygregory good morning
12:47 openstackgerrit Peter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip  https://review.openstack.org/373805
12:48 openstackgerrit Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/381492
12:50 ducttape_ joined #puppet-openstack
13:01 moravec joined #puppet-openstack
13:07 openstackgerrit Mykyta Karpin proposed openstack/puppet-nova: Add class for migration options configuration in qemu.conf  https://review.openstack.org/378754
13:15 jperry joined #puppet-openstack
13:26 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Make MySQL client default host be same as bind address  https://review.openstack.org/381834
13:35 ducttape_ joined #puppet-openstack
13:35 jaosorior joined #puppet-openstack
13:38 fultonj joined #puppet-openstack
13:47 mnaser I'm running into this quite annoying issue where keystone::service::end seems to be triggered by almost anything, has anyone else noticed it?
13:47 mnaser ex: installing neutron vpnaas agent.. Info: /Stage[main]/Neutron::Agents​::Vpnaas/Package[openswan]: Scheduling refresh of Anchor[keystone::service::end]
13:48 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Split installed packages per role  https://review.openstack.org/381228
13:48 mnaser https://github.com/openstack/puppet-keyston​e/blob/stable/mitaka/manifests/deps.pp#L24 -- i can only assume that somehow something is notifying that service?
13:49 mnaser the annoying issue is that it results in an httpd restart, which with rdo, takes a while because if has compresses the horizon assets on every stop/start/restart
13:49 dmsimard EmilienM: did we not have stackviz in puppet logs ?
13:50 EmilienM we do have
13:50 mwhahaha mnaser: https://github.com/openstack/puppet-keystone/​blob/stable/mitaka/manifests/deps.pp#L52-L53
13:50 dmsimard EmilienM: don't see them in http://logs.openstack.org/75/380875/2/c​heck/gate-puppet-openstack-integration-​4-scenario002-tempest-centos-7/2d5e67b/
13:50 mnaser oh, i did not catch that mwhahaha
13:50 mwhahaha probably that, we probably need to change the package notifications to not just be tag == 'openstack' cause that can be a whole lot of people
13:50 mwhahaha s/people/packages
13:51 dmsimard mnaser: yeah this is actually something I was sort of worried about moving everything under wsgi is that we blindly restart apache everytime which means we're restarting a lot of services
13:51 EmilienM dmsimard: I'll look in a few
13:51 mnaser doesnt sound like its' an easy fix :<
13:51 dmsimard I /think/ there is a way to restart individually wsgi services but I don't know.
13:51 mnaser there is, with mod_wsgi you just touch a file somewhere
13:51 dmsimard so puppet should be doing that, not restart httpd
13:52 mnaser https://code.google.com/archive/p/mo​dwsgi/wikis/ReloadingSourceCode.wiki
13:52 mwhahaha yea that's getting too complicated now
13:52 mwhahaha lets not
13:53 dmsimard mwhahaha: going to need to be addressed
13:53 dmsimard it doesn't make sense to restart everything when just updating nova config
13:53 dmsimard oh let's restart keystone, nova, ceilometer and what have you
13:53 mnaser doesnt seem straightforward
13:53 dmsimard not saying it is :)
13:53 mwhahaha that's what happens when you put them all under apache on the same node
13:54 mwhahaha but by fronting it with haproxy reduces the impact
13:54 mnaser well we have 3 controllers and one of them is gone out of service during the time of a puppet run (with the smallest changes ever)
13:54 mnaser some might argue that's a good thing but i dont know
13:55 mwhahaha so this is treading into opinionated land
13:55 mwhahaha so i'd rather just document this and let the installers figure out that restart methodology they want
13:56 moravec joined #puppet-openstack
13:57 dfisher joined #puppet-openstack
14:02 openstackstatus NOTICE: The Gerrit service on review.openstack.org is being restarted to address performance degradation and should return momentarily
14:07 sharatss_ joined #puppet-openstack
14:09 ducttape_ joined #puppet-openstack
14:12 ducttape_ joined #puppet-openstack
14:14 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Cleanup the firewall logic.  https://review.openstack.org/381875
14:19 iurygregory Hey people we will have our weekly meeting in 40minutes, if you would like to discuss something please update the agenda https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20161004
14:25 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Make MySQL client default host be same as bind address  https://review.openstack.org/381834
14:42 mwhahaha I might be a minute or two late for the meeting.
14:44 iurygregory well the agenda is not to big we can wait =D
14:46 ayoung joined #puppet-openstack
14:49 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Release 9.4.1  https://review.openstack.org/381901
14:50 iurygregory EmilienM, 9.4.1 should not be in stable/newton?
14:50 EmilienM I'll backport it
14:50 iurygregory ack
14:51 iurygregory +2 already
14:51 EmilienM thx
14:59 openstackgerrit Sven Anderson proposed openstack/puppet-tempest: WIP Test - DO NOT MERGE  https://review.openstack.org/374537
15:00 iurygregory mwhahaha, https://review.openstack.org/#/c/380531/ is green
15:03 openstackgerrit Merged openstack/puppet-nova: Fix region support for nova providers  https://review.openstack.org/381295
15:14 moravec joined #puppet-openstack
15:16 openstackgerrit Sven Anderson proposed openstack/puppet-tempest: WIP Test - DO NOT MERGE  https://review.openstack.org/374537
15:17 paramite joined #puppet-openstack
15:19 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: Release 9.4.1  https://review.openstack.org/381916
15:20 dgurtner joined #puppet-openstack
15:26 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Meeting #92  https://review.openstack.org/381923
15:28 xarses joined #puppet-openstack
15:33 ansiwen chem: hi
15:33 ansiwen chem: I'm back from holidays, and I immediately want you to look at something. :-)
15:33 ansiwen chem: http://logs.openstack.org/37/374537/3/check/ga​te-puppet-tempest-puppet-unit-4.5-centos-7/36b​9196/console.html#_2016-10-04_15_25_28_458440
15:34 ansiwen EmilienM: or if you have a second, you also probably immediately know what's wrong here ^^^
15:36 mwhahaha ansiwen: it's ruby so it's true not True
15:37 mwhahaha ansiwen: commented on the review
15:38 ansiwen mwhahaha: thanks! but I don't know why it only fails on the latest versions then.
15:38 mwhahaha it failed on all of them
15:39 openstackgerrit Mykyta Karpin proposed openstack/puppet-nova: Add class for migration options configuration in qemu.conf  https://review.openstack.org/378754
15:39 ansiwen mwhahaha: http://logs.openstack.org/37/374537/3​/check/gate-puppet-tempest-puppet-uni​t-3.8-centos-7/b837ed2/console.html
15:39 ansiwen mwhahaha: here not
15:40 ansiwen mwhahaha: by the way, I copy pasted that from here: https://github.com/openstack/pupp​et-keystone/blob/master/README.md
15:40 mwhahaha oh i was going on what was reported back, they might have been less senstive to case in older versions
15:40 mwhahaha we'll have to fix that
15:41 openstackgerrit Sven Anderson proposed openstack/puppet-tempest: WIP Test - DO NOT MERGE  https://review.openstack.org/374537
15:42 ansiwen mwhahaha: thanks a lot
15:42 mwhahaha wonder if it needed to be wrapped in quotes or something
15:43 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Fix boolean typo in documentation  https://review.openstack.org/381938
15:49 EmilienM mwhahaha: you ok with https://review.openstack.org/#/c/381901/ ?
15:50 mwhahaha yea
15:54 EmilienM thx
15:56 ansiwen mwhahaha: looks good so far
15:59 moravec joined #puppet-openstack
16:03 morazi joined #puppet-openstack
16:05 dgurtner joined #puppet-openstack
16:12 EmilienM Vinsh, degorenko: see my question on swift https://review.openstack.org/#/c/378950
16:12 EmilienM I'm concerned about upgrades
16:15 karthiks joined #puppet-openstack
16:15 degorenko EmilienM, not sure. But we can reflect and mark this change in release note - set purge to true in case first time upgrade from old version to new
16:15 mwhahaha _ody: is $require a variable like $name or $title?
16:16 Vinsh It behaves like it is atleast mwhahaha
16:16 mwhahaha you sure it wasn't undef? :D
16:16 _ody mwhahaha: Hmm.  Never personaly validated its usage.
16:17 openstackgerrit Merged openstack/puppet-ec2api: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381391
16:17 EmilienM mwhahaha: wdyt about the puppet-swift change? should we purge config by default to make it clean from now?
16:18 mwhahaha we don't purge in other modules do we?
16:18 Vinsh EmilienM: The upgrade should produce the same file.. if we set purge by default it shouldn't change anything, unless a user has un-set a class/filter.
16:18 Vinsh we purge in many other modules
16:18 Vinsh neutron for example
16:18 mwhahaha do we purge by default tho
16:18 Vinsh set to false by default there
16:19 degorenko +1, if we will set to true - it will broke idempotency
16:19 EmilienM I'm fine approving it, just making sure we don't break upgrades
16:19 openstackgerrit Merged openstack/puppet-magnum: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381404
16:19 mwhahaha i'd be hesitant to set purge by default, i'd rather if we choose to do that warn people for a cycle
16:20 Vinsh mwhahaha: https://docs.puppet.com/puppet/latest​/reference/metaparameter.html#require
16:20 openstackgerrit Merged openstack/puppet-manila: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381405
16:20 Vinsh mwhahaha: So $require is a metaparamater that any resource has really
16:21 _ody Oooh that's what you mean by $require.
16:21 mwhahaha right but that doesn't say anything about require -> $require
16:21 degorenko erm, not sure that this spec is saying about $require as is - it tells about require - as parameter i think
16:21 _ody Yeah. I've never see $require used and can't find it reference as a variable anywhere.
16:21 mwhahaha or do all meta parameters automagically get a variable named after them
16:22 degorenko :D
16:22 mwhahaha i'm pretty sure  they don't :D which is why I asked for a test around that
16:22 * _ody goes source spelunking
16:22 _ody Yeah.  Seem unlikely to me.
16:22 degorenko Vinsh, can you provide working example? :)
16:22 Vinsh https://review.openstack.org/#/c/378​950/10/spec/classes/swift_proxy_spec.rb
16:23 Vinsh line 81
16:23 mwhahaha glorious undocumented functionality
16:23 mwhahaha perhaps we shouldn't use it :D
16:24 Vinsh its like subscribe.. or any of those
16:24 mwhahaha but i've never used $subscribe
16:24 mwhahaha or any of those :D
16:24 mwhahaha or ever seen it referenced like that
16:24 degorenko Vinsh, erm, 81 line is for require parameter and for proxy
16:24 openstackgerrit Merged openstack/puppet-ovn: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381418
16:24 degorenko not for service file
16:25 mwhahaha it's contain_service
16:25 degorenko ah right
16:25 degorenko magical construction
16:25 mwhahaha so i guess it works, but i totally see that breaking someday due to a puppet regression
16:26 degorenko but if it works, we can use it
16:26 Vinsh I consider it a basic tenant/principle of puppet
16:26 Vinsh Didn't realize it wasn't widely known that way
16:26 degorenko yeah, tests for puppet 3.x and 4.x passed
16:26 degorenko really :D i see it first time
16:27 Vinsh I bet if I were to add "$require" as a paramater of swift_service.. it would barf
16:27 openstackgerrit joined #puppet-openstack
16:27 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Change section for setting package_service backend  https://review.openstack.org/381971
16:28 degorenko no :D no need
16:28 degorenko now, i think, when we know about this "feature" we good
16:28 degorenko mwhahaha, thoughts ?
16:28 mwhahaha yea let me rereview it
16:28 degorenko ok, once you will +2 i'll +A :)
16:29 _ody Eventhough I've never seen it, it it logically after thinking it through that all parameters, meta parametes like require become variables available to the local scope because they aren't special beyond being implicitly added to every resource.
16:29 openstackgerrit joined #puppet-openstack
16:29 degorenko Vinsh, but really, good catch with this case. nice knowledge
16:29 Vinsh Then I will begin on the work to remove templates and use ini in the object/container/account portion.. and after that the module will be template free.
16:30 Vinsh degorenko: Thanks :)  The spec tests really helped me learn on this.
16:30 degorenko Vinsh, spec tests helped many of us to learn puppet :D
16:30 openstackgerrit joined #puppet-openstack
16:31 mwhahaha or learn to hate it
16:31 Vinsh Necessary evil
16:31 degorenko +1
16:31 degorenko especially providers :D
16:31 Vinsh providers sort of require a wrapper class to spec even
16:31 Vinsh otherwise It's beyond me
16:32 Vinsh I want to work on landing this next after the object template change is odne
16:32 Vinsh *done
16:32 Vinsh https://review.openstack.org/#/c/327298/8/li​b/puppet/provider/swift_storage_policy/ruby.rb
16:32 degorenko oh
16:32 Vinsh I pretty much had to re-read all of "shit gary says" 10x to learn that
16:32 degorenko nightmare :)
16:32 Vinsh prefect... and all that jaz
16:33 Vinsh The basic ini and openstack ini provider isn't enough for the way storage policies operate, so had to extend it
16:34 Vinsh The puppet types and providers book is ok too.
16:36 Vinsh I appreciate the detailed and helpful review on that template change, the review made it much better.
16:43 degorenko it's in merge conflict
16:44 degorenko i'll review it tomorrow
16:44 Vinsh oh, the storage policy one? yes. I need to rebase/update. I'll set it to WIP till I do that.
16:45 jperry joined #puppet-openstack
16:47 degorenko Vinsh, you already have WIP in commit title :D
16:47 Vinsh Yeah, but it's official now.
16:50 mwhahaha uh oh http://logs.openstack.org/82/381582/1/check/gat​e-puppet-pacemaker-puppet-syntax-4-centos-7/c8e​712f/console.html#_2016-10-04_16_02_40_649768
16:51 mwhahaha Hunner: why you gotta set ruby dependencies :(
16:51 degorenko mwhahaha, would you +A https://review.openstack.org/#/c/381971/ ? :)
16:52 degorenko thanks
16:52 karthiks joined #puppet-openstack
16:58 openstackgerrit Merged openstack/puppet-openstack-guide: Meeting #92  https://review.openstack.org/381923
17:02 jperry joined #puppet-openstack
17:03 openstackgerrit Merged openstack/puppet-tripleo: Use FallbackResource instead of Rewrite for UI  https://review.openstack.org/381276
17:03 _ody Vinsh: I believe this validates the behavior you found witn $require...
17:03 _ody https://github.com/puppetlabs/puppet/blob/m​aster/lib/puppet/resource/type.rb#L347-L387
17:03 _ody https://github.com/puppetlabs/puppet/blo​b/master/lib/puppet/parser/scope.rb#L928
17:03 openstackgerrit Merged openstack/puppet-keystone: Fix boolean typo in documentation  https://review.openstack.org/381938
17:03 openstackgerrit Merged openstack/puppet-aodh: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381385
17:03 openstackgerrit Merged openstack/puppet-barbican: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381386
17:04 openstackgerrit Merged openstack/puppet-ceilometer: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381387
17:04 openstackgerrit Merged openstack/puppet-ceph: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381390
17:04 openstackgerrit Merged openstack/puppet-cinder: fixed typo in ./manifests/backend/{netapp,iscsi,bdd}.pp  https://review.openstack.org/381585
17:04 openstackgerrit Merged openstack/puppet-cinder: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381388
17:05 Vinsh _ody: ah yeah. Nice
17:05 Vinsh puppet guts
17:07 social joined #puppet-openstack
17:08 Vinsh I like that "with_guarded_scope"
17:09 openstackgerrit Merged openstack/puppet-murano: Change section for setting package_service backend  https://review.openstack.org/381971
17:16 openstackgerrit Merged openstack/puppet-glare: Add hooks for external install & svc management  https://review.openstack.org/380026
17:27 ayoung joined #puppet-openstack
17:44 openstackgerrit Maksym Yatsenko proposed openstack/puppet-glance: [WIP]  https://review.openstack.org/382016
17:58 rcernin joined #puppet-openstack
18:00 ccamacho joined #puppet-openstack
18:00 moravec joined #puppet-openstack
18:01 ayoung joined #puppet-openstack
18:13 ducttape_ joined #puppet-openstack
18:46 _ody Vinsh: https://github.com/puppetlabs/p​uppet-specifications/issues/88
18:47 Vinsh :D right on!
18:48 _ody s/Someone/Adam Vinsh/
18:49 _ody Hope you don't mind having you name attached to it.
18:49 Vinsh Not at all
18:50 ansiwen somebody saw that before?: "NoMethodError: undefined method `type' for nil:NilClass"
18:50 ansiwen http://logs.openstack.org/42/356442/13/check/ga​te-puppet-openstack-integration-puppet-lint/789​8dc2/console.html#_2016-10-04_17_37_29_242365
18:50 EmilienM dmsimard: mwhahaha: do we have some ongoing work to support nova placement API thing?
18:50 dmsimard no idea what that is so I'll say no
18:50 Vinsh _ody: The more I learn about puppet internals, its amazing how some simple blocks do so much.
18:51 Vinsh more so then in other projects/languages
18:51 Vinsh The whole recursive nature of it all.
18:51 _ody Puppet it super meta.
18:51 Vinsh It's so meta I'm not even going to describe it to you.
18:51 mwhahaha EmilienM: probably not
18:53 EmilienM ok
18:53 mwhahaha isnt' it built into the existing api?
18:53 _ody Doing this as long as I have has definitely changed me.  My default for figuring out OpenStack behaviors is also to follow the python bread crumbs.
18:53 mwhahaha EmilienM: or do you mean writing a provider to manage it
18:54 EmilienM mwhahaha: I thought it was a new service
18:54 mwhahaha well i found https://launchpad.net/ubuntu/yak​kety/+package/nova-placement-api but i have no idea :D
18:54 EmilienM it's a new endpoint iirc
18:54 mwhahaha so i guess i'll go loking
18:54 iurygregory ansiwen, did you run bundle exec rake lint and bundle exec rake syntax in your local machine? .-.
18:54 EmilienM mwhahaha: look at devstack :-P
18:54 ansiwen iurygregory: no, that's the CI
18:54 EmilienM mwhahaha: really not urgent atm though
18:55 EmilienM just giving more work here :P
18:55 mwhahaha look at devstack, what am i a python developer? :D
18:56 colonwq mwhahaha, Isn't there an awk script to convert python to perl?
18:56 mwhahaha rm -rf /
18:59 ansiwen iurygregory: oh, you mean I should have done that before?
19:00 iurygregory ansiwen, maybe i'm pushing your patch locally to test
19:00 iurygregory =)
19:01 rwsu_ joined #puppet-openstack
19:01 iurygregory ansiwen, http://paste.openstack.org/show/584289/
19:01 ansiwen iurygregory: I think I found something
19:02 ansiwen iurygregory: yes, I also found it just now... thanks :-)
19:02 iurygregory ansiwen, np =)
19:05 openstackgerrit Sven Anderson proposed openstack/puppet-openstack-integration: scenario003: Enable ec2api tempest tests  https://review.openstack.org/356442
19:14 openstackgerrit Merged openstack/puppet-tempest: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381423
19:15 openstackgerrit Merged openstack/puppet-neutron: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381413
19:15 openstackgerrit Merged openstack/puppet-nova: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381414
19:18 johnmilton joined #puppet-openstack
19:18 mwhahaha http://logs.openstack.org/28/381428/1/gate/g​ate-tripleo-ci-centos-7-nonha-multinode/c5e6​317/console.html#_2016-10-04_18_21_17_722013 :/
19:24 mesamoo joined #puppet-openstack
19:24 openstackgerrit Merged openstack/puppet-openstack_extras: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381581
19:26 EmilienM jayg: ^
19:28 mwhahaha iurygregory: https://review.openstack.org/#/c/378002/ +A?
19:29 EmilienM jayg: could you look at this error please?
19:33 mwhahaha looks like it's happening on all the voting jobs now
19:33 iurygregory mwhahaha, done
19:34 cdelatte joined #puppet-openstack
19:35 openstackgerrit Merged openstack/puppet-openstack-guide: Rework release page  https://review.openstack.org/378002
19:43 iurygregory ansiwen, any reason to not use v3 in https://review.openstack.org/#/c/374537 ?
19:43 iurygregory v2 is deprecated since M i think
19:44 ansiwen iurygregory: I know, I'm trying both. I had problems with V3, so for now I go with v2 and when it's basically working I will investigate that
19:45 iurygregory i asked because i saw this error Error: Puppet::Type::Keystone_use​r_role::ProviderOpenstack: Could not find user with name [ec2api-tester] and domain [Default]
19:46 openstackgerrit Merged openstack/puppet-sahara: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381420
19:46 openstackgerrit Merged openstack/puppet-trove: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381427
19:47 ansiwen iurygregory: now in the last run?
19:47 ansiwen iurygregory: I had other errors with V3 before... let's see
19:48 iurygregory i'll try to take a look when i have time
19:48 iurygregory i'll look at older logs
19:50 moravec joined #puppet-openstack
19:55 jayg EmilienM: I know there is currently some work going on with dlrn issues for newton and some of the rdo guys are working on it, but I don't know what the fixes are myself
19:55 jayg they are in progress though, see #rdo
19:55 * jayg afk for a few
19:56 iurygregory oh recheck i'll not help for now =( maybe we should wait
20:06 paramite joined #puppet-openstack
20:06 openstackgerrit Merged openstack/puppet-openstacklib: normalize_ip_for_uri: allow to give an array of IP addresses  https://review.openstack.org/381545
20:13 colonwq dmsimard, Can you help with a question about patches 381083/377158 and tempest?
20:14 openstackgerrit Merged openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
20:14 dmsimard colonwq: wassup
20:15 colonwq dmsimard, so 377158 is testing 002 for cinder-backup but 001 is failing because tempest is looking for cinder-backup even though backup = false in the tempest.conf.
20:15 dmsimard let me see
20:16 colonwq The opposite occurs 377158 is the other way around. 002 is failing when it shoudn't from my understanding.
20:19 mwhahaha EmilienM: dmsimard: found this magical nova-placement-api which of course is packaged but is just a bin and an apache conf openstack-nova-placement-api-14.0.0-0.2​0160929203854.59653c6.el7.centos.noarch so we need to provide a puppet class for it since the rpm config file gets nuked by our stuff
20:20 EmilienM cool
20:20 EmilienM like nova::api::placement
20:20 EmilienM and deploy the wsgi stuff
20:21 mwhahaha yea
20:21 mwhahaha i'll create a bug
20:22 EmilienM mwhahaha: feel free to assign it to me, I'm happy to look at it after newton release
20:23 mwhahaha https://bugs.launchpad.net​/puppet-nova/+bug/1630365
20:23 openstack Launchpad bug 1630365 in puppet-nova "nova-placement-api support" [Medium,Confirmed] - Assigned to Emilien Macchi (emilienm)
20:24 EmilienM i'll do it once we have ocata packages
20:24 openstackgerrit Merged openstack/puppet-glance: Changed the home-page to point Openstack Puppet Homepage  https://review.openstack.org/381395
20:25 mwhahaha it exists in newton
20:25 mwhahaha so whenever
20:25 EmilienM mwhahaha: yeah but really low prio in my list now :)
20:26 EmilienM and we won't backport it I think since it's a feature
20:26 mwhahaha i'm sure someone will ask for it but we'll cross that bridge later
20:26 EmilienM if we add a .pp file in puppet-nova, ok for backporting it
20:27 EmilienM if it affects the interface, we might need to revisit that
20:27 openstackgerrit Sven Anderson proposed openstack/puppet-tempest: WIP Test - DO NOT MERGE  https://review.openstack.org/382059
20:27 EmilienM mwhahaha: so all CI is now blocked?
20:27 mwhahaha yea
20:28 EmilienM sigh
20:28 mwhahaha anyone with a voting tripleo
20:28 mwhahaha cause it's like completely hosed
20:36 EmilienM mwhahaha: sorry, I don't get the thing about tripleo
20:37 mwhahaha not all modules have the voting tripleo ci
20:37 mwhahaha the ones that do are blocked now because of the dlrn problem
20:37 EmilienM right
20:37 EmilienM ovb jobs are not voting
20:37 EmilienM tripleo jobs are also blocked by delorean?
20:38 mwhahaha yea
20:38 mwhahaha that's what i pasted, http://logs.openstack.org/28/381428/1/gate/g​ate-tripleo-ci-centos-7-nonha-multinode/c5e6​317/console.html#_2016-10-04_18_21_17_722013
20:38 iurygregory ci is orange XD
20:38 mwhahaha http://logs.openstack.org/01/381901/1/check/g​ate-tripleo-ci-centos-7-nonha-multinode/b204d​be/console.html#_2016-10-04_20_01_39_913983
20:39 mwhahaha it's related to those rename specs i guess
20:39 mwhahaha somehow something changed such that now it can't find the source
20:39 shardy joined #puppet-openstack
20:40 EmilienM mwhahaha: ok thanks
20:51 dmsimard colonwq: haven't forgot you, sort of stuck in a fire right now
21:03 colonwq dmsimard, no problem. I will be back in a (long) while. Just let me know if you come up with an idea or a correction for me.
21:30 Vinsh mwhahaha: Hey -> https://review.openstack.org/#/c/360032/3
21:30 mwhahaha there's an old one
21:30 mwhahaha what's up
21:30 Vinsh the statsd stuff from proxy-server.conf.erb was dropped in my last patch
21:30 mwhahaha booo
21:30 Vinsh There was no statsd_enabled variable in proxy.pp
21:31 Vinsh so it was never being used
21:31 mwhahaha hmm
21:31 mwhahaha so i think it was used in a different way or something
21:32 mwhahaha i vaguely remember it being a pain tracking all the places that the configs got generated
21:33 Vinsh There would have to be a $statsd_enabled in proxy.pp to enable that
21:33 Vinsh if it was in a filter.. it wouldn't apply to that concat segment in proxy.p;p
21:33 Vinsh anyway,  I can get a patch up soon to add it to the new ini style
21:35 mwhahaha k
21:35 mwhahaha i don't think anyone was using it, i think i did it because of a bug
21:35 moravec joined #puppet-openstack
21:36 Vinsh I had found that part when I was working on this and thought "huh, wonder when that was added".. then I was just syncing our internal repo with upstream and saw your patch :)
21:36 Vinsh Now it all makes sense. ^_^
21:38 ducttape_ joined #puppet-openstack
21:50 openstackgerrit Matthew J Black proposed openstack/puppet-barbican: Update barbican for Debian based systems  https://review.openstack.org/376556
21:53 Vinsh This one I missed reviewing.. but it doesn't do what it claims : https://review.openstack.org/#/c/363003/
21:53 Vinsh The pipeline ordering is set in proxy.pp.. and depends on the user to set it properly.
21:53 Vinsh Merly ordering the concat segments wouldn't matter.
22:14 ducttape_ joined #puppet-openstack
22:32 17SAAKTPZ joined #puppet-openstack
22:40 xarses joined #puppet-openstack
23:03 ducttape_ joined #puppet-openstack
23:16 sthillma joined #puppet-openstack
23:19 ducttape_ joined #puppet-openstack
23:26 iurygregory_ joined #puppet-openstack
23:39 ducttape_ joined #puppet-openstack
23:41 ducttape_ joined #puppet-openstack

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