You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As correctly pointed out in #19, the documentation was incorrect here, the second example should work as expected but with cluster_name instead of name e.g.
describe google_container_node_pool(project: PROJECT_NAME, zone: CLUSTER_ZONE, cluster_name: CLUSTER_NAME, nodepool_name: 'default-pool') do
it { should exist }
its('name') { should eq 'default-pool' }
its('status') { should eq 'in_use' }
its('config.disk_size_gb') { should eq 100 }
its('config.machine_type') { should eq 'n1-standard-1' }
Rest assured the documentation will be updated to better reflect the arguments.
* Ensure resources have consistent exists methods.
* Corrected a coupld of arguments in example doc strings.
Signed-off-by: Stuart Paterson <[email protected]>
Summary
Cannot test
node_pool
, undocumented and untested requirement forcluster_id
.Reproduce
OR
Actual
Expected
Test should work with cluster name. I don't know about some cluster id, it is not in the source code or integration tests.
Using this for Documentation:
Notes
The text was updated successfully, but these errors were encountered: