| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-28 | |||
| 12:08:15 | sahid | all is green | |
| 16:50:04 | greatgatsby | gibi: thanks so much for the info! I'm in and out of the office today, so I'll try to take a look. | |
| 17:19:59 | opendevreview | Merged openstack/placement master: Changed /tmp/migrate-db.rc to /root/migrate-db.rc https://review.opendev.org/c/openstack/placement/+/882436 | |
| #openstack-nova - 2023-09-29 | |||
| 08:42:28 | gibi | greatgatsby: what you are seeing feels like a bug somewhere in nova but at the moment I have no idea where so without some reproduction I cannot fix it | |
| 09:19:32 | bauzas | gibi: I just looked at yesterday's convo, can you summarize it ? (sorry yesterday was off) | |
| 09:31:01 | opendevreview | Franciszek Przewoźny proposed openstack/osc-placement master: Add missing Xena version in release notes https://review.opendev.org/c/openstack/osc-placement/+/896920 | |
| 09:47:49 | opendevreview | Franciszek Przewoźny proposed openstack/osc-placement master: Fix overline length https://review.opendev.org/c/openstack/osc-placement/+/896922 | |
| 09:48:41 | opendevreview | Franciszek Przewoźny proposed openstack/osc-placement master: Add missing Xena version in release notes https://review.opendev.org/c/openstack/osc-placement/+/896920 | |
| 10:08:21 | opendevreview | Franciszek Przewoźny proposed openstack/osc-placement master: Add xena to toctree https://review.opendev.org/c/openstack/osc-placement/+/896923 | |
| 10:09:29 | opendevreview | Franciszek Przewoźny proposed openstack/osc-placement master: Add missing Xena version in release notes https://review.opendev.org/c/openstack/osc-placement/+/896924 | |
| 11:19:56 | opendevreview | Merged openstack/osc-placement master: Update master for stable/xena https://review.opendev.org/c/openstack/osc-placement/+/808458 | |
| 11:19:57 | opendevreview | Merged openstack/osc-placement stable/zed: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869753 | |
| 11:21:27 | opendevreview | Merged openstack/osc-placement master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/osc-placement/+/894069 | |
| 11:51:30 | opendevreview | Merged openstack/osc-placement stable/yoga: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869754 | |
| 12:10:22 | opendevreview | Merged openstack/osc-placement stable/xena: Use pypi released version of placement in functional tests https://review.opendev.org/c/openstack/osc-placement/+/869768 | |
| 12:35:27 | greatgatsby | gibi: really appreciate the help. I grepped through the logs and do not see the "Refreshing aggregate..." string anywhere in there for the full day on which the sync drift occurred | |
| 12:42:14 | greatgatsby | gibi: sorry, ignore that last message, I need to look into something here. Again, I really appreciate the help. We're monitoring this very closely, so hopefully I can keep provide more info if/when it occurs again. | |
| 14:04:11 | opendevreview | Merged openstack/nova stable/2023.1: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888160 | |
| 15:53:56 | auniyal | Hi dansmith, can you please review this https://review.opendev.org/c/openstack/nova/+/895162 sometime, this is related to device tagging bug. | |
| 15:54:37 | auniyal | I was trying to add a reproducer specifically here- https://review.opendev.org/c/openstack/nova/+/895162/6/nova/tests/functional/libvirt/test_pci_sriov_servers.py#4252 | |
| 15:54:44 | auniyal | but the test is passing for me | |
| 15:55:13 | auniyal | I must be missing something, | |
| #openstack-nova - 2023-09-30 | |||
| 08:53:38 | gibi | greatgatsby: do you have DEBUG log enabled for nova-compute? | |
| 08:55:03 | gibi | greatgatsby: as of summary. I still trying to figure out where nova-compute updated an internal caches about the placement aggregates (as that is later flushed back to placemnet). The DEBUG log above would be can place where nova-compute updates the internal cache. But honeslty these are just stabs in the blind | |
| 09:06:43 | gibi | s/can/one/ | |
| 12:16:02 | greatgatsby | gibi: unfortunately it seems we didn't have debug logging enabled at the time the agg unsync occurred. I've enabled it and I'll search for the "Refreshing... " message if/when this happens again. Again, thanks for the help. | |
| #openstack-nova - 2023-10-01 | |||
| 14:41:48 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| 14:42:01 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| 18:10:27 | opendevreview | sean mooney proposed openstack/nova master: remove py2 only syntax https://review.opendev.org/c/openstack/nova/+/896984 | |
| 18:10:28 | opendevreview | sean mooney proposed openstack/nova master: pyupgrade py 3.0 -> 3.6 https://review.opendev.org/c/openstack/nova/+/896986 | |
| 18:10:28 | opendevreview | sean mooney proposed openstack/nova master: remove use of % format strings https://review.opendev.org/c/openstack/nova/+/896985 | |
| 18:10:29 | opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | |
| 18:10:29 | opendevreview | sean mooney proposed openstack/nova master: add precommit pyupgrade hook and tox env https://review.opendev.org/c/openstack/nova/+/896987 | |
| #openstack-nova - 2023-10-02 | |||
| 06:58:22 | bauzas | good morning Nova | |
| 07:26:15 | gibi | o/ | |
| 07:45:45 | 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:46 | dvo-plv | and check dependent patch for it https://review.opendev.org/c/openstack/nova/+/876075 | |
| 07:53:35 | bauzas | yup, specs reviews is on my agenda this week | |
| 09:08:17 | opendevreview | sean mooney proposed openstack/nova master: remove py2 only syntax https://review.opendev.org/c/openstack/nova/+/896984 | |
| 09:08:18 | opendevreview | sean mooney proposed openstack/nova master: pyupgrade py 3.0 -> 3.6 https://review.opendev.org/c/openstack/nova/+/896986 | |
| 09:08:18 | opendevreview | sean mooney proposed openstack/nova master: remove use of % format strings https://review.opendev.org/c/openstack/nova/+/896985 | |
| 09:08:19 | opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | |
| 09:08:19 | opendevreview | sean mooney proposed openstack/nova master: add precommit pyupgrade hook and tox env https://review.opendev.org/c/openstack/nova/+/896987 | |
| 09:09:45 | opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | |
| 09:10:22 | opendevreview | sean mooney proposed openstack/nova master: ignore pyupgrade series in git blame https://review.opendev.org/c/openstack/nova/+/896988 | |
| 09:26:37 | dvo-plv | bauzas, good to hear,thank you | |
| 10:20:34 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Remove crud from conf.py file https://review.opendev.org/c/openstack/nova/+/896224 | |
| 10:20:35 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Use pre-commit for pep8 target https://review.opendev.org/c/openstack/nova/+/896227 | |
| 10:20:35 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Update plugin versions https://review.opendev.org/c/openstack/nova/+/896226 | |
| 10:20:36 | opendevreview | Stephen Finucane proposed openstack/nova master: Blacken codebase https://review.opendev.org/c/openstack/nova/+/896229 | |
| 10:20:36 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Add mypy https://review.opendev.org/c/openstack/nova/+/896228 | |
| 10:20:37 | opendevreview | Stephen Finucane proposed openstack/nova master: Ignore black change https://review.opendev.org/c/openstack/nova/+/896231 | |
| 10:20:37 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Integrate black https://review.opendev.org/c/openstack/nova/+/896230 | |
| 10:20:38 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Use native hacking pre-commit hook https://review.opendev.org/c/openstack/nova/+/897005 | |
| 12:38:34 | opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | |
| 12:55:30 | 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 | |
| 14:56:47 | 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:57:41 | bauzas | stephenfin: tbc, I'm not a big fan of pre-commit hooks | |
| 14:57:59 | bauzas | they slow our development process and avoid new contributors to easily push some code asking for help | |
| 14:58:15 | stephenfin | but you don't need to use them | |
| 14:58:17 | bauzas | in general, I do atomic commits and then I squash | |
| 14:58:39 | bauzas | the more pre-commit hooks we add, the slower a git commit happens | |
| 14:58:45 | stephenfin | only if you enable them | |
| 14:58:54 | stephenfin | which I'm guessing you don't? | |
| 14:58:56 | bauzas | sure, but you're adding them by default | |
| 14:59:13 | bauzas | which relates to my other concern | |
| 14:59:14 | stephenfin | I am? | |
| 14:59:23 | bauzas | and yeah, I know I can disable hooks | |
| 14:59:37 | stephenfin | hooks aren't enabled by default. It's a manual step to install them | |
| 14:59:45 | bauzas | that's why I'm proposing an half-way solution, which is to use pre-push hooks | |
| 15:00:07 | bauzas | it's an automatic step on my env | |
| 15:00:28 | stephenfin | before we talk about that, back up: where is this being added by default? | |
| 15:00:52 | stephenfin | if you don't run 'pre-commit install' the hooks are irrelevant | |
| 15:03:04 | bauzas | I need then to check something | |
| 15:03:17 | bauzas | because I'm sure it's pulling pre-commit without my request | |
| 15:04: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:05:28 | bauzas | I do | |
| 15:05:59 | stephenfin | Okay, if it's annoying couldn't you just remove it? The CI will still catch things for you | |
| 15:06:03 | bauzas | and yeah I wonder how that package went into my env | |
| 15:06:34 | bauzas | stephenfin: I surely can remove that package | |
| 15:07:34 | 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:51 | bauzas | I'm not sure everyone talks the same language | |
| 15:11:14 | stephenfin | We already have expectations of a git commit. The hard ones are encoded in the pep8 job. Nothing has changed | |
| 15:12:40 | 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:14:18 | clarkb | did precommit fix the problem of not installing tools from pypi? | |
| 15:15:28 | stephenfin | That'd be a non-goal since PyPI is a Python-specific repository and pre-commit is intentionally language agnostic | |
| 15:15:55 | stephenfin | at least, I imagine that to be the case? tbh I've never looked :) | |
| 15:18:13 | bauzas | I just think that not a lot of people exactly understand which tools are run and why | |
| 15:18:29 | bauzas | and if they see a pre-commit YAML file, they'll pull it | |
| 15:19:41 | bauzas | but if we are clear on the line that pre-commit is optional, then I question the need | |
| 15:19:47 | 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:51 | bauzas | for our CI, I guess ? | |
| 15:20:32 | 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:21:02 | sean-k-mooney | yes but the pep8 job aslo runs the same checks | |
| 15:21:17 | sean-k-mooney | as does tox -e pep8 | |
| 15:22:16 | bauzas | sean-k-mooney: sure, but then I question the benefits | |
| 15:22:40 | sean-k-mooney | its faster for one | |
| 15:22:45 | sean-k-mooney | vs what we are doing today | |
| 15:23:11 | sean-k-mooney | you can also skip pre-commit whenever you want with --no-verify | |
| 15:23:19 | sean-k-mooney | if you have it installed | |