| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-16 | |||
| 12:56:32 | opendevreview | Takashi Natsume proposed openstack/nova master: trivial: Add a white space in an error message https://review.opendev.org/c/openstack/nova/+/823223 | |
| 13:34:00 | yuval | hello wizards | |
| 13:34:06 | yuval | I have a small question | |
| 13:34:23 | yuval | after I reboot my nova compute node | |
| 13:34:38 | yuval | I need to manually run modprobe nvme-tcp | |
| 13:35:01 | yuval | is there any module that does it? or its the operator responsibility | |
| 13:41:36 | gibi | yuval: I think that is operator responsibility | |
| 13:42:23 | sean-k-mooney | yuval: you should add it to /etc/modules i belive to have it auto load | |
| 13:42:25 | gibi | yuval: you can make loading a module automatic via /etc/modules | |
| 13:42:45 | sean-k-mooney | it depends on your disto but its not really a nova related thing | |
| 13:43:38 | yuval | doesnt it make sense that the os-brick connector - which counts on nvme to verify it or just do it | |
| 13:44:58 | gibi | bauzas: fyi, fresh gate bug https://bugs.launchpad.net/nova/+bug/1961068 not blocking as it is not super frequent | |
| 13:45:07 | bauzas | shit | |
| 13:45:08 | gibi | bug report thanks to elodilles | |
| 13:45:56 | gibi | either a memory leak somewhere or we simply need to switch to a bigger flavor for testing | |
| 13:49:51 | Uggla | sean-k-mooney, thanks. many project that provide plugins like neutron provide sample local.confs --> my point is just that, I think it is something that should be more enlighten in the doc. | |
| 13:50:12 | sean-k-mooney | yep in docs not in local.conf | |
| 13:50:25 | sean-k-mooney | the docs and readme can certenly be improved | |
| 13:52:00 | Uggla | sean-k-mooney, sorry maybe I was unclear, the idea is to complete the doc maybe with links to project local.conf | |
| 13:55:24 | sean-k-mooney | you could yes. although one of the design goals fo devstack and intetion is that as you use it you read the code and learn the options as and when you need them | |
| 13:55:29 | sean-k-mooney | its why its written in bash | |
| 13:55:39 | sean-k-mooney | to keep the barrier to entry as low as possibel | |
| 13:56:34 | sean-k-mooney | but ya you should feel free to submit patches to improve devstack. its developed on #openstack-qa | |
| 14:11:41 | sean-k-mooney | stephenfin: can you take a look at https://review.opendev.org/c/openstack/nova/+/825496 its the uefi SMM patch | |
| 14:12:02 | stephenfin | sure, will check before EOD | |
| 14:12:18 | sean-k-mooney | thanks | |
| 14:34:51 | gmann | gibi: thanks | |
| 14:44:34 | opendevreview | Takashi Natsume proposed openstack/osc-placement master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/osc-placement/+/808565 | |
| 15:36:08 | opendevreview | Sylvain Bauza proposed openstack/nova master: neutron: Allow to spawn VMs with port without IP address https://review.opendev.org/c/openstack/nova/+/669411 | |
| 15:51:21 | opendevreview | Merged openstack/osc-placement master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/osc-placement/+/808565 | |
| 16:26:44 | opendevreview | Kashyap Chamarthy proposed openstack/placement master: bindep: s/python-devel/python3-devel/ https://review.opendev.org/c/openstack/placement/+/829573 | |
| 19:02:29 | sean-k-mooney | stephenfin: bauzas gibi im -1 on https://review.opendev.org/c/openstack/nova/+/823941 adding multi path support for nvme over fabic mainly due to paper work | |
| 19:02:41 | sean-k-mooney | there is no approve blueprint or spec for this | |
| 19:02:46 | sean-k-mooney | but its also trivial | |
| 19:03:12 | sean-k-mooney | so im supportive fo approving it as a specless blueprint even this late based on the size | |
| 19:04:03 | sean-k-mooney | bauzas: gibi what do you think stephenfin is +2 on it and honestly i was also going to +2 it until i notice the commit message did not reference a blueprint or bug | |
| 19:05:19 | sean-k-mooney | the author seams to be a new contibutor https://review.opendev.org/q/owner:lior.friedman%2540kioxia.com | |
| 19:06:12 | sean-k-mooney | so i dont want lack of familarity with our process or deadlines to dicurage them but technially we shoudl defer this to Zed at this point since we are past the spec approval deadline | |
| 19:07:04 | sean-k-mooney | so ya im open to makeing an expciton since it tiny but was wondering how other felt | |
| 19:08:21 | opendevreview | Merged openstack/nova stable/xena: skip test_tagged_attachment in nova-next https://review.opendev.org/c/openstack/nova/+/828542 | |
| 19:22:30 | opendevreview | Merged openstack/nova stable/xena: api: Reject duplicate port IDs in server create https://review.opendev.org/c/openstack/nova/+/827120 | |
| 19:22:37 | opendevreview | Merged openstack/nova stable/wallaby: Reproduce bug 1952941 https://review.opendev.org/c/openstack/nova/+/827870 | |
| 20:32:36 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 20:32:36 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 20:32:37 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 21:24:20 | opendevreview | Julia Kreger proposed openstack/nova master: WIP Ironic - Handle instance/node host on rebalance https://review.opendev.org/c/openstack/nova/+/813897 | |
| 22:43:16 | opendevreview | Julia Kreger proposed openstack/nova master: Ironic - Don't query the API for instance counts https://review.opendev.org/c/openstack/nova/+/829613 | |
| #openstack-nova - 2022-02-17 | |||
| 03:22:55 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Register defaults for undefined hw image properties https://review.opendev.org/c/openstack/nova/+/800708 | |
| 03:38:30 | opendevreview | Ghanshyam proposed openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 04:16:30 | opendevreview | Ghanshyam proposed openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 05:19:40 | opendevreview | Ghanshyam proposed openstack/nova master: Test PROJECT_ADMIN APIs with no legacy rule case https://review.opendev.org/c/openstack/nova/+/824845 | |
| 05:20:05 | opendevreview | Ghanshyam proposed openstack/nova master: Move rule_if_system() method to base test class https://review.opendev.org/c/openstack/nova/+/824475 | |
| 05:20:08 | opendevreview | Ghanshyam proposed openstack/nova master: Convert SYSTEM_ADMIN|READER to Admin and system scope https://review.opendev.org/c/openstack/nova/+/819390 | |
| 05:20:33 | opendevreview | Ghanshyam proposed openstack/nova master: Make more project level APIs scoped to project only https://review.opendev.org/c/openstack/nova/+/828670 | |
| 05:22:27 | opendevreview | Ghanshyam proposed openstack/nova master: Server actions APIs scoped to project scope https://review.opendev.org/c/openstack/nova/+/824358 | |
| 05:22:36 | opendevreview | Ghanshyam proposed openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | |
| 05:39:18 | opendevreview | Ghanshyam proposed openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | |
| 05:39:32 | opendevreview | Ghanshyam proposed openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 06:01:41 | opendevreview | Ghanshyam proposed openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | |
| 06:03:28 | opendevreview | Ghanshyam proposed openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 06:03:43 | opendevreview | Ghanshyam proposed openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 08:23:54 | gibi | sean-k-mooney: re nvme multipath: I'm not against an exception if it is low risk (and it seems so). But I defer to bauzas to make the decision with the PTL hat. | |
| 08:24:47 | sean-k-mooney[m] | ack | |
| 08:34:25 | bauzas | gibi: sean-k-mooney: I agree with sean's proposal for a specless bp approval during Feb 22th meeting | |
| 08:34:38 | bauzas | this way, we will see whether we have a consensus | |
| 08:38:17 | bauzas | sean-k-mooney: gibi: procedural -2 for https://review.opendev.org/c/openstack/nova/+/823941 | |
| 08:38:39 | gibi | bauzas: ack | |
| 08:39:07 | bauzas | sean-k-mooney: if you want, you can review this change and +2 it | |
| 08:39:37 | bauzas | so, in case we agree on accepting it for Yoga, I'd just remove the -2 and do +W | |
| 08:40:37 | sean-k-mooney[m] | i have actully reviewed it and the lack of bluepinrt/bug was the main thing that stopped me form +2ing before | |
| 08:40:50 | sean-k-mooney[m] | but sure ill take a look again | |
| 08:41:07 | sean-k-mooney[m] | and ya we can decide on tuesday | |
| 08:42:39 | bauzas | ++ | |
| 08:42:57 | bauzas | sean-k-mooney: thanks btw. for having reviewed it | |
| 09:18:25 | yuval | hello | |
| 09:18:53 | yuval | I am just making sure the feature freeze is the week of the 21 meaning -> 25.2 right? | |
| 09:26:15 | gibi | yuval: I think the cut date for us will be 2.24 EOB but things that are approved can be rebased and merged after it | |
| 09:35:22 | yuval | whats EOB stand for? | |
| 09:36:12 | yuval | I have this: https://review.opendev.org/c/openstack/nova/+/821606 | |
| 09:36:30 | yuval | its not complicate - its just waiting for os-brick release to pass zuul | |
| 09:40:57 | gibi | end of business | |
| 09:41:25 | gibi | so basically end of the day of cores on the west coast | |
| 09:42:27 | yuval | got it | |
| 09:42:31 | gibi | yuval: as far as I see your os-brick patch is not part of that release https://review.opendev.org/c/openstack/os-brick/+/823654 | |
| 09:42:36 | gibi | as this is not landed yetr | |
| 09:43:46 | yuval | gibi: this is not the important patch for me, although I do think brian want it in the release | |
| 09:44:07 | yuval | os-brick 5.2 dont have the lightos connector in it | |
| 09:44:15 | yuval | so zuul is failing in nova for it | |
| 09:44:29 | gibi | ahh so you only need https://review.opendev.org/c/openstack/os-brick/+/821603 | |
| 09:44:53 | yuval | yes | |
| 09:45:02 | gibi | and that is already landed and will be in os-brick 5.2 | |
| 09:45:03 | gibi | cool | |
| 09:45:10 | gibi | then I think you are on the good track | |
| 09:45:20 | yuval | ok, thanks | |
| 09:45:23 | gibi | I will check you nova patch soon | |
| 09:45:31 | yuval | that would be great | |
| 10:15:56 | sean-k-mooney | gibi: regarding the placment changes i think your current implemenation is likely ok to proceed with. we could add followup patches to try the alternitive approch if we do think the perfload resulst are refective fo a real regression | |
| 10:16:55 | gibi | sean-k-mooney: I have to figure out how to make a less noisy perfload measurement because currenty the noise is comparabel with the amount of degradation I can measure | |
| 10:17:19 | gibi | but at least I did not make the performance halved :D | |
| 10:18:00 | gibi | sean-k-mooney: I'm happy to make followup patches trying to improve especially as you gave directions with the SQL I can try to apply and measure | |