-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
DLP: Added sample for create stored infotype #3233
DLP: Added sample for create stored infotype #3233
Conversation
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
6fd54df
to
ba3bcdd
Compare
This sample requires resources like valid |
We should move to using public dataset given in protocol example. That leaves us with a dependency on output path. |
61530b5
to
697462f
Compare
@dinesh-crest please reference comment I made here to resolve the dlp workflow failure #3240 (comment) |
1e2040f
to
049ef1c
Compare
049ef1c
to
c13e627
Compare
@dinesh-crest Relevant to the github actions error, this PR should be unblocked when this is merged in #3279. |
e9e5703
to
44b5bd7
Compare
44b5bd7
to
445ef9b
Compare
DLP: Added sample for create stored infotype
Added unit test cases for the same
PS:- I have marked the test cases to skip as the required resources are not available. I have verified locally and the test cases seem to be working fine if resources are provided.
Description
Reference:- https://cloud.google.com/dlp/docs/creating-stored-infotypes#create-storedinfotye
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
npm test
(see Testing)npm run lint
(see Style)