Thursday, 2025-01-09

*** __ministry is now known as Guest538704:26
*** __ministry is now known as Guest539507:25
chandankumarHello 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
chandankumaropenstack-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#L140909:22
chandankumarI just need to install proper zuul github app in openstack-k8s-operators repo.09:22
chandankumarI 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
chandankumarCan you point me to the link of opendev zuul github app? Thank you :-)09:23
chandankumarI had added the depends-on here https://review.opendev.org/c/openstack/watcher/+/938776 but no zuul job have started.09:25
fricklerchandankumar: I think no special app should be needed, let me check the zuul executor log about that change09:28
chandankumarsure!09:30
fricklerinfra-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
fricklerchandankumar: so iiuc that's an issue on our side09:33
fricklerforgot 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
chandankumarfrickler: I see, thank you for the links.09:37
chandankumarIf 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
fricklerchandankumar: 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 #opendev09:50
chandankumarsure!09:52
opendevreviewDmitriy Rabotyagov proposed openstack/project-config master: End gating for qdrouterd role  https://review.opendev.org/c/openstack/project-config/+/93818811:11
opendevreviewDmitriy Rabotyagov proposed openstack/project-config master: Remove qdrouterd role from the config  https://review.opendev.org/c/openstack/project-config/+/93819411:11
noonedeadpunkfolks, 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
fricklernoonedeadpunk: done, thx for the reminder, I missed that the gov change was merged for the last one. config-core ^^13:14
noonedeadpunkgov changes is another thing to push...13:17
opendevreviewMerged openstack/project-config master: End gating for qdrouterd role  https://review.opendev.org/c/openstack/project-config/+/93818813:44
opendevreviewMerged openstack/project-config master: End gating for Freezer DR project  https://review.opendev.org/c/openstack/project-config/+/93818013:44
opendevreviewMerged openstack/project-config master: Remove Murano/Senlin/Sahara OSA roles from infra  https://review.opendev.org/c/openstack/project-config/+/93568313:49
*** haleyb|out is now known as haleyb14:37
fricklernoonedeadpunk: 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=020:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!