Skip to content

Conversation

@lafentres
Copy link
Contributor

@lafentres lafentres commented Aug 19, 2025

Description

This PR updates the request forwarding docs to show the logs to expect when an agent has successfully connected with request forwarding enabled. These logs changed in v1.15.1 of the agent, so I've updated the docs back to that version.

Testing Plan

You'll need an HCP Terraform account with the Premium tier entitlement.

  1. Follow instructions in the docs to connect an agent, making sure to turn the request forwarding flag on and debug logging enabled -request-forwarding=true -log-level=debug
  2. Check that the successful connection logs match what is here.
  3. Check the docs for any typos.

@lafentres lafentres self-assigned this Aug 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 19, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Wed Aug 20 16:06:58 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Wed Aug 20 16:01:24 UTC 2025

@github-actions
Copy link
Contributor

github-actions bot commented Aug 19, 2025

Broken Link Checker

No broken links found! 🎉

@lafentres lafentres changed the title [DRAFT] Update logs that indicate a successful request forwarding connection Update logs that indicate a successful request forwarding connection Aug 20, 2025
@lafentres lafentres force-pushed the lafentres.update-broker-connection-success-docs branch from 980dfe9 to 0f0e7ed Compare August 20, 2025 14:21
@lafentres lafentres marked this pull request as ready for review August 20, 2025 14:22
@lafentres lafentres requested a review from a team as a code owner August 20, 2025 14:22
@lafentres lafentres force-pushed the lafentres.update-broker-connection-success-docs branch from 0f0e7ed to 850e90f Compare August 20, 2025 15:47
Copy link
Contributor

@natalie-todd natalie-todd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing this! It is always a bit of a challenge to keep log docs up to date as logs are subject to change so frequently.

@lafentres lafentres merged commit 5e6b550 into main Aug 22, 2025
9 checks passed
@lafentres lafentres deleted the lafentres.update-broker-connection-success-docs branch August 22, 2025 14:17
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.

2 participants