Tuesday, 2015-03-03

openstackgerritDouglas Mendizábal proposed openstack/python-barbicanclient: Add secret_type to Secret constructor  https://review.openstack.org/16056100:02
*** zz_dimtruck is now known as dimtruck00:02
*** kfox1111 has joined #openstack-barbican00:05
kfox1111So, what can I do to get more reviewers looking at the spec and such?00:09
kfox1111Still would kind of like to get it into kilo if possible.00:09
openstackgerritOpenStack Proposal Bot proposed openstack/barbican: Updated from global requirements  https://review.openstack.org/16056400:13
*** chlong has quit IRC00:15
*** igueths has quit IRC00:17
*** jkf has quit IRC00:41
*** kfox1111 has quit IRC01:12
*** dimtruck is now known as zz_dimtruck01:38
*** bdpayne has quit IRC01:45
*** lisaclark1 has joined #openstack-barbican01:52
*** crc32 has joined #openstack-barbican02:49
*** lisaclark1 has quit IRC02:50
*** bdpayne has joined #openstack-barbican03:07
openstackgerritSteve Heyman proposed openstack/python-barbicanclient: Updated test to use both identity v2 and v3  https://review.openstack.org/16061903:16
*** bdpayne has quit IRC03:21
*** zz_dimtruck is now known as dimtruck03:30
*** crc32 has quit IRC04:22
*** dimtruck is now known as zz_dimtruck04:29
*** david-lyle_afk has joined #openstack-barbican05:06
*** rm_you has joined #openstack-barbican05:19
*** rm_you has quit IRC05:19
*** rm_you has joined #openstack-barbican05:19
*** dave-mccowan has quit IRC05:38
*** david-lyle_afk has quit IRC05:44
*** rm_you| has joined #openstack-barbican06:17
*** rm_you has quit IRC06:19
*** redrobot has quit IRC06:21
*** gyee has quit IRC06:22
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Fix functional tests to use new auth provider interface  https://review.openstack.org/16045506:24
*** redrobot has joined #openstack-barbican06:25
*** redrobot is now known as Guest3254406:25
*** jaosorior has joined #openstack-barbican06:30
*** russell_h has quit IRC06:38
*** david-lyle_afk has joined #openstack-barbican06:44
*** jroll has quit IRC06:49
*** jroll has joined #openstack-barbican06:59
*** xaeth_afk has quit IRC07:04
*** dstufft has joined #openstack-barbican07:14
*** xaeth_afk has joined #openstack-barbican07:16
*** AnxiousGarlic has joined #openstack-barbican07:20
*** woodster_ has quit IRC07:20
*** AnxiousGarlic has left #openstack-barbican07:20
*** openstackgerrit has quit IRC07:22
*** openstackgerrit has joined #openstack-barbican07:22
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Use urljoin instead of os.path.join  https://review.openstack.org/16025007:33
*** dstufft has left #openstack-barbican07:56
*** eglute has quit IRC08:26
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Use urljoin instead of os.path.join  https://review.openstack.org/16025008:30
*** eglute has joined #openstack-barbican08:31
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Get rid of Repositories class  https://review.openstack.org/16032908:44
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-barbicanclient: Add default max payload to functional test conf registry  https://review.openstack.org/16071810:00
*** darrenmoffat has quit IRC10:09
*** darrenmoffat has joined #openstack-barbican10:11
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Use oslo_log in db-manage script  https://review.openstack.org/16073510:58
*** nickrmc83 has quit IRC11:33
*** nickrmc83 has joined #openstack-barbican11:39
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-barbicanclient: Do not filter empty strings  https://review.openstack.org/16077212:32
*** dave-mccowan has joined #openstack-barbican12:48
*** jroll has quit IRC13:07
*** jroll has joined #openstack-barbican13:07
*** woodster_ has joined #openstack-barbican13:37
*** alee has quit IRC13:42
*** rellerreller has joined #openstack-barbican13:47
*** nkinder has quit IRC14:24
*** chlong has joined #openstack-barbican14:27
jaosoriorwoodster_, jvrbanac, hockeynut:  so yesterday I had a fox for the gate issues but I had messed up the import order, but here's the fixed CR https://review.openstack.org/#/c/160455/14:29
openstackgerritSteve Heyman proposed openstack/python-barbicanclient: Updated functional tests to use both identity v2 and v3  https://review.openstack.org/16061914:35
hockeynutjaosorior sounds good.  always a good idea to have a fox at the gate :-)14:39
hockeynutjaosorior also thx for catching the "V2" in my CR...updated version is up14:39
jaosoriorLol fix14:50
*** kgriffs|afk is now known as kgriffs14:51
*** alee has joined #openstack-barbican14:54
*** zz_dimtruck is now known as dimtruck14:58
woodster_arunkant, I added a question to the audit spec please15:06
*** nkinder has joined #openstack-barbican15:08
*** ametts has quit IRC15:10
*** xaeth_afk is now known as xaeth15:11
*** igueths has joined #openstack-barbican15:13
woodster_all: please review this one, to finally get rid of the passed-around repos!: https://review.openstack.org/#/c/16032915:14
jaosorioryay :D15:17
jaosoriorand sorry for the big commit15:17
woodster_I didn't realize how invasive that repo stuff really was!15:18
jaosoriorhockeynut: can you elaborate on your comment here? https://review.openstack.org/#/c/160250/4/functionaltests/common/client.py15:18
hockeynutjaosorior sure.  we've merged https://review.openstack.org/#/c/127865/ so the /v1 is no more in the URL15:19
hockeynutjust suggesting we remove that from the comment15:19
jaosoriorhockeynut: while v1 is no longer in the url, it will still be used to access the API, as far as I can tell. It is even reflected by this change: https://review.openstack.org/#/c/158132/15:21
jaosoriorso the change you are referring to is a change to the keystone endpoints, but the version will still be used when accessing the server15:22
jaosorioror... did I get it wrong?15:22
hockeynutI think that means that its host/secrets and not host/v1/secrets15:22
jaosoriorhockeynut: it will still be host/v1/secrets15:23
jaosorioronly thing is that the keystone endpoint is now registered to host, and not host/v115:23
hockeynutjaosorior so we will add the v1 ourselves rather than depending on it coming out of service catalog from keystone15:23
jaosoriorat least that's the way it's implemented in the client at the moment15:23
jaosoriorI guess getting that out of the keystone catalog should be done in another way?15:24
jaosoriorat the moment doing host/secrets returns 40415:24
*** paul_glass has joined #openstack-barbican15:24
hockeynuthmm...checking on my side too15:24
jaosoriorwhich is what used to happen accidentally before I appended the slash there15:24
jaosorioractually, yeah, keystone should be returning the version. At least readon from the tempest code. But still, the version is going to be in the url15:30
jaosorioror, well, it's needed in the url15:31
jaosoriorbut actually, the intent of the comment wasn't related to the version, but actually, to the fact that URLs without the slash in the end would not be joined correctly with the urlparse.join15:31
jaosorior* urlparse.urljoin15:31
jaosoriorme adding the localhost/v1 was actually just an example15:32
*** lisaclark1 has joined #openstack-barbican15:36
*** SheenaG1 has joined #openstack-barbican15:40
jaosoriorhockeynut: do you happen to know why the tests are ran inside the post_test_hook.sh in the functionaltests folder?15:41
hockeynutjaosorior good question - I will guess that it is to ensure that we run AFTER everything else.  Might want to check with jvrbanac since I believe he added this initially15:44
*** kfarr has joined #openstack-barbican15:44
hockeynutjaosorior ok, my confusion is coming to an end :-)  I see that we DO have v1 in our URL...just not in the service catalog15:48
hockeynutso I retract my comment in your CR15:48
*** kfarr has quit IRC16:04
*** rellerreller has quit IRC16:04
*** atiwari has joined #openstack-barbican16:14
*** dave-mccowan has quit IRC16:15
*** kfox1111 has joined #openstack-barbican16:23
*** lisaclark1 has quit IRC16:28
*** lisaclark1 has joined #openstack-barbican16:28
jaosoriorkfox1111: evening.16:29
*** dimtruck is now known as zz_dimtruck16:29
*** lisaclark2 has joined #openstack-barbican16:32
*** lisaclark1 has quit IRC16:32
kfox1111so I noticed the Sahara spec.16:34
jaosoriorWell, it's evening here :P . Uhm... haven't checked that. Got a link?16:35
kfox1111Reading through it, it mentions a draback being it still has to handle the secrets. I think that would be unnessisary with the vm integration spec.16:35
kfox1111just a sec...16:35
*** david-lyle_afk has quit IRC16:37
jaosoriorShit, it's merged16:41
jaosoriorWas gonna comment on it16:41
*** SheenaG11 has joined #openstack-barbican16:41
kfox1111Maybe put a comment on the vm-integration spec then too?16:42
jaosoriorYup, gotta give it a read16:42
*** SheenaG1 has quit IRC16:42
jaosoriorWill do after climbing16:42
kfox1111ok. thx.16:43
*** zz_dimtruck is now known as dimtruck16:43
hockeynutreaperhulk https://review.openstack.org/#/c/160055/1 just needs werkflow16:43
jaosoriorFirst time I read it I was gonna comment that I wanted a bit more description on the use-case. But I got distracted and I didn't comment. Not sure now how the spec is at the moment since you have updated it already16:44
kfox1111look at the comments in version 3. I've been fleshing out the use case in there,16:44
kfox1111and when its clear, I'll post it in the spec.16:44
reaperhulkhockeynut: k16:45
jaosoriorhockeynut, reaperhulk: actually I didn't feel entirely good about that NewRelic CR either, but I didn't come up with an alternative. That's why I didn't comment on it16:46
hockeynutjaosorior please express your concerns16:47
reaperhulkProbably the same ones I have16:47
reaperhulkIt's highly specific to new relic16:47
reaperhulkIt'd be nice for it to be agnostic, but I don't know how to accomplish that without violating hacking and causing other annoyances16:47
reaperhulkSo basically until I see someone else say "I want <not new relic>" we can ignore it. When that person appears we will have them refactor it in a more generic manner16:48
reaperhulkwhatever that manner may be :D16:48
hockeynutsome kind of monitoring plugin perhaps?  then we can provide newrelic as one type?16:48
Guest32544all we need is moar plugin16:48
reaperhulkThat might work, but would like to stay as lightweight as possible16:48
reaperhulk(aka plz no plugins)16:49
hockeynutI use plugin in a generic way here.  extension, configuration, add-on pick your favorite16:49
jaosoriorreaperhulk: actually yeah. It felt too specific to that implementation16:50
*** Guest32544 is now known as redrobot16:51
*** gyee has joined #openstack-barbican16:58
openstackgerritMerged openstack/barbican: Fix functional tests to use new auth provider interface  https://review.openstack.org/16045517:06
openstackgerritDouglas Mendizábal proposed openstack/barbican-specs: Spec for secret pre-encryption  https://review.openstack.org/12840117:09
*** russell_h has joined #openstack-barbican17:14
*** russell_h has quit IRC17:14
*** russell_h has joined #openstack-barbican17:14
*** chlong has quit IRC17:17
*** lisaclark2 has quit IRC17:17
openstackgerritMerged openstack/barbican: Adding NewRelic monitoring to worker tasks  https://review.openstack.org/16005517:25
openstackgerritMerged openstack/python-barbicanclient: Add secret_type to Secret constructor  https://review.openstack.org/16056117:31
*** kfox1111 has quit IRC17:33
openstackgerritOpenStack Proposal Bot proposed openstack/barbican: Updated from global requirements  https://review.openstack.org/16056417:38
*** dimtruck is now known as zz_dimtruck17:48
*** zz_dimtruck is now known as dimtruck17:51
*** lisaclark1 has joined #openstack-barbican17:58
*** bdpayne has joined #openstack-barbican18:00
*** alee is now known as alee_lunch18:11
*** dave-mccowan has joined #openstack-barbican18:12
*** crc32 has joined #openstack-barbican18:20
jaosoriorreaperhulk: 12-pack related code -> https://review.openstack.org/#/c/160329/18:32
*** kgriffs is now known as kgriffs|afk18:40
*** rellerreller has joined #openstack-barbican18:41
jaosoriorkfox1111: are you around?18:46
openstackgerritMerged openstack/python-barbicanclient: Updated functional tests to use both identity v2 and v3  https://review.openstack.org/16061918:47
*** lisaclark1 has quit IRC18:58
iguethsjaosorior: Seems he dropped...18:59
jaosorioryeah, I saw19:00
*** alee_lunch is now known as alee19:15
*** lisaclark1 has joined #openstack-barbican19:22
*** lisaclark2 has joined #openstack-barbican19:26
*** lisaclark1 has quit IRC19:28
elmikohey barbicaneers, i'm trying to get a working barbicanclient with keystone middleware authenticating against my devstack.19:35
elmikoso, far i've got barbican running on my dev machine. i'm able to make http calls from my devstack machine to the barb instance.19:36
elmikoand i've added key-manager service to my catalog19:36
elmikobut i keep getting 404's from the barbicanclient when attempting to store a secret, any advice?19:37
redrobotelmiko does the service catalog entry include "v1" in the URL?19:37
elmikoi tried to copy the config from contrib/devstack/lib/barbican19:37
*** lisaclark2 has quit IRC19:38
elmikothe main difference is that i don't have a barbican service user created, i am using the default admin user for access from barbican-api-paste.ini19:38
*** lisaclark1 has joined #openstack-barbican19:39
*** lisaclark1 has quit IRC19:39
elmikoi should probably just try and integrate it into my devstack, but i wanted to walk through the steps to be sure i understand how it fits into the stack19:39
*** lisaclark1 has joined #openstack-barbican19:39
redrobotelmiko hmm...  I would try to set up a break point and examine the url that gives you the 404.19:40
elmikoredrobot good idea, i'll dig a little further. thanks =)19:41
lisaclark1barbicaneers:  i've got CRs that I'd love to get eyes on!  any takers?19:43
reaperhulkholy shit jaosorior, that is a monster CR19:45
jaosoriorreaperhulk: Yeaaaah... I'm not keen of it being so big myself. But I started trying to make it by parts, and it didn't work out19:45
jaosoriorreaperhulk: I'm happy, though, that it's more negative lines than positive :P19:46
*** kfox1111 has joined #openstack-barbican19:46
jaosorior* more deletions than additions19:46
openstackgerritThomas Dinkjian proposed openstack/python-barbicanclient: Adds positive orders functional tests  https://review.openstack.org/15845419:46
reaperhulkYeah I looked and it is one of those situations where it's super hard to split it up...19:46
woodster_But that cr removes soooo much cruft! :)19:48
openstackgerritThomas Dinkjian proposed openstack/python-barbicanclient: Adds positive orders functional tests  https://review.openstack.org/15845419:49
jaosoriorhaving written all that, now I'm not that sure how keen I am of the singleton factories from a maintenance point of view. Since there is a lot of witchery going on by patching those factory methods... buuuut it does look better than the Repositories class19:50
openstackgerritThomas Dinkjian proposed openstack/python-barbicanclient: Moved DD test from smoke to functional  https://review.openstack.org/16049019:54
elmikoredrobot: ok, for reasons that aren't exactly clear to me yet the calls to store the secret is filling in my keystone endpoint for the _barbican_endpoint in the Client19:56
*** jorge_munoz_ has joined #openstack-barbican19:57
reaperhulkjaosorior: Yeah the lazy module global really frustrates me19:58
reaperhulkI've got some vague ideas on how to resolve that, but we'll see19:58
redrobotelmiko so, if you're using the catalog, make sure that you're not giving an endpoint to the Client object, and that the catalog entry for 'key-manager' is the barbican url19:58
openstackgerritNathan Reller proposed openstack/barbican: Fixed Binary Encoding to Secret Stores  https://review.openstack.org/15741019:58
jaosoriorrellerreller: you added your tags accidentally to that CR19:59
reaperhulk(mostly around whether we can call parse_args almost immediately and therefore not have to lazy init things after import)20:00
elmikoredrobot: i'm creating the client with an Session taken from a keystoneclient.v3.Client object that i created. should that possible?20:00
elmiko*that be*20:00
rellerrellerjaosorior Aargh!!!20:00
redrobotelmiko like this? http://docs.openstack.org/developer/python-barbicanclient/authentication.html20:00
elmikoredrobot: not exactly, sec. i'll paste20:00
*** jorge_munoz_ has quit IRC20:01
elmikoredrobot: http://paste.openstack.org/show/186760/20:01
reaperhulkjaosorior: merged, god help us ;)20:02
openstackgerritNathan Reller proposed openstack/barbican: Fixed Binary Encoding to Secret Stores  https://review.openstack.org/15741020:02
rellerrellerjaosorior Pushed another version without tags. Thanks for catching that. I should put that in .gitignore.20:03
elmikoredrobot: just for reference, http://paste.openstack.org/show/186761/ (service catalog)20:04
jaosoriorreaperhulk: shit's gonna get real20:04
rellerrellerreaperhulk redrobot Can you review CR 157410, https://review.openstack.org/#/c/157410/ woodster_ gave me +1 but wanted you guys to comment on it before +2.20:04
jaosoriorelmiko: looks alright20:05
rellerrellerI need this to get the content type work done.20:05
redrobotrellerreller will look at it today after meetings20:06
elmikojaosorior: i keep getting 404s when i try to store a secret from the client in the first paste. it's pulling the keystone endpoint and trying to post http://{KEYSTONE_ENDPOINT}/v1/secrets20:06
rellerrellerredrobot thanks!20:06
redrobotelmiko very strange... code looks correct. :-\20:08
jaosorioryou're sure that that endpoint works, right?20:08
jaosoriorI mean, you have tried doing manually a request, using curl or something of the sort20:08
elmikojaosorior: yup, i've tried httpie commands from the node that keystone is on, and from the node that i'm running the script on20:09
redrobotelmiko I would pdb into the keystone session stuff, b/c get_endpoint looks good in our code http://docs.openstack.org/developer/python-barbicanclient/authentication.html20:09
redrobotelmiko I mean https://github.com/openstack/python-barbicanclient/blob/master/barbicanclient/client.py#L11120:09
elmikoredrobot: thanks, back to the mines!20:09
jamielennoxelmiko: you shouldn't need to do that20:18
elmikojamielennox: to do what?20:19
jamielennoxelmiko: if you are doing user pass use keystoneclient.auth.identity.v3.Password(auth_url=..., username=..., password-...)20:19
jamielennoxthen just keystoneclient.session.Session(auth=plugin)20:19
jamielennoxdon't steal it from ksc20:19
elmikojamielennox: part of this is that i'm trying to do hacking to understand how we'll fit this into sahara. we have some utility functions to create keystone Client objects, maybe i'll just need to extend that and allow getting Session objects as well.20:20
jamielennoxelmiko: the intent with a lot of this session work is to seperate the CRUD aspects of the client from the auth aspects20:21
jamielennoxelmiko: if you find yourself having to create a client with auth parameters then stop20:21
jamielennoxi don't know how sahara works though20:21
redrobotjamielennox just curious.  the barbicanclient supports sessions.  are you saying the session should not be shared?  only the auth object?20:22
elmikojamielennox: i can certainly add utility functions to get a Session instead, i would like to understand the differences better though.20:22
jamielennoxredrobot: you can share sessions, just there is a circular dependency that was put in place as a hack for keystoneclient, if you create keystoneclient the old way it creates a session, but the auth is weird20:23
elmikojamielennox: is there a good place in the keystone docs that illuminated the differences between Clients and Sessions?20:23
redrobotelmiko http://docs.openstack.org/developer/python-keystoneclient/using-sessions.html20:23
jamielennoxi had a blog a while ago20:23
elmikoredrobot: thanks20:23
elmikojamielennox: awesome, thanks20:24
jamielennoxolder: http://www.jamielennox.net/blog/2014/02/24/client-session-objects/20:24
reaperhulkat some point soon we should have some code merging...20:24
elmikocool, i've got some nice light afternoon reading now =)20:25
jaosoriorjamielennox: as feedback I must say that I've found your blog really useful20:26
jamielennoxjaosorior: excellent! i'm never sure if people read it20:26
jaosoriorjamielennox: if you've found views from Finland (or Sweden) in the statistics. That was provably me20:27
*** gyee has quit IRC20:28
jamielennoxi haven't looked at the stats in ages20:28
openstackgerritMerged openstack/barbican: Get rid of Repositories class  https://review.openstack.org/16032920:28
reaperhulkand now I owe jaosorior beers20:29
jamielennoxsome of the posts take a fair bit of time, and then you look at the stats and like 90% of my traffic was from a 2 paragraph 'why am i getting this error message in auth_token middleware'20:29
jaosoriorreaperhulk: hehehe20:29
*** kebray has joined #openstack-barbican20:29
*** lisaclark1 has quit IRC20:31
openstackgerritKevin Fox proposed openstack/barbican-specs: Spec for vm-integration  https://review.openstack.org/15957120:33
*** lisaclark1 has joined #openstack-barbican20:34
elmikojust to followup, it does work if i use the method from the docs20:34
*** kebray has quit IRC20:41
*** lisaclark1 has quit IRC20:44
*** rellerreller has quit IRC20:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Enable secret decrypt through 'payload' resource  https://review.openstack.org/15706820:46
jaosoriorwoodster_: just had to rebase for that CR ^^20:47
jaosoriorredrobot: this CR originated from a recommendation of yours :D https://review.openstack.org/#/c/160250/ got some minutes to check it out?20:54
jaosoriorwoodster_: ping20:55
*** lisaclark1 has joined #openstack-barbican20:56
redrobotjaosorior shouldn't you be sleeping?20:57
redrobotjaosorior LGTM20:58
jaosoriorredrobot: got no meetings tomorrow, so I'll work from home (a bar...)20:59
*** dimtruck is now known as zz_dimtruck20:59
jaosoriorso no need to wake up early20:59
jaosoriorredrobot: ...and and my current state is: Drinking craft beer, eating manchego cheese and reviewing code. I'm quite entertained at the moment20:59
*** kebray has joined #openstack-barbican21:00
elmikojaosorior: ++21:01
*** paul_glass has quit IRC21:02
*** paul_glass has joined #openstack-barbican21:04
*** lisaclark1 has quit IRC21:07
*** zz_dimtruck is now known as dimtruck21:07
iguethsjaosorior: += insert_metal_band_name_here21:08
jaosoriorthe band is Capharnaum, at the moment :P21:10
iguethsAh ha!21:10
*** jkf has joined #openstack-barbican21:11
*** kebray has quit IRC21:20
*** rellerreller has joined #openstack-barbican21:29
*** kgriffs|afk is now known as kgriffs21:50
woodster_jaosorior: hello21:54
jaosoriorwoodster_: Have you had any news regarding the DigiCert dude? Haven't heard from that :/21:58
openstackgerritMerged openstack/barbican: Use urljoin instead of os.path.join  https://review.openstack.org/16025022:03
*** dave-mccowan has quit IRC22:11
hockeynutjaosorior ping?22:20
jaosoriorhockeynut: pong22:22
hockeynuthaving a problem with tempest auth_provider fix.  I think you have a newer tempest than I22:22
hockeynutcan you check pip freeze | grep tempest ?22:22
jaosoriorprobably, can you paste the error?22:23
jaosorior...and, are you using the stable version of tempest?22:23
jaosoriorcause the fix was for the latest22:23
jaosoriorbasically this commit: https://github.com/openstack/tempest/commit/90012355b4ea24a7321f66b329b543f306d2cefe22:24
hockeynutmine fails because tempest_manger is a module22:24
hockeynutbasically I had to take the old code and change tempest_client to tempest_manager to convince to work with my version of tempest22:24
hockeynutwondering if my version works with BOTH...22:25
jaosoriornow that would be cool22:26
openstackgerritMerged openstack/barbican: Validate character set used for payload when encoding is base64  https://review.openstack.org/16017222:26
*** lisaclark1 has joined #openstack-barbican22:27
*** lisaclark1 has quit IRC22:27
*** rellerreller has quit IRC22:30
*** gyee has joined #openstack-barbican22:33
jaosoriorredrobot: ping22:34
jaosoriorhockeynut: To be honest I just tested against the latest tempest (meaning I got it from github...since from what I understood, that's what the gate tests with), and I didn't check if it was compatible from what you get from pip. Which was a mistake. So I'm guessing code to make it backwards compatible will be needed. And just now that you told me I realized.22:36
jaosoriorWant me to code that? or do you wanna do it?22:36
hockeynutI made a small change - worked with my older tempest, now I've updated to 4.stuff and trying that now22:37
hockeynut...and it worked.22:37
hockeynutI'll slam it up - take a look22:37
openstackgerritMerged openstack/barbican: Update devstack to run tests both sequentially and in parallel  https://review.openstack.org/16001622:38
openstackgerritMerged openstack/barbican: Updated from global requirements  https://review.openstack.org/16056422:39
openstackgerritSteve Heyman proposed openstack/barbican: Let functional tests run with tempest 3.dev*  https://review.openstack.org/16101422:41
hockeynutjaosorior take a peek: https://review.openstack.org/#/c/161014/22:42
openstackgerritMerged openstack/barbican: Use oslo_log in db-manage script  https://review.openstack.org/16073522:42
*** SheenaG11 has quit IRC22:43
openstackgerritMerged openstack/barbican: Fix bug in tests assuming order is active  https://review.openstack.org/15792022:44
jaosoriorhockeynut: that's how the code used to look, right?22:44
hockeynutyes, but uses tempest.manager rather than tempest.client22:44
jaosoriorlets see22:45
jaosoriorbecause as far as I could see from the tempest commit I showed you, the manager shouldn't have that call anymore22:45
jaosoriorso probably there will need to be a try: ... except: to use one way or another22:46
hockeynutI mean tempest_manager and tempest_clients22:46
jaosoriorlets see when the CI shows the result22:46
hockeynutthat works22:47
hockeynut(meaning that's ok with me!)22:47
iguethsHow long should it /normally/ take for a full Tox run to complete?22:47
hockeynutigueths with a clean env it takes a while to download/install all of the reqs22:47
hockeynutigueths otherwise only a few mins22:48
iguethshockeynut: Yeah I already have an environment that I'm working out of, so in theory it shouldn't take too long...Looking back I can actually see that it's running things. Part of it too was that when I first tried it I was on a shit network with upwards of 3000 plus MS of latency.22:50
*** openstackgerrit has quit IRC22:51
*** openstackgerrit has joined #openstack-barbican22:52
hockeynutjaosorior the error I got before my fix: AttributeError: 'module' object has no attribute 'get_auth_provider'22:52
jaosoriorhockeynut: yeah, that's because the function was moved from the class to being a function in the module in the commit I pointed out22:54
jaosoriorso, I propose it ends like this (update to your CR incoming)22:54
jaosoriorwell, lets wait for the CI first, not sure how long it will take22:54
jaosoriorhockeynut: damn...it's gonna take a while22:55
hockeynutjaosorior I still see get_auth_provider(credentials) in tempest/manager.py (line 62)22:55
hockeynutjaosorior devstack.run(speed='slowest')22:56
jaosoriorhockeynut: well, the CI hasn't even started yet running the tests22:56
hockeynutI have to run for daughter violin lesson - I'll check when I get back22:57
jaosoriormight not be here though22:57
jaosoriorbut just in case22:57
jaosoriorif the tests fail, a try ... except AttributeError: ... would be needed22:57
jaosoriorbut get_auth_provider in line 62 is no longer in the class, but it's a function in the module22:59
jaosoriorthe indentation went to the beginning of the line22:59
jaosorioranyway, see ya22:59
*** jkf has quit IRC23:04
*** nkinder has quit IRC23:04
*** igueths has quit IRC23:06
*** xaeth is now known as xaeth_afk23:10
*** paul_glass has quit IRC23:15
*** dimtruck is now known as zz_dimtruck23:16
*** alee has quit IRC23:19
openstackgerritMerged openstack/python-barbicanclient: Convert CRUD tests to requests-mock  https://review.openstack.org/14844723:23
openstackgerritMerged openstack/python-barbicanclient: Remove cyclical dependency  https://review.openstack.org/14844823:26
*** chlong has joined #openstack-barbican23:32
kfox1111seems like today has become the ghost of meeting day. :/23:43

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