Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-07-28

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

All times shown according to UTC.

Time Nick Message
00:50 xarses joined #puppet-openstack
01:26 ducttape_ joined #puppet-openstack
01:39 mfisch joined #puppet-openstack
01:39 mfisch joined #puppet-openstack
01:52 openstackgerrit Matt Fischer proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/344464
02:04 ducttape_ joined #puppet-openstack
02:27 ayoung joined #puppet-openstack
02:29 bklei joined #puppet-openstack
02:35 ducttape_ joined #puppet-openstack
02:36 ayoung joined #puppet-openstack
02:40 rwsu joined #puppet-openstack
02:50 ducttape_ joined #puppet-openstack
03:01 guessi joined #puppet-openstack
03:15 iurygregory joined #puppet-openstack
03:28 alee_afk joined #puppet-openstack
03:44 guessi joined #puppet-openstack
03:50 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
03:51 ducttape_ joined #puppet-openstack
03:57 guimaluf joined #puppet-openstack
05:53 ducttape_ joined #puppet-openstack
06:22 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/348140
06:22 openstackgerrit Ade Lee proposed openstack/puppet-tempest: Add unit test for tempest_roles  https://review.openstack.org/348141
06:42 tesseract- joined #puppet-openstack
06:54 ducttape_ joined #puppet-openstack
07:02 jtomasek joined #puppet-openstack
07:06 atom_ joined #puppet-openstack
07:07 openstackgerrit Xiaohua Yuan proposed openstack/puppet-nova: add missing options for monkey_patch  https://review.openstack.org/346494
07:13 permalac joined #puppet-openstack
07:14 hjensas joined #puppet-openstack
07:25 danpawlik joined #puppet-openstack
07:37 dgurtner joined #puppet-openstack
07:37 dgurtner joined #puppet-openstack
07:52 alee_afk joined #puppet-openstack
08:00 shardy joined #puppet-openstack
08:14 derekh joined #puppet-openstack
08:26 permalac joined #puppet-openstack
08:34 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
08:37 paramite joined #puppet-openstack
08:37 paramite joined #puppet-openstack
08:56 ducttape_ joined #puppet-openstack
09:16 openstackgerrit Karthik S proposed openstack/puppet-neutron: WIP:Configure the max_vfs for SRIOV interfaces  https://review.openstack.org/344918
09:44 chem is there someone from mirantis that could confirm that error from here https://review.openstack.org/#/c/347982/2 and there https://review.openstack.org/#/c/347980/1 are transiants
09:55 gfidente joined #puppet-openstack
09:56 ducttape_ joined #puppet-openstack
10:22 EmilienM hello
10:23 EmilienM iberezovskiy: hey
10:23 iberezovskiy hey
10:23 EmilienM could we fix fuel asap?
10:23 iberezovskiy sure
10:23 iberezovskiy I'm on it
10:23 EmilienM thx
10:23 iberezovskiy thx you for patience
10:36 openstackgerrit Saravanan KR proposed openstack/puppet-openstack-integration: Add puppet-kmod modules to the integration  https://review.openstack.org/348222
10:40 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile  https://review.openstack.org/343346
10:59 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
11:02 openstackgerrit Saravanan KR proposed openstack/puppet-openstack-integration: Add puppet-kmod modules to the integration  https://review.openstack.org/348222
11:03 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
11:06 alee degorenko, reading your comments ..
11:07 alee degorenko, if I understand your idea correctly - we could pass the tempest_roles as a parameter to openstack_integration::tempest in scenario002
11:08 alee degorenko, that would allow us to move the role creation to barbican.py and swift.py accordingly ..
11:08 degorenko alee, right
11:08 alee rather than creating the roles each time.  yeah - I like that - will try it
11:12 openstackgerrit Merged openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/348007
11:12 openstackgerrit Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/348140
11:14 openstackgerrit Merged openstack/puppet-tempest: Add unit test for tempest_roles  https://review.openstack.org/348141
11:15 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/348248
11:15 EmilienM degorenko, iberezovskiy: could you review ^ please?
11:17 degorenko EmilienM, why not cherry-pick with adaptation?
11:17 EmilienM degorenko: we can't
11:17 degorenko EmilienM, why?
11:17 EmilienM degorenko: it's outstanding
11:17 EmilienM because code is really not the same
11:17 EmilienM we don't use puppet-oslo :)
11:17 EmilienM in mitaka
11:17 degorenko i know
11:17 degorenko it should be adaptation :)
11:18 degorenko ok
11:18 degorenko let it go as it now
11:18 EmilienM thanks
11:18 EmilienM degorenko: we found a bug in tripleo
11:18 EmilienM where nova console is really unstable if you don't use the options
11:18 EmilienM maybe in fuel you have the bug too
11:20 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/348248
11:23 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
11:24 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
11:26 degorenko EmilienM, waiting for ci and will +2
11:27 EmilienM degorenko: you rocks
11:41 alee degorenko, EmilienM - whats the difference between the regular and non-voting gate jobs?
11:41 EmilienM non voting jobs don't vote
11:41 EmilienM and regular jobs vote
11:41 EmilienM was it the question?
11:42 alee EmilienM, thanks -- what I meant was integration-4 vs integration-3
11:42 EmilienM ah
11:42 EmilienM the version of puppet
11:42 EmilienM integration-4 is puppet4
11:42 EmilienM and other is puppet4
11:42 EmilienM and other is puppet3 err
11:43 EmilienM alee: did you make progress on barbican?
11:43 skramaja joined #puppet-openstack
11:43 alee EmilienM, ok - so it seems that the failure on puppet 4 for the barbican test is due to a puppet-4/3 problem ..
11:43 EmilienM looking
11:44 alee EmilienM, yeah - I just waited a bit, did a few rechecks and things started working
11:44 alee EmilienM, goign to make changes as suggested by degorenko
11:45 EmilienM yeah I think it was a cache issue in the nodepool images
11:45 EmilienM and degorenko reviewed the patch
11:45 EmilienM his remark is good
11:45 EmilienM it's a puppet4 thing I think I'm investigating
11:45 alee EmilienM, the puppet4 failure looks like a permissions issue
11:45 EmilienM no, it's ordering I think
11:46 EmilienM let me a sec
11:46 alee ok
11:53 openstackgerrit Emilien Macchi proposed openstack/puppet-barbican: db: remove useless pick  https://review.openstack.org/348267
11:58 karthiks joined #puppet-openstack
11:58 ducttape_ joined #puppet-openstack
12:00 morazi joined #puppet-openstack
12:01 EmilienM alee: let's try again
12:01 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
12:02 openstackgerrit Merged openstack/puppet-oslo: cache: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347980
12:02 openstackgerrit Merged openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347982
12:03 alee EmilienM, ok -- I have be out for a few hours .. I'll make the degorenko changes when I return.
12:03 EmilienM alee: what need?
12:04 EmilienM degorenko: let me know if you want us to move things in the barbican patch :)
12:04 ducttape_ joined #puppet-openstack
12:04 alee EmilienM, if you're good as-is, then I'm not going to complain.
12:05 alee EmilienM, otherwise I'm hoping we can finally merge this :)
12:05 EmilienM it would be awesome
12:05 EmilienM we just don't want to break puppet4 CI
12:05 alee EmilienM, I'll check in when I get back ..
12:08 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
12:09 ducttape_ joined #puppet-openstack
12:10 alee_afk joined #puppet-openstack
12:10 morazi joined #puppet-openstack
12:13 openstackgerrit Merged openstack/puppet-horizon: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347981
12:13 iurygregory morning people
12:15 bogdando joined #puppet-openstack
12:16 EmilienM iurygregory: o/
12:16 iurygregory \o
12:21 colonwq EmilienM, For RGW/KeystonV3, the initial answer is 'yes' to need pki support for revocation lists only. It is not for the token itself.
12:21 colonwq I'm still waiting for a final confirmation.
12:22 EmilienM colonwq: oki
12:24 bogdando joined #puppet-openstack
12:26 openstackgerrit Merged openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/328347
12:27 openstackgerrit Xiaohua Yuan proposed openstack/puppet-nova: add missing options for monkey_patch  https://review.openstack.org/346494
12:35 degorenko EmilienM, ok, will look
12:36 degorenko EmilienM, typo in patch
12:37 EmilienM degorenko: arg
12:37 EmilienM where?
12:37 degorenko EmilienM, in barbican class
12:39 iurygregory degorenko, tks for the review i'll update after lunch =)
12:40 degorenko iurygregory, sure
12:41 EmilienM degorenko: looking
12:41 EmilienM degorenko: lol wtf
12:41 EmilienM thx!
12:42 degorenko EmilienM, also commented here https://review.openstack.org/#/c/348248
12:43 EmilienM degorenko: awesome
12:44 EmilienM degorenko: /me running tests this time
12:45 degorenko EmilienM, /me running tests every time :D
12:45 EmilienM degorenko: yeah, not me... shame!
12:51 openstackgerrit Merged openstack/puppet-barbican: db: remove useless pick  https://review.openstack.org/348267
12:54 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Configure Ceph RGW to use Keyston V3  https://review.openstack.org/348293
13:12 ericksonsantos :set nu
13:12 ericksonsantos ops
13:13 richm joined #puppet-openstack
13:13 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/332854
13:13 openstackgerrit Merged openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/347315
13:14 openstackgerrit Merged openstack/puppet-tripleo: Remove global openstack-core resource  https://review.openstack.org/347472
13:16 chem|lunch joined #puppet-openstack
13:16 gfidente joined #puppet-openstack
13:16 gfidente joined #puppet-openstack
13:16 ducttape_ joined #puppet-openstack
13:17 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/348248
13:30 markvoelker joined #puppet-openstack
13:36 dfisher joined #puppet-openstack
13:38 karthiks joined #puppet-openstack
13:42 EmilienM iurygregory: you making good progress with authtoken? you need help?
13:42 EmilienM iurygregory: I would like us to finish this task before end of Newton
13:42 degorenko EmilienM, how many time we have?
13:43 ayoung joined #puppet-openstack
13:43 morazi joined #puppet-openstack
13:43 ducttape_ joined #puppet-openstack
13:43 EmilienM 1 months and some days
13:43 degorenko ah, ok
13:43 EmilienM but we have a lot of modules :)
13:43 degorenko yeah, sure
13:43 degorenko just wondering about time
13:44 ducttape_ joined #puppet-openstack
13:44 EmilienM degorenko: releases.openstack.org/newton/schedule.html
13:44 shardy joined #puppet-openstack
13:44 EmilienM Feature freeze Aug 29-02
13:45 EmilienM I want to keep September for stabilization
13:45 EmilienM and bug fix
13:45 degorenko yep, sure, we will
13:46 EmilienM degorenko: about https://review.openstack.org/#/c/346685/
13:46 EmilienM don't we want to wait until we have one module done ? (ie barbican)?
13:46 EmilienM to make sure it's consistent
13:46 degorenko EmilienM, it uses same approach as barbican, isnt it?
13:46 EmilienM I haven't reviewed it
13:46 EmilienM I'm doing now
13:47 degorenko EmilienM, if you feel, that it should be postponed on some time - just one your word sir :)
13:47 EmilienM degorenko: no, it's fine
13:48 degorenko iurygregory, don't forget about zaqar :) https://review.openstack.org/#/c/346544/
13:52 iurygregory EmilienM, can you review barbican to see if need any changes besides the ones pointed by degorenko and alee_afk ? https://review.openstack.org/#/c/347105
13:52 iurygregory degorenko, sure
13:53 dfisher chem|lunch: regarding https://review.openstack.org/#/c/347463/ and https://review.openstack.org/#/c/347468/ - it has something to do with openstack_setting's exists? method.  The l3_agent file I'm working with comments out router_id so exists? is returning false which appears to short-circuit the transform methods
13:53 EmilienM I'm not available during the next 60 minutes
13:53 degorenko EmilienM, iurygregory i have question, regarding our authtoken things - related to glance
13:53 degorenko glance has 3 services, which needs auth parameters (glare, api, registry)
13:54 degorenko where i should put authtoken params?
13:54 degorenko not sure, that creating 3 classes under keystone directory is good idea
13:55 degorenko i guess all of them should be created in appropriate dirs?
13:59 EmilienM mhh, make them common
13:59 EmilienM ok /me away now :P
14:00 degorenko all of them have different configs
14:00 skramaja joined #puppet-openstack
14:04 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
14:04 iurygregory degorenko, i'm in a meeting right now i'll look in a few minutes =)
14:05 degorenko sure
14:18 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/346685
14:25 openstackgerrit Drew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/347463
14:34 shardy joined #puppet-openstack
14:40 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/347739
14:40 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo  https://review.openstack.org/347250
14:40 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo  https://review.openstack.org/346975
14:40 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes  https://review.openstack.org/347385
14:40 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail config puppet module for tripleO  https://review.openstack.org/346938
14:40 chem joined #puppet-openstack
14:46 dfisher chem: latest bits solve the issue i was seeing where the first run didn't work.  Not sure it's the completely correct thing to do, however
14:46 chem dfisher: I'm going to have a look
14:49 chem dfisher: nice catch.  That makes a lot of sense. the "value=" is only for modification, and "create" is the right one for creation.
14:49 dfisher i hardcoded the direction to be :to though
14:50 dfisher not sure if I need to get into some kind of begin/rescue thing
14:50 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail database puppet module for tripleO  https://review.openstack.org/348406
14:51 chem dfisher: for create that's the :to that we need, so that's good.  it's only for retrieving the info that we need the :from
14:51 * dfisher nods
14:51 chem dfisher: for begin/rescue, well if that's failing it's for a good reason, so let it fail.
14:51 dfisher chem: ok.
14:52 xarses joined #puppet-openstack
14:53 chem dfisher: so now first run create and second run does nothing ?
14:53 dfisher checking ...
14:55 dfisher chem: http://paste.openstack.org/show/543288/  it looks like the second run does something but it doesn't change the value.
14:55 EmilienM iurygregory, degorenko: ok back. I'll review it shortly
14:56 chem dfisher: run it with --detailed-exitcode and check $?, should be 0, but it looks like it's 2.
14:56 mwhahaha 0 or 2
14:56 chem dfisher: something is still missing
14:56 mwhahaha 2 means success and things changed
14:57 mwhahaha 0 is nothing changed
14:57 chem 2
14:57 chem must be 0, but it's going to be 2
14:57 dfisher $ echo $?
14:57 dfisher 2
14:57 chem dfisher: so there is something still missing to make it work completly right
14:58 chem dfisher: we should have 0 here
14:58 dfisher chem: nod.  will keep poking at it...
14:58 mwhahaha looks like you might need to trim the line break
14:59 dfisher not entirely sure where :)
14:59 mwhahaha where it's getting read from initially would be my guess
14:59 mwhahaha cause you're setting 'provider' but it thought it was 'provider\n'
15:00 dfisher right.
15:01 vinsh joined #puppet-openstack
15:01 chem dfisher: ah, yes that could be.  I added a comment on the review  as well, but try first mwhahaha's idea
15:01 mwhahaha probably could just do it in teh value= call
15:01 dfisher so in python, you can use %r to print out a raw string:
15:01 dfisher >>> a = "\n  \tfoo"
15:01 dfisher >>> print a
15:01 dfisher foo
15:01 dfisher >>> print "%r" % a
15:01 dfisher '\n  \tfoo'
15:01 dfisher what's the ruby equivalent of that?
15:02 chem dfisher: add .chop
15:03 chem dfisher: "foo\n".chop -> "foo"
15:03 * dfisher nods
15:09 dfisher it ended up being in the from_uuid() method in neutron_l3_agent_config/ini_settings.rb
15:09 dfisher $ sudo puppet apply --detailed-exitcode test.pp
15:09 dfisher Notice: Compiled catalog for openstack-x52-26.us.oracle.com in environment production in 0.10 seconds
15:09 dfisher Notice: Finished catalog run in 1.28 seconds
15:09 dfisher $ echo $?
15:09 dfisher 0
15:09 jmiu_ joined #puppet-openstack
15:10 chem dfisher: so we're good :)
15:10 openstackgerrit Sanjay Upadhyay proposed openstack/puppet-neutron: sriov check for physical_device_mappings and exclude_devices  https://review.openstack.org/333465
15:10 dfisher i think so...
15:10 dfisher redoing from scratch
15:10 chem dfisher: push it so that the ci can play with it as well :)
15:10 dfisher will in a sec.
15:11 dfisher first run exits with 2 (successful but with changes).    second exits with 0
15:11 dfisher exactly how we want it
15:11 chem dfisher: yep, super nice work!
15:12 openstackgerrit Drew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/347463
15:13 openstackgerrit Drew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/347468
15:13 dfisher ok, there's both
15:13 iurygregory degorenko, we have release for barbican (newton 9.0.0, 9.1.0)
15:13 degorenko iurygregory, oh, right
15:14 iurygregory should I break or not? =(
15:14 degorenko iurygregory, but next question about - auth_type and auth_strategy
15:14 iurygregory let me see
15:14 chem dfisher: I was wondering if we put that up in the transform method directly but I think that letting the client decide what it has to do with the data is better
15:14 degorenko we are using auth_type in authtoken
15:14 chem dfisher: wdyt ?
15:15 dfisher chem: the chop?
15:15 chem dfisher:yep
15:15 dfisher i think it's a bug with neutron client, tbh
15:15 dfisher but i think the client should probably decided how to handle it
15:15 iurygregory degorenko, auth_type in api should be auth_strategy or 'barbican_auth_type'
15:16 chem dfisher: ack, so let it as it is.
15:16 iurygregory keystone::authtoken have auth_type because we need to select what authentication we will use...
15:18 degorenko iurygregory, why? we don't setting it in config
15:18 openstackgerrit joined #puppet-openstack
15:19 iurygregory degorenko, we don't set auth_type?
15:19 degorenko iurygregory, where? i don't see, only if
15:20 EmilienM iurygregory, degorenko: sorry wa sa bit away
15:20 EmilienM yes you can break puppet-barbican
15:21 EmilienM there AFIK no user now
15:21 iurygregory EmilienM, ack
15:21 EmilienM fuel, tripleo, are not using it
15:21 dfisher chem: both to/from need that chop.  for some reason, setting it initially seems to strip the value.
15:21 EmilienM and AFIK no bug report
15:21 EmilienM iurygregory: so break it and make it beautifule
15:21 EmilienM -e
15:21 iurygregory so there is no 'auth_type' anymore for api.pp =P
15:21 degorenko iurygregory, anyway, why you want change auth_type (in barbican meaning) to auth_strategy? i don't see any reasons
15:22 iurygregory is not the correct name =)
15:22 EmilienM what is important is: keep puppet module able to deploy barbican WITHOUT keystone
15:22 iurygregory auth_strategy make more sense than auth_type for barbican
15:22 iurygregory because we choose keystone or any other thing
15:22 degorenko we have auth_type everywhere
15:23 degorenko i don't see any pros for this renaming
15:24 iurygregory degorenko, http://git.openstack.org/cgit/openstack​/puppet-neutron/tree/manifests/init.pp
15:24 iurygregory the auth_type in barbican is the same idea for auth_strategy in neutron
15:24 iurygregory and auth_type is a configuration for keystone_authtoken
15:25 degorenko ok
15:25 degorenko but we don't have stable branches
15:25 degorenko so i guess we can just rename then
15:26 iurygregory degorenko, yeah i'll do this ;)
15:40 iurygregory EmilienM, since barbican was not released i need to add a release note?
15:41 ducttape_ joined #puppet-openstack
15:41 EmilienM iurygregory: no
15:41 EmilienM consider it like a new module
15:41 iurygregory EmilienM, sure ;)
15:43 dfisher EmilienM: nice work adding the solaris logo to your slide deck! :)
15:43 EmilienM dfisher: heh!
15:44 EmilienM dfisher: free add
15:44 dfisher +1 from me!
15:44 EmilienM free ad, I mean
15:44 dfisher :D
15:46 bkero Solaris, wow
15:46 dfisher used by like … 14 people!
15:47 bkero I guess corporations are people now.
15:47 dfisher lol
15:47 bkero (and nibalizer)
15:49 * nibalizer runs svcs -xv on bkero
15:49 bkero ouch
15:50 dfisher svc:/system/bkero:default (FOSS developer)
15:50 dfisher State: online since Thu Jul 21 18:29:49 2016
15:50 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
15:50 EmilienM iberezovskiy, degorenko: https://review.openstack.org/#/c/348248/ is green
15:52 EmilienM mwhahaha: if you have time also ^
15:54 EmilienM thanks guys :)
15:55 openstackgerrit Merged openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
16:00 xarses colonwq: can you add a reno file to that?
16:00 colonwq xarses, "reno"?
16:00 xarses the release notes yaml
16:01 xarses http://docs.openstack.org/developer/reno/
16:01 xarses you can use tox config in the repo to spawn the page
16:02 xarses really its just a unique slug that gets collected when we build the release notes
16:02 colonwq Ah thanks. I'll get it worked up.
16:02 xarses also, more importantly, shouldn't we deprecate the admin_token since ceph says we should consider it to be
16:03 colonwq It only exists for the keystone v2. v3 does not use it.
16:03 colonwq Does setting admin_token to undef delete it if it exists from the ceph.conf?
16:03 xarses correct, but rgw is planning to drop support for it
16:04 xarses no, undef doesn't do anything, it just ignores it
16:04 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/347105
16:04 colonwq How would I delete it if it exists in the conf file?
16:04 iurygregory EmilienM, degorenko done :D
16:06 EmilienM cool
16:07 xarses uh, I'm not sure, worst case, we can switch from the conf file provider to ini_setting and ensure absent, but there should be a better way
16:07 xarses https://github.com/puppetlabs/puppet​labs-inifile#beginning-with-inifile
16:08 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/347105
16:09 xarses EmilienM: mwhahaha degorenko iurygregory ^^ is there an established way for the conf file provider to ensure absent a setting?
16:10 iurygregory xarses, if is $::os_service_default you don't need to ensure absent
16:10 degorenko xarses, like we did os_service_default?
16:11 EmilienM xarses, colonwq: http://docs.openstack.org/developer/pu​ppet-openstack-guide/coding-style.html​#config-file-defaults-and-parameters
16:12 iurygregory if is spec test you can use nil or not contain =)
16:13 xarses EmilienM: oh, I didn't realize thats the magic it does. fantastic thanks
16:13 EmilienM http://storage2.static.itmages.ru/i/15/0​225/h_1424862257_4796128_90f68c0a44.gif
16:14 iurygregory magic https://openstackreactions.files.wor​dpress.com/2014/04/magick.gif?w=326 =)
16:14 EmilienM almost the same!
16:14 iurygregory yeah
16:14 iurygregory lol
16:14 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
16:14 EmilienM ok now reviewing barbican
16:15 dfisher what ended up being the logo, btw?
16:16 xarses thats in puppet-openstacklib right?
16:16 xarses ya
16:16 xarses colonwq: we can just use $::os_service_default after we add it as a dep to puppet-ceph
16:17 colonwq That should become the logo for the "it works" page.
16:17 EmilienM iurygregory: commented
16:18 iurygregory EmilienM, oh i thougt that i should just break auth_type sorry =)
16:19 EmilienM iurygregory: break it all!
16:19 iurygregory alee_afk, ^
16:19 iurygregory XD
16:19 EmilienM sorry if I was not clear
16:20 iurygregory EmilienM, no problem =)
16:20 EmilienM iurygregory: alee_afk and I will work together on p-o-i
16:23 colonwq xarses, things I should parameter checks? keystone_version = v3 and admin_token not servcie_default should fail.
16:23 colonwq or should I trust users to do the right thing.
16:24 alee EmilienM, so - how is it going?
16:25 EmilienM alee: let me see
16:25 ntpttr__ joined #puppet-openstack
16:25 EmilienM test_encrypted_cinder_volumes fail :(
16:26 EmilienM looking at diff
16:27 EmilienM I don't get why it fails
16:27 EmilienM only diff is ordering https://review.openstack.org/#/c/339806/42..44/manifests/barbican.pp
16:27 alee EmilienM, tempest role is Member only again
16:28 EmilienM tempest_roles=Member
16:28 EmilienM right
16:28 alee EmilienM, we must be running into cache problems again
16:28 EmilienM yes
16:28 EmilienM let me report it on infra channel
16:28 alee EmilienM, ok
16:29 pabelanger joined #puppet-openstack
16:29 EmilienM pabelanger: hi
16:29 pabelanger o/
16:29 EmilienM pabelanger: we have a patch in p-o-i that depends on a patch that is merged
16:30 EmilienM but I think the version of puppet-tempest in the image is old and not updated
16:30 EmilienM let me show
16:30 pabelanger ok
16:30 EmilienM context https://review.openstack.org/#/c/339806/
16:30 EmilienM logs are coming, a sec
16:30 EmilienM http://logs.openstack.org/06/339806/44/c​heck/gate-puppet-openstack-integration-3​-scenario002-tempest-centos-7/4bf1d45/co​nsole.html#_2016-07-28_12_49_16_315962
16:30 EmilienM 2016-07-28 12:49:16.315962 | INFO:zuul.Cloner:Prepared openstack/puppet-tempest repo with branch master at commit 16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe
16:30 EmilienM 16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe is not master
16:30 EmilienM see https://github.com/openstack/​puppet-tempest/commits/master
16:31 EmilienM so something is broken somewhere
16:31 EmilienM and we have CI issues because of it
16:31 EmilienM same thing with https://review.openstack.org/#/c/344916/
16:31 EmilienM it looks like it's only on some nodes
16:31 EmilienM let me tell you
16:32 pabelanger EmilienM: well, 16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe is in the master branch, just not the latest commit.  I think there is something going on with your usage of zuul-cloner, let me look at your code
16:32 EmilienM what?
16:32 pabelanger 16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe is from your master branch
16:32 EmilienM pabelanger: https://github.com/openstack/puppet-openstac​k-integration/blob/master/functions#L37-L42
16:32 EmilienM pabelanger: yeah but it's not latest commit
16:32 EmilienM until now, we never had this issue
16:32 pabelanger you are just expecting it to be 03d1e9762074c463cb7d148dbf27e4daf9e12f74
16:32 EmilienM something must have changed
16:33 pabelanger zuul-cloner did get an update recently
16:33 openstackgerrit Merged openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/348248
16:33 pabelanger so, I wonder if this is a regression
16:33 pabelanger EmilienM: lets go back to openstack-infra and ask fungi and clarkb
16:33 EmilienM oki doc
16:37 v1k0d3n_ joined #puppet-openstack
16:42 v1k0d3n joined #puppet-openstack
16:43 alee joined #puppet-openstack
16:44 ntpttr__ joined #puppet-openstack
17:08 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/347105
17:11 dfisher r10k question .. if I want to change where the git cache is, is that based on $HOME?
17:12 dfisher it looks like that's the case:
17:12 dfisher r10k/git/cache.rb:  def_setting_attr :cache_root, File.expand_path(ENV['HOME'] ? '~/.r10k/git': '/root/.r10k/git')
17:13 iurygregory i've never run into this problem O.o
17:14 iurygregory why change git cache? just wondering =)
17:21 EmilienM alee: it's an infra thing
17:21 EmilienM we're stuck now
17:22 alee EmilienM, ok - any eta on resolution?
17:24 EmilienM no  idea
17:27 dfisher iurygregory: oracle networking stuff.  My homedirectory is on a system in Austin TX but I'm doing my dev work on a system in Santa Clara CA
17:27 dfisher so, NFS things are … slow
17:28 iurygregory yeah  we use NFS here at the university in the lab =(
17:28 dfisher so, bundle exec rake spec can take *hours* to run
17:28 dfisher well, hours to run r10k install
17:28 iurygregory that's why i use my notebook XD
17:29 dfisher yeah, i should have.
17:29 alee EmilienM, ok :/  well, I guess we wait then.
17:38 iurygregory alee, can you review https://review.openstack.org/#/c/347105 ?
17:39 alee iurygregory, will do
17:39 iurygregory alee, tks o/
17:50 ducttape_ joined #puppet-openstack
17:57 v1k0d3n joined #puppet-openstack
18:01 ducttape_ joined #puppet-openstack
18:49 iurygregory mwhahaha, can you review 347105? authtoken =) i've update the bug as well
18:50 mwhahaha k
18:56 EmilienM iurygregory: reviewing it too
18:56 iurygregory tks o/
18:56 iurygregory CI is slow =(
18:57 iurygregory waiting for lint and beaker-xenial only
18:57 EmilienM iurygregory: Alex has a good question inline
18:58 EmilienM iurygregory, mwhahaha: I also find it weird to fail by default if you just declare ::barbiban::api without any parameter
18:58 EmilienM because it will include the authtoken class but since we don't have a default keystone password
18:58 EmilienM what do you think?
18:59 iurygregory yeah i saw that with the tests .-.
18:59 iurygregory about auth_uri we can set but we dont need both
18:59 EmilienM but it's fine
18:59 EmilienM for password
18:59 EmilienM it's already failing in other modules if unset
18:59 EmilienM it's just now we have a new class
18:59 mwhahaha iurygregory: which one do you need? is auth_url ok as a standalone?
19:00 iurygregory auth_url work well for barbican =) since beaker is ok
19:01 EmilienM beaker doesn't test much
19:01 EmilienM keep in mind that ;)
19:01 EmilienM what will really test barbican is what alee and I are doing in p-o-i
19:02 iurygregory EmilienM, sure o/ maybe i should provide a poc for a component that run integration?
19:03 EmilienM sure
19:03 iurygregory EmilienM, ack will do tonight =)
19:03 EmilienM iurygregory: I can do it now
19:04 EmilienM I'll update my puppet cinder patch now
19:04 iurygregory EmilienM, ok o/
19:12 iurygregory EmilienM, do you have any talk for Barcelona?
19:12 dfisher seriously, i'll vote.
19:12 dfisher HE SAID SOLARIS.  +1 FOREVERS!
19:12 EmilienM yeah, I have 2 proposals
19:12 EmilienM about authtoken, i would like to notify that some patches might be different for backward compatibility
19:12 EmilienM mwhahaha, iurygregory  ^
19:13 iurygregory EmilienM, nice i'll look =D and ask my teammattes to vote :D
19:13 EmilienM the structure will be the same
19:13 mwhahaha yea
19:13 EmilienM iurygregory: thx
19:13 EmilienM but some params will be different to maintain bp
19:13 EmilienM bc*
19:13 iurygregory yeag
19:13 iurygregory yeah*
19:13 dfisher Search results for "Solaris"
19:13 dfisher •There are no presentations that match your criteria.
19:13 * dfisher sobs
19:14 mwhahaha where's your presentation then?
19:14 mwhahaha mr. solaris
19:14 dfisher i'm so farkin' slammed right now :(
19:14 dfisher hell, i didn't even write an abstract for Oracle Open World.
19:15 mwhahaha oracle "open" world
19:15 * dfisher sighs
19:15 dfisher this again?
19:15 dfisher ok.
19:15 mwhahaha :D
19:15 iurygregory lol
19:15 mwhahaha it's ok, haven't you read the openstack-dev mailing list this week? mirantis is out to get everyone
19:16 dfisher at this point, i pretty much just right-click -> mark as read on that
19:16 mwhahaha still not sure what it's about other than someone is taking a marketing PR a bit too seriously
19:16 dfisher i don't have time to wade through 99.9% cruft to find gems  :(
19:16 mwhahaha ZOMG CONTAINERS AER OURS
19:17 dfisher hey, speaking of that ...
19:17 * mwhahaha goes back to actually attempting to work
19:17 dfisher https://twitter.com/satokaz​/status/757730426324525056
19:17 mwhahaha ha
19:18 dfisher so, Docker's now in Solaris.
19:18 mwhahaha but can it kuberentes
19:19 dfisher that's next.
19:19 dfisher k8 & docker 1.13 for swarm
19:20 dfisher there's already a PR for Solaris support in Docker 1.13
19:20 dfisher that team is working on it.
19:33 openstackgerrit craig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/344464
19:34 openstackgerrit craig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/344464
19:34 cdelatte joined #puppet-openstack
19:35 hjensas joined #puppet-openstack
19:39 openstackgerrit Guilherme Balzana proposed openstack/puppet-monasca: Using puppet-openstack standard gitignore  https://review.openstack.org/348528
19:44 guimaluf EmilienM, it's so odd to setup everything a monasca package should do in puppet, e.g user, groups, upstart files, etc. Should we go that way while there is no package at all or wait/build monasca packages?
19:47 mwhahaha package all the things!
19:55 alee joined #puppet-openstack
20:25 openstackgerrit Tim Rozet proposed openstack/puppet-tripleo: Adds OpenDaylight  https://review.openstack.org/335173
20:44 ducttape_ joined #puppet-openstack
20:48 ntpttr__ joined #puppet-openstack
21:09 ntpttr__ joined #puppet-openstack
21:20 ducttape_ joined #puppet-openstack
21:25 dfisher how do I look at manifest / file content from a failed test in CI?
21:26 dfisher http://logs.openstack.org/63/347463/6/ch​eck/gate-puppet-openstacklib-puppet-beak​er-rspec-centos-7/4c0cf7b/console.html - I'd like to find
21:26 dfisher http://logs.openstack.org/63/347463/6/check/gate-​puppet-openstacklib-puppet-beaker-rspec-centos-7/​4c0cf7b/console.html#_2016-07-28_15_25_58_529642 | localhost $ scp /tmp/beaker20160728-15434-115mn1y centos-70-x64:/tmp/apply_manifest.pp.j68p8p {:ignore => }
21:26 dfisher the contents of that file
21:28 mwhahaha it's the acceptance blob
21:28 dfisher in the keystone repo?
21:28 dfisher puppet-keystone*
21:29 dfisher ajh
21:29 dfisher ah*
21:29 dfisher found it
21:29 dfisher openstacklib_config_provider_spec.rb
21:29 dfisher mwhahaha: than ks
21:30 mwhahaha memcache_pool_conn_get_timeout
21:30 mwhahaha er
21:30 mwhahaha https://github.com/openstack/puppet-open​stacklib/blob/master/spec/acceptance/ope​nstacklib_config_provider_spec.rb#L9-L38
21:30 * dfisher nods
21:30 dfisher thanks!
21:49 dfisher mwhahaha: got time for yet more dumb questions from me?
21:49 mwhahaha for you? no ;)
21:49 mwhahaha what's up
21:50 dfisher ok, so i see why CI is yelling at me.
21:50 dfisher in neutron_l3_agent_config/ini_settings.rb we set transform_to / transform_to=
21:50 dfisher https://review.openstack.org/#/c/347468/7/lib/pup​pet/provider/neutron_l3_agent_config/ini_setting.rb
21:50 dfisher well, I do, currently :)
21:51 dfisher the issue is that in https://review.openstack.org/#/c/347463/6/lib​/puppet/provider/openstack_config/ini_setting.rb, i get a puke from line 54
21:51 dfisher with
21:51 dfisher Error: Could not set 'present' on ensure: undefined local variable or method `transform_to' for #<Puppet::Type::Keystone_config::Pr​oviderIni_setting:0x00000002dd2258> at 3:/var/tmp/keystone.pp
21:51 mwhahaha so you probably need to stub i t out in the openstack_config type
21:51 dfisher $ cat ../../type/openstack_config.rb
21:51 dfisher Puppet::Type.newtype(:openstack_config) do
21:51 dfisher newparam(:transform_to)
21:51 dfisher end
21:51 dfisher i am :(
21:52 dfisher well, chem did this piece :)
21:52 mwhahaha hmm
21:52 mwhahaha the magical land of providers
21:52 dfisher hah
21:54 mwhahaha so either it should be @transform_to or perhaps you need a getter/setter not completely sure
21:54 dfisher ok
21:55 dfisher will poke
21:55 dfisher thanks
21:55 mwhahaha like on line 40
21:55 mwhahaha where there's a def ensure_absent_val
21:55 mwhahaha wonder if you need an equivalent for the tranform_to
21:57 v1k0d3n joined #puppet-openstack
21:58 mwhahaha so a def transform_to\n  resource[:transform_to]\n end ;)
21:58 mwhahaha you get the idea
21:58 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
21:58 EmilienM I GOT IT
21:58 EmilienM sorry
21:59 EmilienM guimaluf: /me no opinion on topic
21:59 guimaluf np!
21:59 * guimaluf package all the things!
22:00 EmilienM iurygregory, mwhahaha: ok I did cinder authtoken https://review.openstack.org/342905 - when you have time to look
22:00 EmilienM have a good night
22:00 mwhahaha k
22:03 dfisher mwhahaha:
22:03 dfisher def transform_to
22:03 dfisher resource[:transform_to]
22:03 dfisher end
22:03 dfisher that did it
22:03 * dfisher tips hat
22:04 openstackgerrit Drew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/347463
22:06 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
22:08 vinsh joined #puppet-openstack
22:08 xarses colonwq: should we fail/warn if both auth_token and user are set?
22:13 _ducttape_ joined #puppet-openstack
22:19 colonwq xarses, I think so. I would do a test of 'v2.0' needs auth_token and not the other 4 and 'v3' is the opposite.
22:20 xarses something like that, we also need tenant and password defined in v3 case
22:21 xarses if v2.0 and auth_token {} elseif v3 and user and password or something
22:22 colonwq project instead of tenant. I'll round out the tests.
22:25 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Configure Ceph RGW to use Keyston V3  https://review.openstack.org/348293
22:36 ntpttr- joined #puppet-openstack
23:10 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
23:41 iurygregory_ joined #puppet-openstack
23:51 ducttape_ joined #puppet-openstack

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