-
Notifications
You must be signed in to change notification settings - Fork 187
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
Run integration tests against an external agent process #552
Conversation
/assign @jkh52 |
Please fix the lint errors:
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jkh52, tallclair The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/label tide/merge-method-squash |
I fixed the security issues by excluding tests from the security linter. I think it's more likely to just create noise on test files. |
/lgtm |
arm64 build timeout /retest |
This PR enables the agent executed by integration tests to be launched as a separate process from a prebuilt binary, chosen by the
--agent-path
flag.Aside from the new agent runner, most of the changes in this PR are around the way metrics are tested, since they need to be scraped from the external process when used.
I will follow up with the external ProxyServer runner as a separate PR.
For #519