-
Notifications
You must be signed in to change notification settings - Fork 60
sru/multipath-tools: initial minor release exception #355
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
I have read the proposed changes, and will add comments to it, but at first glance it sounds like you could use the existing generic MRE allowance from https://documentation.ubuntu.com/project/SRU/reference/requirements/#new-upstream-microreleases There is more in the linked document, but it states:
|
|
|
||
| Beginning with version 0.10, upstream releases stable updates on `stable-0.x.y` branches on `GitHub <https://github.com/opensvc/multipath-tools>`__. | ||
| These contain small bug fixes with low regression risk that are cherry-picked from the staging area from `openSUSE <https://github.com/openSUSE/multipath-tools/tree/queue>`__. | ||
| The stable branches are maintained by the `multipath-tools` maintainers on a best-effort basis. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't sound like a good release policy: "best effort basis". What else can be said about the policy to include fixes in these stable branches?
- what qualifies a fix for a stable branch?
- are the fixes in the stable branch always accompanied by a new or updated test?
- it's weird to mention the opensuse staging area. I see this text was copied from upstream, but what does it mean really? What's the opensuse policy for including fixes in their staging area?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They say "Small bug fixes with low regression risk from will be cherry-picked to these branches from the staging area.", which I would interpret as 'pure, well understandable bugfixes' and since the stable branches have CI tests, that those should all pass and would be updated.
I didn't see something more specific being stated somewhere explicitly but this sounds like the maintainers intend to collect all bugfix commits affecting a stable release, likely like linux stable branches work, just done by opensuse engineering when their time allows.
| - ``basic-build-and-ci`` | ||
|
|
||
| All of them are relevant for us, since they test multiple architectures and execute the unit tests. | ||
| Another very important fact is that these pipelines also use Ubuntu and Debian as their base OS, which makes the results much more reliable for direct Ubuntu integration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I quickly checked, they use ubuntu 22.04 as a base, but the actual test runs in a container for the "rolling distros", which are debian-sid, alpine, opensuse-tumbleweed, and fedora-rawhide. That doesn't seem to match the Ubuntu LTS releases targeted by this proposal, could you check?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the only ubuntu test seems to be for trusty, but they have tests for trixie
multiarch test for stable distros / build-old (debian-trixie, 386) (push) -> https://github.com/opensvc/multipath-tools/blob/stable-0.12.y/.github/workflows/multiarch-stable.yaml and sid https://github.com/opensvc/multipath-tools/blob/stable-0.12.y/.github/workflows/multiarch.yaml
| ^^^^^^^^^^^^ | ||
|
|
||
| The Debian/Ubuntu packages also carry autopkgtests, which check if a multipath devicemapping actually works. | ||
| Especially the test ``tgtbasedmpaths`` validates multipath usage over iSCSI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That seems to be the only multipath test, actually, as kpartx-file-loopback only tests kpartx a bit. I'm a bit concerned that the autopkgtest is the only test we will run on the Ubuntu releases getting these updates. Depending on the changes we get with these upstream branches, we should perhaps add more testing to it, even if it's manual. Do you have an idea of what kind of testing that would be like, given past updates to the upstream stable release branches?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what testing do you imagine is needed in addition? that tests does the setup, path adding, path failure and data transfer verification.
the updates in 0.12.x change manpages, ci workflows, and add new devices. 0.11.x had the same, and conditional c-standard bumping, fixing shell quoting for uuids, fixing a logical ordering bug in kpartx, additional error handling, memory cleanup improvements, a new path discovery flag, fixing path group index invalidation, and a new systemd service that disables queuing on stop (multipathd-queueing.service).
i would hope these changes are covered by our existing testing if the path adding/removing and general storage functionality works.
04819e8 to
9c19654
Compare
|
Thanks! I've addressed your comments where possible :) |
multipath-tools releases stable updates now, i was kindly informed by mail from their maintainer.
this is the minor release exception so we can backport the stable releases more easily.