| *** bauzas2 is now known as bauzas | 00:14 | |
| opendevreview | James E. Blair proposed openstack/project-config master: Fix zuul status node requests graph https://review.opendev.org/c/openstack/project-config/+/953914 | 00:38 |
|---|---|---|
| *** bauzas7 is now known as bauzas | 06:20 | |
| *** bauzas7 is now known as bauzas | 07:34 | |
| opendevreview | Merged openstack/project-config master: Fix zuul status node requests graph https://review.opendev.org/c/openstack/project-config/+/953914 | 09:12 |
| sp-bmilanov1 | Hi Infra Team, I just learned about the new "Signed-off-by" trailer rule for new commits, but that makes backporting (cherry-picking) merged changes from the Gerrit UI impossible (Gerrit rejects the change because "Signed-off-by" is missing). Cherry-picking from Git's CLI works fine though | 14:23 |
| frickler | sp-bmilanov1: ah, yes, I wasn't aware of cherry-picks, but the same applies to rebases of existing changes. I don't think a solution is possible though other than the one you mention | 14:32 |
| fungi | sp-bmilanov1: this was mentioned in the announcement i sent to the mailing list too | 14:37 |
| fungi | it's a temporary situation, new commits merging will have signed-off-by and those should cherry-pick fine | 14:37 |
| fungi | it's just commits that were made and pushed to gerrit before the enforcement switch that you'll need to backport with the cli | 14:38 |
| fungi | maybe quite a few of those right now, but the number should fall fairly quickly | 14:39 |
| sp-bmilanov1 | frickler: fungi: yep, thanks, I found the ML announcement, so all is good | 14:40 |
| fungi | and as covered in the cinder meeting, you *can* use the webui to backport or rebase those, since gerrit gives you an opportunity to edit the commit message before it creates the change | 14:59 |
| fungi | so just add your signed-off-by at that time | 15:00 |
| clarkb | I'm always a little surprised by how many people use the web ui for tasks like that. I must just be stuck in my routine of using git for as much as I can. Really the only time I try to use the web ui is when my ssh keys are not loaded and I'm too lazy to load them for something small | 15:02 |
| fungi | yeah, i really try not to even touch the webui if i can help it | 15:05 |
| opendevreview | James E. Blair proposed openstack/project-config master: Revise zuul-launcher dashboards https://review.opendev.org/c/openstack/project-config/+/953973 | 16:18 |
| opendevreview | Merged openstack/project-config master: Revise zuul-launcher dashboards https://review.opendev.org/c/openstack/project-config/+/953973 | 17:37 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Don't pin setuptools for all tests https://review.opendev.org/c/openstack/pbr/+/953982 | 18:21 |
| opendevreview | Merged openstack/pbr master: tests: Skip tests that break on newer setuptools https://review.opendev.org/c/openstack/pbr/+/953839 | 20:15 |
| opendevreview | Merged openstack/pbr master: tests: Add hint if running integration tests locally https://review.opendev.org/c/openstack/pbr/+/953892 | 20:23 |
| opendevreview | Merged openstack/pbr master: tests: Set GNUPGHOME in tests https://review.opendev.org/c/openstack/pbr/+/953840 | 20:23 |
| opendevreview | Merged openstack/pbr master: Add openstack-tox-py313 job https://review.opendev.org/c/openstack/pbr/+/940117 | 20:27 |
| opendevreview | Merged openstack/pbr master: pre-commit: Bump versions https://review.opendev.org/c/openstack/pbr/+/953893 | 20:27 |
| opendevreview | Merged openstack/pbr master: Don't pin setuptools for all tests https://review.opendev.org/c/openstack/pbr/+/953982 | 21:12 |
| *** prometheanfire is now known as Guest21259 | 23:09 | |
| *** Guest21259 is now known as prometheanfire | 23:14 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!