Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-01-12

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

All times shown according to UTC.

Time Nick Message
00:03 openstackgerrit Merged openstack/puppet-ceilometer: Add required db subclass to README.md  https://review.openstack.org/265049
00:04 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/242330
00:04 _ody EmilienM: Can you point me at a failed job resulting from what https://review.openstack.org/#/c/266069 is reverting?
00:05 _ody I don't see failures from rabbitmq related things on the patch that was previously merged.
00:05 EmilienM _ody: all recent jobs
00:05 EmilienM look at https://review.openstack.org/266004 for example
00:06 EmilienM I reported the bug https://bugzilla.redhat.com/show_bug.cgi?id=1297580
00:06 openstack bugzilla.redhat.com bug 1297580 in openstack-cinder "cinder GID is stolen by epmd" [Urgent,New] - Assigned to eharney
00:06 EmilienM but basically, all jobs for scenario001 on centos7 are failing because of this bug
00:07 EmilienM you can look at all scenario001/centos7 jobs since 5th january
00:07 EmilienM _ody: ^
00:07 rahulait joined #puppet-openstack
00:07 _ody ok. I was just expecting to see it in the orginal change's related tests.
00:07 _ody I must just be overlooking something
00:10 zhangjn joined #puppet-openstack
00:17 xarses joined #puppet-openstack
00:19 _ody EmilienM: Is this a case where the init system claims rabbitmq launched successfully what it doesn't actually do so?  I can see cinder is unable to connect to rabbitmq but I don't see the reason why.
00:22 openstackgerrit Merged openstack/puppet-neutron: Adds the ability to force metadata with dhcp agent  https://review.openstack.org/265048
00:33 EmilienM _ody: ok let me explain
00:34 EmilienM I did that patch because at the end we will want to have rabbitmq ready *before* any openstack service
00:34 EmilienM but the problem is https://bugzilla.redhat.com/show_bug.cgi?id=1297580 which is preventing us to do so
00:34 openstack bugzilla.redhat.com bug 1297580 in openstack-cinder "cinder GID is stolen by epmd" [Urgent,New] - Assigned to eharney
00:35 EmilienM when this bug ^ is fixed in Fedora, we'll be able to install rabbitmq before openstack (cinder included)
00:35 EmilienM and so the orchestration that we want
00:35 EmilienM but until that, we need a workaround (the revert I proposed)
00:35 EmilienM so we can have our integration jobs green
00:36 EmilienM in the meantime, if you observe openstack logs, you might notice openstack services can start before rabbitmq but they'll try to reconnect to amqp again
00:36 EmilienM thanks to something in oslo.messaging
00:36 EmilienM _ody: is it fine now?
00:40 ntpttr joined #puppet-openstack
00:46 openstackgerrit Merged openstack/puppet-openstack-integration: Revert "rabbitmq: make sure RabbitMQ resources are created before running OpenStack"  https://review.openstack.org/266069
00:46 ntpttr left #puppet-openstack
00:58 zhangjn joined #puppet-openstack
00:58 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: use new parameters to configure neutron/notifications  https://review.openstack.org/264387
00:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Improve logging for OpenStack services  https://review.openstack.org/265426
01:00 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Aodh on ubuntu platform  https://review.openstack.org/265061
01:00 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
01:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
01:20 markvoelker joined #puppet-openstack
01:25 markvoelker joined #puppet-openstack
01:57 openstackgerrit Xiaohua Yuan proposed openstack/puppet-designate: Add related parameters to designate::api  https://review.openstack.org/260024
02:01 xingchao joined #puppet-openstack
02:48 ilbot3 joined #puppet-openstack
02:48 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
02:50 xingchao_ joined #puppet-openstack
03:05 zhangjn joined #puppet-openstack
03:10 spredzy joined #puppet-openstack
03:17 openstackgerrit Merged openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/258136
03:18 openstackgerrit Merged openstack/puppet-murano: Change default admin_user and admin_tenant_name  https://review.openstack.org/254250
03:19 zhangjn joined #puppet-openstack
03:38 openstackgerrit Merged openstack/puppet-keystone: Fix multiple typos in README  https://review.openstack.org/265812
03:45 baker joined #puppet-openstack
03:46 cwolferh joined #puppet-openstack
03:52 baker_ joined #puppet-openstack
03:56 michchap joined #puppet-openstack
04:04 openstackgerrit Xingchao Yu proposed openstack/puppet-nova: Change section name for memcached_servers param  https://review.openstack.org/266136
04:27 rahulait joined #puppet-openstack
04:30 morazi joined #puppet-openstack
04:38 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/264538
04:38 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/266142
04:42 cwolferh joined #puppet-openstack
04:46 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/266142
05:15 michchap joined #puppet-openstack
05:25 zerda joined #puppet-openstack
05:35 zhangjn joined #puppet-openstack
05:47 jaosorior joined #puppet-openstack
05:50 zhangjn joined #puppet-openstack
05:59 markvoelker joined #puppet-openstack
06:35 zhangjn joined #puppet-openstack
06:38 zhangjn_ joined #puppet-openstack
06:51 xingchao joined #puppet-openstack
07:01 xingchao_ joined #puppet-openstack
07:09 jaosorior joined #puppet-openstack
07:30 zhangjn joined #puppet-openstack
07:52 degorenko joined #puppet-openstack
07:55 openstackgerrit Christian Schwede proposed openstack/puppet-gnocchi: Add missing metricd service  https://review.openstack.org/265772
07:56 jpena joined #puppet-openstack
07:57 jtomasek joined #puppet-openstack
08:00 dgurtner joined #puppet-openstack
08:06 jistr joined #puppet-openstack
08:16 zhangjn joined #puppet-openstack
08:19 karimb joined #puppet-openstack
08:26 openstackgerrit Christian Schwede proposed openstack/puppet-gnocchi: Add missing metricd service  https://review.openstack.org/265772
08:27 samuelBartel joined #puppet-openstack
08:52 openstackgerrit Merged openstack/puppet-openstack-integration: use new parameters to configure neutron/notifications  https://review.openstack.org/264387
08:52 xingchao joined #puppet-openstack
08:58 zhangjn joined #puppet-openstack
09:05 openstackgerrit Xingchao Yu proposed openstack/puppet-glance: Add memcached_servers parameter  https://review.openstack.org/266248
09:09 openstackgerrit Denis Egorenko proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2  https://review.openstack.org/249711
09:11 gfidente joined #puppet-openstack
09:11 gfidente joined #puppet-openstack
09:13 xarses joined #puppet-openstack
09:14 xarses joined #puppet-openstack
09:15 xarses joined #puppet-openstack
09:18 openstackgerrit joined #puppet-openstack
09:20 jistr joined #puppet-openstack
09:22 igajsin joined #puppet-openstack
09:27 karimb joined #puppet-openstack
09:28 chem joined #puppet-openstack
09:32 paramite joined #puppet-openstack
09:35 derekh joined #puppet-openstack
09:37 igajsin joined #puppet-openstack
09:38 openstackgerrit tobe proposed openstack/puppet-keystone: Use recommended abbreviation of the classes  https://review.openstack.org/266259
09:51 clayton joined #puppet-openstack
09:53 dh64 joined #puppet-openstack
09:55 derekh joined #puppet-openstack
09:55 social joined #puppet-openstack
09:56 morazi joined #puppet-openstack
10:02 markvoelker joined #puppet-openstack
10:05 igajsin joined #puppet-openstack
10:16 vinsh_ joined #puppet-openstack
10:18 xingchao_ joined #puppet-openstack
10:19 derekh joined #puppet-openstack
10:19 morazi joined #puppet-openstack
10:25 igajsin joined #puppet-openstack
10:30 rochaporto joined #puppet-openstack
10:31 openstackgerrit Gael Chamoulaud proposed openstack/puppet-heat: [WIP] Revise keystone/trusts config to avoid deprecated options  https://review.openstack.org/265865
10:55 jaosorior joined #puppet-openstack
11:00 zhangjn joined #puppet-openstack
11:11 openstackgerrit Liao Penghui proposed openstack/puppet-keystone: Fix bug for wrong config section of ssl configs.  https://review.openstack.org/266295
11:18 openstackgerrit joined #puppet-openstack
11:23 openstackgerrit Merged openstack/puppet-gnocchi: Add missing metricd service  https://review.openstack.org/265772
11:25 openstackgerrit Merged openstack/puppet-vitrage: Update typo  https://review.openstack.org/263177
11:27 openstackgerrit Merged openstack/puppet-zaqar: Update typo  https://review.openstack.org/263176
11:28 openstackgerrit Merged openstack/puppet-barbican: Update typo  https://review.openstack.org/263185
11:31 openstackgerrit Merged openstack/puppet-ceilometer: Improve format of the documentations.  https://review.openstack.org/265670
11:31 openstackgerrit Merged openstack/puppet-aodh: Update typo  https://review.openstack.org/263186
11:32 openstackgerrit Merged openstack/puppet-gnocchi: Update typo  https://review.openstack.org/263183
11:42 openstackgerrit yuan.lu proposed openstack/puppet-cinder: Fix wrong location for lock_path of params  https://review.openstack.org/266308
11:43 IvanBerezovskiy1 joined #puppet-openstack
11:50 jaosorior joined #puppet-openstack
12:03 markvoelker joined #puppet-openstack
12:07 dgurtner joined #puppet-openstack
12:07 dgurtner joined #puppet-openstack
12:10 dalees joined #puppet-openstack
12:12 IvanBerezovskiy1 joined #puppet-openstack
12:14 zhangjn joined #puppet-openstack
12:16 openstackgerrit Liao Penghui proposed openstack/puppet-keystone: Fix bug for wrong config section of ssl configs.  https://review.openstack.org/266295
12:22 EmilienM hello
12:27 EmilienM degorenko: hey, have you looked at https://etherpad.openstack.org/p/puppet-openstack-ci-mitaka ? We have some sahara issues on master
12:37 iurygregory joined #puppet-openstack
12:37 degorenko EmilienM, hey, yeah, i saw your letter on openstack-dev list, i'm trying find out what's wrong
12:58 dprince joined #puppet-openstack
13:04 markvoelker joined #puppet-openstack
13:09 ducttape_ joined #puppet-openstack
13:13 dgurtner_ joined #puppet-openstack
13:13 dgurtner_ joined #puppet-openstack
13:14 openstackgerrit Martin Mágr proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/234728
13:16 derekh joined #puppet-openstack
13:22 narasimhasv joined #puppet-openstack
13:22 EmilienM degorenko: are you able to reproduce?
13:22 EmilienM degorenko: I can help you to reproduce
13:23 EmilienM degorenko: you follow https://github.com/openstack/puppet-openstack-integration#all-in-one
13:23 EmilienM but before running all-in-one.sh, you change the scenario in the script. Sahara is in scenario002. Also you need to run on centos7
13:27 openstackgerrit Denis Egorenko proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/261326
13:27 degorenko EmilienM, i'm able to reproduce
13:27 omartsyniuk_ joined #puppet-openstack
13:27 degorenko some internal tasks
13:27 degorenko i'm going to run second deploy
13:27 degorenko for scenario002
13:29 degorenko EmilienM, btw, for sahara floating bug - as developers said me - it can be problem with pymysql driver in sahara, or in pymysql driver :) that's all what i have
13:35 markvoelker joined #puppet-openstack
13:37 degorenko EmilienM, and i've reproduced problem with sahara on mitaka. Looks like package problem.
13:37 EmilienM degorenko: any logs?
13:37 EmilienM degorenko: join #rdo if packaging problem
13:37 degorenko EmilienM, the same as you gave - ImportError: cannot import name main
13:38 EmilienM degorenko: that's a weird error
13:38 degorenko after when i will find problem
13:39 zhangjn joined #puppet-openstack
13:39 zhangjn joined #puppet-openstack
13:42 degorenko EmilienM, lol, on my env heat, neutron and glance are  also dead
13:43 EmilienM yeah? weird
13:43 iurygregory crazy things o.o
13:45 degorenko EmilienM, you also have this problem
13:45 degorenko take a look
13:45 degorenko http://logs.openstack.org/35/255635/15/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/b42f618/console.html#_2016-01-12_01_21_42_296
13:45 degorenko random line
13:45 degorenko glance, ironic and etc
13:46 degorenko really looks like package problem
13:46 degorenko the same for neutron
13:46 EmilienM degorenko: it's not a n issue
13:47 EmilienM this line is not an issue AFIK, I've seen that lot of times
13:48 degorenko EmilienM, what's about log files then: http://logs.openstack.org/35/255635/15/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/b42f618/logs/glance/api.txt.gz#_2016-01-12_01_31_19_157
13:48 EmilienM degorenko: this one is a valid error
13:49 degorenko ok, then continue investigating sahara problem
13:50 iurygregory left #puppet-openstack
13:50 iurygregory joined #puppet-openstack
13:51 EmilienM degorenko: if you have anything blocking in rdo, go ahead on #rdo channel, they are very responsive
13:53 openstackgerrit Ryan Bak proposed openstack/puppet-monasca: Add support for symlinked agent virtualenv  https://review.openstack.org/265357
13:55 xingchao joined #puppet-openstack
13:59 degorenko EmilienM, i've found problem
13:59 degorenko service py files are empty
14:00 EmilienM degorenko: w00t ?
14:01 degorenko take a look on rdo chat
14:01 degorenko i've posted link to paste.openstack
14:02 jpena|mtg joined #puppet-openstack
14:04 openstackgerrit Oleksiy Molchanov proposed openstack/puppet-ceph: [WIP] Add possibility to use mod_proxy for radosgw  https://review.openstack.org/266027
14:05 karimb joined #puppet-openstack
14:10 richm joined #puppet-openstack
14:11 openstackgerrit Ryan Bak proposed openstack/puppet-monasca: Add support for symlinked agent virtualenv  https://review.openstack.org/265357
14:19 openstackgerrit Oleksiy Molchanov proposed openstack/puppet-ceph: [WIP] Add possibility to use mod_proxy for radosgw  https://review.openstack.org/266027
14:26 openstackgerrit Xingchao Yu proposed openstack/puppet-glance: Add memcached_servers parameter  https://review.openstack.org/266248
14:28 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Use HAProxy 'transparent' bind option instead of sysctl ip_nonlocal  https://review.openstack.org/265309
14:35 karimb_ joined #puppet-openstack
14:39 mdorman joined #puppet-openstack
14:41 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/200877
14:44 EmilienM our weekly meeting will start in 15 min on #openstack-meeting-4, you can look at our agenda: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20160112
14:48 EmilienM social: do you have an update to give on https://review.openstack.org/182281 ?
14:48 EmilienM do you need some help?
14:50 degorenko EmilienM, my patch for sahara is merged :) waiting for updated package and will try to run tests again :)
14:50 EmilienM degorenko: cool
14:50 mkarpin joined #puppet-openstack
14:51 mwoodson joined #puppet-openstack
14:51 astupnikov joined #puppet-openstack
14:52 mflobo Hi EmilienM, question about gerrit process
14:53 mflobo EmilienM, what I need to move forward with puppet-ec2api puppet modules? https://review.openstack.org/#/c/252959/ and https://review.openstack.org/#/c/251857/
14:54 _ody SURPRISE! I got out of bed before meeeting time today.
14:54 mflobo EmilienM, maybe just wait for more +1s ? :)
14:54 mflobo oh, right, meeting now
14:55 _ody mflobo: Bringing them up in my browser, I'll review them with my first cup of coffee.
14:55 mflobo _ody, thanks
14:56 _ody and we'll need to ping infra too, they have to merge.
14:57 degorenko will look these patches too
14:57 ducttape_ joined #puppet-openstack
14:58 mflobo degorenko, thanks too!
15:02 dfisher joined #puppet-openstack
15:02 mwoodson joined #puppet-openstack
15:02 openstackgerrit Gael Chamoulaud proposed openstack/puppet-heat: Revise keystone/trusts config to avoid deprecated options  https://review.openstack.org/265865
15:04 openstackgerrit Merged openstack/puppet-cinder: Remove outdated scheduler_driver from documentation  https://review.openstack.org/265691
15:06 skolekonov joined #puppet-openstack
15:09 v1k0d3n joined #puppet-openstack
15:11 openstackgerrit Merged openstack/puppet-glance: Create glance::cache::logging class  https://review.openstack.org/238096
15:18 EmilienM gchamoul: feel free to join our upstream meeting if you need help on unit tests for swift
15:19 gchamoul EmilienM: yes I will join once the daughter will be back home from school
15:21 openstackgerrit Merged openstack/puppet-trove: Add rabbit_ha_queues option  https://review.openstack.org/214458
15:22 baker joined #puppet-openstack
15:23 openstackgerrit Merged openstack/puppet-neutron: Rename admin_state_up method to avoid conflict with accessor  https://review.openstack.org/265373
15:23 openstackgerrit Merged openstack/puppet-neutron: Do not manage network, subnet, router resources if prefetch fails  https://review.openstack.org/265386
15:24 openstackgerrit Merged openstack/puppet-trove: Introduce amqp_durable_queues option  https://review.openstack.org/214477
15:25 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Improve logging for OpenStack services  https://review.openstack.org/265426
15:29 kbyrne joined #puppet-openstack
15:30 kbyrne joined #puppet-openstack
15:30 dmsimard EmilienM: hmm, I think there's a problem with this line: https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh#L138 ?
15:30 dmsimard fails for me http://paste.openstack.org/show/483605/
15:30 EmilienM dmsimard: upstream meeting
15:30 dmsimard oops, okay.
15:36 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API  https://review.openstack.org/254181
15:37 rahulait joined #puppet-openstack
15:37 xarses joined #puppet-openstack
15:43 EmilienM dmsimard: weird, you're missing a variable I guess
15:44 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Fix bug for deprecated vnc options.  https://review.openstack.org/266443
15:45 openstackgerrit Merged openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters  https://review.openstack.org/255869
15:46 dhollinger joined #puppet-openstack
15:48 dmsimard EmilienM: found it, missing lsb_release command
15:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
15:51 EmilienM chem: ^ I added it in Depends-On
15:57 rochaporto joined #puppet-openstack
16:02 EmilienM dmsimard: cool
16:03 _ody gchamoul: Can you please shoot me a link to you failing tests so I can throw it on my queue to look at?
16:04 gchamoul _ody: https://review.openstack.org/#/c/250233/
16:06 gchamoul _ody: that's not related by code introduction in puppet-swift ... but some rspec tests were failing since puppet 4.3.0 came out and that change was expecting to fix that
16:07 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Allow passing the whole contents of bifrost_global_vars  https://review.openstack.org/222914
16:08 _ody gotcha.  They unified some of the parser code in 4.3 and it caused us (Puppet OpenStack) some issues.  More than I'd expect from a point release.
16:09 kbyrne joined #puppet-openstack
16:12 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ceilometer: Support identity_uri and auth_uri properly and add auth_version option  https://review.openstack.org/182281
16:12 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Change SSL options to [ssl] section.  https://review.openstack.org/266454
16:17 openstackgerrit yuan.lu proposed openstack/puppet-cinder: Fix wrong location for lock_path of params  https://review.openstack.org/266308
16:18 openstackgerrit Merged openstack/puppet-openstack_extras: Add os_service_default fact for tests  https://review.openstack.org/265472
16:25 _ody gchamoul: I think it is a bug in puppet.
16:28 gchamoul _ody: so if it is, it affects puppet 4.3.0 and 4.3.1
16:34 _ody gchamoul: It does indeed.
16:35 _ody https://tickets.puppetlabs.com/browse/PUP-5660
16:36 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Make Horizon DB sync optional  https://review.openstack.org/262011
16:41 gchamoul _ody: thx
16:41 openstackgerrit Merged openstack/puppet-keystone: Use recommended abbreviation of the classes  https://review.openstack.org/266259
16:42 _ody gchamoul: I escalated internally and got word that fixes are coming soon.  Final regression tests will be ready tomorrow.
16:43 openstackgerrit Oleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/266027
16:43 gchamoul _ody: so no way to workaround before the fix, right?
16:43 _ody no
16:46 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect configuration for Murano Service Broker API  https://review.openstack.org/254181
16:47 openstackgerrit Ryan Bak proposed openstack/puppet-monasca: Add support for symlinked agent virtualenv  https://review.openstack.org/265357
16:48 dmsimard EmilienM: I don't think this line is necessary if you add read privileges to all the files in the command right after: https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh#L168
16:48 dmsimard mind if I remove it ?
16:50 EmilienM dmsimard: we need directory access which is given by "x" I think
16:51 dmsimard EmilienM: We can set all folders to 755 and files to 644 I guess
16:51 EmilienM yeah
16:51 dmsimard ok i'll figure something out and submit it - ty
16:58 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Fix bug for deprecated vnc options.  https://review.openstack.org/266443
17:00 dmsimard EmilienM: added you on https://review.openstack.org/266482
17:03 EmilienM dmsimard: ack
17:03 rochaporto joined #puppet-openstack
17:03 rochaporto joined #puppet-openstack
17:09 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: [WIP] scenario002: enable application catalog service  https://review.openstack.org/250436
17:09 openstackgerrit yuan.lu proposed openstack/puppet-ceilometer: Set auth_region to $::os_service_default  https://review.openstack.org/266489
17:37 openstackgerrit yuan.lu proposed openstack/puppet-horizon: Improve format of the documentations.  https://review.openstack.org/266504
17:42 xingchao joined #puppet-openstack
17:44 openstackgerrit yuan.lu proposed openstack/puppet-ceilometer: Set auth_region to $::os_service_default  https://review.openstack.org/266489
17:48 xingchao joined #puppet-openstack
17:48 EmilienM \o/ mitaka jobs are green for scenario001/centos and scenario002/trusty
17:48 EmilienM https://review.openstack.org/#/c/255635/
17:48 EmilienM dmsimard: ^
17:48 xingchao joined #puppet-openstack
17:48 dmsimard EmilienM: wat?
17:49 dmsimard Oh
17:49 dmsimard I thought you said they were all green
17:49 EmilienM dmsimard: can you check if versions are good ? http://logs.openstack.org/35/255635/16/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/cfbb631/logs/rpm-qa.txt.gz
17:50 iurygregory yeah \o/
17:50 EmilienM _degorenko|afk: I confirm chem's patch works for puppet-neutron
17:52 IvanBerezovskiy1 left #puppet-openstack
17:57 EmilienM mwhahaha: fyi https://bugs.launchpad.net/puppet-nova/+bug/1533257
17:57 openstack Launchpad bug 1533257 in puppet-nova "VNC options need to move to [vnc] section" [Medium,New] - Assigned to Liao Penghui (liaoishere)
17:57 EmilienM I think i'm going to work on it
17:57 EmilienM if nobody is already on it, because we have this warning for long time now
17:57 mwhahaha ok
17:58 EmilienM _degorenko|afk: ping https://bugs.launchpad.net/puppet-nova/+bug/1533269 for deprecations
17:58 openstack Launchpad bug 1533269 in puppet-nova "SSL options is deprecating and move to [ssl] section" [Undecided,Confirmed] - Assigned to Liao Penghui (liaoishere)
17:58 iurygregory left #puppet-openstack
17:58 iurygregory joined #puppet-openstack
18:16 angdraug joined #puppet-openstack
18:27 jaosorior joined #puppet-openstack
18:29 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Fix bug for deprecated vnc options.  https://review.openstack.org/266443
18:30 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Change SSL options to [ssl] section.  https://review.openstack.org/266454
18:32 openstackgerrit Liao Penghui proposed openstack/puppet-ceilometer: Move workers options to specific sctions.  https://review.openstack.org/266523
18:37 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
18:37 EmilienM dmsimard: ^ will look results after my run
18:37 dmsimard EmilienM: cool
18:37 EmilienM hopefully sahara is fixed
18:37 dmsimard colonwq: got your email, will review today although I no longer have an environment to test it in
18:39 dmsimard EmilienM: would love your +1 on https://review.openstack.org/#/c/266482/ if you think it makes sense
18:39 colonwq dmsimard, thanks. I just put names together. I could ship you my Dockerfiles. They work well engough to pass the RHEL/CentOS tests.
18:39 openstackgerrit Liao Penghui proposed openstack/puppet-ceilometer: Move workers options to specific sctions.  https://review.openstack.org/266523
18:40 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Change SSL options to [ssl] section.  https://review.openstack.org/266454
18:41 openstackgerrit Liao Penghui proposed openstack/puppet-nova: Fix bug for deprecated vnc options.  https://review.openstack.org/266443
18:42 dmsimard colonwq: sure
19:00 jaosorior joined #puppet-openstack
19:10 guimaluf joined #puppet-openstack
19:14 dmsimard colonwq: reviewed
19:32 jaosorior joined #puppet-openstack
19:41 xarses joined #puppet-openstack
19:43 dmsimard EmilienM: we have success on both scenario001 and scenario002 for mitaka trunk :)
19:54 EmilienM dmsimard: nice
19:54 EmilienM dmsimard: congrats
19:59 EmilienM dmsimard: can we promote ? :)
20:08 mwhahaha are we going to run liberty and mitaka jobs or are we just going to do a complete switch
20:09 colonwq dmsimard, Thanks. I saw them just as I left for lunch. I will work on correcting the issues.
20:09 EmilienM mwhahaha: complete I hope
20:09 EmilienM mwhahaha: we're close.
20:10 EmilienM only aodh is failing now on trusty
20:10 mwhahaha personally i'd like to have both
20:10 mwhahaha but ok
20:10 dmsimard EmilienM: I'd love to have tripleo coverage on it first
20:10 EmilienM mwhahaha: me too!
20:10 EmilienM dmsimard: otherwise, I'll keep using this delorean URL
20:10 EmilienM until you have a promoted repo that works
20:11 EmilienM if I keep using this URL, it should be stable no?
20:11 EmilienM except if you update deps I guess
20:11 dmsimard EmilienM: well, stable in the sense that it won't change
20:11 dmsimard deps don't move that much
20:11 EmilienM cool it works for me
20:11 dmsimard but we will create a mitaka one soon I believe
20:11 EmilienM as long we run mitaka
20:12 EmilienM so maybe aodh is fixed by today
20:12 EmilienM I'm working on it
20:25 baker joined #puppet-openstack
20:41 mfisch Anyone know why puppet-keystone calls this '/usr/bin/openstack domain show --format shell Default' like 50 times?
20:41 mfisch in a row?
20:41 mfisch EmilienM: %
20:41 mfisch richm: ^
20:41 mfisch ?
20:43 mfisch oh wow its calling once per tenant
20:43 mfisch what a waste
20:43 mfisch and once per user
20:44 mdorman yeah.
20:56 richm mfisch: kilo?  liberty?
20:56 mfisch kilo
20:57 mfisch kilo openstack we're on an older version of master puppet
20:57 mfisch I wonder why it needs a domain show per tenant
20:57 mfisch let me get a log
21:00 mfisch so much domain calls: https://gist.github.com/matthewfischer/00b382e0b42b2ca4e4f8
21:02 EmilienM it would be great to know which commit you are in puppet-keystone
21:03 mfisch commit 849d18bc7dc73fb0aaca1f7baa0b25d95495ee56
21:03 mfisch Author: Alex Schultz <aschultz@mirantis.com>
21:03 mfisch Date:   Wed Dec 30 11:30:56 2015 -0700
21:03 mwhahaha wasn't me :o
21:03 mfisch is this new behavior?
21:03 mfisch I just noticed but doesnt mean its new
21:05 mwhahaha i doubt it's new
21:06 mwhahaha would seem like a lack of caching or some scoping related thing when it goes to populate all of the resources defined
21:06 rahulait joined #puppet-openstack
21:06 mfisch im curious why it needs to run once per project and once per user
21:07 mfisch in staging we had like 20 test domains and that was way way way worse
21:07 mfisch I had to clean them up
21:07 mwhahaha how many users do you have listed in your catalog?
21:07 mfisch it was spending 1/2 the puppet run in that code
21:07 mfisch mwhahaha: looking
21:08 mfisch 100ish
21:08 mfisch 80 projects
21:09 mwhahaha i want to say i ran accross this if you don't define the domain in the name or something
21:10 mfisch in the name of what?
21:10 mfisch we dont do anything with domains in puppet
21:10 mwhahaha to populate the resource
21:10 mwhahaha sec
21:10 mwhahaha https://github.com/openstack/puppet-keystone/blob/stable/kilo/lib/puppet/provider/keystone.rb#L53-L66
21:12 mfisch whats the relationship between this and projects/users?
21:13 mwhahaha wait are you on the kilo module or master?
21:14 mwhahaha nm just scrolled up
21:14 mfisch master
21:14 mwhahaha so i think it is related to the v3 stuff
21:14 mfisch well I agree
21:14 mfisch I just want to know what its doing
21:14 mwhahaha i ran into something similar
21:14 mfisch domain show doesn't take a tenant for context does it?
21:14 mwhahaha there's also https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L116-L120
21:15 mfisch if domain show doesnt pass in a tenant or user there's no point to call it 500 times
21:15 mwhahaha I'm not sure it's related to what I saw (because i thought it was fixed) but what I ran into was it was trying to populate the default domains tuff
21:15 mwhahaha let me see if i can find the change I had related to my problem
21:16 mfisch it all works, I should be clear on that
21:16 mfisch just seems unncessary
21:18 mwhahaha wonder if it was https://bugs.launchpad.net/puppet-keystone/+bug/1517187
21:18 openstack Launchpad bug 1517187 in puppet-keystone "Default domain not calculated when no keystone_user/keystone_tenant/keystone_user_role is defined." [Critical,Fix committed] - Assigned to Sofer Athlan-Guyot (sofer-athlan-guyot)
21:18 mwhahaha so if you're not providing the default domain it goes and pulls it or something
21:18 mwhahaha or something to that effect
21:19 mwhahaha where it's something that was introduced with the v2/v3 support magic in the providers
21:20 mfisch so its mad that my keystone resources have no domain specified?
21:20 itsuugo joined #puppet-openstack
21:21 mwhahaha it might be, the provider magic that we do around these is so hard to follow :/
21:21 ntpttr joined #puppet-openstack
21:21 mwhahaha https://review.openstack.org/#/c/244900/ or it might be related to that
21:21 mwhahaha that was my problem
21:21 mwhahaha forgot it was in neutron and it tries to pull keystone tenant stuff
21:31 mwhahaha mfisch: i bet it's from https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L215-L229 which always gets called
21:36 dgurtner joined #puppet-openstack
21:36 dgurtner joined #puppet-openstack
21:38 cwolferh joined #puppet-openstack
21:43 mfisch looking
21:44 mfisch mwhahaha: under what context is that used, for iterating projects/users?
21:45 mwhahaha i have no idea i can't find a reference to it
21:45 mfisch me either
21:46 mwhahaha but those are the only two places where domain show is done as far as i can tell
21:48 mfisch that code has to be unused
21:48 mwhahaha yea i think it's coming from the domain_name_from_id
21:48 mwhahaha which is part of keystone_tenant, keystone_user and keystone_user_role
21:48 mwhahaha also your gist has a domain of 'Default' in it
21:48 mfisch yep
21:50 mfisch but domain_name_from_id should cache
21:50 mfisch it tries to anyway
21:50 mwhahaha https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_tenant/openstack.rb#L72-L83
21:51 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: Remove unused provider code  https://review.openstack.org/266585
21:51 mfisch lets see if its really used ^
21:51 mwhahaha it should but i wonder are the best practices for doing caching on such things
21:52 mfisch I dont know enough ruby but is that hash not getting saved?
21:53 mfisch @foo means basically static method var right?
21:53 mfisch (I should not use the Cism)
21:53 mwhahaha local scope var i think
21:54 mfisch does it stick around once filled?
22:02 _ody @ is an instance variable so scoped to the instance of the class the method belongs to
22:08 mwhahaha so basically it has to make the call for every instance of keystone_tenant, keystone_user and keystone_user_role right?
22:10 TheBallPI joined #puppet-openstack
22:10 TheBallPI What's the best way to install the ceph-puppet moduel? https://github.com/openstack/puppet-ceph
22:12 _ody mwhahaha: With something that is self.method it should only be once, when the type/provider combo is first used.
22:12 _ody methods without self are contained to the instance of the resource.
22:13 ntpttr joined #puppet-openstack
22:14 mwhahaha so is it because in keystone_tenant it is not called via self, it's not getting attached to the type/provider combo but instead contained to the instance of the resource?
22:14 mwhahaha but that wouldn't work for https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_tenant/openstack.rb#L72-L83 because we could call it with multiple domains
22:15 mwhahaha so it seems as those the caching in keystone.rb need to be improved and the usage switched to self
22:15 mwhahaha to allow for caching multiple domains
22:16 mwhahaha oh nevermind it should be ok so we just need to update it to use self.domain_name_from_id
22:16 mwhahaha because it's using the ID for @domain_hash
22:19 mwhahaha mfisch: you should try updating the usages of domain_name_from_id to self.domain_name_from_id and see if it fixes it :D
22:25 mfisch I put some prints in the code
22:25 mfisch none of this block gets hit at all
22:25 mfisch https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L106-L109
22:25 mfisch so I dont know where the shows come from
22:43 baker_ joined #puppet-openstack
22:51 mfisch mwhahaha: fetch_domain is what's being called over and over
22:52 mfisch and the caller of that is self.set_domain_for_name
22:52 mfisch which we couldn't figure out where it was called...
22:52 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Override Service provider for mon and rgw services  https://review.openstack.org/249267
22:56 cwolferh joined #puppet-openstack
22:59 mwhahaha hmmm
23:03 mwhahaha so it looks like that needs some caching or something
23:03 ntpttr joined #puppet-openstack
23:05 openstackgerrit yuan.lu proposed openstack/puppet-ceilometer: Set auth_region to $::os_service_default  https://review.openstack.org/266489
23:06 mfisch mwhahaha: I found it
23:06 mfisch mwhahaha: set_domain_for_name is called by the user and project providers and does NO caching
23:06 mfisch mwhahaha: its not the id code
23:06 mfisch I will see if I can fix
23:06 mwhahaha well we can cache fetch_domain
23:06 mwhahaha that's the fix
23:07 mwhahaha we should only call fetch_domain once i would assume, the issue i guess is when we make changes to the domain but if we're only fetching the ID does is there anything that might cause problem?
23:08 mwhahaha but if the only thing we're looking for is domain -> ID which you can't change, caching shouldn't be a problem
23:08 mfisch what is set_domain_for_name returning
23:08 mfisch the lack of return foo is annoying me in ruby
23:09 dfisher the implicit "return the last thing you touched" thing?
23:09 dfisher because I *hate* that about Ruby.
23:09 mfisch yeah its just not as readable as it could be
23:10 mwhahaha @domain_hash[id] is what it's returning
23:10 mfisch we just need a reverse of the other hash
23:10 mfisch domain[name] = id
23:11 mwhahaha for fetch domain we should just do something similar to the @domain_hash elsewhere
23:11 mfisch Im debating if it makes sense in fetch_domain or in the domain_for_name code
23:12 mwhahaha the question i would have is what data is returned from fetch_domain
23:12 mwhahaha is it just a name and an id?
23:13 mwhahaha if so fetch domain should have a @domain_hash like in domain_name_from_id
23:13 mwhahaha where we key of a domain name
23:13 mfisch domain show has desc, enabled, id, and name
23:13 mfisch I think we only use id and name
23:13 mwhahaha so the issue comes in when domains are updated
23:13 mfisch I think thats all we need
23:13 mfisch domains should not be given a new ID
23:13 mfisch id <-> name shouldnt change
23:14 mwhahaha right
23:14 mwhahaha but enabled might
23:14 mwhahaha or name might
23:14 mfisch a name might be new or deleted
23:14 mwhahaha which fetch_domain also pulled
23:14 mfisch if domain[name] is not found, we run the show
23:14 mfisch that handles new domains, that handles renames
23:14 mfisch deletes fail regardless of cache
23:14 mfisch fail to lookup I mean
23:14 mfisch we're not using enabled are we? I didnt see it
23:15 mwhahaha not sure
23:15 mfisch let me hack up some code and try ity
23:15 mwhahaha that would be the only dangerous thing
23:16 mwhahaha description updates might have issues too
23:16 mwhahaha just pointing out where we need to be careful :)
23:19 mfisch unless we need those fields this new hash is just an exact opposite of the other hash
23:20 mwhahaha we could also create a new function to just fetch domain id
23:20 mwhahaha and only cache that
23:25 mfisch what I have so far
23:25 mfisch https://gist.github.com/matthewfischer/9a8a9b32be444a32f0a3
23:25 mwhahaha so shouldn't it be self.domain_id_from_name
23:26 mwhahaha based on what _ody said previously?
23:26 mfisch thats what I have?
23:26 mwhahaha https://gist.github.com/matthewfischer/9a8a9b32be444a32f0a3#file-domain-diff-L34
23:26 mwhahaha on line 34
23:26 mfisch ah with a self
23:27 mfisch I wonder if they can share var names
23:29 mwhahaha probably not
23:29 mwhahaha unless it's doing the exact same thing
23:29 mwhahaha then i couldn't see why not
23:29 mfisch yeah agreed
23:30 mwhahaha well it's domain by name by id and domain id by name right?
23:30 mwhahaha they should be able to share
23:30 mfisch the hashes are backwards of each other keys and values switched
23:31 mfisch Error: Could not prefetch keystone_tenant provider 'openstack': can't convert Symbol into Integer
23:31 mfisch have a typo somewhere
23:31 mwhahaha There's probably an optimization to be had, i'd use two vars for now to prove it out :)
23:32 mfisch agree
23:32 mfisch I want to see it work first then I can improve it
23:38 ntpttr joined #puppet-openstack
23:42 logan2 joined #puppet-openstack
23:49 mdorman mfisch:   any particular reason you do the conversion to integer here?  https://github.secureserver.net/cloudplatform/openstack-puppet-nova/blob/master/lib/puppet/provider/nova.rb#L201   with nova cells, the aggregate id’s look something like this:  api!cell_01@22   so for my needs I need it treated like a string.

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