Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:03 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/252664
00:03 zhangjn joined #puppet-openstack
00:09 RichardRaseley joined #puppet-openstack
00:14 nlahouti joined #puppet-openstack
00:21 baker joined #puppet-openstack
00:22 xingchao joined #puppet-openstack
00:26 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/252664
00:44 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/252664
00:55 zhangjn joined #puppet-openstack
00:58 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/252693
01:02 xingchao joined #puppet-openstack
01:03 xarses joined #puppet-openstack
01:03 xarses joined #puppet-openstack
01:24 rahulait joined #puppet-openstack
01:29 aimon joined #puppet-openstack
01:35 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/252693
01:36 xingchao joined #puppet-openstack
01:42 xarses joined #puppet-openstack
02:07 nlahouti joined #puppet-openstack
02:38 mdorman joined #puppet-openstack
02:54 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/252693
02:54 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Only require netaddr when needed  https://review.openstack.org/252720
02:55 zhangjn joined #puppet-openstack
03:01 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/248316
03:04 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
03:29 ducttape_ joined #puppet-openstack
03:36 xarses_ joined #puppet-openstack
04:00 openstackgerrit Clayton O'Neill proposed openstack/puppet-openstack-specs: Enhance inifile option deprecations spec  https://review.openstack.org/240515
04:09 xingchao_ joined #puppet-openstack
04:20 jerrygb joined #puppet-openstack
04:34 gildub joined #puppet-openstack
04:45 zhangjn joined #puppet-openstack
05:00 jamespage joined #puppet-openstack
05:25 markvoelker joined #puppet-openstack
05:26 zhangjn joined #puppet-openstack
05:27 zhangjn joined #puppet-openstack
05:28 markvoelker_ joined #puppet-openstack
05:31 markvoelker joined #puppet-openstack
05:37 mflobo joined #puppet-openstack
05:38 markvoelker_ joined #puppet-openstack
06:22 jerrygb joined #puppet-openstack
06:31 karmix joined #puppet-openstack
06:34 nlahouti joined #puppet-openstack
06:50 markvoelker joined #puppet-openstack
06:51 mflobo joined #puppet-openstack
06:51 xingchao joined #puppet-openstack
06:57 xarses joined #puppet-openstack
06:57 nlahouti joined #puppet-openstack
06:58 xarses joined #puppet-openstack
06:59 xarses joined #puppet-openstack
07:00 xarses_ joined #puppet-openstack
07:02 bkero joined #puppet-openstack
07:10 bkero joined #puppet-openstack
07:10 nlahouti1 joined #puppet-openstack
07:19 rcernin joined #puppet-openstack
07:24 rcernin_ joined #puppet-openstack
07:34 nlahouti joined #puppet-openstack
07:53 nlahouti joined #puppet-openstack
07:55 migi joined #puppet-openstack
08:01 beddari joined #puppet-openstack
08:14 zhangjn joined #puppet-openstack
08:15 mgrosz joined #puppet-openstack
08:15 mgrosz Hello, can someone please review new puppet-vitrage init project patch https://review.openstack.org/#/c/252214/? Thanks
08:21 mgrosz Someone is online?
08:25 markvoelker joined #puppet-openstack
08:27 jpena joined #puppet-openstack
08:38 openstackgerrit Matt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check. Hyphens will be matched as a word boundry, so add a space to avoid false positives. For example, with an exisitng "demo-type1" we can't create a new "type1".  https://review.openstack.org/252821
08:41 openstackgerrit Matt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check.  https://review.openstack.org/252821
08:43 openstackgerrit Matt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check.  https://review.openstack.org/252821
08:49 dgurtner joined #puppet-openstack
08:49 dgurtner joined #puppet-openstack
08:53 mgrosz joined #puppet-openstack
08:53 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/252664
09:08 openstackgerrit Javier Peña proposed openstack/puppet-neutron: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245229
09:10 openstackgerrit Javier Peña proposed openstack/puppet-neutron: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245229
09:11 openstackgerrit Javier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245715
09:12 openstackgerrit Javier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245721
09:14 mgrosz joined #puppet-openstack
09:14 openstackgerrit Javier Peña proposed openstack/puppet-ironic: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245892
09:41 paramite joined #puppet-openstack
09:59 degorenko joined #puppet-openstack
09:59 rcernin joined #puppet-openstack
09:59 zhangjn joined #puppet-openstack
10:03 IvanBerezovskiy joined #puppet-openstack
10:05 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-glance: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/246926
10:07 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/247467
10:09 jistr joined #puppet-openstack
10:10 derekh joined #puppet-openstack
10:13 openstackgerrit Merged openstack/puppet-murano: Add ability to deploy Murano CloudFoundry service broker API  https://review.openstack.org/245043
10:17 mgrosz joined #puppet-openstack
10:26 markvoelker joined #puppet-openstack
10:33 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/252664
10:34 yottatsa joined #puppet-openstack
10:43 social EmilienM: can you also have look at https://review.openstack.org/#/c/221273/ ?
10:43 mgrosz joined #puppet-openstack
10:48 dgurtner joined #puppet-openstack
11:02 derekh joined #puppet-openstack
11:04 gfidente joined #puppet-openstack
11:04 gfidente joined #puppet-openstack
11:06 jayg|g0n3 joined #puppet-openstack
11:09 rochaporto joined #puppet-openstack
11:16 yottatsa joined #puppet-openstack
11:18 kylo joined #puppet-openstack
11:25 mgrosz joined #puppet-openstack
11:27 iurygregory good morning =)
11:35 mkarpin Hey!
11:49 xingchao joined #puppet-openstack
11:55 openstackgerrit Merged openstack/puppet-nova: Extend nova quota support  https://review.openstack.org/249671
12:04 degorenko clayton, hey o/ can you please merge https://review.openstack.org/250741 and https://review.openstack.org/250288 ?
12:04 degorenko crinkle, spredzy sbadia &&& guys, please :)
12:05 degorenko and this one: https://review.openstack.org/229895
12:05 iurygregory https://review.openstack.org/#/c/252536/ this one too XD
12:06 clayton looking, you can review https://review.openstack.org/#/c/252720/ for me
12:08 degorenko clayton, nice catch!
12:08 clayton well, it broke a patch of mine :)
12:09 degorenko i guess, next patch with hooks? :)
12:09 clayton yeap
12:09 degorenko looking
12:09 clayton you're welcome to review that one too, but it's huge
12:17 derekh joined #puppet-openstack
12:23 degorenko clayton, few comments after first quick review :)
12:24 degorenko do not forget please about this cherry pick please :) https://review.openstack.org/250741
12:27 markvoelker joined #puppet-openstack
12:28 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/252693
12:30 openstackgerrit Merged openstack/puppet-horizon: Fix wrong use of os_package_type  https://review.openstack.org/250288
12:33 openstackgerrit joined #puppet-openstack
12:36 zhangjn joined #puppet-openstack
12:41 openstackgerrit Merged openstack/puppet-glance: Update Glance Swift backend configuration  https://review.openstack.org/250741
12:44 EmilienM good morning
12:44 EmilienM social: sure
12:44 dprince joined #puppet-openstack
12:44 openstackgerrit Merged openstack/puppet-openstacklib: Catch HTTP 403 response  https://review.openstack.org/251477
12:45 iurygregory left #puppet-openstack
12:45 iurygregory joined #puppet-openstack
12:45 openstackgerrit Merged openstack/puppet-openstack_extras: Fix installation instruction  https://review.openstack.org/248375
12:46 mkarpin need core review here https://review.openstack.org/#/c/237035/ please! thanks
12:47 mkarpin degorenko,crinkle, mfisch, sbadia, spredzy : ^^
12:47 ducttape_ joined #puppet-openstack
12:51 openstackgerrit Merged openstack/puppet-glance: Stop collecting glance services by name  https://review.openstack.org/240630
12:52 openstackgerrit Merged openstack/puppet-horizon: Remove installation of lesscpy package  https://review.openstack.org/248162
12:56 openstackgerrit Merged openstack/puppet-nova: Remove docs for unused cluster_id parameter  https://review.openstack.org/243896
12:58 degorenko EmilienM, morning :)
12:59 iurygregory hey denis can you take a look https://review.openstack.org/#/c/252536/
12:59 iurygregory =)
12:59 degorenko mkarpin, what about second part of $::os_service_default for heat? :)
13:00 degorenko iurygregory, will look
13:00 mkarpin degorenko: its ready to be reviewed also :)
13:01 degorenko mkarpin, link :)
13:01 mkarpin https://review.openstack.org/#/c/249711/
13:05 degorenko mkarpin, -1 for 237035
13:05 degorenko take a look
13:09 degorenko mkarpin, for heat os_service_default also -1 :) you forgot about init.pp class
13:09 degorenko we should have two patches :) as it was discussed on summit :) first for ensure => absent and second for all another
13:09 degorenko it can be one common, if module is not so big
13:10 iurygregory denis https://github.com/openstack/puppet-heat/blob/master/manifests/init.pp init already have $::os_service_default
13:10 mkarpin degorenko: i switched all  what i thought can be switched to os_service_default in init.pp in patch part1
13:10 degorenko not all parameters
13:11 iurygregory oh ^^
13:11 mkarpin all other i think is different from default values in docs
13:11 mkarpin but i will double check
13:13 degorenko yes please, if all parameters are ok - then i will +2A
13:14 degorenko iurygregory, why you didn't change logging and db classes for magnum?
13:16 iurygregory sure i'll update =)
13:18 openstackgerrit joined #puppet-openstack
13:21 jerrygb joined #puppet-openstack
13:24 mkarpin degorenko: ok, will do, thanks for review!
13:28 markvoelker joined #puppet-openstack
13:30 zhangjn joined #puppet-openstack
13:34 openstackgerrit Clayton O'Neill proposed openstack/puppet-monasca: Remove obsolete tenant parameter  https://review.openstack.org/252954
13:38 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: add @default_facts to magnum  https://review.openstack.org/252536
13:39 morazi joined #puppet-openstack
13:50 markvoelker joined #puppet-openstack
13:52 markvoelker_ joined #puppet-openstack
13:54 openstackgerrit Merged openstack/puppet-nova: Only require netaddr when needed  https://review.openstack.org/252720
13:56 bdossant joined #puppet-openstack
13:57 bdossant Hi! I'm trying to run unit tests locally with "bundle exec rake spec" but i get /root/puppet-nova/openstack/puppet-openstack-integration/functions: line 51: /usr/local/share/gems/bin/r10k: No such file or directory
13:57 bdossant can anyone point me in the right direction?
13:57 zhangjn joined #puppet-openstack
13:57 iurygregory when i get this i remove my directory with gems and do bundle install again
13:57 bdossant "bundle exec rspec spec/acceptance" requires a vagrant node, can i run this locally?
13:58 iurygregory if you have everything you can =)
13:58 bdossant ok
13:58 iurygregory i only run bundle exec rspec spec
13:58 zhangjn joined #puppet-openstack
13:58 bdossant so remove "/usr/local/share/gems/bin" ?
13:59 iurygregory no XD
13:59 bdossant but do you have vagrant on your box?
13:59 iurygregory in you puppet-<module>
13:59 iurygregory directory
13:59 iurygregory you have .bundle_gems ?
14:00 iurygregory bdossant, http://my1.fr/blog/puppet-module-functional-testing-with-vagrant-openstack-and-beaker/ take a look at the shell script here =)
14:00 zhangjn joined #puppet-openstack
14:01 zhangjn joined #puppet-openstack
14:02 bdossant thnaks
14:02 bdossant i thought i could run this locally
14:02 iurygregory i run locally on my virtual machine o.o
14:03 bdossant ah
14:04 bdossant so i just need the second script
14:04 bdossant if i install vagrant in my vm
14:05 iurygregory i just use the shell script in my virtual machine and it works =( but you can set up vagrant very easily
14:09 RichardRaseley joined #puppet-openstack
14:10 zhangjn joined #puppet-openstack
14:11 yottatsa joined #puppet-openstack
14:14 social IvanBerezovskiy: thanks, looks like I should take some typing lessons
14:30 bdossant_ joined #puppet-openstack
14:44 zhangjn joined #puppet-openstack
14:47 xingchao joined #puppet-openstack
14:53 derekh joined #puppet-openstack
14:54 bdossant joined #puppet-openstack
14:57 dfisher joined #puppet-openstack
15:00 ducttape_ joined #puppet-openstack
15:00 vinsh joined #puppet-openstack
15:02 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/250436
15:04 dhollinger joined #puppet-openstack
15:05 openstackgerrit Lukas Bezdicka proposed openstack/puppet-trove: Remove trove ubuntu package hack  https://review.openstack.org/221273
15:07 iurygregory degorenko, it's done ;) https://review.openstack.org/#/c/252536/
15:10 EmilienM aderyugin: I dropped a comment on https://review.openstack.org/#/c/250436/ - I'm happy to see more testing !!!
15:11 karimb joined #puppet-openstack
15:18 bdossant joined #puppet-openstack
15:20 ntpttr joined #puppet-openstack
15:22 degorenko iurygregory, just a little comment - update please commit message :)
15:23 degorenko and title
15:23 iurygregory oh sorry XD
15:23 degorenko now your patch is fully switch os_service_defualt :)
15:24 degorenko EmilienM, spredzy https://review.openstack.org/#/c/237970/ take a look please :) and this one https://review.openstack.org/246988
15:24 degorenko vinsh, https://review.openstack.org/246988 i've updated patch :) take a look please
15:24 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Switch magnum to $::os_service_default  https://review.openstack.org/252536
15:24 EmilienM degorenko: ok
15:25 iurygregory degorenko, done
15:25 degorenko iurygregory, +2 :)
15:25 EmilienM degorenko: glance::api::db ?
15:25 EmilienM it should be glance::db
15:25 EmilienM why separating it?
15:26 degorenko EmilienM, we have separated options
15:26 degorenko for glance_api
15:26 degorenko and for glance_registry
15:26 degorenko https://github.com/openstack/puppet-glance/blob/master/manifests/api.pp#L268-L283
15:27 degorenko https://github.com/openstack/puppet-glance/blob/master/manifests/registry.pp#L191-L206
15:27 degorenko EmilienM, ^
15:27 EmilienM degorenko: ok
15:27 EmilienM I'll review them today
15:30 openstackgerrit Merged openstack/puppet-nova: Create networks only if nova-network is enabled  https://review.openstack.org/229895
15:32 trown EmilienM: the selinux module just returns nil if there is no selinux support https://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/selinux.rb#L85
15:32 EmilienM trown: mhh
15:32 EmilienM so I guess we can go ahead with your patch
15:32 trown EmilienM: so I think we do not need a conditional to use seltype ... and a conditional inside of file is pretty gross
15:32 EmilienM ok
15:33 EmilienM trown: +2
15:33 trown EmilienM: awesome. thanks!
15:35 mkarpin degorenko: I answered on your comment https://review.openstack.org/#/c/237035/ , so what do you think?
15:35 openstackgerrit Alexey Deryugin proposed openstack/puppet-tempest: Allow to activate Murano Service Broker tests  https://review.openstack.org/253019
15:36 degorenko mkarpin, looks like hardcode :)
15:36 vinsh degorenko: Will do
15:37 degorenko we finally need to switch glance to os_service_default
15:37 degorenko spredzy, do you working at this? ^^^
15:39 xarses_ joined #puppet-openstack
15:41 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/250436
15:41 baker joined #puppet-openstack
15:41 EmilienM aderyugin: maybe we can align on https://review.openstack.org/#/c/253019/1/manifests/init.pp,cm ?
15:42 degorenko EmilienM, i have news for you :) oslo.messaging dropped support of qpid :)
15:42 EmilienM degorenko: yeah?
15:42 degorenko i will create a card for it and update our manifests
15:43 degorenko EmilienM, yeah :)
15:43 EmilienM degorenko: that means openstack does not support qpid at all?
15:43 degorenko i guess yes
15:43 degorenko from mitaka
15:43 chem joined #puppet-openstack
15:44 degorenko EmilienM, https://github.com/openstack/oslo.messaging/commit/925eb734a9d3cb46bb89a89ec1a78281d2d7afe9
15:45 chem left #puppet-openstack
15:45 EmilienM degorenko: thanks
15:45 derekh joined #puppet-openstack
15:45 degorenko i will upload changes :)
15:45 degorenko EmilienM, fyi, progress will be here: https://trello.com/c/nQe9qrkt/76-remove-support-of-qpid
15:46 EmilienM degorenko: I suggest we send a warning if QPID options are set, and backport it in stable/liberty
15:46 EmilienM so people using stable/liberty have 5 months to move to something else but qpid
15:46 degorenko EmilienM, of course, i will deprecated all qpid for mitaka
15:46 degorenko and liberty
15:46 EmilienM we need also to deprecate parameters
15:46 EmilienM degorenko: you awesome
15:46 degorenko :D
15:47 degorenko EmilienM, as i said on proposing me to core: will do my best ;)
15:48 EmilienM :)
15:53 clayton so is keystone v2 just not supported by puppet-keystone anymore?
15:54 clayton I'm trying to upgrade to 6.1.0 of the module, and everything appears to be hard coded for Puppet::Provider::Openstack::CredentialsV3 now
15:55 markvoelker joined #puppet-openstack
16:06 clayton it is always setting OS_IDENTITY_API_VERSION=3 and there doesn't appear to be a way to override that.
16:07 EmilienM good question, you should be able to use v2 iirc
16:07 clayton I can't figure out how
16:07 EmilienM chem and richm are not here ...
16:08 clayton version is set using this "self.class.to_s.sub(/.*V/,'').sub('_','.')"
16:08 clayton and it's hardcoded to use CredentialsV3
16:13 EmilienM clayton: I think you can still use v2 somehow, iirc our functional tests used to test both
16:13 clayton well, I'd love to be wrong, this has me dead in the water on upgrading
16:13 EmilienM https://github.com/openstack/puppet-keystone/blob/stable/kilo/spec/acceptance/keystone_wsgi_apache_spec.rb#L142
16:13 EmilienM clayton: this shows we run tests against v2 too
16:14 clayton @EmilienM this is specifically for keystone_user
16:14 EmilienM you need to specifcy --os-identity-api-version 2
16:14 clayton sorry, I should have been more clear
16:14 EmilienM ok
16:14 EmilienM sorry I missed that
16:14 EmilienM I'll ping richm to make sure
16:17 openstackgerrit Denis Egorenko proposed openstack/puppet-aodh: Deprecate qpid messaging parameters  https://review.openstack.org/253042
16:17 openstackgerrit Denis Egorenko proposed openstack/puppet-ceilometer: Deprecate qpid messaging parameters  https://review.openstack.org/253043
16:21 chem joined #puppet-openstack
16:27 iurygregory EmilienM, https://review.openstack.org/252536 o/ sorry i've update the commit message
16:27 EmilienM chem: you were offline but clayton mentionned he can't use keystone v2 for keystone_user provider in 6.1.0 (kilo release) - could you check that?
16:28 EmilienM logs: http://irclog.perlgeek.de/puppet-openstack/
16:28 chem EmilienM: thanks :)
16:37 clayton my issue specifically is that I had a /root/openrc that existed, but had auth_url set to the v2 endpoint
16:38 clayton but the keystone_user provider was explicitly setting OS_API_VERSION to 3
16:38 clayton which caused openstackclient to barf
16:38 openstackgerrit Nate Potter proposed openstack/puppet-cinder: Remove references to puppet-openstack from README  https://review.openstack.org/248379
16:39 chem clayton: I'm having a look
16:39 openstackgerrit Tim Rozet proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/249390
16:40 aglarendil joined #puppet-openstack
16:43 ddmitriev joined #puppet-openstack
16:50 openstackgerrit Denis Egorenko proposed openstack/puppet-heat: Deprecate qpid messaging parameters  https://review.openstack.org/253062
16:53 rcernin joined #puppet-openstack
16:53 chem clayton: there is nothing specify about the keystone_user provider 6.1.0, all the other keystone provider (tenant, user_role, ...) are using CredentialsV3.  Did I misunderstand something here /
16:53 chem ?
16:54 clayton the keystone_user provider requires v3 api be used, right?
16:54 chem clayton: all of them use CredentialsV3
16:54 chem clayton: but they also parse the $HOME/openrc for auth info
16:54 clayton that was a change in 6.0.0 that wasn't mentioned in the release notes
16:56 chem clayton: I think the problem may be comming from your /root/openrc in a case that was not foreseen
16:56 EmilienM chem: clayton wants to use v2, that's the point.
16:56 chem EmilienM: I got this :)
16:56 EmilienM sorry :)
16:56 clayton well, I'm going to just remove the openrc and move on, just kind of annoyed the failure mode is so obscure and the change wasn't documented
16:57 openstackgerrit Denis Egorenko proposed openstack/puppet-ironic: Deprecate qpid messaging parameters  https://review.openstack.org/253066
16:58 chem clayton: as I said, it was not foreseen that an openrc would have the v2 info.  I could debug the stuff and try to backport the change if I find the source of the problem.  The module is supposed to be backward comptatible, that's just something that has been missed.
16:58 nlahouti joined #puppet-openstack
16:58 chem clayton: could you paste me the openrc (without the real creadential) so I invistage further
16:58 chem ?
16:58 clayton personally I've never really understood why it reads from /root/openrc
16:59 cwolferh joined #puppet-openstack
16:59 EmilienM clayton: I wrote this release note I think - so that's my fault probably :(
16:59 clayton EmilienM have you been following the stuff for reno?
16:59 EmilienM you're right, it's not really clear in https://github.com/openstack/puppet-keystone/blob/stable/kilo/CHANGELOG.md
17:00 EmilienM clayton: no
17:00 EmilienM ah yes I did
17:00 EmilienM in fact
17:00 clayton reno is the new tool all the core openstack projects are switching to for release notes in mitaka
17:00 clayton seems like a good approach, or at least a big improvement
17:00 holser_ joined #puppet-openstack
17:00 clayton EmilienM it's just hard to do release notes after the fact
17:01 iurygregory EmilienM it's not only your fault is our fault too because we reviewed =)
17:01 EmilienM clayton: I'll look at reno and see if our project can use it
17:02 EmilienM clayton: https://trello.com/c/upZVtMps/77-look-at-reno-for-release-notes
17:03 chem If it has happened to clayton that may be happening to others.  I'm going to debug (openrcv2 and puppet-keystone-6.1.0) this and check what we can do.  That is, I think, a bug.
17:04 xingchao joined #puppet-openstack
17:05 clayton chem: https://gist.github.com/dvorak/32a85fa6ab0d6cbbe56b
17:05 openstackgerrit Denis Egorenko proposed openstack/puppet-manila: Deprecate qpid messaging parameters  https://review.openstack.org/253071
17:05 chem clayton: thanks
17:05 nlahouti joined #puppet-openstack
17:10 EmilienM clayton: I think reno is really worth to use it for us
17:15 EmilienM mwhahaha: could you reconsider https://review.openstack.org/252562 please?
17:17 openstackgerrit Denis Egorenko proposed openstack/puppet-mistral: Deprecate qpid messaging parameters  https://review.openstack.org/253085
17:19 RichardRaseley joined #puppet-openstack
17:21 baker joined #puppet-openstack
17:25 yottatsa joined #puppet-openstack
17:25 openstackgerrit Denis Egorenko proposed openstack/puppet-neutron: Deprecate qpid messaging parameters  https://review.openstack.org/253090
17:31 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Deprecate qpid messaging parameters  https://review.openstack.org/253097
17:33 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Remove deprecated parameters since Liberty release  https://review.openstack.org/253099
17:42 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Deprecate qpid messaging parameters  https://review.openstack.org/253105
17:57 openstackgerrit Denis Egorenko proposed openstack/puppet-trove: Deprecate qpid messaging parameters  https://review.openstack.org/253116
18:12 chem clayton: Just two quick questions: are you upgrading from 5.2.0 (juno) or 6.0.0, and does  curl http://127.0.0.1:5000/v3 works on your setup ?  I'm trying to reproduce the problem and wouldn't want to make wrong assumption.
18:13 clayton v3 does actually work for this
18:13 clayton so removing the openrc resolved the issue for now
18:18 chem clayton: oki, I could reproduce the bug.
18:18 chem clayton: checking what's wrong with this now :)
18:21 davideagle_ joined #puppet-openstack
18:25 chem clayton: Oki, I got the idea.  The keystone_user is special in that for checking the password the classic CredentialV3 class is bypassed and a direct 'token issue' is made.  That's where the problem is and that's why it happen only with keystone_user.  I'm filling the bug now and will fix it (hopefully) tomorrow.
18:27 clayton cool, thanks for looking at it
18:30 iurygregory clayton, can you look https://review.openstack.org/#/c/252536/ ? ^^
18:33 holser_ joined #puppet-openstack
18:37 nlahouti joined #puppet-openstack
18:45 iurygregory joined #puppet-openstack
18:51 ericksonsantos joined #puppet-openstack
18:54 aimon joined #puppet-openstack
19:02 davideagle joined #puppet-openstack
19:02 chem clayton: https://bugs.launchpad.net/puppet-keystone/+bug/1522541
19:02 openstack Launchpad bug 1522541 in puppet-keystone "Having /root/openrc filled with v2 auth_url makes keystone_user fails." [Undecided,New]
19:04 openstackgerrit Merged openstack/puppet-nova: Allow customization of db sync command line  https://review.openstack.org/241009
19:09 derekh joined #puppet-openstack
19:11 openstackgerrit Merged openstack/puppet-heat: Allow customization of db sync command line  https://review.openstack.org/241299
19:12 chem clayton: you're bumping into a double bug in your case as your auth_url ends with '/' which make the overall url http://127.0.0.1:5000/v2.0//v3 (do not happen anymore in 7.0.0)
19:12 chem EmilienM: should we make another bug to this version only ^ ?
19:13 EmilienM chem: if it's a different patch, yes I think
19:13 chem EmilienM: ack
19:22 bnemec joined #puppet-openstack
19:29 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/253157
19:30 chem clayton: if you remove the last "/" from your openrc this will fix the issue ^
19:31 chem EmilienM: how do I do a patch for 6.1.0 only, should I checkout 6.1.0 and patch against it ?
19:31 EmilienM nope
19:31 EmilienM stable/kilo
19:31 EmilienM chem: ^
19:31 chem EmilienM: ack
19:31 EmilienM maybe we need to patch master first?
19:33 chem EmilienM: the patch above is for master and should be backported after, but I have another bug https://bugs.launchpad.net/puppet-keystone/+bug/1522550 which is specific to 6.1.0 (stable/kilo)
19:33 openstack Launchpad bug 1522550 in puppet-keystone "If /root/openrc filled with v2 auth_url ending with "/" make keystone_user fails." [Undecided,New]
19:33 chem yeah ^
19:33 EmilienM chem: ok, so then only stable/kilo
19:50 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: keystone_user fails if auth_url ends with '/' in Openrc  https://review.openstack.org/253170
19:50 chem clayton: with the two patches above you're good to go even with /root/openrc
19:52 davideagle_ joined #puppet-openstack
20:20 chem joined #puppet-openstack
20:25 RichardRaseley joined #puppet-openstack
20:26 openstackgerrit Clayton O'Neill proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/253188
20:26 openstackgerrit Clayton O'Neill proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/253189
20:27 EmilienM clayton: you need to wait the patch is merged in master before any backport
20:27 EmilienM because now you're missing the "Cherry-picked from"
20:27 EmilienM and the commit id
20:28 clayton ah.  that's annoying.  that's an easy way for it to get forgotten.
20:48 openstackgerrit joined #puppet-openstack
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Fix keystone configuration.  https://review.openstack.org/247274
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/248412
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/248413
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/248410
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Allow distro packages to install dependencies, etc  https://review.openstack.org/247277
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config.  https://review.openstack.org/248406
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/248404
20:59 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Default Rabbit and Qpid ports to 5672  https://review.openstack.org/247269
21:10 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
21:12 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
21:12 karimb joined #puppet-openstack
21:18 ianw joined #puppet-openstack
21:21 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
21:23 nlahouti joined #puppet-openstack
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Fix keystone configuration.  https://review.openstack.org/247274
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/248412
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/248413
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/248410
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Allow distro packages to install dependencies, etc  https://review.openstack.org/247277
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config.  https://review.openstack.org/248406
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/248404
21:24 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Default Rabbit and Qpid ports to 5672  https://review.openstack.org/247269
21:38 cdelatte joined #puppet-openstack
22:14 rahulait joined #puppet-openstack
22:29 nlahouti joined #puppet-openstack
22:43 gildub joined #puppet-openstack
23:04 baker joined #puppet-openstack
23:28 aimon joined #puppet-openstack
23:33 bnemec joined #puppet-openstack
23:38 openstackgerrit Merged openstack/puppet-keystone: keystone_user fails if auth_url ends with '/' in Openrc  https://review.openstack.org/253170
23:43 EmilienM clayton: ^ merged
23:50 openstackgerrit Merged openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/253157
23:53 ntpttr joined #puppet-openstack
23:53 ntpttr left #puppet-openstack

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