| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 00:11 |
|---|---|---|
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 00:23 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 00:41 |
| mhu | Clark: fungi got it, I feared it was a load issue, still thought I'd mention it though | 07:08 |
| frickler | maybe one day we'll need to look at terminating tls on the lb. then we could also try and see whether we can balance git client requests differently from others, i.e. have dedicated git workers | 08:37 |
| mhu | how "scandalous" would it be to only allow git+ssh and completely drop git over http? If anonymous access is potentially broken by DoS anyway ... | 08:54 |
| *** liuxie is now known as liushy | 11:21 | |
| mhu | okay, so nevermind my issue ... I think it's not an opendev server load issue. If I run the ls-remote command from the host VM it completes successfully. But it fails from the pod running in microshift ... | 11:44 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 14:56 |
| fungi | frickler: yeah, now that we've blocked direct connections to the backends anyway, i don't think we lose anything by switching from layer 4 to 7 in haproxy | 15:04 |
| zigo | Looks like the merge job of my patch is stuck: https://review.opendev.org/c/openstack/rally-openstack/+/971955 | 15:23 |
| zigo | "Ready to submit" means it should be merging, right? | 15:23 |
| frickler | zigo: it needs a rebase onto the version of the parent that actually merged. the orange "(Merged)" is a slight hint at that | 15:24 |
| zigo | Is there something I should do then? | 15:25 |
| fungi | yeah, gerrit's ui is sadly misleading there | 15:25 |
| fungi | it needs to be rebased onto the current branch tip which contains the version of the parent which actually merged rather than the outdated parent that change's present commit has in its history | 15:26 |
| fungi | zuul tested by rebasing the change onto the current branch tip (because that's what it always does) and that worked and didn't encounter any merge conflict, hence the verified +2 result, but zuul won't rebase your change in gerrit you'd still need to do that and get it reapproved | 15:29 |
| fungi | the way to avoid this in the future is, any time you alter a change in an existing series, make sure to rebase all the changes after it in that series onto the updated version of the change | 15:30 |
| fungi | in this case it looks like andriy rebased your change onto his, but then later updated his change and forgot to rebase yours again when he did so | 15:33 |
| zigo | Thanks. I've hit the rebase button, should be enough then. | 15:51 |
| clarkb | it may need to be reapproved after rebasing too | 15:57 |
| clarkb | I've got a morning meeting, but will work on a meeting agenda for later today in about an hour | 16:01 |
| clarkb | I do plan to host a meeting and start to catch up on everything that may have been missed over the last coupel of weeks | 16:01 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 16:12 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 16:16 |
| clarkb | I made a few small updates to the agenda (updates for zuul registry prune fixes and opendev matrix move) | 16:38 |
| clarkb | is there anything else that should get added/edited/deleted? | 16:38 |
| clarkb | I'll send that out in about half an hour either way | 16:38 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 16:44 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 16:53 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 17:03 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 17:14 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 17:15 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Cleanup upload-pypi role https://review.opendev.org/c/zuul/zuul-jobs/+/972051 | 17:25 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix(upload-pypi): Ensure twine is installed before using it https://review.opendev.org/c/zuul/zuul-jobs/+/972244 | 17:26 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: refactor: Cleanup upload-pypi role https://review.opendev.org/c/zuul/zuul-jobs/+/972051 | 17:31 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: fix(upload-pypi): Ensure twine is installed before using it https://review.opendev.org/c/zuul/zuul-jobs/+/972244 | 17:31 |
| fungi | https://peps.python.org/pep-0819/ may have some implications for pbr | 17:42 |
| clarkb | pbr already writes a pbr.json file so is again ahead of the game. But likely not in a way that will save it from needing a bunch of updates | 17:43 |
| fungi | yeah | 17:51 |
| fungi | i think in this case setuptools is responsible for writing it anyway, so probably not pbr's job | 17:52 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 18:17 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Cleanup upload-pypi role https://review.opendev.org/c/zuul/zuul-jobs/+/972051 | 18:17 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Cleanup upload-pypi role https://review.opendev.org/c/zuul/zuul-jobs/+/972051 | 18:18 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Ensure twine is installed before using it https://review.opendev.org/c/zuul/zuul-jobs/+/972244 | 18:18 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Ensure twine is installed before using it https://review.opendev.org/c/zuul/zuul-jobs/+/972244 | 18:18 |
| clarkb | how does this look for the matrix move announcement: https://etherpad.opendev.org/p/R_-SxhUgoc5uoFDWuXVr | 21:49 |
| clarkb | infra-root ^ | 21:49 |
| clarkb | oh shoot I forgot about the etherpad upgrade I staged over the holidays. I'm not in a rush to do that but if others can test the held node that would be great | 21:50 |
| corvus | clarkb: lgtm | 21:51 |
| corvus | the note | 21:51 |
| clarkb | thanks! | 21:51 |
| clarkb | I'll send that out around 22:30 if there isn't any other feedback | 21:52 |
| clarkb | should be in your inboxes momentarily | 22:33 |
| fungi | thanks! sorry i didn't get a chance to review but i'm sure it's fine | 22:37 |
| clarkb | ya I think the main thing was to ensure I didn't leave anything super important out. But I got the link to the room and the spec in there and noted you can join the room today | 22:37 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Rewrite upload-pypi test-playbook https://review.opendev.org/c/zuul/zuul-jobs/+/972252 | 22:45 |
| clarkb | apparently Gerrit 3.12 changes the "merged" blob color in gerrit from green to grey to match googles internal code review system | 22:46 |
| fungi | it's always nice for public open source projects to strive for consistency with internal google corporate standards | 22:48 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Cleanup upload-pypi role https://review.opendev.org/c/zuul/zuul-jobs/+/972051 | 23:48 |
| opendevreview | Damian Fajfer proposed zuul/zuul-jobs master: Ensure twine is installed before using it https://review.opendev.org/c/zuul/zuul-jobs/+/972244 | 23:48 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!