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

Fixed replacement of instance_pool_id in databricks_cluster #850

Merged
merged 1 commit into from
Oct 6, 2021
Merged

Conversation

nfx
Copy link
Contributor

@nfx nfx commented Oct 6, 2021

Added corner case to fix issue #824 where driver_instance_pool_id was not explicitly specified and old driver instance pool sent in cluster update request

Added corner case to fix issue #824 where `driver_instance_pool_id` was not explicitly specified and old driver instance pool sent in cluster update request
@nfx nfx added this to the v0.3.8 milestone Oct 6, 2021
@codecov
Copy link

codecov bot commented Oct 6, 2021

Codecov Report

Merging #850 (bc4f06c) into master (241aae5) will decrease coverage by 0.00%.
The diff coverage is 71.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #850      +/-   ##
==========================================
- Coverage   83.03%   83.02%   -0.01%     
==========================================
  Files          92       92              
  Lines        8204     8211       +7     
==========================================
+ Hits         6812     6817       +5     
- Misses        893      894       +1     
- Partials      499      500       +1     
Impacted Files Coverage Δ
compute/resource_cluster.go 78.11% <71.42%> (-0.21%) ⬇️

@nfx nfx merged commit 365e278 into master Oct 6, 2021
@nfx nfx deleted the issue/824 branch October 6, 2021 12:45
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
…bricks#850)

Added corner case to fix issue databricks#824 where `driver_instance_pool_id` was not explicitly specified and old driver instance pool sent in cluster update request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ISSUE] Replacement of databricks_instance_pool causes error in update of cluster
1 participant