Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-30
22:42:04 mriedem sdague: https://review.openstack.org/#/c/513243/ and then we can backport
22:42:39 mriedem ah i see
22:43:01 mriedem forgot all about the policy stuff for placement
22:47:05 mgagne I'll see what I can hack on my side to get passed that, maybe just load the role name from nova.conf so I don't have to include nova.policy in there. and way for official fix later
22:54:06 mriedem i hope lbragstad doesn't find out we haven't satisfied the queens community goal for policy in code for placement...
22:56:54 lbragstad mriedem: looks like you register at least one rule
22:56:55 lbragstad https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/policy.py#L45
23:05:28 mgagne local patch I will try to apply and use: https://gist.github.com/mgagne/b43c1e085c1f1d50bebc054a7d387688
23:09:19 mgagne and now I'm off, gonna check on Monday and report back on my experience with placement api =)
23:10:06 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: General policy sample file for placement https://review.openstack.org/524425
23:10:07 mriedem mgagne: starting stuff here ^ but i don't plan on working on this really
23:11:06 mgagne will try to see if I get time to check/review, if I have a workaround, I might not make it a priority though =)
23:13:14 lbragstad mgagne: if you need help with the policy-in-code bits, let me know
23:14:01 mriedem i don't think he's planning on writing the code :)
23:14:11 mgagne lbragstad: thanks for the offer. I'm working with old code for now so I don't know yet how relevant my findings will be. will check against master and report back accordingly
23:31:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
23:46:10 efried mriedem No, not going to hold it up for that.
#openstack-nova - 2017-12-01
00:24:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/523194
00:24:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027
00:25:19 mriedem alex_xu: is this kind of what you were looking for? https://review.openstack.org/#/c/523194/2/nova/api/openstack/compute/rest_api_version_history.rst
00:34:23 tetsuro jay: are you there?
01:19:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
01:39:01 openstackgerrit Merged openstack/nova master: Hyper-V: fix live migration with CSVs https://review.openstack.org/380448
01:40:02 openstackgerrit Merged openstack/nova master: Remove unused global variables https://review.openstack.org/524126
01:40:33 openstackgerrit Merged openstack/nova master: Vzstorage: synchronize volume connect https://review.openstack.org/505708
01:41:04 openstackgerrit Merged openstack/nova master: Fix incorrect known vcpuset when CPUPinningUnknown raised https://review.openstack.org/517592
01:41:31 openstackgerrit Merged openstack/nova master: Use NoDBTestCase for powervm driver tests https://review.openstack.org/520118
01:42:03 openstackgerrit Merged openstack/nova master: [placement] Fix format in placement API ref https://review.openstack.org/521483
01:42:30 openstackgerrit Merged openstack/nova master: [placement] Fix parameter order in placement API ref https://review.openstack.org/516233
01:42:57 openstackgerrit Merged openstack/nova master: Fix TypeError of _get_project_id when project_id is None https://review.openstack.org/522100
01:43:23 openstackgerrit Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505
01:43:24 openstackgerrit Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
01:43:30 openstackgerrit Merged openstack/nova master: VMware: Handle concurrent registrations of the VC extension https://review.openstack.org/484667
01:44:03 openstackgerrit Merged openstack/nova master: [placement] Add 'CUSTOM_' prefix description in API ref https://review.openstack.org/521495
01:44:32 openstackgerrit Merged openstack/nova master: Update description of Rebuild in server_concepts.rst https://review.openstack.org/522086
02:08:46 gmann gibi: thanks
02:10:18 gmann mriedem: on query schema BP - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-query-param
02:11:06 gmann mriedem: only API left is cell (v1), which anyway going to be deprecated and removed so i am thinking to skip the schema for this
02:11:33 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007
02:14:05 gmann mriedem: else this BP is completed.
02:19:13 gmann mriedem: we did for deprecated APIs but cell one is not just deprecation case but going to be removed that why we can skip this ? or its easy schema for cell i can implement if you want
02:28:07 openstackgerrit Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443
02:53:19 openstackgerrit Merged openstack/nova master: Remove old-style quotas code https://review.openstack.org/511689
03:01:32 alex_xu jaypipes: sir, good evening, do you want to unlock this patch https://review.openstack.org/#/c/517119/?
03:01:47 alex_xu jaypipes: or anything I can help to unlock that patch
03:58:48 efried alex_xu I believe jaypipes said he was working on it locally.
04:00:09 efried alex_xu Never mind - he rebased it, but I think he thought he was done at that point :)
04:02:41 openstackgerrit jichenjc proposed openstack/nova master: Refactor placement version check https://review.openstack.org/512497
04:07:56 openstackgerrit jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215
04:33:53 openstackgerrit Merged openstack/nova stable/pike: doc: fix flavor notes https://review.openstack.org/509438
04:43:13 openstackgerrit Nguyen Van Trung proposed openstack/nova-specs master: Add doc8 to pep8 check for nova-specs project https://review.openstack.org/519561
04:44:00 alex_xu efried: cool, then just let us waiting for the update
04:50:38 openstackgerrit Merged openstack/nova master: Do not use “-y” for package install https://review.openstack.org/518238
05:02:42 openstackgerrit jichenjc proposed openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
05:21:01 openstackgerrit jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392
05:21:02 openstackgerrit jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397
06:25:36 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
06:31:29 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
06:45:05 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653
07:11:07 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
07:44:55 nsingh efried: To create keystoneauth session we are using some parameters like insecure, verify, cacert, cert, and key. And to creating sessionClient we are passing keystoneauth session for nova/glance/cinder. but these conf options from session are not used anywhere while creating session client. so is it only necessary to create session?
08:13:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506
08:17:50 openstackgerrit Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443
08:39:40 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
08:39:41 openstackgerrit Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
08:49:46 openstackgerrit sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457
09:00:17 openstackgerrit Erik Berg proposed openstack/nova stable/pike: Fix binary name https://review.openstack.org/524531
09:00:19 openstackgerrit Merged openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871
09:16:51 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
10:11:30 openstackgerrit Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309
10:15:01 stephenfin Man, Gerrit is slooowww this morning
10:19:44 bauzas stephenfin: probably because of the close week-end ? :p
10:20:00 bauzas or because it's snowing here
10:20:16 bauzas FWIW, resorts are opening this week-end \o/
10:20:30 openstackgerrit Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309
11:10:47 openstackgerrit Merged openstack/nova stable/pike: Fix race in delete allocation in ServerMovingTests https://review.openstack.org/508872
12:08:54 efried nsingh I have noticed that too - In Nova we go through all this trouble to create a session from conf parameters, and then we create the client for the service which creates its own session.
12:09:18 efried nsingh Eventually the vision (per mordred) is to get rid of the clients altogether and just use the ksa adapter for the API communication.
12:32:51 nsingh efried: thank you for the info. :)
12:38:12 ildikov johnthetubaguy: hi, daily pretty please: https://review.openstack.org/#/c/330285/
12:43:52 openstackgerrit Merged openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655
13:22:22 fried_rice Friday!
13:24:16 openstackgerrit Chen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command https://review.openstack.org/524588
13:26:08 fried_rice stephenfin You around?
13:27:34 fried_rice or bauzas or gibi or alex_xu ? Easy review, which will merge another patch for a backportable bug: https://review.openstack.org/#/c/524245/
13:40:46 stephenfin fried_rice: I am. Looking now
13:40:53 fried_rice stephenfin Thanks!
13:42:14 finucannot fried_rice: Are you finding Gerrit slow today too?
13:42:28 fried_rice finucannot Yeah, totally. I asked in -infra, but nobody bit yet.
13:42:43 finucannot So not my network. Good to know.
13:42:52 fried_rice That was an hour ago - more people are awake now. I'll ask again...
13:43:05 jaypipes fried_rice: mornin
13:43:10 fried_rice jaypipes Howdy!
13:43:19 cdent I’ve been out of a bed for a long time, but I make no claims to being awake.
13:43:28 leakypipes fried_rice: what was the thing with alex from earlier?
13:43:46 fried_rice leakypipes He was asking about https://review.openstack.org/#/c/517119/
13:44:04 leakypipes fried_rice: ah, k
13:44:06 fried_rice I had thought you were making local changes to it, but looking back it seemed you had only rebased it.
13:44:13 leakypipes fried_rice: yeah, I'll work on that this mroning.
13:44:17 fried_rice Cool beans.

Earlier   Later