| opendevreview | Thierry Carrez proposed openstack/project-config master: Update openstack-mirroring secrets https://review.opendev.org/c/openstack/project-config/+/973060 | 10:40 |
|---|---|---|
| opendevreview | Thierry Carrez proposed openstack/project-config master: Update openstack-mirroring secrets https://review.opendev.org/c/openstack/project-config/+/973060 | 10:43 |
| opendevreview | Merged openstack/project-config master: Update openstack-mirroring secrets https://review.opendev.org/c/openstack/project-config/+/973060 | 11:08 |
| frickler | github mirror job for ^^ passed, so at least this part is fine and working https://zuul.opendev.org/t/openstack/build/e942db0dfb7e49cba64d3f896dba7c3a | 11:38 |
| *** dmellado4 is now known as dmellado | 11:41 | |
| fungi | yay! | 13:16 |
| fungi | 973029 should hopefully get proposal and release jobs working too | 13:17 |
| fungi | once that merges i'll work with the openstack release team on a test run | 13:17 |
| *** mrunge_ is now known as mrunge | 13:48 | |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Update api_token for pypi access https://review.opendev.org/c/openstack/project-config/+/973095 | 13:49 |
| opendevreview | Merged openstack/project-config master: Update api_token for pypi access https://review.opendev.org/c/openstack/project-config/+/973095 | 14:09 |
| opendevreview | Merged openstack/project-config master: Refresh Gerrit SSH for proposal and release bots https://review.opendev.org/c/openstack/project-config/+/973029 | 14:26 |
| opendevreview | Michal Nasiadka proposed openstack/project-config master: kolla: Introduce a version of Ironic core two tier structure https://review.opendev.org/c/openstack/project-config/+/972508 | 15:16 |
| mnasiadka | frickler: updated ^^ - but still I’m puzzled why removelabel entries do not work... | 15:20 |
| mnasiadka | https://review.opendev.org/Documentation/access-control.html#category_remove_label | 15:20 |
| fungi | mnasiadka: we would need to add that as an allowed directive in our acl linter | 15:23 |
| fungi | we're careful about which gerrit features we let projects enable, since we want to make sure we're comfortable supporting them | 15:23 |
| fungi | you'll want to propose the addition to https://opendev.org/openstack/project-config/src/commit/3d06ad5/tools/normalize_acl.py#L192-L201 | 15:25 |
| fungi | or maybe the section above that | 15:25 |
| fungi | we do have removeLabel-Review-Priority allowed, for example | 15:26 |
| fungi | oh, that's what you're adding... | 15:27 |
| fungi | i guess i'm confused as to what the question is | 15:28 |
| mnasiadka | fungi: it’s added in the acl linter | 15:31 |
| mnasiadka | It seems it’s a UI bug | 15:31 |
| mnasiadka | Because I could remove Review-Priority using API | 15:31 |
| mnasiadka | (See https://review.opendev.org/c/openstack/kolla-ansible/+/922098) | 15:31 |
| mnasiadka | But it seems I’m not alone - https://issues.gerritcodereview.com/issues?q=removeLabel | 15:33 |
| fungi | ah, okay. i was going to ask whether you force-refreshed your browser tab after the acl change, but sounds like you did and it's a legit gerrit bug | 15:34 |
| opendevreview | Michal Nasiadka proposed openstack/project-config master: kolla: Introduce a version of Ironic core two tier structure https://review.opendev.org/c/openstack/project-config/+/972508 | 15:34 |
| fungi | one of those issues calls out the suspected code, i suppose that would be a low-hanging fix for someone wanting to get some experience hacking on upstream gerrit | 15:36 |
| mnasiadka | Yeah, it’s been fixed in 3.13 - https://gerrit-review.googlesource.com/c/gerrit/+/512902 | 15:41 |
| mnasiadka | Asked if they plan to backport it to 3.12/3.11 | 15:48 |
| clarkb | we can theoretically manage patches like that in our local builds, but we've been wary of doing so since we finally got away from running a forked gerrit | 15:49 |
| clarkb | in this case backporting upstream patches is probably better than carrying our own patches like we did but still | 15:49 |
| mnasiadka | Well, API works, so no rush - and only Kolla has it in acls :) | 15:51 |
| fungi | stepping out to lunch, bbiab | 15:51 |
| clarkb | are there any outstanding secret rotations that need to be reviewed? | 15:52 |
| clarkb | looks like the main ones from the weekend have been done | 15:52 |
| clarkb | I'm rechecking https://review.opendev.org/c/zuul/zuul-jobs/+/680178 again to see how swift in rax handles log uploads. Hoping we can reenable that cloud today for log uploads | 15:54 |
| clarkb | zuul's components list reports version 13.1.1.dev2 for all components which implies to me that our normal upgrade and reboot system worked over the weekend | 16:08 |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Update pypi api_token https://review.opendev.org/c/openstack/project-config/+/973113 | 16:13 |
| clarkb | infra-root 680178 reported success and I can see logs in its build reports. I think we should go ahead and land https://review.opendev.org/c/opendev/base-jobs/+/972756 to put rackspace swift back into the log upload rotation | 16:45 |
| frickler | ack, approved | 16:49 |
| frickler | can I get a second review on https://review.opendev.org/c/openstack/project-config/+/973113, too? | 16:50 |
| frickler | I think we can reenqueue the failed release test after that merges | 16:51 |
| clarkb | frickler: I went ahead and approved it since it is hard to review those changes beyond "shape is correct" | 16:52 |
| clarkb | as you mention the reenqueue to test it is the real check so lets get to that point | 16:52 |
| clarkb | so far fungi's test IP is the only one to hit the honeypot on docs.opendev.org. I'm thinking that robots.txt may not be a sufficient trap or maybe crawlers are just less interested in docs.opendev.org content. I think we have two options for next steps. The first is expand this to somewhere we have had problems with crawlers (lists?) the other is updating docs.opendev.org content | 16:55 |
| clarkb | with a more attractive trap | 16:55 |
| clarkb | we can discuss that in our meeting tomorrow | 16:55 |
| opendevreview | Merged opendev/base-jobs master: Revert "Disable job log uploads to rax swift" https://review.opendev.org/c/opendev/base-jobs/+/972756 | 16:59 |
| *** darmach2 is now known as darmach | 17:09 | |
| opendevreview | Merged openstack/project-config master: Update pypi api_token https://review.opendev.org/c/openstack/project-config/+/973113 | 17:12 |
| opendevreview | Takashi Kajinami proposed openstack/project-config master: Migrate base propose-updates job for p-o-i to Jammy https://review.opendev.org/c/openstack/project-config/+/961475 | 17:24 |
| opendevreview | Takashi Kajinami proposed openstack/project-config master: Migrate base propose-updates job to Jammy https://review.opendev.org/c/openstack/project-config/+/961475 | 17:26 |
| clarkb | I've done a first pass update on the meeting agenda. Let me know if there is naything else to call out that I haven't added | 18:26 |
| clarkb | I'm going to pop out for a bike ride but when I get back I'll get the meeting agenda sent (so let me know about any needed updates or feel free to make them) and catch up on any more secret rotations that may need reviews | 20:45 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!