| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-12 | |||
| 06:44:57 | frickler | mugsie: seems the unit test will need added mocking for the new calls you inserted, let me know if you want help with that | |
| 07:02:00 | mhen | Hello! | |
| 07:03:17 | mhen | This channel is also related to the OpenStack client, correct? | |
| 07:08:28 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068 | |
| 07:11:51 | frickler | mhen: yes, feel free to ask your real question. we should also consider making the channel topic more inviting | |
| 07:12:22 | mhen | okay | |
| 07:20:34 | mhen | Is there no IRC meeting for the client anymore? I found a schedule and agendas online but the most recent seems to be mid 2017. | |
| 07:24:04 | mhen | background is: I'm trying to implement image signing functionality into the OpenStack client (which currently has to be done manually using 'openssl' commands). I'd like to discuss if this would be worth a contribution and if so, where it would be best to put related crypto code (openstack-sdk or cursive library?). | |
| 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. | |