Skip to content

Update proxytest configuration#384

Merged
khushijain21 merged 4 commits intoelastic:mainfrom
khushijain21:addproxy
Jan 29, 2026
Merged

Update proxytest configuration#384
khushijain21 merged 4 commits intoelastic:mainfrom
khushijain21:addproxy

Conversation

@khushijain21
Copy link
Copy Markdown
Contributor

@khushijain21 khushijain21 commented Jan 28, 2026

What does this PR do?

proxytest from elastic-agent repo was moved to elastic-agent-libs which also included a bug fix in [this PR]#358). But a follow up PR to remove this package from elastic-agent was missed.

In the meantime, some changes were made to proxytest package on elastic-agent repo in this PR.

To avoid anymore future divergence and consolidate our changes into one package - I am updating the changes here. After this, will immediately delete the proxytest package on elastic-agent repo by this pull request

Why is it important?

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works

Related issues

@khushijain21 khushijain21 requested a review from a team as a code owner January 28, 2026 14:39
@khushijain21 khushijain21 requested review from andrzej-stencel and mauri870 and removed request for a team January 28, 2026 14:39
mauri870
mauri870 previously approved these changes Jan 28, 2026
Copy link
Copy Markdown
Member

@mauri870 mauri870 left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
@khushijain21 khushijain21 enabled auto-merge (squash) January 29, 2026 04:58
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

@khushijain21 khushijain21 merged commit 86c452f into elastic:main Jan 29, 2026
2 of 5 checks passed
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.

4 participants