*** __ministry is now known as Guest5387 | 04:26 | |
*** __ministry is now known as Guest5395 | 07:25 | |
chandankumar | Hello OpenDev Infra, I want to test https://github.com/openstack-k8s-operators/sg-core/pull/11 via depends-on in openstack watcher code review. | 09:20 |
---|---|---|
chandankumar | openstack-k8s-operators/sg-core is already under untrusted project under github section in project config https://github.com/openstack/project-config/blob/master/zuul/main.yaml#L1409 | 09:22 |
chandankumar | I just need to install proper zuul github app in openstack-k8s-operators repo. | 09:22 |
chandankumar | I was going through this doc https://zuul-ci.org/docs/zuul/latest/drivers/github.html#configure-github But it does not mention about opendev zuul github app name. | 09:23 |
chandankumar | Can you point me to the link of opendev zuul github app? Thank you :-) | 09:23 |
chandankumar | I had added the depends-on here https://review.opendev.org/c/openstack/watcher/+/938776 but no zuul job have started. | 09:25 |
frickler | chandankumar: I think no special app should be needed, let me check the zuul executor log about that change | 09:28 |
chandankumar | sure! | 09:30 |
frickler | infra-root: ^^ this seems to be an issue with the token I generated, I was wondering whether some extra scope would have been needed, but I saw none on the old one. I'm going to generate a new token with public_repo scope now https://paste.opendev.org/show/bT7OhWDH6Bbvzrnrlmo9/ | 09:32 |
frickler | chandankumar: so iiuc that's an issue on our side | 09:33 |
frickler | forgot to include the adjacent traceback in the paste that shows that this is really triggered by the above review. maybe zuul should also have some better error handling there rather than trying to reference a None object? https://paste.opendev.org/show/bhR6r4D0CaeUZHUwyt3l/ | 09:35 |
chandankumar | frickler: I see, thank you for the links. | 09:37 |
chandankumar | If I get it correctly. in opendev, We are using api_token, which does not have proper scope. That's why Zuul is not able to pull the depends-on and it failed. | 09:49 |
frickler | chandankumar: yes, I've updated the token now, but it will require a restart in order to be picked up, will continue that discussion over in #opendev | 09:50 |
chandankumar | sure! | 09:52 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: End gating for qdrouterd role https://review.opendev.org/c/openstack/project-config/+/938188 | 11:11 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Remove qdrouterd role from the config https://review.opendev.org/c/openstack/project-config/+/938194 | 11:11 |
noonedeadpunk | folks, can I please bring your attention to some reviews regarding project retirement: https://review.opendev.org/c/openstack/project-config/+/938188 https://review.opendev.org/c/openstack/project-config/+/938180/ and https://review.opendev.org/c/openstack/project-config/+/935683 | 11:14 |
frickler | noonedeadpunk: done, thx for the reminder, I missed that the gov change was merged for the last one. config-core ^^ | 13:14 |
noonedeadpunk | gov changes is another thing to push... | 13:17 |
opendevreview | Merged openstack/project-config master: End gating for qdrouterd role https://review.opendev.org/c/openstack/project-config/+/938188 | 13:44 |
opendevreview | Merged openstack/project-config master: End gating for Freezer DR project https://review.opendev.org/c/openstack/project-config/+/938180 | 13:44 |
opendevreview | Merged openstack/project-config master: Remove Murano/Senlin/Sahara OSA roles from infra https://review.opendev.org/c/openstack/project-config/+/935683 | 13:49 |
*** haleyb|out is now known as haleyb | 14:37 | |
frickler | noonedeadpunk: oh my, the last ^^ seems to have triggered a whole new bunch of zuul config errors, like https://zuul.opendev.org/t/openstack/config-errors?project=openstack%2Fopenstack-ansible-tests&skip=0 | 20:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!