Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:38 gildub joined #puppet-openstack
00:58 zhangjn joined #puppet-openstack
01:10 markvoelker joined #puppet-openstack
01:25 baker joined #puppet-openstack
01:39 baker joined #puppet-openstack
01:42 baker joined #puppet-openstack
01:45 baker joined #puppet-openstack
02:13 zhangjn joined #puppet-openstack
03:11 markvoelker joined #puppet-openstack
04:35 zhangjn joined #puppet-openstack
04:37 mgagne joined #puppet-openstack
04:38 mentat joined #puppet-openstack
05:12 markvoelker joined #puppet-openstack
05:21 zhangjn joined #puppet-openstack
06:01 zhangjn joined #puppet-openstack
06:41 zhangjn joined #puppet-openstack
06:57 zhangjn joined #puppet-openstack
06:58 zhangjn joined #puppet-openstack
07:01 rcernin joined #puppet-openstack
07:04 zhangjn joined #puppet-openstack
07:05 mjb joined #puppet-openstack
07:06 openstackstatus joined #puppet-openstack
07:10 openstackgerrit joined #puppet-openstack
07:13 markvoelker joined #puppet-openstack
07:20 openstackgerrit Andrew Spiers proposed openstack/puppet-designate: Use actual service names for RH osfamily  https://review.openstack.org/257175
07:26 rmart04 joined #puppet-openstack
07:47 openstackgerrit Andrew Spiers proposed openstack/puppet-designate: Use actual service names for RH osfamily  https://review.openstack.org/257175
08:31 jpena joined #puppet-openstack
08:35 paramite joined #puppet-openstack
08:49 openstackgerrit Mykyta Karpin proposed openstack/puppet-tempest: CI test - do not merge  https://review.openstack.org/257221
08:53 gfidente joined #puppet-openstack
08:53 gfidente joined #puppet-openstack
08:55 openstackgerrit Michael Chapman proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/249390
08:55 samuelBartel joined #puppet-openstack
09:03 zhangjn joined #puppet-openstack
09:07 darkxploit joined #puppet-openstack
09:12 jistr joined #puppet-openstack
09:14 markvoelker joined #puppet-openstack
09:20 openstackgerrit Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router  https://review.openstack.org/255913
09:25 rochaporto joined #puppet-openstack
09:26 leanderthal joined #puppet-openstack
09:30 admin0 joined #puppet-openstack
09:30 morazi joined #puppet-openstack
09:41 rahulait joined #puppet-openstack
09:47 rmart04_ joined #puppet-openstack
10:01 zhangjn joined #puppet-openstack
10:01 leanderthal joined #puppet-openstack
10:03 zhangjn joined #puppet-openstack
10:06 jtomasek joined #puppet-openstack
10:08 openstackgerrit Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router  https://review.openstack.org/255913
10:10 davideagle__ joined #puppet-openstack
10:16 paramite joined #puppet-openstack
10:41 mgrosz joined #puppet-openstack
10:42 mgrosz Hello everyone, can some one please review and +2 and +workflow change https://review.openstack.org/#/c/255431/? Thanks
10:43 rleander joined #puppet-openstack
10:47 omartsyniuk joined #puppet-openstack
10:57 leanderthal joined #puppet-openstack
11:01 devvesa joined #puppet-openstack
11:01 IvanBerezovskiy joined #puppet-openstack
11:04 rleander joined #puppet-openstack
11:13 zhangjn joined #puppet-openstack
11:14 markvoelker joined #puppet-openstack
11:19 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Ability to configure api, collector and notification workers  https://review.openstack.org/226011
11:21 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes.  https://review.openstack.org/211593
11:25 devvesa Hi, anyone knows why https://review.openstack.org/#/c/251319/
11:25 devvesa some reviews are failing on puppet tests?
11:26 devvesa It is something introduced since December 12th
11:28 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ceilometer: Remove POSIX users and groups  https://review.openstack.org/195135
11:30 mgrosz joined #puppet-openstack
11:47 derekh joined #puppet-openstack
11:53 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/257294
11:54 slagle joined #puppet-openstack
12:01 mgrosz joined #puppet-openstack
12:06 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/251319
12:07 admin0 joined #puppet-openstack
12:09 dgurtner joined #puppet-openstack
12:09 dgurtner joined #puppet-openstack
12:22 EmilienM good morning
12:24 devvesa Hi EmilienM: Good morning
12:24 devvesa I noticed that suddenly my gate-puppet jobs failed
12:25 devvesa Look this one since Dec 13th: https://review.openstack.org/#/c/251319/
12:25 EmilienM I'm looking
12:25 devvesa This is another one in puppet-midonet
12:25 devvesa sorry
12:25 devvesa puppet-neutron
12:25 EmilienM this one is a bad example, it's not a puppet openstack module but tripleo, they way it's tested is different
12:25 devvesa https://review.openstack.org/#/c/255913/
12:26 EmilienM but there is something with psych, indeed
12:26 devvesa Yes. I solved in puppet-tripleo with this patch
12:26 devvesa https://review.openstack.org/#/c/257294/
12:27 devvesa I don't know the reason why they started to fail
12:27 devvesa Another example: https://review.openstack.org/#/c/257175/
12:28 EmilienM sbadia: I think we also need to add the dependency in our modules
12:28 EmilienM sbadia: can you double check too? I think it's ok
12:28 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: CI test - never merge  https://review.openstack.org/257312
12:33 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Adding psych on Gemfile explicitly  https://review.openstack.org/257317
12:33 EmilienM devvesa: doing some testing in puppet openstack modules
12:34 devvesa EmilienM: your test one already has failed
12:34 devvesa https://jenkins05.openstack.org/job/gate-puppet-​tripleo-puppet-unit-3.3-dsvm-centos7/26/console
12:34 devvesa sorry, bad link
12:35 devvesa https://jenkins06.openstack.org/job/gate-puppet-​neutron-puppet-syntax-3-dsvm-centos7/99/console
12:36 EmilienM devvesa: taking a coffee and I'm back ;-)
12:37 * devvesa lunch time
12:37 devvesa :)
12:38 dprince joined #puppet-openstack
12:40 ddmitriev joined #puppet-openstack
12:44 ducttape_ joined #puppet-openstack
12:45 zhangjn joined #puppet-openstack
12:45 markvoelker joined #puppet-openstack
12:49 mgrosz @EmilienM: Good Morning!
12:51 sbadia hi here
12:51 sbadia EmilienM: ok, I take a look
13:09 sbadia EmilienM: hum, only two gems are changed between the two builds (https://review.openstack.org/#/c/257175/ (fail) and https://review.openstack.org/#/c/255393/ (ok))
13:09 sbadia EmilienM: specinfra 2.44.7 → specinfra 2.45.0 and bundler 1.10.6 → bundler 1.11.0
13:10 sbadia (maybe rubygem itself but I'm not sure, this message isn't present on the « ok » test http://logs.openstack.org/75/257175/2/check/gat​e-puppet-designate-puppet-syntax-3-dsvm-centos7​/f0e38b7/console.html#_2015-12-14_07_47_52_179 )
13:11 devvesa sbadia: I would say it is the bundle version
13:11 devvesa *bundler
13:11 devvesa https://github.com/bundler/bundler/issues/2068
13:11 sbadia indeed :/
13:12 sbadia let's pin bundler version \o/
13:12 sbadia devvesa: you already made the patch?
13:12 devvesa That's the other option :)
13:12 devvesa I made one for puppet-tripleo
13:12 devvesa https://review.openstack.org/#/c/257294/
13:12 sbadia ok!
13:14 EmilienM it seems we have 2 options, yes
13:14 EmilienM I'm in favor of keeping bundle from latest release
13:14 EmilienM and add psych as a new dep
13:15 EmilienM devvesa, sbadia, like https://review.openstack.org/#/c/257317/ (look in the code my commend)
13:15 EmilienM comment*
13:15 sbadia EmilienM: hum…
13:16 sbadia this is a bundler bug, let's patch it in project-config (top wide)
13:16 sbadia in order to not add this in all our projects…
13:19 devvesa sbadia: In that case, someone could experience the bug locally and once you know it, you would push patches blindly...
13:19 IvanBerezovskiy joined #puppet-openstack
13:19 sbadia yep yep sure!
13:21 EmilienM sbadia: if you patch the fix in project-config (where by the way?), people who run tests locally would fail
13:22 EmilienM I think we have to fix all Gemfiles
13:23 sbadia or fix bundler :D
13:23 sbadia https://github.com/bundler/bundler/issues/4149
13:24 EmilienM sbadia: it's fixed but not released iiuc
13:24 devvesa According to the issue, it will be released on bundle 1.12
13:25 EmilienM and we don't know when that happens
13:26 devvesa I have to wait some reviews on tripleo patches, if you have a spreadsheet or something with the list of puppet modules where to apply
13:26 devvesa I can help you to push patches in all of them
13:29 mgrosz @EmilienM: hey. Any chances my puppet-vitrage will be merged today?
13:31 sbadia devvesa: oh thanks! we can maybe use modulesync for that no? https://github.com/openstack/puppet-modules​ync-configs/blob/master/config_defaults.yml
13:32 sbadia (for the list https://github.com/openstack/puppet-modulesy​nc-configs/blob/master/managed_modules.yml)
13:33 devvesa sbadia: Oh, I didn't know this project! How does it work?
13:33 sbadia it's magic :-)
13:33 sbadia sorry, no it use the tool « modulesync »
13:34 sbadia https://github.com/puppet-community/modulesync
13:34 devvesa Being magic is good for good stuff but horrible for bad ones :)
13:34 sbadia ;-)
13:35 igajsin joined #puppet-openstack
13:36 igajsin left #puppet-openstack
13:41 zhangjn joined #puppet-openstack
13:43 openstackgerrit Sebastien Badia proposed openstack/puppet-modulesync-configs: Add new managed modules  https://review.openstack.org/257357
13:45 zhangjn joined #puppet-openstack
13:45 EmilienM sbadia: monasca?
13:46 EmilienM not sure monasca is compliant
13:46 EmilienM the msync will change a lot of things I think
13:47 jaosorior joined #puppet-openstack
13:55 dslevin_ joined #puppet-openstack
13:56 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters  https://review.openstack.org/255869
13:56 devvesa I have a question regarding on puppet-modulesync-configs: Should I push the change on this module and it will be applied automatically to others with a CI job?
13:56 EmilienM devvesa: sbadia is working on it
13:56 devvesa Oh... I finished mine already :)
13:57 zeus joined #puppet-openstack
13:57 devvesa (with the --noop)
13:57 devvesa Ok, all yours sbadia :)
13:58 sbadia devvesa: ok cool then!
13:58 sbadia you already patched all our modules?
13:58 devvesa BTW, EmilienM: Do you have +2 permissions on puppet-tripleo?
13:58 sbadia devvesa: yep (me too)
13:58 EmilienM devvesa: yes
13:58 devvesa Can you merge the puppet-tripleo one, then?
13:59 devvesa https://review.openstack.org/257294
13:59 devvesa sbadia: Yes. I already patches all the modules
13:59 EmilienM devvesa: I would like to see an inline comment
13:59 devvesa EmilienM: ok
13:59 sbadia and a unique topic
13:59 EmilienM devvesa: see https://review.openstack.org/257317
14:00 sbadia devvesa: then do not hesitate to submit your patchs
14:00 sbadia it's no use that we will both manage that, I leave you manage in this case.
14:00 michchap psych issue hit me today
14:01 michchap I thought it was interesting it only affected the versions < 4.0.0
14:02 richm joined #puppet-openstack
14:02 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/257294
14:03 devvesa sbadia: I have not dared to run the process without the --noop. Will I create a patch for each project? Is this what it is expected to happen?
14:04 sbadia devvesa: this case is a bit special (because we need to add a comment inline)
14:05 sbadia we can maybe manage this the old way using a loop on managed module (in order to do that quickly)
14:05 sbadia devvesa: do you want to split the module, or it's ok?
14:05 devvesa Uhm, ok. I think I give it to you. I don't want to mess up with all the modules...
14:05 zhangjn joined #puppet-openstack
14:05 EmilienM sbadia: hopefully Bundle will provide a release soon
14:06 EmilienM because it also break stable/* CI
14:06 sbadia devvesa: ok, ok, let's go then
14:07 jaosorior richm: Hey man, just saw the CR you sent me: https://review.openstack.org/#/c/256248/ I think I could use that for what I need.
14:07 EmilienM sbadia: what do we do for stable branches?
14:08 jaosorior richm: Instead of wiring an exec with sed. Which would work, but somehow I like this better
14:08 sbadia EmilienM: the latest bundler is always fetched even in stable branch
14:08 EmilienM sbadia: yeah, that's what I'm saying: stable/liberty, kilo etc are broken too
14:09 sbadia EmilienM: we can maybe query to bundler team when the release come?
14:09 EmilienM sbadia: what should we do?
14:09 EmilienM sbadia ++
14:15 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/251319
14:22 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters  https://review.openstack.org/255869
14:26 openstackgerrit Merged openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/257294
14:29 sbadia spam ready :)
14:29 * EmilienM ready for review
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-aodh: Adding psych on Gemfile explicitly  https://review.openstack.org/257379
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-barbican: Adding psych on Gemfile explicitly  https://review.openstack.org/257381
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-ceilometer: Adding psych on Gemfile explicitly  https://review.openstack.org/257382
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-ceph: Adding psych on Gemfile explicitly  https://review.openstack.org/257383
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-cinder: Adding psych on Gemfile explicitly  https://review.openstack.org/257384
14:29 openstackgerrit Sebastien Badia proposed openstack/puppet-designate: Adding psych on Gemfile explicitly  https://review.openstack.org/257385
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-glance: Adding psych on Gemfile explicitly  https://review.openstack.org/257386
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-gnocchi: Adding psych on Gemfile explicitly  https://review.openstack.org/257387
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-heat: Adding psych on Gemfile explicitly  https://review.openstack.org/257388
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-horizon: Adding psych on Gemfile explicitly  https://review.openstack.org/257389
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-ironic: Adding psych on Gemfile explicitly  https://review.openstack.org/257390
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-keystone: Adding psych on Gemfile explicitly  https://review.openstack.org/257391
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-magnum: Adding psych on Gemfile explicitly  https://review.openstack.org/257392
14:30 openstackgerrit Sebastien Badia proposed openstack/puppet-manila: Adding psych on Gemfile explicitly  https://review.openstack.org/257393
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-mistral: Adding psych on Gemfile explicitly  https://review.openstack.org/257394
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-monasca: Adding psych on Gemfile explicitly  https://review.openstack.org/257395
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-murano: Adding psych on Gemfile explicitly  https://review.openstack.org/257396
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-nova: Adding psych on Gemfile explicitly  https://review.openstack.org/257397
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-openstacklib: Adding psych on Gemfile explicitly  https://review.openstack.org/257398
14:31 openstackgerrit Sebastien Badia proposed openstack/puppet-sahara: Adding psych on Gemfile explicitly  https://review.openstack.org/257399
14:32 openstackgerrit Sebastien Badia proposed openstack/puppet-swift: Adding psych on Gemfile explicitly  https://review.openstack.org/257400
14:32 openstackgerrit Sebastien Badia proposed openstack/puppet-tempest: Adding psych on Gemfile explicitly  https://review.openstack.org/257401
14:32 openstackgerrit Sebastien Badia proposed openstack/puppet-trove: Adding psych on Gemfile explicitly  https://review.openstack.org/257402
14:32 openstackgerrit Sebastien Badia proposed openstack/puppet-tuskar: Adding psych on Gemfile explicitly  https://review.openstack.org/257404
14:32 openstackgerrit Sebastien Badia proposed openstack/puppet-vswitch: Adding psych on Gemfile explicitly  https://review.openstack.org/257405
14:33 openstackgerrit Sebastien Badia proposed openstack/puppet-zaqar: Adding psych on Gemfile explicitly  https://review.openstack.org/257406
14:34 EmilienM sbadia: monasca is ok
14:34 EmilienM sbadia: oh I actually have another idea
14:35 EmilienM since it's only temporary
14:35 EmilienM and it's only for unit tests: maybe we can only patch https://github.com/openstack/puppet-openstack-i​ntegration/blob/master/install_modules_unit.sh
14:36 EmilienM sbadia: or is it also affecting syntax?
14:36 EmilienM I think we can avoid all these patches
14:36 EmilienM 1/ patch https://github.com/openstack/puppet-openstack-i​ntegration/blob/master/install_modules_unit.sh and install the gem manually
14:36 EmilienM 2/ patch project-config in syntax jobs and add the gem manually too
14:37 EmilienM that way, we only have 2 patches, it easily backportable and people can still run tests locally
14:37 EmilienM devvesa, sbadia: wdyt?
14:37 sbadia https://review.openstack.org/#/q/topic:bug/1525929,n,z
14:37 sbadia (sorry scrum meeting right now)
14:38 EmilienM sbadia: I think we can avoid that in fact
14:38 sbadia EmilienM: it's also syntax indeed
14:38 EmilienM yeah but we can fix project-config for that one
14:38 EmilienM (syntax)
14:38 EmilienM I'm going to test it
14:39 sbadia and for the local tests (devlopper laptop)
14:40 samuelBartel joined #puppet-openstack
14:41 dgurtner joined #puppet-openstack
14:41 dgurtner joined #puppet-openstack
14:43 markvoelker joined #puppet-openstack
14:44 iurygregory hello people o/
14:45 EmilienM sbadia: do you think people run syntax tests?
14:45 sbadia EmilienM: yes :)
14:46 devvesa Sorry, I don't get it. Why people will be able to run locally?
14:46 devvesa is puppet-openstack-integration a common dependency?
14:46 EmilienM yes
14:46 devvesa Ok, +1 then :)
14:46 EmilienM see Rakefile
14:47 EmilienM having 26 patches each time is really heavy
14:47 EmilienM I would avoid it when possible
14:47 iurygregory i run syntax tests before send =D
14:48 EmilienM we need to poke Bundler folks to provide a release
14:48 devvesa Agree. But as far as I understood, the problem is not the pysch is not installed
14:48 sbadia … let's abandon all this patchs then…
14:48 EmilienM sbadia: no!
14:48 EmilienM sbadia: just 'pause'
14:48 devvesa But is not installed by bundle
14:48 EmilienM sbadia: we might need your patches
14:49 EmilienM I'm just looking for a simple wway
14:49 EmilienM because we will have this issue again for another dep one day
14:49 EmilienM and we need a scalable solution
14:49 EmilienM patching 26 modules (+ backports so ~100 patches) is not scalable
14:49 sbadia I didn't remember the git clone of integration repo on local laptops
14:49 EmilienM of course it does
14:49 EmilienM sbadia: read Rakefile ;-)
14:49 sbadia but it's the best way to manage that…
14:50 sbadia so patching all modules is not needed
14:50 EmilienM you +2'ed the patches
14:50 sbadia we juste need to patch integration repo for local and project-config for openstack infra
14:50 sbadia and that it
14:50 EmilienM sbadia: yeah but people who run syntax on their laptop will be broken
14:50 EmilienM but that's not related to our modules
14:51 EmilienM they can install the gem manually I guess?
14:51 sbadia yes maybe
14:52 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters  https://review.openstack.org/255869
14:52 zhangjn joined #puppet-openstack
14:55 ducttape_ joined #puppet-openstack
14:56 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/257420
14:57 EmilienM sbadia: do not abandon your patches yet
14:57 devvesa Since puppet-tripleo is outside the puppet-openstack modules, I've backported to stable/liberty the master's patch ^
15:00 richm jaosorior: yeah, just trying to figure out some alternatives for you to use
15:01 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Adding psych on Gemfile explicitly  https://review.openstack.org/257421
15:02 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Test https://review.openstack.org/257421  https://review.openstack.org/257422
15:02 EmilienM sbadia: ^ if the test pass, I think we can go ahead with the change, but let me know what do you think
15:07 zhangjn joined #puppet-openstack
15:09 mwhahaha ugh looks like it broke the fuel syntax checks as well
15:09 bklei joined #puppet-openstack
15:11 EmilienM mwhahaha: mouahahahahah
15:11 mwhahaha Happy Monday!
15:11 EmilienM (I had too)
15:17 openstackgerrit Michael Polenchuk proposed openstack/puppet-keystone: Make request by service token if got unauth error  https://review.openstack.org/251490
15:20 EmilienM 166 jobs in the gate, we have time to wait :)
15:21 dfisher joined #puppet-openstack
15:22 mwhahaha hmm i don't get the psych error locally
15:23 EmilienM maybe because you already have the gem
15:23 EmilienM mwhahaha: gem list | grep psych?
15:26 mwhahaha ah yes because it's a default gem on my system
15:26 mwhahaha blah
15:28 jaosorior richm: Thanks dude
15:30 baker joined #puppet-openstack
15:33 migi joined #puppet-openstack
15:42 xarses joined #puppet-openstack
15:43 admin0 joined #puppet-openstack
15:47 mdorman joined #puppet-openstack
15:48 baker joined #puppet-openstack
16:02 dhollinger joined #puppet-openstack
16:13 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters  https://review.openstack.org/255869
16:21 davideagle__ joined #puppet-openstack
16:24 rahulait joined #puppet-openstack
16:25 dh64 joined #puppet-openstack
16:27 openstackgerrit Jay_Clark proposed openstack/puppet-zaqar: continuing work on the initial pass of the zaqar module.  https://review.openstack.org/229036
16:27 openstackgerrit Jay_Clark proposed openstack/puppet-zaqar: Adding manifest work for Debian installs.  https://review.openstack.org/257469
16:43 gfidente joined #puppet-openstack
16:43 gfidente joined #puppet-openstack
16:49 pleia2 joined #puppet-openstack
16:51 rahulait1 joined #puppet-openstack
16:52 cwolferh joined #puppet-openstack
17:03 yottatsa joined #puppet-openstack
17:04 EmilienM sbadia: my fix does not work
17:04 EmilienM http://logs.openstack.org/22/257422/1/check/ga​te-puppet-neutron-puppet-unit-3.3-dsvm-centos7​/d6bfdb4/console.html#_2015-12-14_16_35_34_101
17:04 EmilienM see http://logs.openstack.org/22/257422/1/check/ga​te-puppet-neutron-puppet-unit-3.3-dsvm-centos7​/d6bfdb4/console.html#_2015-12-14_16_35_33_436
17:12 xarses EmilienM: https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20151215 appears to be empty
17:13 EmilienM xarses: yeah I need to feed it
17:13 EmilienM doing it right now
17:13 xarses ok
17:18 jtomasek joined #puppet-openstack
17:23 openstackstatus joined #puppet-openstack
17:38 yottatsa joined #puppet-openstack
17:42 openstackgerrit Andrew Woodward proposed openstack/puppet-ceph: Move intergration tests out of acceptance  https://review.openstack.org/257511
17:44 EmilienM I think we need to pin bundler
17:45 xarses related to https://launchpad.net/bugs/1525929 ?
17:45 openstack Launchpad bug 1525929 in puppet-vswitch "Unit-tests fails due to an update of bundler (Psych is double-loaded unless it's in the Gemfile)" [Undecided,In progress] - Assigned to Sebastien Badia (sbadia)
17:45 EmilienM yes
17:45 EmilienM except if they do a release
17:46 EmilienM https://github.com/bundler/bundler/​issues/4149#issuecomment-164507268
17:47 xarses (off-topic) there was also a tripple-o thread of similar note (about puppetlabs-mysql) they where asking about pinning third-party modules to a release also. I'm not sure if you caught it, it didn't have [puppet] in the topic
17:47 EmilienM I did, I'll look at it
17:47 EmilienM thanks
17:58 dgurtner joined #puppet-openstack
17:58 dgurtner joined #puppet-openstack
18:04 openstackgerrit Ben Kero proposed openstack/puppet-trove: Typo in README.md  https://review.openstack.org/257515
18:11 bkero >_> drive-by review
18:16 EmilienM the CI is still broken, unit tests
18:16 EmilienM I'm on it today, if anyone wants to help
18:16 mgrosz joined #puppet-openstack
18:18 bkero Is CI not running, or are jobs just failing?
18:19 yottatsa joined #puppet-openstack
18:23 xarses bkero: CI appears to be backlogged
18:23 xarses http://status.openstack.org/zuul/
18:23 xarses you can search for your job there
18:24 aimon joined #puppet-openstack
18:25 bkero ok
18:30 openstackgerrit Dan Prince proposed openstack/puppet-nova: Drop use of deprecated neutron auth options  https://review.openstack.org/257522
18:32 openstackgerrit Dan Prince proposed openstack/puppet-nova: Drop use of deprecated neutron auth options  https://review.openstack.org/257522
18:46 IvanBerezovskiy left #puppet-openstack
18:48 dslevin_ joined #puppet-openstack
18:51 EmilienM crinkle, _ody: I would like your insights on 1525929
18:52 EmilienM https://launchpad.net/bugs/1525929
18:52 openstack Launchpad bug 1525929 in puppet-vswitch "Unit-tests fails due to an update of bundler (Psych is double-loaded unless it's in the Gemfile)" [Undecided,In progress] - Assigned to Sebastien Badia (sbadia)
18:52 EmilienM we have several options:
18:52 EmilienM - patch all modules, like sbadia did: https://review.openstack.org/#/q/topic:bug/1525929,n,z
18:53 EmilienM - try to fix it only in puppet-openstack-integration, like I'm doing (and is failing): https://review.openstack.org/257421
18:53 EmilienM - pin bundler in project-config
18:54 EmilienM nibalizer ^
18:56 EmilienM a very quick workaround would be to pin bundler
18:56 crinkle EmilienM: the unit tests run with `bundle exec` to installing gems with `gem install` won't fix things
18:56 crinkle i think pinning bundler or doing what sbadia did are both fine
18:56 EmilienM crinkle: I figured :(
18:57 EmilienM crinkle: I'm in favor of pin because what sbadia did does not scale for 26 modules (+ stable branches so 100 patches)
18:57 xarses EmilienM: if pining bundler works, I think fixing it in project-config seems good
18:57 EmilienM it's a temporary fix, so if we could avoid 100 patches
18:57 EmilienM cool I just want to make sure our group like it
18:57 xarses 200, eventually we will back it out right?
18:57 EmilienM xarses: sigh.
19:00 yottatsa joined #puppet-openstack
19:07 yottatsa joined #puppet-openstack
19:10 mdorman all the keystone_* providers are supposed to work under keystone v2 api as well as v3, right?   or are they all coded now such that they only use v3?
19:11 chem mdorman: right
19:11 chem mdorman: should work with both
19:14 mdorman ok.  found a bug where keystone_user is calling ‘openstack user list’ w/o a —domain option under v3 api.  so i’m trying to figure out how to patch this such that it still works for v2 as well.
19:14 mdorman https://github.com/openstack/puppet-​keystone/blob/master/lib/puppet/prov​ider/keystone_user/openstack.rb#L10  does this line imply that keystone_user always uses v3?
19:15 chem mdorman: not that simple :)
19:15 mdorman i figured as much
19:15 chem mdorman: did you do a bug report about what you've found ?
19:16 mdorman i’m about to.  but wanted to have a fix mostly working on my end first so i can make sure i’m describing it correctly
19:16 mdorman is there some way i can programatically determine what api version it’s using?
19:17 chem mdorman: most of the time it's going to be /v3, but in some cases '/2.0' endpoint would be used for password checking in kesytone_user
19:18 mdorman ok.   well what i need to do is figure out if it’s 2 or 3 here:  https://github.com/openstack/puppet-​keystone/blob/master/lib/puppet/prov​ider/keystone_user/openstack.rb#L150
19:18 mdorman under v2, it works as is.  but for v3 we need to do a domain list first and then a user list for each domain individually
19:18 mdorman i have it working here for v3, but i know it’ll break for v2 (b/c there is no such ‘openstack domain list’ for v2)
19:21 chem going to lend on '/v3'
19:21 chem mdorman: but before you go further, I think you got it wrong
19:22 mdorman ?
19:22 mdorman i’m writing up the bug now
19:22 chem mdorman: or maybe you have a use case ...
19:23 mdorman not sure i follow
19:24 yottatsa joined #puppet-openstack
19:24 mdorman ‘openstack user list’ on the command line using the v3 api fails
19:25 mdorman https://bugs.launchpad.net/p​uppet-keystone/+bug/1526031
19:25 openstack Launchpad bug 1526031 in puppet-keystone "keystone_user provider missing --domain parameter on user list call under v3 API" [Undecided,New]
19:25 chem mdorman: I'm looking
19:26 EmilienM our CI should be back very soon
19:26 EmilienM once https://review.openstack.org/#/c/257465/ is merged
19:28 chem mdorman: I think this cannot be the case, all the ci would be broken if it was true, as listing user is the basis to not recreate them.
19:28 chem mdorman: there must be something more
19:28 mdorman i think the problem is in the test:  https://github.com/openstack/puppet-key​stone/blob/master/spec/unit/provider/ke​ystone_user/openstack_spec.rb#L82-L87
19:28 mdorman this improperly assumes that ‘openstack user list’ will return a proper user listing…  when in fact it fails under v3.
19:29 mdorman (if i am reading that test right)
19:30 chem mdorman: you are but it certainly sure this work, for instance, this works https://github.com/openstack/puppet-​keystone/blob/master/spec/acceptance​/keystone_wsgi_apache_spec.rb#L123
19:35 mdorman chem:   i wonder if ‘openstack user list’ only fails w/o —domain if keystone actually has more than the default domain configured
19:35 mdorman let me test that theory
19:36 chem mdorman: in the ci there are multiple domains configured
19:36 chem mdorman: those 2: https://github.com/openstack/puppet-k​eystone/blob/master/spec/acceptance/k​eystone_wsgi_apache_spec.rb#L25-L34 and the Default one
19:39 mdorman fair enough
19:41 mdorman ok, the problem is when domain_specific_drivers_enabled is turned on.  when that’s enabled, —domain is needed for user list call.  when it’s disabled, user list works fine by defaulting to the default domain.
19:42 mdorman not sure why it’s not obeying the default domain in that base
19:42 mdorman *case
19:43 chem user list will show the user of all the domain if you use the admin user
19:44 chem mdorman: ^ I think
19:46 mdorman https://bugs.launchpad.net/keystone/+bug/1479578
19:46 openstack Launchpad bug 1479578 in OpenStack Identity (keystone) "Domain-specific config breaks some ops" [Undecided,New]
19:46 mdorman it doesn’t work if you have domain-specific config turned on in keystone
19:46 chem mdorman: when you activate domain_specific_drivers_enabled you have to have your backend working for any request to work
19:47 yottatsa joined #puppet-openstack
19:48 mdorman they keystone backend?  yes, i know.  and it is.
19:48 mdorman it’s a keystone bug.
19:48 chem mdorman: sorry, it's late :)
19:49 imcsk8 joined #puppet-openstack
19:49 mdorman np
19:50 mdorman as usual, this was quite a rabbit hole
19:52 chem mdorman: deep down.  Could you update the bug report, we have a patch that should manage domain_specific driver, there https://review.openstack.org/#/c/219289/24, it's not fully tested in the CI as we don't setup the proper ldap backend (for instance), so we just check that we create the right files.  In the meantime the openstack cli used in the bug report above is old If my memory is ok.  So before the patch get commited I would like
19:52 chem to check if that problem still exists.  Thanks.
19:53 chem mdorman: or when, what and try to have it fixed if necessary
19:54 chem mdorman: this is the end fo my day so please put any finding in the bug report and I will get back to you through it
19:54 dprince joined #puppet-openstack
19:55 mdorman chem:  yup i updated the bug report referencing the keystone bug
19:55 chem mdorman: thank
19:55 chem s
19:59 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/219289
20:00 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend.  https://review.openstack.org/238164
20:02 rahulait joined #puppet-openstack
20:06 EmilienM _ody: I think https://review.openstack.org/#/c/256644/ is safe to merge also
20:10 yottatsa joined #puppet-openstack
20:12 vinsh joined #puppet-openstack
20:17 openstackgerrit Sergey Kolekonov proposed openstack/puppet-tempest: [DO NOT MERGE] Test commit  https://review.openstack.org/257556
20:27 mdorman joined #puppet-openstack
20:33 _ody EmilienM: We talked about one failure with the pick() function last week.  What was the actual result of that?
20:33 EmilienM _ody: I did not investigate yet
20:33 _ody IMO it didn't look related to the module updates.
20:33 EmilienM but puppet-openstack_extras has unit tests broken
20:33 EmilienM nope it did not
20:33 _ody Ah.
20:34 _ody So do think it was already broken?
20:38 ducttape_ joined #puppet-openstack
20:42 _ody EmilienM: Can you please update https://review.openstack.org/#/c/256644/ and comment that you tested the patch and the results?
20:43 EmilienM _ody: done
20:43 EmilienM _ody: yes it was already broken
20:44 EmilienM _ody: I'll look at it if nobody does before
20:44 mflobo joined #puppet-openstack
20:47 v1k0d3n joined #puppet-openstack
20:47 _ody Lame bundler bug.
20:49 EmilienM _ody: :)
20:49 EmilienM _ody: your first +2, heh!
20:50 _ody nah.  I believe I did one last week too but that failure was on the day's first.
20:52 dhollinger joined #puppet-openstack
20:54 v1k0d3n joined #puppet-openstack
20:58 v1k0d3n joined #puppet-openstack
20:58 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: implement class and defined type for cinder GPFS driver  https://review.openstack.org/257565
20:58 karimb joined #puppet-openstack
21:01 _ody EmilienM: weird.  I can't comment on https://review.openstack.org/#/c/257565
21:04 EmilienM _ody: comment?
21:05 _ody Nevermind.  It was just a UI thing I've never noticed.
21:09 dslevin_ joined #puppet-openstack
21:12 v1k0d3n joined #puppet-openstack
21:13 EmilienM dmsimard: how do you feel about testing mitaka in puppet CI for centos jobs? Have you heard about some blockers?
21:13 EmilienM I see https://launchpad.net/~ubuntu-cloud-a​rchive/+archive/ubuntu/mitaka-staging
21:13 EmilienM maybe worth to test it
21:33 yottatsa joined #puppet-openstack
21:49 _ody by randome chance, someone here whould have happened to backport rally to jessie?
21:50 yottatsa joined #puppet-openstack
21:50 * _ody is down a rabbit hole of dependencies
21:53 yottatsa joined #puppet-openstack
21:56 yottatsa joined #puppet-openstack
21:59 mwhahaha zigo could probably answer that
22:02 baker joined #puppet-openstack
22:07 yottatsa joined #puppet-openstack
22:14 mdorman mfisch:   ping?
22:15 mfisch mdorman: yo
22:16 mdorman re:  https://bugs.launchpad.net/p​uppet-keystone/+bug/1523643   just wondering if you had any patches in flight for that?
22:16 openstack Launchpad bug 1523643 in puppet-keystone "glance & keystone fighting over openstack client" [High,In progress] - Assigned to Matt Fischer (mfisch)
22:16 mfisch mdorman: nope
22:16 mfisch oh wait
22:16 mfisch sorry yes I do
22:16 mfisch they all landed
22:16 mfisch I backported the one that fixes it to liberty
22:16 EmilienM I had some news from canonical, and they'll have mitaka packaging next week
22:16 mdorman i am running into a related problem where if you specify a version (vs. ‘present’ or ‘absent’) to keystone::client, it forces openstackclient to be set to the same version.  i want to indicate independent version numbers for each.
22:16 EmilienM in the meantime, RDO packaging seems ready
22:17 EmilienM I think we could bump our CI during Xmas holidays
22:17 mdorman mfisch:   ok maybe i just need to pull then
22:17 EmilienM so we'll run mitaka very soon
22:17 mfisch mdorman: what we did was have Keystone and glance just include ::openstacklib::openstackclient
22:17 mfisch mdorman: want to BP them?
22:17 EmilienM maybe I should ask ML
22:17 mfisch mdorman: https://review.openstack.org/#/c/254824/  https://review.openstack.org/#/c/253311/
22:17 mfisch I will +2 a BP
22:19 mdorman i’m off master i just pulled longer ago than when these langed
22:19 mdorman *landed
22:19 mfisch ok
22:19 mdorman i can create the backport reviews, though, if i can remember/figure out how to do it.
22:19 mfisch you do it in gerrit but I'd rather not
22:19 openstackgerrit Mike Dorman proposed openstack/puppet-keystone: Let openstacklib manage openstackclient  https://review.openstack.org/257583
22:20 mfisch the 3rd review "fixes" it for most people without being intrusive
22:20 mfisch but if you think people want it then I'm +2
22:20 openstackgerrit Mike Dorman proposed openstack/puppet-glance: Let openstacklib manage openstackclient  https://review.openstack.org/257584
22:20 mdorman the third review?
22:21 EmilienM mfisch: any patches we can help now?
22:21 mfisch mdorman: the 3rd review is in liberty
22:23 mdorman oh.  so i should not have backported those to stable/liberty
22:23 mdorman so no Closes-Bug on those?  none of them show up in the bug report
22:24 mfisch mdorman: well its debatable
22:24 mfisch give me 1 min
22:24 mfisch This one solves the actual bug: https://review.openstack.org/#/c/254502/
22:24 mfisch the other 2 in general make things cleaner and better but might not need  BP
22:24 mfisch make sense?
22:24 mfisch like without that last review you can't compile the catalog
22:26 gildub joined #puppet-openstack
22:27 mdorman sure you can?  if python-openstackclient package is defined in openstacklib::openstackclient and everywhere else it’s just include’d ?
22:27 mfisch one was including the class, one was including the package
22:27 mfisch I think glance & keystone respectively
22:27 ducttape_ joined #puppet-openstack
22:27 mfisch we covered this in the meeting last week and on the ML to a lesser extent
22:28 mdorman kk, sorry for dup questions here.   i see what you’re saying, i didn’t realize there was a difference in glance.  at the moment i’m only concerned with keystone
22:28 yottatsa joined #puppet-openstack
22:28 mfisch iirc keystone was doing it wrong
22:29 mdorman i guess i have no opinion on the backport to stable/liberty for those other two patches.  like i said i’m more or less running off master so i don’t really care.
22:29 mfisch okay lets skip for now then
22:29 mfisch unless we have a good reason
22:30 mdorman sounds good.
22:43 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
22:43 EmilienM crinkle, _ody, nibalizer: do you do something for puppethack day tomorrow?
22:46 nibalizer ill be puppeting likely
22:46 nibalizer not super sure what
22:46 nibalizer i might just try to slog through puppet community pull requests
22:47 baker joined #puppet-openstack
22:48 EmilienM nice
22:48 nibalizer https://twitter.com/nibalize​r/status/676534071929868288
22:48 EmilienM nibalizer: rt'ed
22:48 nibalizer haha
22:48 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
22:50 _ody EmilienM: It kinda just lines up with the day I generally dedicate my whole day to Puppet OpenStack stuff...so I'll coincidentally probably just be hacking on that.
22:51 EmilienM \o/
22:52 mgrosz joined #puppet-openstack
23:01 baker joined #puppet-openstack
23:26 mgrosz joined #puppet-openstack
23:26 ducttape_ joined #puppet-openstack
23:36 davideagle joined #puppet-openstack
23:54 openstackgerrit Merged openstack/puppet-openstack-integration: Update Puppetfile to latest releases / branches  https://review.openstack.org/256644
23:54 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: CI test - never merge  https://review.openstack.org/257611

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