| opendevreview | Merged openstack/releases master: Release sushy 5.3.0 for epoxy https://review.opendev.org/c/openstack/releases/+/933079 | 07:50 |
|---|---|---|
| opendevreview | Merged openstack/releases master: Release cloudkitty for stable/2023.1 https://review.opendev.org/c/openstack/releases/+/925338 | 07:56 |
| opendevreview | Merged openstack/releases master: nova: Release 2023.1 Antelope 27.5.1 https://review.opendev.org/c/openstack/releases/+/932406 | 07:56 |
| opendevreview | Merged openstack/releases master: Add --except-type to list-deliverables command https://review.opendev.org/c/openstack/releases/+/932900 | 08:03 |
| opendevreview | Merged openstack/releases master: [tools] Do not transition tempest plugins https://review.opendev.org/c/openstack/releases/+/911597 | 08:05 |
| slaweq | frickler hi, I think this will be good channel to talk about this rtd thing | 09:28 |
| slaweq | I have access to the admin panel of the tobiko project in rtd | 09:28 |
| slaweq | and webhook works fine if I run it with curl and pass in the req body secret, something like: | 09:29 |
| slaweq | curl -X POST https://readthedocs.org/api/v2/webhook/tobiko/84947/ -d "token=<secret_from_rtd>" | 09:29 |
| slaweq | it is only tobiko job impacted? I see in https://zuul.opendev.org/t/openstack/builds?job_name=trigger-readthedocs-webhook&skip=0 that basically this job is failing for all projects | 09:30 |
| slaweq | and I think it is caused by https://github.com/readthedocs/readthedocs.org/pull/11083 | 09:30 |
| frickler | oh, indeed, it seems to be failing for all repos that use it, last success was a month ago. so actually we should maybe rather discuss this in #opendev | 09:33 |
| frickler | (sorry for channel hopping, but with this information it seems more like some infra issue than release topic) | 09:34 |
| slaweq | yes, it seems like this role which calls this webhook needs to be changed to accept secret token | 09:38 |
| slaweq | it seems this role already supports "integration_token" so I can try to update tobiko job and set it there | 09:44 |
| slaweq | frickler can you take a look if this patch makes sense https://review.opendev.org/c/x/tobiko/+/933248 ? I can't really test it before merging it so I would like to at least know from someone familiar with zuul jobs if that could work | 10:37 |
| frickler | slaweq: iiuc the secret name (and contents, which I can't check) should match the role definition https://zuul-ci.org/docs/zuul-jobs/latest/general-roles.html#rolevar-trigger-readthedocs.rtd_credentials | 11:11 |
| frickler | clarkb: fungi: ^^ maybe you have more experience with this? | 11:13 |
| slaweq | frickler so are you saying that the secret's name should be "rtd_credentials" and then in data I should have "integration_token: ... "? | 12:02 |
| scrungus | hi, would it be possible to get a +2 on this from anyone? https://review.opendev.org/c/openstack/releases/+/931913 | 12:11 |
| ttx | yes, sorry it did not appear in my dashboard due to a bug | 12:34 |
| opendevreview | Merged openstack/releases master: magnum-capi-helm: Add 1.2.0 release https://review.opendev.org/c/openstack/releases/+/931913 | 12:42 |
| scrungus | thanks! | 12:43 |
| frickler | slaweq: that's my understanding, yes | 13:00 |
| slaweq | frickler please check once again when you will have few minutes https://review.opendev.org/c/x/tobiko/+/933248 | 13:06 |
| frickler | slaweq: looks like that name is already defined in project-config. let's try to get help from some other infra-root, I'll ask in #opendev now | 13:11 |
| slaweq | frickler thx | 13:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!