Skip to content
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

[Flaky Test]: TestProxyURL_ProxyInThePolicyTakesPrecedence – did not find requests to the proxy defined in the policy #4333

Closed
rdner opened this issue Feb 23, 2024 · 5 comments
Assignees
Labels
flaky-test Unstable or unreliable test cases. Team:Elastic-Agent Label for the Agent team

Comments

@rdner
Copy link
Member

rdner commented Feb 23, 2024

Failing test case

TestProxyURL_ProxyInThePolicyTakesPrecedence

Error message

did not find requests to the proxy defined in the policy

Build

https://buildkite.com/elastic/elastic-agent/builds/7468#018dd6d2-937b-4f06-9079-dd95111ef62d

OS

Windows

Stacktrace and notes

proxytest.go:62: [proxy-proxy-1] [77f2d3d0-78f2-4208-8f22-0f6601e891f5] DONE 200 - POST http://localhost:49970/api/fleet/agents/proxy-url-agent-id/checkin? HTTP/1.1 [::1]:49987
    proxytest.go:62: [proxy-proxy-1] [8578b348-0418-4255-ae73-0f04ae99385b] STARTING - POST http://fleet.elastic.co/api/fleet/agents/proxy-url-agent-id/checkin? HTTP/1.1 [::1]:49987
    proxytest.go:62: [proxy-proxy-1] original URL: http://fleet.elastic.co/api/fleet/agents/proxy-url-agent-id/checkin?, new URL: http://localhost:49970/api/fleet/agents/proxy-url-agent-id/checkin?
    fleetserver.go:123: [fleet-server] [b858478b-31b9-4184-915b-1b669af84c2a] STARTING - POST /api/fleet/agents/proxy-url-agent-id/checkin? HTTP/1.1 [::1]:49982
    fleetserver.go:123: [fleet-server] [b858478b-31b9-4184-915b-1b669af84c2a] DONE 200 - POST /api/fleet/agents/proxy-url-agent-id/checkin? HTTP/1.1 [::1]:49982
    proxytest.go:62: [proxy-proxy-1] [8578b348-0418-4255-ae73-0f04ae99385b] DONE 200 - POST http://localhost:49970/api/fleet/agents/proxy-url-agent-id/checkin? HTTP/1.1 [::1]:49987
    proxy_url_test.go:242: 
        	Error Trace:	C:/Users/windows/agent/testing/integration/proxy_url_test.go:242
        	Error:      	Condition never satisfied
        	Test:       	TestProxyURL_ProxyInThePolicyTakesPrecedence
        	Messages:   	did not find requests to the proxy defined in the policy. Want [http://localhost:49972] on [GET - http localhost:49970 http://localhost:49970/api/status?]
    fixture_install.go:211: [test TestProxyURL_ProxyInThePolicyTakesPrecedence] Inside fixture cleanup function
    fixture_install.go:227: collecting diagnostics; test failed
    fixture.go:632: >> running binary with: [C:\Program Files\Elastic\Agent\elastic-agent.exe diagnostics -f C:\Users\windows\agent\build\diagnostics\TestProxyURL_ProxyInThePolicyTakesPrecedence-diagnostics-2024-02-23T17-07-51Z.zip]
    fixture.go:632: >> running binary with: [C:\Program Files\Elastic\Agent\elastic-agent.exe uninstall --force]
    fixture_install.go:186: Dumping running processes in C:\Users\windows\agent\build\diagnostics\TEST-TestProxyURL_ProxyInThePolicyTakesPrecedence-windows-amd64-ProcessDump.json
--- FAIL: TestProxyURL_ProxyInThePolicyTakesPrecedence (351.73s)
@rdner rdner added Team:Elastic-Agent Label for the Agent team flaky-test Unstable or unreliable test cases. labels Feb 23, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

@rdner
Copy link
Member Author

rdner commented Feb 26, 2024

Related failures: TestProxyURL_EnrollProxyAndNoProxyInThePolicy and TestProxyURL_RemoveProxyFromThePolicy failed in https://buildkite.com/elastic/elastic-agent/builds/7481#018de487-bbd5-4596-8a58-5b3e2856b046

@AndersonQ
Copy link
Member

Related failures: TestProxyURL_EnrollProxyAndNoProxyInThePolicy and TestProxyURL_RemoveProxyFromThePolicy failed in https://buildkite.com/elastic/elastic-agent/builds/7481#018de487-bbd5-4596-8a58-5b3e2856b046

I had a look at this build, those failures do not seem to be related to the originak issue. Why do you think they're related?

@rdner
Copy link
Member Author

rdner commented Mar 12, 2024

@AndersonQ you're right, it seems to be #4377 not this one. Please ignore #4333 (comment)

The proxy tests are still flaky though, so this issue is still valid.

@pierrehilbert
Copy link
Contributor

Closing as it seems to be a duplicate of #4861

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Unstable or unreliable test cases. Team:Elastic-Agent Label for the Agent team
Projects
None yet
Development

No branches or pull requests

5 participants