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/puppetlabs-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/puppet-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/0225/h_1424862257_4796128_90f68c0a44.gif |
16:14 |
iurygregory |
magic https://openstackreactions.files.wordpress.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/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/4bf1d45/console.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-openstack-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/check/gate-puppet-openstacklib-puppet-beaker-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-openstacklib/blob/master/spec/acceptance/openstacklib_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/puppet/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::ProviderIni_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 |