| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-22 | |||
| 17:56:58 | imacdonn | thanks* | |
| 17:58:30 | imacdonn | melwitt: Actually, though, this still seems to suggest that I can "just not run the service" ... and that's not working for me | |
| 17:59:19 | melwitt | yeah, sorry. indeed you are supposed to be able to not run the service | |
| 18:00:08 | melwitt | imacdonn: do you have a single cell deployment or multiple cells? | |
| 18:00:30 | imacdonn | melwitt: single cell, not counting cell0 | |
| 18:02:10 | melwitt | do you have a trace you can pastebin of the nova-api log and nova-novnc | |
| 18:03:22 | imacdonn | melwitt: I guess I'll enable debug logging and see what I can get ... wanted to make sure I wasn't missing a required config change or something first | |
| 18:05:39 | melwitt | here's examples from the gate of what the configs should look like for the cell and compute host http://logs.openstack.org/58/540258/13/check/tempest-full-py3/f59f34a/controller/logs/etc/nova/nova_cell1_conf.txt.gz http://logs.openstack.org/58/540258/13/check/tempest-full-py3/f59f34a/controller/logs/etc/nova/nova-cpu_conf.txt.gz | |
| 18:19:03 | melwitt | imacdonn: I made a mistake, sorry. you do have to run the consoleauth service in Rocky still. being able to discontinue running it will be in Stein | |
| 18:19:30 | imacdonn | melwitt: Oooh okay .... so ... uhm ... it's not really deprecated yet, then ....... ? | |
| 18:20:10 | melwitt | imacdonn: it's deprecated but not yet removed. in rocky it's doing things side-by-side, then in stein we can remove it | |
| 18:20:16 | imacdonn | melwitt: where my definition of "deprecated" is something like "you can still do this, but you really shouldn't, and you should do it the new way instead" | |
| 18:21:20 | melwitt | yeah, I see what you mean. and the docs are wrong saying you can stop running it in rocky, so I need to fix that | |
| 18:21:44 | imacdonn | melwitt: yes, that was going to be my next point ;) | |
| 18:22:37 | melwitt | yeah, apologies for that. I'll work on some updates to the docs right now | |
| 18:23:01 | melwitt | mriedem_lunch ^ think this will be another thing for rc3 | |
| 18:23:46 | imacdonn | no problem! Thanks for working through it with me | |
| 18:24:40 | imacdonn | part of my practice for each new release is to review logs for deprecation warnings, and eliminate as many as I can | |
| 18:25:33 | melwitt | that's a good practice. sorry that doc threw a wrench in it | |
| 18:26:11 | imacdonn | Glad to catch it before the release! | |
| 18:26:34 | melwitt | yes, definitely. thank you for that | |
| 18:27:27 | imacdonn | np | |
| 19:00:18 | mriedem | imacdonn: can you report a bug that the docs are wrong? | |
| 19:00:41 | imacdonn | mriedem: ok | |
| 19:01:04 | melwitt | I'm in the middle of writing up a bug | |
| 19:02:15 | melwitt | heh | |
| 19:02:34 | mriedem | https://www.youtube.com/watch?v=Vvwq1FZ6gxE | |
| 19:03:30 | imacdonn | :) | |
| 19:05:10 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Handle binding_failed vif plug errors on compute restart https://review.openstack.org/595317 | |
| 19:06:59 | openstack | Launchpad bug 1788470 in OpenStack Compute (nova) "release notes and docs erroneously say that nova-consoleauth doesn't have to run in Rocky " [Medium,New] - Assigned to melanie witt (melwitt) | |
| 19:06:59 | melwitt | https://bugs.launchpad.net/nova/+bug/1788470 | |
| 19:54:39 | mriedem | so uh, i could use some reviews on this rc3 potential fix https://review.openstack.org/#/c/594508/ | |
| 19:54:41 | mriedem | "fix" | |
| 19:55:42 | melwitt | I'll review the rc3 things today. it's been a distracting day | |
| 19:56:59 | mriedem | i'll review yours if you review mine | |
| 19:57:24 | mriedem | the other one, approved yesterday morning, is still humping its way through the gate | |
| 20:01:04 | melwitt | ok, cool. I saw on the todos etherpad | |
| 20:28:36 | mriedem | dansmith: i'm slowly climbing multi-cell list batch mountain | |
| 20:28:41 | mriedem | nearly at the summit | |
| 20:28:49 | mriedem | oxygen....depleting... | |
| 20:41:21 | melwitt | I found that the console code in compute is storing token auths in both the database and nova-consoleauth for rocky, and I'm struggling to understand why. it was like that before I took on the patch series. I assume it was just so operators could keep using nova-consoleauth as the storage backend for a cycle, if they for some reason wanted to | |
| 20:41:48 | melwitt | initially, I thought maybe it was to support rolling upgrades, but I don't think storing new tokens in nova-consoleauth helps with that | |
| 20:46:11 | eandersson | Is compute_driver used by any other service then nova-compute? | |
| 20:46:51 | eandersson | Can you have nova-compute-ironic on the same box as nova-conductor, api etc? | |
| 20:56:38 | tssurya | mriedem: ack, I'll repropose.. | |
| 20:57:25 | openstackgerrit | Dan Smith proposed openstack/nova master: Make CELL_TIMEOUT a constant https://review.openstack.org/594570 | |
| 20:57:26 | openstackgerrit | Dan Smith proposed openstack/nova master: Stash the cell uuid on the context when targeting https://review.openstack.org/594571 | |
| 20:57:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Make RecordWrapper record RequestContext and expose cell_uuid https://review.openstack.org/594572 | |
| 20:57:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Clarify which context is used by do_query() https://review.openstack.org/595250 | |
| 20:57:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | |
| 20:57:30 | openstackgerrit | Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717 | |
| 20:57:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131 | |
| 20:57:32 | openstackgerrit | Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265 | |
| 20:57:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577 | |
| 21:00:03 | melwitt | eandersson: no, compute_driver shouldn't be used for any other service (though I see one check for it in the novnc proxy since the xen driver doesn't support a new enough novnc version, IIRC). and AFAIK, you could run nova-compute-ironic on the same box as nova-conductor, etc, but I'm not sure it's recommended. maybe TheJulia can advise | |
| 21:00:40 | eandersson | I see - we figured as much, and didn't see any issues | |
| 21:00:55 | TheJulia | best to keep it to where you run ironic-conductor or ironic-api anyhow | |
| 21:01:20 | TheJulia | minimize latency between connections and also keep access to logs relatively close if they are needed | |
| 21:01:36 | mriedem | the api looks at compute_driver in one case for bdm device name | |
| 21:01:37 | mriedem | i think | |
| 21:01:38 | TheJulia | But that is just my take for sanity :) | |
| 21:01:52 | mriedem | if you're using xen | |
| 21:02:05 | melwitt | ok, I didn't see that in my grepping | |
| 21:04:24 | mriedem | eandersson: http://codesearch.openstack.org/?q=is_xenapi&i=nope&files=&repos= | |
| 21:04:37 | mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/cmd/novncproxy.py#n43 | |
| 21:10:20 | eandersson | Thanks! | |
| 21:41:06 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | |
| 22:53:55 | openstackgerrit | Merged openstack/nova master: Filter out instances without a host when populating AZ https://review.openstack.org/594050 | |
| 22:54:02 | openstackgerrit | Merged openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | |
| #openstack-nova - 2018-08-23 | |||
| 00:04:12 | openstackgerrit | Sam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558 | |
| 00:36:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | |
| 00:47:20 | mriedem | anyone, off the top of your head, does shelve of a volume-backed server create a snapshot of the root volume like createImage will? | |
| 00:48:50 | mriedem | oh we don't even snapshot for volume-backed | |
| 00:48:53 | mriedem | we just offload | |
| 00:49:46 | mriedem | well hells to the yeah | |
| 00:49:52 | mriedem | something went right with shelve for a change | |
| 01:12:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | |
| 01:16:58 | Kevin_Zheng | :) | |
| 02:37:37 | openstackgerrit | zhufl proposed openstack/nova master: Blacklist test_create_server_with_tags for nova-cells-v1 https://review.openstack.org/595399 | |
| 03:24:15 | openstackgerrit | Chen proposed openstack/nova master: Normalize dashless 'resource provider create' uuid https://review.openstack.org/567191 | |
| 04:12:18 | openstackgerrit | Chen proposed openstack/nova master: Fix create_resource_provider docstring https://review.openstack.org/595453 | |
| 04:57:33 | openstackgerrit | melanie witt proposed openstack/nova master: Correct the release notes related to nova-consoleauth https://review.openstack.org/595455 | |
| 06:50:49 | openstackgerrit | zhufl proposed openstack/nova master: Blacklist test_create_server_with_tags for nova-cells-v1 https://review.openstack.org/595399 | |
| 07:41:18 | zigo | http://paste.openstack.org/show/728659/ <--- Excuse my french but ... WTF ?!? | |
| 08:25:29 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Only lock when race condition happens in context.set_target_cell https://review.openstack.org/595530 | |
| 09:21:02 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Only lock when race condition happens in context.set_target_cell https://review.openstack.org/595530 | |
| 09:39:44 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add functional test to verify presence of policy https://review.openstack.org/595559 | |
| 11:33:14 | openstackgerrit | Moshe Levi proposed openstack/nova master: libvirt: set vfio driver in interface hostdev https://review.openstack.org/595592 | |
| 11:34:23 | mriedem | ahoy mateys! | |
| 11:42:05 | sean-k-mooney | mriedem: so i have not got a chance to test your patches for the livemigration issue yet(ill do that today) but i have noticed one other issue. | |
| 11:42:27 | sean-k-mooney | its not so much a bug just something unfortunate | |
| 11:44:01 | sean-k-mooney | mriedem: when hybrid_plug=false and we are not useing dpdk we hit https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L216-L223 | |
| 11:44:37 | sean-k-mooney | in this config os-vif only creates the bridge and the actual pluggin is done by the hypervior e.g. libvirt | |
| 11:44:55 | sean-k-mooney | so we dont get the speed up on live migrate | |
| 11:47:16 | mriedem | what speed up on live migrate? | |
| 11:47:17 | sean-k-mooney | i know how to fix this in two way and ill start working on it after i test your changes but ya. i noticed up to a 20 second network outage when using kernel ovs with ovs contrac firewall driver but none when using ip tables as a result. | |
| 11:47:58 | sean-k-mooney | ya mriedem effectivly the plug does not happen untill libvirt creates the vm on the dest. | |
| 11:48:04 | mriedem | ok | |
| 11:48:08 | mriedem | i'll take your word for it :) | |
| 11:48:14 | jaypipes | morning supernovas | |