| *** amoralej|off is now known as amoralej | 07:02 | |
| opendevreview | Marios Andreou proposed openstack/releases master: Release os-net-config for stable/victoria https://review.opendev.org/c/openstack/releases/+/825807 | 07:34 |
|---|---|---|
| opendevreview | Merged openstack/releases master: Release PBR 5.8.1 https://review.opendev.org/c/openstack/releases/+/828016 | 07:37 |
| damani | fungi, i will check | 08:28 |
| opendevreview | Hervé Beraud proposed openstack/releases master: [independent] release oslo deliverables https://review.opendev.org/c/openstack/releases/+/827524 | 09:15 |
| opendevreview | Merged openstack/releases master: Release trove for stable/victoria https://review.opendev.org/c/openstack/releases/+/825827 | 09:29 |
| opendevreview | Merged openstack/releases master: [xena] release oslo deliverables https://review.opendev.org/c/openstack/releases/+/827675 | 10:16 |
| opendevreview | Merged openstack/releases master: Release ironic 18.2.1 for xena https://review.opendev.org/c/openstack/releases/+/826254 | 10:16 |
| opendevreview | Merged openstack/releases master: [victoria] release oslo deliverables https://review.opendev.org/c/openstack/releases/+/827672 | 10:16 |
| opendevreview | LiZekun proposed openstack/releases master: Remove unicode from code https://review.opendev.org/c/openstack/releases/+/826575 | 10:20 |
| opendevreview | Merged openstack/releases master: Add release liaisons for manila https://review.opendev.org/c/openstack/releases/+/826496 | 10:21 |
| *** amoralej is now known as amoralej|lunch | 12:56 | |
| *** amoralej|lunch is now known as amoralej | 13:53 | |
| knikolla | for a few cycles we missed merging the reno-<release> reviews for keystonemiddleware https://review.opendev.org/q/owner:infra-root%2540openstack.org+project:openstack/keystonemiddleware+status:open | 15:46 |
| knikolla | they do have a semver: feature flag in the commit message | 15:47 |
| knikolla | will that cause issues if we merge them one by one during a short timespan? | 15:47 |
| clarkb | I'm not going to speak for the release team, but it seems like if there aren't any features to add then adding that after the fact will lead to misleading version numbers. | 15:48 |
| fungi | they're only for stable branches though. has anyoner merged any other changes to those branches? | 15:54 |
| knikolla | the reno docs changes which contains the semver flag are proposed to master. | 15:59 |
| knikolla | I don't see anything that merged in stable/victoria or above. | 16:02 |
| *** amoralej is now known as amoralej|off | 16:22 | |
| priteau | fungi: they're master commits, so potentially there would be three sem-ver: feature commits in succession added to master | 16:24 |
| fungi | oh, right, they get proposed to master after branching, these aren't the ones which get proposed to the newly created branches | 16:27 |
| fungi | anyway, multiple sem-ver: feature footers should be fine, they just indicate that the minor version number will need to be increased at the next release. multiple commits with sem-ver: feature footers are the same as one | 16:29 |
| fungi | if you don't want the minor version number to increase at the next release, then the commit messages will need to be edited to remove those | 16:30 |
| knikolla | makes sense, thanks for clarifying. i'll go ahead and remove the semver, since these are just doc changes, and the releases corresponding to them have already happened. | 16:32 |
| *** marios is now known as marios|out | 16:48 | |
| *** noonedeadpunk_ is now known as noonedeadpunk | 19:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!