| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-30 | |||
| 21:18:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 21:18:46 | efried | jaypipes ack | |
| 21:19:33 | efried | jaypipes +1. Ship it! | |
| 21:19:40 | rwellum | mriedem: that indeed fixed the database sync error! However I still am getting 'No valid host was found. There are not enough hosts available' | |
| 21:19:58 | mriedem | rwellum: have you created cell1 and discovered hosts? | |
| 21:20:07 | mriedem | rwellum: i thought kolla automated all of this? | |
| 21:20:54 | dansmith | I thought kolla was just containers of stuff | |
| 21:23:42 | mriedem | i thought there was an ansible part to kolla for actually deploying openstack in those containers | |
| 21:23:52 | rwellum | Yeah kolla is just the images and some config built into the image. I'm orchestrating with kolla-kubernetes. Works with ocata btw. | |
| 21:23:55 | mriedem | clearly this isn't the place to ask about wtf kolla is or does :) | |
| 21:24:30 | mriedem | i remember the kolla guys asking about the cellsv2 discover_hosts stuff in pike | |
| 21:24:34 | rwellum | Yeah my bad. More wanted to know where to start with with a generic error like the above. | |
| 21:24:36 | mriedem | so that is automated somewhere in the kolla stadium | |
| 21:24:57 | rwellum | I'll try to find who did it. There's only a few of them. | |
| 21:25:06 | mriedem | http://git.openstack.org/cgit/openstack/kolla-ansible/ ? | |
| 21:25:46 | mriedem | http://git.openstack.org/cgit/openstack/kolla-ansible/tree/ansible/roles/nova/tasks/simple_cell_setup.yml ? | |
| 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 | |