| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-29 | |||
| 12:50:16 | sean-k-mooney | they are just as imporant as SLURPS | |
| 12:50:32 | bauzas | is there any reason for nova to use sphinx 8.0 ? | |
| 12:50:51 | 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:52 | sean-k-mooney | i belive we have been for some time | |
| 12:50:56 | bauzas | sean-k-mooney: I'm not devaluating the non-SLURP releases, I'm just evaluating the risks | |
| 12:51:07 | frickler | why not use the latest and greatest version as we do for everything? | |
| 12:51:16 | sean-k-mooney | bauzas: this is blockign seveeral feature that are close to merging in this release | |
| 12:51:33 | sean-k-mooney | so the trade of to fixing this now is FFE into the early rc period | |
| 12:52:14 | 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:33 | bhola | auniyal, Just a sec. Let me see in the log. | |
| 12:52:45 | 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:55 | sean-k-mooney | none | |
| 12:53:03 | sean-k-mooney | but this is currently break our ablity to merge patches | |
| 12:53:20 | bauzas | hence my proposal to skip the bump, for sure | |
| 12:53:24 | 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:54:02 | 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:04 | 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:20 | 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:31 | 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:41 | sean-k-mooney | bhola: no we have a unified release when you install you normally install that unified release. | |
| 12:54:58 | sean-k-mooney | so while ech componet may have a diffent release number you isntall one release of openstack | |
| 12:55:03 | bauzas | a regression sounds a good reason to me :) | |
| 12:55:12 | stephenfin | bauzas: Right. But again, sphinx is not a runtime dependency. So talk of an RC2 doesn't make any sense. | |
| 12:55:58 | bauzas | this is true, but I wouldn't want our dalmatian branch to be hit by some unknown yet regression | |
| 12:56:27 | bauzas | after RC1, you know that we would have this 2 weeks period when no backports can be done on stables | |
| 12:57:31 | bauzas | anyway, seems to me the ship has sailed. https://review.opendev.org/c/openstack/nova/+/927471 is +Wd | |
| 12:57:45 | 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:58:00 | stephenfin | +1 | |
| 12:58:05 | sean-k-mooney | +1 | |
| 12:58:10 | 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:26 | stephenfin | #shipit 🚢 | |
| 12:58:40 | sean-k-mooney | you wanted qourm on if we shoudl unblock the gate | |
| 12:58:44 | sean-k-mooney | ? | |
| 12:58:57 | bauzas | no and you know that | |
| 12:59:11 | bauzas | I wanted quorum on the approach we would take on fixing the gate | |
| 12:59:27 | 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:36 | bauzas | anyway, 2 +2s and +W, you no longer need me | |
| 12:59:59 | bauzas | sean-k-mooney: I was about to propose that and you know that too | |
| 13:00:09 | sean-k-mooney | no i dont | |
| 13:00:19 | bauzas | then, look at previous FFs | |
| 13:00:34 | sean-k-mooney | bauzas: you were form my perspecitve taking a very non contibutor feindly stance | |
| 13:00:43 | sean-k-mooney | we normally dont extend into next week | |
| 13:00:55 | sean-k-mooney | we normaly say if its +w by friday it can merge until monday | |
| 13:01:30 | bauzas | we had a couple of milestone-3 freaking issues with gate that were providing our code to merge | |
| 13:01:33 | sean-k-mooney | i was suggesting we aloow review to continue until tuesday and +w until firday to give addtional review time | |
| 13:01:35 | bauzas | in the past | |
| 13:01:47 | bauzas | gate stability isn't code review | |
| 13:02:00 | bauzas | anyway, I have many other things to do today so I stop talking | |
| 13:05:46 | bauzas | ok | |
| 13:06:01 | bauzas | (sorry, wrong window) | |
| 13:06:56 | 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:10:08 | bhola | sean-k-mooney, horizon shows 22.1.1 | |
| 13:11:01 | 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:36 | sean-k-mooney | bhola: we gennerally recommend isntalling openstack with an installer the ifrst time before trying a manual install | |
| 13:12:07 | 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:18 | frickler | sean-k-mooney: except our (openstack's) install guide tells a different story | |
| 13:12:27 | frickler | bhola: let's move to #openstack | |
| 13:12:45 | sean-k-mooney | frickler: the one that distors were ment to maintain not the project that were inhired form the docs project | |
| 13:13:17 | frickler | sean-k-mooney: yes, that's why I agree that the discussion should not in this channel | |
| 13:13:27 | frickler | +continue | |
| 13:13:40 | bhola | frickler, I am there now | |
| 13:14:07 | opendevreview | Merged openstack/nova master: Delete ./doc/source/_static/support-matrix.css file https://review.opendev.org/c/openstack/nova/+/927471 | |
| 13:14:23 | sean-k-mooney | frickler: ^ :) | |
| 13:15:16 | frickler | ok, now to recheck stephen's doc tools stack | |
| 13:18:32 | sean-k-mooney | ya ill abandon my targeted patchand we can proceed with his more comprehensive ones | |
| 14:30:16 | 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:31:03 | gibi | left comments inline in the sereis | |
| 14:31:05 | gibi | series | |
| 14:31:45 | Uggla | gibi, thx | |
| 14:36:09 | gibi | I will dig deeper tot he instance.update case as that feels very strange | |
| 14:37:42 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:38:47 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:39:15 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:39:18 | dansmith | guh, sorry for the noise | |
| 14:39:59 | sean-k-mooney | has the missing client support landed | |
| 14:40:05 | dansmith | yes | |
| 14:40:30 | sean-k-mooney | cool, looks like that green before the rebase so ill take a look at it now | |
| 14:40:56 | dansmith | well, before it was running without the change | |
| 14:41:05 | dansmith | but the DNM on top was including it | |
| 14:41:16 | sean-k-mooney | ok i tough it had a depend on before | |
| 14:41:18 | dansmith | we had to test with and without the glanceclient change and with and without the actual server-side change, | |
| 14:41:27 | dansmith | hence the two patches and complex deps linkage | |
| 14:41:43 | dansmith | so this next run will be the first with updated server and client from package, etc | |
| 14:41:52 | dansmith | but it should work the same as the top patch did before | |
| 14:42:06 | sean-k-mooney | ack, but we expect that to be thet same as with the depends on more or less | |
| 14:42:10 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/891036/14..17//COMMIT_MSG | |
| 14:42:13 | dansmith | yep, just sayin' | |
| 14:43:24 | bauzas | Uggla: ack, for now, I'm giving some chance to other patches series to land before FF | |
| 14:43:42 | bauzas | those have been left overdue for too long | |
| 14:43:56 | bauzas | folks, I'm also considering an extra day before calling FF | |
| 14:44:36 | bauzas | I was less oriented to reviews this week and I was mostly off the whole August timeframe | |
| 14:54:07 | 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:56:14 | melwitt | frickler: thanks for finding a better way to fix it. I had wondered if something like that could work but wasn't sure | |
| 15:27:54 | 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:55 | opendevreview | Elod Illes proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| 17:27:23 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |
| 18:25:55 | opendevreview | Merged openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | |
| 21:22:21 | opendevreview | Merged openstack/nova master: api: Fix typo https://review.opendev.org/c/openstack/nova/+/924722 | |
| 21:22:42 | opendevreview | Merged openstack/nova master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/nova/+/899753 | |
| 23:15:10 | opendevreview | Merged openstack/python-novaclient master: [doc] change directive noindex to no-index https://review.opendev.org/c/openstack/python-novaclient/+/927519 | |