-
Notifications
You must be signed in to change notification settings - Fork 21
add arm support docs #558
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
base: main
Are you sure you want to change the base?
add arm support docs #558
Conversation
Vercel Previews Deployed
|
Broken Link CheckerNo broken links found! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for putting this together! Let me know if you have any questions about any of my feedback and I'm happy to give it another 👀 when it is ready.
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agent-pools.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agent-pools.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agent-pools.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agents.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/changelog.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Natalie Todd <[email protected]>
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agent-pools.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agents.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Natalie Todd <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just did a review via the preview so most of this is about formatting and section organization.
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/agent-pools.mdx
Show resolved
Hide resolved
|
||
Refer to [Change the agent pool architecture](/terraform/cloud-docs/agents/agent-pools#change-the-agent-pool-architecture) for instructions on how to switch between architectures. | ||
|
||
## Operating systems |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## Operating systems | |
## Operating Systems |
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Show resolved
Hide resolved
```plaintext | ||
│ Error: Incompatible provider version | ||
│ | ||
│ Provider registry.terraform.io/<namespace>/<name_version> does not have a package available for your current platform, <os_architecture>. | ||
│ | ||
│ Provider releases are separate from Terraform CLI releases, so not all providers are available for all platforms. Other versions of this provider may have | ||
│ different platforms supported. | ||
You can call the [Terraform provider registry API](/terraform/internals/provider-registry-protocol#list-available-versions) to confirm the architectures available for a provider. | ||
Sample request: | ||
```shell-session | ||
$ curl 'https://registry.terraform.io/v1/providers/hashicorp/random/versions' | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how but I think we could reformat this to be more readable. Right now it looks like this:
I think part of the issue is this part got formatted with the error message:
You can call the Terraform provider registry API to confirm the architectures available for a provider.
Sample request:
The sample response looks good though!
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Show resolved
Hide resolved
|
||
## Operating systems | ||
|
||
You can run the agent on Linux distributions for x86-64 or ARM64. You can also run the agent in a Docker container using the official [Terraform Cloud Agent image on Docker hub](https://hub.docker.com/r/hashicorp/tfc-agent). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can run the agent on Linux distributions for x86-64 or ARM64. You can also run the agent in a Docker container using the official [Terraform Cloud Agent image on Docker hub](https://hub.docker.com/r/hashicorp/tfc-agent). | |
[Agents](https://releases.hashicorp.com/tfc-agent/) support x86_64 bit and ARM64 Linux operating systems. You can also run the agent within Docker using our official [HCP Terraform Agent Docker container](https://hub.docker.com/r/hashicorp/tfc-agent). |
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did another review with the preview to better assess formatting
@natalie-todd - thanks for reviewing. I implemented your feedback where applicable. Some requests are in conflict with our style guide and information architecture strategy. Also, local previews are broken for me, so I can't verify some of the formatting until Vercel builds a preview. Please let me know if there are any changes to address the accuracy of the information. |
content/terraform-docs-agents/v1.22.x/docs/cloud-docs/agents/requirements.mdx
Outdated
Show resolved
Hide resolved
@@ -123,6 +123,58 @@ The agent maintains a registration and a liveness indicator within HCP Terraform | |||
|
|||
After initiating a graceful shutdown by either of these methods, the terminal user or parent program should wait for the agent to exit. The amount of time this exit takes depends on the agent's current workload. The agent waits for any current operations to complete before deregistering and exiting. | |||
|
|||
## Address provider compatibility errors | |||
|
|||
If a provider does not support your agent’s architecture, the following error appears in the agent’s TRACE logs, the CLI, or the UI: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the agent’s TRACE logs
Will this error text show up in the agent logs and not just the run logs @natalie-todd? I'd think no, since those provider incompatibility errors are emitted to stdout by the Terraform CLI during terraform init
, but maybe we're doing some parsing of stdout for that command that I don't know about. I'd expect that the agent logs will only show that Terraform exited with a non-zero exit code.
This PR adds docs for supporting for ARM-compiled agents.