| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-07 | |||
| 12:48:33 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Store and allow libvirt instance device buses and models to be updated https://review.opendev.org/c/openstack/nova-specs/+/799811 | |
| 12:56:11 | lyarwood | Does anyone know of an API where we change the response for admins based on a microversion? | |
| 12:56:34 | lyarwood | Oh events, instance faults etc me-- | |
| 13:07:21 | ralonsoh | hi folks, do you know why if I have two compute node hosts (opentack host list) | |
| 13:07:32 | ralonsoh | I can't map the second one to cell1? | |
| 13:07:47 | ralonsoh | "nova-manage cell_v2 discover_hosts", then "nova-manage cell_v2 list_hosts" | |
| 13:07:52 | ralonsoh | +-----------+--------------------------------------+----------+ | |
| 13:07:53 | ralonsoh | +-----------+--------------------------------------+----------+ | |
| 13:07:53 | ralonsoh | | cell1 | db04030a-5cca-439d-b1c8-df1aa8d53d4f | u20ovn | | |
| 13:07:53 | ralonsoh | | Cell Name | Cell UUID | Hostname | | |
| 13:08:03 | ralonsoh | one is missing | |
| 13:09:10 | sean-k-mooney | you need to run discover hosts | |
| 13:09:20 | sean-k-mooney | nova-manage cellv2 discover_hosts | |
| 13:09:24 | ralonsoh | I did | |
| 13:09:44 | sean-k-mooney | from the controler | |
| 13:09:47 | ralonsoh | yes | |
| 13:10:04 | sean-k-mooney | can you run that with -v or --debug and past the output somewhere | |
| 13:10:09 | ralonsoh | let me check | |
| 13:11:26 | ralonsoh | sean-k-mooney, nothing, no output | |
| 13:12:06 | sean-k-mooney | does it show up in openstack hyperviors list | |
| 13:12:20 | ralonsoh | yes | |
| 13:12:30 | sean-k-mooney | the os-host api is deprecated by the way so im not sure it works proerly | |
| 13:12:35 | ralonsoh | | u20ovn | conductor | internal | | |
| 13:12:35 | ralonsoh | | u20ovn | scheduler | internal | | |
| 13:12:35 | ralonsoh | +-----------+-----------+----------+ | |
| 13:12:35 | ralonsoh | | Host Name | Service | Zone | | |
| 13:12:35 | ralonsoh | stack@u20ovn:/opt/stack$ openstack host list | |
| 13:12:37 | ralonsoh | | u20ovn | conductor | internal | | |
| 13:12:39 | ralonsoh | | u20ovn | compute | nova | | |
| 13:12:41 | ralonsoh | | u20ovn2 | compute | nova | | |
| 13:12:43 | ralonsoh | +-----------+-----------+----------+ | |
| 13:13:00 | sean-k-mooney | very odd indeed | |
| 13:13:37 | ralonsoh | do I need any service on the second node? | |
| 13:13:44 | ralonsoh | devstack@n-cond-cell1.service | |
| 13:13:47 | stephenfin | bauzas: Trivial bugfix here if you have a moment? https://review.opendev.org/c/openstack/nova/+/794006 | |
| 13:13:57 | bauzas | stephenfin: I can do | |
| 13:14:29 | stephenfin | ta | |
| 13:14:59 | bauzas | stephenfin: same thought than gmann | |
| 13:15:10 | stephenfin | oh, missed that comment | |
| 13:15:16 | bauzas | wow, takes a lot of time to update | |
| 13:28:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Handle invalid volume UUIDs during spawn https://review.opendev.org/c/openstack/nova/+/794006 | |
| 13:28:34 | stephenfin | bauzas: ^ | |
| 13:28:45 | bauzas | ta, looking | |
| 13:28:50 | stephenfin | gmann, lyarwood: either of you care to re-review also. Super trivial ^ | |
| 13:29:18 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Replace 'nova boot' with 'openstack server create' https://review.opendev.org/c/openstack/nova/+/794007 | |
| 13:29:28 | stephenfin | lyarwood: I also addressed your comments on that doc | |
| 13:48:01 | lyarwood | ack looking | |
| 14:00:33 | lyarwood | stephenfin: I'm sure you still need --bootable when creating the volume | |
| 14:01:12 | stephenfin | Possibly in the past, but I tested it locally with a basic DevStack deployment and you didn't | |
| 14:02:02 | lyarwood | I've just copy pasted from my env, it's failing for me | |
| 14:03:27 | lyarwood | unless cinder is somehow automatically setting this under a later microversion | |
| 14:03:37 | lyarwood | or we are somehow updating the bootable flag | |
| 14:03:43 | lyarwood | I can't see how this ever worked | |
| 14:04:32 | stephenfin | What I was seeing was that the bootable flag would be False in the response returned from the create request, but then True when I did a show call | |
| 14:04:44 | stephenfin | so cinder was changing things under the good | |
| 14:04:47 | stephenfin | *hood | |
| 14:05:15 | stephenfin | I'm pretty sure I traced through cinder to find what was setting it too, but it seems I didn't take notes on the patch like I usually would | |
| 14:05:42 | lyarwood | weird | |
| 14:07:49 | lyarwood | oh wait that's with the --bootable flag | |
| 14:08:02 | lyarwood | haha nice, I bet the osc command does it in two calls or something weird | |
| 14:08:40 | stephenfin | wdym? | |
| 14:10:12 | lyarwood | https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=create-a-volume-detail#volumes-volumes - because c-api doesn't take bootable as an arg when creating the volume | |
| 14:12:34 | lyarwood | https://github.com/openstack/python-openstackclient/blob/4f358367986525625157bee164c694e65573c30c/openstackclient/volume/v2/volume.py#L231-L232 and yeah osc creates and then optionally marks it as bootable if you pass the flag | |
| 14:13:48 | lyarwood | https://github.com/openstack/nova/blob/dfda0c0482acf579452742860cec2bdd64ac616a/nova/block_device.py#L685-L686 is where n-api rejects it btw | |
| 14:21:22 | stephenfin | lyarwood: I'm still seeing the same behavior http://paste.openstack.org/show/807231/ | |
| 14:22:12 | stephenfin | So if I create a volume from an image then the initial response from cinder is bootable='false', but this changes to bootable='true' once the volume is fully created | |
| 14:22:30 | lyarwood | yeah osc must be doing that | |
| 14:22:38 | lyarwood | that makes sense if you've passed --image | |
| 14:23:38 | stephenfin | Not that I can see | |
| 14:23:46 | stephenfin | and python-cinderclient isn't doing anything too funky either | |
| 14:23:53 | lyarwood | yeah weird, maybe c-api then, TIL | |
| 14:24:06 | lyarwood | and I wasn't passing --image so that's why I didn't see it | |
| 14:24:24 | stephenfin | ah, yes, I explicitly called that fact out https://review.opendev.org/c/openstack/nova/+/794007/4/doc/source/user/launch-instance-from-volume.rst#265 | |
| 14:24:29 | stephenfin | Cinder makes a volume bootable when ``--image`` parameter is passed. | |
| 14:26:41 | lyarwood | Yup apologies | |
| 14:28:52 | opendevreview | Jay Faulkner proposed openstack/nova stable/wallaby: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/799772 | |
| 14:34:53 | gmann | stephenfin: done, thanks for adding test. | |
| 14:36:36 | stephenfin | ta | |
| 15:22:33 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Add connection_info to responses of the os-volume_attachments API for admins https://review.opendev.org/c/openstack/nova-specs/+/799850 | |
| 15:44:29 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Add connection_info to responses of the os-volume_attachments API for admins https://review.opendev.org/c/openstack/nova-specs/+/799850 | |
| 15:48:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 15:48:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect extended_resource_request neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 15:50:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 15:50:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 15:52:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 15:55:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 15:57:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 15:57:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 15:59:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 16:01:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 16:02:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 16:03:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 16:05:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 16:15:10 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Add nova-manage commands to show and refresh connection_info https://review.opendev.org/c/openstack/nova-specs/+/799624 | |
| 16:45:42 | stephenfin | zzzeek: If you've any spare time whatsoever this week, I'd appreciate your attention on an issue I'm having during the sqla-migrate -> alembic switch. Basically, my migration code to create shadow tables doesn't seem to be working with in-memory SQLite databases, but I can't figure out how to fix it | |
| 16:45:58 | stephenfin | zzzeek: I have a pretty minimal reproducer here https://github.com/stephenfin/alembic-sqlite-memory-issue | |
| 16:46:28 | stephenfin | zzzeek: and the series to do the conversion for nova ends here, fwiw https://review.opendev.org/c/openstack/nova/+/799530/ | |
| 16:47:16 | stephenfin | zzzeek: I _think_ I'm seeing an issue like the one in the cookbook, but I don't understand the issue well enough yet to say /o\ Any input would be much appreciated https://alembic.sqlalchemy.org/en/latest/cookbook.html#sharing-a-connection-with-a-series-of-migration-commands-and-environments | |
| 17:06:55 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Store and allow libvirt instance device buses and models to be updated https://review.opendev.org/c/openstack/nova-specs/+/799811 | |
| 17:07:54 | lyarwood | stephenfin: hope you're happy, I pretty much always forget to manually run pep8 in projects outside of Nova now thanks to pre-commit ;) | |