Conversation
mandre
left a comment
There was a problem hiding this comment.
Would it make more sense to have these in openstack/networking/v2/extensions/fwaas_v2 and openstack/networking/v2/extensions/bgpvpns, as a similar fashion that we're doing for MTU for example?
|
To be honest I don't know. Existing support for rbac policies extension already resides in the quota package. |
82c811d to
b5851c1
Compare
|
@mandre I think once these new struct members are pointers, it is ok to keep them here. They will be ignored if they're not defined. |
|
I'm not sure either. Let's ask the other maintainers. Should we add them to |
|
I'd be personally in favour of the former, if this is possible, seeing as these are quotas for an extension. Any extension can define its own quota, so this could quickly get out of hand otherwise. My 2c |
|
For context, |
|
I shared my thoughts on moving this into a new extension in #3447 (comment). Using pointers for these struct members should be safe and should not introduce issues. Creating separate packages for each individual option, however, could add unnecessary complexity and negatively impact the developer experience for SDK users. |
An addition to #1742