Skip to content
This repository was archived by the owner on Sep 9, 2023. It is now read-only.

samples: ucaip sample batch 2 of 6 #15

Merged
merged 14 commits into from
Nov 6, 2020
Merged

Conversation

munkhuushmgl
Copy link
Contributor

batch 2 of 6

@munkhuushmgl munkhuushmgl requested a review from a team September 30, 2020 04:39
@munkhuushmgl munkhuushmgl requested a review from a team as a code owner September 30, 2020 04:39
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 30, 2020
@munkhuushmgl munkhuushmgl added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Sep 30, 2020
@stephaniewang526
Copy link

stephaniewang526 commented Sep 30, 2020

Looks like environment variable UCAIP_PROJECT_ID is missing from the testing project. Please make sure to set it in Secret Manager and re-run ITs. Thanks.

@munkhuushmgl munkhuushmgl added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@munkhuushmgl munkhuushmgl added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@codecov
Copy link

codecov bot commented Sep 30, 2020

Codecov Report

Merging #15 (e3d8e30) into master (e44abbc) will decrease coverage by 0.83%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #15      +/-   ##
============================================
- Coverage     76.57%   75.73%   -0.84%     
  Complexity      606      606              
============================================
  Files            48       48              
  Lines          5617     5617              
  Branches         69       22      -47     
============================================
- Hits           4301     4254      -47     
  Misses         1227     1227              
- Partials         89      136      +47     
Impacted Files Coverage Δ Complexity Δ
...gle/cloud/aiplatform/v1beta1/JobServiceClient.java 56.46% <0.00%> (-5.37%) 83.00% <0.00%> (ø%)
...loud/aiplatform/v1beta1/PipelineServiceClient.java 51.48% <0.00%> (-4.96%) 23.00% <0.00%> (ø%)
...iplatform/v1beta1/SpecialistPoolServiceClient.java 54.63% <0.00%> (-4.13%) 23.00% <0.00%> (ø%)
...cloud/aiplatform/v1beta1/DatasetServiceClient.java 61.40% <0.00%> (-4.10%) 43.00% <0.00%> (ø%)
...ud/aiplatform/v1beta1/PredictionServiceClient.java 39.28% <0.00%> (-3.58%) 11.00% <0.00%> (ø%)
...e/cloud/aiplatform/v1beta1/ModelServiceClient.java 60.00% <0.00%> (-3.53%) 43.00% <0.00%> (ø%)
...loud/aiplatform/v1beta1/EndpointServiceClient.java 50.84% <0.00%> (-3.39%) 31.00% <0.00%> (ø%)
...oud/aiplatform/v1beta1/MigrationServiceClient.java 51.47% <0.00%> (-2.95%) 11.00% <0.00%> (ø%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e44abbc...e3d8e30. Read the comment docs.

@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/install-without-bom/pom.xml
  • samples/snapshot/pom.xml
  • samples/snippets/pom.xml

@snippet-bot
Copy link

snippet-bot bot commented Nov 6, 2020

Here is the summary of changes.

You added 20 region tags.

To update this comment, add snippet-bot:force-run label.

@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/install-without-bom/pom.xml
  • samples/snapshot/pom.xml
  • samples/snippets/pom.xml

@munkhuushmgl munkhuushmgl added the automerge Merge the pull request once unit tests and other checks pass. label Nov 6, 2020
@gcf-merge-on-green
Copy link

Your PR has conflicts that you need to resolve before merge-on-green can automerge

1 similar comment
@gcf-merge-on-green
Copy link

Your PR has conflicts that you need to resolve before merge-on-green can automerge

@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/install-without-bom/pom.xml
  • samples/snapshot/pom.xml
  • samples/snippets/pom.xml

@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/install-without-bom/pom.xml
  • samples/snapshot/pom.xml
  • samples/snippets/pom.xml

@gcf-merge-on-green gcf-merge-on-green bot merged commit 8b3f746 into master Nov 6, 2020
@gcf-merge-on-green gcf-merge-on-green bot deleted the ucaip_samples_batch_2 branch November 6, 2020 19:54
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Nov 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants