| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-10 | |||
| 10:07:10 | kashyap | Gave my paltry ACK on the Tempest change, with a note | |
| 10:37:06 | sean-k-mooney1 | gmann: gibi_pto ye might find https://review.opendev.org/c/openstack/keystonemiddleware/+/787822 interesting its the implementation of X-Project-Id passthru that allows you to use project ids with system scoped tokens | |
| 11:11:38 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Query the service group API within check_instance_host https://review.opendev.org/c/openstack/nova/+/802317 | |
| 14:38:07 | artom | Am I high? Why am I seeing api_models.py in Newton? I thought cells v2 was way after that... | |
| 14:43:07 | artom | I guess we'd already started by then... | |
| 14:44:25 | sean-k-mooney | i dont think that was new to cells v2 | |
| 14:45:03 | sean-k-mooney | even with v1 i think we had a seperate top level db that we then had sync tasks runing btween the cells | |
| 14:45:41 | sean-k-mooney | i try to block out cell_v1's existance form my mind as much as i can | |
| 14:57:45 | stephenfin | It was new to cells v2 | |
| 14:58:03 | stephenfin | we only turned it on in Pike, but there was ground work way before that | |
| 15:58:39 | gmann | sean-k-mooney: ack, thanks | |
| 18:49:48 | b1tsh1ft3r | Hey all, had a compute node hardware failure. Ive removed the compute node and re-added according to the docs here: https://docs.openstack.org/openstack-ansible/train/admin/scale-environment.html#add-compute-host | |
| 18:50:35 | b1tsh1ft3r | i'm having issues however being able to live migrate instances to the compute node. I'm not seeing any real errors in logs or the UI. It looks as if it works, but doesnt. | |
| 18:54:27 | sean-k-mooney | i would check that ssh is workign between the compute nodes and that the host name resolve correctly | |
| 18:55:02 | sean-k-mooney | you need passworad less ssh to work as the use nova is running as and yuou will need to have exchanged keys | |
| 18:55:41 | sean-k-mooney | assuming you are getting that far | |
| 18:56:24 | sean-k-mooney | b1tsh1ft3r: in general thougfh you might be better of asking the osa developers | |
| 19:02:15 | b1tsh1ft3r | it has nothing to do with ansible deployment of the new compute node itself. The plays ran through clean. its purely a nova issue or config of nova but i cant imagine what it might be since the config hasnt changed between this node and any of the other nodes when it was setup. | |
| 19:02:35 | b1tsh1ft3r | SSH access is open between both nodes without prompts | |
| 19:02:58 | sean-k-mooney | b1tsh1ft3r: just becasue the play ran thoguh clean does not mean the plays were correct | |
| 19:03:45 | b1tsh1ft3r | Yes but the config hasn't changed. | |
| 19:04:01 | sean-k-mooney | have you used this procdeure to add other nodes | |
| 19:09:38 | b1tsh1ft3r | yes, its worked fine. my only guess here is something is left over residual somewhere from removing the node, but its probably burried in db in galera somewhere is my only real guess. | |
| 19:10:15 | sean-k-mooney | when you added the new node did you give it a new host name | |
| 19:10:33 | sean-k-mooney | its generally not a good idea to reuse the hostname | |
| 19:13:08 | b1tsh1ft3r | used same node name, so my guess is thats probably the issue here. i guess i can change it, but... since its a replacement for a hardware failure i wanted to keep the same name ideally | |
| 19:13:54 | sean-k-mooney | you can keep the same name but there are extra steps you ahve to take and osa is know to not do the replacement properly | |
| 19:14:27 | sean-k-mooney | b1tsh1ft3r: if you are ligitimatly replaceing the node not adding one its important to ensure that you do not delete the compute sesrvice record for the node for example | |
| 19:15:06 | sean-k-mooney | that way it will keep the same service uuid and placement resouce provider | |
| 19:15:43 | sean-k-mooney | if you delete the compute serivce record the service uuid will change and cause a conflict in placment when trying to create a resouce provier with a different uuid but same host name | |
| 23:05:40 | opendevreview | melanie witt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/c/openstack/nova/+/712137 | |
| 23:05:41 | opendevreview | melanie witt proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/c/openstack/nova/+/712140 | |
| 23:05:41 | opendevreview | melanie witt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/c/openstack/nova/+/712141 | |
| 23:05:42 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 23:05:42 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 23:05:43 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 23:05:43 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 23:05:44 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 23:05:45 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 23:05:45 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 23:05:47 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 23:05:47 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 23:05:49 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 23:21:03 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| #openstack-nova - 2021-08-11 | |||
| 00:52:02 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs https://review.opendev.org/c/openstack/nova/+/764292 | |
| 06:21:35 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 06:21:35 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 06:21:36 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 06:21:36 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 06:21:37 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 06:52:28 | brinzhang | hi stephenfin, gibi: do you know which patch removed the tags from fake_compute_get_all(), https://review.opendev.org/c/openstack/nova/+/764292/29/nova/tests/unit/api/openstack/fakes.py#422 | |
| 08:11:44 | opendevreview | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs https://review.opendev.org/c/openstack/nova/+/764292 | |
| 11:21:04 | opendevreview | likui proposed openstack/osc-placement stable/train: Update command help information https://review.opendev.org/c/openstack/osc-placement/+/804164 | |
| 11:22:36 | opendevreview | likui proposed openstack/osc-placement stable/ussuri: Update command help information https://review.opendev.org/c/openstack/osc-placement/+/804165 | |
| 11:25:15 | opendevreview | likui proposed openstack/osc-placement stable/stein: Update command help information https://review.opendev.org/c/openstack/osc-placement/+/804166 | |
| 12:19:25 | opendevreview | Lee Yarwood proposed openstack/nova master: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/804229 | |
| 12:19:25 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/804230 | |
| 12:19:26 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move OSBrickFixture into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804231 | |
| 12:19:26 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move _migrate_stub into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804232 | |
| 12:19:27 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Mock all unix connectors in os-brick fixture https://review.opendev.org/c/openstack/nova/+/804233 | |
| 14:15:56 | lyarwood | hmm I'm quickly trying to spin a new microversion change to add attachment_id to os-volume_attachments | |
| 14:16:10 | lyarwood | shouldn't tox -e api_samples auto generate the samples for me? | |
| 14:16:31 | lyarwood | it's failing with FileNotFoundError for the samples | |
| 14:16:45 | lyarwood | or do I still need to define the tpl files? | |
| 14:19:42 | lyarwood | well that would make sense wouldn't it me-- | |
| 14:19:56 | lyarwood | my bad, I do this so little I always forget | |
| 15:56:37 | prometheanfire | looks like nova is still failing on fasteners 0.16.3 https://review.opendev.org/804246 | |
| 16:17:58 | opendevreview | Lee Yarwood proposed openstack/nova master: api: 2.89 adding attachment_id to os-volume_attachments responses https://review.opendev.org/c/openstack/nova/+/804275 | |
| 16:28:10 | lyarwood | https://paste.opendev.org/show/808016/ - fun | |
| 16:28:45 | lyarwood | stephenfin: think I'll try to finish your mypy stuff in the block device layers this cycle assuming you're not going ahead with it | |
| 16:29:03 | stephenfin | knock yourself out. Happy to review it if you do polish it | |
| 16:43:40 | spatel | any idea why disk_cachemodes = network=writeback doesn't working with wallaby release? | |
| 16:44:33 | spatel | looking at here - https://bugs.launchpad.net/nova/+bug/1727558 | |
| 16:59:01 | opendevreview | Merged openstack/nova master: tox: Encode specific Python versions https://review.opendev.org/c/openstack/nova/+/797054 | |
| 17:30:08 | sean-k-mooney | stephenfin: why ^ | |
| 17:30:36 | sean-k-mooney | i at least really dont want the python version encoded there | |
| 17:31:01 | sean-k-mooney | because i may or may not have python 3.8 and 3.6 installed on all my dev systems | |
| 17:34:33 | opendevreview | sean mooney proposed openstack/nova master: Revert "tox: Encode specific Python versions" https://review.opendev.org/c/openstack/nova/+/804168 | |
| 17:42:41 | sean-k-mooney | gibi_pto: stephenfin i have proposed a revert for the tox version pinning i would like to land that now and then we can discuss this properly per our fast revert policy | |
| 17:42:57 | opendevreview | Lee Yarwood proposed openstack/nova master: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/804229 | |
| 17:42:58 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/804230 | |
| 17:42:58 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move OSBrickFixture into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804231 | |
| 17:42:59 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move _migrate_stub into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804232 | |
| 17:42:59 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Mock all unix connectors in os-brick fixture https://review.opendev.org/c/openstack/nova/+/804233 | |
| 17:43:00 | opendevreview | Lee Yarwood proposed openstack/nova master: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/804285 | |
| 17:43:00 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP fup: Remove unused legacy block_device_info format https://review.opendev.org/c/openstack/nova/+/804286 | |
| 17:46:31 | opendevreview | sean mooney proposed openstack/nova master: Revert "tox: Encode specific Python versions" https://review.opendev.org/c/openstack/nova/+/804168 | |
| 18:52:07 | opendevreview | sean mooney proposed openstack/nova master: this change replaces all hardcoded tox enve with generative envs https://review.opendev.org/c/openstack/nova/+/804292 | |
| 18:52:58 | opendevreview | Lee Yarwood proposed openstack/nova master: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/804285 | |
| 18:52:58 | opendevreview | Lee Yarwood proposed openstack/nova master: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/804229 | |
| 18:52:59 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/804230 | |
| 18:52:59 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move OSBrickFixture into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804231 | |
| 18:53:00 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move _migrate_stub into base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/804232 | |
| 18:53:00 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Mock all unix connectors in os-brick fixture https://review.opendev.org/c/openstack/nova/+/804233 | |
| 18:53:01 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP fup: Remove unused legacy block_device_info format https://review.opendev.org/c/openstack/nova/+/804286 | |
| 18:54:36 | lyarwood | artom: ^ should be good now btw, had some fun with the legacy block_device_info cruft but that should be sorted now | |
| 18:57:02 | artom | lyarwood, ack, I can remove my -1 because you answered my question. Full review will most likely wait until tomorrow. I feel guilty parking the kids in front of the TV | |
| 18:57:16 | lyarwood | np | |
| 19:08:25 | opendevreview | Lee Yarwood proposed openstack/nova master: api: Introduce microversion 2.89 adding attachment_id to responses https://review.opendev.org/c/openstack/nova/+/804275 | |
| 20:01:06 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Make connection_info returned by CinderFixture unique per attachment https://review.opendev.org/c/openstack/nova/+/804298 | |