opendevreview | Tony Breeds proposed openstack/ptgbot master: Set SO_REUSEADDR and SO_REUSEPORT to enable faster service restarts https://review.opendev.org/c/openstack/ptgbot/+/897272 | 01:26 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/897273 | 02:14 |
opendevreview | Tony Breeds proposed openstack/ptgbot master: Set SO_REUSEADDR and SO_REUSEPORT to enable faster service restarts https://review.opendev.org/c/openstack/ptgbot/+/897272 | 06:25 |
opendevreview | Michal Nasiadka proposed openstack/project-config master: Add nested-virt-debian-bookworm https://review.opendev.org/c/openstack/project-config/+/897331 | 12:47 |
*** pmatulis1 is now known as pmatulis | 14:01 | |
opendevreview | Merged openstack/ptgbot master: Fix errors in iCal https://review.opendev.org/c/openstack/ptgbot/+/897269 | 17:18 |
opendevreview | Merged openstack/ptgbot master: Set SO_REUSEADDR and SO_REUSEPORT to enable faster service restarts https://review.opendev.org/c/openstack/ptgbot/+/897272 | 17:18 |
frickler | clarkb: fungi: please check https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/896832 as part of the release housekeeping | 19:18 |
clarkb | approved thanks | 19:22 |
frickler | ty | 19:24 |
frickler | gmann: I didn't see a patch to make tox-py311 voting yet, do you want to do that? | 19:25 |
opendevreview | Merged openstack/openstack-zuul-jobs master: Add stable/2023.2 to periodic-stable templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/896832 | 19:33 |
opendevreview | Merged openstack/openstack-zuul-jobs master: Start translations for 2023.2 (Bobcat) stable branch https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/896416 | 19:33 |
gmann | frickler: I have it up but kept WIP. let me open it for review | 19:36 |
gmann | frickler: this one, please review https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/891238 | 19:37 |
fungi | https://review.opendev.org/896949 is also safe to approve now that the release is done | 19:38 |
fungi | oh, also reviews on https://review.opendev.org/896944 for signing key rotation appreciated | 20:13 |
fungi | wanting to merge that monday | 20:13 |
opendevreview | Vladimir Kozhukalov proposed openstack/project-config master: Add Allow-Post-Review to openstack-helm-core group https://review.opendev.org/c/openstack/project-config/+/897366 | 21:51 |
opendevreview | Vladimir Kozhukalov proposed openstack/project-config master: Add Allow-Post-Review to openstack-helm-core group https://review.opendev.org/c/openstack/project-config/+/897366 | 21:53 |
clarkb | kozhukalov: re ^ are you using the standard container upload jobs/roles? | 21:59 |
clarkb | if not maybe the time and effort is better spent moving to the proven tools than trying to debug something else? | 22:00 |
opendevreview | Vladimir Kozhukalov proposed openstack/project-config master: Add Allow-Post-Review to openstack-helm-core group https://review.opendev.org/c/openstack/project-config/+/897366 | 22:30 |
kozhukalov | @clarkb we have jobs inherited from the standard jobs (using upload-docker-image and promote-docker-image roles). I am trying to understand why this failed https://zuul.opendev.org/t/openstack/build/8efffe85e45f4c2ea04bfff505452c90 | 22:49 |
kozhukalov | however upload was successful. I tried to compare the log with other similar promote jobs that succeeded earlier. Looks like promote job failed for no reason. | 22:51 |
fungi | yeah, unfortunately that failed "promote-docker-image: Get dockerhub JWT token" task has its output censored to avoid exposing credentials | 23:21 |
fungi | it could have simply been a blip in the dockerhub api | 23:22 |
clarkb | I think a good piece of info is knowing whether or not that is a persistent failure (in which case it is likely somethign to do with the credential or its associated permissions) or if it was a one off in which case api failure seems likely | 23:25 |
clarkb | the opendev team just promoted something like 18 images? and that worked so I think their api is generally working | 23:26 |
clarkb | maybe before we resort to changing gerrit acls we can rerun and see if it persists? | 23:27 |
clarkb | also I'm not sure how modifying the acl helps. The acl allows you to run jobs pre merge. But promotion does not make sense pre merge | 23:28 |
fungi | also, if you could somehow attempt that job pre-merge, i guess the intent would be to run a modified copy that provided verbose output of the failing task? which would risk leaking the credentials it used into published logs | 23:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!