opensvc / multipath-tools

Other
59 stars 47 forks source link

multipath-tools 0.9.5 #64

Closed mwilck closed 1 year ago

mwilck commented 1 year ago

Hi Christophe,

it's a relatively small PR this time, mostly bug fixes.

@bmarzins (15): multipathd: make pr registration consistent libmultipath: make prflag an enum multipathd: handle no active paths in update_map_pr multipathd: add missing newline to cli_del_map reply libmultipath: skip extra vector work in remove_maps libmultipath: orphan paths if coalesce_paths frees newmp libmultipath: use select_reload_action in select_action libmultipath: select resize action even if reload is forced libmultipath: cleanup ACT_CREATE code in select_action libmultipath: keep renames from stopping other multipath actions multipath.rules: fix "smart" bug with failed valid path check libmultipath: return 'pending' state when port is in transition (fixes #60) libmultipath: set init failure message when init fails libmultipath: reset nr_timeouts if we freed the context libmultipath: limit paths that can get wwid from environment

@brianatpurestorage (1): libmultipathd: Avoid parsing errors due to unsupported designators

@dirkmueller (1): multipath-tools tests: use assert_true/false instead of assert() (fixes #63)

@mwilck (11): libmultipath: is_path_valid(): check if device is in use libmpathpersist: use conf->timeout for updating persistent reservations (fixes #45) libmultipath: pathinfo: don't fail for devices lacking INQUIRY properties (fixes #56) libmultipath: bump ABI version to 18.0.0 libmpathpersist: fix resource leak in update_map_pr() libmultipath.version: bump symbol version of get_uid multipath-tools Makefiles: handle dependency files correctly multipathd.service: remove "Also=multipathd.socket" (fixes #65) libmultipath: use directio checker for LIO targets (fixes #54) GitHub workflows (basic): remove Ubuntu 18.04, add 22.04 libmultipath: bump version to 0.9.5

@thesamesam (1): multipath-tools tests: fix quoting of CFLAGS in Makefile

All patches have been reviewed, except for the version bumps and the GitHub workflow changes (as usual).

mwilck commented 1 year ago

The CI job for Alpine has failed to download the sources. The same job for the same code has succeeded in the openSUSE repo. @cvaroqui, if you have time, please just restart the job (I have no permissions to do that).

cvaroqui commented 1 year ago

@mwilck done

mwilck commented 1 year ago

Coverity scan found no new defects.

mwilck commented 1 year ago

I apologize for the multiple updates, I was having some fun with the update of the basic-build-and-ci GitHub workflow from Ubuntu bionic top jammy, and it took me a few attempts to get it right.

mwilck commented 1 year ago

Hold on once more, we need another fixup for #54.

mwilck commented 1 year ago

@bmarzins, @cvaroqui: gentle reminder