Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-30
22:30:58 mriedem wordsmithing that a bit
22:31:00 mriedem in the commit message
22:31:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Downgrade log for keystone verify client fail https://review.openstack.org/513243
22:31:15 mriedem efried: see if ^ makes sense
22:32:38 mriedem hmm now that i think about it,
22:32:43 efried mriedem Hum, I don't think "like there isn't a keystone v3 endpoint" is right - that would have been covered by the previous except
22:32:45 mriedem your change should go on top of chen's
22:33:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Use ksa adapter for keystone conf & requests https://review.openstack.org/507693
22:33:16 mriedem i rebased your ksa adapter change on top of his, so we can backport his w/o weirdness
22:33:51 efried Dang, mine was so close to merging :)
22:34:53 mriedem you going to hold this up on the "like there isn't a keystone v3 endpoint" thing or should we just shoot the moon?
22:35:00 mriedem else i'll get sdague to +W
22:38:15 mgagne is the 'admin' role hardcoded in the placement api (in newton)? :O https://github.com/openstack/nova/blob/stable/newton/nova/api/openstack/placement/handler.py#L153
22:39:21 mriedem see the TODO in there
22:39:32 mgagne yea, got fixed in ocata
22:39:46 mgagne but we no longer have an "admin" role in our setup :-/
22:39:46 melwitt mriedem: I'm requesting a g-r bump for the crazy compute host reboot thing https://review.openstack.org/#/c/524413
22:40:04 mriedem melwitt: on master?
22:40:17 melwitt mriedem: yeah
22:40:24 mriedem melwitt: i thought your issue was in newton?
22:40:35 melwitt it is, but it's still broken on master
22:41:18 mriedem mgagne: this? https://github.com/openstack/nova/commit/cb9300fd37225c0b744f1f85e4c3bb940e402a8b
22:41:22 melwitt the instance can 'nova start' after a compute host reboot, but the volume isn't readable. I tested it with devstack
22:41:39 melwitt *the encrypted volume isn't readable
22:41:57 mgagne mriedem: now it's hardcoded at line 39 of nova/api/openstack/placement/policy.py
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

Earlier   Later