| *** efried1 is now known as efried | 05:18 | |
| bauzas | good morning Nova | 06:58 |
|---|---|---|
| gibi | o/ | 07:26 |
| dvo-plv | Hello, I would like to remind about to take a look on the reproposed spec file ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | 07:45 |
| dvo-plv | and check dependent patch for it https://review.opendev.org/c/openstack/nova/+/876075 | 07:45 |
| bauzas | yup, specs reviews is on my agenda this week | 07:53 |
| *** Continuity__ is now known as Continuity | 08:03 | |
| opendevreview | sean mooney proposed openstack/nova master: remove py2 only syntax https://review.opendev.org/c/openstack/nova/+/896984 | 09:08 |
| opendevreview | sean mooney proposed openstack/nova master: remove use of % format strings https://review.opendev.org/c/openstack/nova/+/896985 | 09:08 |
| opendevreview | sean mooney proposed openstack/nova master: pyupgrade py 3.0 -> 3.6 https://review.opendev.org/c/openstack/nova/+/896986 | 09:08 |
| opendevreview | sean mooney proposed openstack/nova master: add precommit pyupgrade hook and tox env https://review.opendev.org/c/openstack/nova/+/896987 | 09:08 |
| opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | 09:08 |
| opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | 09:09 |
| opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | 09:10 |
| dvo-plv | bauzas, good to hear,thank you | 09:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: doc: Remove crud from conf.py file https://review.opendev.org/c/openstack/nova/+/896224 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Update plugin versions https://review.opendev.org/c/openstack/nova/+/896226 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tox: Use pre-commit for pep8 target https://review.opendev.org/c/openstack/nova/+/896227 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Add mypy https://review.opendev.org/c/openstack/nova/+/896228 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Blacken codebase https://review.opendev.org/c/openstack/nova/+/896229 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Integrate black https://review.opendev.org/c/openstack/nova/+/896230 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Ignore black change https://review.opendev.org/c/openstack/nova/+/896231 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Use native hacking pre-commit hook https://review.opendev.org/c/openstack/nova/+/897005 | 10:20 |
| *** Continuity__ is now known as Continuity | 10:24 | |
| opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | 12:38 |
| opendevreview | Merged openstack/os-vif stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/os-vif/+/894062 | 12:55 |
| *** haleyb_out is now known as haleyb | 13:33 | |
| stephenfin | bauzas: I'm not sure I understand your comment on https://review.opendev.org/c/openstack/nova/+/896226/? Does my reply make sense? | 14:56 |
| bauzas | stephenfin: tbc, I'm not a big fan of pre-commit hooks | 14:57 |
| bauzas | they slow our development process and avoid new contributors to easily push some code asking for help | 14:57 |
| stephenfin | but you don't need to use them | 14:58 |
| bauzas | in general, I do atomic commits and then I squash | 14:58 |
| bauzas | the more pre-commit hooks we add, the slower a git commit happens | 14:58 |
| stephenfin | only if you enable them | 14:58 |
| stephenfin | which I'm guessing you don't? | 14:58 |
| bauzas | sure, but you're adding them by default | 14:58 |
| bauzas | which relates to my other concern | 14:59 |
| stephenfin | I am? | 14:59 |
| bauzas | and yeah, I know I can disable hooks | 14:59 |
| stephenfin | hooks aren't enabled by default. It's a manual step to install them | 14:59 |
| bauzas | that's why I'm proposing an half-way solution, which is to use pre-push hooks | 14:59 |
| bauzas | it's an automatic step on my env | 15:00 |
| stephenfin | before we talk about that, back up: where is this being added by default? | 15:00 |
| stephenfin | if you don't run 'pre-commit install' the hooks are irrelevant | 15:00 |
| bauzas | I need then to check something | 15:03 |
| bauzas | because I'm sure it's pulling pre-commit without my request | 15:03 |
| stephenfin | I'm very confused. Do you have pre-commit installed? If not, it isn't capable of installing itself or anything like that | 15:04 |
| bauzas | I do | 15:05 |
| stephenfin | Okay, if it's annoying couldn't you just remove it? The CI will still catch things for you | 15:05 |
| bauzas | and yeah I wonder how that package went into my env | 15:06 |
| bauzas | stephenfin: I surely can remove that package | 15:06 |
| bauzas | stephenfin: can we actually discuss that topic into our PTG sessions if you can attend ? black is a different matter, but I'd like us to agree on what are the expectations of a git commit | 15:07 |
| bauzas | I'm not sure everyone talks the same language | 15:07 |
| stephenfin | We already have expectations of a git commit. The hard ones are encoded in the pep8 job. Nothing has changed | 15:11 |
| stephenfin | I can understand the merit of discussing the black stuff there, but pre-commit is simply a wrapper around tooling that we've already agreed on (autopep8, hacking, mypy). Do we need 20+ people discussing a tool like that? | 15:12 |
| clarkb | did precommit fix the problem of not installing tools from pypi? | 15:14 |
| stephenfin | That'd be a non-goal since PyPI is a Python-specific repository and pre-commit is intentionally language agnostic | 15:15 |
| stephenfin | at least, I imagine that to be the case? tbh I've never looked :) | 15:15 |
| bauzas | I just think that not a lot of people exactly understand which tools are run and why | 15:18 |
| bauzas | and if they see a pre-commit YAML file, they'll pull it | 15:18 |
| bauzas | but if we are clear on the line that pre-commit is optional, then I question the need | 15:19 |
| sean-k-mooney | bauzas: they way we use pre-commit now its entirly optional to install the hook locally and all automations are enforced in the tox jobs already | 15:19 |
| bauzas | for our CI, I guess ? | 15:19 |
| bauzas | sean-k-mooney: the way it's done is that *if* you have pre-commit installed, then you're tied to its rules | 15:20 |
| sean-k-mooney | yes but the pep8 job aslo runs the same checks | 15:21 |
| sean-k-mooney | as does tox -e pep8 | 15:21 |
| bauzas | sean-k-mooney: sure, but then I question the benefits | 15:22 |
| sean-k-mooney | its faster for one | 15:22 |
| sean-k-mooney | vs what we are doing today | 15:22 |
| sean-k-mooney | you can also skip pre-commit whenever you want with --no-verify | 15:23 |
| sean-k-mooney | if you have it installed | 15:23 |
| clarkb | how is it faster if it runs the same checks? | 15:23 |
| sean-k-mooney | if you dont have it installed the you can run it with tox | 15:23 |
| sean-k-mooney | and not install it or change your workflow | 15:23 |
| sean-k-mooney | clarkb: by default its flake8 check just the files you modifed | 15:23 |
| sean-k-mooney | so its the same as our old fast8 tox env | 15:24 |
| sean-k-mooney | for other things its not neessisarly any faster | 15:24 |
| dansmith | it's still faaar too slow for run-on-commit for me :) | 15:24 |
| bauzas | I just feel we absolutely need to amend https://docs.openstack.org/nova/latest/contributor/development-environment.html#using-the-pre-commit-hook | 15:24 |
| bauzas | to make it optional | 15:24 |
| sean-k-mooney | it is optional | 15:25 |
| bauzas | and specify how to skip it | 15:25 |
| bauzas | sean-k-mooney: our docs aren't that clear | 15:25 |
| sean-k-mooney | if you dont install it then it does not run | 15:25 |
| sean-k-mooney | if you do its just --no-verify | 15:25 |
| sean-k-mooney | but we can updated it to be more clear | 15:25 |
| melwitt | you can also just do -n | 15:25 |
| bauzas | I think I can and shall update that docv | 15:26 |
| melwitt | and the doc was clear enough to me, fwiw. i.e. I understood pre-commit doesn't happen unless you install the pre-commit package | 15:26 |
| sean-k-mooney | melwitt: oh good to know | 15:26 |
| sean-k-mooney | yep "This must be enabled locally to function:" | 15:26 |
| sean-k-mooney | was ment to capture that in the doc | 15:26 |
| bauzas | then I'll remove my vote on the pre-commit additions change | 15:27 |
| melwitt | (I'm not objecting to improving the doc btw, just saying at least to me I've used the doc and it made sense for me) | 15:27 |
| sean-k-mooney | bauzas: for what its worht i have other related but seperate changes | 15:28 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/896987 | 15:28 |
| sean-k-mooney | i was going to add a "tech-debt reduction and code modernization" topic to the ptg (items for this seession would by hyperv and vmware driver removal. black and pyupgrade) | 15:32 |
| sean-k-mooney | we have blueprints filed for at least 3 of those. https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver https://blueprints.launchpad.net/nova/+spec/pyupgrade and https://blueprints.launchpad.net/nova/+spec/remove-vmware-support | 15:34 |
| sean-k-mooney | i dont know if we have or need one for automated formating but we can at least have the conversation at the ptg | 15:34 |
| bauzas | sean-k-mooney: I'd be in favor of more fine-scoped topic names | 15:35 |
| stephenfin | bauzas: Thanks. Good to hear that confusion is cleared up (y) | 15:35 |
| dansmith | I'm prepared to die on black hill | 15:35 |
| bauzas | at least for clarifying the intent and clearly timekeep on subjects | 15:35 |
| bauzas | one topic I could add could be "volume attachment improvements" but that's exactly kind of the same vague bucket :) | 15:36 |
| sean-k-mooney | i wanted to group related topics | 15:36 |
| sean-k-mooney | hence the sub items | 15:36 |
| bauzas | sean-k-mooney: if you don't mind, stick with items per each, and I'll just make sure we loop thru them as a group | 15:37 |
| stephenfin | dansmith: You'll be happy to hear sean-k-mooney also does not like the spreading of arguments across multiple lines 0:) Apparently there's some way to change that but I didn't look into it yet | 15:37 |
| sean-k-mooney | those 3 are all tech debth reduction and as an fyi i have fully implemted pyupgrade adn vmware removal | 15:37 |
| bauzas | (fwiw, the vmware one seems to me quite non-controversial :) ) | 15:38 |
| sean-k-mooney | i have one open question on it | 15:38 |
| sean-k-mooney | baiscally i intentionl didn not make any api changes but i proably should | 15:38 |
| bauzas | if you stopped a dead corner, then sure :) | 15:38 |
| sean-k-mooney | i.e. i did not remove the mks console server or any of the extra specs validation ectra | 15:38 |
| bauzas | spotted* | 15:38 |
| sean-k-mooney | but want to hear how people want that to be done | 15:38 |
| bauzas | yeah, I thought it was API-related | 15:38 |
| sean-k-mooney | the currnt patch is not api related | 15:39 |
| bauzas | we have a solid experience on API stability while we remove backends | 15:39 |
| sean-k-mooney | just the virt driver and remvoe teh virt dirver spcifric cofnig options and docs. | 15:39 |
| sean-k-mooney | i belive gman has also already done hyperv and has the same open | 15:39 |
| sean-k-mooney | i.e. waht to do with hyperv specific api aspect | 15:39 |
| sean-k-mooney | so i wanted to group does disucssions togetehr | 15:40 |
| sean-k-mooney | since hte approch should likely be the same | 15:40 |
| bauzas | sounds a good idea to me | 15:41 |
| bauzas | thanks for catching it | 15:41 |
| sean-k-mooney | do we have the ptg etherpad yet | 15:41 |
| bauzas | yep | 15:42 |
| bauzas | oh but you weren't in the last meeting, sec | 15:42 |
| sean-k-mooney | ya still catching up form PTO | 15:42 |
| bauzas | np | 15:42 |
| bauzas | https://etherpad.opendev.org/p/nova-caracal-ptg | 15:43 |
| bauzas | (didn't made any huge effort for the naming) | 15:43 |
| sean-k-mooney | cool ill add something to that then and you cna feel frre to rearragne things | 15:44 |
| bauzas | cool | 15:45 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: doc: clarify that pre-commit is optional https://review.opendev.org/c/openstack/nova/+/897057 | 15:53 |
| stephenfin | sean-k-mooney: I wonder if we could start setting and exposing ACPI index for network devices, as a consistent identifier that will be respected (unless PCI device addresses) https://libvirt.org/formatdomain.html#network-interfaces | 16:08 |
| sean-k-mooney | hum let me read | 16:08 |
| sean-k-mooney | but maybe | 16:08 |
| sean-k-mooney | """Since 7.3.0, one can set the ACPI index against network interfaces. With some operating systems (eg Linux with systemd), the ACPI index is used to provide network interface device naming, that is stable across changes in PCI addresses assigned to the device. This value is required to be unique across all devices and be between 1 and (16*1024-1).""" | 16:09 |
| sean-k-mooney | interesting so ya maybe | 16:09 |
| sean-k-mooney | stephenfin: of the top of my head i dont see why not | 16:10 |
| sean-k-mooney | well hum | 16:11 |
| sean-k-mooney | there might be some issues with that | 16:11 |
| sean-k-mooney | so the problem iw going to be how to corralate it with the port | 16:11 |
| sean-k-mooney | so we willl likely need to store it in the virtural interfaces table | 16:11 |
| sean-k-mooney | so that its accesable to the api or store it in the binding_profile on the neutorn port | 16:12 |
| sean-k-mooney | ok so testing the acpi index theory <acpi index="4"/> is accepted in the xml | 16:52 |
| sean-k-mooney | and the interface name becomes eno4 | 16:52 |
| sean-k-mooney | so that is a good sigh | 16:52 |
| sean-k-mooney | sean@upstream-devstack:~$ cat /sys/class/net/eno4/device/acpi_index | 16:53 |
| sean-k-mooney | 4 | 16:53 |
| sean-k-mooney | stephenfin: ^ so ya that is trivially discoverable | 16:53 |
| stephenfin | \o/ | 16:53 |
| stephenfin | noice | 16:53 |
| sean-k-mooney | so if we were to add that to the metadata that would work | 16:53 |
| sean-k-mooney | im going to make it 42 just to be sure it actully is working the way i think | 16:54 |
| sean-k-mooney | sean@upstream-devstack:~$ cat /sys/class/net/eno42/device/acpi_index | 16:55 |
| sean-k-mooney | 42 | 16:55 |
| sean-k-mooney | yep so it will result in a stable device name as well as the docs suggest | 16:55 |
| opendevreview | Dan Smith proposed openstack/nova master: Sanity check that new hosts have no instances https://review.opendev.org/c/openstack/nova/+/897076 | 18:22 |
| opendevreview | Dan Smith proposed openstack/nova master: Sanity check that new hosts have no instances https://review.opendev.org/c/openstack/nova/+/897076 | 19:35 |
| opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | 20:38 |
| opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint issues in releasenotes https://review.opendev.org/c/openstack/nova/+/897087 | 22:04 |
| opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint issues in api guide https://review.opendev.org/c/openstack/nova/+/897088 | 22:04 |
| opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | 22:04 |
| opendevreview | sean mooney proposed openstack/nova master: [codespell] start fixing all the typos https://review.opendev.org/c/openstack/nova/+/897092 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!