Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:12 gildub joined #puppet-openstack
00:35 markvoelker joined #puppet-openstack
00:35 xingchao joined #puppet-openstack
00:40 ducttape_ joined #puppet-openstack
00:47 xingchao joined #puppet-openstack
00:52 crinkle why are these different? http://git.openstack.org/cgit/openstack/puppet-keystone/tree/manifests/init.pp#n622 and http://git.openstack.org/cgit/openstack/puppet-neutron/tree/manifests/init.pp#n356
00:56 gildub joined #puppet-openstack
01:02 openstackgerrit zhongshengping proposed openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/276551
01:10 bkero joined #puppet-openstack
01:11 xingchao joined #puppet-openstack
01:11 bkero Hi there. I'm trying to run puppet-openstack-integration's all-in-one (or just run_tests.sh scenario003). I'm getting failure with "Error: nova --os-username admin --os-password a_big_secret --os-tenant-name openstack --os-auth-url http://127.0.0.1:5000/v2.0 flavor-create m1.micro 84 128 0 1 returned 1 instead of one of [0]"
01:12 bkero Could it be because I don't have cpu virtualization inside my kvm VM?
01:17 brandon joined #puppet-openstack
01:18 crinkle bkero: if you run the command manually you can get more information beyond 'returned 1'
01:18 bkero crinkle: Ah, good idea!
01:23 openstackgerrit zhongshengping proposed openstack/puppet-ironic: Add api_paste type/provider for Ironic  https://review.openstack.org/276565
01:23 * bkero adds kvm_intel.nested=1 just in case
01:27 xingchao joined #puppet-openstack
01:29 ducttape_ joined #puppet-openstack
01:32 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Fix rabbitmq ssl logic  https://review.openstack.org/280462
01:33 bkero Guh, seems to be something in cinder-volumes. Stderr: u'  Volume group "cinder-volumes" not found\n  Cannot process volume group cinder-volumes\n'
01:37 johnmilton joined #puppet-openstack
01:47 ducttape_ joined #puppet-openstack
01:55 bkero Also ImportError: No module named oslo.middleware
01:55 bkero :|
01:56 * bkero wondering if there's something different between my VM and the nodepool VMs
01:59 xingchao joined #puppet-openstack
02:14 openstackgerrit zhongshengping proposed openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/276551
02:15 bkero Uhh, I don't think it's a request_id compatibility shim, but that's the only occurrence of 'oslo.middleware' I can find.
02:36 markvoelker 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 baker joined #puppet-openstack
02:56 xingchao joined #puppet-openstack
03:08 fedexo joined #puppet-openstack
03:08 ducttape_ joined #puppet-openstack
03:27 llua joined #puppet-openstack
03:28 gildub joined #puppet-openstack
03:28 ducttape_ joined #puppet-openstack
03:33 bkero crinkle: Problem was my VM was unclean. Had delorean-kilo repo in it :(
04:14 xingchao joined #puppet-openstack
04:34 gildub joined #puppet-openstack
04:36 markvoelker joined #puppet-openstack
04:46 yottatsa joined #puppet-openstack
05:04 nlahouti joined #puppet-openstack
05:07 nlahouti1 joined #puppet-openstack
05:47 yottatsa joined #puppet-openstack
06:19 yottatsa joined #puppet-openstack
06:20 jaosorior joined #puppet-openstack
06:37 markvoelker joined #puppet-openstack
06:45 baker joined #puppet-openstack
07:37 rcernin joined #puppet-openstack
07:45 IvanBerezovskiy joined #puppet-openstack
07:46 aarefiev joined #puppet-openstack
07:46 jprs joined #puppet-openstack
07:46 Anticimex joined #puppet-openstack
07:47 sbadia joined #puppet-openstack
07:51 dgurtner joined #puppet-openstack
07:51 dgurtner joined #puppet-openstack
07:52 nlahouti joined #puppet-openstack
07:56 samuelBartel joined #puppet-openstack
08:13 rstarmer joined #puppet-openstack
08:14 rstarmer joined #puppet-openstack
08:18 mkarpin left #puppet-openstack
08:20 hjensas joined #puppet-openstack
08:21 mkarpin joined #puppet-openstack
08:21 yottatsa joined #puppet-openstack
08:22 jpena joined #puppet-openstack
09:12 rstarmer joined #puppet-openstack
09:24 paramite joined #puppet-openstack
09:45 xingchao joined #puppet-openstack
09:47 rstarmer joined #puppet-openstack
09:48 Aegil joined #puppet-openstack
10:14 openstackgerrit Merged openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/280251
10:14 openstackgerrit Merged openstack/puppet-neutron: Deprecate router[dhcp]_delete_namespaces parameter  https://review.openstack.org/276674
11:00 mkarpin left #puppet-openstack
11:14 xingchao joined #puppet-openstack
11:18 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
11:20 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/280181
11:34 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/280181
11:45 rbabyuk joined #puppet-openstack
11:58 rstarmer joined #puppet-openstack
12:01 dprince joined #puppet-openstack
12:06 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
12:10 baker joined #puppet-openstack
12:13 ericksonsantos joined #puppet-openstack
12:23 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Allow setting of public_addr from ::profile::mon and ::mon  https://review.openstack.org/280351
12:33 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow sending random kernel arguments to the IPA  https://review.openstack.org/280652
12:34 EmilienM good morning
12:38 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
12:38 johnmilton joined #puppet-openstack
13:01 EmilienM bkero: our integration jobs do not run on kilo
13:07 bdossant joined #puppet-openstack
13:12 ducttape_ joined #puppet-openstack
13:14 mflobo left #puppet-openstack
13:21 chem joined #puppet-openstack
13:32 rstarmer joined #puppet-openstack
13:38 chem mwhahaha: EmilienM could you review  https://review.openstack.org/#/c/272146/ when you've got 1 min ?  I'm going to request for the associated integration job to be merged as well
13:39 markvoelker joined #puppet-openstack
13:40 EmilienM chem: +2
13:40 chem EmilienM: cool
13:42 EmilienM degorenko: can you look https://review.openstack.org/#/c/277223/ please ,
13:43 degorenko EmilienM, sure
13:44 degorenko EmilienM, i have only one nit comment from 4 patch set
13:44 degorenko what do you think about it?
13:46 EmilienM degorenko: it's a problem fixed last week
13:46 EmilienM mistral package was broken
13:47 degorenko EmilienM, no, i'm about inherits + include for params class
13:47 EmilienM how the inherits stuff
13:47 EmilienM ok, I can drop it
13:47 degorenko EmilienM, wait
13:47 degorenko i'm looking for puppetlabs docs for this
13:47 degorenko it's just my opinion
13:53 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Allow to configure Murano Engine workers  https://review.openstack.org/280705
13:54 EmilienM degorenko: it's already like this in other modules
13:54 EmilienM degorenko: let's merge as it is, it does not break anything now
13:54 EmilienM we'll clean it later if we find that's not useful
13:54 degorenko EmilienM, ok
13:54 EmilienM thanks
14:07 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] allow sending random kernel arguments to the IPA  https://review.openstack.org/280652
14:09 openstackgerrit Merged openstack/puppet-mistral: Implement WSGI support for mistral-api  https://review.openstack.org/277223
14:09 yottatsa joined #puppet-openstack
14:17 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Enable to run boostrap without an upstream repo.  https://review.openstack.org/272146
14:17 jaosorior joined #puppet-openstack
14:18 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
14:21 openstackgerrit Merged openstack/puppet-manila: remove POSIX users, groups and file modes  https://review.openstack.org/276551
14:23 rstarmer joined #puppet-openstack
14:25 openstackgerrit Merged openstack/puppet-openstack_extras: auth file: default urls to keystone v3  https://review.openstack.org/280344
14:32 baker joined #puppet-openstack
14:33 openstackgerrit Merged openstack/puppet-ironic: Add api_paste type/provider for Ironic  https://review.openstack.org/276565
14:34 yottatsa joined #puppet-openstack
14:35 dfisher joined #puppet-openstack
14:36 mflobo joined #puppet-openstack
14:40 yottatsa joined #puppet-openstack
14:42 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate database configuration for ironic-inspector  https://review.openstack.org/279695
14:46 baker joined #puppet-openstack
14:50 EmilienM meeting in 10 min, agenda is here: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20160216
14:51 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/280742
14:55 aarefiev joined #puppet-openstack
14:55 aarefiev joined #puppet-openstack
14:57 ducttape_ joined #puppet-openstack
14:57 hjensas joined #puppet-openstack
14:57 hjensas joined #puppet-openstack
14:57 ducttape_ joined #puppet-openstack
14:59 morazi joined #puppet-openstack
15:02 gchamoul joined #puppet-openstack
15:07 baker joined #puppet-openstack
15:08 rstarmer joined #puppet-openstack
15:14 baker joined #puppet-openstack
15:17 openstackgerrit Merged openstack/puppet-openstack-integration: Use $default_domain for ::keystone::endpoint  https://review.openstack.org/280383
15:29 mdorman joined #puppet-openstack
15:31 rochaporto joined #puppet-openstack
15:32 bkero EmilienM: Yeah, was a mistake on my part. Unclean vm had an old repo that was pulling in packages.
15:34 EmilienM no problem
15:34 EmilienM bkero: we're in upstream meeting atm
15:35 xarses_ joined #puppet-openstack
15:36 bkero Ah ok. Don't have that one on my schedule.
15:37 EmilienM bkero: https://wiki.openstack.org/wiki/Meetings/Puppet
15:37 rochaporto EmilienM: i got late, but wanted to ask about https://review.openstack.org/#/c/275223/
15:37 bkero Thanks
15:37 rstarmer joined #puppet-openstack
15:38 rochaporto mostly ask if someone could have a look :) and the magnum-api one from nate which seems to be waiting
15:41 bkero EmilienM: Thanks again, I'll try to attend in the future.
15:47 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Replace defines for managing cinder types with providers  https://review.openstack.org/273513
16:06 haplo37 joined #puppet-openstack
16:49 dfisher joined #puppet-openstack
16:51 mdorman_ joined #puppet-openstack
17:03 nlahouti joined #puppet-openstack
17:03 nlahouti joined #puppet-openstack
17:04 dprince joined #puppet-openstack
17:05 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
17:09 rcernin joined #puppet-openstack
17:43 EmilienM degorenko: do you have progress on https://review.openstack.org/#/c/261326/ ?
17:43 degorenko EmilienM, it is in my activity list, but i don't have time for now, i will try to land it on this week
17:43 EmilienM no problem
17:43 EmilienM just checking
17:43 EmilienM if you need help let me know
17:44 degorenko ok, thanks
17:44 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
17:44 degorenko https://review.openstack.org/226862 is also under refactoring
17:44 rstarmer joined #puppet-openstack
17:45 delattec joined #puppet-openstack
17:46 dgurtner joined #puppet-openstack
17:46 dprince joined #puppet-openstack
18:00 rahulait joined #puppet-openstack
18:00 baker joined #puppet-openstack
18:03 delattec joined #puppet-openstack
18:08 xingchao joined #puppet-openstack
18:08 openstackgerrit Merged openstack/puppet-horizon: Leave an ability to trigger collectstatic for debian/ubuntu  https://review.openstack.org/280181
18:10 v1k0d3n joined #puppet-openstack
18:10 yottatsa joined #puppet-openstack
18:11 rahulait joined #puppet-openstack
18:17 baker joined #puppet-openstack
18:19 yottatsa joined #puppet-openstack
18:22 bkero Hi #puppet-openstack people. I'm working on adding some RDO support for puppet-openstack-integration's scripts. Before I start a review for this, I thought I'd share this here. Is the upstream interested in taking changes that would facilitate this? http://paste.openstack.org/show/487154/
18:29 dmsimard bkero: what's the objective ?
18:29 bkero dmsimard: To be able to use puppet-openstack-integration and it's scenarios with delorean's openstack-puppet-modules.rpm
18:37 dmsimard I'm not personally a fan of putting something like that upstream, it's kind of perverting it with downstream things
18:37 baker joined #puppet-openstack
18:38 EmilienM right, things like this should be discussed on #rdo channel
18:39 bkero Okay
18:40 EmilienM puppet-openstack-integration is for upstream CI
18:40 EmilienM if downstream projects want to use it, but we won't add specific bits in there.
18:41 gfidente joined #puppet-openstack
18:41 EmilienM AFIK RDO is already using this repo as it is, and it works fine
18:41 EmilienM dmsimard: ^
18:41 EmilienM err, bkero ^ ( dmsimard did the work )
18:42 bkero Ah, got it
18:42 EmilienM bkero: they use puppet-openstack-integration to gate all RDO packages
18:42 EmilienM thanks for that, our upstream CI tend to never breaks
18:42 EmilienM (because of RDO packaging)
18:43 dmsimard It was made to be generic and usable outside of the context of RDO or specific bits, Ubuntu/Canonical could use the tests with their own internal repositories if they wanted to
18:43 EmilienM that would be nice
18:43 EmilienM but I haven't seen them interested by doing that
18:43 EmilienM AFIK they gate on their juju charms
18:43 dmsimard yeah I don't know how they test things
18:43 dmsimard oh ew
18:43 bkero dmsimard: You mean by using the specific fixtures, but not the all-in-one/run-tests.sh?
18:43 EmilienM which is very limited to a few services
18:45 dmsimard bkero: this is what we re-use puppet-openstack-integration tests in RDO CI https://github.com/openstack/puppet-openstack-integration/commit/28007a665d8488b0c3d0a629b2423bcd5e26d426
18:45 dmsimard So anyone can configure their own repositories prior to running integration tests
18:46 EmilienM I think that's what bkero is looking at.
18:46 bkero Right, that makes sense
18:46 dmsimard Adding a feature upstream to essentially add support for installing forks of the upstream modules it's meant to test is a bit awkward
18:47 bkero dmsimard: It's essentially adding support to disable the Puppetfile fetch
18:47 dmsimard bkero: so add a switch to disable that, then
18:48 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/254548
18:48 dmsimard but don't bake the fork installation right in
18:48 dmsimard and do that in your job/test environment like RDO does
18:48 bkero Okay
18:49 dmsimard also, the manage_puppet_repo switch is probably not necessary, yum-plugin-priorities should make yum pick the right package just like when EPEL is enabled
18:49 bkero That would also work, thank you
18:52 baker joined #puppet-openstack
18:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/280350
18:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - do not merge  https://review.openstack.org/280883
18:56 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Do not merge - CI test  https://review.openstack.org/280385
18:57 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test - do not merge  https://review.openstack.org/280884
18:58 EmilienM dgurtner: if you don't mind, I might need your guidance on a CI patch I'm working on, to deploy ceph: https://review.openstack.org/#/c/279826/3/manifests/ceph.pp
18:58 yottatsa joined #puppet-openstack
19:01 nlahouti joined #puppet-openstack
19:02 baker joined #puppet-openstack
19:03 yottatsa joined #puppet-openstack
19:13 baker joined #puppet-openstack
19:19 rahulait joined #puppet-openstack
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/279349
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::messaging::mongodb  https://review.openstack.org/279348
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/279351
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/279350
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/279747
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::management::mongodb  https://review.openstack.org/279347
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/279746
19:23 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Implement zaqar::init class  https://review.openstack.org/279346
19:27 jtomasek_ joined #puppet-openstack
19:29 baker joined #puppet-openstack
19:31 EmilienM dprince: where is used python_redis_package_nam in https://review.openstack.org/279346 ?
19:32 dprince EmilienM: oh, I didn't submit that patch yet did I? Sorry.
19:33 dprince EmilienM: i can remove it too
19:33 EmilienM thanks
19:33 EmilienM lgtm otherwise
19:33 dprince EmilienM: I have redis patches... but they can come after this
19:34 EmilienM dprince: wait then
19:34 EmilienM we can land it as it is, we are bootstraping the module anyway
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/279349
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::messaging::mongodb  https://review.openstack.org/279348
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/279351
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/279350
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/279747
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::management::mongodb  https://review.openstack.org/279347
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/279746
19:34 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Implement zaqar::init class  https://review.openstack.org/279346
19:34 EmilienM too late
19:34 EmilienM boom
19:35 dprince EmilienM: oh, sorry. Just caught your message
19:35 EmilienM :) no problem
19:35 dprince EmilienM: it is fine. I can re-add it
19:35 EmilienM looking again, and I'll +2
19:35 EmilienM dprince: it's clean now!
19:35 EmilienM dprince: +2 on init patch
19:38 EmilienM dprince: quick comment on https://review.openstack.org/#/c/279349/
19:40 EmilienM dprince: and https://review.openstack.org/#/c/279350/
19:43 EmilienM dprince: all reviewed
19:49 haplo37 joined #puppet-openstack
19:49 guimaluf joined #puppet-openstack
19:56 dhollinger joined #puppet-openstack
19:56 baker joined #puppet-openstack
20:00 dprince EmilienM: I used enherits because before I was defaulting parameters based on what was set in params.pp
20:00 dprince EmilienM: now that I removed that though the include you suggest is fine. I will switch to that
20:00 EmilienM ok
20:11 jfluhmann joined #puppet-openstack
20:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/280350
20:32 EmilienM _ody: we're still having a warning on puppet-vswitch http://logs.openstack.org/42/280742/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/8f52da4/console.html#_2016-02-16_15_38_27_214 -- any idea?
20:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Update manifests to clean-up warnings  https://review.openstack.org/280919
20:38 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server class  https://review.openstack.org/279349
20:38 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/279351
20:38 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::server_instance class.  https://review.openstack.org/279350
20:38 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::wsgi class  https://review.openstack.org/279747
20:38 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add zaqar::transport::websocket class  https://review.openstack.org/279746
20:44 openstackgerrit Dan Prince proposed openstack/puppet-zaqar: Add acceptance spec for zaqar with mongo.  https://review.openstack.org/279351
20:49 brandon joined #puppet-openstack
20:53 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: ceph: deploy keys & pools  https://review.openstack.org/279826
21:05 openstackgerrit Colleen Murphy proposed openstack/puppet-keystone: Add keystone::disable_admin_token_auth class  https://review.openstack.org/280929
21:06 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Add project_name parameter for auth_file  https://review.openstack.org/280930
21:06 rahulait joined #puppet-openstack
21:10 _ody EmilienM: Different than the one I fixed.  I'll take a quick look.
21:10 EmilienM maybe something we introduced recently, I'm aware we merged patches lately
21:13 rahulait joined #puppet-openstack
21:13 _ody EmilienM: I see the issue, looks like I fixed something else but I am confused as to why I didn't see this issue when I fixed the last one.
21:14 EmilienM cool :)
21:15 _ody I'll go ahead and fix.
21:15 EmilienM thanks
21:20 brandon_ joined #puppet-openstack
21:21 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: Drop User/group resource dependency  https://review.openstack.org/195684
21:27 nlahouti joined #puppet-openstack
21:29 openstackgerrit Cody Herriges proposed openstack/puppet-vswitch: Remove duplicate loading of constants  https://review.openstack.org/280937
21:29 _ody EmilienM: That should fix that other warning but...I just noticed, this module has near no tests.
21:29 EmilienM we have integration tests
21:30 _ody Yeah.  We'll have to rely on that.
21:31 EmilienM that's how I saw the WARNING
21:51 EmilienM crinkle: I might need your help on https://review.openstack.org/#/c/280350/ -- looking at zuul, it's still failing. Any idea of what I'm doing wrong?
21:55 crinkle EmilienM: need to set project_name => 'openstack'
21:55 EmilienM crinkle: it's default, isn't?
21:56 EmilienM crinkle: https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/auth_file.pp#L96
21:56 crinkle EmilienM: oh it wasn't before
21:58 EmilienM yeah I did it, you reviewed it I think
22:00 crinkle EmilienM: i'm not sure then, we might need to set OS_IDENTITY_API_VERSION=3 but then i'm not sure why the keystone one was okay
22:00 EmilienM crinkle: identity_api_version is set to 3 by default already https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/auth_file.pp#L108
22:00 crinkle welp
22:01 * EmilienM goes on http://www.urbandictionary.com/define.php?term=Welp
22:02 nibalizer joined #puppet-openstack
22:06 crinkle EmilienM: it's failing on glance_image which wasn't tested by the keystone module :(
22:07 EmilienM it's also failing on neutron I think and also tempest
22:08 EmilienM that's why I'm adding your new class in our integ repo
22:08 EmilienM to test it like you would have in prod
22:08 EmilienM crinkle: did you test it in infra cloud?
22:09 crinkle EmilienM: nope
22:09 EmilienM it is maybe an orchestration issue, I'm currently trying to reproduce the bug in my vm
22:11 EmilienM crinkle: the weird thing is, your new class has no effect on centos, because /etc/keystone/keystone-paste.ini does not exist
22:12 EmilienM crinkle: (it's in RDO master now, so it will exist soon)
22:12 EmilienM so the problem is really in the openrc
22:13 crinkle i'm running glance func tests with the new keystone class to see if i can reproduce
22:19 rstarmer joined #puppet-openstack
22:20 EmilienM crinkle: so I can upload the image, bot not list it
22:20 EmilienM oh wait, no image create also fails sorry
22:20 EmilienM got 400
22:21 rahulait joined #puppet-openstack
22:21 EmilienM crinkle: "Expecting to find domain in project"
22:23 EmilienM crinkle: I found it
22:24 EmilienM I'm testing it before
22:25 EmilienM yeah it works
22:25 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: authfile: fix OS_IDENTITY_API_VERSION  https://review.openstack.org/280960
22:25 EmilienM crinkle: ^
22:25 crinkle oh ha
22:26 EmilienM sorry for that
22:26 EmilienM I'll RTFM next time
22:27 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: disabling the admin_token after bootstrap  https://review.openstack.org/280350
22:29 dfisher FYI: I just pushed v6.1.0 of cinder, glance, heat, horizon, ironic, keystone, neutron, nova, swift and openstacklib into Solaris 12
22:30 openstackgerrit Ben Kero proposed openstack/puppet-openstack-integration: Add ability to pass puppet args in as env var  https://review.openstack.org/280964
22:33 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Remove keystone_disable_admin_token_auth beaker test  https://review.openstack.org/280385
22:33 openstackgerrit Ben Kero proposed openstack/puppet-openstack-integration: Add ability to manually install puppet modules  https://review.openstack.org/280965
22:35 rahulait joined #puppet-openstack
22:37 dgurtner EmilienM: ping re: https://review.openstack.org/#/c/279826/
22:37 EmilienM dgurtner: hey
22:38 EmilienM weird it fails, while it works on my laptp
22:38 dgurtner EmilienM: it fails due to lack of idempotency, no?
22:38 dgurtner EmilienM: I think this http://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bcc77e6/console.html#_2016-02-16_21_53_17_540
22:38 EmilienM I'm checking
22:39 EmilienM http://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/6f3b383/console.html#_2016-02-16_21_29_46_962
22:39 dgurtner EmilienM: and this http://logs.openstack.org/26/279826/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/bcc77e6/console.html#_2016-02-16_21_53_17_722
22:39 EmilienM looking both links now
22:40 dgurtner EmilienM: the module actually lacks idempotency there
22:40 EmilienM dgurtner: right we have a bug in puppet-ceph about idempotency
22:40 EmilienM no prob, we'll fix it :-)
22:40 dgurtner EmilienM: when using cephx a file resource gets created and later the file gets removed via rm
22:40 EmilienM the good news is, it's only for one Exec
22:41 EmilienM dgurtner: why do we remove it?
22:41 dgurtner it's a temporary file in /tmp/ which contains the key
22:42 EmilienM we need to fix the "unless"
22:43 dgurtner https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L116-L119
22:43 dgurtner https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L180-L186
22:43 dgurtner first adds the file, second one removes it
22:43 EmilienM dgurtner: I'm more concerned about https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L182
22:43 EmilienM why do we have /bin/true
22:44 EmilienM that's terrible no?
22:44 EmilienM we need to cleanup the file after the keyring creation, isn't?
22:44 dgurtner after monitor bootstrap
22:44 EmilienM dfisher: here https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L159 ?
22:45 dgurtner the problem is that the "keyring_path" file shouldn't exist, but needs to be there
22:45 dfisher EmilienM: pardon?
22:45 dfisher you might have meant 'dgurtner' :)
22:45 EmilienM dfisher: sorry wrong nickname.
22:45 dfisher no worries!
22:45 EmilienM I'm so lazy, I just type "d" and tab
22:46 dfisher :)
22:46 EmilienM dgurtner: can we cleanup in another place?
22:46 dgurtner we probably shouldn't have a file resource on https://github.com/openstack/puppet-ceph/blob/master/manifests/mon.pp#L116
22:47 EmilienM or we can do something else
22:47 EmilienM in the rm-keyring-${id} Exec
22:48 EmilienM add a refreshonly => true
22:48 dgurtner not really, the file shouldn't exist
22:48 EmilienM so this exec will be executed only if we have a change in the keyring
22:48 dgurtner the problem is the file resource will just recreate it
22:49 dgurtner and that file will always contain the monitor key, which is a security issue
22:52 EmilienM dgurtner: we can still create an exec instead of FIle resource
22:52 EmilienM so we can add conditionals to manage it
22:53 EmilienM dgurtner: or stop using /tmp
22:53 EmilienM maybe use /etc/ceph/tmp ?
22:57 EmilienM dgurtner: wdyt?
22:57 dgurtner the solution is probably something ugly like this: http://paste.openstack.org/show/487200/
22:57 dgurtner EmilienM: ^
22:57 EmilienM yes
22:58 EmilienM it's ok
22:58 EmilienM dgurtner: let's do it
22:58 EmilienM dgurtner: you want to send a patch ? or you want me to do it?
22:59 dgurtner EmilienM: I can send it. the problem is if I send patches we need 2 of the other core-reviewers to approve ;)
22:59 dgurtner EmilienM: or in all fairness, given I already wrote this, we probably need 2 anyways
23:00 EmilienM this is not a critical bug, and not very urgent. So we can still ping xarses and dmsimard for a review tomorrow
23:00 EmilienM dgurtner: I'll let you send it, and I'll review / test it in our CI
23:00 dgurtner EmilienM: works for me
23:00 EmilienM if you can do it, otherwise I'll push it, I have a bit of time now
23:02 xarses ?
23:02 xarses which now?
23:02 EmilienM xarses: dgurtner is about to send a patch, to fix idempotency in ceph::profile::mon class
23:02 xarses ok
23:02 EmilienM we found a bug
23:03 xarses Ya, I'm not surprised =(
23:03 EmilienM but AFIK that's the last bug I've seen that is blocking our CI
23:04 EmilienM other than that, I can deploy puppet-ceph, create pools with idempotency. Which is cool
23:06 xarses yay!
23:09 nlahouti joined #puppet-openstack
23:21 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Fix idempotency when bootstrapping ceph::mon with cephx  https://review.openstack.org/280984
23:22 dgurtner EmilienM: xarses ^ I want to add an acceptance test which uses cephx to test idempotency on this
23:22 dgurtner apart from that I think this should do the trick
23:23 xarses dgurtner: ok, cephx was only being tested integration in rgw / swift
23:23 dgurtner xarses: but probably not with idempotency
23:23 EmilienM it sounds good to me, I'll test it
23:24 xarses there was some acceptance tests that worked with it, but thats the only integration
23:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: ceph: deploy keys & pools  https://review.openstack.org/279826
23:24 EmilienM dgurtner: ^ testing in progress
23:27 _ody_ joined #puppet-openstack
23:27 dgurtner my hotel has random internet issues, so I'm currently having issues testing locally as it times out downloading gems :/
23:28 brandon joined #puppet-openstack
23:28 mdorman_ joined #puppet-openstack
23:30 mwhahaha_ joined #puppet-openstack
23:30 reynir1 joined #puppet-openstack
23:30 rhefner_ joined #puppet-openstack
23:32 Mosibi_ joined #puppet-openstack
23:33 kgalanov_ joined #puppet-openstack
23:35 itsuugo_ joined #puppet-openstack
23:36 EmilienM_ joined #puppet-openstack
23:36 clayton_ joined #puppet-openstack
23:38 Hunner joined #puppet-openstack
23:38 Hunner joined #puppet-openstack
23:38 Guest28836 crinkle: CI still does not pass on https://review.openstack.org/#/c/280350/ -- while it worked in my env... I'm a bit lost
23:39 crinkle Guest28836: :'(
23:39 Guest28836 damn my nick is broken
23:41 Guest28836 joined #puppet-openstack
23:42 EmilienM crinkle: it was me ! lol
23:43 mdavidson joined #puppet-openstack
23:47 dalees joined #puppet-openstack
23:47 nibalizer joined #puppet-openstack
23:55 johnmilton joined #puppet-openstack

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