Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-31
15:35:46 nightmare_unreal understood, I wanted to implement some missing functionalities in openstack cli
15:35:52 nightmare_unreal that are in novaclient
15:49:03 mordred awesome!
15:55:10 nightmare_unreal :)
16:16:03 dtruong hi. can I get a review on this: https://review.opendev.org/#/c/715255/
16:38:44 smcginnis dtruong: Probably good to fix the test failures first.
17:08:33 mordred dtruong: yeah - it looks ok - other than the test failures :)
17:16:40 openstackgerrit Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
17:31:28 dtruong ok, thanks. The test failures seem to be unrelated to my changes. So just keep rechecking until it passes?
17:45:59 smcginnis dtruong: Yeah, unfortunately. If the failure comes back as the same thing consistently, then we probably have something that snuck in that needs to be fixed first.
18:02:36 mordred the previous failure has been fixed in devstack
18:02:44 mordred so this one shold be green
18:11:55 openstackgerrit Andreas Jaeger proposed openstack/keystoneauth master: Update hacking for Python3 https://review.opendev.org/716334
19:49:36 openstackgerrit Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
20:38:42 dtruong Ok, the test failures are gone. Thanks for the help. Appreciate any reviews on it: https://review.opendev.org/#/c/715255/
22:00:58 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
22:06:46 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
23:50:29 openstackgerrit Merged openstack/openstacksdk master: Remove kwargs validation for identity project updates https://review.opendev.org/715255
#openstack-sdks - 2020-04-01
00:54:19 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
02:39:09 openstackgerrit Merged openstack/python-openstackclient master: Cleanup Python 2.7 support https://review.opendev.org/715446
03:47:33 openstackgerrit wangshenqiang proposed openstack/openstacksdk master: fixbug: error when import queue, supoort py2 & py3 at the same time. https://review.opendev.org/716479
07:19:09 openstackgerrit Merged openstack/keystoneauth master: Update hacking for Python3 https://review.opendev.org/716334
07:37:43 openstackgerrit Daniel Bengtsson proposed openstack/osc-lib master: Fix the requirement of keystoneauth1. https://review.opendev.org/695022
09:39:20 nightmare_unreal stephenfin: ping
09:45:08 AJaeger nightmare_unreal: please don't leave naked pings, see https://blogs.gnome.org/markmc/2014/02/20/naked-pings/
09:45:40 AJaeger nightmare_unreal: best to just leave a message and either stephenfin will answer later or somebody else might be able to help
09:45:57 nightmare_unreal okay , understood :)
09:48:57 nightmare_unreal can this be merged ? https://review.opendev.org/#/c/713881/ , it's code review is done too
09:51:27 gtema nightmare_unreal: I would really need to see a functional test for that in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/tests/functional/common/test_quota.py
09:55:01 AJaeger nightmare_unreal: we need two +2s, so stephenfin would not merge it, another reviewer is needed to give a +2 and approve
09:55:44 AJaeger gtema: isn't there a test in https://review.opendev.org/#/c/713881/10/openstackclient/tests/unit/common/test_quota.py ?
09:55:56 gtema this is just unit test
09:55:59 nightmare_unreal okay I thought one just need +2 .
09:56:09 AJaeger gtema: Ah, unit vs functional
09:56:13 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
09:56:14 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
09:56:16 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
09:56:17 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
09:56:18 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
09:56:19 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537
09:56:31 gtema in the current situation with team merges I guess we should not trust on only unit, since there is still no big understanding
09:57:15 AJaeger nightmare_unreal: https://docs.openstack.org/project-team-guide/review-the-openstack-way.html#code-review-plus-2
09:57:40 AJaeger gtema: I'm not objecting, just helping nightmare_unreal to understand
09:57:50 nightmare_unreal sure sure
09:57:51 gtema yes, no worries
11:03:52 openstackgerrit Merged openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
11:56:13 nightmare_unreal hey I am working on this : https://storyboard.openstack.org/#!/story/2007489 . I am not able to figure out how to call live_migration_abort function from novaclient to openstackclient.
12:08:47 nightmare_unreal https://www.irccloud.com/pastebin/N7zNj7FF/compute_client
12:09:04 nightmare_unreal I get following error : list() missing 1 required positional argument: 'server'
12:14:45 openstackgerrit Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
15:55:57 openstackgerrit Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
16:39:10 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: DNM testing inline comment fix https://review.opendev.org/716659
16:58:00 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: DNM testing inline comment fix https://review.opendev.org/716659
23:26:57 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
#openstack-sdks - 2020-04-02
00:12:44 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
01:13:51 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
09:57:45 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908
10:56:54 nightmare_unreal how can I mock migration ID ?
10:57:20 nightmare_unreal i am working on osc. trying to implement some nova migration commands.
10:57:33 nightmare_unreal I need to make test case so need to know how can I mock migration ID ?
11:20:20 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908
11:43:54 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
11:43:55 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
11:43:56 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
11:43:57 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
11:43:58 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
11:43:59 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537
11:48:38 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: Provides live migration abort feature in openstackclient https://review.opendev.org/716960
11:59:30 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
11:59:31 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
11:59:32 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
11:59:33 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
11:59:34 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537
12:03:10 nightmare_unreal can someone leave some comments on this : https://review.opendev.org/716960 . Kind of stuck
12:54:52 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: Add '--force; parameter to 'openstack quota set' https://review.opendev.org/713881
13:37:43 openstackgerrit Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
13:40:14 gtema holly cow, looking finally to failures from ansible module figured out new funny stuff
13:40:40 gtema in the block_storage volume.backup: on create param is "incremental" on get "is_incremental"
13:41:13 mordred gtema: like in the rest api?
13:41:18 gtema yes
13:41:21 mordred _wow_
13:41:26 gtema https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=create-a-backup-detail,show-backup-detail-detail#backups-backups
13:42:05 mordred gtema: yup. you are correct there
13:42:53 gtema how should we solve this: "alias", "aka", etc?
13:43:05 gtema I think "aka" should be fine for that
14:01:22 nightmare_unreal smcginnis: saw your comment on https://review.opendev.org/716960 . Can you look at the test case I made. To be honest I imitated the same test case of ( novaclient/server_migrations) for the openstackclient. I am checking the live_migration_abort() .
14:22:51 smcginnis nightmare_unreal: Sure, I'll try to get to that soon.
14:23:05 nightmare_unreal Thanks :)
14:27:05 openstackgerrit OpenStack Release Bot proposed openstack/keystoneauth stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/717028
14:27:07 openstackgerrit OpenStack Release Bot proposed openstack/keystoneauth stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/717029
14:27:08 openstackgerrit OpenStack Release Bot proposed openstack/keystoneauth master: Update master for stable/ussuri https://review.opendev.org/717030
14:30:06 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908
14:35:52 openstackgerrit Merged openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
14:42:09 openstackgerrit Merged openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
14:42:10 openstackgerrit Merged openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
14:48:54 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: Add '--force; parameter to 'openstack quota set' https://review.opendev.org/713881
15:13:37 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Volume.backup API attr name fixes https://review.opendev.org/717047
15:24:03 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537
15:27:27 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
15:27:28 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992

Earlier   Later