| *** dmellado9 is now known as dmellado | 04:03 | |
| *** dmellado0 is now known as dmellado | 04:11 | |
| *** dmellado8 is now known as dmellado | 04:46 | |
| *** dmellado3 is now known as dmellado | 05:46 | |
| *** jroll01 is now known as jroll0 | 08:55 | |
| opendevreview | Merged opendev/irc-meetings master: Update Cinder weekly team meeting location https://review.opendev.org/c/opendev/irc-meetings/+/964179 | 09:00 |
|---|---|---|
| stephenfin | clarkb: I usually get a load of coding done in downtime during summit, but it was too short this go round 😅 I think we need to add proper PEP-517 support so that we can drop our setup.py files. Before that though, we need to remove our last use of easy_install and pkg_resources. That's happening here https://review.opendev.org/c/openstack/pbr/+/960133 | 09:34 |
| stephenfin | In fact, we might get away without the pyproject.toml stuff for now, since it's explicitly the use of easy_install that setuptools is warning about | 09:34 |
| mattcrees | Hi everyone, I'm trying to submit a patch to ansible-collections-openstack but I'm getting denied when running trying to set up `git review -s`: "mattcrees@review.opendev.org: Permission denied (publickey)." This is only happening on this project in particular, has anyone got any idea why this could be the case? | 10:31 |
| frickler | mattcrees: it is very unlikely for this to happen only for a specific repo. can you retry with a fresh clone and verify you are using your correct gerrit username? | 10:37 |
| stephenfin | fungi: I'm guessing the py27 jobs need to go in pbr? https://review.opendev.org/c/openstack/pbr/+/960133/ If so, is it time to formally drop py27 support? | 10:39 |
| mattcrees | Strangely is does still happen with a fresh clone. I'm definitely using the right username. My colleague just confirmed it works fine for him, probably something else wrong on my end... | 10:40 |
| frickler | mattcrees: maybe run the command with "-v" added and show the (possibly redacted) output on paste.o.o? | 10:41 |
| mattcrees | https://paste.opendev.org/show/bWK50k0joHL6OW3ix4JK/ | 10:45 |
| tonyb | mattcrees: and your certain that ~/.ssh/opendev-ssh-key matches the key you set at https://review.opendev.org/settings/#SSHKeys | 10:47 |
| frickler | mattcrees: that looks like the issue is with the commit hook. I think I've seen that before, but don't remember the solution. what host OS are you using? | 10:48 |
| frickler | you can also try to simple copy the file from some other repo | 10:48 |
| mattcrees | Yes the key definitely matches. I'm using Ubuntu 24.04.3 | 10:49 |
| mattcrees | I tried a fresh checkout of sandbox, it now fails there too, so you're right it's not actually project specific | 10:49 |
| tonyb | what version of git review ? `git review --version` | 10:50 |
| mattcrees | 2.3.1 | 10:50 |
| tonyb | Ah. that's it | 10:51 |
| tonyb | I'll look for the details | 10:51 |
| frickler | mattcrees: ah, right, you need a newer version | 10:52 |
| frickler | https://review.opendev.org/c/opendev/git-review/+/910275 is the relevant change | 10:52 |
| mattcrees | Interesting, looks like I'm on the newest Ubuntu 24.04 has. I see I can install with pip instead, will give it a try | 10:54 |
| tonyb | mattcrees: 2.4.0 or newer | 10:54 |
| tonyb | `pip install --user git-review` is probably safe? | 10:55 |
| mattcrees | That's worked now, cheers guys :) | 10:56 |
| tonyb | mattcrees: See https://opendev.org/opendev/git-review/commit/d633541ecc5cafbf5660e0bf57ecbd96bfb77da0 for a few more details. | 10:56 |
| tonyb | basically OpenSSH deprecated scp | 10:56 |
| mattcrees | Funky | 10:57 |
| frickler | mattcrees: trixie has 2.4.0-1, maybe you want to open a bug against ubuntu to make them sync it | 10:57 |
| frickler | ah, updated in plucky https://launchpad.net/ubuntu/+source/git-review. also interesting list of bugs https://bugs.launchpad.net/ubuntu/+source/git-review | 10:59 |
| tonyb | I bet they can all be closed :) | 11:02 |
| stephenfin | fungi: nvm, I figured it out | 11:50 |
| *** amoralej__ is now known as amoralej | 12:01 | |
| mnasiadka | frickler: what is funny it seems Debian OpenStack (cc: zigo) is maintaining that, maybe there’s an option to ,,backport’’ that to Noble? | 12:41 |
| fungi | mnasiadka: ubuntu has their "sru" (stable release update, i think) process for backporting fixes to existing ubuntu releases | 12:56 |
| mnasiadka | Right, maybe they just don’t know it’s broken on Noble | 12:58 |
| Clark[m] | stephenfin yes I don't think dropping setup.py is necessary for this particular setup tools deprecation pass. We just need to get out of the things that are going away | 13:04 |
| Clark[m] | We should probably focus on the specific things needed to keep working in a week and not worry about setup.py until afterwards | 13:05 |
| stephenfin | right, then what I have up _should_ do the trick | 13:05 |
| stephenfin | (I basically just vendored everything /o\) | 13:05 |
| Clark[m] | Re python 2.7 we can still test it on newer platforms and the swift release which last supported python2.7 is like a year old at this point. I think we're getting close to being able to drop it but if we can keep 2.7 for now that is likely safest | 13:06 |
| Clark[m] | Ok so we should review the stack at 960133 and then get a release out soon? | 13:07 |
| stephenfin | I think so | 13:07 |
| Clark[m] | I can review that in a few hours. Have school related stuff this morning first | 13:07 |
| *** dhill is now known as Guest29731 | 14:19 | |
| fserucas | hi there, is there any problem with centos-stream mirror? | 15:14 |
| fungi | fserucas: maybe start with describing what problem you're observing | 15:17 |
| fungi | but it looks like the centos-stream mirror volume ran out of quota around 13 days ago and hasn't been updating | 15:18 |
| fungi | are you seeing stale content? | 15:18 |
| fungi | https://grafana.opendev.org/d/9871b26303/afs?orgId=1&from=now-30d&to=now&timezone=utc&viewPanel=panel-36 | 15:18 |
| fungi | i'll work on extending the quota now | 15:19 |
| fungi | #status log Increased the mirror.centos-stream AFS volume quota from 400000000 to 500000000 since it filled up around 2025-10-10 | 15:25 |
| fserucas | is seems outdated for 13 days ago | 15:26 |
| fserucas | thank you | 15:27 |
| fungi | yes, i | 15:27 |
| fungi | i'm manually running the update script now to catch it up | 15:28 |
| fungi | looks like our statusbot is also not in here any longer, i'll look into that next | 15:28 |
| fungi | #status log Increased the mirror.centos-stream AFS volume quota from 400000000 to 500000000 since it filled up around 2025-10-10 | 15:30 |
| opendevstatus | fungi: finished logging | 15:34 |
| stephenfin | Clark[m]: https://review.opendev.org/c/openstack/pbr/+/964716 | 17:18 |
| stephenfin | jfyi | 17:18 |
| stephenfin | that's as far as I've gotten | 17:18 |
| stephenfin | *got | 17:18 |
| clarkb | ack (sorry I'm on my irc client now for maximum confusion) | 17:20 |
| clarkb | I'm slowly reviewing the bottom of that stack. Want to diff the vendored code before I +2 that change | 17:20 |
| clarkb | stephenfin: in https://review.opendev.org/c/openstack/pbr/+/960029/3/pbr/_compat/commands.py a longer term followup may be to drop the localinstallscripts as a general thing anyway since pip apparently does that now without the inefficient setuptools imports and entrypoints | 17:23 |
| opendevreview | James E. Blair proposed opendev/system-config master: static: redirect zuulci.org to zuul-ci.org https://review.opendev.org/c/opendev/system-config/+/964296 | 17:27 |
| clarkb | stephenfin: I think I found a bug on https://review.opendev.org/c/openstack/pbr/+/960031 | 17:34 |
| clarkb | stephenfin: ok I think I'm mostly caught up on reviewing that stack. I think things are mostly good until 960031 but did leave comments here and there on the whole stack if you wnt to read them before updating anything | 18:05 |
| fungi | did i happen to review those? it's been a while and i can't remember (trying to dig myself out from under paperwork and post-summit backlog still) | 18:09 |
| clarkb | fungi: no I'm the only reviewer on the latest versions I think | 18:12 |
| fungi | okay, i'll try to remember to get to those, thanks | 18:13 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Find tox/nox siblings with normalized package names https://review.opendev.org/c/zuul/zuul-jobs/+/964724 | 18:17 |
| clarkb | infra-root the openinfra ptg is next week. I'd like to put the meetpad servers in the emergency file to avoid any unexpected container updates to the service during the ptg. Any objections to me doing that now? | 18:27 |
| fungi | no, please do! | 18:28 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Find tox/nox siblings with normalized package names https://review.opendev.org/c/zuul/zuul-jobs/+/964724 | 18:29 |
| clarkb | #status log Added jvb02 and meetpad02 to the emergency file to avoid updates during the PTG | 18:31 |
| opendevstatus | clarkb: finished logging | 18:31 |
| clarkb | rosmaita: fungi I just tested meetpad recording in an up to date chrome installation and it captured both my audio and video | 18:36 |
| clarkb | it does look like you have to ensure that you tell it to capture teh correct tab but it was selected by default when the pop up to allow it came up | 18:36 |
| rosmaita | clarkb: ack | 18:37 |
| clarkb | I guess the other thing to note is that I'm using X windows. I know that wayland impacts screen capturing which I think may eb related to recording as well | 18:37 |
| clarkb | I think the underlying mechanism for sharing your screen is very similar to the mechanism used to record the video | 18:37 |
| rosmaita | ok, i'll see if jbernard can run a test with me and try it out | 18:38 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Find tox/nox siblings with normalized package names https://review.opendev.org/c/zuul/zuul-jobs/+/964724 | 18:44 |
| clarkb | this is what i get for trying to think while jet lagged. Apologies for the noise on issues that shouldn't even happen in the first palce | 18:45 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Force gitea http(s) connectivity through the load balancer https://review.opendev.org/c/opendev/system-config/+/964728 | 21:18 |
| clarkb | feedback welcome this is something we discussed as I was trying to head to the summit. I think the change itself is straightforward but we should consider what the impacts to our ability to manage gitea rae and whether or not the tradeoff is worthwhile | 21:18 |
| tonyb | clarkb: +2 from me | 22:02 |
| tonyb | clarkb: Even with 964724, I'm seeing https://zuul.opendev.org/t/openstack/build/27b40f3997a0431499e5502e0188ddb4/log/job-output.txt#662-680 | 22:10 |
| tonyb | clarkb: Do you have time to look at it or should I? | 22:11 |
| clarkb | tonyb: Ithink that is a separate issue with logging? | 22:43 |
| clarkb | tonyb: it assumes the venv for the env exists. Does that target maybe not use the same venv name? | 22:45 |
| clarkb | I think there is autodetection for that by reading the tox config but maybe something around that is broken? | 22:45 |
| nhicher[m] | tonyb, clarkb: I've proposed a review to use linux bridge instead openvswitch ansible module, could be good if you can review it https://review.opendev.org/c/zuul/zuul-jobs/+/959393 thanks =) | 22:46 |
| tonyb | can see that /home/zuul/src/opendev.org/openstack/requirements/.tox/py310-check-uc/ exists, but make the log subdir /home/zuul/src/opendev.org/openstack/requirements/.tox/py310-check-uc/log/ doesn't? | 22:47 |
| tonyb | nhicher[m]: I did a while ago, but I don't have +2 there | 22:47 |
| tonyb | clarkb: I guess I'm trying to figure a) what makes that dir, and b) what changed to stop it doing so | 22:48 |
| clarkb | tonyb: I think tox does (or did) create that log dir for its own logs | 22:50 |
| clarkb | tonyb: maybe tox is failing to do so due to a failure or it just logs elsewhere now? When we did the nox port for these roles we just stopped doing the logging to disk as nox writes everything to stdout/stderr iirc | 22:51 |
| tonyb | Hmm I'll look at nox as a comparison point | 22:52 |
| clarkb | it certainly looks like tox ran successfully without running any tests | 22:53 |
| tonyb | Yeah, there was a tox release but that was 3 weeks ago so I suspect we would have seen this befoer now | 22:54 |
| clarkb | https://opendev.org/openstack/requirements/src/branch/master/tox.ini I don't see any envlogdir or envdir configs where requirements is overriding things | 22:54 |
| clarkb | https://tox.wiki/en/latest/config.html#env_log_dir seems to imply that tox should still be creating that dir | 22:55 |
| clarkb | maybe it is a permissions thing? though I woudl expect a different error I think for that | 22:55 |
| clarkb | tonyb: for debugging I think you can add a task between the steps that does an ls of the tree under .tox to see what is there | 22:56 |
| clarkb | tonyb: basically after tox without tests runs and before siblings happens | 22:56 |
| tonyb | Yeah. I'll do that. | 22:56 |
| clarkb | nhicher[m]: I'll try to take a look tomorrow /me makes a note now | 22:58 |
| clarkb | we can probably make the tox role more robust by trying to open the file in a try except and if it can't be opened then oh well emit a warning rather than an error situation | 23:01 |
| nhicher[m] | clarkb: sure, no rush, I've proposed it few weeks ago =) | 23:01 |
| tonyb | clarkb: and/or add an os.makedirs(os.path.dirname(log_file)) or similar | 23:02 |
| tonyb | It's only really fatal because it's in a finally clause | 23:03 |
| tonyb | It's gotta be something specific to the requirements repo/check-uc toxenv or we'd be seeing red on every tox job right? | 23:23 |
| clarkb | thats what I'm wondering. I note that you have this set: https://opendev.org/openstack/requirements/src/branch/master/tox.ini#L16-L17 and maybe that causes it to skip enough work between no installs and no tests so it doesn't log anything to the log file? | 23:24 |
| clarkb | tonyb: in aprticular I think those log files may predominantly record the installation steps | 23:24 |
| tonyb | Ah, yeah I guess that could be an unexpected side-effect | 23:25 |
| clarkb | tonyb: https://zuul.opendev.org/t/openstack/build/9736bd34a5b64d32bb4c97b1408745b4/log/tox/functional/1-install_package.log here is an example from the change rosmaita was checking | 23:25 |
| tonyb | That's a solid lead | 23:26 |
| clarkb | tonyb: so maybe in that deps list instead of deps = empty string and no install you could test dropping the no install and setting deps = setuptools or something along those lines | 23:26 |
| tonyb | thanks clarkb | 23:26 |
| clarkb | to force it to do something noopy but just enough work to create logs? | 23:26 |
| clarkb | tonyb: I checked that glance job and tox doesn't seem to log when it creats the log dir so hard to narrow it down that way | 23:36 |
| tonyb | clarkb: I'm confident that you're very close with that guess. It's a fairly substantial difference from all the other tox jobs we run | 23:38 |
| tonyb | clarkb: https://zuul.opendev.org/t/openstack/status?change=964731 hopefully that's helpful. As we have several check jobs I tried a few different things | 23:39 |
| clarkb | cool I guess we wait and see now | 23:42 |
| tonyb | Yup we need to install *something*, now to make it the least possible | 23:45 |
| tonyb | clarkb: I think you should look at https://review.opendev.org/c/opendev/system-config/+/963642 .. re docker not being present | 23:49 |
| Clark[m] | tonyb: aha thanks. I actually think that is better for us as we may be able to drop service disable if the package doesn't install at all. But we need to figure out the paths to the socket and if that works without docker installed. I'll take a look in the morning. Off to dinner now | 23:55 |
| Clark[m] | Oh also the command for compat so ya a few things to figure out before we do real cleanup | 23:56 |
| tonyb | We need /usr/bin/docker or all the ansible tasks where we use 'docker pull' or 'docker inspect' fail | 23:58 |
| tonyb | I tried the "right" thing first and discovered the fallout | 23:59 |
| tonyb | enjoy dinner | 23:59 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!