*** markvoelker has joined #openstack-sdks | 00:42 | |
*** markvoelker has quit IRC | 00:44 | |
*** markvoelker has joined #openstack-sdks | 00:45 | |
*** ricolin has joined #openstack-sdks | 01:00 | |
*** whoami-rajat has joined #openstack-sdks | 01:19 | |
*** irclogbot_1 has quit IRC | 01:44 | |
*** edleafe has quit IRC | 01:48 | |
*** Dinesh_Bhor has joined #openstack-sdks | 01:57 | |
*** bobh has joined #openstack-sdks | 02:25 | |
*** bobh has quit IRC | 02:34 | |
*** bobh has joined #openstack-sdks | 02:34 | |
*** bobh has quit IRC | 02:39 | |
*** e0ne has joined #openstack-sdks | 05:31 | |
*** e0ne has quit IRC | 05:39 | |
*** Luzi has joined #openstack-sdks | 05:49 | |
*** e0ne has joined #openstack-sdks | 05:56 | |
*** e0ne has quit IRC | 06:04 | |
*** markvoelker has quit IRC | 06:43 | |
*** ITD27M01 has joined #openstack-sdks | 06:50 | |
*** slaweq has joined #openstack-sdks | 06:53 | |
*** slaweq has quit IRC | 06:56 | |
*** slaweq has joined #openstack-sdks | 07:01 | |
*** ITD27M01 has quit IRC | 07:03 | |
*** gtema has joined #openstack-sdks | 07:09 | |
*** tosky has joined #openstack-sdks | 07:19 | |
*** e0ne has joined #openstack-sdks | 07:24 | |
*** gtema has quit IRC | 07:33 | |
*** ralonsoh has joined #openstack-sdks | 07:35 | |
*** holser_ has joined #openstack-sdks | 07:41 | |
*** gtema has joined #openstack-sdks | 07:44 | |
*** jangutter has joined #openstack-sdks | 07:49 | |
*** yolanda_ has joined #openstack-sdks | 07:52 | |
*** e0ne has quit IRC | 07:52 | |
*** jpena|off is now known as jpena | 07:54 | |
*** jpich has joined #openstack-sdks | 07:55 | |
*** yolanda_ has quit IRC | 08:11 | |
*** yolanda_ has joined #openstack-sdks | 08:11 | |
*** dtantsur|afk is now known as dtantsur | 08:35 | |
*** e0ne has joined #openstack-sdks | 08:37 | |
*** markvoelker has joined #openstack-sdks | 08:45 | |
*** zigo_ has joined #openstack-sdks | 08:47 | |
*** zigo_ is now known as zigo | 08:50 | |
*** ITD27M01 has joined #openstack-sdks | 09:07 | |
ITD27M01 | mordred: Hi. If you have a time, can you please participate in discussion http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004695.html ? | 09:08 |
---|---|---|
*** yolanda_ has quit IRC | 09:09 | |
*** ttsiouts has joined #openstack-sdks | 09:11 | |
*** tobberydberg has quit IRC | 09:12 | |
*** tbarron_ has joined #openstack-sdks | 09:13 | |
*** markvoelker has quit IRC | 09:18 | |
*** e0ne has quit IRC | 09:27 | |
*** tobberydberg has joined #openstack-sdks | 09:27 | |
*** e0ne has joined #openstack-sdks | 09:34 | |
*** cdent has joined #openstack-sdks | 09:39 | |
*** holser_ is now known as holser|lunch | 09:40 | |
*** finucannot is now known as stephenfin | 09:40 | |
*** e0ne has quit IRC | 10:06 | |
gtema | ITD27M01: which API filters are you missing on the nova side? Let's say covering existing filters in SDK is not a problem | 10:10 |
*** yolanda_ has joined #openstack-sdks | 10:11 | |
*** e0ne has joined #openstack-sdks | 10:13 | |
*** markvoelker has joined #openstack-sdks | 10:16 | |
*** ttsiouts has quit IRC | 10:17 | |
*** ttsiouts has joined #openstack-sdks | 10:17 | |
*** ttsiouts_ has joined #openstack-sdks | 10:20 | |
*** ttsiouts has quit IRC | 10:22 | |
ITD27M01 | gtema: The basic idea is to transfer the filtering to nova API itself. In this way, the nova api will produce only requested data and we will get less work on the client side. As I said, the benefit will be the same for the server itself - fewer data will be transferred to the client. For large environments, this does lead to problems. Significant delays in processing large amounts of data. | 10:29 |
gtema | that's clear. As I read your message you want to extend nova filtering. What are you missing here. It currently support a lot (what we do not support so far, I'm preparing change for it now) | 10:31 |
*** ttsiouts_ has quit IRC | 10:31 | |
*** ttsiouts has joined #openstack-sdks | 10:32 | |
*** ttsiouts has quit IRC | 10:36 | |
*** yolanda_ has quit IRC | 10:38 | |
mordred | ITD27M01: responded to email | 10:46 |
*** yolanda_ has joined #openstack-sdks | 10:48 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Improve error message for version_kwargs mismatch https://review.openstack.org/650845 | 10:49 |
mordred | dtantsur: ^^ fixed your suggestion in a followup | 10:49 |
*** markvoelker has quit IRC | 10:49 | |
ITD27M01 | mordred: Thank you, I am in the process of understanding. | 10:52 |
*** holser|lunch is now known as holser_ | 10:54 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix get_server to work with use_direct_get https://review.openstack.org/540389 | 10:54 |
*** e0ne has quit IRC | 11:02 | |
gtema | mordred: I'm currently working on switching Image in OSC from glanceclient to SDK and found, that OSC has a use-case to only create "placeholder" for image without data. | 11:03 |
dtantsur | thanks mordred | 11:03 |
gtema | we currently do not "support" that in SDK. Any thoughts? Simply implement? | 11:03 |
mordred | gtema: you can't just create an Image resource? | 11:06 |
mordred | gtema: oh - you mean a remote image without uploading data to it yet? | 11:06 |
gtema | well, you can, but currently with a deprecated warning | 11:07 |
gtema | yes. We have create_image method, but it throws deprecated warn | 11:07 |
gtema | coming from cloud restruct | 11:07 |
mordred | yeah - I don't see any reason to not support doing that | 11:08 |
gtema | ok, will implement it then | 11:08 |
mordred | that said -upload image should be the deprecatred one and create_image should be the good one | 11:08 |
gtema | P.S. would be then very good to get it released soon not to block OSC switch | 11:09 |
*** e0ne has joined #openstack-sdks | 11:09 | |
mordred | ++ | 11:09 |
gtema | ack. In work :D | 11:10 |
*** ttsiouts has joined #openstack-sdks | 11:13 | |
ITD27M01 | gtema: If I understand it correctly the "metadata" of server currently cannot be used for API-side filtering. Only those items that described at the link I have provided. Is your question about this? | 11:13 |
gtema | I mean I understood your message, that you miss some filtering capabilities in nova. I was asking - which. If you only mean 'metadata' - ok | 11:14 |
*** markvoelker has joined #openstack-sdks | 11:15 | |
*** Luzi has quit IRC | 11:15 | |
ITD27M01 | gtema: Yes, metadata. | 11:16 |
gtema | ok, got it | 11:17 |
*** dtantsur is now known as dtantsur|brb | 11:20 | |
*** jpena is now known as jpena|lunch | 11:27 | |
*** ITD27M01 has quit IRC | 11:35 | |
openstackgerrit | LIU Yulong proposed openstack/openstacksdk master: Add floating IP port forwarding related methods https://review.openstack.org/650063 | 11:39 |
mordred | gtema: do we have support in Resource for listing two different remote keys that a field MIGHT be? | 11:43 |
gtema | not as I know | 11:43 |
mordred | ok. fiddle. | 11:44 |
* mordred is starting to work through server stuff, starting with flavors, which are "easy" | 11:44 | |
gtema | you mean sdk_name vs remote_name? | 11:44 |
mordred | I mean over time there are two possible remote names | 11:44 |
mordred | for instance, extra_specs vs. OS-FLV-WITH-EXT-SPECS:extra_specs | 11:45 |
gtema | btw, wait with server - I will push in a minute refresh with new attrs, complete query_params | 11:45 |
mordred | OS-FLV-WITH-EXT-SPECS:extra_specs being teh old name from an old nova extension - extra_specs being the current name | 11:45 |
mordred | gtema: cool. | 11:45 |
mordred | gtema: I'm going to start thinking about the caching layer soon | 11:45 |
*** e0ne has quit IRC | 11:46 | |
gtema | ah, so you mean basically multiple remote attr names for single sdk attr | 11:46 |
gtema | great | 11:46 |
mordred | yeah. obviousl will only work sanely for read - not for write | 11:46 |
gtema | sure | 11:46 |
*** e0ne has joined #openstack-sdks | 11:51 | |
*** e0ne has quit IRC | 11:53 | |
*** bobh has joined #openstack-sdks | 12:03 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Update compute.server resource https://review.openstack.org/650868 | 12:06 |
*** mgariepy has joined #openstack-sdks | 12:11 | |
*** bobh has quit IRC | 12:21 | |
*** jpena|lunch is now known as jpena | 12:22 | |
mordred | gtema: left a couple of comments - only the microversion one is important (and dtantsur|brb - check me that I'm right about that) | 12:41 |
gtema | mordred: ok, thanks | 12:42 |
*** ttsiouts has quit IRC | 12:44 | |
*** edleafe has joined #openstack-sdks | 12:44 | |
*** ttsiouts has joined #openstack-sdks | 12:45 | |
*** dtantsur|brb is now known as dtantsur | 12:47 | |
* dtantsur checks | 12:47 | |
*** Luzi has joined #openstack-sdks | 12:48 | |
dtantsur | mordred: you're right about that | 12:48 |
*** needssleep is now known as TheJulia | 12:49 | |
dtantsur | API SIG folks, mordred, cdent, elmiko, edleafe, would you be interested in a guideline on discovering openstack API endpoints with mdns? | 12:49 |
*** gtema has quit IRC | 12:49 | |
*** jroll has quit IRC | 12:50 | |
*** ttsiouts has quit IRC | 12:50 | |
cdent | dtantsur: are you using that? | 12:50 |
*** jroll has joined #openstack-sdks | 12:50 | |
edleafe | dtantsur: um, I don't have a lot of experience (read: any experience) with mdns. What would be the main advantage? | 12:51 |
dtantsur | cdent: I have an ask to add discovery of ironic API via mdns | 12:51 |
* cdent doesn't speak to people who use "ask" in that fashion | 12:51 | |
dtantsur | it could replace endpoint_override for standalone inter-service interaction, but my use case is a bit more specific: | 12:51 |
* dtantsur mutes cdent | 12:51 | |
cdent | \o/ | 12:52 |
dtantsur | we have a ramdisk in ironic that has to talk back to ironic and/or ironic-inspector on booting | 12:52 |
cdent | it does rather make sense in that context | 12:52 |
dtantsur | to do that, it needs to know ironic URL (keystone service catalog is out of question, we don't have credentials, and probably don't have access to keystone at all) | 12:52 |
cdent | but in others, now so sure | 12:52 |
cdent | s/now/not/ | 12:52 |
dtantsur | currently we pass the ironic URL via kernel command line when PXE booting. works, but is somewhat annoying. | 12:53 |
dtantsur | now people want to do things like pre-flashing ironic ramdisk to hard drives or using a flash drive with ironic or something | 12:53 |
dtantsur | no PXE here, no way to pass kernel params | 12:54 |
dtantsur | okay, this was the background. the question for this channel is whether it's something we're interested to provide guidance one | 12:54 |
cdent | I think I'd wait | 12:54 |
cdent | if you get two people wanting to do something, then's the time for guidance? | 12:55 |
dtantsur | right, okay | 12:55 |
edleafe | It does sound more like a "how to" than a guideline | 12:55 |
dtantsur | well, there is at least a question of providing a standard service fqdn | 12:55 |
dtantsur | e.g. ironic._http._tcp or openstack-baremetal._http._tcp or something more complex | 12:55 |
*** Shrews has quit IRC | 12:56 | |
dtantsur | (I'm still looking at subtypes to have something like baremetal._http._tcp._openstack) | 12:56 |
dtantsur | TheJulia: if you're curious ^^^ | 12:56 |
TheJulia | curious and interested... and have 3 hours of meetings ahead of me | 12:56 |
dtantsur | ouch :( | 12:56 |
edleafe | OK, the fqdn does sound like a good candidate for guidance, so that various services don't implement contrary designs | 12:57 |
dtantsur | right | 12:57 |
*** gtema has joined #openstack-sdks | 12:57 | |
cdent | I'm still curious if we can up with a situation where something other than Ironic would find this useful (I suspect we can, but I'm not thinking of it off the top of my head) | 12:58 |
*** Shrews has joined #openstack-sdks | 12:58 | |
cdent | That is: it's not something we'd want to encourage as "the other way of getting endpoints" because having two ways would be tireseome, presumably? | 12:59 |
mordred | oh my- many scrollbacks | 12:59 |
* cdent considers coughing on mordred | 12:59 | |
cdent | nope, I got nothing | 12:59 |
mordred | dtantsur: hrm. interesting | 13:00 |
mordred | dtantsur: so - we've got the well-known url entry (Which we really need to advertise more) for finding the keystone auth_url - I don't think that helps here though. .. HOWEVER | 13:01 |
mordred | if we built off of that so that the base URL/domain one would be doing a well-known url check on would similarly be the domain on which the SRV record would be looked up on - then that might not be super terrible | 13:02 |
mordred | I'd _definitely_ go with service-type keys rather than name though - baremetal._http._tcp vs ironic | 13:03 |
mordred | dtantsur: also - jamielennox started some work in keystoneauth AGES ago to be able to look things up optionally using dns | 13:04 |
*** bobh has joined #openstack-sdks | 13:04 | |
cdent | (yeah, I was kind of taking service-type (not name) for granted) | 13:04 |
mordred | https://review.openstack.org/#/c/223429/ | 13:05 |
dtantsur | mordred: but how do we know *any* URL? (esp. if we don't have keystone, but we can assume we do for the sake of the exercise) | 13:05 |
mordred | dtantsur: ^^ might be worth looking at - it's more about finding auth_url - and we wound up using well-known for that | 13:06 |
*** ttsiouts has joined #openstack-sdks | 13:06 | |
dtantsur | ah, so you lookup for _openstack_keystone | 13:06 |
mordred | dtantsur: yeah - totally - I could see using mDNS for finding a root domain for local things - but sort of just thinking through it | 13:06 |
dtantsur | btw why _openstack_keystone, not just _openstack? | 13:07 |
mordred | yeah - but I agree - there's a different use case for "I'm inside of a firewall looking for the local ironic" - and "I'm looking for the auth_url to a cloud that I know is over there" | 13:08 |
mordred | dtantsur: agree. I think since the well-known url location is /.well-known/openstack/api ... then finding the auth_url via SRV record, if we supported that, would want to jst look for _openstack | 13:09 |
mordred | but then, I think given your usecase, a SRV record for _baremetal._openstack would be potentially nice - but it still requires pre-knowing a domain | 13:09 |
mordred | which would be where an mDNS option would come in I'd guess | 13:10 |
* mordred is mostly just thinking out loud here | 13:11 | |
*** cmorpheus is now known as cmurphy | 13:11 | |
mordred | dtantsur: I think this _is_ a thing where we should consider a holistic design - because for sanity sake client-side support for it as an alternate endpoint finding method would want to go into keystoneauth | 13:12 |
mordred | given where in the stack doing these actions happens | 13:12 |
mordred | which means at the very least agreeing to and defining a protocol for it - even if ironic is the one one that implements it server-side for now | 13:13 |
dtantsur | our ramdisk doesn't use keystoneauth, otherwise I agree | 13:13 |
dtantsur | I will think more about it when I don't have to listen to a meeting :( | 13:14 |
* dtantsur dives into https://www.ietf.org/rfc/rfc6763.txt | 13:16 | |
mordred | dtantsur: fair enough re: ramdisk | 13:16 |
mordred | dtantsur: it sounds like a thing that would be a nice usability feature for ironic users anyway though - so I think definitely worth more discussion | 13:16 |
mordred | zomg | 13:27 |
* mordred really hates things that mock object internals rather than API boundaries | 13:27 | |
*** whoami-rajat has quit IRC | 13:28 | |
*** gtema has quit IRC | 13:29 | |
dtantsur | ++ | 13:29 |
* cdent shows mordred nova | 13:30 | |
* mordred shows cdent https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/tests/unit/test_proxy_base.py#L203-L216 | 13:31 | |
* cdent burns everything down | 13:32 | |
mordred | there is some really complicated testing that the mock system mocks the things that were mocked and some really tiny slices of the behavior work correctly when the rest of the stack is mocked out | 13:32 |
mordred | *basically* I think that's used to verify that self._list is called | 13:32 |
cdent | it's a sickness, a bit like heroin addiction | 13:32 |
mordred | yeah | 13:33 |
* mordred has been _very_ pleased with the shade layer unittests once we switched it all to mocking http interactions (and only http interactions) using requests_mock | 13:33 | |
cdent | \o/ | 13:34 |
mordred | sometimes setup takes a little more work - but you get to test "does this payload from the server result in the correct object behavior in the client" - which is, of course, the thing you actually care about | 13:34 |
mordred | "Does layer 1 call hidden method 2" is completely useless to know | 13:34 |
* mordred throws a cdent at the wet cats | 13:34 | |
cdent | that's the same spirit that drives gabbi, on the other side | 13:34 |
mordred | ++ | 13:35 |
dtantsur | this file is awful indeed | 13:35 |
mordred | dtantsur: I keep wanting to rip out all of the unittests using it and replace them with requests-mock based tests- but it took us about 8 months to do that with shade and it was MINDNUMBING so I haven't gotten angry enough to do it with these yet | 13:36 |
dtantsur | right | 13:36 |
dtantsur | we could start with replacing it with mocking Session. it's not perfect, but tests much more. | 13:37 |
dtantsur | as you know, I tend to write my unit tests like that | 13:37 |
mordred | yeah - althuogh we've already got a Session in each unit test that is set up with requests-mock - like, the plumbing is all there, so we don't really even have to mock session | 13:38 |
*** whoami-rajat has joined #openstack-sdks | 13:39 | |
mordred | with auth and catalogs already even! :) | 13:40 |
mordred | I think the real issue is that we need to dig in to "What is this test supposed to be testing" | 13:40 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add logger to Proxy object https://review.openstack.org/650901 | 13:54 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support microversion 2.61 for nova flavors https://review.openstack.org/650902 | 13:54 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Use resource layer for compute flavors https://review.openstack.org/650903 | 13:54 |
mordred | that stack is a bit of a mess still - but it's getting close | 13:54 |
*** Luzi has quit IRC | 14:32 | |
elmiko | dtantsur: hey, just saw the conversation | 14:47 |
elmiko | i would need to read more about mdns before giving an opinion, but is it something we want to promote across openstack? | 14:48 |
dtantsur | I don't think I want to promote it :) | 14:48 |
elmiko | lol | 14:48 |
elmiko | i guess that's my position re: new guidelines, is it something we think everyone /should/ be doing? | 14:48 |
dtantsur | I don't think everyone should. My thought was: if ppl other than ironic do it, maybe they should do it consistently. | 14:49 |
elmiko | ok, i can definitely get behind that | 15:07 |
elmiko | so perhaps this is more us setting a good example through the guidance to help generate consistency if needed? | 15:08 |
dtantsur | yeah | 15:13 |
*** e0ne has joined #openstack-sdks | 15:14 | |
elmiko | dtantsur: given all that, i think you've made a good case for writing something about it. no objection from me =) | 15:21 |
dtantsur | cool :) I'll try to be brief | 15:22 |
*** ttsiouts has quit IRC | 15:25 | |
*** ttsiouts has joined #openstack-sdks | 15:26 | |
*** ttsiouts has quit IRC | 15:30 | |
*** ttsiouts has joined #openstack-sdks | 15:31 | |
*** dave-mccowan has joined #openstack-sdks | 15:50 | |
*** e0ne has quit IRC | 16:02 | |
*** e0ne has joined #openstack-sdks | 16:05 | |
*** ttsiouts has quit IRC | 16:14 | |
*** e0ne has quit IRC | 16:17 | |
*** ITD27M01 has joined #openstack-sdks | 16:20 | |
*** jpich has quit IRC | 16:24 | |
*** dtantsur is now known as dtantsur|afk | 16:27 | |
*** jpena is now known as jpena|off | 16:45 | |
*** e0ne has joined #openstack-sdks | 16:46 | |
*** slaweq has quit IRC | 16:57 | |
*** cdent has quit IRC | 16:59 | |
*** ralonsoh has quit IRC | 17:13 | |
*** slaweq has joined #openstack-sdks | 17:14 | |
*** e0ne has quit IRC | 17:16 | |
*** slaweq has quit IRC | 17:19 | |
*** slaweq has joined #openstack-sdks | 17:24 | |
*** slaweq has quit IRC | 17:28 | |
*** ricolin has quit IRC | 18:00 | |
*** johnsom has quit IRC | 18:20 | |
*** Shrews has quit IRC | 18:20 | |
*** amito has quit IRC | 18:20 | |
*** Shrews has joined #openstack-sdks | 18:20 | |
*** amito has joined #openstack-sdks | 18:20 | |
*** fyx has quit IRC | 18:20 | |
*** fyx has joined #openstack-sdks | 18:21 | |
*** johnsom has joined #openstack-sdks | 18:21 | |
*** slaweq has joined #openstack-sdks | 18:25 | |
*** cdent has joined #openstack-sdks | 18:27 | |
*** slaweq has quit IRC | 18:30 | |
*** e0ne has joined #openstack-sdks | 18:33 | |
*** slaweq has joined #openstack-sdks | 18:46 | |
*** slaweq has quit IRC | 18:50 | |
*** cdent has quit IRC | 18:54 | |
*** e0ne has quit IRC | 18:55 | |
*** slaweq has joined #openstack-sdks | 19:07 | |
*** holser_ has quit IRC | 19:10 | |
*** dave-mccowan has quit IRC | 20:05 | |
*** dave-mccowan has joined #openstack-sdks | 20:06 | |
*** whoami-rajat has quit IRC | 20:08 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support microversion 2.61 for nova flavors https://review.openstack.org/650902 | 20:23 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.openstack.org/650903 | 20:23 |
mordred | Shrews: look! ^^ using the resource layer for flavors! | 20:24 |
* Shrews looks away | 20:25 | |
*** bobh has quit IRC | 21:07 | |
*** ttsiouts has joined #openstack-sdks | 21:18 | |
*** whoami-rajat has joined #openstack-sdks | 21:45 | |
*** slaweq has quit IRC | 22:07 | |
*** tobiash_ has quit IRC | 22:17 | |
*** tobiash has joined #openstack-sdks | 22:21 | |
*** slaweq has joined #openstack-sdks | 22:23 | |
*** slaweq has quit IRC | 22:27 | |
*** ttsiouts has quit IRC | 22:35 | |
*** ttsiouts has joined #openstack-sdks | 22:36 | |
*** ttsiouts has quit IRC | 22:40 | |
*** dave-mccowan has quit IRC | 22:58 | |
*** whoami-rajat has quit IRC | 23:58 | |
*** tosky has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!