Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-03-24
21:19:36 openstackgerrit Ian Wienand proposed openstack/js-openstack-lib master: Replace openstack.org git:// URLs with https:// https://review.openstack.org/646426
22:23:51 openstackgerrit Ian Wienand proposed openstack-infra/shade master: Replace openstack.org git:// URLs with https:// https://review.openstack.org/647135
#openstack-sdks - 2019-03-25
00:44:21 openstackgerrit Merged openstack-infra/shade master: Replace openstack.org git:// URLs with https:// https://review.openstack.org/647135
10:58:09 openstackgerrit sahid proposed openstack/os-service-types master: add python 3.7 unit test job https://review.openstack.org/647397
11:02:00 openstackgerrit sahid proposed openstack/openstacksdk master: add python 3.7 unit test job https://review.openstack.org/647401
12:59:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: add python 3.7 unit test job https://review.openstack.org/647401
13:00:06 mordred dtantsur: ^^ I fixed the broken test. https://stackoverflow.com/questions/51700960/runtimeerror-generator-raised-stopiteration-every-time-i-try-to-run-app is a fun one
13:13:28 dtantsur oh, this is a good one indeed
13:14:54 dtantsur mordred: an idea inline
13:19:20 mordred dtantsur: oh - does that work? neat
13:19:35 dtantsur I think I remember somebody using it in one of our projects
13:20:36 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: add python 3.7 unit test job https://review.openstack.org/647401
13:20:45 mordred dtantsur: yup! works great
13:45:56 efried There's also
13:45:56 efried exc = self.assertRaises(Exception, ...)
13:45:56 efried self.assertIsInstance(exc, ...)
13:46:01 efried but what you've done is way nicer.
13:48:25 mordred \o/
16:34:00 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
16:34:15 mordred fungi: ^^ updated that with tests
16:37:24 fungi thanks mordred!!!
16:58:19 mordred Shrews: if you have a sec and feel like assaulting your brain by looking at code I wrote ^^ is in support of adding a storyboard feature for attaching files/images to stories and storing them in swift on the backend
17:14:58 Shrews mordred: ack. lemme get to a break point on this nodepool issue first
17:20:23 mordred Shrews: no worries / rush
18:28:02 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
19:27:07 Shrews mordred: other than a doc nit,lgtm
19:33:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
19:33:23 mordred Shrews: sweet - thanks - fixed the nit and added a release note ^^
19:34:46 Shrews mordred: oh yah, good idea
#openstack-sdks - 2019-03-26
10:03:37 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Split OpenStackCloud into reasonable pieces https://review.openstack.org/642218
11:52:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Split OpenStackCloud into reasonable pieces https://review.openstack.org/642218
12:46:49 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Make PATCH a first class operation and support it for baremetal https://review.openstack.org/647730
12:46:51 dtantsur mordred: early comments welcome ^^
13:50:59 mordred dtantsur: ooh fun. will look in a bit
13:51:12 mordred dtantsur: (seems like a good thing to have as a first-class operation)
13:51:25 gtema ++
15:20:29 openstackgerrit Merged openstack/openstacksdk master: add python 3.7 unit test job https://review.openstack.org/647401
15:27:53 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Split OpenStackCloud into reasonable pieces https://review.openstack.org/642218
15:56:51 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Make PATCH a first class operation and support it for baremetal https://review.openstack.org/647730
15:56:54 dtantsur this is it, assuming the ironic CI passes ^^
17:34:27 dtruong mordred I'm using openstacksdk 0.26.0 and am seeing connections that are not being closed when calling network APIs.
17:34:35 dtruong If you run this file: http://paste.openstack.org/show/748415/
17:35:02 dtruong and watch the number of open connections of that process, it just keeps increasing and using more memory
17:36:04 dtruong I run that file as python blah.py
17:36:22 dtruong and watch the connections like this: watch -n 1 "pgrep -o -f blah | xargs sudo lsof -i 4 -a -p "
17:48:08 dtruong mordred It looks like this started with openstacksdk 0.18.0
17:48:34 dtruong With openstacksdk 0.17.2 it only has 2 or 3 connections open when running my sample program
17:58:43 dtantsur regressions \o/
18:00:40 dtruong actually, looks like using openstacksdk master doesn't have this problem
18:01:06 dtruong so I guess you something changed between 0.26.0 and currently master that fixed it?
18:06:11 dtruong Looking at the commit history, I'm guessing this commit fixed it:
18:06:12 dtruong https://opendev.org/openstack/openstacksdk/commit/e7a9b52998f23d7d7af99ecd9adb0554102f9604
19:33:16 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
19:33:33 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
19:34:34 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
21:34:29 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
21:34:47 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
22:09:19 dtruong mordred I created a bug report for the issue I described above: https://storyboard.openstack.org/#!/story/2005306
#openstack-sdks - 2019-03-27
11:43:25 dtantsur mordred: hi! it seems like this whole thing is broken nowadays: https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/openstack.py#L145-L154
11:43:40 dtantsur (in addition to os_ironic_node having authentication half-broken as well)
11:51:33 dtantsur sigh, I just wanted to fix some deprecations >_<
12:58:04 openstackgerrit ITD27M01 proposed openstack/openstacksdk master: Adds missing "params" attribute for creating a Mistral workflow execution https://review.openstack.org/648117
13:00:51 openstackgerrit ITD27M01 proposed openstack/openstacksdk master: Adds missing "params" attribute for creating a Mistral workflow execution https://review.openstack.org/648117
13:02:48 openstackgerrit ITD27M01 proposed openstack/openstacksdk master: Adds missing "params" attribute for creating a Mistral workflow execution https://review.openstack.org/648117
14:14:45 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
16:22:05 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_transfer_request.py and test_snapshot.py https://review.openstack.org/647892
16:40:11 openstackgerrit Ian Y. Choi proposed openstack/openstacksdk master: DNM - Start building pdfs of docs https://review.openstack.org/601659
20:39:52 openstackgerrit Pierre Riteau proposed openstack/openstacksdk master: Do not disregard tags when updating stacks https://review.openstack.org/643195
#openstack-sdks - 2019-03-28
00:31:49 dtruong mordred Would it be possible to cut a new openstacksdk release off master branch? That would fix the connection leak problem for Senlin.
00:32:06 dtruong I also send an email to openstack-discuss with the same question: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/004295.html
06:46:28 mordred dtruong: yes, I thnk that should be no problem
06:49:42 mordred dtantsur|afk: boo to that being broken. :(
07:02:42 mordred dtruong: oh - I say that - doing it as a release for stein might be harder because of the various freezes.
07:06:12 mordred dtruong: I'm going to ask the release team what they think
07:48:12 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for generating form-post signatures https://review.openstack.org/639760
07:48:25 mordred timburke: ^^ thanks! nice catch
07:49:17 mordred Shrews, dtantsur|afk: ^^ timburke found a bug in that. I fixed the bug and added unit tests to actually exercise the methods in question which would have shown how horribly broken they were
08:59:02 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move set_temp_url_key logic into resource objects https://review.openstack.org/648347
09:01:09 gtema mordred, dtantsur: can you please do a next quick review on https://review.openstack.org/#/c/642218/ (split openstackcloud.py)
09:01:24 gtema I only did a "rebase"
09:01:50 gtema I would like to ask you to try to integrate it soon, as rebasing in it is a terrible mess with each new change
09:16:02 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove and rearrange object bonged proxy tests https://review.openstack.org/648352
09:16:39 mordred gtema: +2 from e
09:16:58 gtema thks mordred
09:18:36 gtema btw, I would like to modify service_descriptions.__set__ and __delete__ to be able to be able to override default descriptors (you know OTC case with the DNS implementation)
09:18:55 gtema are you ok with that?
09:20:41 mordred gtema: I *think* that should be ok - although my brain isn't super functional today, so I might be missing an implication :)
09:22:08 gtema ok, will prepare patch. My problem is that in a particular cloud a service might differ from upstream. We currently do have possibility to add service implementation in runtime, but not to "replace" the ones delivered in SDK
09:22:21 mordred yah
09:22:32 gtema so for me DNS implementation should be replaced
09:22:38 mordred of course, insert obligatory grumbling about clouds having incompatible services ...
09:22:50 gtema yupp
09:23:00 gtema thks
10:03:08 dtantsur gtema: sure thing! btw, how did you visa application go?
10:05:08 dtantsur and unrelated, but we need to fix the masakari job: http://logs.openstack.org/18/642218/9/check/openstacksdk-functional-devstack-masakari/bd6a050/job-output.txt.gz#_2019-03-26_16_03_12_675170
11:49:08 openstackgerrit Merged openstack/openstacksdk master: Split OpenStackCloud into reasonable pieces https://review.openstack.org/642218
11:49:09 openstackgerrit Merged openstack/openstacksdk master: Do not disregard tags when updating stacks https://review.openstack.org/643195
11:57:38 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Make PATCH a first class operation and support it for baremetal https://review.openstack.org/647730
11:59:26 gtema dtantsur: thanks. Visa is there. And I was really surprised to get it on 3 days after interview
12:00:45 gtema s/on/in/

Earlier   Later