Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-06-14
00:51:04 igordc sean-k-mooney, since you're here, can I ask if you had a look at https://review.opendev.org/#/c/576607/ after Stamatis last comment?
00:51:18 igordc cc stephenfin
00:51:31 sean-k-mooney i have not
00:51:42 izza_ ok noted..i already sent message to rally channel...yet no answer..just looking for a chance..if anyone knows how to resolv that issue..but thanks:)
02:22:13 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.74: Support Specifying AZ to unshelve https://review.opendev.org/665136
02:56:48 openstackgerrit Merged openstack/nova stable/rocky: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/662584
04:00:22 openstackgerrit zhufl proposed openstack/nova master: Fix :param: in docstring https://review.opendev.org/665289
05:27:48 openstackgerrit Boxiang Zhu proposed openstack/nova master: Validate requested host/node during servers create https://review.opendev.org/661237
05:27:49 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
06:33:17 openstackgerrit Merged openstack/nova master: fup: Merge machine_type_mappings into get_default_machine_type https://review.opendev.org/664036
07:45:16 openstackgerrit Merged openstack/nova master: Warn for duplicate host mappings during discover_hosts https://review.opendev.org/651947
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 https://review.opendev.org/627765
08:32:07 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912
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: conf: Rename 'configuration drive' to 'config drive' https://review.opendev.org/641723
14:18:54 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rework all things metadata'y https://review.opendev.org/640730
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: Update resize doc https://review.opendev.org/665412
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: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: Update resize doc https://review.opendev.org/665412
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: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: tests: Stop starting consoleauth in functional tests https://review.opendev.org/652966
15:07:30 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove references to nova-consoleauth https://review.opendev.org/652965
15:07:31 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-status: Remove consoleauth workaround check https://review.opendev.org/652968
15:07:31 openstackgerrit Stephen Finucane proposed openstack/nova master: xvp: Start using consoleauth tokens https://review.opendev.org/652967
15:07:32 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove ConsoleAuthToken.to_dict https://review.opendev.org/652970
15:07:32 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova-consoleauth https://review.opendev.org/652969
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: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431
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: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

Earlier   Later