Skip to content
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

Create Java code sample for wordList example - Exception list #3120

Merged
merged 2 commits into from
Jun 10, 2020
Merged

Conversation

chloeliu123
Copy link
Contributor

To be linked from https://cloud.google.com/dlp/docs/creating-custom-infotypes-dictionary#exception_list

Fixes #issue

It's a good idea to open an issue first for discussion.

[yes] I have followed Sample Format Guide
[yes] pom.xml parent set to latest shared-configuration
[nothing new] Appropriate changes to README are included in PR
[nothing new] API's need to be enabled to test (tell us)
[nothing new] Environment Variables need to be set (ask us to set them)
[yes] Tests pass (mvn -P lint clean verify)
(Note- Checkstyle passing is required; Spotbugs, ErrorProne, PMD, etc. ERROR's are advisory only)
[yes] Please merge this PR for me once it is approved.

@chloeliu123 chloeliu123 requested a review from a team June 10, 2020 16:39
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 10, 2020
@chloeliu123
Copy link
Contributor Author

The test failure seems to be caused by a flaky test, looks unrelated to this change:

[ERROR] Tests run: 20, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 2,347.024 s <<< FAILURE! - in dlp.snippets.InspectTests
[ERROR] dlp.snippets.InspectTests.testInspectGcsFile Time elapsed: 907.113 s <<< FAILURE!
java.lang.AssertionError:

Expected: a string containing "Job status: DONE"
but: was "Job created: projects/java-docs-samples-testing/locations/global/dlpJobs/i-2561321144851358899
Job was not completed after 15 minutes.

@averikitsch averikitsch added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jun 10, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 10, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit f4d921c into GoogleCloudPlatform:master Jun 10, 2020
Sita04 pushed a commit that referenced this pull request Feb 7, 2023
#3120)

To be linked from https://cloud.google.com/dlp/docs/creating-custom-infotypes-dictionary#exception_list

Fixes #issue

It's a good idea to open an issue first for discussion.

[yes] I have followed Sample Format Guide
[yes] pom.xml parent set to latest shared-configuration
[nothing new] Appropriate changes to README are included in PR
[nothing new] API's need to be enabled to test (tell us)
[nothing new] Environment Variables need to be set (ask us to set them)
[yes] Tests pass (mvn -P lint clean verify)
(Note- Checkstyle passing is required; Spotbugs, ErrorProne, PMD, etc. ERROR's are advisory only)
[yes] Please merge this PR for me once it is approved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants