Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-12-07
18:21:59 elmiko edleafe: =)
18:28:11 elmiko cdent: i hope it didn't sound like i was advocating the other way. it sounds like we have a good consensus though
18:28:38 cdent somebody may wish to clarify? not sure, I'm not caught up on the thread
18:29:08 elmiko good idea, i'll take another look to see if i was ambiguous and could potentially make it clearer
18:29:15 elmiko or maybe just link to this talk in chat
18:32:15 cdent but we have our professional reputations to consider!!
18:34:40 openstackgerrit Ed Leafe proposed openstack/api-sig master: Add idempotent DELETE guidance https://review.openstack.org/616610
18:35:06 edleafe elmiko: cdent: ^^ Cleaned up the wording. Let me know if you think it's better
18:36:06 elmiko edleafe: ack
#openstack-sdks - 2018-12-10
11:26:37 dtantsur cdent, edleafe, /me git rm -f /all/the/things && git commit -m "My work here is done" && git push -f
11:26:54 cdent yes!!!
12:31:14 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Fix requesting specific fields from ironic https://review.openstack.org/623059
13:38:00 edleafe dtantsur:
13:38:32 edleafe dtantsur: oh, crap - now we're all out of a job. Nothing left to do!
13:47:30 dtantsur what do you mean? we still can rewrite everything in rust!
13:48:07 cdent I think you just used up openstack's ninth life. game over.
13:48:19 dtantsur lol
13:58:30 dtantsur mordred: morning! new ironic bug fix (sigh :) ready for review: https://review.openstack.org/#/c/623059/
14:50:22 mordred yay bug fixes!
14:53:40 mordred dtantsur: in https://review.openstack.org/#/c/623059/4/openstack/baremetal/v1/chassis.py you set _max_microversion = 1.8 ... is that supposed to be min?
14:58:09 dtantsur mordred: it's max, because we don't know/support higher versions for chassis (there is none so far)
15:00:29 mordred kk. just checking
15:01:36 cdent APPARENTLY MY HINT WAS TOO PASSIVE. GIVE ME COFFEE
15:09:22 mordred dtantsur: if you have a sec, https://review.openstack.org/#/c/621153 from gtema looks good to me, but it touches many files
15:09:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
15:09:41 dtantsur mordred: will check
15:09:50 mordred and it's in support of that patch, that I just rebased
15:16:23 mordred mriedem: would you look at https://review.openstack.org/#/c/621695 and let me know if the api calls added to the compute proxy seem like the right thing to add? (they look fine to me, but I've never worked with host aggregates)
15:19:16 mriedem ack
15:19:55 mordred dtantsur, Shrews: also, eyeballs on https://review.openstack.org/#/c/618993 would be welcome
15:22:40 cdent make sure you form that query right or you'll get a partial cdent resource
15:23:09 dtantsur OMG
15:26:24 mordred cdent: did you know you can use graphql inside of react javascript just as a form to manipulate in-memory data objects?
15:26:48 cdent well, thank goodness for that, all my problems are solved
15:30:30 mordred cdent: I only know this because it's used in the gatsby static site generator tool: https://github.com/emonty/jubilant-fortnight/blob/master/src/templates/blog-index.js#L55 (and which might be the best and most natural use I've seen for the concept, tbh)
15:31:41 mordred the underlying framework sucks in all of the markdown files that exist and puts them into memory, and then you an graphql that data structure as needed to generate content rather than needing to nested for loop a bunch
15:32:22 cdent sounds rather reasonable
16:17:02 openstackgerrit Merged openstack/openstacksdk master: Fix requesting specific fields from ironic https://review.openstack.org/623059
16:23:01 openstackgerrit Merged openstack/openstacksdk master: Add propagate_uplink_status to port https://review.openstack.org/586687
16:35:00 openstackgerrit Merged openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193
16:48:22 openstackgerrit Merged openstack/openstacksdk master: block_storage.backup func tests to use configurable timeout https://review.openstack.org/622933
16:57:15 openstackgerrit Merged openstack/openstacksdk master: Register proxy directly in add_service https://review.openstack.org/620398
16:57:16 openstackgerrit Merged openstack/openstacksdk master: Fix query parameters of network.port_forwarding https://review.openstack.org/622963
17:20:42 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Supports router gateway IP QoS https://review.openstack.org/622130
18:04:00 openstackgerrit Merged openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993
18:21:23 openstackgerrit Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139
18:50:16 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: Adding suport for readthedocs.org https://review.openstack.org/624165
18:51:58 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
19:16:20 mriedem mordred: done re https://review.openstack.org/#/c/621695/
19:27:35 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
19:38:02 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
19:45:46 mordred mriedem: thank you
20:09:25 openstackgerrit Merged openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
20:11:07 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Don't try to upload to images in saving or queued state https://review.openstack.org/624188
22:31:42 openstackgerrit Merged openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
#openstack-sdks - 2018-12-11
02:22:14 openstackgerrit Gergely Csatari proposed openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
07:33:20 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
07:36:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
13:51:54 openstackgerrit Telles Mota Vidal Nóbrega proposed openstack/os-api-ref master: Fixing duplicate label issue. https://review.openstack.org/624400
14:47:13 openstackgerrit Merged openstack/os-api-ref master: readthedocs.io support https://review.openstack.org/623485
17:00:36 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Import code for building ironic-compatible configdrives https://review.openstack.org/624441
17:00:38 dtantsur mordred: dunno if you're going to hate this ^^^ :)
17:05:18 Shrews dtantsur: why would he hate it? seems very convenient to me
17:05:59 cdent elmiko, edleafe: we've just an EOY API-SIG report request in the mail, can I leave that in your capable hands?
17:06:01 dtantsur dunno, it's not about directly interacting with the API
17:11:32 edleafe cdent: Yeah, I can do that for the SIG
17:11:43 cdent kewl
17:24:19 elmiko edleafe: thanks =)
17:24:55 edleafe elmiko: I should have something in time for the office hour on Thursday for you to look over.
17:40:22 mordred dtantsur: yeah - I agree with Shrews - I think it's a good idea - kind of like the image signing code
17:40:30 dtantsur cool :)
17:43:52 elmiko edleafe: awesome, thanks again for taking the lead on that
20:23:10 Shrews mordred: i think the latest dogpile.cache release breaks us
20:23:25 Shrews http://logs.openstack.org/41/624441/1/check/nodepool-functional-py35-src/88da860/controller/logs/screen-nodepool-builder.txt.gz#_Dec_11_17_48_29_135083
20:23:45 Shrews dogpile.cache 0.7.0 release about 17 hours ago
20:27:53 openstackgerrit David Shrewsbury proposed openstack/openstacksdk master: Avoid dogpile.cache 0.7.0 https://review.openstack.org/624485
20:35:13 Shrews also, dtantsur|afk since that is affecting your configdrive change
20:48:18 Shrews mordred: we're doing a custom version of cache_on_arguments() so i wonder if the fault is ours?
20:48:39 Shrews mordred: educated guess at the commit that broke us: https://github.com/sqlalchemy/dogpile.cache/commit/7e8d9b0a2eb552adb6734c8570c63eb6ea2e3a7a
20:51:48 Shrews for the life of me, i cannot remember why we have this headache-inducing piece of code
21:02:38 Shrews ah ha: https://review.openstack.org/171456
21:03:40 Shrews might need to involve SpamapS
23:17:51 mordred Shrews: aWESOME
23:17:57 mordred kmalloc: ^^
#openstack-sdks - 2018-12-12
00:36:14 kmalloc Nice
00:37:50 kmalloc Heh
04:38:19 openstackgerrit Merged openstack/python-openstackclient master: Replace assertEqual(True/False, expr) with assertTrue/assertFalse https://review.openstack.org/617862
07:01:24 openstackgerrit Merged openstack/os-api-ref master: Fixing duplicate label issue. https://review.openstack.org/624400
13:07:30 openstackgerrit Merged openstack/python-openstackclient master: Supports router gateway IP QoS https://review.openstack.org/622130
13:42:06 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Avoid dogpile.cache 0.7.0 https://review.openstack.org/624485
13:42:18 mordred Shrews: ^^ added requirements patch and a depends-on
14:36:50 mordred kmalloc: whence you awaken, if you have any spare cycles today ...
14:37:18 mordred kmalloc: something is broken with ksa discovery caching (I think)
14:38:45 mordred kmalloc: you can see it by using openstacksdk like "import openstack ; openstack.enable_logging(http_debug=True) ; conn=openstack.connect(cloud='some cloud') ; print(conn.compute.servers()) ; print(conn.compute.servers())" and note that multiple discover http calls get made
14:57:57 kmalloc Will look as soon as I coffee
14:58:18 kmalloc I have cycles for this for sure
15:24:48 Shrews mordred: well, the thing is, i'm not convinced that we aren't doing something weird/unexpected that's causing the failure
15:29:03 mordred Shrews: oh - totally. I'm not sure dogpile itself is broken - but for now the combo of sdk and dogpile is - so we should go ahead and block it
15:29:09 mordred until we can figure it out
15:29:36 mordred Shrews, kmalloc: the ksa discovery cache issue is separate from the dogpile cache issue fwiw

Earlier   Later