Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-21
10:42:03 ITD27M01 gtema: Ok, Can you please explain me the logic of the "proxy" in sdk, I have heard from you many times, but did not understand.
10:43:53 gtema see example in https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_image.py#L75
10:44:30 gtema cloud/_dns.py:list_recordsets should use self.dns.recordsets function instead of doing request itself
10:45:36 gtema unfortunately I do not have today to fix this. But all of the functions in the cloud/_dns should use methods defined in openstack/dns/v2/_proxy.py instead of manual requests
10:45:55 gtema this is however also a "bigger" change with respect to adapting tests properly
10:51:10 ITD27M01 gtema: Ok, I will create bug request for it so we don't lose it.
10:52:24 gtema great
11:29:43 brtknr Hello
11:29:59 brtknr Whats the protocol for backporting changes in ansible
11:30:23 brtknr I have these two backports from dev branch to 2.8 and 2.7: https://github.com/ansible/ansible/pull/56708 https://github.com/ansible/ansible/pull/56709
11:30:49 gtema brtknr: https://docs.ansible.com/ansible/latest/community/development_process.html
11:36:55 brtknr I have followed the protocol here:
11:36:57 brtknr https://github.com/ansible/ansible/pull/56710
11:40:16 brtknr And here: https://github.com/ansible/ansible/pull/56711
13:42:08 mordred gtema: I look forward to getting that "use proxy layer in cloud layer" project done
13:42:19 mordred having proper pagination support everywhere will be nice
13:42:39 gtema hehehe
13:42:50 gtema can be done soon
13:43:16 gtema would appreciate your thoughts on vendor_hook and rework of stat reporting
13:43:33 gtema and then can go for "use proxy"
13:43:34 mordred yes! they are on my review-today list
13:43:41 gtema great
15:04:45 ITD27M01 gtema: https://storyboard.openstack.org/#!/story/2005749
15:06:10 gtema ITD27M01: taken.
15:06:17 gtema thanks
15:20:44 ITD27M01 gtema: I have also another proposal for you today. What do you think about filtering feature in Ansible openstack inventory plugin?
15:20:56 ITD27M01 gtema: https://github.com/ansible/ansible/pull/51401
15:21:11 ITD27M01 In some cases, the host patterns cannot help to split infrastructure into small pieces. For example when openstack project/tenant contains
15:21:11 ITD27M01 thousands of instances. And a lot of users works with such a project.
15:21:33 ITD27M01 The aws inventory plugin is an example of such filtering:
15:21:43 ITD27M01 https://docs.ansible.com/ansible/latest/plugins/inventory/aws_ec2.html
15:21:47 ITD27M01 In the case of server-side filtering, we can get a significant performance gain.
17:28:40 openstackgerrit sean mooney proposed openstack/python-openstackclient master: add openstack server resurrect command https://review.opendev.org/643578
#openstack-sdks - 2019-05-22
06:51:39 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: allow updating name and extra fields of an allocation https://review.opendev.org/660578
08:37:37 ITD27M01_ gtema: Hello! I have also another proposal for you today. What do you think about filtering feature in Ansible openstack inventory plugin?
08:37:39 ITD27M01_ https://github.com/ansible/ansible/pull/51401
08:39:01 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: allow updating name and extra fields of an allocation https://review.opendev.org/660578
10:15:32 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: allow updating name and extra fields of an allocation https://review.opendev.org/660578
10:23:56 gtema ITD27M01_, with respect to ansible pool you mentioned. We have now finally all the possible filters from nova inside SDK
10:24:20 gtema what we still need, however, is support all of those in the cloud/_compute.py
10:24:53 gtema This will be likely covered same as for DNS during switching all of those methods to use service proxies
10:25:31 gtema I will start working on those next week
10:37:48 ITD27M01_ gtema: Do you mean that cloud.list_servers can be changed?
10:37:53 ITD27M01_ gtema: I want to invite you to participate in the discussion and review of the pull-request.
10:38:00 ITD27M01_ https://github.com/ansible/ansible/pull/51401
10:39:18 gtema well, to be honest I see currently that cloud.list_servers already uses proxy layer, meaning all the filters should be already supported
10:39:56 gtema only that there is no filter for metadata
11:01:46 ITD27M01_ gtema: Only one caveat about inventory module in cloud. There is search_hosts method which uses the _utils._filter_list for filtering.
11:01:47 ITD27M01_ https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/inventory.py#L79
11:02:04 openstackgerrit Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstacksdk master: Add ability to provide qos_policy_id for port https://review.opendev.org/660680
11:06:18 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: allow updating name and extra fields of an allocation https://review.opendev.org/660578
11:06:44 ITD27M01_ gtema: Remembering the conversation about the division of filters into the server-side and client-side. I do not quite agree with this implementation - there will be a lot of calculations on the client side (in large installations).
13:11:35 stingrayza mnaser: could I pester you for an updated shipit on https://github.com/ansible/ansible/pull/56453 please? (it seems as if the changes I made after invalidated your original one)
13:11:55 mnaser stingrayza: let me have a look
13:12:02 stingrayza thanks! :)
13:12:33 mnaser stingrayza: voilaa
13:14:02 stingrayza mnaser: thanks a mil :)
13:37:09 stingrayza (I'd love a shipit from anybody else that can throw one up... https://github.com/ansible/ansible/pull/56453)
13:55:55 stingrayza mordred: :hat-tip: thanks :)
14:39:48 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:46:48 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'openstack server resurrect' command https://review.opendev.org/643578
19:49:06 openstackgerrit Dean Troyer proposed openstack/osc-lib master: Final switch to stestr https://review.opendev.org/660815
#openstack-sdks - 2019-05-23
02:36:20 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373
04:37:14 openstackgerrit Merged openstack/osc-lib master: Final switch to stestr https://review.opendev.org/660815
06:00:53 openstackgerrit Chinmay Naik proposed openstack/keystoneauth master: Resolves a typo in a link to use Application Credentials https://review.opendev.org/660886
06:38:23 openstackgerrit zhulingjie proposed openstack/keystoneauth master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655016
07:24:04 openstackgerrit ZhouHeng proposed openstack/python-openstackclient master: openstack port create support --extra-dhcp-option https://review.opendev.org/657519
07:43:24 openstackgerrit ZhouHeng proposed openstack/python-openstackclient master: openstack port create support --extra-dhcp-option https://review.opendev.org/657519
08:08:06 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
08:17:32 dtantsur mordred, gtema, 2x +2, anything blocking approval? https://review.opendev.org/#/c/660578/
08:18:01 gtema :) - done
08:18:04 dtantsur thx!
08:37:16 openstackgerrit Merged openstack/osc-lib master: Dropping the py35 testing https://review.opendev.org/654659
09:44:38 openstackgerrit Vishakha Agarwal proposed openstack/keystoneauth master: Bump bandit version and cap sphinx https://review.opendev.org/660956
09:55:27 openstackgerrit Merged openstack/openstacksdk master: baremetal: allow updating name and extra fields of an allocation https://review.opendev.org/660578
10:11:21 openstackgerrit Merged openstack/openstacksdk master: Add support for vendor hooks https://review.opendev.org/657848
10:21:44 openstackgerrit Merged openstack/openstacksdk master: Add ability to provide qos_policy_id for port https://review.opendev.org/660680
11:58:26 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:26:48 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'openstack server resurrect' command https://review.opendev.org/643578
14:16:50 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Make 'destination' optional for live migration https://review.opendev.org/626949
14:29:47 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'openstack server resurrect' command https://review.opendev.org/643578
14:52:48 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Compute: Add user id support for keypair https://review.opendev.org/576507
15:46:18 openstackgerrit Sergii Golovatiuk proposed openstack/openstacksdk master: Log OpenStackCloudResourceNotFound exceptions https://review.opendev.org/637169
16:00:03 edleafe Wow! Time for another API-SIG Office Hour!!
16:03:15 cdent I wonder if the api-sig saw the discussion between me and cmurphy about the errors guideline?
16:03:59 cdent this thread: http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006377.html
16:04:21 cdent and colleen's comments: http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006425.html
16:05:30 elmiko o/
16:05:57 edleafe I saw it, but I wasn't around for any of the original discussions, so I can't add much
16:11:35 edleafe elmiko: you +W'd that change - any impressions?
16:14:23 elmiko i'm reading now
16:14:55 cdent but if you've got stuff to say please add to the thread
16:17:05 elmiko i'm really not sure how to respond to colleen's question
16:17:23 elmiko i mean, having a list of errors seemed like a good idea at the time because more than one could be returned
16:17:44 elmiko i think it was also inspired by prior art within the openstack community
16:17:47 edleafe Are you aware of any other API that does anything similar?
16:17:56 edleafe jinxish
16:23:03 elmiko i vaguely recall at the time we had examples of others doing it that way
16:28:54 elmiko after looking around, maybe we just need to revisit that guideline?
16:29:15 elmiko i don't remember how we arrived at that, and i can't seem to find the reference materials

Earlier   Later