Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-30
21:26:17 mriedem https://review.openstack.org/#/c/456920/ is the change i remember
21:26:37 mriedem i think
21:26:42 openstackgerrit melanie witt proposed openstack/nova master: Improve the CellDatabases test fixture and usage https://review.openstack.org/508432
21:26:51 mriedem eduardo is definitely the person to ask
21:26:53 mriedem rwellum: ^
21:27:28 rwellum ty mriedem
21:33:27 rwellum mriedem: our code (not ansible - kubernetes) is calling: "nova-manage cell_v2 simple_cell_setup"
21:35:14 mriedem rwellum: probably want to read through this then https://docs.openstack.org/nova/latest/user/cells.html#step-by-step-for-common-use-cases
21:37:11 rwellum perfect thanks.
21:49:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle spawning error on unshelving https://review.openstack.org/378009
21:53:48 mriedem dansmith: +2 on the generic lister split
21:54:42 dansmith mriedem: thanks, did you comment on the migration stuff or should I?
21:54:44 mriedem hmm, the bottom patch failed tests http://logs.openstack.org/53/524253/3/check/openstack-tox-py27/540f78a/testr_results.html.gz
21:54:48 mriedem but those look unrelated
21:54:52 mriedem dansmith: i haven't yet
21:55:00 dansmith I rechecked because yeah
21:56:31 dansmith okay I commented on the one you suggested rolling back to PS21
21:58:58 mriedem thanks
22:26:58 mriedem efried: with the ksa adapter stuff, just to check my understanding, valid_interfaces defaults to (internal, public) and iterates through them when trying to find the endpoint?
22:27:08 efried mriedem yes
22:27:10 mriedem cool
22:27:21 mriedem was making sure i was explaining that correctly here https://review.openstack.org/#/c/513243/2
22:29:41 efried mriedem lgtm
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

Earlier   Later