Segment security profile resource #878
Conversation
Signed-off-by: Anna Khmelnitsky <[email protected]>
Signed-off-by: Anna Khmelnitsky <[email protected]>
|
/test-all |
|
@annakhm, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
|
/test-all |
Signed-off-by: Anna Khmelnitsky <[email protected]>
|
@annakhm, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
| Optional: true, | ||
| Default: true, | ||
| }, | ||
| "non_ip_traffic_block_enabled": { |
There was a problem hiding this comment.
Aren't we usually adding description to resource attributes?
There was a problem hiding this comment.
This file was generated by policygen, and descriptions in spec are often too long to appear here in the code. We do have them in documentation. I'm not sure description in the code has any practical purpose.
There was a problem hiding this comment.
Makes sense. I wonder how much the provider image will shrink if we remove all these strings :)
No description provided.