-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update the doc link and add the example link to the cutom policy guide #378
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #378 +/- ##
==========================================
- Coverage 25.40% 25.39% -0.01%
==========================================
Files 195 195
Lines 16784 16788 +4
==========================================
Hits 4264 4264
- Misses 12302 12306 +4
Partials 218 218 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Custom policy template and custom policy examples in doc seem to be scattered.
b2e1a59
to
13b55f9
Compare
…e creation and assignment in the policy guide Signed-off-by: Ramya Bangera <bangerar@vmware.com>
13b55f9
to
abea42c
Compare
…in the custom policy assignment is not found Initially this check was added as the custom template creation via TMC TF provider was not supported. Now since the provider supports creation of the custom template removing this check, to let user create the custom template and assign it to custom policy on the fly. Signed-off-by: Ramya Bangera <bangerar@vmware.com>
4c9b895
to
7df2dcd
Compare
Overall LGTM, hope the doc changes are tested using the tool for doc preview |
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
What this PR does / why we need it:
Initially this check was added as the custom template creation via TMC TF provider was not supported. Now since the provider supports creation of the custom template removing this check,
to let user create the custom template and assign it to custom policy on the fly.
Which issue(s) this PR fixes
Additional information
Special notes for your reviewer