Earlier  
Posted Nick Remark
#openstack-sdks - 2018-08-23
19:37:01 openstackgerrit Mathieu Gagné proposed openstack/openstacksdk master: Add the ability to extend a volume size https://review.openstack.org/595873
#openstack-sdks - 2018-08-24
01:24:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242
02:06:20 mordred dtantsur|afk: ok. SO - I dug all the way down the the rabbit hole in to keystoneauth and I don't think we can support versioned endpoints for ironic with the noauth / standalone without some updates to the version response from ironic
02:06:25 mordred we can update to fail more better
02:06:34 mordred but the version response has no microversion data
02:06:40 mordred so we have to assume v1
02:07:01 mordred that said - I think it wouldn't be awful to add the microversion info to the versioned response
02:07:18 mordred and we can update keystoneauth slightly to make it upwards compatible when ironic grows doing that
02:16:44 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add support for ironic single-version responses https://review.openstack.org/595980
02:16:51 mordred dtantsur|afk, TheJulia: ^^
02:20:48 mordred dtantsur|afk: for now, in sdk, I think just putting in the try/except around the discovery failure seems to 'work' for endpoint things ... although given the goal is supporting newer microversions I'm not sure it fully achieves all of the goals
02:22:06 mordred dtantsur|afk: we COULD try to add a warning if someone tries to use ironic through sdk with an endpoint_override that ends in /v1 - saying that microversions canot be detected
02:22:08 mordred hrm
02:23:02 mordred the discovery path might do the wrong thing is given an unversioned endpoint in endpoint_override
14:26:38 mnaser mordred: does shade have a way of listing stack resources, i don't see one right off the bat..
14:29:13 Shrews mnaser: i don't see one either
14:29:56 mnaser Shrews: is there a way to get a client via shade or not anymore? would need to go to openstacksdk for that?
14:31:23 Shrews mnaser: there should be a orchestration_client property
14:32:18 mnaser Shrews: that's wonderful, thank you
14:54:53 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: WIP: Functional tests for core functionality https://review.openstack.org/596359
14:57:00 pooja_jadhav_ hello all, I am wrirring functional_tests in openstacksdk for masakari. In that, for test_host module. for get_host_test, trying assert host objects. (expected, actual). But its failing because of timestamp in created_at field.
14:58:48 pooja_jadhav_ while creating host its giving correct datetime with timestamp. but while saving host record in DB, its saves only date nd time (not timestamp). so while getting that host, created_at field will append timestamp as 0000. its mismatching reference and actual object.
14:58:58 pooja_jadhav_ please anyone can help me in this?
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

Earlier   Later