opendevreview | Boushra Sondos Bettir proposed openstack/ironic master: [WIP] Currently working on adding support for OVN type switches which Neutron already supports. https://review.opendev.org/c/openstack/ironic/+/900568 | 00:49 |
---|---|---|
opendevreview | Adam Rozman proposed openstack/ironic-python-agent master: fix multipathd error handling release notes https://review.opendev.org/c/openstack/ironic-python-agent/+/900575 | 07:54 |
rpittau | good morning ironic, happy Friday! o/ | 08:00 |
opendevreview | Verification of a change to openstack/ironic master failed: Make sqlalchemy-2x job voting again https://review.opendev.org/c/openstack/ironic/+/900537 | 08:47 |
rpittau | TheJulia. JayF, dtantsur, re: bugfix branches, reading through the chat logs, I'm ok with the retirement/delete procedure, we need to document that somewhere, can look into it, and of course talk to the release team, also to get a confirmation that we're not missing something | 08:53 |
rpittau | also probably would be good to track this in launchpad | 08:58 |
rpittau | I guess we don't care about renaming the branches as we're just removing them | 09:21 |
rpittau | wondering if we should add a "step 0" sending a mail informing that we're removing the bugfix branches, and probably better move the removal of the open changes as "step 1" | 09:33 |
opendevreview | Verification of a change to openstack/ironic master failed: Make sqlalchemy-2x job voting again https://review.opendev.org/c/openstack/ironic/+/900537 | 11:03 |
rpittau | we can probably use this https://opendev.org/openstack/project-config/src/branch/master/roles/copy-release-tools-scripts/files/release-tools/eol_branch.sh to remove the eol tagged bugfix branches | 11:55 |
rpittau | now I just need to understand how to add the tag in a proper way, I guess git push won't work and ui neither :) | 11:56 |
dtantsur | rpittau: probably via the release tooling? | 12:01 |
iurygregory | morning Ironic | 12:45 |
iurygregory | happy friday | 12:45 |
rpittau | dtantsur: yes, I was looking at the repos but no luck finding something related :/ | 13:13 |
TheJulia | good morning | 13:52 |
rpittau | I asked in release and found a promising answer :) | 13:57 |
TheJulia | I guess my huge concern anything we're documenting as process introduces risk for human error. we ideally need to be "one step" and move on so human interaction is minimal and the tooling enforces. At least, that is my opinion | 13:58 |
opendevreview | Verification of a change to openstack/ironic master failed: Make sqlalchemy-2x job voting again https://review.opendev.org/c/openstack/ironic/+/900537 | 13:59 |
rpittau | TheJulia: I agree with you, I just think that even if we automate the process we need to document the workflow in case automation for some reason fails, and also to provide understanding of the process itself | 13:59 |
TheJulia | oh, yes | 13:59 |
TheJulia | agree | 13:59 |
TheJulia | detail what we expect and ensure reality matches | 14:00 |
rpittau | exactly | 14:00 |
rpittau | there are also some prerequisites that may not be obvious, like the need of an openpgp key | 14:00 |
rpittau | anyway, now I'm much more confident about the process and what we need to provide at least for the correct workflow, I think I can fix the bugfix branches that need eol/delete and provide the documentation during the next week | 14:04 |
rpittau | then work on the automation | 14:04 |
TheJulia | rpittau: I'd almost say the automation needs to be the focus to do the cleanup, just from a standpoint of if it cleans it up then we know it is in a good-ish shape process wise, where as manual cleanup is not ideal at the moment | 14:55 |
TheJulia | ... I'm surprised by the pgp key comment, I mean it makes sense if we are doing the tag,but if the tooling is, then that would be surprising | 14:55 |
rpittau | TheJulia: tags need to be signed tags | 14:56 |
TheJulia | cannot the infra scripts do that today when executing on the configuration update in the CI pipeline? | 14:59 |
opendevreview | Julia Kreger proposed openstack/ironic master: Change snmp job to not use a focal node https://review.opendev.org/c/openstack/ironic/+/893824 | 15:10 |
rpittau | AFAICS the infratools do not create a tag unless you also create or modify a branch, and we don't want to have new branches or pollute the yaml files with tons of bugfix eol entries | 15:14 |
rpittau | We could do like we do for stable branches, so just change the name of the branch adding eol, but my understanding is that we just want to add the tag | 15:15 |
rpittau | And I don't think we should delegate infratools for that but have an ironic specific tool, that can take advantage of some infratools functions | 15:15 |
TheJulia | Perhaps this is a better topic to discuss with the release team, because I think "our own tooling" risk creating the exact same problem we're trying to avoid, and perhaps an entirely uniform automated process migth, indeed, be better | 15:32 |
rpittau | Sure, even though we're not moving at all from the release process, the tool will just mark eol and delete the branch, that's it | 15:36 |
rpittau | Anyway, I need to leave now, have to drive for a bit, have a great weekend everyone! o/ | 15:37 |
opendevreview | Julia Kreger proposed openstack/sushy master: Adds an option for setting the http boot uri https://review.opendev.org/c/openstack/sushy/+/718276 | 16:18 |
TheJulia | rpittau: I guess maybe actually talking about it might be best, because I'm worried about any manual process at this point | 16:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!