| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-06-29 | |||
| 14:31:40 | bauzas | but, tbc, I don't want to see other changes just changing types without thinking about this | |
| 14:32:30 | bauzas | stephenfin: fwiw, I'd also have loved if you could have cut this change in two and not adding two properties as well in the same | |
| 14:32:42 | bauzas | given this wasn't needed | |
| 14:33:14 | stephenfin | It wouldn't really help though | |
| 14:33:37 | bauzas | anyway, reviewing it | |
| 14:33:56 | stephenfin | You'd save about four lines in the first patch (the properties), and then you'd have to review a second patch that's reworks virtually everything you changed in the first one | |
| 14:54:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: Test the state of VMware NSX 3pp CI https://review.opendev.org/734114 | |
| 15:17:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/train: Guard against missing image cache directory https://review.opendev.org/738455 | |
| 17:21:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Warn at controller start if there are older than N-1 computes https://review.opendev.org/738482 | |
| 17:38:08 | openstackgerrit | Merged openstack/nova master: objects: Add MigrationTypeField https://review.opendev.org/706013 | |
| #openstack-nova - 2020-06-30 | |||
| 04:15:12 | openstack | bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 04:15:12 | openstackgerrit | Merged openstack/nova stable/queens: Reproduce bug 1862633 https://review.opendev.org/729539 | |
| 06:04:18 | openstackgerrit | Alexandre Arents proposed openstack/nova master: Limit the number of concurrent snapshots https://review.opendev.org/736169 | |
| 07:05:35 | aarents | Good morning Nova, | |
| 07:05:50 | aarents | basic gerrit workflow question , should I rebase this https://review.opendev.org/#/c/734776/ , because I repushed nits on https://review.opendev.org/#/c/736169 (first one is on top of 2nd) | |
| 07:14:18 | bauzas | good morning nova | |
| 07:14:21 | bauzas | aarents: /me looks | |
| 07:15:01 | bauzas | aarents: indeed but I can +W it | |
| 07:15:12 | bauzas | since stephenfin and dansmith accepted it | |
| 07:16:26 | aarents | bauzas: thanks, doing | |
| 07:16:36 | bauzas | aarents: ping me when you're done | |
| 07:16:51 | aarents | k | |
| 07:17:00 | gibi | aarents: yeah, in general you don't have to be affraid loosing +2 or +A. if it was there we will be put it back easily | |
| 07:17:43 | aarents | great | |
| 07:22:54 | openstackgerrit | Alexandre Arents proposed openstack/nova master: Snapshot: offload glance upload in a native thread https://review.opendev.org/734776 | |
| 08:52:07 | openstackgerrit | Wenping Song proposed openstack/nova master: Add instance project_id for cyborg arq https://review.opendev.org/738428 | |
| 10:29:49 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: test_hardware: code formating for multiline construct https://review.opendev.org/728793 | |
| 11:08:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Update keypairs when saving an instance https://review.opendev.org/683043 | |
| 11:08:37 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Reset changes for nested fields https://review.opendev.org/738583 | |
| 12:00:14 | openstackgerrit | Jinsheng Zhang proposed openstack/nova master: Add API to support multiple boot-volumes with boot order selection https://review.opendev.org/737913 | |
| 12:01:16 | gibi | dansmith: I need your guidance in https://review.opendev.org/#/c/738482 (see comments inline) | |
| 13:03:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Rework how we check for neutron extensions https://review.opendev.org/705792 | |
| 13:05:21 | sean-k-mooney2 | stephenfin: ^ why | |
| 13:06:17 | sean-k-mooney2 | more or less that seams to just be renames but you have also mixed in some parmater changes | |
| 13:09:41 | stephenfin | the only parameter change is that I use the extension alias to do extension lookup rather than the name | |
| 13:09:54 | stephenfin | the alias is a slug, so that seems more correct IMO | |
| 13:10:17 | sean-k-mooney2 | you changed to useing the named args instead of postiional too but that is correct | |
| 13:11:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use neutronclient's port binding APIs https://review.opendev.org/706295 | |
| 13:11:42 | sean-k-mooney2 | also most people wont be familar with the term slugs its a term more or less exclucivly used in web dev form old newpaper printing | |
| 13:12:05 | sean-k-mooney2 | that said i agree that using the stevedor alias is better then the name | |
| 13:12:31 | sean-k-mooney2 | the entry point alias is required to be stable the name is not | |
| 13:14:24 | sean-k-mooney2 | strictly speaking i think you could make the argument the extenion names could be translatable | |
| 13:31:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Don't use generic 'Field' container https://review.opendev.org/738239 | |
| 13:31:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Use imports instead of type aliases https://review.opendev.org/738018 | |
| 13:31:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove unnecessary type aliases, exceptions https://review.opendev.org/738240 | |
| 13:31:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove wrappers around ovo mixins https://review.opendev.org/738019 | |
| 13:31:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add type hints https://review.opendev.org/738020 | |
| 13:36:48 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add type hints https://review.opendev.org/738020 | |
| 13:43:59 | tacco | hi there.. anyone knows where the volumes_attached: information in "openstack server show" comes from? | |
| 13:44:31 | tacco | because i deleted a volume long time ago.. but it still exists in that particular place.. and thats why i cannot resize this instance.. but the volume was successfull deleted long time ago. | |
| 13:44:36 | tacco | any idea? | |
| 13:44:43 | tacco | tryed to fix this in the DB but no chance for nor. | |
| 13:44:50 | tacco | s/nor/now/ | |
| 13:49:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.opendev.org/676209 | |
| 13:49:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to top-level modules https://review.opendev.org/705658 | |
| 13:49:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add nova.cmd, nova.conf, nova.console https://review.opendev.org/705657 | |
| 13:53:26 | stephenfin | tacco: That's this code https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/servers.py#L714-L733 | |
| 13:59:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove custom comparison methods https://review.opendev.org/472285 | |
| 14:03:37 | tacco | stephenfin: thanks a lot will have a look | |
| 14:19:58 | tacco | cannot find the right way.. trough.. all i can find is in cinder that the volume is marked deleted and the volume does not exists anymore. But in the volumes_attached it still apeares. So strange.. | |
| 14:22:14 | tacco | is there any way to force detatch a volume from the instance by nova? | |
| 14:50:13 | gibi | tacco: look into the block_device_mapping table of the nova cell database | |
| 15:00:52 | aarents | It seems that check and gate are failing due to : 'Build of instance b29da99e-65e8-4b0a-b542-a368f6b30154 aborted: rbd python libraries not found' on nova-live-migration and nova-grenade-multinode | |
| 15:04:12 | aarents | wondering if it is related to mail "devstack dropping python 2 support" | |
| 15:11:51 | tacco | gibi: found it and seems to be the point i was looking for.. thanks a lot. | |
| 15:21:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Update keypairs when saving an instance https://review.opendev.org/683043 | |
| 15:24:26 | stephenfin | dansmith: I tried to do this https://review.opendev.org/#/c/738583/ and it exploded. Guess we're relying on fields staying dirty or something :( | |
| 15:25:44 | stephenfin | so I've gone with the 'keypairs' field-only fix in https://review.opendev.org/#/c/683043/ though I suspect the issues I highlighted in ^ might still be true | |
| 15:26:47 | dansmith | stephenfin: one failure in unit tests that is asserting something equal? | |
| 15:27:26 | dansmith | meaning, something that compared two objects and failed because of the changed list, which is likely just someone encoding what caused the test to pass before | |
| 15:28:16 | dansmith | what explosion specifically? | |
| 15:34:17 | stephenfin | dansmith: No, the functional test | |
| 15:34:20 | stephenfin | *s | |
| 15:35:08 | dansmith | the numa one seems like it *has* to be a bug | |
| 15:35:14 | dansmith | I dunno about the port ones | |
| 15:35:56 | stephenfin | If I filter out the numa_topology and resources fields before doing the rest, the few failing tests I tried passed again | |
| 15:36:24 | stephenfin | but idk where to even start figuring out what's going on | |
| 15:42:11 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Don't allow "reserving" file-backed memory https://review.opendev.org/736219 | |
| 15:55:54 | mandre | hi, when setting glance annotation to enable a specific driver, for instance virtio-scsi, that restricts this image to a given hypervisor, correct? For example with virtio-scsi, this limits it to libvirt/kvm, right? BM nodes won't be able to boot from the same image? | |
| 17:38:46 | sean-k-mooney | well for the equal case we shoudl never compure object with out serialising them as the dirty state is actully checked | |
| 17:39:34 | sean-k-mooney | so just resetting all fields recurcivly woudl break any chang that was looking at raw equals comparison | |
| 17:40:18 | sean-k-mooney | its one of the reason we call to_primative and compure the dits normally then jsut doing ovo == other_ovo | |
| 17:40:47 | sean-k-mooney | or self.assertEquals(ovo, other_ovo) i guess | |
| 17:44:01 | sean-k-mooney | mandre: if the dirver does not support it it will ignore it | |
| 17:44:10 | sean-k-mooney | mandre: so it should work on an ironic node | |
| 17:44:25 | sean-k-mooney | i have that set on my images and they can be used with ironic | |
| 17:50:35 | mandre | great, thanks sean-k-mooney! | |
| 17:51:46 | sean-k-mooney | the only thing that would prevent you from sharing the image beyond driver in the image for the hardware would be if you enabled a filter or prefilter that was makeing some assertion regrating support for a feature in the schduler | |
| 17:51:59 | sean-k-mooney | non of the defualt ones shoudl be an issue | |
| 17:52:52 | sean-k-mooney | well things like have a numa toplogy in the image might cause the node to be rejected if you have the numa toplogy filter enabled because we dont report a numa toplogy for ironic nodes | |
| 17:53:24 | sean-k-mooney | but hw_disk_bus wont break anything unless you enable the prefilter that checks that | |
| 17:53:44 | mandre | ack, that shouldn't be an issue in my case I think but I'll keep that in mind | |
| 17:57:26 | sean-k-mooney | mandre: strangly enough i was just deploy ironic nodes with vm image and i just hit what i assume was a dirver issue | |
| 17:57:56 | sean-k-mooney | cetos8.1 booted fine form me by ubuntu 18.04 and 20.04 was not getting an ip correctly | |
| 17:58:17 | sean-k-mooney | when i used my vm image on my ironic hosts | |
| 17:59:30 | sean-k-mooney | all of those iamge have hw_disk_bus=scsi and hw_scsi_model=virtio-scsi and booted fine other then that | |
| 19:41:35 | ajitha | Hi all, We are running a third party cinder driver, we are testing multi attach functionality. and i am getting "Multiattach volumes are only supported starting with compute API version 2.60" error. | |
| 19:42:39 | ajitha | i found as using --os-compute-api-version is one option | |
| 19:43:13 | ajitha | how to set the compute-api-version permanently | |
| 20:41:16 | lseki | ajitha: https://docs.openstack.org/tempest/latest/sampleconf.html | |