public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Chris Leech <cleech@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/iscsi-initiator-utils] rawhide: fix regression in node record of type "fw" or "static" with untagged upstream fix
Date: Thu, 25 Jun 2026 21:37:23 GMT	[thread overview]
Message-ID: <178242344371.1.4946025689758969274.rpms-iscsi-initiator-utils-b2ffc3d0c9ff@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/iscsi-initiator-utils
Branch : rawhide
Commit : b2ffc3d0c9ffb1c77b9fd21511dee1e8ebe86661
Author : Chris Leech <cleech@redhat.com>
Date   : 2026-06-25T14:37:06-07:00
Stats  : +64/-1 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/iscsi-initiator-utils/c/b2ffc3d0c9ffb1c77b9fd21511dee1e8ebe86661?branch=rawhide

Log:
fix regression in node record of type "fw" or "static" with untagged upstream fix

---
diff --git a/0001-Fix-incorrect-parsing-of-node.discovery_type-static-.patch b/0001-Fix-incorrect-parsing-of-node.discovery_type-static-.patch
new file mode 100644
index 0000000..473f65f
--- /dev/null
+++ b/0001-Fix-incorrect-parsing-of-node.discovery_type-static-.patch
@@ -0,0 +1,59 @@
+From 290d16d6c6b6f4c78af119eb874484b2f995dc23 Mon Sep 17 00:00:00 2001
+From: Mike Andrews <gubblebozer@gmail.com>
+Date: Sun, 28 Dec 2025 19:19:39 -0500
+Subject: [PATCH 1/1] Fix incorrect parsing of node.discovery_type "static" and
+ "fw" (#518)
+
+Remove the unused DISCOVERY_TYPE_SLP enumeration value so that the strings
+"static" and "fw" map to the correct enum values.
+
+Previous versions supported an "slp" discovery_type.  In d1d456cf28cd89, the
+"slp" string was removed from the list of TYPE_INT_O string arguments when
+processing DISC_TYPE and NODE_DISC_TYPE records in idbm_recinfo_discovery and
+_idbm_node_rec_link.  But, it was left in the discovery_type enum.  The
+resulting parsing logic would then map the string values "static" and "fw" to
+the wrong enum values, and thus logic with unique action on
+DISCOVERY_TYPE_STATIC and DISCOVERY_TYPE_FW did the wrong thing.  Specifically,
+
+- "static" mapped to DISCOVERY_TYPE_SLP, and
+- "fw" mapped to DISCOVERY_TYPE_STATIC.
+
+One manifestation of this bug is that you can no longer use iscsiadm to
+manipulate or delete nodes with discovery_type = "static", as it falls through
+to the default case in setup_disc_to_node_link.
+
+Fixes #463.
+---
+ libopeniscsiusr/idbm.h | 2 --
+ usr/config.h           | 2 --
+ 2 files changed, 4 deletions(-)
+
+diff --git a/libopeniscsiusr/idbm.h b/libopeniscsiusr/idbm.h
+index 0b201f4..512851d 100644
+--- a/libopeniscsiusr/idbm.h
++++ b/libopeniscsiusr/idbm.h
+@@ -78,8 +78,6 @@ enum discovery_type {
+ 	DISCOVERY_TYPE_SENDTARGETS,
+ 	DISCOVERY_TYPE_ISNS,
+ 	DISCOVERY_TYPE_OFFLOAD_SENDTARGETS,
+-	/* SLP is Not Used --  kept as place holder */
+-	DISCOVERY_TYPE_SLP_NOT_IMPLEMENTED,
+ 	DISCOVERY_TYPE_STATIC,
+ 	DISCOVERY_TYPE_FW,
+ };
+diff --git a/usr/config.h b/usr/config.h
+index a38a25d..94a8ad0 100644
+--- a/usr/config.h
++++ b/usr/config.h
+@@ -161,8 +161,6 @@ typedef enum discovery_type {
+ 	DISCOVERY_TYPE_SENDTARGETS,
+ 	DISCOVERY_TYPE_ISNS,
+ 	DISCOVERY_TYPE_OFFLOAD_SENDTARGETS,
+-	/* SLP is Not Used --  kept as place holder */
+-	DISCOVERY_TYPE_SLP_NOT_IMPLEMENTED,
+ 	DISCOVERY_TYPE_STATIC,
+ 	DISCOVERY_TYPE_FW,
+ } discovery_type_e;
+-- 
+2.54.0
+

diff --git a/iscsi-initiator-utils.spec b/iscsi-initiator-utils.spec
index 06dca4a..fb83dd5 100644
--- a/iscsi-initiator-utils.spec
+++ b/iscsi-initiator-utils.spec
@@ -10,13 +10,14 @@
 Summary: iSCSI daemon and utility programs
 Name: iscsi-initiator-utils
 Version: 6.%{open_iscsi_version}.%{open_iscsi_build}
-Release: 0.git%{shortcommit0}%{?dist}.5
+Release: 0.git%{shortcommit0}%{?dist}.6
 License: GPL-2.0-or-later
 URL: https://github.com/open-iscsi/open-iscsi
 Source0: https://github.com/open-iscsi/open-iscsi/archive/%{commit0}.tar.gz#/open-iscsi-%{shortcommit0}.tar.gz
 Source4: 04-iscsi
 Source5: iscsi-tmpfiles.conf
 
+Patch00: 0001-Fix-incorrect-parsing-of-node.discovery_type-static-.patch
 Patch01: 0001-meson-don-t-hide-things-with-Wno-all.patch
 
 # https://github.com/open-iscsi/open-iscsi/pull/394/
@@ -272,6 +273,9 @@ systemctl --no-reload preset iscsi.service iscsi-starter.service &>/dev/null || 
 %endif
 
 %changelog
+* Thu Jun 25 2026 Chris Leech <cleech@redhat.com> - 6.2.1.11-0.git4b3e853.6
+- fix regression in node record of type "fw" or "static" with untagged upstream fix
+
 * Fri Jun 12 2026 Yaakov Selkowitz <yselkowi@redhat.com> - 6.2.1.11-0.git4b3e853.5
 - Rebuilt for openssl 4.0
 

                 reply	other threads:[~2026-06-25 21:37 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178242344371.1.4946025689758969274.rpms-iscsi-initiator-utils-b2ffc3d0c9ff@fedoraproject.org \
    --to=cleech@redhat.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox