bzp2010 commented on code in PR #12179:
URL: https://github.com/apache/apisix/pull/12179#discussion_r2067800589


##########
conf/config.yaml.example:
##########
@@ -165,6 +165,7 @@ nginx_config:                     # Config for render the 
template to generate n
   meta:
     lua_shared_dict:              # Nginx Lua shared memory zone. Size units 
are m or k.
       prometheus-metrics: 15m
+      standalone-config: 10m

Review Comment:
   I don't know about this, this PR is just providing a new data source for the 
config_yaml module, other than that, it's reusing previous code.
   If the user is just starting a pure stream subsystem, then the Admin API is 
not possible to turn on, that's a server block in `http {}`, but turning on 
both subsystems at the same time and updating the stream route configuration 
via the Admin API should be allowed.
   
   The standalone not being supported in the stream subsystem is not expected, 
and if it is indeed not allowed it must be fixed in a future PR. Some 
Kubernetes scenarios require it to work correctly.



-- 
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