Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-16
17:02:08 mordred I don't know that I've ever encountered a vX.Y url with a service that doesn't have a working discovery document
17:02:36 elmiko any guidance, or opinion, you could share on the pr would help =)
17:02:53 mordred so while we do account for crazy things - most of them are tied to crazy things people have actually done - not crazy things people might have done
17:02:58 mordred I will leave a comment
17:03:15 elmiko yeah, and the speculation business is a dangerous one
17:03:17 elmiko thanks
17:03:20 mordred ++
17:11:40 dustinc dtantsur|afk or anyone else able to answer a question about node.patch for baremetal...the ironicclient equivalent would raise an exception if a node is locked when you try to patch it, but the docstring for it does not how any exceptions. I dug in a bit and did not see any possible exception sources (other than parent class, but only if patch is not enabled, so not relevant for node) can someone confirm?
17:11:52 dustinc implemented here: https://review.opendev.org/#/c/647730/
17:12:13 dustinc **the docstring for sdk**
19:09:56 openstackgerrit Merged openstack/python-openstackclient master: Update sphinx requirement. https://review.opendev.org/659279
#openstack-sdks - 2019-05-17
00:20:52 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove code migrated to osc-lib long ago https://review.opendev.org/659693
04:35:35 openstackgerrit Merged openstack/python-openstackclient master: Aggregate functional test tweak https://review.opendev.org/659552
07:32:51 dtantsur dustinc: I'm not sure I entirely get the question.. you'll get HTTP CONFLICT if the node is locked, both ironicclient and openstacksdk will retry it for a bit
07:48:58 openstackgerrit Dmitry Tantsur proposed openstack/api-sig master: Update Sphinx requirements and actually use Python 3 for docs https://review.opendev.org/659739
07:49:07 dtantsur elmiko, edleafe, pretty urgent ^^^
07:49:19 dtantsur gtema: ^^
07:53:21 gtema dtantsur: great, was wanting to do that as well. Thanks
10:39:51 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
10:41:42 tssurya dtroyer: addressed your comments ^ hope it looks ok now
11:19:14 dtroyer tssurya: thank you… it looks like you still have a problem with requirements though… novaclient 14.0 is not the common minimum yet. you may have to check that version too unless it gets raised globally
11:20:14 tssurya dtroyer: so its not enough I bump it here: https://review.opendev.org/#/c/659124/4/lower-constraints.txt ?
11:20:23 tssurya sorry for the stupid questions (haven't done this before)
11:21:48 tssurya apart from that I also relealised some problems with the logic (https://review.opendev.org/#/c/659124/4/openstackclient/compute/v2/server.py@1217) I'll work on that before pinging you
11:24:10 dtroyer tssurya: We can allow an older version than the global minimum, we can not go the other way. I would prefer to work with as wide a range of dependencies as possible
11:25:12 tssurya dtroyer: ah ok can you point me to the global min requirements ?
11:25:27 tssurya not that I will change that, I'll do the check like you stated then
11:26:06 dtroyer all of the requirements are in https://opendev.org/openstack/requirements
11:27:02 tssurya thanks
11:33:08 dtantsur dtroyer, tssurya, IIRC minimum requirements are no longer tracked in the central repository
11:33:33 dtantsur each project can bump requirements.txt + lower-constraints.txt, as long as it does not contradict global requirements
11:33:33 tssurya dtantsur, dtroyer: yea I was also looking as to where the min requirements are
11:33:53 tssurya I only see max possible ones: https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L221
11:34:00 dtantsur right
11:34:29 tssurya cool thanks :)
12:29:56 mordred elmiko: responded on https://review.opendev.org/#/c/459405 - I was misreading that section yesterday, I have a better answer today
12:30:43 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
12:31:13 edleafe dtantsur: lloks good. Fast-approved
12:31:17 edleafe lloks*
12:31:25 edleafe heh
12:31:28 edleafe looks
12:37:32 openstackgerrit Merged openstack/api-sig master: Update Sphinx requirements and actually use Python 3 for docs https://review.opendev.org/659739
12:41:09 dtantsur thx edleafe!
12:51:09 openstackgerrit Merged openstack/api-sig master: Add guideline on DNS-based service discovery https://review.opendev.org/651222
12:53:01 dtantsur \o/
13:07:52 brtknr mordred: I found a bug in the os_stack module I submitted months ago, which appears to fail when using new Ansible with openstacksdk older than 0.28.0, which I've fixed in this PR: https://github.com/ansible/ansible/pull/56575
13:10:49 elmiko mordred: ack, thanks!
13:16:17 stingrayza Hi folks, would anybody mind casting an eye on https://review.opendev.org/659366/ please. (It's required for https://github.com/ansible/ansible/pull/56453)
13:17:49 dtantsur stingrayza: could you add a release note?
13:17:58 brtknr mordred: thanks :)
13:18:39 stingrayza dtantsur: sure thing. will take a quick look how ;)
13:19:13 dtantsur stingrayza: https://docs.openstack.org/reno/latest/user/usage.html
13:21:06 brtknr any other Ansible reviewers please able to look at/apply shipit to: https://review.opendev.org/659366/
13:21:07 stingrayza dtantsur: gotcha, thanks!
13:21:30 brtknr any other Ansible reviewers please able to look at/apply shipit to: https://github.com/ansible/ansible/pull/56575
13:30:59 openstackgerrit Raimund Hook (stingrayza) proposed openstack/openstacksdk master: Adding dns_domain parameter into create_network https://review.opendev.org/659366
13:36:18 stingrayza dtantsur: thanks! :D
14:03:10 dtantsur np
15:06:00 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
16:02:30 openstackgerrit Merged openstack/openstacksdk master: Cap sphinx for py2 to match global requirements https://review.opendev.org/659442
17:08:57 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP add influxdb stats reporting. https://review.opendev.org/659841
17:27:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP rework statistics reporting https://review.opendev.org/659841
17:28:24 openstackgerrit Merged openstack/openstacksdk master: Add image.stage methods https://review.opendev.org/652981
19:11:32 efried mordred: I forgot to hit you up yesterday, now a good time to brainstorm sdk config bootstrap in nova?
20:58:36 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.opendev.org/568549
21:33:34 openstackgerrit Merged openstack/python-openstackclient master: Fix bug in endpoint group deletion https://review.opendev.org/655889
21:44:20 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Serialize more aggregate functional tests https://review.opendev.org/659883
23:07:58 openstackgerrit Merged openstack/python-openstackclient master: Fix RuntimeError when showing project which has extra properties https://review.opendev.org/530207
#openstack-sdks - 2019-05-18
00:15:36 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Serialize more aggregate functional tests https://review.opendev.org/659883
02:42:35 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373
03:51:53 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373
12:57:09 openstackgerrit Merged openstack/openstacksdk master: Adding dns_domain parameter into create_network https://review.opendev.org/659366
18:06:04 openstackgerrit Merged openstack/python-openstackclient master: Serialize more aggregate functional tests https://review.opendev.org/659883
18:47:44 openstackgerrit Merged openstack/python-openstackclient master: Remove code migrated to osc-lib long ago https://review.opendev.org/659693
#openstack-sdks - 2019-05-19
17:03:08 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated compute commands https://review.opendev.org/612781
17:16:17 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated compute commands https://review.opendev.org/612781
17:57:55 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated compute commands https://review.opendev.org/612781
17:57:55 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated identity commands and args https://review.opendev.org/612798
19:44:22 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Batch up minor cleanups for release https://review.opendev.org/659982
21:30:12 olaph B556
22:07:56 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated image commands https://review.opendev.org/659431
23:22:38 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated network options https://review.opendev.org/613644
#openstack-sdks - 2019-05-20
07:25:25 openstackgerrit yanpuqing proposed openstack/python-openstackclient master: Client should parse string to boolean for valuse 'is_domain' https://review.opendev.org/660038
08:17:37 openstackgerrit yanpuqing proposed openstack/python-openstackclient master: Client should parse string to boolean for valuse 'is_domain' https://review.opendev.org/660038
10:10:27 openstackgerrit yanpuqing proposed openstack/python-openstackclient master: Client should parse string to boolean for valuse 'is_domain' https://review.opendev.org/660038
10:28:22 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP rework statistics reporting https://review.opendev.org/659841
12:05:56 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP rework statistics reporting https://review.opendev.org/659841
13:41:51 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Batch up minor cleanups for release https://review.opendev.org/659982
13:48:07 efried mordred: ...or today?
13:51:42 mordred efried: yes!
13:51:50 mordred efried: today is a great day to discuss such things
13:52:00 efried (gtema, dustinc, dtantsur)
13:52:39 efried mordred: So today we have oslo.config for e.g. [placement]=>ks_loading.get_{auth|session|adapter}_config_opts
13:53:00 efried I'm thinking we could inject [placement]cloud_region=<str>
13:53:12 efried (is it called a "cloud region", this section in clouds.yaml?)
13:53:50 efried The code looking at that config section will use the presence of 'cloud_region' to decide whether to use the ksa opts or look at clouds.yaml.
13:54:28 efried that's... pretty much it.
13:55:13 mordred yes - I think that's about right. and I think maybe what we want is a function similar to get_{}_config_opts in sdk that ads the parameter
13:55:43 efried yeah, that would work. Except it's only one opt, so meh?
13:55:46 mordred (the section and parameter in sdk is just called "cloud" - a CloudRegion object is a cloud + a region_name

Earlier   Later