Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-07-30
13:20:58 mordred dtroyer: http://logs.openstack.org/62/586262/6/check/osc-functional-devstack-tips/3dc18eb/testr_results.html.gz
13:21:27 mordred dtroyer: I'm not sure the right way to fix either - the first seems to be that we're getting an os_hidden field back from glanceclient that we're not consuming
13:22:04 mordred the second is just that v1 has gone away - is there an appropriate 'only run this test if the cloud has XXX' cantrip?
13:55:48 samueldmq g'morning
13:55:58 samueldmq does -sdk have meetings in the PTG?
14:03:29 mordred samueldmq: we don't have any scheduled ahead of time - but we can use ptgbot to schedule some when we're there
14:03:57 samueldmq mordred: gotcha, thanks!
14:06:11 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Add a simple baremetal functional job https://review.openstack.org/586574
14:54:28 mordred corvus, Shrews: if'n you get a few moments today: https://review.openstack.org/#/q/topic:swift-streaming
15:40:42 dtroyer mordred: I think we have to invent that catnip, it'll be a cousin to the nova-net/neutron selector
15:44:20 mordred dtroyer: nod
15:46:35 dtantsur mordred: sup! what's the difference between NotFoundException and ResourceNotFound? :)
15:48:56 dtantsur I'm asking because baremetal.get_node documents raising ResourceNotFound, but raises NotFoundException (its parent)
15:56:25 mordred dtantsur: I can come up with no reason for that - probably mistake from the sdk/shade merge
15:56:46 mordred dtantsur: we should probalby make ResourceNotFound an alias to NotFoundException instead of a subclass
15:57:37 dtantsur maybe, yeah. because the docstrings for at least baremetal are wrong
15:57:42 mordred dtantsur: ResourceNotFound seems to be the most used
15:58:50 mordred so maybe we should replace use of NotFoundException in the codebase with ResourceNotFound but keep NotFoundException as an alias?
15:59:59 dtantsur yep. wanna me do it?
16:10:24 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Add a simple baremetal functional job https://review.openstack.org/586574
16:49:43 mordred dtantsur: sure!
16:50:02 mordred dtantsur: I'm also excited about that baremetal functional job ...
16:52:29 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Defer object interactions to openstacksdk https://review.openstack.org/586728
16:55:45 dtantsur mordred: yeah, I hope it finally passes :)
16:55:58 mordred :)
16:58:00 dtantsur mordred: do we anyhow track the "dirty" flag for nested JSON attributes of resources?
16:59:22 dtantsur given that I don't see mentions of jsonpatch in the baremetal stuff, I suspect it's completely broken
17:05:51 mordred I suspect the same thing
17:11:51 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for static routes https://review.openstack.org/587163
17:12:08 mordred slaweq: ^^ cherry-pick of the shade patch
17:16:15 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Adds toggle port security on network create https://review.openstack.org/587166
17:16:16 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: python-shade expose MTU setting. https://review.openstack.org/587167
17:16:23 mordred slaweq: and two more
18:06:36 openstackgerrit Merged openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
18:17:46 openstackgerrit Merged openstack-infra/shade master: python-shade expose MTU setting. https://review.openstack.org/578861
18:24:31 slaweq mordred: thx, looking
18:24:55 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Add a simple baremetal functional job https://review.openstack.org/586574
19:26:13 dtroyer mordred: so the brokkeness in volume is due to the compat bits being removed from the python API, not something we currently test for.
19:26:28 dtroyer what pisses me off is they removed it the day before cutting the stable release
19:26:32 dtroyer no warning
19:34:43 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix broken functional tests https://review.openstack.org/587203
19:35:00 dtroyer mordred: lets see if ^^^ does it
19:35:28 mordred dtroyer: sigh (re volume)
19:36:09 mordred dtroyer: maybe in stein we can make progress in removing that vector of failure
19:36:37 dtroyer cinder just jumped uo to my #2 for getting off the python lib, after glance
19:36:55 mordred dtroyer: nod. I will order my efforts accordingly
19:43:16 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Adds toggle port security on network create https://review.openstack.org/587166
19:43:16 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: python-shade expose MTU setting. https://review.openstack.org/587167
19:43:36 mordred slaweq: ^^ sorry - had to fix exception names for pep8/py35
19:49:51 slaweq mordred: thx, sorry that I missed them :)
21:21:27 openstackgerrit Merged openstack/openstacksdk master: Support passing profile to get_one https://review.openstack.org/585780
21:35:37 mordred dtantsur|afk: zomg. the ironic test passed!
23:02:13 openstackgerrit Merged openstack/openstacksdk master: Add support for static routes https://review.openstack.org/587163
23:27:58 openstackgerrit Merged openstack/openstacksdk master: Add method for returning a raw response for an object https://review.openstack.org/586262
23:33:27 openstackgerrit Merged openstack/openstacksdk master: Add support for streaming object responses https://review.openstack.org/586250
23:33:28 openstackgerrit Merged openstack/openstacksdk master: Remove the auto-creation of containers in create_object https://review.openstack.org/586616
#openstack-sdks - 2018-07-31
01:15:57 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: Remove volume replication v1 support. https://review.openstack.org/587008
01:38:20 dtroyer this ^^^ and https://review.openstack.org/#/c/587005/ will likely need to be merged to pass tests
02:05:13 openstackgerrit yanpuqing proposed openstack/openstacksdk master: Add an error message when router add subnet https://review.openstack.org/586996
02:24:39 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005
02:56:02 openstackgerrit Merged openstack/openstacksdk master: Add some backoff to find_best_address https://review.openstack.org/575580
06:03:31 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005
06:08:33 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005
07:48:17 openstackgerrit Fan Zhang proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005
11:02:30 openstackgerrit Merged openstack/openstacksdk master: Adds toggle port security on network create https://review.openstack.org/587166
11:06:10 openstackgerrit Merged openstack/openstacksdk master: python-shade expose MTU setting. https://review.openstack.org/587167
13:12:01 mordred dtroyer: so we should squash 587008 and 587005 ?
13:15:24 mordred fanzhang: looks like you're on top of it - let me know if you'd like me to help in any way
13:32:19 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005
13:32:31 mordred fanzhang: ^^ pushed up an update
13:37:45 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass microversion info through from Profile https://review.openstack.org/587435
14:11:39 mordred cdent: youknow all the things, right?
14:11:53 cdent mordred: I know everything except what you want to know.
14:11:59 mordred sweet. perfect
14:12:25 mordred cdent: do you happen to know what response bodies from nova errors look like? they don't seem to be documented in the api docs
14:12:48 mordred cdent: I have just learned that neutron sends a json dict {'NeutronError': 'some string'}
14:12:51 mordred because of course
14:13:11 cdent the nova api is murky to me, but I should be able to find something clearly relatively quickly
14:14:02 mordred cool - thanks! I *think* it might be a list of dicts with a message key - since we've got sdk code to handle that case
14:16:40 cdent mordred: looks like a dict keys on the fault name: https://github.com/openstack/nova/blob/03ccb1f9333bcfbdb2caf8a8c6e98622a63088cc/nova/api/openstack/wsgi.py#L1080-L1083
14:17:25 mordred cdent: ah - cool. that would explain the code in sdk currently
14:17:30 mordred I will add a comment
14:22:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for neutron error messages https://review.openstack.org/587483
14:34:44 mordred cdent: ^^ just in case you are curious.
14:42:14 cdent I'm sure we made this http://specs.openstack.org/openstack/api-wg/guidelines/errors.html for some reason but I can't remember why
15:01:37 openstackgerrit Merged openstack/openstacksdk master: Support for microversions in baremetal resources https://review.openstack.org/583532
15:01:39 openstackgerrit Merged openstack/openstacksdk master: Add a simple baremetal functional job https://review.openstack.org/586574
20:13:11 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005
20:19:13 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005
20:23:32 dtroyer mordred: ^^^ should get us really close… it needs a way to detect Image API v1 yet
20:24:05 mordred dtroyer: like - needs to be able to detect whether the cloud has v1?
20:24:41 dtroyer yes
20:24:55 mordred https://review.openstack.org/#/c/585029/ <--
20:25:13 mordred dtroyer: that is what you need - unfortunately I did not manage to get it in to keystoneauth before the feature freeze
20:25:29 dtroyer I've hard-coded it 'not there' in that rev just to get this through… ah, nice
20:25:52 mordred dtroyer: yah - if you're ok with the test suite being a little wonk until we can get another ksa release cut - there's a path to it
20:26:09 dtroyer I'll think about this harder after the 30,000 foot fatigue is less
20:26:28 dtroyer I'm fine, this is better than ripping it all out
20:26:33 mordred dtroyer: alternately, the code in the versions show command that does it client-side would also work - it'll just be more expensive than would normally be pleasant
20:26:44 mordred cool

Earlier   Later