Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-13
13:03:23 stephenfin good way to get used to having working internet again
13:12:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove functional test specific nova code https://review.opendev.org/683609
13:31:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove functional test specific nova code https://review.opendev.org/683609
13:40:55 efried stephenfin: alex_xu already has to deal with Intel's firewall, the gfwoc is nbd.
13:41:20 efried I work around it by using a personal computer for real work.
13:41:21 bauzas I wonder whether we should propose some Chinese IRC server
13:41:30 efried bauzas: isn't that called WeChat?
13:41:42 bauzas efried: sure, but WeChat doesn't support IRC
13:41:47 efried oh, you mean a server for actual IRC, got it.
13:41:50 bauzas having the same client would be nice
13:42:06 efried I'm not sure the technology is the problem; it's the connectivity.
13:42:07 bauzas like, I could use this Chinese server plus the Freenode one
13:42:27 efried isn't the problem that the bits can't get in & out?
13:42:31 bauzas of course, but I'm pretty sure a IRC server would be supported by the chinese government :)
13:42:59 bauzas I mean, when running in a chinese cloud ;)
13:43:45 bauzas alex_xu: FWIW, I'll still continue to use WeChat
13:54:04 kashyap efried: Hi, when you get a moment, is a Blueprint required for this: https://bugs.launchpad.net/nova/+bug/1852437 (Allow ability to disable individual CPU features via `cpu_model_extra_flags`)
13:54:04 openstack Launchpad bug 1852437 in OpenStack Compute (nova) "Allow ability to disable individual CPU features via `cpu_model_extra_flags`" [Undecided,New]
13:55:28 efried kashyap: at a glance, a blueprint seems entirely necessary. A spec, not as sure. Perhaps put it on tomorrow's meeting agenda for discussion?
13:56:00 kashyap efried: Spec / BP seems like an overkill, IMHO. But sure, can discuss tomm
13:56:42 efried we're talking about enhancing syntax of a conf option in what seems like a nontrivial way.
13:56:46 kashyap It is essentially aims to implement what I stated at the end of this commit in paranthesis: https://opendev.org/openstack/nova/commit/cc27a2007f314
13:56:57 kashyap efried: Hmm, phrased that way...
13:57:18 kashyap efried: But what do you think of it? Is there a better way you can think of than the +/- notion?
13:59:08 efried kashyap: no, that seems reasonable, just seems like a thing that ought to have a "design" somewhere written down that we can agree on. Just having it in a RFE bug might be sufficient, but really that's what blueprints are for. And it's the kind of thing I would not expect to backport.
14:01:36 kashyap efried: Sure, can file a simple BluePrint
14:01:59 efried thanks kashyap.
14:02:40 kashyap I first wanted to file it, but went the bug route, thinking it is a "simple idea". :D
14:09:02 kashyap Done: https://blueprints.launchpad.net/nova/+spec/allow-disabling-cpu-flags
14:19:26 kashyap mriedem: Maybe we can just close this, as we'll be tracking it in the Blueprint? - https://bugs.launchpad.net/nova/+bug/1852437
14:19:26 openstack Launchpad bug 1852437 in OpenStack Compute (nova) "Allow ability to disable individual CPU features via `cpu_model_extra_flags`" [Wishlist,New]
14:20:58 mriedem wfm
14:43:45 efried mriedem: what's the easiest/best way to "discover" a compute node's UUID (the one that'll match the placement root RP)?
14:44:41 efried (I'm trying to middleman here, not actually sure if we know anything about the node beforehand)
14:48:41 mriedem you mean to get the ComputeNode object?
14:49:00 mriedem for non-ironic nodes the compute node uuid is randomly generated when the record is created the first time
14:49:13 mriedem to look up the computenode record, you want to use the host/nodename
14:49:26 mriedem if you're on compute and it's not ironic, you can just use CONF.host
14:49:42 mriedem host == node for non-ironic
14:49:52 mriedem otherwise you get the nodenames from the driver
14:50:05 mriedem see ComputeManager.update_available_resource
14:50:47 mriedem or you're trying to do something outside of nova to try to expose a vulnerability?
14:51:09 bauzas efried: yeah, you need to know the (host, node) tuple
14:51:44 bauzas mriedem: and no, AFAICR, for some virt drivers, node != host
14:52:04 mriedem bauzas: yeah, ironic
14:52:05 mriedem i said that
14:52:13 bauzas mriedem: not only for ironic
14:52:17 mriedem which ones?
14:52:25 mriedem if you're thinking vcenter, you're thinking of kilo era
14:52:29 bauzas I don't remember, lemme look
14:53:29 efried I think I can work with this, thanks.
14:53:30 bauzas I do wonder for HyperV
14:53:41 mriedem hyperv has only 1 node per host
14:54:39 mriedem http://cloudbase-ci.com/nova/693937/1/windows/logs/n-h2-693937-1/nova-compute.log.gz
14:54:43 mriedem 2019-11-13 05:46:51.769 5012 103574784 GreenThread-1 INFO nova.compute.resource_tracker [req-320b7c3d-8917-4c50-b48e-18dd405c7877 - - - - -] Compute node record created for n-h2-693937-1:n-h2-693937-1 with uuid: 77e9e8d8-bba6-4aba-81f9-5e9f69ca1db9
14:55:15 mriedem ironic is the only weirdo
14:56:12 bauzas anyway, looks you're right
14:57:08 bauzas https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L9189
14:59:33 usr2033 Hi, can i pxe boot nova/kvm instances?
14:59:43 mriedem note that https://docs.openstack.org/nova/latest/admin/configuration/hypervisors.html doesn't have a subpage for ironic as a compute driver in nova - that's probably a decent sized gap given all of the edge cases we could describe with ironic in nova as a compute driver
14:59:47 openstackgerrit Merged openstack/python-novaclient master: Add minor version [21] to the test_versions https://review.opendev.org/688599
15:00:08 mriedem not to mention scaling issues when using the ironic driver to manage lots of nodes from a single compute service, re the ML threads on RT perf issues
15:02:10 mriedem usr2033: not natively no
15:02:13 mriedem see https://serverfault.com/questions/469479/does-nova-support-pxe-boot
15:03:38 mriedem usr2033: you may be interested in https://openstack-virtual-baremetal.readthedocs.io/en/latest/index.html
15:06:01 efried I may be choppy today, going to be trying to work remotely (like, more remotely than usual).
15:09:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'os-consoles' API https://review.opendev.org/687907
15:09:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova-console' service, 'os-consoles' API https://review.opendev.org/687908
15:09:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
15:15:06 openstackgerrit Merged openstack/nova master: Remove TODOs around claim_resources_on_destination https://review.opendev.org/693635
15:15:41 mriedem bauzas: https://bugs.launchpad.net/nova/+bug/1852446
15:15:41 openstack Launchpad bug 1852446 in OpenStack Compute (nova) "Hypervisors in nova - no subpage details for ironic" [Undecided,New]
15:16:09 bauzas ack, good point
15:18:07 stephenfin mriedem: It's not urgent, but if you can rebase https://review.opendev.org/#/c/693425/ today I'm happy to push it through
15:19:29 mriedem ack, let me rebase the entire cross-cell series first quick so i can destroy the gate
15:21:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Make API always RPC cast to conductor for resize/migrate https://review.opendev.org/693937
15:21:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224
15:21:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253
15:21:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591
15:21:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
15:21:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
15:21:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
15:21:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013
15:21:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058
15:21:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
15:21:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075
15:21:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
15:21:50 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
15:21:50 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453
15:21:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647
15:21:51 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.opendev.org/638046
15:21:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047
15:21:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
15:22:11 mriedem dansmith: gibi_off: https://review.opendev.org/#/c/693937/ is fixed now
15:22:15 mriedem py2 vs py3 weirdness
15:27:09 openstackgerrit Merged openstack/nova master: Remove now invalid TODO from ComputeManager._confirm_resize https://review.opendev.org/693427
15:27:21 openstackgerrit Merged openstack/nova master: Use ListOfUUIDField from oslo.versionedobjects https://review.opendev.org/693258
15:27:37 openstackgerrit Merged openstack/nova master: Add known limitation about resize not resizing ephemeral disks https://review.opendev.org/691915
15:27:46 openstackgerrit Merged openstack/nova master: api-ref: re-work resize action post-conditions https://review.opendev.org/691918
15:27:56 openstackgerrit Merged openstack/nova master: Provide a better error when _verify_response hits a TypeError https://review.opendev.org/693042

Earlier   Later