| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-30 | |||
| 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 | |
| 13:42:39 | artom_ | bauzas, when's the spec review day again? May 13? | |
| 13:42:59 | bauzas | artom: shit, forgot to send the email | |
| 13:43:31 | bauzas | artom: May the 10th https://meetings.opendev.org/meetings/nova/2022/nova.2022-04-26-16.00.html | |
| 13:43:54 | artom | Next Tuesday fsck me | |
| 16:08:06 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: PCI device tracking in Placement https://review.opendev.org/c/openstack/nova-specs/+/791047 | |
| 16:08:27 | gibi | sean-k-mooney, bauzas: ^^ this is a long but hopefully interesting read | |
| 16:51:42 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Proposes to remove keypair generation https://review.opendev.org/c/openstack/nova-specs/+/840217 | |
| 16:53:07 | opendevreview | ribaudr proposed openstack/nova-specs master: libvirt: Allow Manila shares to be directly attached to instances https://review.opendev.org/c/openstack/nova-specs/+/833669 | |
| 17:04:49 | erlon | sean-k-mooney: hey Sean, can you give a look on this bugfix backport when you have a chance so I can continue the backporting throught the other releases? | |
| 17:04:49 | erlon | https://review.opendev.org/c/openstack/nova/+/838788 | |
| 17:04:49 | erlon | https://review.opendev.org/c/openstack/nova/+/836014 | |
| 19:37:05 | mnaser | sean-k-mooney: i feel like you might have some context on this -- but thoughts at isolcpus=X vs CPUAffinity+systemd? | |
| #openstack-nova - 2022-05-03 | |||
| 02:02:45 | opendevreview | melanie witt proposed openstack/nova master: Log the exception returned from a cell during API.get() https://review.opendev.org/c/openstack/nova/+/840260 | |
| 02:07:29 | opendevreview | melanie witt proposed openstack/nova master: Log the exception returned from a cell during API.get() https://review.opendev.org/c/openstack/nova/+/840260 | |
| 07:04:20 | gibi | o/ | |
| 07:46:48 | Uggla | o/ | |
| 07:50:59 | bauzas | \o | |
| 07:52:22 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: PCI device tracking in Placement https://review.opendev.org/c/openstack/nova-specs/+/791047 | |
| 07:52:46 | Uggla | gibi, bauzas if you have time, can you have a look at https://review.opendev.org/c/openstack/nova-specs/+/833669 and especially the proposed states. We discuss that with Artom and we thought that this naming is clearer. | |
| 07:53:28 | gibi | Uggla: added to my review list | |
| 07:55:16 | Uggla | thx, it will be interesting to have your thoughts, by the way I have out what we discussed regarding notifications. I hope that's clear enough. | |
| 07:58:00 | gibi | bauzas: btw, your feedback forum session got accepted too | |
| 07:58:22 | bauzas | gibi: ah ok, haven't seen the results yet | |
| 07:58:43 | gibi | you got a mail from the organizers askig for confirming the timing of the session | |
| 07:59:38 | bauzas | shit | |
| 07:59:43 | bauzas | can't see in my free.fr inbox | |
| 08:00:20 | gibi | you got it from Sarah Krawiecki | |
| 08:00:30 | gibi | with subject: OpenInfra Summit Berlin - Forum Session Accepted! | |
| 08:00:37 | bauzas | damn shit, neither in the spam folder | |
| 08:00:56 | bauzas | I know my free.fr ISP is having some problems | |
| 08:01:15 | gibi | I can forward it. should I try it to free.fr or your work mail address? | |
| 08:01:52 | bauzas | saw your reply to Sarah | |
| 08:02:40 | bauzas | confirmed, nothing got received. | |
| 08:03:10 | bauzas | I should ask Sarah whether she got a bot reply | |
| 08:03:25 | bauzas | thanks gibi for the notice | |
| 08:03:43 | gibi | OK, cool | |
| 08:09:03 | chayan | Hi everyone I recently joined the project and am looking for some beginner friendly issues to contribute to can you please help me ? | |
| 08:10:49 | kashyap | chayan: Hi, have you come acros this document yet? - https://docs.openstack.org/nova/latest/contributor/index.html | |
| 08:11:56 | kashyap | chayan: Are you familiar with Gerrit workflow? If not, you might want to check out the sandbox: https://docs.opendev.org/opendev/infra-manual/latest/sandbox.html | |
| 08:11:57 | bauzas | chayan: welcome | |
| 08:12:12 | bauzas | chayan: what kashyap said is a good starting point | |