Skip to content

Monitoring: Regenerate VPCSC tests to include NotificationChannelService and UptimeCheckService.#5057

Merged
chingor13 merged 2 commits intogoogleapis:masterfrom
steinwaywhw:refresh-vpcsc-test
May 6, 2019
Merged

Monitoring: Regenerate VPCSC tests to include NotificationChannelService and UptimeCheckService.#5057
chingor13 merged 2 commits intogoogleapis:masterfrom
steinwaywhw:refresh-vpcsc-test

Conversation

@steinwaywhw
Copy link
Copy Markdown
Contributor

Hi, this is a refresh of an auto-generated test to include NotificationChannelService. This has been tested locally. The pull request is sent on behalf of @michaelsafyan.

@steinwaywhw steinwaywhw requested a review from a team May 2, 2019 22:00
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 2, 2019
@steinwaywhw
Copy link
Copy Markdown
Contributor Author

The failed tests in Kokoro - Test: OSX Java 8 is a failed connection in the cloud-datastore tests. It is not related to my change in this PR.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 3, 2019

Codecov Report

Merging #5057 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5057      +/-   ##
============================================
- Coverage     50.38%   50.38%   -0.01%     
  Complexity    23735    23735              
============================================
  Files          2248     2248              
  Lines        226478   226478              
  Branches      24954    24954              
============================================
- Hits         114116   114113       -3     
- Misses       103769   103772       +3     
  Partials       8593     8593
Impacted Files Coverage Δ Complexity Δ
...om/google/cloud/logging/MonitoredResourceUtil.java 50% <0%> (-0.93%) 8% <0%> (ø)
...src/main/java/com/google/cloud/ServiceOptions.java 40.61% <0%> (-0.88%) 27% <0%> (ø)

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 00380da...a02785f. Read the comment docs.

@steinwaywhw steinwaywhw changed the title Regenerate test to include NotificationChannelService. Monitoring: Regenerate VPCSC tests to include NotificationChannelService and UptimeCheckService. May 3, 2019
@chingor13 chingor13 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2019
Copy link
Copy Markdown
Contributor

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're working through issues with our OSX test executor. LGTM.

@chingor13 chingor13 merged commit 6f637c9 into googleapis:master May 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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