Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-04-12
07:29:58 frickler mhen: I haven't seen an IRC meeting and the schedule on eavesdrop doesn't seem to list one. probably outdated information in the wiki?
07:30:59 frickler mhen: regarding the other question I have no idea, maybe dtroyer_zz or mordred have some idea. but that topic may also better be suited for a discussion on the dev-ml
07:37:04 mhen For reference, this is the meeting schedule I'm referring to: http://eavesdrop.openstack.org/#OpenStackClient_Team_Meeting
07:39:51 mhen frickler, you referring to this I guess? http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
07:41:45 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477
07:42:19 frickler mugsie: fixed a small glitch in your patch first ^^
07:44:41 frickler mhen: you are correct about th mailing list
07:45:46 frickler mhen: and no idea why I missed the meeting in the schedule. probably should get dropped there if it doesn't happen anyway
07:46:12 mhen frickler, okay. Thanks for your guidance! I will try there :)
08:57:50 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: Add designate to the services deployed for functional tests https://review.openstack.org/560831
09:01:18 frickler mugsie: FYI ^^ waiting to see what others think about that now
09:17:02 openstackgerrit Graham Hayes proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477
09:19:02 openstackgerrit Graham Hayes proposed openstack/openstacksdk master: DNM: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560545
09:19:43 mugsie frickler: cool. ^^ should fix the unit tests, and I reup'd the testing patch on top as well
13:11:57 openstackgerrit Nick Jones proposed openstack/os-client-config master: Remove DataCentred from list of vendors https://review.openstack.org/560894
13:47:22 mordred mugsie, frickler: I'd be fine just adding designate to the normal functional test config for openstacksdk as long as doing so doesn't increase the memory load on the devstack such that stuff starts failing
13:49:02 mordred mugsie, frickler: however, if adding it to the default doesn't work, the senlin folks have a patch up adding a senlin-enabled job, we could also add designate to that (and name it somethin gdifferent)
14:01:10 gtmanfred mordred: have you had a chance to look into the floating ip thing? This is basically all we are doing right now, is create_server with wait=True, and auto_ip=True https://github.com/saltstack/salt/blob/2018.3/salt/cloud/clouds/openstack.py#L644
14:04:09 mugsie mordred: cool - I don't think we will add a huge amount of extra pressure, and chewing up an entire extra node for 3 tests feels like a bad thing :/ - lets hope it fits :)
16:00:46 elmiko cdent around?
16:00:53 cdent dag nabbit
16:14:34 openstackgerrit Merged openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234
16:37:28 edleafe cdent: elmiko: https://etherpad.openstack.org/p/api-sig-newsletter is ready for your review
16:37:37 elmiko edleafe: ack, looking
16:38:19 elmiko edleafe: lgtm
16:38:31 cdent edleafe: yeah, agree, lgtm, short and sweet
16:38:46 edleafe ok, sending it to the clouds...
16:58:39 openstackgerrit Merged openstack/os-client-config master: Remove DataCentred from list of vendors https://review.openstack.org/560894
17:07:43 cdent elmiko, edleafe: the merge of the http split has broken the gate jobs: https://review.openstack.org/#/c/554921/
17:08:04 cdent I'm on the phone right now, but if nobody steps up I'll see if I can make it go
17:12:23 elmiko cdent: does it just need a kick?
17:14:47 elmiko ok, those errors look a little beyond my current knowledge of the system. i think you better check it when you can cdent
17:19:01 cdent elmiko: roger elmiko
17:19:49 elmiko cdent: sorry, wish i could be more helpful =(
17:19:54 cdent no problem
17:23:53 edleafe the error on that doesn't make sense, does it?
17:24:10 elmiko it didn't to me, but then i'm really out of touch with this stuff
17:24:28 edleafe it just says that the docs have to end in .rst or .json
17:26:48 cdent elmiko, edleafe: it is reading the dir 'http' as a file, yes? (still on phone)
17:27:57 elmiko hmm, maybe
17:29:44 edleafe cdent: how did the http patch ever make it through, then?
17:30:26 edleafe oh, it didn't run the py27 tests on that
17:32:03 edleafe Unknow directive type "toctree" - http://logs.openstack.org/21/554921/1/gate/openstack-tox-py27/f3ef842/job-output.txt.gz#_2018-04-12_17_00_26_938269
17:36:08 edleafe Yeah, I just checked that test. It's doing glob.glob("guidelines/*"), which pulls in the 'guidelines/http' entry
17:36:16 edleafe Want me to push a fix?
17:40:32 cdent edleafe: if you can that would be dandy
17:43:05 openstackgerrit Ed Leafe proposed openstack/api-wg master: Fix the checking of directories for extensions https://review.openstack.org/560984
17:43:16 edleafe cdent: ^^
17:43:37 edleafe Do we have to revert the HTTP merge first?
17:43:47 cdent I don't think so
17:44:41 cdent edleafe: I assume you tried it locally and was fine?
17:44:57 edleafe yep
17:46:56 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068
17:48:56 edleafe cdent: looking at that test, it bugs me that we use the builtin name 'file' as a local variable name. I'll push a fix for that, but no rush on that
17:49:23 cdent yeah, that stuff is super old, predates us I think
17:55:29 openstackgerrit Ed Leafe proposed openstack/api-wg master: Remove use of Python builtin name for variables https://review.openstack.org/560994
17:55:39 edleafe cdent: no rush on this ^^
17:55:50 cdent yay!
17:57:34 edleafe looks like etowes is the culprit, from Idf0dc2aa1fd1e10ed3dfb953838e6ea469c76933
17:58:26 edleafe but cdent +1'd that one! :-P
17:58:57 cdent oh the humanity
17:59:12 cdent but in a tool, it's a good opportunity for a newbie to come along and clean up later...newbie
18:00:42 edleafe I always hated giving interviewees live coding challenges. IMO, they are a terrible predictor of what they know and how they work. Instead, I wrote some code with varying degrees of wrongness buried within.
18:01:24 edleafe I then asked them to review the code, and that showed how well they understood Python better than pretty much anything else I could come up with
18:02:51 cdent that seems a reasonable too
18:02:54 cdent l
18:03:05 cdent did you contextualize the environment?
18:03:14 cdent I fear that in openstack we fail to do that
18:03:19 cdent and tend to overshoot
18:05:40 edleafe "contextualize the environment" - you and your big city words
18:18:45 cdent says the man wh spent a lot of time in new york
19:53:30 openstackgerrit Merged openstack/api-wg master: Fix the checking of directories for extensions https://review.openstack.org/560984
20:00:57 edleafe cdent: elmiko: ^^
20:28:46 elmiko edleafe: \o/
20:36:47 openstackgerrit Chris Dent proposed openstack/api-wg master: Update the errors guidance to use service-type for code https://review.openstack.org/554921
20:47:44 openstackgerrit Chris Dent proposed openstack/api-wg master: Add guidance on needing cache-control headers https://review.openstack.org/550468
20:52:07 cdent elmiko, edleafe ^ fixed the merge conflict
20:53:18 edleafe w00t!
22:46:56 elmiko cdent: nice, thanks!
#openstack-sdks - 2018-04-13
00:15:24 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix _get_entity to accept a dict https://review.openstack.org/561078
00:16:00 mordred ianw: ^^ there it is in patach form
00:18:01 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Abort waiting for image on 404 https://review.openstack.org/561080
00:26:29 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix _get_entity to accept a dict https://review.openstack.org/561078
00:32:09 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Abort waiting for image on 404 https://review.openstack.org/561080
00:32:09 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Don't throw errors when deleting nonexistant images https://review.openstack.org/561081
00:33:13 mordred ianw: ^^ three different patches that should make image deleting better
00:33:33 mordred ianw: one of them was already half-written locally - I'd just forgotten about it
07:39:22 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: Fix DNS Recordset CRUD https://review.openstack.org/560477
11:35:04 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068
13:27:58 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add methods to get all of the version data https://review.openstack.org/559154
13:33:25 frickler mugsie: seems there are two issues still with 560477: a) we need different zone names because the func tests seem to run in parallel. maybe autogenerate random zone names?
13:34:01 mugsie frickler: yeah, I was looking at that. I think a random name is a good idea.
13:34:03 frickler b) get_recordset doesn't work with a name and I'm not sure whether we really want to fix that, might get more complicated than it's worth
13:34:24 frickler http://logs.openstack.org/68/559068/7/check/openstacksdk-functional-devstack-tips/9d04dba/testr_results.html.gz
13:35:13 mugsie damn. I think that could be s follow up patch in anycase
13:35:17 mugsie be a*
13:35:34 frickler the third failure is on me, I'll look into that
13:35:52 mugsie sounds like a plan
13:41:31 frickler oh, that's because the rebase rebased me off my other patch
13:42:22 frickler mugsie: maybe after mordred already kind-of approved this, you can merge https://review.openstack.org/560831 into your patch so that it becomes self-testing

Earlier   Later