Skip to content

[core] Deflake test_state_api.py#53804

Merged
edoakes merged 2 commits intoray-project:masterfrom
edoakes:eoakes/deflake-test-state-api
Jun 13, 2025
Merged

[core] Deflake test_state_api.py#53804
edoakes merged 2 commits intoray-project:masterfrom
edoakes:eoakes/deflake-test-state-api

Conversation

@edoakes
Copy link
Copy Markdown
Collaborator

@edoakes edoakes commented Jun 13, 2025

Failing on postmerge because the agent doesn't start up due to port conflict.

The port selection logic is a total mess, so for now I'm just following the pattern of other tests in this file.

edoakes added 2 commits June 13, 2025 10:00
Signed-off-by: Edward Oakes <ed.nmi.oakes@gmail.com>
Signed-off-by: Edward Oakes <ed.nmi.oakes@gmail.com>
@edoakes edoakes requested a review from a team June 13, 2025 15:01
@edoakes edoakes added the go add ONLY when ready to merge, run all tests label Jun 13, 2025
Copy link
Copy Markdown
Contributor

@can-anyscale can-anyscale left a comment

Choose a reason for hiding this comment

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

@edoakes edoakes merged commit fa07ade into ray-project:master Jun 13, 2025
3 of 5 checks passed
elliot-barn pushed a commit that referenced this pull request Jun 18, 2025
Failing on postmerge because the agent doesn't start up due to port
conflict.

The port selection logic is a total mess, so for now I'm just following
the pattern of other tests in this file.

---------

Signed-off-by: Edward Oakes <ed.nmi.oakes@gmail.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
elliot-barn pushed a commit that referenced this pull request Jul 2, 2025
Failing on postmerge because the agent doesn't start up due to port
conflict.

The port selection logic is a total mess, so for now I'm just following
the pattern of other tests in this file.

---------

Signed-off-by: Edward Oakes <ed.nmi.oakes@gmail.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants