bzp2010 commented on code in PR #2440:
URL: 
https://github.com/apache/apisix-ingress-controller/pull/2440#discussion_r2203706054


##########
docs/en/latest/upgrade-guide.md:
##########
@@ -64,7 +64,7 @@ etcdserver:
 
 ##### Controller-Only Configuration Source
 
-In 2.0.0, all data plane configurations must originate from the Ingress 
Controller. Configurations via Admin API or any external methods are no longer 
supported and will be ignored or may cause errors.
+In APISIX Ingress Controller 2.0.0, a full-sync mode is used. All Data Plane 
configurations must originate from the Ingress Controller. Any additional route 
configurations made via the Admin API or other external methods will be ignored.

Review Comment:
   This is ambiguous and should be edited.
   
   > Any additional route configurations made via the Admin API or other 
external methods will be ignored.
   
   Manual Admin API operations outside of the ingress controller will be 
overridden by regular full updates. We used to allow those user-created 
resources to co-exist with those created by the ingress controller, which was 
frankly a mistake in itself.
   
   The description should be made clearer to clarify that both Admin API and 
standalone mode are supported.



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