| opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | 00:15 |
|---|---|---|
| *** bauzas_ is now known as bauzas | 00:24 | |
| melwitt | the openstack-tox-docs job appears to be failing 100% of the time which might be related to an upper-constraints bump of sphinx earlier today. I have posted to the ML https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/TSZUKN6MPOZVMQJQO7XRYIL37INH6PWM/ | 01:48 |
| melwitt | cc stephenfin ^ | 01:48 |
| melwitt | (sphinx expert) | 01:49 |
| *** __ministry is now known as Guest1775 | 02:08 | |
| *** bauzas_ is now known as bauzas | 03:02 | |
| melwitt | stephenfin: update: I found what the problem is, sphinx 8.0.0 made overwriting of user files an error by default and the sphinx-feature-classification extension is doing an overwrite. it needs to use force=True to maintain its behavior, I have proposed a patch based on local testing https://review.opendev.org/c/openstack/sphinx-feature-classification/+/927449 | 03:06 |
| melwitt | that ^ will need a version requirement on 8.0.0 or some other way to only pass force=True if >= 8.0.0. so I'm not sure how to handle that | 03:07 |
| melwitt | stephenfin: I suspect it will be best to roll back the sphinx version bump in upper-constraints for now | 03:09 |
| melwitt | cc bauzas ^ | 03:35 |
| opendevreview | Amit Uniyal proposed openstack/nova master: VMware: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926522 | 05:25 |
| opendevreview | Amit Uniyal proposed openstack/nova master: zvm: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926523 | 05:25 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Ironic: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926524 | 05:25 |
| mikal | melwitt: when is feature freeze? I'm feeling like I'm unlikely to get this patch series landed before then? | 06:56 |
| mikal | Yeah, I think its this week which means this thing isn't going to happen. | 07:04 |
| opendevreview | Jens Harbott proposed openstack/nova master: Delete ./doc/source/_static/support-matrix.css file https://review.opendev.org/c/openstack/nova/+/927471 | 07:28 |
| frickler | melwitt: ^^ maybe this is the better solution? sphinx tries to copy .tox/docs/lib/python3.10/site-packages/sphinx_feature_classification/support-matrix.css over it anyway. at least that seemed to fix the error for me locally | 07:30 |
| frickler | but if that doesn't work for some reason I have missed so far, I'm also fine with reverting the sphinx bump until next cycle | 07:31 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: hardware: Correct log https://review.opendev.org/c/openstack/nova/+/927473 | 08:14 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/926558 | 08:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926559 | 08:18 |
| bauzas | frickler: saw the email from melwitt about the sphinx update that hits the docs CI ? | 08:20 |
| bauzas | oh shit, just saw your points | 08:20 |
| opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | 08:23 |
| bauzas | frickler: do you know if others projects are impacted ? | 08:23 |
| bauzas | I'm just afraid we would find other bugs so close to FF | 08:36 |
| frickler | I didn't see other failures yet, but also didn't explicitly look for them so far | 09:14 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tox: Add Python-specific functional envs https://review.opendev.org/c/openstack/python-novaclient/+/927250 | 09:42 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Fix Python 3.12 compatibility https://review.opendev.org/c/openstack/python-novaclient/+/927251 | 09:42 |
| opendevreview | sean mooney proposed openstack/nova master: update nova-next to use ubuntu 24.04 https://review.opendev.org/c/openstack/nova/+/922492 | 11:08 |
| *** bauzas_ is now known as bauzas | 11:33 | |
| opendevreview | sean mooney proposed openstack/nova master: [DNM] testing uc cap https://review.opendev.org/c/openstack/nova/+/927500 | 12:14 |
| sean-k-mooney | melwitt: i think https://review.opendev.org/c/openstack/openstackdocstheme/+/927496/3/openstackdocstheme/page_context.py will fix the openstackdocstheme | 12:18 |
| sean-k-mooney | and i think https://review.opendev.org/c/openstack/requirements/+/927499 is what allowed the newer sphinx build | 12:19 |
| sean-k-mooney | well the bump to 4.0.0 | 12:19 |
| frickler | sean-k-mooney: sphinx is explicitly pinned in u-c. but also please check https://review.opendev.org/c/openstack/nova/+/927471 | 12:20 |
| sean-k-mooney | that not the error we are seeign in the gate as far as i can see | 12:21 |
| sean-k-mooney | or its not related to it | 12:21 |
| frickler | sean-k-mooney: then maybe we are talking about two different issues, can you point me to one of your failures? | 12:22 |
| sean-k-mooney | there is a new sphinx removal warning for sphinx 9 | 12:22 |
| frickler | yes, but afaict that is non-fatal | 12:23 |
| sean-k-mooney | we treat warnings as errors | 12:23 |
| sean-k-mooney | so in the gate it is | 12:23 |
| frickler | in the docs build? are you telling me my patch passes in check but will fail in gate pipeline? that would be a pretty bad design IMO | 12:24 |
| sean-k-mooney | no | 12:25 |
| sean-k-mooney | i thnk you test willl fail now that the new verison of sphinx is allwoed | 12:25 |
| frickler | no, it ran after that | 12:25 |
| sean-k-mooney | maybe we are exluding | 12:27 |
| sean-k-mooney | /home/zuul/src/opendev.org/openstack/nova/.tox/docs/lib/python3.10/site-packages/openstackdocstheme/page_context.py:70: RemovedInSphinx90Warning: Sphinx 9 will drop support for representing paths as strings. Use "pathlib.Path" or "os.fspath" instead. | 12:27 |
| sean-k-mooney | src_file = full_src_file[len(str(app.builder.env.srcdir)):].lstrip('/') | 12:27 |
| auniyal | but shouldn't we just delete too, if its problametic, and deleting fixes the issue also, like what this patch is doing https://review.opendev.org/c/openstack/nova/+/927471 | 12:27 |
| sean-k-mooney | form what is fatal | 12:27 |
| auniyal | or it can cause any other error ? | 12:28 |
| frickler | check https://ca0e8b358c93d8e83ff8-d2c9747c19735a1cd74d2b5484a5c4b3.ssl.cf5.rackcdn.com/927471/1/check/openstack-tox-docs/8034b4b/job-output.txt , using sphinx-8.0.2, lots of warning, no failure | 12:28 |
| frickler | the failure is this: WARNING: Aborted attempted copy from /home/zuul/src/opendev.org/openstack/nova/.tox/docs/lib/python3.10/site-packages/sphinx_feature_classification/support-matrix.css to /home/zuul/src/opendev.org/openstack/nova/doc/build/html/_static/support-matrix.css (the destination path has existing data). [misc.copy_overwrite] | 12:30 |
| frickler | build finished with problems, 1 warning. | 12:30 |
| sean-k-mooney | so your current change is passign so im just going to merge it but i think we regruest how our docs job works at some point | 12:31 |
| bhola | Hi. Looking for some help. I have installed openstack following the installation guide on openstack site. Everything seems to be working good but I cannot launch an instance. It throws the error "Block Device Mapping is Invalid: failed to get image" if I use image. If I use pre-defined volume based on image it throws error "Block Device Mapping is Invalid: failed to get volume" | 12:32 |
| bhola | Any idea how to fix this issue? | 12:32 |
| frickler | sean-k-mooney: regruest = regressed? (I can parse most of your typos but not sure on this one) | 12:32 |
| sean-k-mooney | yes regressed | 12:34 |
| sean-k-mooney | we run "sphinx-build -W --keep-going -b html -j auto doc/source doc/build/html" | 12:35 |
| sean-k-mooney | -W is fail-on-warnings | 12:35 |
| auniyal | bhola can you check nova and cinder logs for detailed traceback | 12:37 |
| sean-k-mooney | bhola: as a general rule we do not provide support on this channel. it for development but if you can point ot a spefici error messge then we may be able to point you in the right direction | 12:40 |
| sean-k-mooney | bhola: also without know which version of openstack you installed we cant really say what might be the issue | 12:40 |
| sean-k-mooney | and which distro | 12:41 |
| sean-k-mooney | frickler: there are some followups we likely shoudl do for https://review.opendev.org/c/openstack/nova/+/927471 | 12:46 |
| sean-k-mooney | it should be in .gitignore for one | 12:47 |
| sean-k-mooney | but lest unblock things and do that after | 12:47 |
| bauzas | I was awaiting people to arrive before deciding anything | 12:47 |
| stephenfin | sean-k-mooney: frickler: yeah, https://review.opendev.org/c/openstack/nova/+/927471 looks like the proper approach. I'm +2 on that also. | 12:48 |
| bauzas | I tend to be conservative as we're close to the FF and probably ask ourselves to temporarly avoid the sphinx bump | 12:48 |
| bauzas | Dalmatian is a non-SlURP release, I feel less obliged to have all our libs being upgraded, if we don't really need the upgrade itself | 12:49 |
| bauzas | and other issues may arise | 12:49 |
| sean-k-mooney | i dont think that a good approch to take | 12:49 |
| sean-k-mooney | we shoudl not devalue non-slurp release | 12:50 |
| sean-k-mooney | they are just as imporant as SLURPS | 12:50 |
| bauzas | is there any reason for nova to use sphinx 8.0 ? | 12:50 |
| frickler | IMHO we have the freeze coming up soon in order to still have enough time before the release to fix issues, not in order to avoid bumping things even before that | 12:50 |
| sean-k-mooney | i belive we have been for some time | 12:50 |
| bauzas | sean-k-mooney: I'm not devaluating the non-SLURP releases, I'm just evaluating the risks | 12:50 |
| frickler | why not use the latest and greatest version as we do for everything? | 12:51 |
| sean-k-mooney | bauzas: this is blockign seveeral feature that are close to merging in this release | 12:51 |
| sean-k-mooney | so the trade of to fixing this now is FFE into the early rc period | 12:51 |
| sean-k-mooney | there are several small feature and bugfix pathces that i dont think would be reasonabel to just defer to the next release | 12:52 |
| bhola | auniyal, Just a sec. Let me see in the log. | 12:52 |
| bauzas | sean-k-mooney: for the sake of evaluating the need, which features are depending on the very recent sphinx bump ? (I may have missed them) | 12:52 |
| sean-k-mooney | none | 12:52 |
| sean-k-mooney | but this is currently break our ablity to merge patches | 12:53 |
| bauzas | hence my proposal to skip the bump, for sure | 12:53 |
| stephenfin | bauzas: sphinx isn't a runtime dependency, the docs are not a "feature" (so talk of FFE doesn't make sense), and Nova is doing something wrong and Sphinx is only pointing it out now | 12:53 |
| sean-k-mooney | stephenfin: i ment for all the feature patches that failed to meger last night and that we cant merge today until this is fixed | 12:54 |
| bhola | sean-k-mooney, The versioning system of openstack always confuses me. I think there are different versions of every service. Are you referring to openstack client toll version? | 12:54 |
| bauzas | stephenfin: I'm not against docs, for sure, I'm just evaluating the risk of discovering a late bug that would require some RC2 | 12:54 |
| stephenfin | I don't know if folks like frickler that work on deps would agree, but IMO we should need a *very* good reason to hold back a dep bump | 12:54 |
| sean-k-mooney | bhola: no we have a unified release when you install you normally install that unified release. | 12:54 |
| sean-k-mooney | so while ech componet may have a diffent release number you isntall one release of openstack | 12:54 |
| bauzas | a regression sounds a good reason to me :) | 12:55 |
| stephenfin | bauzas: Right. But again, sphinx is not a runtime dependency. So talk of an RC2 doesn't make any sense. | 12:55 |
| bauzas | this is true, but I wouldn't want our dalmatian branch to be hit by some unknown yet regression | 12:55 |
| bauzas | after RC1, you know that we would have this 2 weeks period when no backports can be done on stables | 12:56 |
| bauzas | anyway, seems to me the ship has sailed. https://review.opendev.org/c/openstack/nova/+/927471 is +Wd | 12:57 |
| frickler | so IMO https://review.opendev.org/c/openstack/nova/+/927471 is a very simple, no-risk fix, and much better than reverting the u-c bump for everyone | 12:57 |
| stephenfin | +1 | 12:58 |
| sean-k-mooney | +1 | 12:58 |
| bauzas | I was just explaining why I didn't reviewed such patch while the gate was blocked : I was awaiting quorum before proceeding | 12:58 |
| stephenfin | #shipit 🚢 | 12:58 |
| sean-k-mooney | you wanted qourm on if we shoudl unblock the gate | 12:58 |
| sean-k-mooney | ? | 12:58 |
| bauzas | no and you know that | 12:58 |
| bauzas | I wanted quorum on the approach we would take on fixing the gate | 12:59 |
| sean-k-mooney | bauzas: if we didnt unblock this in a timely manner i would be asking use to extend our FF deadline into next week | 12:59 |
| bauzas | anyway, 2 +2s and +W, you no longer need me | 12:59 |
| bauzas | sean-k-mooney: I was about to propose that and you know that too | 12:59 |
| sean-k-mooney | no i dont | 13:00 |
| bauzas | then, look at previous FFs | 13:00 |
| sean-k-mooney | bauzas: you were form my perspecitve taking a very non contibutor feindly stance | 13:00 |
| sean-k-mooney | we normally dont extend into next week | 13:00 |
| sean-k-mooney | we normaly say if its +w by friday it can merge until monday | 13:00 |
| bauzas | we had a couple of milestone-3 freaking issues with gate that were providing our code to merge | 13:01 |
| sean-k-mooney | i was suggesting we aloow review to continue until tuesday and +w until firday to give addtional review time | 13:01 |
| bauzas | in the past | 13:01 |
| bauzas | gate stability isn't code review | 13:01 |
| bauzas | anyway, I have many other things to do today so I stop talking | 13:02 |
| bauzas | ok | 13:05 |
| bauzas | (sorry, wrong window) | 13:06 |
| bhola | sean-k-mooney, I am sorry but I must admit that I am new to openstack. I installed it on Ubunto 22.04.4 server version without setting up any special repository. How I can check it? openstack --version returns 5.8.0 | 13:06 |
| bhola | sean-k-mooney, horizon shows 22.1.1 | 13:10 |
| sean-k-mooney | honestly im sorry to say we likely dont have time to help and this proably somethign that hsould be brought up on the mailing list instead | 13:11 |
| sean-k-mooney | bhola: we gennerally recommend isntalling openstack with an installer the ifrst time before trying a manual install | 13:11 |
| sean-k-mooney | if you did the manual instlal there are two many way you could have incorrectly configured or installed it and if your using the disto packages they can have divergance form the upstream release | 13:12 |
| frickler | sean-k-mooney: except our (openstack's) install guide tells a different story | 13:12 |
| frickler | bhola: let's move to #openstack | 13:12 |
| sean-k-mooney | frickler: the one that distors were ment to maintain not the project that were inhired form the docs project | 13:12 |
| frickler | sean-k-mooney: yes, that's why I agree that the discussion should not in this channel | 13:13 |
| frickler | +continue | 13:13 |
| bhola | frickler, I am there now | 13:13 |
| opendevreview | Merged openstack/nova master: Delete ./doc/source/_static/support-matrix.css file https://review.opendev.org/c/openstack/nova/+/927471 | 13:14 |
| sean-k-mooney | frickler: ^ :) | 13:14 |
| frickler | ok, now to recheck stephen's doc tools stack | 13:15 |
| sean-k-mooney | ya ill abandon my targeted patchand we can proceed with his more comprehensive ones | 13:18 |
| gibi | Uggla: bauzas: re further testing the virtiofs series: rescue works as expected, rebuild still needs to be blocked (or fixed not to lose the share). Server events works well. Notification mostly works but shares are always null for the instance.update notification specifically. | 14:30 |
| gibi | left comments inline in the sereis | 14:31 |
| gibi | series | 14:31 |
| Uggla | gibi, thx | 14:31 |
| gibi | I will dig deeper tot he instance.update case as that feels very strange | 14:36 |
| opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | 14:37 |
| opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | 14:38 |
| opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | 14:39 |
| dansmith | guh, sorry for the noise | 14:39 |
| sean-k-mooney | has the missing client support landed | 14:39 |
| dansmith | yes | 14:40 |
| sean-k-mooney | cool, looks like that green before the rebase so ill take a look at it now | 14:40 |
| dansmith | well, before it was running without the change | 14:40 |
| dansmith | but the DNM on top was including it | 14:41 |
| sean-k-mooney | ok i tough it had a depend on before | 14:41 |
| dansmith | we had to test with and without the glanceclient change and with and without the actual server-side change, | 14:41 |
| dansmith | hence the two patches and complex deps linkage | 14:41 |
| dansmith | so this next run will be the first with updated server and client from package, etc | 14:41 |
| dansmith | but it should work the same as the top patch did before | 14:41 |
| sean-k-mooney | ack, but we expect that to be thet same as with the depends on more or less | 14:42 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/891036/14..17//COMMIT_MSG | 14:42 |
| dansmith | yep, just sayin' | 14:42 |
| bauzas | Uggla: ack, for now, I'm giving some chance to other patches series to land before FF | 14:43 |
| bauzas | those have been left overdue for too long | 14:43 |
| bauzas | folks, I'm also considering an extra day before calling FF | 14:43 |
| bauzas | I was less oriented to reviews this week and I was mostly off the whole August timeframe | 14:44 |
| sean-k-mooney | there are also man other that have been but we can do our bset to get what wwe can over the line | 14:54 |
| *** bauzas_ is now known as bauzas | 14:56 | |
| melwitt | frickler: thanks for finding a better way to fix it. I had wondered if something like that could work but wasn't sure | 14:56 |
| *** __ministry is now known as Guest1815 | 15:00 | |
| opendevreview | Elod Illes proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | 15:27 |
| opendevreview | Elod Illes proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | 15:39 |
| *** __ministry is now known as Guest1822 | 17:12 | |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | 17:27 |
| opendevreview | Merged openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | 18:25 |
| *** bauzas_ is now known as bauzas | 20:34 | |
| opendevreview | Merged openstack/nova master: api: Fix typo https://review.opendev.org/c/openstack/nova/+/924722 | 21:22 |
| opendevreview | Merged openstack/nova master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/nova/+/899753 | 21:22 |
| opendevreview | Merged openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | 23:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!