* [pve-devel] applied: [PATCH storage v5] fix #957: iscsi: improve iscsi_test_portal logic
[not found] <20250320142319.20411-2-linuxmaniac@torreviejawireless.org>
@ 2025-03-25 18:35 ` Thomas Lamprecht
0 siblings, 0 replies; only message in thread
From: Thomas Lamprecht @ 2025-03-25 18:35 UTC (permalink / raw)
To: Victor Seva, pve-devel
Am 20.03.25 um 15:23 schrieb Victor Seva:
> Check if there is already a logged session present and
> fall back to previous TCP check port connection.
>
> pvestatd is calling check_connection every 10 seconds.
> This check produces a lot of noise at the iscsi server logging.
>
> Signed-off-by: Victor Seva <linuxmaniac@torreviejawireless.org>
> ---
>
> changes since v4:
>
> * rework commit message wording and subject in proper format
>
> * iscsi_test_session():
> use [0-9] instead of \d
> simplify return logic checking $state is defined too
>
> * iscsi_test_portal():
> fallback to TCP check connection if session is not LOGGED_IN
>
> * iscsi_discovery():
> rename $target to $target_in to avoid variable shadowing
>
> src/PVE/Storage.pm | 2 +-
> src/PVE/Storage/ISCSIPlugin.pm | 39 ++++++++++++++++++++++++++--------
> 2 files changed, 31 insertions(+), 10 deletions(-)
>
>
applied with Mira's and Friedrich's R-b and T-b trailers, thanks!
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2025-03-25 18:36 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20250320142319.20411-2-linuxmaniac@torreviejawireless.org>
2025-03-25 18:35 ` [pve-devel] applied: [PATCH storage v5] fix #957: iscsi: improve iscsi_test_portal logic Thomas Lamprecht
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal