Wednesday, 2016-10-19

« Tuesday, 2016-10-18 Index Thursday, 2016-10-20 »
*** ahsa518 has joined #craton00:17
*** ahsa518 has quit IRC00:21
*** ahsa518 has joined #craton00:21
syed_sulo:01:08
syed_i am getting this error when i try to pass the net_interface_update test01:08
syed_https://gist.github.com/ahsan518/9e34867ef8fed9c77f280ccaabce002201:08
syed_sqlalchemy.exc.InvalidRequestError: Object '<NetInterface at 0x7f5fa5beb1d0>' is already attached to session '84' (this is '83')01:08
syed_So it says that object already exists01:08
syed_https://gist.github.com/ahsan518/9e34867ef8fed9c77f280ccaabce0022#file-error-L901:09
syed_i guess this is causing the issue maybe ....01:09
syed_oops01:19
syed_sorry, i figured that one out01:19
syed_my bad guys !01:19
*** valw has joined #craton02:02
*** valw_ has joined #craton02:04
*** valw has quit IRC02:07
*** VW has joined #craton02:45
*** VW has quit IRC03:18
*** valw_ has quit IRC03:29
*** ahsa518 has quit IRC03:29
*** syed_ has quit IRC04:24
*** VW has joined #craton05:19
*** VW has quit IRC05:23
sulosayed_: i left some comments on your patch https://review.openstack.org/#/c/38481709:45
sulosmall fixes needed09:45
sulojimbaker: yeah, i think its probably wise to not spend more time on the workflow .. i doubt that this will be ready for any sort of demo10:37
sulojimbaker: think we should just spend time on the actual presentation10:37
suloi will have some screenshots of workflow from resolver etc .. just to give context10:38
*** VW has joined #craton12:19
*** VW has quit IRC12:23
sulosigmavirus: i think i know why that test passed12:49
sigmavirussulo: oh?12:50
sulosigmavirus: i think that mock order is wrong (probalby my doing)12:50
sigmavirusaha12:50
sigmaviruslet me investigate that12:50
sigmavirussulo: looking at the docs, that is indeed the problem12:52
sigmavirus(unittest.mock docs)12:52
sigmavirusso pushing a new patchset with the user_id set to 1, and the patch ordering swapped12:52
sulodamn unit test, can make anything pass12:53
sulobut i guess kinda my fault for using the same id and not testing the first bit12:54
suloawesome catch though :)12:54
sigmavirussulo: tests caught it12:58
sigmavirusThis is also, why I like to check more than just one attribute in at est12:58
sigmavirusbut *shrug*12:58
sigmavirusI have what appears to be a very bizarre approach to testing12:59
sigmavirussulo: fixed :D https://review.openstack.org/#/c/386646/13:12
*** valw has joined #craton13:55
*** valw has quit IRC14:03
*** Tamayo has joined #craton14:07
jimbakersulo, re workflow support - i believe there's some good stuff there, but clearly not ready yet14:25
suloyeah, lets revisit the workflow stuff after the summit14:29
*** syed_ has joined #craton14:47
jimbakersulo, in terms of implementation :)14:50
jimbakeri expect the actual architecture/design will be a big focus for next week14:50
jimbakerin particular: using tooling like docker swarm14:51
syed_sulo: Thanks for the reviews. Have updated the patch.14:54
*** VW has joined #craton15:02
* sigmavirus thinks he has a way to get keystone working with craton15:03
jimbakersigmavirus, awesome15:12
*** chrisspencer_ has joined #craton15:13
*** Mudpuppy has joined #craton15:45
*** Mudpuppy has quit IRC15:48
lcastell\o/15:56
sigmavirusSo far, no joy but Soon™15:57
syed_sigmavirus: thanks Ian for the reviews15:57
syed_Glad for the keystone work :)15:57
syed_i hope it works15:57
sigmavirusI doubt it will16:00
sigmavirusEverything I touch turns to sadness16:00
jimbakersigmavirus, i'm just hearing the voice of marvin the robot now in my head...16:01
jimbakerhttp://images.amcnetworks.com/bbcamerica.com/wp-content/uploads/2014/05/marvin.jpg16:02
syed_haha16:03
syed_:D16:03
* sigmavirus has no clue who Marvin teh Robot is16:11
jimbakersigmavirus, how to get in touch with one's inner marvin: http://www.bbcamerica.com/anglophenia/2014/05/happy-towel-day-six-life-lessons-live-marvin-paranoid-android16:15
* sigmavirus keeps kicking himself for having never read that book16:20
sulosoo shits be broken ? bytes is not a 16-char string16:55
suloMessage: UnknownException("500: (builtins.ValueError) bytes is not a 16-char string16:55
jimbakersulo, context?17:01
sulosigmavirus: http://paste.openstack.org/show/586439/17:04
sigmavirussulo: how'd you encounter that?17:05
sigmavirusi.e., what was the request you made?17:05
suloget /hosts17:06
*** VW has quit IRC17:07
sigmavirussulo: what project id did you use?17:08
*** VW has joined #craton17:08
sulosome random string17:08
jimbakervs using something like uuidgen?17:09
sigmavirussulo: so with docker we use the same projectid every time17:10
*** VW has quit IRC17:12
sulonot sure i follow17:12
sigmavirussulo: so the project-id that we set-up by default is in the Dockerfile17:13
sigmavirusOr are you testing random inputs?17:13
suloi am just testing random input17:13
sigmavirusAha17:13
sulothe generation bit worked17:13
suloi guess its not too big a deal, as users will follow uuid here17:14
sigmavirusWe can add better validation in the middleware if we ant17:14
suloyeah +117:14
sigmavirusuuidutils in oslo.utils will handle that for us17:14
*** chrisspencer_ has quit IRC17:15
*** chrisspencer has joined #craton17:15
jimbakerack, we can make it better/more robust17:16
*** VW has joined #craton17:17
jimbakerjovon, http://logs.openstack.org/43/386743/2/gate/gate-craton-python35/077f0e2/console.html#_2016-10-19_17_25_05_34940117:32
jimbakersome change in self.message vs self.msg ?17:36
jimbakeranyway, should be easy enough to fix17:37
*** VW has quit IRC17:38
*** Mudpuppy has joined #craton17:39
*** valw has joined #craton17:39
*** VW has joined #craton17:39
*** VW has quit IRC17:43
*** valw has quit IRC17:56
*** VW has joined #craton17:57
*** valw has joined #craton18:01
jimbakersyed_, so merged in your update work. thanks for that18:03
jimbakerplease work with jovon on the resulting merge conflict in his code, in addition to msg vs message conflict that the subsequent tox picked up18:04
syed_Cool. Thanks jim18:04
syed_I dont think jovon is in office today18:04
jimbakersyed_, no worries, we can update that on his behalf then18:10
jimbakersyed_, do you want to take care of that?18:10
jimbakersigmavirus, sulo, https://gist.github.com/jimbaker/18123ddb6e63e84ad66b2de44fc0ff72 with the cli work in https://review.openstack.org/#/c/388091/18:11
jimbakerso that specific change looks fine to me - just some bit rot on the project id parsing here18:11
*** VW has quit IRC18:12
*** VW has joined #craton18:16
*** VW has quit IRC18:16
*** VW has joined #craton18:17
jimbakerfwiw, i think we are going to want to spend some time on the craton cli today - it seems sort of borked. so getting stuff like this18:19
jimbaker$ craton --craton-url http://127.0.0.1:8080/v1  results in ERROR: b"'Namespace' object has no attribute 'func'"18:19
sulohmm you getting that error even with my patch ?18:20
jimbakersulo, yeah - i just applied your patch18:20
suloahh .. actually i see why, so yeah other stuff broken too18:20
jimbakerand it's now in master - it's necessary, just not sufficient18:20
suloalso its incomplete18:21
suloand also missing networks / projects/ users/ netdevices etc18:21
*** valw has quit IRC18:21
jimbakeryeah - just a lot of missing stuff there18:21
suloits just not really usuable in general18:21
*** VW has quit IRC18:22
jimbakeranyway, now that core client code is actually usable and seemingly robust, we can put some effort into it18:22
jimbakersulo, btw, besides the fact that the workflow code i shared with you (everyone else here, https://gist.github.com/jimbaker/9cfc95a1a683bb22562b2c0d87811cd0) is incomplete - did that make sense?18:23
jimbakeri believe the basic layout is fine, although i would definitely would want to revisit some of the structuring. but plenty of time to think about design issues next week18:25
*** valw has joined #craton18:26
*** valw has quit IRC18:29
*** valw has joined #craton18:29
sulojimbaker: yeah i think so but i didnt go though it properly tbh18:31
jimbakersulo, np18:34
syed_Guys, https://review.openstack.org/#/c/388858/18:34
jimbakersyed_, thanks18:35
syed_@jimbaker sure18:35
syed_i will update that18:35
*** valw has quit IRC18:47
*** valw has joined #craton18:55
*** keekz has quit IRC19:10
*** keekz has joined #craton19:12
*** valw has quit IRC19:12
*** valw has joined #craton19:14
*** Tamayo has quit IRC19:24
*** valw has quit IRC19:59
*** Mudpuppy has quit IRC20:03
*** valw has joined #craton20:09
*** valw has quit IRC20:13
sigmavirusso progress on keystone integration: Keystoneauth works fine but when it sends the token to Craton I get a 422 that isn't logged *anywhere* in craton20:17
sigmavirusso, I"m trying to figure out why20:17
sigmavirusAnd I'm out for the day. See you all tomorrow20:17
*** VW has joined #craton20:33
*** Mudpuppy has joined #craton20:56
*** VW_ has joined #craton21:02
*** VW_ has quit IRC21:02
*** VW_ has joined #craton21:03
jimbakersigmavirus, thanks for the investigation21:05
*** VW has quit IRC21:05
*** valw has joined #craton21:14
jimbakersyed_, thanks for making that change for jovon - hopefully it will get through jenkins now (but i did test separately with tox!)21:23
jimbaker(and passed jenkins)21:25
*** VW_ has quit IRC21:37
syed_Thanks jimbaker21:41
*** VW has joined #craton22:21
*** VW has quit IRC22:21
*** VW_ has joined #craton22:26
*** VW_ has quit IRC22:27
*** VW_ has joined #craton22:27
*** VW_ has quit IRC22:28
*** VW_ has joined #craton22:29
*** VW_ has quit IRC22:29
*** VW has joined #craton22:30
*** harlowja has quit IRC22:33
*** valw has quit IRC22:40
*** harlowja has joined #craton22:48
*** VW has quit IRC23:01
*** VW has joined #craton23:02
*** VW has quit IRC23:02
*** VW has joined #craton23:03
*** VW has quit IRC23:04
*** VW has joined #craton23:05
*** VW has quit IRC23:06
*** VW has joined #craton23:07
*** Mudpuppy has quit IRC23:19
*** Mudpuppy has joined #craton23:19
*** Mudpuppy has quit IRC23:20
*** Mudpuppy has joined #craton23:20
*** VW has quit IRC23:41
*** Mudpuppy has quit IRC23:51
« Tuesday, 2016-10-18 Index Thursday, 2016-10-20 »

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