Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-17
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
13:56:04 efried oh, okay. I definitely want to be told what a good name for the opt is.
13:56:23 mordred yeah - might just be meh - mostly thinking about how we can make it easy for other non-nova services to follow the lead
13:56:29 dtantsur cloud_name is probably fine
13:56:34 efried 'cloud' might be a bit too broad. But I don't know ...
13:56:49 mordred well - cloud is what is used everywhere else such a value is consumed
13:57:08 mordred so while I do wish we'd named it better ... probably more confusing for the oslo opt to be different
13:58:02 efried I can live with 'cloud'. I don't think we have anything else in nova that'll conflict, at least.
13:58:35 mordred oh - also - fwiw, if there is only one cloud entry defined in clouds.yaml, the won't need to put a cloud= setting in nova.ini
13:58:54 efried Yeah, but then we would have no way to know that we're looking for clouds.yaml instead of ksa opts.
13:59:36 mordred efried: this is a good point, actually
13:59:51 efried And I'm an opt conservative - fewer options rather than more wherever possible - so I'd like to avoid use_clouds_yaml = $bool; cloud = <str:optional>
14:00:03 mordred efried: and in fact, we ignore clouds.yaml files in the current patch - so we'll probably want someone to be explicit anyway so we can avoid avoiding the file
14:00:14 mordred ++
14:00:25 mordred I agree = no use_clouds_yaml = $bool
14:00:35 efried cool, so here's another angle:
14:00:45 efried eventually I think we probably want to deprecate and remove the ksa oslo.config stuff entirely.
14:01:07 efried And the theory behind clouds.yaml is more like define-once...
14:01:25 efried so eventually I would think we want to have the 'cloud' option defined in just one place in nova.conf, rather than in every section.
14:01:40 efried Because presumably one nova wouldn't want different services to talk to different clouds?
14:03:08 efried so like [DEFAULT]cloud=$str, or a section that somehow denotes [shit-the-sdk-uses]
14:03:48 mordred yes. I think this is spot on
14:04:07 efried Would rather not put it in [DEFAULT] I guess; suggestions for what to call that second thing?

Earlier   Later