SkyeYoung commented on code in PR #12383:
URL: https://github.com/apache/apisix/pull/12383#discussion_r2221865942


##########
apisix/plugins/prometheus/exporter.lua:
##########
@@ -211,6 +221,14 @@ end
 
 
 function _M.stream_init()
+    -- FIXME:
+    -- Now the HTTP subsystem loads the stream plugin unintentionally, which 
shouldn't happen.
+    -- It breaks the initialization logic of the plugin,
+    -- here it is temporarily fixed using a workaround.
+    if ngx.config.subsystem ~= "stream" then
+        return
+    end

Review Comment:
   As mentioned in the comments, the http subsystem also loads the stream 
plugin. This is an issue that needs to be resolved.
   
   <img width="1580" height="182" alt="Image" 
src="https://github.com/user-attachments/assets/d3891539-5b76-4100-b782-8d0b04b91b57";
 />
   
   <img width="1597" height="586" alt="Image" 
src="https://github.com/user-attachments/assets/3580b141-4eda-4df7-862d-e537524b0f11";
 />
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to