<div dir="ltr">Looks good to me, thanks for finding this issue. And, since we do need the 'enable' strings to be templated in right, I am asking Dean to re-run this testsuite with your patch applied. <div><br></div><div>Thomas, I gather this series will not be applied/go through next-dts since it is not primarily a DTS series, but please let me know if it should. I am asking you since I see it is delegated to you on Patchwork.</div><div><br></div><div>Reviewed-by: Patrick Robb <<a href="mailto:probb@iol.unh.edu">probb@iol.unh.edu</a>></div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Jan 15, 2026 at 6:02 PM Lukas Sismis <<a href="mailto:sismis@dyna-nic.com">sismis@dyna-nic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The f-string syntax was missing the 'f' prefix, causing<br>
string interpolation to fail.<br>
<br>
Fixes: cfd851f264 ("dts: add QinQ strip and VLAN extend to testpmd shell")<br>
Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
<br>
Signed-off-by: Lukas Sismis <<a href="mailto:sismis@dyna-nic.com" target="_blank">sismis@dyna-nic.com</a>><br>
---<br>
 dts/api/testpmd/__init__.py | 8 ++++----<br>
 1 file changed, 4 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/dts/api/testpmd/__init__.py b/dts/api/testpmd/__init__.py<br>
index a20dedc25a..703cae487e 100644<br>
--- a/dts/api/testpmd/__init__.py<br>
+++ b/dts/api/testpmd/__init__.py<br>
@@ -227,7 +227,7 @@ def start_tx_first(self, burst_num: int, verify: bool = True) -> None:<br>
             InteractiveCommandExecutionError: If `verify` is :data:`True` and forwarding fails to<br>
                 start or ports fail to come up.<br>
         """<br>
-        self.send_command(f"start tx_first {burst_num if burst_num is not None else ""}")<br>
+        self.send_command(f"start tx_first {burst_num if burst_num is not None else ''}")<br>
         if verify:<br>
             # If forwarding was already started, sending "start" again should tell us<br>
             start_cmd_output = self.send_command("start")<br>
@@ -898,11 +898,11 @@ def set_qinq_strip(self, port: int, enable: bool, verify: bool = True) -> None:<br>
             vlan_settings = self.show_port_info(port_id=port).vlan_offload<br>
             if enable ^ (vlan_settings is not None and VLANOffloadFlag.QINQ_STRIP in vlan_settings):<br>
                 self._logger.debug(<br>
-                    f"Failed to {"enable" if enable else "disable"}"<br>
+                    f"Failed to {'enable' if enable else 'disable'}"<br>
                     f"QinQ strip on port {port}: \n{qinq_cmd_output}"<br>
                 )<br>
                 raise InteractiveCommandExecutionError(<br>
-                    f"Failed to {"enable" if enable else "disable"} QinQ strip on port {port}"<br>
+                    f"Failed to {'enable' if enable else 'disable'} QinQ strip on port {port}"<br>
                 )<br>
<br>
     def set_mac_address(self, port: int, mac_address: str, verify: bool = True) -> None:<br>
@@ -1535,7 +1535,7 @@ def set_port_mbuf_fast_free(<br>
             InteractiveCommandExecutionError: If mbuf_fast_free could not be set successfully.<br>
         """<br>
         mbuf_output = self.send_command(<br>
-            f"port config {port_id} tx_offload mbuf_fast_free {"on" if on else "off"}"<br>
+            f"port config {port_id} tx_offload mbuf_fast_free {'on' if on else 'off'}"<br>
         )<br>
<br>
         if verify and "Error" in mbuf_output:<br>
-- <br>
2.43.7<br>
<br>
</blockquote></div>