| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-29 | |||
| 10:51:21 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 10:51:22 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 11:26:37 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add workaround to skip compareCPU() check on the destination https://review.opendev.org/c/openstack/nova/+/838926 | |
| 11:26:42 | kashyap | gibi: --^ | |
| 11:27:13 | kashyap | Forgot to add your signed-off-by for the ut | |
| 11:27:44 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add workaround to skip compareCPU() check on the destination https://review.opendev.org/c/openstack/nova/+/838926 | |
| 11:32:11 | gibi | kashyap: ack, I will take a look | |
| 11:55:50 | opendevreview | Merged openstack/nova master: Add nova-emulation to the experimental queue too https://review.opendev.org/c/openstack/nova/+/836792 | |
| 11:57:18 | opendevreview | Merged openstack/python-novaclient master: Remove unnecessary packages in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/838199 | |
| 12:08:29 | opendevreview | Merged openstack/nova master: List auth plugin parameters for [keystone] section https://review.opendev.org/c/openstack/nova/+/824458 | |
| 12:08:37 | opendevreview | Merged openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | |
| 12:25:44 | opendevreview | Merged openstack/nova master: Store pf_mac_address and vf_num in extra_info https://review.opendev.org/c/openstack/nova/+/834006 | |
| 13:17:23 | opendevreview | Merged openstack/nova master: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/839227 | |
| 13:20:47 | sean-k-mooney | yay ^ | |
| 13:22:32 | sean-k-mooney | stephenfin: bauzas im +2 on gibis series coudl either of ye review https://review.opendev.org/q/topic:bug%252F1969496 | |
| 13:22:43 | bauzas | I can | |
| 13:22:49 | sean-k-mooney | thanks | |
| 13:26:01 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/nova/+/829248 is now in merge conflict by the way, the neuton patches are all merged if you respin that perhaps on top of ^ so that there are no conflict i think that would be all the main pci related bugfixes that are in flight landed | |
| 13:28:46 | gibi | sean-k-mooney: I don't see the merge conflict | |
| 13:29:43 | sean-k-mooney | i think its because the direct parent was rebased | |
| 13:30:19 | gibi | now I see, It seem gerrit cached something for me | |
| 13:30:42 | gibi | I will do the rebase on top of the other series | |
| 13:30:50 | gibi | later today | |
| 13:30:54 | gibi | thanks for the review! | |
| 13:31:58 | sean-k-mooney | no worries there are a set of sriov/limgrtation patches that were all in flight recnetly and i was spending time reviewign this morning so wanted to see if we could land some of them | |
| 13:32:33 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add a workaround to skip compareCPU() on destination https://review.opendev.org/c/openstack/nova/+/838926 | |
| 13:56:55 | ygk_12345 | hi all I need some information on nova availability zones | |
| 13:57:17 | ygk_12345 | i tried googling but cant fin d the information | |
| 13:58:15 | ygk_12345 | why is it that we have two concepts of aggregates and AZs ? is one not enough ? like exposing aggregates and creating flavors with extra specs to match ? why do we need AZs also ? | |
| 14:05:44 | gibi | sean-k-mooney: quick question do we have the pci parent - child handling in case of pure type-PCI device? or that is only for type-PF and type-VF? | |
| 14:06:34 | gibi | sean-k-mooney: so if we limit the scope of the PCI tracking spec for type-PCI for now, then can we ignore the consume PF -> reserve child VFs logic? | |
| 14:19:28 | gibi | or even if we don't want to handle neutron based PCI requests we still need to handle type-PF and type-VF? | |
| 14:19:39 | gibi | as those can be requested via alias too | |
| 16:33:56 | opendevreview | Merged openstack/nova master: VMware: StableMoRefProxy for moref recovery https://review.opendev.org/c/openstack/nova/+/832164 | |
| 16:34:03 | opendevreview | Merged openstack/nova master: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/839766 | |
| 16:46:50 | opendevreview | Merged openstack/nova master: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/838553 | |
| 16:46:58 | opendevreview | Merged openstack/nova master: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/838554 | |
| 17:34:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/829248 | |
| 20:18:53 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| #openstack-nova - 2022-04-30 | |||
| 02:02:51 | opendevreview | Ghanshyam proposed openstack/placement master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/placement/+/840018 | |
| 02:02:53 | opendevreview | Ghanshyam proposed openstack/python-novaclient master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/python-novaclient/+/840019 | |
| 02:02:55 | opendevreview | Ghanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-vif/+/840020 | |
| 02:03:01 | opendevreview | Ghanshyam proposed openstack/nova master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/nova/+/840021 | |
| 02:06:36 | gmann | melwitt: dansmith ^^ you will be interesting in these. | |
| 02:07:06 | melwitt | gmann: ack thanks | |
| 02:47:38 | opendevreview | Merged openstack/nova stable/xena: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821348 | |
| 12:58:09 | opendevreview | Artom Lifshitz proposed openstack/nova stable/yoga: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839921 | |
| 12:59:09 | opendevreview | Artom Lifshitz proposed openstack/nova stable/yoga: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/839922 | |
| 19:37:37 | opendevreview | Ghanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-vif/+/840020 | |
| #openstack-nova - 2022-05-01 | |||
| 00:39:29 | opendevreview | Ghanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-vif/+/840020 | |
| 00:39:43 | opendevreview | Ghanshyam proposed openstack/placement master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/placement/+/840018 | |
| #openstack-nova - 2022-05-02 | |||
| 07:52:09 | Uggla | Hello Nova o/ | |
| 08:21:40 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Repropose volume backed server rebuild spec https://review.opendev.org/c/openstack/nova-specs/+/840155 | |
| 08:26:10 | gibi | o/ | |
| 08:36:14 | bauzas | \Å“ | |
| 08:36:17 | bauzas | wow | |
| 08:36:20 | bauzas | \o even | |
| 09:09:27 | Uggla | bauzas, Hi, I would like to add a graphviz schema in my spec any objection to add sphinx.ext.graphviz extension to the configuration ? | |
| 09:10:00 | bauzas | Uggla: I'm not a sphinx specialist but IIRC, we already support some way to add a schema | |
| 09:10:23 | bauzas | not sure about specs, but at least for our docs | |
| 09:11:09 | Uggla | bauzas, yep it is the case for the doc, the extension is available. But not for the nova-spec part. | |
| 09:11:30 | bauzas | oh | |
| 09:12:54 | bauzas | Uggla: then, create a change and we can discuss on it | |
| 09:13:06 | bauzas | Uggla: some specs were having some schemes | |
| 09:13:11 | Uggla | bauzas, I have just tested, it is just a matter of adding the extension in the conf for specs. Unless you don't want schema in specs. | |
| 09:14:08 | bauzas | in general, we did like this : https://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/granular-resource-requests.html#use-cases | |
| 09:14:14 | Uggla | bauzas, by the way seqdiag are available but not graphviz one (into specs). | |
| 09:14:54 | bauzas | Uggla: honestly, I'm not against | |
| 09:15:32 | Uggla | bauzas, what do you prefer, change included in my specs or a new one, just for that purpose ? | |
| 09:15:44 | bauzas | Uggla: no, as a specific change | |
| 09:16:46 | Uggla | bauzas, ok | |
| 09:17:00 | bauzas | and thanks for having looked at it | |
| 09:17:25 | Uggla | I'm gonna propose a patch for it in a moment. | |
| 09:19:04 | Uggla | bauzas, do we need something in launchpad for this ? (bug ?, bp ?) | |
| 09:19:18 | bauzas | Uggla: no, just a patch | |
| 09:19:32 | gibi | if you can draw the picture in ascii (e.g with https://asciiflow.com/#/) then I would prefer that instead of a dot file as it is directly reviewable in the source | |
| 09:19:39 | bauzas | Uggla: if you want, we can discuss on it at the next meeting | |
| 09:20:12 | bauzas | gibi: that's why we can discuss it in a meeting | |
| 09:20:27 | gibi | but I know that after a certain complexity ascii is not an option | |
| 09:23:34 | Uggla | gibi, yep and you need to change the ascii for each change. | |
| 09:23:49 | bauzas | Uggla: I did that in the past :p | |
| 09:24:28 | Uggla | gibi, and I'm too lazy to redraw the ascii when graphviz will do it for me. :) | |
| 09:31:40 | gibi | I also know the feeling | |
| 09:41:54 | opendevreview | ribaudr proposed openstack/nova-specs master: Allow to use graphviz into specifications https://review.opendev.org/c/openstack/nova-specs/+/840167 | |
| 11:53:55 | gibi | bauzas: do you have an example how we name pGPU RPs? | |
| 11:55:17 | gibi | bauzas: is it like pci_0000_84_00_0 ? | |
| 11:55:41 | gibi | (asking just to make a consistent proposal for PF RPs) | |
| 12:51:54 | bauzas | gibi: sorry was a dad taxi | |
| 12:56:06 | gibi | no worries | |
| 13:07:43 | bauzas | gibi: did you eventually got your answer ? | |
| 13:07:58 | bauzas | if not, lemme give you the link | |
| 13:08:11 | gibi | from the code comments I see nodename_pci_0000_84_00_0 as the pGPU RP name | |
| 13:08:23 | gibi | but would be nice to confirm | |
| 13:10:02 | bauzas | https://github.com/openstack/nova/blob/b8cc5704558d3c08fda9db2f1bb7fecb2bcd985d/nova/virt/libvirt/driver.py#L8784 | |
| 13:10:14 | bauzas | eg. https://github.com/openstack/nova/blob/b8cc5704558d3c08fda9db2f1bb7fecb2bcd985d/nova/virt/libvirt/driver.py#L8756 | |
| 13:10:26 | bauzas | so yeah | |
| 13:11:52 | gibi | thanks | |
| 13:11:59 | bauzas | or https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L362-L363 | |
| 13:12:24 | bauzas | gibi: ^ see the functest example | |
| 13:12:46 | gibi | ack | |