Monday, 2017-01-30

« Sunday, 2017-01-29 Index Tuesday, 2017-01-31 »
*** spotz_zzz is now known as spotz00:08
*** spotz is now known as spotz_zzz00:18
*** dimtruck is now known as zz_dimtruck01:14
*** spotz_zzz is now known as spotz02:09
*** spotz is now known as spotz_zzz02:20
*** spotz_zzz is now known as spotz04:12
*** spotz is now known as spotz_zzz04:21
*** spotz_zzz is now known as spotz05:12
*** spotz is now known as spotz_zzz05:22
*** liujiong has joined #openstack-barbican05:38
*** spotz_zzz is now known as spotz06:13
*** spotz is now known as spotz_zzz06:23
*** toabctl has joined #openstack-barbican06:54
*** spotz_zzz is now known as spotz07:14
*** spotz is now known as spotz_zzz07:24
*** andreas_s has joined #openstack-barbican07:25
*** mkoderer has joined #openstack-barbican07:26
*** shohel has joined #openstack-barbican08:15
*** spotz_zzz is now known as spotz08:15
*** pcaruana has joined #openstack-barbican08:15
*** spotz is now known as spotz_zzz08:25
*** spotz_zzz is now known as spotz10:16
*** mkoderer has quit IRC10:16
*** spotz is now known as spotz_zzz10:26
*** spotz_zzz is now known as spotz11:17
*** spotz is now known as spotz_zzz11:27
*** spotz_zzz is now known as spotz12:18
*** shohel has quit IRC12:24
*** catintheroof has joined #openstack-barbican12:25
*** spotz is now known as spotz_zzz12:28
*** shohel has joined #openstack-barbican12:45
*** dave-mccowan has joined #openstack-barbican13:00
*** jmckind has joined #openstack-barbican13:00
*** spotz_zzz is now known as spotz13:19
*** spotz is now known as spotz_zzz13:29
*** shohel has quit IRC13:31
*** mkoderer has joined #openstack-barbican13:38
*** jperry has joined #openstack-barbican14:26
*** agrebennikov__ has joined #openstack-barbican14:31
*** alee has quit IRC14:34
*** alee has joined #openstack-barbican14:35
*** jperry has quit IRC14:36
*** jperry has joined #openstack-barbican14:38
openstackgerritdane-fichter proposed openstack/barbican-tempest-plugin: Added Secret Metadata client and unit tests  https://review.openstack.org/42481714:38
*** edtubill has joined #openstack-barbican14:38
*** liujiong has quit IRC14:51
*** liujiong has joined #openstack-barbican14:56
*** chris_hultin|AWA is now known as chris_hultin15:06
openstackgerritdane-fichter proposed openstack/barbican-tempest-plugin: Added Secret Metadata client and unit tests  https://review.openstack.org/42481715:09
*** spotz_zzz is now known as spotz15:10
*** nkinder has joined #openstack-barbican15:16
*** kfarr has joined #openstack-barbican15:21
*** jaosorior has joined #openstack-barbican15:23
*** zz_dimtruck is now known as dimtruck15:26
*** jaosorior has quit IRC15:27
*** jaosorior has joined #openstack-barbican15:32
*** kfarr has quit IRC15:33
*** tdink has joined #openstack-barbican15:45
*** jperry has quit IRC15:56
openstackgerritdane-fichter proposed openstack/barbican-tempest-plugin: Added Secret Metadata client and unit tests  https://review.openstack.org/42481716:00
*** tdink has quit IRC16:05
*** tdink has joined #openstack-barbican16:05
openstackgerritAde Lee proposed openstack/barbican: add ipavault plugin  https://review.openstack.org/42682516:13
aleejaosorior, ^^16:13
jaosorioralee: great, lets see16:13
aleejaosorior, all still untested and wip right now ..16:14
aleejaosorior, these are the errors I am currently getting http://paste.openstack.org/show/596911/16:16
jaosorioralee: running tox on it16:16
aleejaosorior, yeah - you're going to need to modify your tox environment by copying stuff in16:16
aleejaosorior, because the ipa stuff is not yet pip installable16:16
jaosorioralee: that's no biggie, it's better for me to see the bug there, that way I can mock it easier16:17
aleejaosorior, (christian working on that)16:17
aleejaosorior, understood.16:17
jaosoriorwow16:17
jaosoriorgot an error with the tox16:17
jaosoriorinstalling python-ldap16:17
aleejaosorior, yeah16:17
jaosoriordafuq16:17
jaosorioralee: did you get those from global requirements?16:18
aleejaosorior, let me recall what I had to copy in ..16:18
jaosorioralee: let me just fix up the requirements16:18
jaosorioralee: do we add those to requirements.txt or to test-requirements.txt ?16:19
aleejaosorior, cp -r /usr/lib/python2.7/site-packages/ipaplatform .tox/py27/lib/python2.7/site-packages/16:19
aleejaosorior, cp -r /usr/lib/python2.7/site-packages/ipapython .tox/py27/lib/python2.7/site-packages/16:19
alee cp -r /usr/lib/python2.7/site-packages/ipalib .tox/py27/lib/python2.7/site-packages/16:19
jaosoriorfuck16:19
jaosoriorpython-ldap is not in openstack requirements16:19
jaosorioralee: we have ldap3 available though, would you be able to use that?16:20
aleejaosorior, of course it isn't16:20
aleejaosorior, maybe - not sure -t he python-ldap requirement comes from within ipa code16:21
aleejaosorior, we don't use it in barbican plugin16:21
aleejaosorior, lets just get it working first - and then try to figure out the deps.16:22
jaosorioralee: so then we probably should add the ipa requirement to bindep.txt16:22
jaosoriorcause adding stuff to requirements.txt that way could get problematic16:22
jaosorioralee: I went with the deps since it's the first thing that failed when trying to do tox16:23
aleejaosorior, no thats a good point16:23
aleejaosorior, whats bindep.txt supposed to be for?16:24
jaosorioralee: any non-pip requirements16:24
jaosoriorAFAIK16:24
aleeah16:24
jaosorioralee: would be better to ask in the infra channel16:24
jaosorioranyway, trying it again without gssapi and ldap as dependencies16:25
*** liujiong_lj has joined #openstack-barbican16:31
*** liujiong has quit IRC16:32
*** nkinder has quit IRC16:34
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: add ipavault plugin  https://review.openstack.org/42682516:36
*** briancurtin has joined #openstack-barbican16:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: add ipavault plugin  https://review.openstack.org/42682516:43
*** briancurtin has left #openstack-barbican16:43
*** andreas_s has quit IRC16:45
*** armstrong has joined #openstack-barbican16:50
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: add ipavault plugin  https://review.openstack.org/42682516:56
*** tdink has quit IRC16:59
*** tdink has joined #openstack-barbican17:04
*** kfarr has joined #openstack-barbican17:04
*** nkinder has joined #openstack-barbican17:07
*** nkinder has quit IRC17:17
*** jsavak has joined #openstack-barbican17:23
openstackgerritKaitlin Farr proposed openstack/castellan: Remove outdated comment  https://review.openstack.org/40988717:41
jaosorioralee: so I copied the paths that you told me but I still don't get to the issue you mentioned17:42
jaosoriorstill getting issues importing ipalib from the mock17:43
jaosoriorI did get past that by mocking the ipalib import17:43
jaosoriorbut now it fails referencing ipav.CONF, which wouldn't be available when the import fails17:43
*** kfarr has quit IRC17:43
aleejaosorior, brb17:45
aleejaosorior, ok - here is what I did ..17:50
aleejaosorior, pull up change, then tox  -r -e py2717:51
aleejaosorior, this fails --- then .. copy in the stuff I mentioned17:52
aleejaosorior, then  tox   -e py2717:52
jaosorioralee: I did just that17:55
aleejaosorior, and you're still getting error referencing ipav.CONF ?17:56
openstackgerritKaitlin Farr proposed openstack/castellan: Add ability to get only metadata  https://review.openstack.org/41255817:56
*** jsavak has quit IRC17:57
openstackgerritKaitlin Farr proposed openstack/castellan: Add ability to get only metadata  https://review.openstack.org/41255817:57
openstackgerritFernando Diaz proposed openstack/barbican: Add Unit Tests for Consumers API Controller  https://review.openstack.org/42554817:57
jaosorioralee: yes17:58
aleejaosorior, can you paste error?17:59
jaosorioralee: http://paste.openstack.org/show/596924/17:59
aleejaosorior, ah ok -- actaully I had that different before ..18:00
*** tdink has quit IRC18:02
aleejaosorior, can you try printing out the ImportError exception?18:02
aleejaosorior, to see if I missed something18:02
jaosorioralee: so that was when I mocked ipalib18:06
jaosorioralee: wait up18:06
jaosoriorafter letting the code as you had it, I get this18:07
jaosorioralee: can you reply to this https://review.openstack.org/#/c/423510/2 ?18:08
jaosorioralee: and here's what I get http://paste.openstack.org/show/596925/18:08
aleejaosorior, sorrry - in meeting - responses delayed ..18:08
aleejaosorior, responded18:13
jaosoriorthanks18:14
aleejaosorior, can you try printing out an exception in case the import failed .. so something like this ..18:17
aleejaosorior, you can move from barbican.plugin import ipavault as ipav to above the try block18:19
aleeso it will always get called18:19
aleeand then you'll see why its failing18:19
jaosorioralee: http://paste.openstack.org/show/596926/18:22
aleejaosorior, and you did copy over ipalib?18:23
jaosorioralee: I did what I pasted there in the beginning of the logs18:23
jaosoriorI mean the paste18:23
aleejaosorior, ok - you missed ipalib18:24
aleecp -r /usr/lib/python2.7/site-packages/ipalib .tox/py27/lib/python2.7/site-packages/18:24
jaosorioralee: ok, it's not fialing in the ldap module18:24
jaosoriorImportError: No module named ldap.dn18:24
aleeok -- yeah - I had that too -- let me see what I did --18:25
aleejaosorior, I thought adding python-ldap in requirements.txt fixed it18:25
jaosorioralee: I can't even do the tox command with python-ldap in the requirements cause it fails building it18:26
jaosorior:/18:26
aleejaosorior, it doesn't fail if you doi manualy ...18:26
*** armstrong has quit IRC18:27
aleejaosorior, I'm pretty sure I didn't  have to copy it in -- but I guess you can do that till we figure it out18:28
*** kfarr has joined #openstack-barbican18:29
aleejaosorior, I'm confused - with python-ldap in requirements.txt , running tox -r -e py27 fails ?18:31
aleejaosorior, or just fails in the gate?18:32
jaosoriorboth18:33
*** tdink has joined #openstack-barbican18:33
kfarrdave-mccowan redrobot the castellan gate (and I'm assuming the python-barbicanclient gate) is broken from the ID work that merged in last week.  I'm guessing https://review.openstack.org/#/c/382643/ will fix it?18:36
dave-mccowankfarr yikes!18:42
openstackgerritKaitlin Farr proposed openstack/barbican: Add CryptoPluginUnsupportedOperation  https://review.openstack.org/41770918:46
*** liujiong_lj has quit IRC18:57
dave-mccowanHappy Monday, Barbicaneers!  Weekly IRC Meeting starts in 45 minutes in #openstack-meeting-alt.19:15
jaosoriorkfarr, dave-mccowan, redrobot: So... will that fix it? Seems that the barbican issue is blocking RDO image promotion19:22
jaosoriorcause it fails the tests19:22
jaosorioralee: ^^19:23
dave-mccowanjaosorior kfarr redrobot it seems like it helps it (fixes for secret.id), but does not fix all the cases (like orders.id)19:24
jaosoriordave-mccowan: one option is to revert that change19:25
dave-mccowanjaosorior that's an option.  we have to do something pretty quick... we're supposed to cut RC-1 this week.19:26
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: Revert "Add ID property to all entities"  https://review.openstack.org/42689319:27
jaosoriordave-mccowan: ^^19:27
*** jaosorior is now known as jaosorior_lunch19:28
aleedave-mccowan, how come the gate tests did not pick this up?  seems like we need to add gate tests for barbican client/ castellan19:32
dave-mccowanalee +1 we should run castellan and client tests for each commit to barbican repo.19:33
aleedave-mccowan, I wont be able to make meeting -- but I'll check online later to see whats up ..19:34
aleedave-mccowan, if we can't fix quickly, we need to revert19:34
dave-mccowanalee i think we need to revert.  especially since we have already tagged the Ocata release for Barbican client.19:35
*** diazjf has joined #openstack-barbican19:35
dave-mccowancan you +1 or +2 https://review.openstack.org/42689319:35
dave-mccowanalee i'd like to get a broad agreement to revert or not.19:36
*** mkoderer has quit IRC19:46
*** jmckind_ has joined #openstack-barbican19:47
diazjfdave-mccowan just commented on that patch19:48
*** jmckind has quit IRC19:49
diazjfWhen is the cutoff date? We still have some time to add bug fixes, but will need someone to commit.19:49
diazjfkfarr, queueing the castellan patches to re-review this week :)19:50
dave-mccowandiazjf we're supposed to tag a release candidate by the end of this week.  barbicanclient's deadline was last week.  (i've already tagged the release, so any changes will be a maintenance release).19:51
dave-mccowandiazjf kfarr castellan is not on the official cycle... maybe we should change that?19:52
diazjfdave-mccowan it makes sense to revert the change then. I will +2. kfarr, I think it should have official releases now that it is more mature. Lmk what you think.19:54
kfarrdave-mccowan, back now19:57
kfarrdave-mccowan, what do official releases give us?  It's been kind of nice not being on a schedule19:59
*** tdink has quit IRC19:59
*** tdink has joined #openstack-barbican20:01
dave-mccowanmeeting starting in #openstack-meeting-alt20:01
*** dimtruck is now known as zz_dimtruck20:29
*** kfarr_ has joined #openstack-barbican20:39
*** kfarr has quit IRC20:39
*** kfarr_ has quit IRC20:40
*** kfarr has joined #openstack-barbican20:40
*** tdink_ has joined #openstack-barbican20:51
*** tdink has quit IRC20:51
*** tdink_ has quit IRC20:53
*** tdink has joined #openstack-barbican20:53
*** zz_dimtruck is now known as dimtruck21:11
aleejaosorior_lunch, you around?21:12
*** diazjf has quit IRC21:19
*** jaosorior_lunch is now known as jaosorior21:21
jaosorioralee: now I am21:21
jaosoriorwhat's up?21:21
aleejaosorior, so first up -- e2e test ..21:21
aleejaosorior, my overcloud deployment failed ..21:22
jaosorioralee: where did it fail?21:22
aleeand it looks like it could not resolve dns21:22
aleethat is -- I go onto the nodes and I see that cloud-init was run ..21:22
jaosorioralee: did you add the DnsServers value? I think it was in your blog21:22
jaosorioralee: cause that's dynamic, so I can't put it in the enable-internal-tls.yaml21:23
aleeah no ... I figured that was in your patches ..21:23
aleejaosorior, let me check my blog ..21:23
jaosorioralee: ok, remember I had mentioned, I can't put the DNS in the environment, cause that will vary per deployment21:23
*** EmilienM has joined #openstack-barbican21:24
EmilienMhey21:24
EmilienMdave-mccowan: https://review.openstack.org/426930 FYI21:24
jaosoriorEmilienM: thanks dude21:24
EmilienMjaosorior: my pleasure21:24
jaosoriordave-mccowan: besides the server-side revert, is there any client-side revert we should do?21:25
aleeEmilienM, cool21:26
dave-mccowanjaosorior i'm not sure.  we should review the merged queue.  hopefully not, since i already cut the Ocata release for python-barbicanclient.21:27
dave-mccowanjaosorior merged queue looks good, i think we're in the clear now.21:28
jaosoriorgreat21:29
*** kfarr has quit IRC21:34
jaosoriordave-mccowan: so EmilienM is adding this patch https://review.openstack.org/#/c/426930/ that will basically test barbican changes via tripleo. We basically just boot a node from an encrypted volume, so it's pretty simple, however, it's good for testing integration21:35
EmilienMyeah and the job is non voting21:36
EmilienMit won't block you never, ever21:36
EmilienMjust tell us when it's read21:36
EmilienMred*21:36
EmilienMand we'll tell if it's because of your patch or not21:37
EmilienMactually ask jaosorior  :P21:37
*** catintheroof has quit IRC21:40
*** tdink has quit IRC21:40
*** catintheroof has joined #openstack-barbican21:42
dave-mccowanthanks EmilienM!  this is great!21:47
jaosoriorEmilienM: why is puppet set as the suffix21:47
EmilienMjaosorior: because we have to duplicate the jobs21:48
EmilienMjaosorior: regular scenarios are triggered on specific files changes21:48
jaosorioroh, right21:48
EmilienMbut zuul v2.5 doesn't support both rules in same time21:48
*** jmckind has joined #openstack-barbican21:48
jaosoriorand we can't trigger it this way cause we want it to be triggered on every barbican change21:48
jaosoriormakes sense21:48
jaosoriorEmilienM: anyway id zuulv3 fixes this?21:48
EmilienMjaosorior: yes21:50
*** jmckind_ has quit IRC21:50
*** catinthe_ has joined #openstack-barbican21:52
jaosoriordave-mccowan: anyway, if you can take a look at that patch, we need your +1 :D21:52
*** catintheroof has quit IRC21:53
*** jmckind has quit IRC22:02
*** diazjf has joined #openstack-barbican22:03
*** catintheroof has joined #openstack-barbican22:04
*** catinthe_ has quit IRC22:07
*** catinthe_ has joined #openstack-barbican22:17
*** catinth__ has joined #openstack-barbican22:18
*** catintheroof has quit IRC22:18
*** catinthe_ has quit IRC22:22
*** diazjf has quit IRC22:25
*** catinth__ has quit IRC22:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/barbican: add ipavault plugin  https://review.openstack.org/42682522:46
*** edtubill has quit IRC23:01
*** catintheroof has joined #openstack-barbican23:50
*** dimtruck is now known as zz_dimtruck23:57
*** jamielennox is now known as jamielennox|away23:58
*** jamielennox|away is now known as jamielennox23:59
« Sunday, 2017-01-29 Index Tuesday, 2017-01-31 »

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