| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-19 | |||
| 19:42:29 | sean-k-mooney | in the api | |
| 19:42:50 | sean-k-mooney | so we would just need to drop the extra vlaidation on hostname when its passed explcitly | |
| 19:43:00 | sean-k-mooney | and live with the fact it can be an fqdn | |
| 19:43:01 | dansmith | for the new microversion yeah | |
| 19:43:14 | sean-k-mooney | and document that it will be passed as is to neutron | |
| 19:43:25 | dansmith | I don't love it, but I like it a lot better than us taking multiple things and constructing FQDNs | |
| 19:43:49 | sean-k-mooney | ya we dissucsed having --fqdn when --hostname was added | |
| 19:43:55 | sean-k-mooney | but we did not want to support fqdns | |
| 19:44:13 | sean-k-mooney | but i guess that is a less invasive change | |
| 19:44:20 | dansmith | okay now hold up, | |
| 19:44:24 | dansmith | what about the multi-create case? | |
| 19:44:46 | sean-k-mooney | we will sufix the fqdn | |
| 19:44:47 | dansmith | that would require the parsing of the hostname in nova to insert the index | |
| 19:44:59 | sean-k-mooney | well or that | |
| 19:45:17 | sean-k-mooney | or make the sufic a prefix | |
| 19:45:18 | dansmith | suffixing the fqdn won't yield anything valid, so no point in doing that | |
| 19:45:27 | dansmith | yeah 1-$hostname would work I think | |
| 19:45:36 | sean-k-mooney | we could change that in the microversion | |
| 19:46:00 | sean-k-mooney | i assume we cant just decied to not support multi create | |
| 19:46:12 | dansmith | well, I was going to say, | |
| 19:46:13 | sean-k-mooney | it would make life eiser in many ways :) | |
| 19:46:19 | dansmith | I wonder how useful hostname really is in the multi case | |
| 19:46:50 | sean-k-mooney | so in artoms spec he nandedl this by saying we woudl continue to suffix the hostname part as we do today | |
| 19:47:06 | sean-k-mooney | but the doamin value would be appended and the same for all instnaces | |
| 19:47:16 | sean-k-mooney | which when you have it as two parts makes sense | |
| 19:47:52 | sean-k-mooney | but if we want to avodi parsing then ya prefix or declare not supported | |
| 19:48:20 | dansmith | or just set them all to what they gave us | |
| 19:48:31 | sean-k-mooney | or that | |
| 19:48:40 | sean-k-mooney | ok its getting late here | |
| 19:48:56 | sean-k-mooney | if we want to explore this this cycle i can update the spec | |
| 19:48:59 | dansmith | ...and I'm sick of obsessing over hostnames :) | |
| 19:49:06 | sean-k-mooney | and formally ask for a spec freeze on the mailing list | |
| 19:49:13 | dansmith | I can't imagine we're going to get to agreement on all this in time | |
| 19:49:38 | sean-k-mooney | thats fine i suspect the same | |
| 19:50:04 | sean-k-mooney | im also kind of burnt out on this but i can prepare a draft or at least link to this in the spec | |
| #openstack-nova - 2022-07-20 | |||
| 02:14:28 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: default by_name image_type to config correctly https://review.opendev.org/c/openstack/nova/+/826526 | |
| 02:14:29 | opendevreview | melanie witt proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | |
| 02:14:29 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 02:14:30 | opendevreview | melanie witt proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 02:14:30 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 02:14:31 | opendevreview | melanie witt proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 02:14:31 | opendevreview | melanie witt proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | |
| 02:14:32 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | |
| 02:14:32 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to image and ephemeral disks https://review.opendev.org/c/openstack/nova/+/826528 | |
| 02:14:34 | opendevreview | melanie witt proposed openstack/nova master: virt: Add block_device_info helper to find encrypted disks https://review.opendev.org/c/openstack/nova/+/826529 | |
| 02:14:34 | opendevreview | melanie witt proposed openstack/nova master: blockinfo: Add encryption details to the disk_info mappings when provided https://review.opendev.org/c/openstack/nova/+/772272 | |
| 02:14:36 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add disk_info_mapping as an optional attribute of Image https://review.opendev.org/c/openstack/nova/+/826530 | |
| 02:14:36 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Consolidate create_cow_image and create_image https://review.opendev.org/c/openstack/nova/+/846246 | |
| 02:14:38 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add encryption support to qemu-img create command https://review.opendev.org/c/openstack/nova/+/826752 | |
| 02:14:38 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Report ephemeral encryption traits based on imagebackend https://review.opendev.org/c/openstack/nova/+/826753 | |
| 02:14:40 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 03:49:20 | opendevreview | melanie witt proposed openstack/nova master: Workaround qemu query-migrate issue during live migration https://review.opendev.org/c/openstack/nova/+/842687 | |
| 05:10:45 | opendevreview | liuhuajie proposed openstack/nova master: Remove None for get method, the default value is None https://review.opendev.org/c/openstack/nova/+/850450 | |
| 07:09:28 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 07:27:50 | opendevreview | Manuel Bentele proposed openstack/nova master: libvirt: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova/+/828675 | |
| 08:06:05 | gibi | o/ | |
| 08:18:22 | bauzas | \o | |
| 08:19:40 | Uggla | o/ | |
| 08:28:50 | kashyap | A patch on SPICE, interesting ... upstream SPICE seems almost inactive | |
| 08:30:31 | kashyap | s/almost inactive/dead/ | |
| 08:36:39 | gibi | sean-k-mooney: you probably saw last evening I pushed an update to the pci tracking series. I fixed most of your comments but some is left open with an answer from me, like the owner trait, or the PciDeviceSpec move to PciDevice. I put my oppinonon about those into the reviews | |
| 08:37:14 | sean-k-mooney | ack i did not but ill try to take a look at them today | |
| 08:37:14 | gibi | I promised couple of refactors on top. I will do those now | |
| 08:37:25 | sean-k-mooney | cool | |
| 08:37:54 | gibi | and I noticed a strange functional failure I have to track down (probably a leaked global state somewhere) | |
| 09:26:21 | sean-k-mooney | gibi: is https://review.opendev.org/c/openstack/nova/+/843834 the first patch in the serise | |
| 09:26:32 | sean-k-mooney | i think so but i know that it splits latter | |
| 09:27:07 | gibi | yes, that is the start | |
| 09:27:22 | gibi | I did not update the secondary thread of patches | |
| 09:27:25 | gibi | just the main one | |
| 09:27:35 | sean-k-mooney | cool ill grab coffee and make my way throuhg it | |
| 09:27:53 | gibi | ohh shit, I have an extra file added to that patch :D | |
| 09:27:55 | sean-k-mooney | in the secondary traead there was one change i suggested pulling out an merging first | |
| 09:28:32 | sean-k-mooney | ah q | |
| 09:28:38 | sean-k-mooney | a very good file in deed | |
| 09:28:53 | gibi | there was the restart compute func test enhancement that you asked to split out I will do that todayt | |
| 09:29:06 | gibi | I haven't checked the secondary thread there might be others there too | |
| 09:29:08 | sean-k-mooney | ya that is the one | |
| 09:29:17 | sean-k-mooney | i just think that woudl be nice ot merge ahead of the rest | |
| 09:29:26 | gibi | let me quckly resping the series to remove "q" :D | |
| 09:29:40 | sean-k-mooney | oh its novs git log | |
| 09:29:53 | sean-k-mooney | fun that is nice an recursvice | |
| 09:30:15 | sean-k-mooney | i never do | |
| 09:30:31 | sean-k-mooney | fable has it that back in the dark dark days of nova | |
| 09:31:07 | sean-k-mooney | someone added the entire ddwrt image or souce to novas git repo | |
| 09:31:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | |
| 09:31:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | |
| 09:31:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | |
| 09:31:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 09:31:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 09:31:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 09:31:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 09:31:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 09:31:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 09:31:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 09:31:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 09:31:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | |
| 09:31:28 | gibi | jeah I heard that story :) | |
| 09:31:52 | sean-k-mooney | jay should me the commit at one point | |
| 09:32:10 | sean-k-mooney | i think it has been purged form the amin repo but still exists in gerrits copy | |