| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-05-16 | |||
| 16:13:15 | edleafe | dtantsur: Weren't you just saying you wanted something to do with all your free time? | |
| 16:13:38 | dtantsur | I'm enjoying fixing ironic CI 24/7 :( | |
| 16:14:27 | openstackgerrit | Colleen Murphy proposed openstack/api-sig master: Update liaison for keystone https://review.opendev.org/659613 | |
| 16:15:57 | elmiko | ouch | |
| 16:19:28 | dtantsur | speaking of which, I'm exhausted, so I'll call it a day a bit earlier.. see you | |
| 16:19:43 | elmiko | rest easy dmitry o/ | |
| 16:20:19 | dtantsur|afk | I have to move some furniture today, otherwise I will :D | |
| 16:22:57 | elmiko | yikes! | |
| 16:25:40 | edleafe | I would update, but I really don't know the answer to the formatting question on https://review.opendev.org/#/c/459405/19/guidelines/consuming-catalog.rst@293 | |
| 16:26:01 | edleafe | too many levels :) | |
| 16:27:46 | elmiko | hmm, that's a good question and i'm not sure the correct answer | |
| 16:31:18 | edleafe | We can wait for when dtantsur|afk has copious free time. | |
| 16:31:40 | elmiko | i wonder if mordred might comment there too, he certainly understands the intent better than most | |
| 16:32:03 | edleafe | I was going to suggest that, but was afraid to summon the spirit of mordred | |
| 16:32:14 | elmiko | i mean, by the reading of the rule it seems like morgan's assumption is correct re the v3 v3.6 example | |
| 16:32:17 | elmiko | haha, fair | |
| 16:33:07 | edleafe | Yeah, but I don't trust my reading of that rule. Maybe that's a sign that a bit more explanation is needed | |
| 16:33:28 | elmiko | i share your concerns, i felt very similar | |
| 16:34:14 | elmiko | i guess i will add that to the comments, at the least it helps expose an area where more clarity is needed | |
| 16:58:53 | mordred | elmiko: what did I do? | |
| 17:01:13 | elmiko | mordred: oh nothing in specific, morgan had brought up a question on the version discovery pr and we thought you might be able to shed some light | |
| 17:01:21 | mordred | oh jeez. what a fun edgecase | |
| 17:01:27 | elmiko | exactly | |
| 17:01:35 | mordred | my first response is "don't do that" | |
| 17:01:42 | elmiko | i just don't know the problem space well enough to provide a solid answer | |
| 17:01:45 | elmiko | lol | |
| 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 | |