| *** amoralej|off is now known as amoralej | 07:21 | |
| opendevreview | Joel Capitao proposed openstack/releases master: Create release 1.5.1 for puppet-pacemaker https://review.opendev.org/c/openstack/releases/+/867766 | 08:24 |
|---|---|---|
| opendevreview | Merged openstack/releases master: [manila] Transition stable/stein to eol https://review.opendev.org/c/openstack/releases/+/867548 | 09:19 |
| opendevreview | Merged openstack/releases master: [manila-ui] Transition stable/stein to eol https://review.opendev.org/c/openstack/releases/+/867550 | 09:19 |
| opendevreview | Merged openstack/releases master: [python-manilaclient] Transition stable/stein to eol https://review.opendev.org/c/openstack/releases/+/867552 | 09:23 |
| opendevreview | Merged openstack/releases master: [kolla] release Zed rc2 https://review.opendev.org/c/openstack/releases/+/867629 | 09:24 |
| opendevreview | Bartosz Bezak proposed openstack/releases master: [kolla] final Zed release https://review.opendev.org/c/openstack/releases/+/867660 | 09:24 |
| opendevreview | Merged openstack/releases master: [networking-ovn] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/865967 | 09:25 |
| opendevreview | Merged openstack/releases master: [mistral-tempest-plugin] Tag ussuri-last & victoria-last https://review.opendev.org/c/openstack/releases/+/864349 | 09:27 |
| opendevreview | Joel Capitao proposed openstack/releases master: Create release 1.5.1 for puppet-pacemaker https://review.opendev.org/c/openstack/releases/+/867766 | 10:06 |
| opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic 21.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867772 | 10:08 |
| opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic-inspector 11.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867773 | 10:09 |
| opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic-python-agent 9.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867774 | 10:10 |
| *** dviroel|out is now known as dviroel|rover | 11:12 | |
| ttx | the new Gerrit dashboard no longer shows PTL+1 which is kinda annoying | 12:37 |
| elodilles | oh, indeed :S | 12:39 |
| ttx | I bet there is a way to get it back but it's not obvious... Maybe fungi knows | 12:39 |
| *** gthiemon1e is now known as gthiemonge | 12:41 | |
| fungi | elodilles: ttx: you can still filter queries on it, but gerrit 3.6 only shows columns for "required" labels in its summary view. we think a workaround would be to make that label required in the acl (for example, you could require that it's non-negative and then not give anyone the ability to set a negative value for it) | 12:45 |
| ttx | hah interesting | 12:46 |
| ttx | let me try that | 12:46 |
| ttx | Gerrit doc is not super clear | 12:50 |
| ttx | ok I think I got it | 12:56 |
| *** amoralej is now known as amoralej|lunch | 13:00 | |
| ttx | fungi: hopefully that is enough for gerrit to consider it "required"? | 13:01 |
| ttx | https://review.opendev.org/c/openstack/project-config/+/867801 | 13:01 |
| ttx | otherwise I'll probably have to understand how submit requirements work, and that's not super clear to me | 13:02 |
| fungi | ttx: yeah i think that's the approach clarkb was suggesting to try yesterday. hopefully he can confirm once awake/around | 13:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: [OpenStackAnsible] Release openstack-ansible for Zed and cut branch https://review.opendev.org/c/openstack/releases/+/864509 | 13:15 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: Release OpenStack-Ansible Zed https://review.opendev.org/c/openstack/releases/+/867803 | 13:18 |
| *** akekane is now known as abhishekk | 13:45 | |
| opendevreview | Bartosz Bezak proposed openstack/releases master: [kolla] Release kayobe for Zed and cut branch https://review.opendev.org/c/openstack/releases/+/864506 | 14:05 |
| *** amoralej|lunch is now known as amoralej | 14:08 | |
| opendevreview | Merged openstack/releases master: [kolla] final Zed release https://review.opendev.org/c/openstack/releases/+/867660 | 14:59 |
| opendevreview | Merged openstack/releases master: Release ironic-python-agent 9.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867774 | 15:20 |
| clarkb | ttx: fungi yup that is what I had in mind to test | 15:25 |
| clarkb | I've approved it. Lets see if it works | 15:25 |
| ttx | crossing fingers emoji | 15:36 |
| opendevreview | Merged openstack/releases master: Release ironic-inspector 11.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867773 | 15:59 |
| opendevreview | Merged openstack/releases master: Release ironic 21.2 bugfix for antelope https://review.opendev.org/c/openstack/releases/+/867772 | 16:06 |
| clarkb | fungi: ttx https://review.opendev.org/q/project:openstack/releases I'm not sure that fixed it :( the deploy job claims it applied the updated acl. But may be we need to confirm that | 16:10 |
| clarkb | the set of functions that trip the behavior may be more limited than just having the ability to block | 16:15 |
| clarkb | https://groups.google.com/g/repo-discuss/c/F3H8ks7QS3Q/m/bj3Bm_lWBQAJ seems to show this is possible (this is the thread I found yesterday that made my more confident this work around would work) | 16:16 |
| ttx | hm | 16:29 |
| ttx | it might actually need a new change to enter the pipe | 16:32 |
| ttx | so we'll give ait some time | 16:33 |
| *** amoralej is now known as amoralej|off | 16:36 | |
| clarkb | oh that could be | 16:37 |
| clarkb | to trigger indexing actions | 16:37 |
| ttx | let's pause for a while and see :) | 16:38 |
| rosmaita | elodilles: question about cinderlib release patch https://review.opendev.org/c/openstack/releases/+/864504 | 16:59 |
| rosmaita | patch says 4.3.0 ... we're dropping py36 support, should this be 5.0.0? | 17:00 |
| rosmaita | also, can i cut the stable/zed branch on that patch, or do you prefer it to be done separately? | 17:00 |
| rosmaita | (we're still about 45 min away from everything merging) | 17:01 |
| clarkb | rosmaita: you should probably clean up https://opendev.org/openstack/cinderlib/src/branch/master/setup.cfg#L23 ? | 17:11 |
| clarkb | https://opendev.org/openstack/cinderlib/src/branch/master/setup.cfg#L11 too? | 17:12 |
| rosmaita | clarkb: thanks for looking, that's done by https://review.opendev.org/c/openstack/cinderlib/+/848846 (still in the gate) | 17:19 |
| clarkb | ah | 17:19 |
| rosmaita | (yeah, we are running late!) | 17:19 |
| elodilles | rosmaita: yes, if you drop support of py36 then please bump MAJOR version | 17:29 |
| rosmaita | elodilles: ty, will do | 17:29 |
| elodilles | thanks too for doing it proper :) | 17:29 |
| opendevreview | Brian Rosmaita proposed openstack/releases master: [cinder] Release cinderlib for Zed and cut branch https://review.opendev.org/c/openstack/releases/+/864504 | 17:56 |
| opendevreview | Jay Faulkner proposed openstack/releases master: EOL Metalsmith for stein https://review.opendev.org/c/openstack/releases/+/867891 | 19:09 |
| *** dviroel|rover is now known as dviroel|rover|afk | 21:16 | |
| *** blarnath is now known as d34dh0r53 | 22:29 | |
| *** sfinucan is now known as stephenfin | 22:55 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!