Skip to content

(docs) Bigtable test cleanup and additional workload generator test #7190

Merged
anguillanneuf merged 4 commits into
mainfrom
bt-cleanup
Jun 22, 2022
Merged

(docs) Bigtable test cleanup and additional workload generator test #7190
anguillanneuf merged 4 commits into
mainfrom
bt-cleanup

Conversation

@billyjacobson
Copy link
Copy Markdown
Member

Add test to validate deployed template
Cleanup non-canonical tests

Fixes #issue

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

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • API's need to be enabled to test (tell us)
  • Environment Variables need to be set (ask us to set them)
  • Tests pass: mvn clean verify required
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
  • Please merge this PR for me once it is approved.

Cleanup non-canonical tests
@billyjacobson billyjacobson requested review from a team and yoshi-approver as code owners June 17, 2022 14:45
@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label Jun 17, 2022
# Conflicts:
#	bigtable/beam/workload-generator/pom.xml
@billyjacobson billyjacobson requested a review from kolea2 June 17, 2022 15:13
@anguillanneuf anguillanneuf merged commit 702e9cf into main Jun 22, 2022
@anguillanneuf anguillanneuf deleted the bt-cleanup branch June 22, 2022 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants