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

samples: fixing machine type for new default image#490

Merged
bradmiro merged 1 commit intomasterfrom
update-samples
Mar 29, 2021
Merged

samples: fixing machine type for new default image#490
bradmiro merged 1 commit intomasterfrom
update-samples

Conversation

@bradmiro
Copy link
Copy Markdown
Contributor

Dataproc 2.0 is the new default image which does not support n1-standard-1. Sample corrects for this.

Fixes https://github.com/googleapis/java-dataproc/issues/485, https://github.com/googleapis/java-dataproc/issues/486, https://github.com/googleapis/java-dataproc/issues/487

@bradmiro bradmiro requested a review from a team March 23, 2021 18:24
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 23, 2021
@product-auto-label product-auto-label bot added api: dataproc Issues related to the googleapis/java-dataproc API. samples Issues that are directly related to samples. labels Mar 23, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 23, 2021

Codecov Report

Merging #490 (69c1ae5) into master (d25a909) will increase coverage by 2.66%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #490      +/-   ##
============================================
+ Coverage     75.52%   78.19%   +2.66%     
- Complexity      534      606      +72     
============================================
  Files            48       48              
  Lines          4552     4402     -150     
  Branches         48       48              
============================================
+ Hits           3438     3442       +4     
+ Misses         1076      912     -164     
- Partials         38       48      +10     
Impacted Files Coverage Δ Complexity Δ
.../v1/stub/AutoscalingPolicyServiceStubSettings.java 80.41% <0.00%> (-7.20%) 15.00% <0.00%> (ø%)
...ta2/stub/AutoscalingPolicyServiceStubSettings.java 80.41% <0.00%> (-7.20%) 15.00% <0.00%> (ø%)
...ud/dataproc/v1/stub/JobControllerStubSettings.java 84.04% <0.00%> (-4.77%) 18.00% <0.00%> (ø%)
...taproc/v1beta2/stub/JobControllerStubSettings.java 84.04% <0.00%> (-4.77%) 18.00% <0.00%> (ø%)
...proc/v1/stub/GrpcAutoscalingPolicyServiceStub.java 91.72% <0.00%> (-0.58%) 12.00% <0.00%> (ø%)
...v1beta2/stub/GrpcAutoscalingPolicyServiceStub.java 91.72% <0.00%> (-0.58%) 12.00% <0.00%> (ø%)
...c/v1/stub/WorkflowTemplateServiceStubSettings.java 84.82% <0.00%> (-0.14%) 19.00% <0.00%> (ø%)
...eta2/stub/WorkflowTemplateServiceStubSettings.java 84.82% <0.00%> (-0.14%) 19.00% <0.00%> (ø%)
...ataproc/v1/stub/ClusterControllerStubSettings.java 88.03% <0.00%> (-0.10%) 20.00% <0.00%> (ø%)
...oc/v1beta2/stub/ClusterControllerStubSettings.java 88.03% <0.00%> (-0.10%) 20.00% <0.00%> (ø%)
... and 38 more

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 d25a909...69c1ae5. Read the comment docs.

@bradmiro bradmiro merged commit c3213c6 into master Mar 29, 2021
@bradmiro bradmiro deleted the update-samples branch March 29, 2021 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: dataproc Issues related to the googleapis/java-dataproc API. 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.

CreateClusterWithAutoscalingTest: createClusterWithAutoscalingTest failed

4 participants