sbrouet commented on issue #1496:
URL: 
https://github.com/apache/apisix-ingress-controller/issues/1496#issuecomment-1342325749

   > If this feature is supported, http should be an object, which would make 
an incompatible change, which would require a V3 API version.
   
   Hello,
   That is a good remark. `spec.http[].plugin_config_name` indeed is a single 
value
   I don't know what is best practice in such cases.
   Versioning (v3) probably is cleaner.
   
   Another - hacky - option would be keeping current 
`spec.http[].plugin_config_name` field and adding an optional array named 
`spec.http[].plugin_config_names` (with plural). In that case old 
configurations would be compatible. Priority would need to be defined when both 
`plugin_config_name` and `plugin_config_names` are present. 
   
   Thanks !


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to