Skip to content

Commit

Permalink
moritz feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
elchead committed Oct 24, 2024
1 parent cd12f21 commit 0651847
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 12 deletions.
17 changes: 6 additions & 11 deletions docs/docs/workflows/terraform-provider.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,8 @@ This example shows how to set up a Constellation cluster with the reference IAM
</TabItem>
<TabItem value="azure" label="Azure">

<Tabs groupId="variant">
<TabItem value="sevsnp" label="SEV-SNP">
When creating a cluster with SEV-SNP, you need to manually patch the policy of the MAA provider before creating the Constellation cluster, as this feature isn't available in Azure's Terraform provider yet. The Constellation CLI provides a utility for patching, but you
can also do it manually.
:::info

Check warning on line 38 in docs/docs/workflows/terraform-provider.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.SentenceLength] Try to keep sentences short (< 30 words). Raw Output: {"message": "[Microsoft.SentenceLength] Try to keep sentences short (\u003c 30 words).", "location": {"path": "docs/docs/workflows/terraform-provider.md", "range": {"start": {"line": 38, "column": 4}}}, "severity": "WARNING"}
On SEV-SNP, you need to manually patch the policy of the MAA provider before creating the Constellation cluster, as this feature isn't available in Azure's Terraform provider yet. The Constellation CLI provides a utility for patching, but you can also do it manually.

```bash
terraform init
Expand All @@ -48,9 +46,7 @@ This example shows how to set up a Constellation cluster with the reference IAM
terraform apply -target constellation_cluster.azure_example # adjust resource path if not using the example configuration
```

Optionally, you can prefix the `terraform apply` command with `TF_LOG=INFO` to collect [Terraform logs](https://developer.hashicorp.com/terraform/internals/debugging) while applying the configuration. This may provide helpful output in debugging scenarios.

Use the following policy if manually performing the patch.
Use the following policy if manually performing the patch.

```
version= 1.0;
Expand All @@ -70,8 +66,8 @@ This example shows how to set up a Constellation cluster with the reference IAM
};
```

</TabItem>
<TabItem value="tdx" label="TDX">
:::

Initialize the providers and apply the configuration.

```bash
Expand All @@ -80,8 +76,7 @@ This example shows how to set up a Constellation cluster with the reference IAM
```

Optionally, you can prefix the `terraform apply` command with `TF_LOG=INFO` to collect [Terraform logs](https://developer.hashicorp.com/terraform/internals/debugging) while applying the configuration. This may provide helpful output in debugging scenarios.
</TabItem>
</Tabs>

</TabItem>

<TabItem value="gcp" label="GCP">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ locals {
location = "northeurope"
control_plane_count = 3
worker_count = 2
instance_type = "Standard_DC4as_v5" // needs to be changed for azure-tdx
instance_type = "Standard_DC4as_v5" // Adjust if using TDX
subscription_id = "00000000-0000-0000-0000-000000000000"

master_secret = random_bytes.master_secret.hex
Expand Down

0 comments on commit 0651847

Please sign in to comment.