Thursday, 2016-07-28

« Wednesday, 2016-07-27 Index Friday, 2016-07-29 »
*** xarses has quit IRC00:19
*** ducttape_ has quit IRC00:22
*** saneax is now known as saneax_AFK00:37
*** xarses has joined #puppet-openstack00:50
*** cdelatte has quit IRC00:55
*** mfisch has quit IRC01:17
*** ducttape_ has joined #puppet-openstack01:26
*** mfisch has joined #puppet-openstack01:39
*** mfisch has quit IRC01:39
*** mfisch has joined #puppet-openstack01:39
openstackgerritMatt Fischer proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446401:52
*** ducttape_ has quit IRC01:59
*** ducttape_ has joined #puppet-openstack02:04
*** ducttape_ has quit IRC02:14
*** ayoung has quit IRC02:25
*** ayoung has joined #puppet-openstack02:27
*** rwsu has quit IRC02:28
*** bklei has joined #puppet-openstack02:29
*** ayoung has quit IRC02:32
*** bklei has quit IRC02:34
*** richm has quit IRC02:34
*** ducttape_ has joined #puppet-openstack02:35
*** ayoung has joined #puppet-openstack02:36
*** ducttape_ has quit IRC02:36
*** rwsu has joined #puppet-openstack02:40
*** ducttape_ has joined #puppet-openstack02:50
*** ducttape_ has quit IRC02:54
*** guessi has joined #puppet-openstack03:01
*** ayoung has quit IRC03:14
*** iurygregory has quit IRC03:15
*** iurygregory has joined #puppet-openstack03:15
*** alee_afk has quit IRC03:16
*** guessi has quit IRC03:28
*** alee_afk has joined #puppet-openstack03:28
*** guessi has joined #puppet-openstack03:45
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795603:50
*** ducttape_ has joined #puppet-openstack03:51
*** guessi has quit IRC03:56
*** ducttape_ has quit IRC03:56
*** guimaluf has quit IRC03:56
*** guimaluf has joined #puppet-openstack03:57
*** leanderthal|afk is now known as leanderthal04:26
*** ducttape_ has joined #puppet-openstack05:53
*** ducttape_ has quit IRC05:58
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34814006:22
openstackgerritAde Lee proposed openstack/puppet-tempest: Add unit test for tempest_roles  https://review.openstack.org/34814106:22
*** tesseract- has joined #puppet-openstack06:42
*** ducttape_ has joined #puppet-openstack06:54
*** ducttape_ has quit IRC06:59
*** alee_afk has quit IRC07:02
*** jtomasek has joined #puppet-openstack07:02
*** atom_ has joined #puppet-openstack07:06
openstackgerritXiaohua Yuan proposed openstack/puppet-nova: add missing options for monkey_patch  https://review.openstack.org/34649407:07
*** permalac has joined #puppet-openstack07:13
*** hjensas has joined #puppet-openstack07:14
*** ifarkas_away is now known as ifarkas07:15
*** danpawlik has joined #puppet-openstack07:25
*** fxpester has quit IRC07:35
*** dgurtner has joined #puppet-openstack07:37
*** dgurtner has quit IRC07:37
*** dgurtner has joined #puppet-openstack07:37
*** alee_afk has joined #puppet-openstack07:52
*** shardy has joined #puppet-openstack08:00
*** derekh has joined #puppet-openstack08:14
*** permalac has quit IRC08:26
*** permalac has joined #puppet-openstack08:26
*** alee_afk is now known as alee08:28
openstackgerritSimon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780308:34
*** paramite has joined #puppet-openstack08:37
*** paramite has quit IRC08:37
*** paramite has joined #puppet-openstack08:37
*** imcsk8 is now known as imcsk8|zZz08:42
*** saneax_AFK is now known as saneax08:51
*** ducttape_ has joined #puppet-openstack08:56
*** ducttape_ has quit IRC09:00
*** saneax is now known as saneax_AFK09:12
openstackgerritKarthik S proposed openstack/puppet-neutron: WIP:Configure the max_vfs for SRIOV interfaces  https://review.openstack.org/34491809:16
*** permalac has quit IRC09:25
chemis 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 transiants09:44
*** gfidente has joined #puppet-openstack09:55
*** _degorenko|afk is now known as degorenko09:56
*** ducttape_ has joined #puppet-openstack09:57
*** ducttape_ has quit IRC10:01
*** saneax_AFK is now known as saneax10:16
*** iberezovskiy|off is now known as iberezovskiy10:16
EmilienMhello10:22
EmilienMiberezovskiy: hey10:23
iberezovskiyhey10:23
EmilienMcould we fix fuel asap?10:23
iberezovskiysure10:23
iberezovskiyI'm on it10:23
EmilienMthx10:23
iberezovskiythx you for patience10:23
openstackgerritSaravanan KR proposed openstack/puppet-openstack-integration: Add puppet-kmod modules to the integration  https://review.openstack.org/34822210:36
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile  https://review.openstack.org/34334610:40
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786010:59
openstackgerritSaravanan KR proposed openstack/puppet-openstack-integration: Add puppet-kmod modules to the integration  https://review.openstack.org/34822211:02
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786011:03
aleedegorenko, reading your comments ..11:07
aleedegorenko, if I understand your idea correctly - we could pass the tempest_roles as a parameter to openstack_integration::tempest in scenario00211:07
aleedegorenko, that would allow us to move the role creation to barbican.py and swift.py accordingly ..11:08
degorenkoalee, right11:08
aleerather than creating the roles each time.  yeah - I like that - will try it11:09
openstackgerritMerged openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/34800711:12
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34814011:13
openstackgerritMerged openstack/puppet-tempest: Add unit test for tempest_roles  https://review.openstack.org/34814111:14
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/34824811:15
EmilienMdegorenko, iberezovskiy: could you review ^ please?11:15
degorenkoEmilienM, why not cherry-pick with adaptation?11:17
EmilienMdegorenko: we can't11:17
degorenkoEmilienM, why?11:17
EmilienMdegorenko: it's outstanding11:17
EmilienMbecause code is really not the same11:17
EmilienMwe don't use puppet-oslo :)11:17
EmilienMin mitaka11:17
degorenkoi know11:17
degorenkoit should be adaptation :)11:18
degorenkook11:18
degorenkolet it go as it now11:18
EmilienMthanks11:18
EmilienMdegorenko: we found a bug in tripleo11:18
EmilienMwhere nova console is really unstable if you don't use the options11:18
EmilienMmaybe in fuel you have the bug too11:19
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/34824811:20
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668511:23
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668511:24
degorenkoEmilienM, waiting for ci and will +211:27
EmilienMdegorenko: you rocks11:27
aleedegorenko, EmilienM - whats the difference between the regular and non-voting gate jobs?11:41
EmilienMnon voting jobs don't vote11:41
EmilienMand regular jobs vote11:41
EmilienMwas it the question?11:42
aleeEmilienM, thanks -- what I meant was integration-4 vs integration-311:42
EmilienMah11:42
EmilienMthe version of puppet11:42
EmilienMintegration-4 is puppet411:42
EmilienMand other is puppet411:42
EmilienMand other is puppet3 err11:42
EmilienMalee: did you make progress on barbican?11:43
*** skramaja has quit IRC11:43
*** skramaja has joined #puppet-openstack11:43
aleeEmilienM, ok - so it seems that the failure on puppet 4 for the barbican test is due to a puppet-4/3 problem ..11:43
EmilienMlooking11:44
aleeEmilienM, yeah - I just waited a bit, did a few rechecks and things started working11:44
aleeEmilienM, goign to make changes as suggested by degorenko11:44
EmilienMyeah I think it was a cache issue in the nodepool images11:45
EmilienMand degorenko reviewed the patch11:45
*** karthiks has quit IRC11:45
EmilienMhis remark is good11:45
EmilienMit's a puppet4 thing I think I'm investigating11:45
aleeEmilienM, the puppet4 failure looks like a permissions issue11:45
EmilienMno, it's ordering I think11:46
EmilienMlet me a sec11:46
aleeok11:46
*** shardy has quit IRC11:50
*** saneax is now known as saneax_AFK11:52
openstackgerritEmilien Macchi proposed openstack/puppet-barbican: db: remove useless pick  https://review.openstack.org/34826711:53
*** karthiks has joined #puppet-openstack11:58
*** skramaja has quit IRC11:58
*** ducttape_ has joined #puppet-openstack11:58
*** saneax_AFK is now known as saneax12:00
*** morazi has joined #puppet-openstack12:00
EmilienMalee: let's try again12:01
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980612:01
openstackgerritMerged openstack/puppet-oslo: cache: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798012:02
openstackgerritMerged openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798212:02
*** ducttape_ has quit IRC12:02
aleeEmilienM, ok -- I have be out for a few hours .. I'll make the degorenko changes when I return.12:03
EmilienMalee: what need?12:04
EmilienMdegorenko: let me know if you want us to move things in the barbican patch :)12:04
*** bogdando has quit IRC12:04
*** ducttape_ has joined #puppet-openstack12:05
aleeEmilienM, if you're good as-is, then I'm not going to complain.12:05
aleeEmilienM, otherwise I'm hoping we can finally merge this :)12:05
EmilienMit would be awesome12:05
EmilienMwe just don't want to break puppet4 CI12:05
aleeEmilienM, I'll check in when I get back ..12:05
*** ducttape_ has quit IRC12:06
*** alee is now known as alee_afk12:07
*** morazi has quit IRC12:08
*** alee_afk has quit IRC12:08
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795612:08
*** gfidente has quit IRC12:09
*** ducttape_ has joined #puppet-openstack12:09
*** alee_afk has joined #puppet-openstack12:10
*** morazi has joined #puppet-openstack12:10
openstackgerritMerged openstack/puppet-horizon: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798112:13
iurygregorymorning people12:13
*** bogdando has joined #puppet-openstack12:15
EmilienMiurygregory: o/12:16
iurygregory\o12:16
colonwqEmilienM, 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
colonwqI'm still waiting for a final confirmation.12:21
*** bogdando has quit IRC12:22
EmilienMcolonwq: oki12:22
*** bogdando has joined #puppet-openstack12:25
openstackgerritMerged openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/32834712:26
openstackgerritXiaohua Yuan proposed openstack/puppet-nova: add missing options for monkey_patch  https://review.openstack.org/34649412:27
*** ducttape_ has quit IRC12:35
degorenkoEmilienM, ok, will look12:35
degorenkoEmilienM, typo in patch12:36
EmilienMdegorenko: arg12:37
EmilienMwhere?12:37
degorenkoEmilienM, in barbican class12:37
iurygregorydegorenko, tks for the review i'll update after lunch =)12:39
degorenkoiurygregory, sure12:40
EmilienMdegorenko: looking12:41
EmilienMdegorenko: lol wtf12:41
EmilienMthx!12:42
degorenkoEmilienM, also commented here https://review.openstack.org/#/c/34824812:42
EmilienMdegorenko: awesome12:43
*** chem is now known as chem|lunch12:43
*** trown|outtypewww is now known as trown12:43
EmilienMdegorenko: /me running tests this time12:44
degorenkoEmilienM, /me running tests every time :D12:45
EmilienMdegorenko: yeah, not me... shame!12:45
openstackgerritMerged openstack/puppet-barbican: db: remove useless pick  https://review.openstack.org/34826712:51
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Configure Ceph RGW to use Keyston V3  https://review.openstack.org/34829312:54
*** saneax is now known as saneax_AFK13:09
ericksonsantos:set nu13:12
ericksonsantosops13:12
*** richm has joined #puppet-openstack13:13
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Implement aodh profiles  https://review.openstack.org/33285413:14
openstackgerritMerged openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/34731513:14
openstackgerritMerged openstack/puppet-tripleo: Remove global openstack-core resource  https://review.openstack.org/34747213:14
*** chem|lunch has quit IRC13:16
*** chem|lunch has joined #puppet-openstack13:16
*** gfidente has joined #puppet-openstack13:16
*** gfidente has quit IRC13:16
*** gfidente has joined #puppet-openstack13:16
*** ducttape_ has joined #puppet-openstack13:16
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/34824813:17
*** ducttape_ has quit IRC13:21
*** karthiks has quit IRC13:26
*** markvoelker has joined #puppet-openstack13:30
*** morazi has quit IRC13:30
*** markvoelker has quit IRC13:34
*** dfisher has joined #puppet-openstack13:36
*** karthiks has joined #puppet-openstack13:38
EmilienMiurygregory: you making good progress with authtoken? you need help?13:42
EmilienMiurygregory: I would like us to finish this task before end of Newton13:42
degorenkoEmilienM, how many time we have?13:42
*** ayoung has joined #puppet-openstack13:43
*** morazi has joined #puppet-openstack13:43
*** ducttape_ has joined #puppet-openstack13:43
EmilienM1 months and some days13:43
degorenkoah, ok13:43
EmilienMbut we have a lot of modules :)13:43
degorenkoyeah, sure13:43
degorenkojust wondering about time13:44
*** ducttape_ has quit IRC13:44
*** ducttape_ has joined #puppet-openstack13:44
EmilienMdegorenko: releases.openstack.org/newton/schedule.html13:44
*** shardy has joined #puppet-openstack13:44
EmilienMFeature freeze Aug 29-0213:44
EmilienMI want to keep September for stabilization13:45
EmilienMand bug fix13:45
degorenkoyep, sure, we will13:45
EmilienMdegorenko: about https://review.openstack.org/#/c/346685/13:46
EmilienMdon't we want to wait until we have one module done ? (ie barbican)?13:46
EmilienMto make sure it's consistent13:46
degorenkoEmilienM, it uses same approach as barbican, isnt it?13:46
EmilienMI haven't reviewed it13:46
EmilienMI'm doing now13:46
degorenkoEmilienM, if you feel, that it should be postponed on some time - just one your word sir :)13:47
EmilienMdegorenko: no, it's fine13:48
degorenkoiurygregory, don't forget about zaqar :) https://review.openstack.org/#/c/346544/13:48
iurygregoryEmilienM, can you review barbican to see if need any changes besides the ones pointed by degorenko and alee_afk ? https://review.openstack.org/#/c/34710513:52
iurygregorydegorenko, sure13:52
dfisherchem|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 methods13:53
EmilienMI'm not available during the next 60 minutes13:53
degorenkoEmilienM, iurygregory i have question, regarding our authtoken things - related to glance13:53
degorenkoglance has 3 services, which needs auth parameters (glare, api, registry)13:54
degorenkowhere i should put authtoken params?13:54
degorenkonot sure, that creating 3 classes under keystone directory is good idea13:54
degorenkoi guess all of them should be created in appropriate dirs?13:55
EmilienMmhh, make them common13:59
EmilienMok /me away now :P13:59
*** xarses has quit IRC14:00
degorenkoall of them have different configs14:00
*** skramaja has joined #puppet-openstack14:00
*** saneax_AFK is now known as saneax14:00
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786014:04
iurygregorydegorenko, i'm in a meeting right now i'll look in a few minutes =)14:05
*** leanderthal is now known as leanderthal|afk14:05
degorenkosure14:05
*** chem|lunch has quit IRC14:17
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668514:18
openstackgerritDrew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746314:25
*** shardy has quit IRC14:31
*** shardy has joined #puppet-openstack14:34
*** jistr is now known as jistr|call14:35
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/34773914:40
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo  https://review.openstack.org/34725014:40
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo  https://review.openstack.org/34697514:40
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes  https://review.openstack.org/34738514:40
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail config puppet module for tripleO  https://review.openstack.org/34693814:40
*** chem has joined #puppet-openstack14:40
dfisherchem: 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, however14:46
chemdfisher: I'm going to have a look14:47
*** jistr|call is now known as jistr14:49
chemdfisher: nice catch.  That makes a lot of sense. the "value=" is only for modification, and "create" is the right one for creation.14:49
dfisheri hardcoded the direction to be :to though14:50
dfishernot sure if I need to get into some kind of begin/rescue thing14:50
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail database puppet module for tripleO  https://review.openstack.org/34840614:50
chemdfisher: for create that's the :to that we need, so that's good.  it's only for retrieving the info that we need the :from14:51
* dfisher nods14:51
chemdfisher: for begin/rescue, well if that's failing it's for a good reason, so let it fail.14:51
dfisherchem: ok.14:51
*** xarses has joined #puppet-openstack14:52
chemdfisher: so now first run create and second run does nothing ?14:53
dfisherchecking ...14:53
dfisherchem: http://paste.openstack.org/show/543288/  it looks like the second run does something but it doesn't change the value.14:55
EmilienMiurygregory, degorenko: ok back. I'll review it shortly14:55
chemdfisher: run it with --detailed-exitcode and check $?, should be 0, but it looks like it's 2.14:56
mwhahaha0 or 214:56
chemdfisher: something is still missing14:57
mwhahaha2 means success and things changed14:57
mwhahaha0 is nothing changed14:57
chem214:57
chemmust be 0, but it's going to be 214:57
dfisher$ echo $?14:57
dfisher214:57
chemdfisher: so there is something still missing to make it work completly right14:57
*** ifarkas is now known as ifarkas_afk14:58
chemdfisher: we should have 0 here14:58
dfisherchem: nod.  will keep poking at it...14:58
mwhahahalooks like you might need to trim the line break14:58
dfishernot entirely sure where :)14:59
mwhahahawhere it's getting read from initially would be my guess14:59
mwhahahacause you're setting 'provider' but it thought it was 'provider\n'14:59
dfisherright.15:00
*** vinsh has joined #puppet-openstack15:01
chemdfisher: ah, yes that could be.  I added a comment on the review  as well, but try first mwhahaha's idea15:01
mwhahahaprobably could just do it in teh value= call15:01
dfisherso in python, you can use %r to print out a raw string:15:01
dfisher>>> a = "\n  \tfoo"15:01
dfisher>>> print a15:01
dfisher        foo15:01
dfisher>>> print "%r" % a15:01
dfisher'\n  \tfoo'15:01
dfisherwhat's the ruby equivalent of that?15:01
chemdfisher: add .chop15:02
chemdfisher: "foo\n".chop -> "foo"15:03
* dfisher nods15:03
dfisherit ended up being in the from_uuid() method in neutron_l3_agent_config/ini_settings.rb15:09
*** jmiu has quit IRC15:09
*** jmiu_ is now known as jmiu15:09
dfisher$ sudo puppet apply --detailed-exitcode test.pp15:09
dfisherNotice: Compiled catalog for openstack-x52-26.us.oracle.com in environment production in 0.10 seconds15:09
dfisherNotice: Finished catalog run in 1.28 seconds15:09
dfisher$ echo $?15:09
dfisher015:09
*** jmiu_ has joined #puppet-openstack15:09
chemdfisher: so we're good :)15:10
openstackgerritSanjay Upadhyay proposed openstack/puppet-neutron: sriov check for physical_device_mappings and exclude_devices  https://review.openstack.org/33346515:10
dfisheri think so...15:10
dfisherredoing from scratch15:10
chemdfisher: push it so that the ci can play with it as well :)15:10
dfisherwill in a sec.15:10
dfisherfirst run exits with 2 (successful but with changes).    second exits with 015:11
dfisherexactly how we want it15:11
chemdfisher: yep, super nice work!15:11
openstackgerritDrew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746315:12
openstackgerritDrew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746815:13
dfisherok, there's both15:13
iurygregorydegorenko, we have release for barbican (newton 9.0.0, 9.1.0)15:13
degorenkoiurygregory, oh, right15:13
iurygregoryshould I break or not? =(15:14
degorenkoiurygregory, but next question about - auth_type and auth_strategy15:14
iurygregorylet me see15:14
chemdfisher: 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 better15:14
degorenkowe are using auth_type in authtoken15:14
chemdfisher: wdyt ?15:14
dfisherchem: the chop?15:15
chemdfisher:yep15:15
dfisheri think it's a bug with neutron client, tbh15:15
dfisherbut i think the client should probably decided how to handle it15:15
iurygregorydegorenko, auth_type in api should be auth_strategy or 'barbican_auth_type'15:15
chemdfisher: ack, so let it as it is.15:16
iurygregorykeystone::authtoken have auth_type because we need to select what authentication we will use...15:16
*** openstackgerrit has quit IRC15:18
degorenkoiurygregory, why? we don't setting it in config15:18
*** openstackgerrit has joined #puppet-openstack15:18
iurygregorydegorenko, we don't set auth_type?15:19
degorenkoiurygregory, where? i don't see, only if15:19
EmilienMiurygregory, degorenko: sorry wa sa bit away15:20
EmilienMyes you can break puppet-barbican15:20
EmilienMthere AFIK no user now15:21
iurygregoryEmilienM, ack15:21
EmilienMfuel, tripleo, are not using it15:21
dfisherchem: both to/from need that chop.  for some reason, setting it initially seems to strip the value.15:21
EmilienMand AFIK no bug report15:21
EmilienMiurygregory: so break it and make it beautifule15:21
EmilienM-e15:21
iurygregoryso there is no 'auth_type' anymore for api.pp =P15:21
degorenkoiurygregory, anyway, why you want change auth_type (in barbican meaning) to auth_strategy? i don't see any reasons15:21
iurygregoryis not the correct name =)15:22
EmilienMwhat is important is: keep puppet module able to deploy barbican WITHOUT keystone15:22
iurygregoryauth_strategy make more sense than auth_type for barbican15:22
iurygregorybecause we choose keystone or any other thing15:22
degorenkowe have auth_type everywhere15:22
degorenkoi don't see any pros for this renaming15:23
iurygregorydegorenko, http://git.openstack.org/cgit/openstack/puppet-neutron/tree/manifests/init.pp15:24
iurygregorythe auth_type in barbican is the same idea for auth_strategy in neutron15:24
iurygregoryand auth_type is a configuration for keystone_authtoken15:24
degorenkook15:25
degorenkobut we don't have stable branches15:25
degorenkoso i guess we can just rename then15:25
iurygregorydegorenko, yeah i'll do this ;)15:26
*** saneax is now known as saneax_AFK15:36
*** ducttape_ has quit IRC15:37
iurygregoryEmilienM, since barbican was not released i need to add a release note?15:40
*** ducttape_ has joined #puppet-openstack15:41
EmilienMiurygregory: no15:41
EmilienMconsider it like a new module15:41
iurygregoryEmilienM, sure ;)15:41
*** jistr is now known as jistr|afk15:42
dfisherEmilienM: nice work adding the solaris logo to your slide deck! :)15:43
*** shardy has quit IRC15:43
EmilienMdfisher: heh!15:43
EmilienMdfisher: free add15:44
dfisher+1 from me!15:44
EmilienMfree ad, I mean15:44
dfisher:D15:44
bkeroSolaris, wow15:46
dfisherused by like … 14 people!15:46
bkeroI guess corporations are people now.15:47
dfisherlol15:47
bkero(and nibalizer)15:47
* nibalizer runs svcs -xv on bkero15:49
bkeroouch15:49
dfishersvc:/system/bkero:default (FOSS developer)15:50
dfisher State: online since Thu Jul 21 18:29:49 201615:50
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795615:50
EmilienMiberezovskiy, degorenko: https://review.openstack.org/#/c/348248/ is green15:50
EmilienMmwhahaha: if you have time also ^15:52
*** atom_ has quit IRC15:54
EmilienMthanks guys :)15:55
openstackgerritMerged openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780315:55
xarsescolonwq: can you add a reno file to that?16:00
colonwqxarses, "reno"?16:00
xarsesthe release notes yaml16:01
xarseshttp://docs.openstack.org/developer/reno/16:01
xarsesyou can use tox config in the repo to spawn the page16:01
xarsesreally its just a unique slug that gets collected when we build the release notes16:02
colonwqAh thanks. I'll get it worked up.16:02
xarsesalso, more importantly, shouldn't we deprecate the admin_token since ceph says we should consider it to be16:02
colonwqIt only exists for the keystone v2. v3 does not use it.16:03
colonwqDoes setting admin_token to undef delete it if it exists from the ceph.conf?16:03
xarsescorrect, but rgw is planning to drop support for it16:04
xarsesno, undef doesn't do anything, it just ignores it16:04
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/34710516:04
colonwqHow would I delete it if it exists in the conf file?16:04
iurygregoryEmilienM, degorenko done :D16:04
EmilienMcool16:06
*** foxhunt_away is now known as foxhunt7216:06
xarsesuh, 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 way16:07
xarseshttps://github.com/puppetlabs/puppetlabs-inifile#beginning-with-inifile16:07
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/34710516:08
xarsesEmilienM: mwhahaha degorenko iurygregory ^^ is there an established way for the conf file provider to ensure absent a setting?16:09
iurygregoryxarses, if is $::os_service_default you don't need to ensure absent16:11
degorenkoxarses, like we did os_service_default?16:11
EmilienMxarses, colonwq: http://docs.openstack.org/developer/puppet-openstack-guide/coding-style.html#config-file-defaults-and-parameters16:11
iurygregoryif is spec test you can use nil or not contain =)16:12
xarsesEmilienM: oh, I didn't realize thats the magic it does. fantastic thanks16:13
EmilienMhttp://storage2.static.itmages.ru/i/15/0225/h_1424862257_4796128_90f68c0a44.gif16:14
iurygregorymagic https://openstackreactions.files.wordpress.com/2014/04/magick.gif?w=326 =)16:14
EmilienMalmost the same!16:14
iurygregoryyeah16:14
iurygregorylol16:14
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786016:14
EmilienMok now reviewing barbican16:14
dfisherwhat ended up being the logo, btw?16:16
xarsesthats in puppet-openstacklib right?16:16
xarsesya16:16
xarsescolonwq: we can just use $::os_service_default after we add it as a dep to puppet-ceph16:17
colonwqThat should become the logo for the "it works" page.16:17
EmilienMiurygregory: commented16:17
iurygregoryEmilienM, oh i thougt that i should just break auth_type sorry =)16:18
*** gfidente has quit IRC16:19
EmilienMiurygregory: break it all!16:19
iurygregoryalee_afk, ^16:19
iurygregoryXD16:19
EmilienMsorry if I was not clear16:19
iurygregoryEmilienM, no problem =)16:20
EmilienMiurygregory: alee_afk and I will work together on p-o-i16:20
colonwqxarses, things I should parameter checks? keystone_version = v3 and admin_token not servcie_default should fail.16:23
colonwqor should I trust users to do the right thing.16:23
*** alee_afk is now known as alee16:24
aleeEmilienM, so - how is it going?16:24
EmilienMalee: let me see16:25
*** ntpttr__ has joined #puppet-openstack16:25
EmilienMtest_encrypted_cinder_volumes fail :(16:25
EmilienMlooking at diff16:26
EmilienMI don't get why it fails16:27
EmilienMonly diff is ordering https://review.openstack.org/#/c/339806/42..44/manifests/barbican.pp16:27
aleeEmilienM, tempest role is Member only again16:27
EmilienMtempest_roles=Member16:28
EmilienMright16:28
aleeEmilienM, we must be running into cache problems again16:28
EmilienMyes16:28
EmilienMlet me report it on infra channel16:28
aleeEmilienM, ok16:28
*** pabelanger has joined #puppet-openstack16:29
EmilienMpabelanger: hi16:29
pabelangero/16:29
EmilienMpabelanger: we have a patch in p-o-i that depends on a patch that is merged16:29
EmilienMbut I think the version of puppet-tempest in the image is old and not updated16:30
EmilienMlet me show16:30
pabelangerok16:30
EmilienMcontext https://review.openstack.org/#/c/339806/16:30
EmilienMlogs are coming, a sec16:30
*** jistr|afk is now known as jistr16:30
EmilienMhttp://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_31596216:30
EmilienM2016-07-28 12:49:16.315962 | INFO:zuul.Cloner:Prepared openstack/puppet-tempest repo with branch master at commit 16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe16:30
EmilienM16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe is not master16:31
EmilienMsee https://github.com/openstack/puppet-tempest/commits/master16:31
EmilienMso something is broken somewhere16:31
EmilienMand we have CI issues because of it16:31
EmilienMsame thing with https://review.openstack.org/#/c/344916/16:31
EmilienMit looks like it's only on some nodes16:31
EmilienMlet me tell you16:32
pabelangerEmilienM: 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 code16:32
EmilienMwhat?16:32
*** tesseract- has quit IRC16:32
pabelanger16aa1cf48fcf8a03846b6cfea0568ee8a14f6afe is from your master branch16:32
EmilienMpabelanger: https://github.com/openstack/puppet-openstack-integration/blob/master/functions#L37-L4216:32
EmilienMpabelanger: yeah but it's not latest commit16:32
EmilienMuntil now, we never had this issue16:32
pabelangeryou are just expecting it to be 03d1e9762074c463cb7d148dbf27e4daf9e12f7416:33
EmilienMsomething must have changed16:33
pabelangerzuul-cloner did get an update recently16:33
openstackgerritMerged openstack/puppet-nova: Implement nova::cache in stable/mitaka  https://review.openstack.org/34824816:33
pabelangerso, I wonder if this is a regression16:33
pabelangerEmilienM: lets go back to openstack-infra and ask fungi and clarkb16:33
EmilienMoki doc16:33
*** v1k0d3n_ has joined #puppet-openstack16:37
*** derekh has quit IRC16:39
*** alee has quit IRC16:40
*** severion has quit IRC16:40
*** v1k0d3n_ has quit IRC16:41
*** v1k0d3n has joined #puppet-openstack16:42
*** ntpttr__ has quit IRC16:43
*** alee has joined #puppet-openstack16:43
*** ntpttr__ has joined #puppet-openstack16:44
*** foxhunt72 is now known as foxhunt_away16:47
*** dgurtner has quit IRC16:49
*** imcsk8|zZz is now known as imcsk816:51
*** chem is now known as chem|away17:01
*** paramite has quit IRC17:02
*** trown is now known as trown|lunch17:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/34710517:09
dfisherr10k question .. if I want to change where the git cache is, is that based on $HOME?17:12
dfisherit looks like that's the case:17:12
dfisherr10k/git/cache.rb:  def_setting_attr :cache_root, File.expand_path(ENV['HOME'] ? '~/.r10k/git': '/root/.r10k/git')17:12
iurygregoryi've never run into this problem O.o17:13
iurygregorywhy change git cache? just wondering =)17:14
*** ducttape_ has quit IRC17:14
EmilienMalee: it's an infra thing17:21
EmilienMwe're stuck now17:21
aleeEmilienM, ok - any eta on resolution?17:22
EmilienMno  idea17:24
dfisheriurygregory: 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 CA17:27
dfisherso, NFS things are … slow17:27
iurygregoryyeah  we use NFS here at the university in the lab =(17:28
dfisherso, bundle exec rake spec can take *hours* to run17:28
dfisherwell, hours to run r10k install17:28
iurygregorythat's why i use my notebook XD17:29
dfisheryeah, i should have.17:29
aleeEmilienM, ok :/  well, I guess we wait then.17:29
*** v1k0d3n has quit IRC17:33
iurygregoryalee, can you review https://review.openstack.org/#/c/347105 ?17:38
aleeiurygregory, will do17:39
iurygregoryalee, tks o/17:39
*** ducttape_ has joined #puppet-openstack17:50
*** ducttape_ has quit IRC17:55
*** v1k0d3n has joined #puppet-openstack17:57
*** ducttape_ has joined #puppet-openstack18:01
*** trown|lunch is now known as trown18:07
*** iberezovskiy is now known as iberezovskiy|off18:20
*** degorenko is now known as _degorenko|afk18:20
*** ayoung has quit IRC18:24
*** ntpttr__ has quit IRC18:26
*** alee has quit IRC18:35
iurygregorymwhahaha, can you review 347105? authtoken =) i've update the bug as well18:49
mwhahahak18:50
EmilienMiurygregory: reviewing it too18:56
iurygregorytks o/18:56
iurygregoryCI is slow =(18:56
iurygregorywaiting for lint and beaker-xenial only18:57
EmilienMiurygregory: Alex has a good question inline18:57
EmilienMiurygregory, mwhahaha: I also find it weird to fail by default if you just declare ::barbiban::api without any parameter18:58
EmilienMbecause it will include the authtoken class but since we don't have a default keystone password18:58
EmilienMwhat do you think?18:58
iurygregoryyeah i saw that with the tests .-.18:59
iurygregoryabout auth_uri we can set but we dont need both18:59
EmilienMbut it's fine18:59
EmilienMfor password18:59
EmilienMit's already failing in other modules if unset18:59
EmilienMit's just now we have a new class18:59
mwhahahaiurygregory: which one do you need? is auth_url ok as a standalone?18:59
iurygregoryauth_url work well for barbican =) since beaker is ok19:00
EmilienMbeaker doesn't test much19:01
EmilienMkeep in mind that ;)19:01
EmilienMwhat will really test barbican is what alee and I are doing in p-o-i19:01
iurygregoryEmilienM, sure o/ maybe i should provide a poc for a component that run integration?19:02
EmilienMsure19:03
iurygregoryEmilienM, ack will do tonight =)19:03
EmilienMiurygregory: I can do it now19:04
EmilienMI'll update my puppet cinder patch now19:04
iurygregoryEmilienM, ok o/19:04
iurygregoryEmilienM, do you have any talk for Barcelona?19:12
dfisherseriously, i'll vote.19:12
dfisherHE SAID SOLARIS.  +1 FOREVERS!19:12
EmilienMyeah, I have 2 proposals19:12
EmilienMabout authtoken, i would like to notify that some patches might be different for backward compatibility19:12
EmilienMmwhahaha, iurygregory  ^19:13
iurygregoryEmilienM, nice i'll look =D and ask my teammattes to vote :D19:13
EmilienMthe structure will be the same19:13
mwhahahayea19:13
EmilienMiurygregory: thx19:13
EmilienMbut some params will be different to maintain bp19:13
EmilienMbc*19:13
iurygregoryyeag19:13
iurygregoryyeah*19:13
dfisherSearch results for "Solaris"19:14
dfisher•There are no presentations that match your criteria.19:14
* dfisher sobs19:14
mwhahahawhere's your presentation then?19:14
mwhahahamr. solaris19:14
dfisheri'm so farkin' slammed right now :(19:14
dfisherhell, i didn't even write an abstract for Oracle Open World.19:14
mwhahahaoracle "open" world19:15
* dfisher sighs19:15
dfisherthis again?19:15
dfisherok.19:15
mwhahaha:D19:15
iurygregorylol19:15
mwhahahait's ok, haven't you read the openstack-dev mailing list this week? mirantis is out to get everyone19:15
dfisherat this point, i pretty much just right-click -> mark as read on that19:16
mwhahahastill not sure what it's about other than someone is taking a marketing PR a bit too seriously19:16
dfisheri don't have time to wade through 99.9% cruft to find gems  :(19:16
mwhahahaZOMG CONTAINERS AER OURS19:17
dfisherhey, speaking of that ...19:17
* mwhahaha goes back to actually attempting to work19:17
dfisherhttps://twitter.com/satokaz/status/75773042632452505619:17
mwhahahaha19:17
dfisherso, Docker's now in Solaris.19:18
mwhahahabut can it kuberentes19:18
dfisherthat's next.19:19
dfisherk8 & docker 1.13 for swarm19:19
dfisherthere's already a PR for Solaris support in Docker 1.1319:20
dfisherthat team is working on it.19:20
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446419:33
*** hjensas has quit IRC19:33
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446419:34
*** cdelatte has joined #puppet-openstack19:34
*** hjensas has joined #puppet-openstack19:35
*** foxhunt_away is now known as foxhunt7219:36
*** foxhunt72 is now known as foxhunt_away19:39
openstackgerritGuilherme Balzana proposed openstack/puppet-monasca: Using puppet-openstack standard gitignore  https://review.openstack.org/34852819:39
*** foxhunt_away is now known as foxhunt7219:40
guimalufEmilienM, 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:44
mwhahahapackage all the things!19:47
*** vinsh has quit IRC19:51
*** foxhunt72 is now known as foxhunt_away19:54
*** alee has joined #puppet-openstack19:55
openstackgerritTim Rozet proposed openstack/puppet-tripleo: Adds OpenDaylight  https://review.openstack.org/33517320:25
*** cdelatte has quit IRC20:26
*** ducttape_ has quit IRC20:38
*** ducttape_ has joined #puppet-openstack20:44
*** ntpttr__ has joined #puppet-openstack20:48
*** trown is now known as trown|outtypewww20:57
*** ducttape_ has quit IRC21:07
*** ntpttr__ has quit IRC21:09
*** ntpttr__ has joined #puppet-openstack21:09
*** ducttape_ has joined #puppet-openstack21:20
dfisherhow do I look at manifest / file content from a failed test in CI?21:25
dfisherhttp://logs.openstack.org/63/347463/6/check/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/4c0cf7b/console.html - I'd like to find21:26
dfisherhttp://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
dfisherthe contents of that file21:26
mwhahahait's the acceptance blob21:28
dfisherin the keystone repo?21:28
dfisherpuppet-keystone*21:28
dfisherajh21:29
dfisherah*21:29
dfisherfound it21:29
dfisheropenstacklib_config_provider_spec.rb21:29
dfishermwhahaha: than ks21:30
mwhahahamemcache_pool_conn_get_timeout21:30
mwhahahaer21:30
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/spec/acceptance/openstacklib_config_provider_spec.rb#L9-L3821:30
* dfisher nods21:30
dfisherthanks!21:30
*** v1k0d3n has quit IRC21:35
dfishermwhahaha: got time for yet more dumb questions from me?21:49
mwhahahafor you? no ;)21:49
mwhahahawhat's up21:49
dfisherok, so i see why CI is yelling at me.21:50
dfisherin neutron_l3_agent_config/ini_settings.rb we set transform_to / transform_to=21:50
dfisherhttps://review.openstack.org/#/c/347468/7/lib/puppet/provider/neutron_l3_agent_config/ini_setting.rb21:50
dfisherwell, I do, currently :)21:50
dfisherthe 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 5421:51
dfisherwith21:51
dfisherError: 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.pp21:51
mwhahahaso you probably need to stub i t out in the openstack_config type21:51
dfisher$ cat ../../type/openstack_config.rb21:52
dfisherPuppet::Type.newtype(:openstack_config) do21:52
dfisher  newparam(:transform_to)21:52
dfisherend21:52
dfisheri am :(21:52
dfisherwell, chem did this piece :)21:52
mwhahahahmm21:52
mwhahahathe magical land of providers21:52
dfisherhah21:52
mwhahahaso either it should be @transform_to or perhaps you need a getter/setter not completely sure21:54
dfisherok21:55
dfisherwill poke21:55
dfisherthanks21:55
mwhahahalike on line 4021:55
mwhahahawhere there's a def ensure_absent_val21:55
mwhahahawonder if you need an equivalent for the tranform_to21:55
*** v1k0d3n has joined #puppet-openstack21:57
mwhahahaso a def transform_to\n  resource[:transform_to]\n end ;)21:58
mwhahahayou get the idea21:58
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290521:58
EmilienMI GOT IT21:58
EmilienMsorry21:58
EmilienMguimaluf: /me no opinion on topic21:59
guimalufnp!21:59
* guimaluf package all the things!21:59
EmilienMiurygregory, mwhahaha: ok I did cinder authtoken https://review.openstack.org/342905 - when you have time to look22:00
EmilienMhave a good night22:00
mwhahahak22:00
dfishermwhahaha:22:03
dfisher  def transform_to22:03
dfisher    resource[:transform_to]22:03
dfisher  end22:03
dfisherthat did it22:03
* dfisher tips hat22:03
openstackgerritDrew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746322:04
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795622:06
*** vinsh has joined #puppet-openstack22:08
xarsescolonwq: should we fail/warn if both auth_token and user are set?22:08
*** morazi has quit IRC22:11
*** _ducttape_ has joined #puppet-openstack22:13
*** ducttape_ has quit IRC22:14
colonwqxarses, 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:19
xarsessomething like that, we also need tenant and password defined in v3 case22:20
xarsesif v2.0 and auth_token {} elseif v3 and user and password or something22:21
colonwqproject instead of tenant. I'll round out the tests.22:22
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Configure Ceph RGW to use Keyston V3  https://review.openstack.org/34829322:25
*** ntpttr- has quit IRC22:31
*** ntpttr- has joined #puppet-openstack22:36
*** _ducttape_ has quit IRC23:07
*** dfisher has quit IRC23:07
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795623:10
*** aderyugin has quit IRC23:17
*** ntpttr__ has quit IRC23:26
*** saneax_AFK is now known as saneax23:34
*** xarses has quit IRC23:39
*** iurygregory_ has joined #puppet-openstack23:41
*** ducttape_ has joined #puppet-openstack23:51
« Wednesday, 2016-07-27 Index Friday, 2016-07-29 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!