Posted	Nick	Remark
#openstack-sdks - 2018-08-24
15:01:51	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: WIP: Functional tests for core functionality https://review.openstack.org/596359
15:03:01	cdent	Shrews: you have any insight on pooja_jadhav_ 's issue?
15:04:13	Shrews	pooja_jadhav_: cdent: my initial thought is to maybe compare individual fields (as test_create() does), or to mock the time function used
15:04:48	Shrews	(if i understand correctly)
15:04:58	Shrews	sorry, have to AFK for a bit now
15:05:35	samueldmq	mordred: cdent: Shrews: would be awesome to have your eyes on https://review.openstack.org/596359
15:05:58	samueldmq	I am sure it's not on shade patterns yet, but I wanted to make sure what I'm doing looks sane
15:06:18	samueldmq	it's for my masters research and I'd like to run that against several clouds really soon
15:06:46	cdent	samueldmq: your masters is apparently more interesting than mine was :)
15:09:07	samueldmq	cdent: what was yours about?
15:09:36	samueldmq	I'm investigating intra-platform interop in iaas, focusing on openstack
15:10:08	cdent	generically info science, but I guess if you had to label it with something it was sort of the intersection of HCI and engelbart's ideas on augmentation. So it wasn't actually boring at all, but it didn't involve assaulting clouds :)
15:13:26	samueldmq	heheh yes :) looks interesting
15:13:38	samueldmq	assaulting clouds seems interesting in the openstack context
15:42:40	pooja_jadhav_	shrews: Can u help me in the how to mock time function while creating any resource like (host or segment)
15:43:31	pooja_jadhav_	shrews: in the functional tests
15:43:59	pooja_jadhav_	shrews: so that i can assert whole object for remaining fields
15:46:21	pooja_jadhav_	shrews: But i think usually we dont use mock in functional tests
17:44:58	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: WIP: Functional tests for core functionality https://review.openstack.org/596359
18:51:13	openstackgerrit	Julia Kreger proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572
19:45:43	openstackgerrit	Julia Kreger proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572
20:07:03	TheJulia	dtantsur|afk: http://logs.openstack.org/25/596425/2/check/bifrost-integration-tinyipa-ubuntu-xenial/85f6c1b/job-output.txt.gz#_2018-08-24_19_52_46_756788 <-- tl;dr openstacksdk master branch state is broken for bifrost :\ But there is a pretty backtrace.
21:09:25	samueldmq	I love comments in openstackcloud
21:09:56	samueldmq	looking at image create code when: "# This makes me want to die inside"
21:10:03	samueldmq	and others :-)
21:16:20	TheJulia	heh
#openstack-sdks - 2018-08-25
06:09:37	mordred	dtantsur|afk: re: pulling microversions from headers - because that's literally not how any of the rest of the services do things, and I'd rather fix ironic if we can instead of adding YET ANOTHER completely different way of doing the exact same thing
06:10:56	mordred	dtantsur|afk: (yes, we can, and probably will add support to do that because it's our job to make this work - but it's ... frustrating to say the least)
06:11:53	mordred	dtantsur|afk: as for the 'remove /v1' - we can't - because the issue here is what to do if the versioned endpoint is passed in directly because of the noauth plugin
06:12:39	mordred	when someone passes in a direct endpoint via endpoint_override (which is the mechanism the endpoint/noauth approach is using) - we don't do *any* modifications to the endpoint passed in - it is assumed that the endpoint the user is passing in is the endpoint they want to use
06:12:53	mordred	and in fact, it is correct - it IS the enpdoint the user wants to use if they pass in the versioned endpoint
06:13:09	mordred	it's just that the endpoint doesn't have a version discover document
06:13:17	mordred	*sigh*
06:15:06	mordred	I guess what we can do is put logic into keystoneauth in the discovery flow that if it can't find the data in the document that it needs to look in the headers
06:16:16	mordred	perhaps if we do it in the same place that that patch above was coming from - we can just extract the data from the headers and normalize it in to the json in the place the rest of the logic is looking for it
06:16:51	mordred	of course -we're going to have to write a fun thing to pull the codename from the service name
06:22:01	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add support for ironic single-version responses https://review.openstack.org/595980
06:22:04	mordred	dtantsur|afk: something like that ^^
06:22:10	mordred	dtantsur|afk: (needs tests, obvs)
11:39:34	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Handle missing endpoint_data in maximum_supported_microversions https://review.openstack.org/596530
11:44:46	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Handle missing endpoint_data in maximum_supported_microversions https://review.openstack.org/596530
11:45:46	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242
11:47:23	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Run bifrost integration test jobs https://review.openstack.org/586564
11:47:41	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572
14:23:48	mordred	dtantsur|afk: I think I have some thoughts (for when it's not late and i'm not weirdly jetlagged) about how to generlize that keystoneauth patch
15:57:21	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Listing containers to return Munch objects https://review.openstack.org/596539
16:55:24	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Listing containers to return Munch objects https://review.openstack.org/596539
16:55:25	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Invalidate cache upon container deletion https://review.openstack.org/596545
16:55:26	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Return body on create and get container https://review.openstack.org/596546
17:02:58	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Return body on create and get container https://review.openstack.org/596546
18:35:24	openstackgerrit	Merged openstack/openstacksdk master: Alias NotFoundException to ResourceNotFound https://review.openstack.org/594158
#openstack-sdks - 2018-08-26
09:34:37	openstackgerrit	Merged openstack/os-client-config master: switch documentation job to new PTI https://review.openstack.org/594378
09:34:39	openstackgerrit	Merged openstack/os-client-config master: add python 3.6 unit test job https://review.openstack.org/594379
09:41:36	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Allow returning object on create and get container https://review.openstack.org/596546
16:22:03	openstackgerrit	Igor Gnatenko proposed openstack/openstacksdk master: compute: fix typo in update_security_groups() https://review.openstack.org/596609
16:26:34	openstackgerrit	Igor Gnatenko proposed openstack/openstacksdk master: compute: fix typo in update_security_groups() https://review.openstack.org/596609
16:30:45	openstackgerrit	Igor Gnatenko proposed openstack/openstacksdk master: compute: fix typo in update_security_groups() https://review.openstack.org/596609
16:49:31	ignatenkobrain	good evening, everyone
16:49:33	ignatenkobrain	would appreciate for review of https://review.openstack.org/#/c/596609/
16:49:42	ignatenkobrain	it's very trivial one-line fix ;)
#openstack-sdks - 2018-08-27
00:21:50	adriant	ignatenkobrain: are you using nova-networks?
00:40:48	openstackgerrit	Adrian Turjak proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544
07:38:00	dtantsur	mordred: hey, do we have some service that exposes the expected information on a versioned endpoint? I tried placement, compute and image so far..
07:42:13	dtantsur	mordred: oh, and we don't have anything in api-sig about it.. so, what is keystoneauth basing its discovery for versioned endpoints on? there seems to be absolutely no prior art for that.
07:42:26	dtantsur	edleafe, if you know ^^^
09:30:03	dtantsur	okay, I found the unfinished spec :( we really should pile on it and get it done
10:45:28	openstackgerrit	Olivier Bourdon proposed openstack/openstacksdk master: Add Python OpenSSL requirements https://review.openstack.org/596687
12:38:57	openstackgerrit	Federico Ressi proposed openstack/os-service-types master: Close descriptor after reading file. https://review.openstack.org/595491
12:38:58	openstackgerrit	Federico Ressi proposed openstack/os-service-types master: Add unittest for os_service_types.data module. https://review.openstack.org/596701
13:35:40	openstackgerrit	Chuck Short proposed openstack-infra/shade master: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/596725
14:06:45	openstackgerrit	Federico Ressi proposed openstack/os-service-types master: Add unittest for os_service_types.data module. https://review.openstack.org/596701
14:20:27	samueldmq	howdy! does anybody else think returning munch objects for create/get of container and object is sane?
14:20:56	samueldmq	I've done it for containers and would like some feedback before I go ahead with object: https://review.openstack.org/#/c/596546/
14:21:22	samueldmq	it's currently returning metadata as a dict, but as an end user I care more about the object than the metadata headers
14:46:33	openstackgerrit	Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Listing objects to return Munch objects https://review.openstack.org/596780
14:48:37	dtantsur	samueldmq: get_container not returning a container looks weird indeed..
14:54:51	samueldmq	dtantsur: yes, hard part is that it returns only metadata today, so I had to add a metadata=True parameter, setting to false returns the container instead of metadata
14:55:17	dtantsur	samueldmq: do you plan to deprecate it eventually? I guess it can be a source of confusion
14:56:31	samueldmq	dtantsur: I don't know what to do, but I agree with you... what we tried to do initially was to follow what the API returns
14:57:07	samueldmq	because yes the object store apis let us CRUD the metadata.. however as an end user I don't care about hte metadata
14:57:31	samueldmq	I just want to CRUD my objects
14:58:15	dtantsur	exactly
14:58:37	dtantsur	I guess we could deprecate metadata=True to allow people to move away from it..
14:59:40	samueldmq	++
14:59:53	samueldmq	we could have separate methods for handling the object metadata
15:00:49	samueldmq	let's try to get some input from others on this too. cc cdent mordred Shrews
15:01:34	samueldmq	dtantsur: I'll go ahead and do the same for create_object and get_object
15:01:46	dtantsur	k
15:04:14	cdent	samueldmq: sorry, can you catch me about this later, add me to a review or something, unfortunately I'm on the way out the door right now
15:05:22	samueldmq	cdent: sure! talk to you later
23:21:35	samueldmq	kk I got something interesting when validating SDK against a given cloud
23:22:09	samueldmq	radosgw's swift api endpoint, which implements swift api in a slightly different way
23:22:32	samueldmq	such as returning 200 OK when you do GET /account/container/OBJ_THAT_DOES_NOT_EXIST
23:22:57	samueldmq	what do we do about that? I suspect we don't care since it's not pure openstack-swift APIs?
23:23:03	samueldmq	mordred: ^ this is interesting, see.
23:35:54	openstackgerrit	Merged openstack/keystoneauth master: Add Keystone2Keystone example https://review.openstack.org/594156
#openstack-sdks - 2018-08-28
06:27:13	openstackgerrit	Federico Ressi proposed openstack/os-service-types master: Close descriptor after reading file. https://review.openstack.org/595491
06:27:13	openstackgerrit	Federico Ressi proposed openstack/os-service-types master: Add unittest for os_service_types.data module. https://review.openstack.org/596701
13:03:30	dtantsur	hi folks, I could use some reviews on https://review.openstack.org/#/c/596530/ please. It fixes bifrost with openstacksdk master.
14:14:21	openstackgerrit	Merged openstack/os-service-types master: Close descriptor after reading file. https://review.openstack.org/595491

Earlier   Later