| *** rlandy is now known as rlandy|out | 00:50 | |
| opendevreview | Ian Wienand proposed opendev/gerrit master: Retire repo https://review.opendev.org/c/opendev/gerrit/+/833943 | 03:20 |
|---|---|---|
| ianw | for reference, this is the list of branches to remove from opendev/gerrit, as discussed in the last meeting https://paste.opendev.org/show/b9Ic1Cpj55IM6arWe2ek/ | 03:31 |
| opendevreview | Merged opendev/gerrit master: Retire repo https://review.opendev.org/c/opendev/gerrit/+/833943 | 04:01 |
| ianw | ok, repo is retired now. we'll keep an eye if we need to purge those changes from master; if you end up at opendev.org when looking at gerrit things that would be a good data point | 04:26 |
| opendevreview | Dr. Jens Harbott proposed openstack/diskimage-builder master: Test building jammy https://review.opendev.org/c/openstack/diskimage-builder/+/836228 | 05:55 |
| frickler | infra-root: seems the twitter part of status bot hasn't been working for almost a year, is that something we still care about? otherwise we might want to mark it as abandoned and remove references from our docs https://twitter.com/openstackinfra | 06:00 |
| frickler | https://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing | 06:00 |
| ianw | huh, i did think that it was working post containerisation, but admit i don't keep close tabs | 06:10 |
| ianw | File "/usr/local/lib/python3.8/site-packages/twitter/api.py", line 4907, in _ParseAndCheckTwitter | 06:17 |
| ianw | raise TwitterError({'Unknown error': '{0}'.format(json_data)}) | 06:17 |
| ianw | it probably died post python3 upgrade | 06:17 |
| ianw | looks like it's this bug | 06:22 |
| ianw | https://github.com/bear/python-twitter/issues/695 | 06:22 |
| frickler | there is a bug in gerrit regarding to attention sets: I'm marked as needing to look at https://review.opendev.org/c/zuul/nodepool/+/836263 since I uploaded a previous PS, but since I'm not the current uploader, I cannot remove myself any longer, so this patch is stuck in my "Your Turn" list | 06:22 |
| frickler | o.k., manually adding myself to the CC: list seems to be a workaround | 06:24 |
| frickler | ianw: so it looks like there isn't a solution yet? | 06:25 |
| ianw | looks like all attention has moved to tweepy | 06:28 |
| frickler | seems etherpad just went down? checking the server now | 06:36 |
| frickler | looks like log rotation hickup, came back without me actually doing anything except checking the docker log | 06:50 |
| *** icey_ is now known as icey | 06:59 | |
| ttx | Looks like ethercalc is down | 07:22 |
| ttx | frickler, ianw: that prevents access to the PTG host keys | 07:22 |
| ttx | (https://ethercalc.openstack.org/ returns 503) | 07:23 |
| *** jpena|off is now known as jpena | 07:37 | |
| ianw | hrm, let me see | 07:37 |
| ianw | ttx: ok now? | 07:40 |
| ttx | yes | 07:40 |
| ttx | thanks! | 07:40 |
| ianw | when i logged in there was no ethercalc node service running | 07:41 |
| ianw | Apr 05 06:30:37 ethercalc02 bash[3586]: /opt/ethercalc/node_modules/redis/index.js:602 | 07:44 |
| ianw | Apr 05 06:30:37 ethercalc02 bash[3586]: throw err; | 07:44 |
| ianw | that's where it died | 07:44 |
| ttx | famous last words | 08:05 |
| frickler | ttx: should be stable again now, please let us know if you see further issues | 08:20 |
| *** ysandeep is now known as ysandeep|lunch | 08:28 | |
| ttx | frickler: thanks! | 08:36 |
| *** ysandeep|lunch is now known as ysandeep | 09:00 | |
| *** pojadhav is now known as pojadhav|afk | 09:50 | |
| *** pojadhav|afk is now known as pojadhav | 10:12 | |
| *** rlandy|out is now known as rlandy | 10:27 | |
| fungi | puppet also would have restarted it on the next periodic deploy run, since the service resource has an ensure=>running | 11:11 |
| fungi | unfortunately it crashes a lot | 11:11 |
| opendevreview | Andy Ladjadj proposed zuul/zuul-jobs master: Update upload-logs azure role to support endpoint override https://review.opendev.org/c/zuul/zuul-jobs/+/836640 | 12:04 |
| opendevreview | Andy Ladjadj proposed zuul/zuul-jobs master: Update upload-logs azure role to support endpoint override https://review.opendev.org/c/zuul/zuul-jobs/+/836640 | 12:06 |
| opendevreview | Andy Ladjadj proposed zuul/zuul-jobs master: Update upload-logs azure role to support endpoint override https://review.opendev.org/c/zuul/zuul-jobs/+/836640 | 12:27 |
| *** artom__ is now known as artom | 13:28 | |
| zigo | There's no release notes for Yoga online, at least for puppet-keystone: https://docs.openstack.org/releasenotes/puppet-keystone/unreleased.html | 13:29 |
| zigo | How can this be fixed? | 13:29 |
| fungi | if a change merges for puppet-keystone, the release notes job should run again and publish content there. did the 20.3.0 tag not upload any release notes? i'll double-check whether that job ran | 13:34 |
| fungi | looks like it ran successfully: https://zuul.opendev.org/t/openstack/build/ab6f733179c34a6699f3d5e5fa78e17c | 13:36 |
| fungi | i think it may be a problem with the design of the job. there was a yoga releasenotes page but the publication of the 20.3.0 tag's releasenotes deleted it: https://zuul.opendev.org/t/openstack/build/ab6f733179c34a6699f3d5e5fa78e17c/console#4/0/7/localhost | 13:38 |
| fungi | https://review.opendev.org/833960 was the release request which triggered that | 13:39 |
| fungi | looks like the stable/yoga branch was created at the same time as the tag, so i wonder if there's a race with the new branch if the tag is pushed at the same moment it's created | 13:41 |
| fungi | zuul did provide a stable/yoga branch for the puppet-keystone repo though: https://zuul.opendev.org/t/openstack/build/ab6f733179c34a6699f3d5e5fa78e17c/console#0/4/12/ubuntu-focal | 13:43 |
| fungi | looks like the yoga releasenotes should automatically get recreated once the next change merges for that repo, but it would be good to figure out why reno didn't build yoga release notes in that run | 13:46 |
| Clark[m] | fungi: re tox and use develop the package is installed by default without use develop. https://tox.wiki/en/latest/config.html#conf-skip_install you need to set skip_install to skip that | 14:15 |
| fungi | Clark[m]: it really isn't though. if skipsdist is true and usedevelop is false or unset, the resulting env doesn't have the project in it unless you explicitly add it to deps (i just tested again with latest tox to confirm my recollections) | 14:28 |
| fungi | i have a project where i'm obsessive about not installing it into any tox testenvs which don't absolutely require it to be installed (in order to also avoid dragging in its requirements), so for the testenvs where the project does need to be installed i achieve that by explicitly adding {toxinidir} to the deps list in the testenv | 14:30 |
| fungi | the rest don't end up with the projects or its requirements in the resulting venvs | 14:30 |
| Clark[m] | Oh skipsdist is like skip_install then? | 14:40 |
| Clark[m] | I thought it was just to pip install . Rather than the tarball | 14:41 |
| Clark[m] | I think that is a behavior change from when I last looked at these settings (years ago I'm sure) | 14:42 |
| fungi | skipsdist avoids explicitly building an sdist of the project, which would otherwise have a side effect of installing the project | 14:43 |
| fungi | in the example project, i have a separate testenv:dist which i use to test that sdist creation works (as well as wheel building, and performs some cursory analysis of the resulting artifacts) | 14:45 |
| Clark[m] | Right but skip_install is a separate setting and I'm pretty sure that skipsdist only controlled the building of the sdist in the past. If you set it in the past it would still install the project just without a separate sdist step | 14:45 |
| Clark[m] | Which would do whatever pip does to install the project rather than a pip install of the resulting sdist iirc | 14:46 |
| fungi | i actually don't recall that being the case. usedevelop also installs the project, which probably led to some confusion. if skipsdist was true *and* usedevelop was false (or unset) then you didn't get an automatic install of the project, but if at least one of those was not the case then it ended up getting installed | 14:46 |
| fungi | i ran into it back when we were first working on bindep adoption, because i wanted it to be possible to `tox -e bindep` in a nova checkout without nova or its requirements getting installed. unsetting usedevelop and making skipsdist true accomplished that | 14:48 |
| fungi | unfortunately, some time later, nova dropped the skipsdist setting from their tox.ini, so now it's no longer working as intended | 14:50 |
| clarkb | ya testing this it works as you say. Its weird to hvae the separate setting which is cimpletely ignored | 14:50 |
| clarkb | fungi: is it worth noting on the thread that the behavior you describe depends on skipsdist? | 14:51 |
| fungi | they also edited my comment in their tox.ini which said tox.skipsdist = True was necessary :/ | 14:51 |
| clarkb | I guess the assumption is that openstack sets that fairly universally? | 14:51 |
| fungi | https://review.openstack.org/622972 (erroneously) removed skipsdist, and then https://review.opendev.org/684775 later cleaned up the comment which said it was needed | 14:52 |
| fungi | i think there's a lot of misconceptions about how those settings work, leading to projects randomly adding/removing them in hopes the black magic will work | 14:53 |
| fungi | i wouldn't assume anything at this point | 14:53 |
| *** pojadhav is now known as pojadhav|afk | 14:56 | |
| clarkb | fungi: might be worth a followup to the thread then? Just to make it clear that usedevelop=true is only responsible for that behavior when skipsdist is also true. Otherwise default tox behavior can exhibit this as well | 14:56 |
| fungi | yep. i'll try to add something once i get a break | 14:57 |
| clarkb | thanks. Ya I'm about to jump on the call I expect you are on. I just have to figure out how to connect from my phone | 14:58 |
| fungi | i'm joining the qa ptg room for the fips testing discussion, yes | 14:58 |
| slittle | please add me as first member of group starlingx-app-istio-core | 15:03 |
| slittle | I can add the rest after that | 15:03 |
| fungi | slittle: will do, just a moment | 15:04 |
| fungi | slittle: done | 15:06 |
| fungi | clarkb: are you going to be in the qa room for the openstack-health topic? it will probably be when i need to be talking about xstatic packages in the horizon room | 15:39 |
| clarkb | ya I can be | 15:39 |
| clarkb | it was on my rough list of things to be around for | 15:40 |
| fungi | not sure whether they need someone from opendev/tact-sig in there but figured it might be in your wheelhouse | 15:40 |
| fungi | cool, thanks! | 15:40 |
| *** marios is now known as marios|out | 15:47 | |
| opendevreview | Marcin Juszkiewicz proposed opendev/system-config master: mirror-update: added EPEL 9 https://review.opendev.org/c/opendev/system-config/+/836670 | 15:48 |
| *** pojadhav|afk is now known as pojadhav | 15:58 | |
| *** ysandeep is now known as ysandeep|out | 16:03 | |
| opendevreview | Marcin Juszkiewicz proposed opendev/system-config master: mirror-update: added EPEL 9 https://review.opendev.org/c/opendev/system-config/+/836670 | 16:12 |
| *** jpena is now known as jpena|off | 16:31 | |
| clarkb | noting this here but openstack jobs have started to see failures due to the pypi served us a stale index file for $package problem | 18:15 |
| clarkb | it seems to be affecting inmotion iad3, rax iad, and iweb mtl01 (all east coast north america ish) and a number of packages | 18:16 |
| clarkb | at this point based on debugging I've done I don't think any of our services are the source of the problem and instead this is what we've debugged in the past where pypi's CDN will fallback to grabbing info from a backup source that is nowhere near up to date which means newre packages are not present in indexes | 18:17 |
| clarkb | also TIL openstack uses a package called "itsdangerous" | 18:22 |
| fungi | ahh, data integrity | 18:33 |
| *** rlandy is now known as rlandy|mtg | 18:33 | |
| *** rlandy|mtg is now known as rlandy | 19:02 | |
| opendevreview | Merged opendev/system-config master: Finish removing git.openstack.org references https://review.opendev.org/c/opendev/system-config/+/654040 | 19:50 |
| clarkb | fungi: are you aware of wednesday sessions we should be in? I'm planning to do the thursday and friday TC stuff I think | 20:01 |
| clarkb | or at least as much as I can manage with my morning time | 20:01 |
| fungi | clarkb: none i'm aware of, the only thing i have for tomorrow is barbican consumers api discussion since it's related to/blocking the image encryption effort | 20:06 |
| TheSov | trying to use gerrit from my desktop and keeping get public key errors, any idea why that is, i made sure the key matched several times | 20:08 |
| clarkb | TheSov: chances are you are on a system that doesn't allow sha1 + rsa keys (fedora, opensuse tumbleweed, or anything with a very new openssh) | 20:09 |
| TheSov | using ubuntu wsl | 20:09 |
| clarkb | We think this will be fixed in gerrit 3.6 whenever it releases. Until then you can tell openssh to allow sha1 keys or use a non rsa key. We've been suggesting a non rsa key | 20:09 |
| clarkb | https://www.openssh.com/txt/release-8.8 documents allowing sha1 with rsa under Potentially-incompatible changes | 20:10 |
| TheSov | thanks for the direction! | 20:10 |
| TheSov | ill try it | 20:10 |
| *** rlandy is now known as rlandy|mtg | 20:26 | |
| *** dviroel is now known as dviroel|out | 20:36 | |
| opendevreview | Ghanshyam proposed openstack/project-config master: Remove tempest-lib from infra https://review.opendev.org/c/openstack/project-config/+/836703 | 22:45 |
| clarkb | ianw: did you see the change to add epel mirroring for centos 9? https://review.opendev.org/c/opendev/system-config/+/836670 I tried to give it proper review both for afs room and appropritae content. It looks good to me | 22:49 |
| *** rlandy|mtg is now known as rlandy | 23:16 | |
| ianw | clarkb: thanks, will look | 23:21 |
| fungi | ianw: also in case you missed scrollback in #openstack-infra, seems there are problems with the state of the f35 mirror. the refreshes have shown a bunch of churn, so hopefully it'll settle down within the coming hours | 23:26 |
| fungi | might be worth keeping an eye on though | 23:26 |
| ianw | yeah i noticed that, and i agree with your analysis of a partial update | 23:28 |
| ianw | as you say, i'll check in a bit and see if it looks better | 23:28 |
| opendevreview | Ghanshyam proposed openstack/project-config master: Retire openstack-health project: end project gating https://review.opendev.org/c/openstack/project-config/+/836707 | 23:43 |
| clarkb | gmann: you'll need a change in health to update the .zuul.yaml | 23:44 |
| clarkb | I guess we have to do noop jobs in project-config so they apply when we delete the content | 23:44 |
| gmann | clarkb: yeah, I am on the next step to remove the content in repo which need noop job ^^ | 23:45 |
| clarkb | yup I approved it I think single core on that one is fine | 23:46 |
| gmann | +1. thanks. | 23:48 |
| *** rlandy is now known as rlandy|out | 23:55 | |
| clarkb | heh all the extra people in my house will be gone tomorrow but its the one day this week where I don't need todial into ptg calls | 23:58 |
| clarkb | didn't plan that very well | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!