-
Notifications
You must be signed in to change notification settings - Fork 676
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
add dbaas example #2649
add dbaas example #2649
Conversation
@@ -35,6 +35,33 @@ resource "ibm_resource_instance" "resource_instance" { | |||
} | |||
``` | |||
|
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.
Can we have a title for this section in Bold something
Provisioning a Hyper Protect DBaaS PostgreSQL Service...
something like this example
https://github.com/IBM-Cloud/terraform-provider-ibm/blob/master/website/docs/r/iam_access_group_policy.html.markdown#access-group-policy-for-all-identity-and-access-enabled-services
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.
@hkantare I added a title. Please review.
Detect secrets failed can you please validate and add false positive secrets |
@hkantare I don't understand. Could you explain how I should do this? |
From this section of code the detect_secret tool is detecting mostly the password as secret .. |
@hkantare thanks a lot for your help, but I couldn't install Homebrew because of connection issues. I tried many times (maybe it's because I'm working from home). Is there any workaround for this secret problem? |
I raised a PR to your brach by updating detect secret ... Can you merge this so that it will update this pr with required file |
add a section title
@hkantare thank you for your help! I merged that PR. |
There are still some conflicts ...so I raised a PR from my end |
closing this PR |
Community Note
Relates OR Closes #0000
Output from acceptance testing: