| opendevreview | efineshi proposed openstack/nova master: Not destroy the vm as it migrate back before evacuate clean https://review.opendev.org/c/openstack/nova/+/887564 | 01:26 |
|---|---|---|
| *** elodilles_pto is now known as elodilles | 06:16 | |
| opendevreview | suzhengwei proposed openstack/nova-specs master: spec: evacuate api return migration https://review.opendev.org/c/openstack/nova-specs/+/888101 | 06:27 |
| opendevreview | sean mooney proposed openstack/nova master: Remove deprecated AZ filter. https://review.opendev.org/c/openstack/nova/+/886779 | 11:20 |
| opendevreview | sean mooney proposed openstack/nova master: Remove deprecated AZ filter. https://review.opendev.org/c/openstack/nova/+/886779 | 11:22 |
| sean-k-mooney | stephenfin: thanks for the review nits adressed and rebased :) | 11:22 |
| sean-k-mooney | dansmith: gmann if ye are devstack cores can ye look at https://review.opendev.org/c/openstack/devstack/+/886972 to unblock ^ | 11:23 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | 11:45 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | 11:45 |
| auniyal | Hi sean-k-mooney | 11:50 |
| auniyal | regarding AZ filter removal from devstack | 11:50 |
| auniyal | after devstack deployment tempest tests runs right ? so we need to remove AZ test from tempest as well? | 11:50 |
| sean-k-mooney | no | 11:50 |
| auniyal | oh okay, thanks | 11:50 |
| sean-k-mooney | i replied to your comment | 11:50 |
| sean-k-mooney | we are not removing the az support | 11:50 |
| sean-k-mooney | we have 2 implematnions | 11:50 |
| sean-k-mooney | we are just remvoing the older one | 11:51 |
| sean-k-mooney | the same functionality exits | 11:51 |
| sean-k-mooney | so there is no visable change form tempest | 11:51 |
| auniyal | ack | 11:51 |
| *** iurygregory_ is now known as iurygregory | 12:08 | |
| opendevreview | Elod Illes proposed openstack/nova stable/victoria: [stable-only] Cap virtualenv/setuptools https://review.opendev.org/c/openstack/nova/+/888681 | 12:14 |
| opendevreview | Elod Illes proposed openstack/nova stable/victoria: [stable-only] Cap virtualenv/setuptools https://review.opendev.org/c/openstack/nova/+/888681 | 13:18 |
| opendevreview | Merged openstack/nova master: db: Avoid relying on autobegin https://review.opendev.org/c/openstack/nova/+/880668 | 14:37 |
| *** iurygregory_ is now known as iurygregory | 14:59 | |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: Bump MIN_{LIBVIRT,QEMU} for "Bobcat" https://review.opendev.org/c/openstack/nova/+/887255 | 15:06 |
| kashyap | gibi: Hey, later when you get a min, with the latest revision of the above, this UT is still failing for me: test_compare_cpu_aarch64_skip_comparison | 15:16 |
| kashyap | (Since you have the env locally, I'd appreciate a second look. Don't worry if you're short on time; I'll take a deeper look tomm) | 15:16 |
| gibi | kashyap: please ping me tomorrow | 15:43 |
| kashyap | Sure, thx! | 15:43 |
| melwitt | sean-k-mooney: I dunno if you remember this but awhile back someone reported a bug downstream where they got a "QueuePool limit of size 5 overflow 50 reached, connection timed out, timeout 30.00" database error after calling the os-extra_specs API repeatedly over time. I think I figured out what was causing it and proposed a patch here https://review.opendev.org/c/openstack/nova/+/888513 I didn't include test coverage bc I'm not sure | 15:59 |
| melwitt | how to test that ... feel free to suggest if you know a way | 15:59 |
| sean-k-mooney | i vaguly recall that but not the details | 16:03 |
| gmann | sean-k-mooney: just saw your message and devstack change is merged already. ping me anything else pending on devstack side | 16:16 |
| sean-k-mooney | gmann: cool i think if that is merge to stop addign it by default we can proceed on the nova side | 16:34 |
| dansmith | melwitt: I've been seeing an increase in frequency of that mkfs timeout thing lately... | 16:48 |
| melwitt | dansmith: me too. I feel like I see it hit most changes. do you always see it in test_stamp_pattern or any other test? I tried a tempest change to add "no discard" to the command and it didn't help | 16:49 |
| dansmith | melwitt: yep that's the one I just saw fail, not sure if the others are the same or not | 16:50 |
| dansmith | this one I did see IO errors in the console log though, | 16:50 |
| dansmith | so perhaps it's something more going on than just the discard thing | 16:50 |
| dansmith | [ 289.096098] Buffer I/O error on dev vdb, logical block 6, lost async page write | 16:51 |
| melwitt | dansmith: this was the change I tried https://review.opendev.org/c/openstack/tempest/+/886991 I have never seen it hit any other test. and I don't see what is unique about this test to be the only one exhibiting this? | 16:51 |
| dansmith | ack | 16:51 |
| melwitt | there's definitely something else going on than discard imho but I hadn't noticed the IO errors | 16:52 |
| melwitt | the test isn't trying to do anything weird, I'm stumped as to why it's only test_stamp_pattern | 16:53 |
| dansmith | yeah | 16:54 |
| dansmith | Jul 17 16:01:08 np0034696204 kernel: iSCSI Login timeout on Network Portal 10.209.97.252:3260 | 16:55 |
| dansmith | several iscsi errors there in the log | 16:57 |
| dansmith | do you have another example log to look at? this one was nova-multi-cell and it was on the compute worker node in the syslog around the time of the failure | 16:57 |
| gmann | sean-k-mooney: lgtm, one question regarding copy/mentioning the upgrade notes to next SLURP release https://review.opendev.org/c/openstack/nova/+/886779/4/releasenotes/notes/az-filter-removal-587f7337040b6216.yaml#2 | 16:59 |
| gmann | I am not sure we have figured that out in TC but in case we need to tag/make a note somehere to remember it for C release | 17:00 |
| gmann | sean-k-mooney: I am +1 and can convert to +2 as ^^ should not be blocker for it. just bringing it here and in TC also to finalize the guidelines. added this in next TC meeting agenda | 17:13 |
| melwitt | dansmith: maybe this one https://zuul.opendev.org/t/openstack/build/f8734471732246aaab8a8c41e3761af3/logs | 17:14 |
| melwitt | and I had opened this bug about it and put an opensearch query in the first comment https://bugs.launchpad.net/tempest/+bug/2024859 | 17:15 |
| dansmith | melwitt: yeah that one has the login timeout message in the controller's syslog | 17:20 |
| gmann | sean-k-mooney: can you check this again as you were +2 already. I asked ykarel to change the exception type to raise (after checking meeting discussion I am ok now to raise exception instead of warning) https://review.opendev.org/c/openstack/nova/+/868419 | 17:20 |
| dansmith | so that could be the common link | 17:20 |
| sean-k-mooney | gmann: it depend if we raise and excption we woudl ant it to stop the agent form startign right | 17:31 |
| sean-k-mooney | i think that is ok for invalid config i.e. enabling this if the min version is not met | 17:31 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | 17:38 |
| auniyal | hi elodilles | 17:48 |
| auniyal | here https://review.opendev.org/c/openstack/nova/+/882914 | 17:48 |
| auniyal | you meant until "enable validations in nova-lvm" get merged in zed right ? | 17:48 |
| auniyal | this: https://review.opendev.org/q/Ia198f712e2ad277743aed08e27e480208f463ac7 | 17:48 |
| auniyal | dansmith , melwitt can these get merged https://review.opendev.org/q/Ia198f712e2ad277743aed08e27e480208f463ac7 ? | 17:51 |
| melwitt | auniyal: yeah I can review | 17:54 |
| opendevreview | Merged openstack/nova stable/wallaby: Remove deleted projects from flavor access list https://review.opendev.org/c/openstack/nova/+/888333 | 19:35 |
| elodilles | auniyal: yes, those are the gate fixing patches | 19:37 |
| auniyal | thanks melwitt, ack elodilles | 19:50 |
| auniyal | stable/2023.1 will be merged in some time, can you please merge this as well - https://review.opendev.org/c/openstack/nova/+/888230 | 19:50 |
| auniyal | melwitt, regarding your comment https://review.opendev.org/c/openstack/nova/+/873648/comment/e8d05cbb_aa954e91/ in "Added context manager for instance lock" as per lyarwood Note we need to create new context to unlock the instance. | 19:57 |
| melwitt | auniyal: yeah, we'll do that one next | 19:59 |
| melwitt | auniyal: oh, hm. sorry I didn't notice that comment, on the surface, I'm not sure why we would need or want a new request-id for the unlock | 19:59 |
| melwitt | so let me think on that for a bit | 20:00 |
| auniyal | ack, thanks, will fix rest meanwhile | 20:00 |
| melwitt | ++ | 20:00 |
| auniyal | if you have time today, can you have another look on resize one as well; I think its somewhat better now; I have documented alot in it for now intentionally will remove them once good from review side. | 20:02 |
| melwitt | ok, I'll try to get to that today | 20:02 |
| melwitt | auniyal: I just added another comment on the review. thanks for pointing out about the unique request-id's | 20:46 |
| dansmith | auniyal: can you please respond to all comments and/or mark them as resolved when they're no longer relevant? You never responded to my question about needing to target in one place and not another, pushed up another rev, and asked me to re-review.. | 21:12 |
| dansmith | it's hard to know whether you intended to change what I commented about because the old comment had no reply and is still unresolved, on a nonsense line in the new patcheset now | 21:12 |
| opendevreview | melanie witt proposed openstack/nova stable/train: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888509 | 22:20 |
| opendevreview | Merged openstack/nova master: Add config option to configure TB cache size https://review.opendev.org/c/openstack/nova/+/868419 | 23:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!