Skip to content

Cloud Workstations - Workstation Config#364

Merged
modular-magician merged 1 commit into
terraform-google-modules:masterfrom
modular-magician:downstream-pr-91d96c897eb8537ba19f66c1652083d14aa32aa6
Feb 24, 2023
Merged

Cloud Workstations - Workstation Config#364
modular-magician merged 1 commit into
terraform-google-modules:masterfrom
modular-magician:downstream-pr-91d96c897eb8537ba19f66c1652083d14aa32aa6

Conversation

@modular-magician

Copy link
Copy Markdown
Collaborator

The leading pull request for this is: GoogleCloudPlatform/magic-modules#7005, that should be merged first. (as I've split up the PRs for this new service)

If this PR is for Terraform, I acknowledge that I have:

  • Searched through the issue tracker for an open issue that this either resolves or contributes to, commented on it to claim it, and written "fixes {url}" or "part of {url}" in this PR description. If there were no relevant open issues, I opened one and commented that I would like to work on it (not necessary for very small changes).
  • Generated Terraform, and ran make test and make lint to ensure it passes unit and linter tests.
  • Ensured that all new fields I added that can be set by a user appear in at least one example (for generated resources) or third_party test (for handwritten resources or update tests).
  • Ran relevant acceptance tests (If the acceptance tests do not yet pass or you are unable to run them, please let your reviewer know).
  • Read the Release Notes Guide before writing my release note below.

Release Note Template for Downstream PRs (will be copied)

google_workstations_workstation_config

Derived from GoogleCloudPlatform/magic-modules#7017

* Add WorkstationConfig resource

* indent correctly

* add the workstationConfig

* Add WorkstationConfig

* Create workstation_config_basic.tf.erb

* Create resource_workstations_workstation_config_test.go.erb

* fix indent

* remove duplicate displayName field

* add `workstationClusterId` param

* set host attribute block

* set the `workstation_cluster_id`

* fix create

* use a default for the container property

* add input true

* set api defaults

* set `host.gceInstance.bootDiskSizeGb` to be computed

* fix machine_type attribute

* fix typo's

* lower case funcs

* Update mmv1/third_party/terraform/tests/resource_workstations_workstation_config_test.go.erb

Co-authored-by: Riley Karson <rileykarson@google.com>

* add location

* add skip test to examples

* add tests to run sequentially instead of paralell

* set `<% unless version == 'ga' -%>`

* Remove extra tests

* align google_workstations_workstation_config resource in update test

* Revert "align google_workstations_workstation_config resource in update test"

This reverts commit 37a4af300e0d78b4110c8029f09511ae1957e6c9.

* Revert "Remove extra tests"

This reverts commit 1fc074464b3fea0293884bbda86725a81e2f08e6.

* revert skip test

* align tests and remove update label

* add encryptionKey attribute

* indent the description

* add encryption key test

* Use a static `account_id` for KMS SA

* address comments on pr

* set some extra defaults

* add 2 new tests to test host and persistent_directories

* Move back up

* have a single set of unless/end

* remove data google project

* Remove faulty import

* remove duplicate output and fmt

* remove output field on displayName

* add displayName update test

* Fix update of display name

---------

Co-authored-by: Riley Karson <rileykarson@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>
@modular-magician modular-magician merged commit f255907 into terraform-google-modules:master Feb 24, 2023
@modular-magician modular-magician deleted the downstream-pr-91d96c897eb8537ba19f66c1652083d14aa32aa6 branch November 18, 2024 03:05
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.

1 participant