Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-02-26

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

All times shown according to UTC.

Time Nick Message
00:04 baker joined #puppet-openstack
00:11 EmilienM mwhahaha: omg
00:11 EmilienM https://review.openstack.org/#/c/284900/
00:12 EmilienM from 45 min to 28 min
00:12 mwhahaha :o
00:12 EmilienM yeah
00:13 EmilienM I increased it early this week to have RBD jobs successful but 60 was too much
00:13 * EmilienM feels like http://goo.gl/T2Kelq
00:13 EmilienM or maybe http://goo.gl/7BiA0g
00:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: generate testrepository.subunit all the time  https://review.openstack.org/283930
00:17 EmilienM mwhahaha: fwiw, scenario003 is still useful, it will help to scale our tests and also have more coverage
00:21 baker joined #puppet-openstack
00:27 rahulait joined #puppet-openstack
00:30 yolanda_ joined #puppet-openstack
00:36 baker joined #puppet-openstack
00:37 xingchao joined #puppet-openstack
00:39 openstackgerrit Merged openstack/puppet-gnocchi: Fix incorrect class params  https://review.openstack.org/281566
00:39 openstackgerrit Merged openstack/puppet-neutron: Set v3 auth related options by default  https://review.openstack.org/282309
00:39 openstackgerrit Merged openstack/puppet-openstack-integration: Add ability to manually install puppet modules  https://review.openstack.org/282020
00:39 openstackgerrit Merged openstack/puppet-trove: Add kombu_reconnect_delay option  https://review.openstack.org/279809
00:50 baker joined #puppet-openstack
00:52 ayoung joined #puppet-openstack
01:02 baker joined #puppet-openstack
01:08 ducttape_ joined #puppet-openstack
01:16 cwolferh joined #puppet-openstack
01:20 openstackgerrit Cody Herriges proposed openstack/puppet-horizon: Making root_url configurable  https://review.openstack.org/283827
01:24 baker joined #puppet-openstack
01:36 baker joined #puppet-openstack
01:47 baker joined #puppet-openstack
02:04 EmilienM ok Ubuntu looks better
02:05 EmilienM mwhahaha: we'll need to land https://review.openstack.org/#/c/284978/
02:07 xingchao joined #puppet-openstack
02:31 openstackgerrit Dan Prince proposed openstack/puppet-keystone: Run keystone-manage bootstrap  https://review.openstack.org/284876
02:34 brandon joined #puppet-openstack
02:36 xarses joined #puppet-openstack
02:37 ducttape_ joined #puppet-openstack
02:45 mwhahaha I'll need to look into that further because that change appears to fail the fuel CI https://ci.fuel-infra.org/view/puppet-openstack/job/puppet-openstack.fuel-library.pkgs.ubuntu.neutron_vlan_ha/148/console
02:45 mwhahaha So we might need to land something first so we don't have a repeat of the ml2 conf problem
02:45 ducttape_ joined #puppet-openstack
02:47 ilbot3 joined #puppet-openstack
02:47 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
02:51 xingchao joined #puppet-openstack
02:53 ducttape_ joined #puppet-openstack
03:05 baker joined #puppet-openstack
03:12 baker joined #puppet-openstack
03:35 mwhahaha actually it might have failed because i think it started prior to our merge for the ml2 conf issue, rerunning to make sure it's ok
03:43 gchamoul joined #puppet-openstack
03:45 fedexo joined #puppet-openstack
04:01 brandon joined #puppet-openstack
04:07 baker joined #puppet-openstack
04:10 openstackgerrit Merged openstack/puppet-openstack-integration: run_tests: generate testrepository.subunit all the time  https://review.openstack.org/283930
04:15 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
04:47 brandon joined #puppet-openstack
04:51 markvoelker joined #puppet-openstack
05:05 ducttape_ joined #puppet-openstack
05:12 severion joined #puppet-openstack
05:29 mwhahaha yea we can land that patch, should be ok
05:38 khappone joined #puppet-openstack
05:58 xingchao joined #puppet-openstack
05:58 rlaurika joined #puppet-openstack
06:02 imcsk8 joined #puppet-openstack
06:53 markvoelker joined #puppet-openstack
07:05 rcernin joined #puppet-openstack
07:09 magicboiz joined #puppet-openstack
07:09 vinsh_ joined #puppet-openstack
07:12 magicboiz left #puppet-openstack
07:52 jtomasek joined #puppet-openstack
07:52 furlongm_ joined #puppet-openstack
07:59 jtomasek_ joined #puppet-openstack
08:21 openstackgerrit Nikolay Starodubtsev proposed openstack/puppet-murano: Switch murano to use keystone API v3 by default  https://review.openstack.org/279444
08:29 dgurtner joined #puppet-openstack
08:31 jpena joined #puppet-openstack
08:41 hjensas joined #puppet-openstack
08:53 markvoelker joined #puppet-openstack
09:22 gfidente joined #puppet-openstack
09:27 jistr joined #puppet-openstack
09:28 rcernin joined #puppet-openstack
09:30 brandon joined #puppet-openstack
09:32 xarses_ joined #puppet-openstack
09:32 v1k0d3n joined #puppet-openstack
09:50 davideagle_ joined #puppet-openstack
10:00 paramite joined #puppet-openstack
10:13 derekh joined #puppet-openstack
10:34 ducttape_ joined #puppet-openstack
10:38 dalees joined #puppet-openstack
10:43 openstackgerrit Merged openstack/puppet-neutron: change default value for lock_path  https://review.openstack.org/284978
10:43 openstackgerrit Merged openstack/puppet-openstack-integration: tempest: reduce compute/build_interval from 60 to 10  https://review.openstack.org/284900
10:47 davideagle_ joined #puppet-openstack
10:48 openstackgerrit Merged openstack/puppet-openstack-integration: ubuntu/nova: configure notifications using keystone v3  https://review.openstack.org/284503
10:53 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs  https://review.openstack.org/284771
10:54 markvoelker joined #puppet-openstack
10:55 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Add support of Block Device backend for Cinder  https://review.openstack.org/284834
11:19 openstackgerrit Merged openstack/puppet-neutron: Link linuxbridge agent to rabbitmq changes  https://review.openstack.org/284842
11:50 xingchao joined #puppet-openstack
11:51 davideagle_ joined #puppet-openstack
11:54 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs  https://review.openstack.org/284771
11:56 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
12:17 EmilienM good morning
12:18 cdelatte joined #puppet-openstack
12:26 EmilienM degorenko: thx a lot for the reviews
12:26 EmilienM so we have Ubuntu almost green now, except for scenario003, with Heat
12:26 EmilienM but I'm working on it so we can re-enable voting maybe today
12:34 davideagle_ joined #puppet-openstack
12:37 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/285273
12:40 markvoelker joined #puppet-openstack
12:50 degorenko EmilienM,  no problem :)
12:50 degorenko EmilienM, btw, https://review.openstack.org/#/c/284771/ green :)
12:50 degorenko i forgot update store parameter in glance_store section
12:51 EmilienM oh nice !!
12:51 EmilienM Excellent
12:51 EmilienM I think we can try to add Cinder on scenario002, using iscsi backend
12:52 degorenko EmilienM, but wait with merging, i've found related issue in glance swift backend class
12:52 EmilienM degorenko: which one?
12:52 degorenko EmilienM, as you see here: https://review.openstack.org/#/c/284771/4/manifests/glance.pp - i passed auth_address for swift
12:53 degorenko because default value without http protocol, without any protocol
12:53 degorenko and it uses https
12:53 EmilienM https://github.com/openstack/puppet-glance/blob/master/manifests/backend/swift.pp#L44
12:53 degorenko do we want leave it as it now
12:53 degorenko ?
12:53 EmilienM we need to fix that
12:53 degorenko ok, i will update it to http protocol then
12:53 EmilienM or?
12:53 EmilienM what is the default in upstream?
12:53 degorenko or update it :)
12:53 * degorenko looking
12:54 degorenko EmilienM, in liberty is none
12:54 degorenko so, i guess we can use http here by default
12:54 EmilienM ok
12:54 EmilienM go ahead
12:54 EmilienM and use Depends-On to test it in the integ patch
12:54 degorenko EmilienM, ok :)
12:55 EmilienM degorenko: I'm happy to see jobs back to ~30 min
12:55 EmilienM instead of 1h or 50 min
12:55 degorenko :D yes
12:57 dprince joined #puppet-openstack
12:58 colonwq EmilienM, The checks are going much faster today. I guess I'll see less of the outside today.
13:01 EmilienM dprince: I'm investigating the Ubuntu thing on https://review.openstack.org/#/c/284876 - you don't have to worry about it
13:03 dprince EmilienM: okay, thanks
13:03 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstack-integration: Fix rspec testing outside of the Openstack CI.  https://review.openstack.org/285285
13:05 EmilienM nice shot ^
13:06 EmilienM mfisch: I think chem is fixing a bug you reported in a last meeting
13:07 chem mfisch: EmilienM, kind of though that It would be easy to provide a how-to, but it wasn't :)
13:07 EmilienM chem: https://bugs.launchpad.net/puppet-keystone/+bug/1548872
13:07 openstack Launchpad bug 1548872 in puppet-keystone "bundle exec rake spec fails to find r10k - tests wont run" [High,Confirmed]
13:08 EmilienM he already created a bug I think, is it the same?
13:08 chem mfisch: EmilienM had to patch the stuff, preparing a mail now
13:08 chem EmilienM: yep, oups sorry
13:08 EmilienM chem: no worries
13:08 EmilienM I close his bug
13:09 chem EmilienM: mfisch I knew that my working env was /luckily/ working, but it kind of bother me for a while ... I think the patch should do
13:10 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Use http protocol by default in auth_address for Swift backend  https://review.openstack.org/285290
13:11 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Use swift backend for glance in scenario002 jobs  https://review.openstack.org/284771
13:13 ducttape_ joined #puppet-openstack
13:15 EmilienM degorenko: can you -A and +A again please?
13:15 EmilienM https://review.openstack.org/#/c/284863/
13:15 EmilienM to trigger zuul
13:15 degorenko done :)
13:16 EmilienM it's not triggered
13:16 EmilienM I need to rebase it I think
13:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: heat: manage heat user role before openrc  https://review.openstack.org/284863
13:16 EmilienM degorenko: a last time? :)
13:17 degorenko boom :)
13:20 markvoelker joined #puppet-openstack
13:24 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Switch creating cinder types to providers from define classes  https://review.openstack.org/275669
13:25 brad[] joined #puppet-openstack
13:26 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
13:27 degorenko EmilienM, do you want to add iscsi to cinder scenario002 or i can do it? :)
13:28 chem EmilienM: degorenko, whoua, that was fast.  Thanks,  hope I didn't break all CI :)
13:29 degorenko chem, can you +1 again me here: https://review.openstack.org/273513 please ? :D
13:29 EmilienM degorenko: go ahead
13:29 degorenko ack :)
13:32 chem degorenko: basically you re-introduced the original type for a deprecation period, got it right ?
13:32 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Add testing Cinder+ISCSI backend for scenario002 jobs  https://review.openstack.org/285300
13:32 degorenko chem, that's right. I totally forgot to go through deprecation cycle for this defines :D
13:33 chem degorenko: not only you :)
13:33 degorenko thanks mwhahaha for reminder :D
13:33 chem degorenko: in the meantime, we don't care about the current error on centos ?
13:34 degorenko chem, well, yes, it should go away after some count of rechecks :D
13:35 degorenko because patch works before this update :) and now it still works for trusty, so it is ci issues
13:35 chem degorenko: looks like Disable_admin_token_auth/ is not not working ok ...  anyway, completly unrelated to your patch
13:36 chem degorenko: +1
13:36 degorenko thanks :)
14:00 johnmilton joined #puppet-openstack
14:00 yolanda joined #puppet-openstack
14:03 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
14:03 richm joined #puppet-openstack
14:07 EmilienM degorenko: you'll need https://review.openstack.org/283786 first
14:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: enable fwaas  https://review.openstack.org/283920
14:09 degorenko EmilienM, i will rebase on your, just to make sure that tests are working
14:09 EmilienM ok
14:09 EmilienM but running tempest plugins require some work, in upstream projects
14:09 EmilienM it won't be easy
14:17 openstackgerrit Merged openstack/puppet-openstack-integration: Fix rspec testing outside of the Openstack CI.  https://review.openstack.org/285285
14:17 vinsh joined #puppet-openstack
14:17 karimb joined #puppet-openstack
14:30 openstackgerrit Merged openstack/puppet-openstack-integration: heat: manage heat user role before openrc  https://review.openstack.org/284863
14:42 xarses_ joined #puppet-openstack
14:44 davideagle_ joined #puppet-openstack
14:47 dfisher joined #puppet-openstack
14:48 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging  https://review.openstack.org/285333
14:53 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
14:56 ducttape_ joined #puppet-openstack
15:04 delattec joined #puppet-openstack
15:06 xingchao joined #puppet-openstack
15:07 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer  https://review.openstack.org/285344
15:07 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: use stevedore names for driver / backends  https://review.openstack.org/285345
15:07 EmilienM richm, chem: I need your eyes on ^
15:07 EmilienM mfisch: also
15:08 chem EmilienM: ack, I add it to my favorite patch :)
15:08 dhollinger joined #puppet-openstack
15:09 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Run keystone-manage bootstrap  https://review.openstack.org/284876
15:14 vinsh joined #puppet-openstack
15:18 baker joined #puppet-openstack
15:26 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer  https://review.openstack.org/285358
15:28 delatte joined #puppet-openstack
15:29 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer  https://review.openstack.org/285358
15:30 baker joined #puppet-openstack
15:30 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer  https://review.openstack.org/285358
15:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin  https://review.openstack.org/283786
15:33 davideagle_ joined #puppet-openstack
15:34 EmilienM degorenko: you'll need to rebase https://review.openstack.org/#/c/250436/ on top of ^
15:34 EmilienM but I doubt 250436 will pass easily
15:34 EmilienM I found many bugs upstream that prevent us to run tempest plugins
15:38 degorenko EmilienM, for 250436 we also don't have rdo murano packages anyway :D
15:39 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging  https://review.openstack.org/285333
15:40 xarses_ joined #puppet-openstack
15:40 EmilienM degorenko: not yet? what is blocking?
15:40 degorenko EmilienM, i don't know :(
15:55 alee joined #puppet-openstack
16:06 haplo37 joined #puppet-openstack
16:13 EmilienM chem: mhh now when I run integ jobs in my servers, it fails to run r10k
16:13 chem EmilienM: your server on CI openstack ?
16:13 EmilienM no
16:13 EmilienM *my* servers
16:14 EmilienM I'm digging
16:14 logan- joined #puppet-openstack
16:14 chem EmilienM: hum you must have hardcoded the GEM_PATH_BIN somewhere
16:17 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules.sh: run gem install with --verbose  https://review.openstack.org/283696
16:20 alee joined #puppet-openstack
16:31 IvanBerezovskiy left #puppet-openstack
16:34 EmilienM degorenko: can you +A https://review.openstack.org/#/c/284031/ please?
16:35 EmilienM ayoung: chem wrote testing for ldap, maybe you can talk each other
16:36 EmilienM chem: ayoung is confused because about puppet-keystone/ldap support
16:36 degorenko EmilienM, so many lines, i believe that everything fine here :) +A
16:36 degorenko oh, cherry pick
16:36 chem ayoung: what is the problem ?
16:37 ayoung chem, EmilienM showed me a log with a referencetop an LDAP Credential backend
16:37 ayoung http://logs.openstack.org/76/284876/4/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/161cc03/console.html#_2016-02-26_11_27_31_950
16:37 ayoung and there is no LDAP credential backend, nor has there ever been.
16:38 chem ayoung: I think I got it
16:39 ayoung cool
16:40 chem ayoung: I didn't have a look at the patch yet, but I know that I could not have keystone to run, in my test.  I just test that files are created and clean it up, because if I try any command it would fails as there is no actual backend
16:40 chem ayoung: it would require installing an actual ldap server and all
16:40 chem ayoung: which at the time was considered good to have but non urgent
16:41 ayoung chem, it looks like a typo, maybe copied over from the Identity backend. That is the only place LDAP should ever be used.  It is  deprecated or removed from all of Keystone but Identityt
16:44 chem ayoung: oki, I just looked at the code and ... it's big change :)
16:44 chem ayoung: the Keystone_config<||> ~> Exec<| title == 'keystone-manage bootstrap'|>
16:45 chem ayoung: line is hard
16:45 ayoung Yeah.  It should actually be a net reduction in calls from Puppet
16:45 ayoung I'm still getting my head around the bootstrap call. Jamie Lennox caught an interesting paradox there:
16:46 ayoung we need to set up the first endpoint, which is the keystone Identity Service/endpoint itself.  Or we can't do anything
16:46 chem ayoung: so as I said the ldap test must not restart keystone, or try to run it as they will *fail*, with this catch all I think that trigger a keystone run that fail
16:46 ayoung chem, So this came from the first time someone tried to run the LDAP tests?
16:48 chem ayoung: I did a quick review of the associated change https://review.openstack.org/#/c/284876/5
16:48 chem ayoung: I think the problem would disapear if the unconditional trigger is removed
16:48 ayoung chem, so let me show you another approach
16:49 ayoung https://review.openstack.org/#/c/284778/
16:49 fedexo joined #puppet-openstack
16:50 chem ayoung: it's friday evening :) what is the link ?
16:50 ayoung chem, it is a call to crete the default domain on demand
16:50 ayoung taking it off your hands
16:51 davideagle_ joined #puppet-openstack
16:51 ayoung "V2 operations create default domain on demand To support deployment utilities that are still using v2, create a default domain for them when they first use the v2 API. "
16:51 chem ayoung: how it is related to ldap ?
16:52 ayoung chem, that was the issue with the DB sync; it used to create the default domain.  We stopped doing that.
16:52 ayoung I am not quite sure the rationale, I didn;'t review the change, but it happened as part of the the every-release-compact-the-migrations process
16:53 ayoung chem, the link you sent had this comment in the review: "Per upstream Keystone Mitaka commit 7b7fea7a3fe7677981fbf9bac5121bc15601163 keystone no longer creates the default domain during the db_sync. This patch updates the keystone class so that it adds a new option to enable_bootstrap (true by default) to re-add this functionality."
16:54 chem ayoung: I send you this as I though it was linked to the error log you sent me :)
16:56 angdraug joined #puppet-openstack
16:56 chem ayoung: do you want to bj, I think we misunderstand each other :)
17:00 EmilienM ayoung: chem did not do this patch, dprince did
17:00 ayoung chem, nah...I have something I am heads down in at the moment.  Is this going to hold you up release wise?
17:00 EmilienM ayoung: what chem helped, is ldap bits in puppet-keystone
17:00 ayoung I need to finish a pretty significant patch, that other people are depending on
17:01 ayoung chem, so, LDAP in Puppet keystone needs to be Identity only.  Ignore all other backends
17:01 ayoung you can remove those capability from puppet-keystone
17:01 ayoung as Keysteon will no longer support anything but Identity, and that is headed toward read-only
17:03 darkxploit joined #puppet-openstack
17:05 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Make sure to use GEM_BIN_DIR everwhere r10k is run  https://review.openstack.org/285438
17:06 chem ayoung: oki.  Not sure what change that imply, but for the test included in the acceptance test, as I said, there are only there to test that the files get created, not to actualy works, as it would require a ldap backend https://trello.com/c/DuNmElKz/70-add-a-ldap-server-to-keystone-acceptance-test-for-testing-ldap-backend
17:06 cwolferh joined #puppet-openstack
17:06 ayoung chem, there is code in Devstack to set up an LDAP server that you can use if you need
17:07 chem ayoung: EmilienM for the error at hand, I'm quite sure that adding the conditional I suggest in the review should make this error go away
17:07 ayoung chem, but I would check the puppet module to see why it is trying to set up a Credential LDAP backend, cuz that is wrong
17:07 dprince joined #puppet-openstack
17:09 openstackgerrit Merged openstack/puppet-trove: Remove trove ubuntu package hack  https://review.openstack.org/284031
17:29 ilbot3 joined #puppet-openstack
17:29 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
17:50 davideagle_ joined #puppet-openstack
18:04 dhollinger joined #puppet-openstack
18:11 RichardRaseley joined #puppet-openstack
18:14 openstackgerrit Merged openstack/puppet-openstack-integration: Add testing Cinder+ISCSI backend for scenario002 jobs  https://review.openstack.org/285300
18:15 openstackgerrit Merged openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/285273
18:18 openstackgerrit Merged openstack/puppet-openstack-integration: neutron: enable fwaas  https://review.openstack.org/283920
18:37 colonwq xarses, Can you give my patch (254548) a hopefully last look over?
18:44 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: use stevedore names when possible and cleanup ldap testing  https://review.openstack.org/285345
18:48 EmilienM crinkle, _ody: would you mind to confirm my statement on https://review.openstack.org/#/c/284876/5/manifests/init.pp ?
18:49 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging  https://review.openstack.org/285333
18:58 EmilienM xarses: I tested https://review.openstack.org/#/c/276141
18:58 EmilienM and even without the patch, restarting a node does not restart ceph
18:58 * xarses slaps forhead
18:58 EmilienM you have to run puppet again.
18:58 EmilienM I have a public VM if you want to try
18:58 imcsk8 joined #puppet-openstack
18:58 EmilienM but I just tried with & without
18:59 xarses thats fine, it's already broken.
18:59 EmilienM puppet-ceph has a bug, we need to fix it
18:59 xarses we can merge this one then
19:00 xarses do you want to fix it here, or somewhere else?
19:00 EmilienM xarses: somewhere else
19:00 EmilienM I like atomic commits
19:00 xarses I'm good with that
19:00 xarses I'll give this another once over
19:01 EmilienM I'm reporting the bug and working on it now.
19:01 EmilienM xarses: and mwhahaha already reported it: https://bugs.launchpad.net/puppet-ceph/+bug/1540744
19:01 openstack Launchpad bug 1540744 in puppet-ceph "puppet-ceph does not ensure services start on boot" [Undecided,Confirmed]
19:02 xarses sigh, I thought that only mucked up the MON's
19:02 EmilienM and he abandonned his patch because rgw was fixed
19:02 EmilienM but not mon & osd
19:02 EmilienM so we need to fix mon & osd
19:03 colonwq EmilienM, which OS does that happen on?
19:03 EmilienM colonwq: centos7
19:04 colonwq EmilienM, could it be related to the 'systemctl daemon-relaod' nastiness of old init scripts? I have not looked at the logs yet.
19:05 xarses thanks EmilienM, I've +2+A'd it
19:05 xingchao joined #puppet-openstack
19:06 EmilienM colonwq: maybe. I need to dig
19:06 EmilienM xarses: thx
19:06 delattec joined #puppet-openstack
19:07 EmilienM xarses: I find you a bit negative on the review. What is broken exactly?
19:07 EmilienM it's an hack I agree, but it's not related to puppet-ceph directly, is it?
19:09 delatte joined #puppet-openstack
19:12 xarses sorry, that reboots are already broken
19:12 EmilienM I'm testing it now
19:12 EmilienM honestly, a lot of things are broken after a reboot
19:13 EmilienM specially when running OpenStack in production, with clustering, etc
19:13 xarses ya, we run an entire suite of tests to try and ensure it's fine
19:13 xarses and still miss things
19:13 xarses I just had it in my mind, that disabeling udev would break reboots for OSD's
19:14 xarses turns out, it was already broken
19:14 EmilienM let's fix it
19:14 dprince joined #puppet-openstack
19:16 dmsimard mtreinish: I know I've asked you this before (probably)
19:16 dmsimard mtreinish: but where does the subunit file need to be for it to be picked up by the health thingie ?
19:16 EmilienM dmsimard: https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L108
19:16 dmsimard mtreinish: at the root of /logs ?
19:17 dmsimard EmilienM: you're putting it in /tmp ...
19:17 dmsimard EmilienM: I guess copy puppet logs puts it somewhere useful
19:17 dmsimard https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh#L101
19:17 dmsimard ok so root of /logs
19:17 xarses_ joined #puppet-openstack
19:18 EmilienM yes
19:21 dprince joined #puppet-openstack
19:21 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Run keystone-manage bootstrap  https://review.openstack.org/284876
19:23 ducttape_ joined #puppet-openstack
19:23 dhollinger joined #puppet-openstack
19:34 openstackgerrit Emilien Macchi proposed openstack/puppet-ceph: fix collectors: Ceph_Config -> Ceph_config  https://review.openstack.org/285510
19:44 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: debug https://review.openstack.org/#/c/280351  https://review.openstack.org/285517
19:44 openstackgerrit Trevor McCasland proposed openstack/puppet-neutron: Requires allocation pools for creation of subnet  https://review.openstack.org/285518
19:57 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Update the README about the third scenario  https://review.openstack.org/285523
20:00 openstackgerrit Merged openstack/puppet-ceph: Disable udev rules in ::osd for releases <= Hammer  https://review.openstack.org/276141
20:00 dmsimard EmilienM: all three scenarios are stable on centos now right ?
20:00 EmilienM yes
20:00 EmilienM and 001/002 on ubuntu too
20:00 dmsimard ok adding the third scenario to our pipeline
20:00 dmsimard thanks
20:00 EmilienM 003 is broken though
20:00 EmilienM cool np
20:03 yolanda joined #puppet-openstack
20:08 mtreinish dmsimard: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/files/logstash/jenkins-log-client.yaml#L14 is where it looks for things after a run
20:09 dmsimard mtreinish: ty, just fixed the packstack jobs to drop it there
20:13 EmilienM mtreinish: I'm still confused by running tempest plugins does not work for us https://review.openstack.org/#/c/283786/
20:15 mtreinish EmilienM: so that looks like it's caused by a bad fwaas plugin tempest-lib is being used but never installed (it's not in requirements) http://logs.openstack.org/86/283786/5/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/4568e94/console.html#_2016-02-26_18_10_30_362
20:16 EmilienM mtreinish: so it's in neutron_fwaas upstream, isn't?
20:17 mtreinish yeah, I think so
20:17 EmilienM https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/tempest_plugin/tests/scenario/test_fwaas.py#L21
20:18 EmilienM https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/tempest_plugin/tests/scenario/base.py#L16-L17
20:18 EmilienM it's a bug in neutron_fwaas
20:19 mtreinish yeah, they don't have tempest-lib in requirements.txt or test-requirements.txt
20:19 mtreinish so nothing is installing tempest-lib and the plugin code imports it
20:20 EmilienM do people actually test things?
20:20 mtreinish heh, doesn't look like it. Although to be fair this wouldn't have been caught before because tempest used to depend on tempest-lib so it would install it for them
20:20 mtreinish but that changed this week
20:22 EmilienM mtreinish: should I patch  https://github.com/openstack/neutron-fwaas/blob/master/test-requirements.txt ?
20:22 mtreinish EmilienM: yeah, that or add a setuptools extras target for the plugin. Either will work
20:23 mtreinish but if it's in test requirements you'll have to make sure to install those too
20:32 EmilienM mtreinish: I did like nova does: https://review.openstack.org/285534
20:33 mtreinish EmilienM: ok that works. Just note when you install fwaas, you'll have to install test-requirements manually, just doing pip install neutron-fwaas won't do that
20:33 EmilienM ok
20:33 EmilienM mtreinish: but in the puppet CI, we use packaging
20:34 EmilienM so even with my patch, we'll still have the bug I think, isn't?
20:34 EmilienM we deploy tempest-lib in a venv
20:34 mtreinish right, I think that'll be the case. Since nothing is installing tempest-lib
20:34 EmilienM so should I have to run pip install tempest-lib before running tests?
20:35 dmsimard EmilienM: that stems from the fact that we don't typically install test-requirements for runtime
20:35 mtreinish EmilienM: that would fix it
20:35 dmsimard EmilienM: i.e, we might install sphinx (from test-requirements) at buildtime only for docs but sphinx won't get installed when you install the package
20:35 EmilienM mtreinish: why installing tempest does not install tempest lib?
20:35 mtreinish btw, this was part of my argument that tempest plugins should be self contained pythong projects
20:36 mtreinish because packaging and installing gets much more complicated when they share a repo
20:36 EmilienM yeah but they still rely on tempest-lib
20:36 EmilienM ok
20:36 EmilienM so let's install tempest-lib system wide
20:36 dmsimard mtreinish: so does tempest-lib become a runtime dependency then ?
20:36 mtreinish EmilienM: because of: https://review.openstack.org/#/c/283691/
20:36 EmilienM dmsimard: iiuc, yes
20:36 mtreinish EmilienM: which is part of: http://specs.openstack.org/openstack/qa-specs/specs/tempest/reintegrate-tempest-lib.html
20:37 EmilienM mtreinish: NEED IT !
20:37 dmsimard then it needs to be in requirements.txt, not test-requirements.txt
20:37 dmsimard and we can pick it up in packaging if necessary
20:37 EmilienM mtreinish: so people should update their tests right?
20:37 dmsimard but I sort of agree with EmilienM that tempest-lib should come bundled with tempest
20:37 mtreinish EmilienM: they should, but there is no requirement to do it. (the old tempest-lib isn't going away)
20:38 mtreinish dmsimard: tempest-lib and tempest are seperate packages
20:38 dmsimard fair enough
20:38 mtreinish tempest isn't even on pypi yet
20:39 mtreinish (although that'll change next week when I push a release)
20:39 dmsimard neither is nova
20:39 dmsimard :)
20:40 mtreinish right, but for requirements in openstack projects that's one of the minimum requirements. Nova isn't supposed to be imported so it doesn't really matter
20:40 dmsimard I see
20:42 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: run tox with -eall-plugin  https://review.openstack.org/283786
20:49 EmilienM mwhahaha: have you some progress on lint/centos7?
20:50 mwhahaha no not at the moment
20:50 mwhahaha got side tracked on a bunch of other stuff
20:50 EmilienM mwhahaha: need help?
20:50 mwhahaha it seems fairly straight forward but did we want to just combine the syntax/lint checks? it seems a waste to have to spin up a vm just for lint
20:51 mwhahaha i think i had asked about that the last time linting took hours
20:58 EmilienM mwhahaha: we have this discussion in the past, let me find the history
20:58 EmilienM https://review.openstack.org/#/c/220527/
20:59 EmilienM oh and look https://review.openstack.org/#/c/209147/
21:00 openstackgerrit Merged openstack/puppet-neutron: Support midonet type driver for ML2  https://review.openstack.org/278207
21:02 brandon joined #puppet-openstack
21:03 EmilienM mwhahaha: well, looks like they won't let it happen
21:03 EmilienM on #openstack-infra
21:05 dgurtner joined #puppet-openstack
21:05 dgurtner joined #puppet-openstack
21:06 mwhahaha yea that was my other concern as i had a feeling they wouldn't be ok with the switch since there wasn't a really good technical reason
21:15 delattec joined #puppet-openstack
21:15 EmilienM mwhahaha: trying something with https://review.openstack.org/285542
21:22 delattec joined #puppet-openstack
21:22 davideagle joined #puppet-openstack
21:56 EmilienM mfisch: can you try out if r10k works for you now?
21:57 spredzy joined #puppet-openstack
21:57 EmilienM chem's patch actually broke me when I deploy integration jobs on my personal servers
21:57 EmilienM (I run all-in-one)
22:13 EmilienM crinkle: where is cacert_content value in infra-cloud?
22:15 EmilienM I guess in system-config
22:17 crinkle EmilienM: here http://git.openstack.org/cgit/openstack-infra/system-config/tree/hiera/common.yaml#n361
22:17 EmilienM thanks
22:23 davideagle_ joined #puppet-openstack
22:23 brandon joined #puppet-openstack
22:34 brandon joined #puppet-openstack
22:38 EmilienM crinkle: do you store ssl_key_file_contents content in hiera too?
22:39 crinkle EmilienM: yes but it's private
22:39 EmilienM ok I see
22:39 EmilienM I wanted to re-use your files :-P
22:40 EmilienM I'll generate it on my laptop for our CI
22:40 crinkle EmilienM: just generate new ones :P
22:40 EmilienM I'm lazy!
22:52 EmilienM crinkle: just by curiosity, how many times do you run puppet on a fresh infra cloud node to deploy it?
22:53 crinkle EmilienM: it works the first time :D
22:54 EmilienM omg
22:54 crinkle EmilienM: we don't use swift
22:54 EmilienM crinkle: I'm wondering because you don't set any orchestration for SSL certificates
22:55 EmilienM and what if they are installed *after* deploying nova
22:55 crinkle EmilienM: there is orchestration i think
22:55 EmilienM indeed
22:56 EmilienM I see notify / require
22:56 EmilienM my bad
23:14 EmilienM crinkle: I see you don't manage /etc/ssl/certs/openstack_infra_ca.pem with Puppet, right?
23:14 nibalizer EmilienM: i think that is managed
23:15 nibalizer certainly it exists in hiera
23:15 EmilienM nibalizer: I don't find it in .pp files
23:15 EmilienM maybe in system-config?
23:15 nibalizer http://git.openstack.org/cgit/openstack-infra/system-config/tree/hiera/common.yaml#n389
23:15 crinkle EmilienM: the infracloud::cacert class has an exec that links /usr/share/local/ca-certificates/thing.crt to /etc/ssl/certs
23:16 crinkle i think it's an ubuntuism
23:16 EmilienM oh.. the execs does that
23:16 EmilienM yeah there is something similar in centos
23:16 EmilienM hence the refreshonly to make sure it's idempotent
23:29 angdraug joined #puppet-openstack
23:30 johnmilton joined #puppet-openstack
23:40 haplo37 joined #puppet-openstack
23:48 yolanda joined #puppet-openstack
23:55 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Fix rabbitmq ssl logic  https://review.openstack.org/280462
23:55 EmilienM just a rebase ^

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