Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-06-25
15:09:36 Shrews hrm, if ironic doesn't gate on it, yeah...
15:15:47 dtantsur TheJulia: we should maybe add a bifrost job and revert ^^^
15:16:05 openstackgerrit Merged openstack/python-openstackclient master: Network: Add tag support for security group https://review.openstack.org/547733
15:41:00 dtantsur mordred, TheJulia, https://review.openstack.org/#/c/577848/ seems to pass the same set of jobs that used to pass before pxe_ilo removal
15:45:47 mordred dtantsur: \o/
15:54:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move clustering additions from shade directories https://review.openstack.org/577654
15:54:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Re-enable clustering functional tests https://review.openstack.org/577852
15:58:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move clustering additions from shade directories https://review.openstack.org/577654
15:58:42 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Re-enable clustering functional tests https://review.openstack.org/577852
16:47:10 openstackgerrit Merged openstack-infra/shade master: Switch bifrost jobs to nonvoting https://review.openstack.org/577846
17:43:35 TheJulia just now able to context switch over, but I concur
18:51:53 openstackgerrit Merged openstack-infra/shade master: Use RequestsMockTestCase everywhere https://review.openstack.org/577658
18:52:28 openstackgerrit Merged openstack-infra/shade master: Convert domain params tests to requests_mock https://review.openstack.org/577659
18:59:27 TheJulia mordred: not a cat? :(
19:03:16 openstackgerrit Merged openstack-infra/shade master: Convert test_caching to requests-mock https://review.openstack.org/577660
19:25:40 mordred TheJulia: well, it might not be a puppet either
19:27:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Convert domain params tests to requests_mock https://review.openstack.org/577663
19:27:40 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Convert test_caching to requests-mock https://review.openstack.org/577664
19:27:45 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Convert image_client mocks in test_shade_operator https://review.openstack.org/577665
19:27:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Finish migrating image tests to requests-mock https://review.openstack.org/577666
21:27:11 flwang mordred: around? i'm a little bit confusing about the openstack support in ansible. based on the code of ansible, seems it's using both openstacksdk and shade, but mainly it's openstacksdk, right?
21:32:27 TheJulia mordred: that was a really bad joke ;)
21:33:15 TheJulia flwang: in the process of migrating as shade was imported into openstacksdk to superceed shade
21:33:28 mnaser flwang: https://docs.openstack.org/openstacksdk/latest/#a-brief-history might be a fun read
21:34:19 openstackgerrit Merged openstack-infra/shade master: Convert image_client mocks in test_shade_operator https://review.openstack.org/577661
21:35:08 openstackgerrit Merged openstack-infra/shade master: Finish migrating image tests to requests-mock https://review.openstack.org/577662
21:39:12 flwang mnaser: so currently, i'm trying to support magnum in ansible, should I just contribute to openstacksdk directly?
21:40:26 flwang TheJulia: mnaser: thanks, that's the thing i want to figure out though I have to some brief answers from mordred
21:41:46 mnaser flwang: step 1 would be to make openstacksdk support it and step 2 would be to add ansible support
21:42:35 flwang mnaser: for step 1, i assume it means a new folder under https://github.com/openstack/openstacksdk/tree/master/openstack like containerinfra, right?
21:42:57 mnaser that's the part where i am not as useful :)
21:42:59 flwang instead of putting code into https://github.com/openstack/openstacksdk/tree/master/openstack/cloud/openstackcloud.py ？
22:30:22 TheJulia flwang: i am also not current pn the desired process, but in the past i've just proposed the desired methods to the main file.
22:35:22 flwang TheJulia: actually, i don't mind adding magnum support for both shade and cloud/openstackcloud.py, and even adding a new folder like /openstack/containerinfra, but I do need to understand in ansible which will be the way ansible calling
22:35:42 flwang not sure if there is a confirmed direction i can follow now
22:54:08 TheJulia flwang: i believe the desired direction is just openstacksdk, fwiw
22:55:53 flwang TheJulia: cheers, that's helpful
#openstack-sdks - 2018-06-26
06:00:45 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute:Support keypair set key-type https://review.openstack.org/577768
12:46:04 mordred flwang: just openstacksdk in openstack/cloud/openstackcloud.py - we can cherry-pick that shade patch you already wrote over
13:08:12 mordred at some point it would also probably be good to add a container_infrastructure_management directory and flesh out sdk-level support - but for now the shade-style methods are the only ones you need for ansible
13:09:06 mordred Shrews: for yourmorning reviewing pleasure - yesterday's shade patches are now green on sdk: https://review.openstack.org/577654 through https://review.openstack.org/577666
13:12:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Magnum cluster support https://review.openstack.org/578097
13:13:44 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Magnum cluster support https://review.openstack.org/578097
13:13:56 mordred flwang: ^^ there's a cherry-pick of your patch
13:20:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Magnum cluster support https://review.openstack.org/578097
14:52:58 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Improve Magnum cluster templates functions https://review.openstack.org/578119
15:50:55 Shrews mordred: so generous of you
16:12:53 mordred Shrews: I like providing joy
17:33:01 openstackgerrit Merged openstack/openstacksdk master: Move clustering additions from shade directories https://review.openstack.org/577654
17:33:02 openstackgerrit Merged openstack/openstacksdk master: Convert domain params tests to requests_mock https://review.openstack.org/577663
17:33:03 openstackgerrit Merged openstack/openstacksdk master: Convert test_caching to requests-mock https://review.openstack.org/577664
17:33:04 openstackgerrit Merged openstack/openstacksdk master: Convert image_client mocks in test_shade_operator https://review.openstack.org/577665
17:33:04 openstackgerrit Merged openstack/openstacksdk master: Finish migrating image tests to requests-mock https://review.openstack.org/577666
18:04:57 openstackgerrit Merged openstack/openstacksdk master: Improve Magnum cluster templates functions https://review.openstack.org/578119
18:28:55 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add Magnum cluster support https://review.openstack.org/578097
19:28:32 openstackgerrit Merged openstack/openstacksdk master: Fix clustering profile type miss list operation https://review.openstack.org/576377
21:22:52 flwang mordred: thanks for the patch
21:23:10 mordred flwang: sure enough! sorry the two repos are in a confusion transition period right now
21:23:49 flwang mordred: so at least for the magnum support in Ansible, we don't need the openstacksdk/openstack/containerinfra ATM, right? just wanna double check
21:24:08 mordred flwang: that is correct
21:24:24 mordred we can totally do that later
21:24:59 flwang cool, that's the main thing i'd like to figure out
21:25:50 flwang as if I propose patch to ansible for openstack stuff, should I @ you as the reviewer/approver?
21:28:54 flwang mordred: ^
21:31:39 mordred flwang: yes - although the ansibot will already do that
21:31:55 mordred flwang: that said - drop a link to the PR in here
21:40:01 flwang mordred: awesome, cheers
21:40:09 flwang will bug you very ofter recently
21:40:24 flwang s/ofter/often
21:41:06 mordred bugging is good
22:14:39 nicolaso_ Hi! I need help with openstack SDK. I tried to perform a live migration but the sdk is sending the wrong parameters. I had to change the source code to make it work: https://github.com/nicolasochem/openstacksdk/commit/4a912a026a218e3b146fe1408c460608873ecfd4
22:14:49 nicolaso_ According to nova api, the "force" parameter is new on version 2.3 of the api. https://developer.openstack.org/api-ref/compute/#live-migrate-server-os-migratelive-action
22:15:01 nicolaso_ I am on pike and the sdk is using api version 2.1. Can I force sdk to use api version 2.3 ?
22:16:05 mordred nicolaso_: yeah - that's actually a bug - we shouldn't be sending force without also negotiating for the right microversion
22:16:44 mordred nicolaso_: I'm just stepping out for the night - but I can get a patch up to fix that in the morning
22:17:01 nicolaso_ That would be great, thanks.
22:23:12 nicolaso_ mordred: I created this bug report for you https://bugs.launchpad.net/python-novaclient/+bug/1778806
22:23:12 openstack Launchpad bug 1778806 in python-novaclient "live migration fails with sdk because of wrong api param" [Undecided,New]
#openstack-sdks - 2018-06-27
01:13:42 openstackgerrit Alan Bishop proposed openstack/python-openstackclient master: Deprecate volume create --project and --user options https://review.openstack.org/575804
01:15:21 openstackgerrit Alan Bishop proposed openstack/python-openstackclient master: Deprecate volume create --project and --user options https://review.openstack.org/575804
02:05:18 openstackgerrit Adrian Turjak proposed openstack/python-openstackclient master: Add an option to container create for storage policy https://review.openstack.org/564665
07:55:33 openstackgerrit tianhui proposed openstack/python-openstackclient master: Change bug url to a correct one https://review.openstack.org/578301
12:33:50 openstackgerrit Chris Dent proposed openstack/api-wg master: Add links to errors-example.json https://review.openstack.org/578369
13:31:20 openstackgerrit Merged openstack/python-openstackclient master: Change bug url to a correct one https://review.openstack.org/578301
14:01:53 cdent edleafe: have you, or do you know about 'use-storyboard' in the project-config for api-wg^wsig?
14:24:40 edleafe yes, I changed that
14:24:48 edleafe actually, added that
14:25:56 cdent edleafe: is it still pending?
14:26:22 edleafe no, storyboard is done
14:27:29 edleafe https://review.openstack.org/#/c/575120/
14:28:59 cdent hmmm. do you know if there's more we need to do make it do something than what I've done in https://review.openstack.org/#/c/578369/
14:29:31 cdent there's no apparent change in https://storyboard.openstack.org/#!/story/1593321 but maybe there isn't supposed to be any?
14:29:47 edleafe Yeah, I don't know about that.
14:30:16 edleafe You could ask in #storyboard
14:32:30 cdent part of my confusion was that I couldn't find your change in project-config, but have now
14:32:41 edleafe I guess I could ask, just as easily
14:33:10 cdent no worries, I'm already in flow
15:06:37 openstackgerrit qingszhao proposed openstack/os-client-config master: Add release note link in README https://review.openstack.org/578433
15:14:49 openstackgerrit qingszhao proposed openstack/os-service-types master: Add release note link in README https://review.openstack.org/578439
15:16:56 frickler Steve Martinelli doesn't seem to hang around in IRC anymore, does he?

Earlier   Later