| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-14 | |||
| 07:45:23 | openstackgerrit | Merged openstack/nova master: Literalize CLI options in docs https://review.opendev.org/665217 | |
| 07:45:32 | openstackgerrit | Merged openstack/nova master: Fix a warning about flags in an expression string https://review.opendev.org/665065 | |
| 08:13:55 | mrch_ | is there redhat production support for? https://www.suse.com/documentation/sles-15/singlehtml/art.amd-sev/art.amd-sev.html | |
| 08:31:17 | aspiers | mrch_: no. I'm working on it | |
| 08:31:38 | aspiers | mrch_: I mean, I'm working on adding SEV support to OpenStack | |
| 08:31:59 | aspiers | not sure if anyone from Red Hat is working on SEV support in the OS | |
| 08:32:07 | openstackgerrit | Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912 | |
| 08:32:07 | openstackgerrit | Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765 | |
| 08:32:16 | aspiers | but the RH folks are being very helpful with my work | |
| 08:32:20 | aspiers | mrch_: see http://specs.openstack.org/openstack/nova-specs/specs/train/approved/amd-sev-libvirt-support.html | |
| 08:32:41 | aspiers | mrch_: what's the nature of your interest? | |
| 08:35:41 | mrch_ | aspiers: thx | |
| 12:25:29 | openstackgerrit | sean mooney proposed openstack/nova-specs master: add libvirt pqos spec https://review.opendev.org/662264 | |
| 13:48:35 | dansmith | lyarwood: did you see mriedem's query about "q35" on the master patch for the sata bus thing? | |
| 14:11:58 | artom | dansmith, lyarwood at a medial appointment, but we've volunteered kashyap to follow up on that just now during our internal bug call :) | |
| 14:13:06 | artom | dansmith, while I have you, is looking at https://review.opendev.org/#/c/644881/ again anywhere near the top of your review list? | |
| 14:14:17 | dansmith | artom: mriedem has been shitting on my reviews all week, so I'm going to wait for him to +2 that | |
| 14:14:21 | kashyap | artom: Got a link to the backport patch? | |
| 14:14:33 | dansmith | kashyap: the backport isn't in question, it's the merged master one | |
| 14:14:52 | dansmith | kashyap: https://review.opendev.org/#/c/663011/ | |
| 14:15:06 | artom | Jinx! | |
| 14:15:28 | artom | dansmith, :( for mriedem. I may have been too pestery with him yesterday | |
| 14:17:27 | kashyap | dansmith: Matt answered the casing question, just saw that :) | |
| 14:18:13 | dansmith | kashyap: oh sorry, hadn't refreshed my tab | |
| 14:18:40 | kashyap | dansmith: No-no, but still it's a good question, the answer is not "complete" | |
| 14:18:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework all things metadata'y https://review.opendev.org/640730 | |
| 14:18:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename 'configuration drive' to 'config drive' https://review.opendev.org/641723 | |
| 14:18:56 | dansmith | I'm guessing that if someone puts Q35 we'll just break, but earlier because we pass that to qemu? | |
| 14:19:35 | kashyap | dansmith: Yes, that's what I'd expect; I wonder if there's equivalent of 'ignore_case' for image properties | |
| 14:20:03 | dansmith | kashyap: well, it's all in the handling of it really | |
| 14:20:13 | kashyap | For now, we're not in the ditch: as Matt noted, libvirt itself reports (in its capabilities) it in lower case | |
| 14:20:48 | dansmith | yeah, the patch doesn't make it worse, which is all that matters for the backport, that's fine | |
| 14:23:55 | kashyap | And we can rely on "instinctive laziness" of most folks, in terms of casing: people tend to "default" to lower case, as it comes "naturally" :D | |
| 14:24:13 | kashyap | s/stole/stone/ | |
| 14:42:20 | openstackgerrit | sean mooney proposed openstack/nova master: extend libvirt video model support https://review.opendev.org/647733 | |
| 14:47:37 | dansmith | temka: I'm having a hard time with instance.get_network_info().get_bind_time_events() | |
| 14:49:55 | temka | dansmith, how so? | |
| 14:50:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document how to revert, confirm a cold migration https://review.opendev.org/664396 | |
| 14:50:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Update resize doc https://review.opendev.org/665412 | |
| 14:50:43 | temka | ... could be None, I suppose | |
| 14:52:01 | dansmith | temka: commenting with some analysis | |
| 14:52:06 | temka | dansmith, much thanks :) | |
| 14:52:20 | cdent | efried: you aware of https://bugs.launchpad.net/nova/+bug/1832164 and https://review.opendev.org/#/c/664193/ . it's blocking placement because of subunit | |
| 14:52:22 | openstack | Launchpad bug 1832164 in OpenStack Compute (nova) "SADeprecationWarning: The joinedload_all() function is deprecated, and will be removed in a future release. Please use method chaining with joinedload() instead" [Undecided,In progress] - Assigned to Takashi NATSUME (natsume-takashi) | |
| 14:58:06 | kashyap | finucannot: If you're respinning https://review.opendev.org/#/c/664396/2, a small grammar thinko in there. | |
| 14:59:17 | finucannot | good catch | |
| 15:00:13 | kashyap | finucannot: Otherwise, looks good. Thanks for documenting. | |
| 15:00:46 | finucannot | done | |
| 15:00:48 | kashyap | finucannot: Speaking of docs, ... you might be horrified to know, that some of the PCIe-related command-line options in QEMU are not even documented :-( | |
| 15:00:52 | kashyap | I asked them here: | |
| 15:01:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document how to revert, confirm a cold migration https://review.opendev.org/664396 | |
| 15:01:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Update resize doc https://review.opendev.org/665412 | |
| 15:01:08 | kashyap | https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg01173.html | |
| 15:01:26 | kashyap | [Err, main e-mail here: https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg01093.html] | |
| 15:01:29 | sean-k-mooney | kashyap: that is not that surprising | |
| 15:02:02 | kashyap | I'll get them documented in the official docs. I find it unacceptable that valid options are hidden like that | |
| 15:02:20 | kashyap | sean-k-mooney: They're getting better, though. From my observation. Still a long way to go. | |
| 15:02:47 | kashyap | One of the thorny "problems with documentation" is that misaligned incentives. | |
| 15:02:58 | sean-k-mooney | kashyap: ya i just am not surpirsed then the more obsucre/less used flags of complex features are not docuemtned | |
| 15:03:20 | sean-k-mooney | somethimgs its because the interfaces were experimental and they did not want to commit to it | |
| 15:03:46 | sean-k-mooney | other times it just slips through the cracks | |
| 15:04:41 | kashyap | In this case, it's officially supported, and not experimental (as you can see from the response). It was the all-too-common: "slipping through the cracks" | |
| 15:05:48 | sean-k-mooney | one of the nice things about having docs in tree is that you can require docs with each change that add a new user faceing change | |
| 15:06:06 | sean-k-mooney | or wehre you have docs generated form code | |
| 15:06:30 | efried | cdent: Will look. Been kind of avoiding because out of my realm, but I can ride on your vote with clear conscience | |
| 15:07:10 | cdent | I think in this case you can rely on the tests, not me | |
| 15:07:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove references to nova-consoleauth https://review.opendev.org/652965 | |
| 15:07:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Stop starting consoleauth in functional tests https://review.opendev.org/652966 | |
| 15:07:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: xvp: Start using consoleauth tokens https://review.opendev.org/652967 | |
| 15:07:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-status: Remove consoleauth workaround check https://review.opendev.org/652968 | |
| 15:07:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova-consoleauth https://review.opendev.org/652969 | |
| 15:07:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove ConsoleAuthToken.to_dict https://review.opendev.org/652970 | |
| 15:07:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework nova console diagram https://review.opendev.org/660147 | |
| 15:13:18 | efried | cdent: That wasn't so scary, found a page that explains it clearly. dansmith, would you please send https://review.opendev.org/#/c/664193/ ? | |
| 15:14:07 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 15:14:07 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431 | |
| 15:14:08 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442 | |
| 15:14:20 | dansmith | efried: link me to the page? because it looks scary on the surface | |
| 15:14:34 | efried | dansmith: https://docs.sqlalchemy.org/en/13/changelog/migration_09.html#new-query-options-api-load-only-option | |
| 15:16:06 | dansmith | efried: that page says that the old methods will be supported "indefinitely" | |
| 15:16:31 | cdent | they spit warnings | |
| 15:16:36 | cdent | lots and lots and lots of warnings | |
| 15:17:09 | dansmith | oh, hmm, that's annoying given the statement | |
| 15:18:28 | dansmith | cdent: your suggestion would be a lot tidier | |
| 15:18:43 | dansmith | cdent: any chance you'd just do that real quick so I and efried can approve _that_ ? | |
| 15:19:02 | dansmith | should just be a few lines of change it looks like | |
| 15:19:04 | cdent | yeah, can do | |
| 15:19:07 | cdent | one mo | |
| 15:28:04 | efried | cdent, dansmith: tbh I would prefer going the other way | |
| 15:28:17 | efried | but whatever | |
| 15:28:20 | dansmith | efried: that's not as easy | |
| 15:28:27 | cdent | i'm basically done | |
| 15:28:27 | dansmith | because we calculate column names in places | |
| 15:28:30 | cdent | yes, that | |
| 15:29:32 | efried | right, I get that, so I'm fine with it. But I actually prefer the explicitness of takashi's patch. But I'm fine either way. | |
| 15:30:14 | cdent | the new diff is way tidier | |
| 15:30:16 | dansmith | joinedload, joinedload_all, joined, etc has brought plenty of confusion in the past, so more consistency for people to copy seems like a win to me | |
| 15:30:56 | dansmith | also, cdent and I are agreeing which means (1) jump on that shit and (2) go buy some lottery tickets | |
| 15:31:08 | cdent | ✔ | |
| 15:32:32 | openstackgerrit | Chris Dent proposed openstack/nova master: Replace joinedload_all with joinedload https://review.opendev.org/664193 | |