Posted	Nick	Remark
#openstack-sdks - 2020-04-01
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
15:34:41	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: Attempt to run Ansible-test from the devel ansible branch https://review.opendev.org/717062
15:59:49	elmiko	API SIG office hour now open \o/
16:01:07	dtantsur|brb	o/
16:16:02	openstackgerrit	Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
16:46:06	smcginnis	mordred: In case you automatically bit bucket the notification emails - https://review.opendev.org/#/q/status:open+project:openstack/releases+branch:master+topic:u-final-lib
16:47:38	mordred	smcginnis: thanks - I do bit bucket those ... lemme double check then ack
16:50:07	brtknr	Anyone know how to get the server version of a service from the openstack client?
16:51:04	mordred	brtknr: no - I can tell you how to get the API version using sdk (and we should totally expose that in osc)
16:51:17	mordred	but to my knowledge teh server version is not publised anywhere api accessible
16:52:25	brtknr	I suppose the server version can be partially inferred from api version but not going to be exaxt
16:52:49	mordred	brtknr: there's a script in the sdk repo - tools/nova_version.py - that will fetch and print the nova api version for every cloud in your clouds.yaml
16:53:09	mordred	it's super ugly - but it might be a cargo-cult starting point
16:53:22	mordred	and this is a good reminder that we should make an OSC command that does the same thing
16:53:25	brtknr	Cool thanks I’ll take a look
16:54:27	mordred	brtknr: I'm sorry - we have one!
16:54:39	mordred	brtknr: openstack versions show
16:54:46	brtknr	Sweet
16:58:15	brtknr	mordred: That’s awesome
17:41:34	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266
18:17:26	openstackgerrit	OpenStack Release Bot proposed openstack/os-client-config stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/717095
18:17:28	openstackgerrit	OpenStack Release Bot proposed openstack/os-client-config stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/717096
18:17:31	openstackgerrit	OpenStack Release Bot proposed openstack/os-client-config master: Update master for stable/ussuri https://review.opendev.org/717097
18:46:57	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Volume.backup API attr name fixes https://review.opendev.org/717047
18:47:50	gtema	our resource didn't like such a loop in the attribute names/aliases -> tiny fix
19:37:28	openstackgerrit	Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with ansible-base https://review.opendev.org/717108

Earlier   Later