| *** Mudpuppy has quit IRC | 01:41 | |
| *** valw has joined #craton | 01:48 | |
| *** Mudpuppy has joined #craton | 02:05 | |
| *** harlowja has quit IRC | 02:57 | |
| *** valw has quit IRC | 04:25 | |
| *** valw has joined #craton | 04:26 | |
| *** valw has quit IRC | 04:31 | |
| *** Mudpuppy has quit IRC | 04:58 | |
| *** tojuvone has joined #craton | 06:13 | |
| *** valw has joined #craton | 06:27 | |
| *** valw has quit IRC | 06:32 | |
| *** tojuvone_ has joined #craton | 06:51 | |
| *** tojuvone__ has joined #craton | 06:51 | |
| *** tojuvone__ has quit IRC | 06:52 | |
| *** tojuvone has quit IRC | 06:52 | |
| *** tojuvone_ has quit IRC | 06:52 | |
| *** tojuvone has joined #craton | 07:01 | |
| sulo | o/ | 08:49 |
|---|---|---|
| *** valw has joined #craton | 10:29 | |
| *** valw has quit IRC | 10:33 | |
| *** tojuvone has quit IRC | 11:42 | |
| *** tojuvone has joined #craton | 11:46 | |
| -openstackstatus- NOTICE: We are currently having capacity issues with our ubuntu-xenial nodes. We have addressed the issue but will be another few hours before new images have been uploaded to all cloud providers. | 12:17 | |
| sulo | is http://paste.openstack.org/show/589870/ something on client side | 13:24 |
| sulo | ? | 13:24 |
| *** VW has joined #craton | 13:33 | |
| sigmavirus | sulo: looks to be | 13:41 |
| sulo | sigmavirus: yeah it looks to be missing a few things | 13:43 |
| sigmavirus | I wouldn't phrase it that way | 14:00 |
| sigmavirus | but I'll take a clook at it | 14:00 |
| sulo | process_header is missing too ... i know you are wokring on redoing a few things there .. so i am wondering | 14:03 |
| sulo | if i should wait untill that to do the functional test thing | 14:04 |
| sulo | or maybe just do with requests for now ? and change to use client later | 14:04 |
| sigmavirus | Oh I didn't realize you were doing functional testing with the client. yeah, I would not suggest that ever | 14:05 |
| sulo | ever ? | 14:05 |
| sigmavirus | Projects tend to use requests directly. It more closely resembles what a regular user might do who isn't tied to python | 14:05 |
| sulo | right | 14:05 |
| sigmavirus | (because requests is rather generic in how it works0 | 14:05 |
| sulo | ok, sounds good to me .. thats what i was doing to begin with .. i thought you jimbaker wanted client used there | 14:06 |
| sulo | my misunderstanding | 14:06 |
| *** valw has joined #craton | 14:30 | |
| *** valw has quit IRC | 14:35 | |
| jimbaker | sulo, sigmavirus, others - meeting in 2 min | 14:59 |
| jimbaker | git-harry, ^^^ | 14:59 |
| jimbaker | palendae, ^^^ | 15:02 |
| jimbaker | in #openstack-meeting-4 of course | 15:02 |
| palendae | Here, vidyo? Have another video meeting at this time | 15:02 |
| jimbaker | palendae, no, just our usual scheduled irc meeting | 15:03 |
| palendae | Roger | 15:03 |
| jimbaker | palendae, but totally optional for you :) | 15:03 |
| palendae | Will at least lurk. Also pop it on my calendar | 15:03 |
| jimbaker | palendae, sounds good | 15:03 |
| jimbaker | i can send you the invite | 15:03 |
| *** Mudpuppy has joined #craton | 15:11 | |
| *** valw has joined #craton | 15:20 | |
| *** VW has quit IRC | 15:26 | |
| *** valw has quit IRC | 15:31 | |
| *** valw has joined #craton | 15:32 | |
| *** valw has quit IRC | 15:33 | |
| *** valw has joined #craton | 15:33 | |
| *** valw has quit IRC | 15:35 | |
| tojuvone | Hi guys | 15:35 |
| *** valw has joined #craton | 15:36 | |
| sigmavirus | Ho tojuvone! | 15:36 |
| sigmavirus | *Hi | 15:36 |
| sigmavirus | I see you found your way from the mailing list :) | 15:36 |
| tojuvone | yeah :) Otherwise not that far.. | 15:36 |
| tojuvone | I'm strugling my devstack env after which can continue to setup craton on it | 15:37 |
| sulo | jimbaker: git-harry: sigmavirus: oops i totally lost track of time here .. missed the meeting ... reading log now | 15:39 |
| jimbaker | sulo, no worries | 15:40 |
| jimbaker | sulo, do you want to just describe your forthcoming week? | 15:40 |
| jimbaker | also, https://review.openstack.org/#/c/399305/ needs to be reviewd | 15:41 |
| jimbaker | virtualized variables should just be a matter of being to dynamically update the resolution order with a plugin; plus some additional scaffolding | 15:42 |
| sulo | ok looks like a short meeting there ... | 15:42 |
| sulo | yeah, so i have a few pending tasks that i need to cleanup | 15:42 |
| sulo | testing : https://review.openstack.org/#/c/397872/ | 15:43 |
| sulo | ready for some review | 15:43 |
| sulo | i have one more patch that i need to cleanup | 15:43 |
| sulo | after that i am going to start poking at workflows again | 15:43 |
| jimbaker | sulo, cool re workflows | 15:43 |
| jimbaker | i suggest we start from that partial patch i started | 15:44 |
| jimbaker | maybe i can clean that up so you have something to build upon? | 15:44 |
| jimbaker | i think the db modeling is good for it | 15:44 |
| sulo | sure | 15:44 |
| sulo | is it in gerrit ? | 15:44 |
| sulo | actually no i dont see anything open for that from you | 15:45 |
| jimbaker | https://gist.github.com/jimbaker/0b7b8fcebce65bbe176116d1c0b11653 | 15:45 |
| jimbaker | sulo, ^^ | 15:45 |
| sulo | ah cool, thx | 15:45 |
| sulo | i can start from that | 15:45 |
| jimbaker | it was too early for gerrit, at least for me :) | 15:45 |
| jimbaker | sulo, here's some code to exercise - https://gist.github.com/jimbaker/9cfc95a1a683bb22562b2c0d87811cd0 | 15:46 |
| jimbaker | basically i had done a partial rework of the original test code | 15:46 |
| sulo | cool. i might team up with git-harry on that go get more eyes on it .. he is in UK so its easy to peer-program on things | 15:47 |
| sulo | if he is up for it that is | 15:47 |
| jimbaker | also the other thing to note is the complete duhh i experienced here, after talking with harlowja - https://gist.github.com/jimbaker/9cfc95a1a683bb22562b2c0d87811cd0#file-cab_workflow-py-L202 | 15:48 |
| git-harry | sulo: sure, we can talk about it tomorrow. | 15:48 |
| jimbaker | if we look at the flow listener api - _task_receiver(self, state, details) means we get the new state in the arg passed; the old state is in the details | 15:49 |
| sulo | git-harry: ok | 15:49 |
| jimbaker | it's possibly questionable we are using _flow_receiver, _task_receiver - but despite the naming of those methods, they are used elsewhere in the taskflow codebase, not just in the module where they are defined | 15:50 |
| jimbaker | sulo, anyway that first gist (the diff on models.py) should be a good start. the second gist just shows one way of how to use | 15:51 |
| jimbaker | after this we need to work on the worker stuff, and nailing that down | 15:51 |
| sulo | jimbaker: cool. i will have to refresh mysefl on all of taskflow again so ill go though all of this | 15:52 |
| jimbaker | sulo, sounds good | 15:52 |
| jimbaker | certainly reach out as you see anything | 15:52 |
| jimbaker | but i'm pretty happy with the db modeling i did | 15:52 |
| jimbaker | it seemed to capture exactly what we wanted - we could readily query a workflow for its tasks, and find out those states. with some indexing, we could have good queries on that too | 15:54 |
| jimbaker | meanwhile the definitions (workflow_def, task_def) are kept separate | 15:54 |
| *** syed__ has joined #craton | 15:55 | |
| jimbaker | it's possible that the description of the work performed in workflow_def, task_def captures enough. my assumption was that if we had packaged with the Dockerfile, then simply running commands sequentially against the worker would suffice | 15:56 |
| jimbaker | but not described here is how the worker gets its environment variables | 15:57 |
| jimbaker | which i assume come out of variables somehow | 15:57 |
| jimbaker | anyway... a number of decision points we can settle on | 15:57 |
| jimbaker | syed__, hi | 15:58 |
| syed__ | Hi jim | 15:58 |
| jimbaker | syed__, http://eavesdrop.openstack.org/meetings/craton/2016/craton.2016-11-21-15.00.html | 15:58 |
| jimbaker | syed__, could you give a brief standup for what you plan to work on this week? | 15:59 |
| syed__ | I will be working over fixing the schemas for client along with tests, and some more client work. Also was looking into the raised bug called need testing for network | 15:59 |
| syed__ | so will be try doing as much as possible in all that per this week | 16:00 |
| syed__ | i have some sample api calls along with responses noted, so maybe i can add that to docs for regions hosts cells individually, will make it look more cooler | 16:00 |
| syed__ | jimbaker: did we put in all the points for rbac in blueprint | 16:02 |
| jimbaker | syed__, no, that's what i'm working on :) | 16:02 |
| syed__ | jimbaker: i see some interesting discussion this meeting for rbac :) | 16:02 |
| syed__ | looking into it more | 16:02 |
| jimbaker | syed__, yes, that's a rough high level outline of what will be in the bp | 16:03 |
| jimbaker | some interesting discussion of whether to be polymorphic or not, but given that's a general approach we have taken, i don't see why a little more wouldn't hurt | 16:04 |
| jimbaker | imho, it has been a great modeling decision for us | 16:05 |
| *** VW has joined #craton | 16:05 | |
| jimbaker | why a little more *would* hurt... get my negatives straight! | 16:05 |
| *** VW has quit IRC | 16:10 | |
| sigmavirus | jimbaker: left a stream-of-consciousness style review for git-harry, sulo, and you Ib459b9824815d6cac0eaa9e544e13b90fb61ecc1 | 16:15 |
| sigmavirus | gah, copied the change-id https://review.openstack.org/400198 | 16:15 |
| *** VW has joined #craton | 16:17 | |
| jimbaker | sigmavirus, git-harry - nice, and i appreciate anything that simplifies our logic and use | 16:17 |
| *** valw has quit IRC | 16:26 | |
| *** valw has joined #craton | 16:35 | |
| *** tojuvone has quit IRC | 16:50 | |
| *** jovon has joined #craton | 16:58 | |
| *** VW has quit IRC | 17:06 | |
| *** VW has joined #craton | 17:19 | |
| *** VW has quit IRC | 17:24 | |
| sigmavirus | jimbaker: so, the conversation on that review is more of what I want your attention on | 17:33 |
| sigmavirus | I think that "simplification" is pointing out rather blatantly that the shape and semantics of our API design presently is bad | 17:33 |
| *** VW has joined #craton | 17:34 | |
| sulo | sigmavirus: we talked abou change that at one point but never did | 17:36 |
| sulo | sigmavirus: this seems to be a good time to do it, since its back in attention | 17:36 |
| sulo | its a smallish change too .. so not too bad | 17:37 |
| sigmavirus | yeah | 17:37 |
| *** VW has quit IRC | 17:38 | |
| sigmavirus | btw, sulo would you mind dumping info into a bug on https://bugs.launchpad.net/python-cratonclient about the problems you ran into with debug logging? | 17:38 |
| sigmavirus | Just want to link my review to a bug | 17:38 |
| sulo | sigmavirus: will do | 17:38 |
| sigmavirus | Thanks sulo | 17:38 |
| sigmavirus | You can also just assign me out of the gate | 17:38 |
| *** VW has joined #craton | 17:40 | |
| sigmavirus | this is going to wreak havoc on the client (when we make those URLs more logical) | 17:44 |
| sigmavirus | removing the Inventory object is already going to be a huge change :/ | 17:44 |
| sulo | https://bugs.launchpad.net/craton/+bug/1643650 | 17:47 |
| openstack | Launchpad bug 1643650 in craton "python-cratonclient: session object seems to be missing some properties" [Undecided,New] - Assigned to Ian Cordasco (icordasc) | 17:47 |
| sigmavirus | Thanks sulo ! | 17:47 |
| sulo | yeah region stuff is broken too .. i didnt bother creating issues for it cuz i thought you are changing it altogether | 17:47 |
| sulo | for example its doing this: https://github.com/openstack/python-cratonclient/blob/master/cratonclient/v1/regions.py#L30 | 17:48 |
| jimbaker | sigmavirus, sulo - sounds good about further refactoring | 17:59 |
| jimbaker | of the api | 17:59 |
| jimbaker | the point of filtered_context was to remove boilerplate. removing even more, or changing the api, sounds good to me | 18:00 |
| *** harlowja has joined #craton | 18:14 | |
| sigmavirus | sulo: just make bugs | 18:36 |
| sigmavirus | bugs bugs bugs | 18:36 |
| sigmavirus | syed__: and jovon can pick up those bugs too if you make them | 18:36 |
| sigmavirus | gives them discrete work items | 18:36 |
| syed__ | Sounds good sigmavirus | 18:37 |
| jovon | agreed | 18:37 |
| *** valw has quit IRC | 18:47 | |
| *** valw has joined #craton | 18:49 | |
| *** valw has quit IRC | 19:05 | |
| *** valw has joined #craton | 19:07 | |
| *** valw has quit IRC | 19:22 | |
| *** valw has joined #craton | 19:23 | |
| *** valw has quit IRC | 19:29 | |
| *** valw has joined #craton | 19:34 | |
| *** VW has quit IRC | 20:01 | |
| *** VW has joined #craton | 20:01 | |
| *** VW has quit IRC | 20:06 | |
| *** VW has joined #craton | 20:10 | |
| *** valw has quit IRC | 20:23 | |
| *** valw has joined #craton | 20:23 | |
| *** valw has quit IRC | 20:28 | |
| *** valw has joined #craton | 20:29 | |
| *** valw has quit IRC | 20:29 | |
| *** valw has joined #craton | 20:30 | |
| *** valw has quit IRC | 20:31 | |
| *** valw has joined #craton | 20:31 | |
| *** VW has quit IRC | 21:06 | |
| *** VW has joined #craton | 21:06 | |
| *** VW has quit IRC | 21:09 | |
| *** VW has joined #craton | 21:10 | |
| *** Mudpuppy_ has joined #craton | 22:01 | |
| *** valw has quit IRC | 22:01 | |
| *** Mudpuppy has quit IRC | 22:04 | |
| *** Mudpuppy_ has quit IRC | 22:05 | |
| jimbaker | +1 | 22:19 |
| *** rainya has joined #craton | 22:28 | |
| *** rainya has quit IRC | 22:32 | |
| *** rainya has joined #craton | 22:33 | |
| *** rainya has quit IRC | 22:41 | |
| *** rainya has joined #craton | 22:44 | |
| *** VW has quit IRC | 22:49 | |
| *** rainya has quit IRC | 22:54 | |
| *** VW has joined #craton | 22:54 | |
| *** rainya has joined #craton | 22:55 | |
| *** rainya has quit IRC | 23:00 | |
| *** VW_ has joined #craton | 23:10 | |
| *** VW has quit IRC | 23:12 | |
| *** VW_ has quit IRC | 23:14 | |
| *** rainya has joined #craton | 23:19 | |
| *** rainya_ has joined #craton | 23:19 | |
| *** rainya has quit IRC | 23:23 | |
| *** VW has joined #craton | 23:39 | |
| *** openstack has joined #craton | 23:44 | |
| *** rainya_ has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!