| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-22 | |||
| 16:38:24 | mriedem | dansmith: are you also going to do ^ for pike? | |
| 16:38:33 | mriedem | because https://review.openstack.org/#/c/591762/ | |
| 16:38:51 | dansmith | I had assumed we would yeah | |
| 16:43:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | |
| 16:43:50 | openstackgerrit | Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717 | |
| 16:43:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131 | |
| 16:43:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265 | |
| 16:43:53 | openstackgerrit | Dan Smith proposed openstack/nova master: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577 | |
| 16:43:54 | openstackgerrit | Dan Smith proposed openstack/nova master: Clarify which context is used by do_query() https://review.openstack.org/595250 | |
| 16:45:30 | dansmith | mriedem: FYI the last patch removes an extra db hit we do in the listing process which was a fixme from the original work | |
| 16:45:59 | dansmith | probably small and not super impactful, but..fewer is better | |
| 17:16:36 | openstackgerrit | Merged openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548 | |
| 17:33:22 | imacdonn | Could someone help me understand (or point me to a good description of) the deprecation of nova-consoleauth? From the way it's described, it sounds like you can just not run the service, but when I do that 'nova get-vnc-console ... novnc' gets an "unknown error" (HTTP 504) | |
| 17:50:29 | melwitt | imacdonn: sec, looking for docs link | |
| 17:55:00 | melwitt | imacdonn: https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#consoleauth-service-and-console-proxies and the release notes https://docs.openstack.org/releasenotes/nova/rocky.html#relnotes-18-0-0-0b2-stable-rocky-upgrade-notes | |
| 17:55:29 | imacdonn | melwitt: OK... will review - I just found https://review.openstack.org/#/c/557489/ too | |
| 17:56:55 | imacdonn | Somehow I missed that upgrade release notes.... maybe was looking at the wrong version of release notes .. thans | |
| 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 | melwitt | https://bugs.launchpad.net/nova/+bug/1788470 | |
| 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: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 | |