Skip to content

tests.unit.test_flow.TestInstalledAppFlow: test_run_local_server_audience failed #381

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

Closed
flaky-bot bot opened this issue Nov 13, 2024 · 3 comments · Fixed by #407 or #368
Closed

tests.unit.test_flow.TestInstalledAppFlow: test_run_local_server_audience failed #381

flaky-bot bot opened this issue Nov 13, 2024 · 3 comments · Fixed by #407 or #368
Assignees
Labels
flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Nov 13, 2024

Note: #308 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.


commit: f0ea4c3
buildURL: Build Status, Sponge
status: failed

@flaky-bot flaky-bot bot added flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. labels Nov 13, 2024
Copy link
Author

flaky-bot bot commented Nov 13, 2024

Looks like this issue is flaky. 😟

I'm going to leave this open and stop commenting.

A human should fix and close this.


When run at the same commit (f0ea4c3), this test passed in one build (Build Status, Sponge) and failed in another build (Build Status, Sponge).

@harkamaljot
Copy link
Contributor

sent in a pull request #394 to fix this

Copy link
Author

flaky-bot bot commented Feb 21, 2025

Oops! Looks like this issue is still flaky. It failed again. 😬

I reopened the issue, but a human will need to close it again.


commit: 9a1dfab
buildURL: Build Status, Sponge
status: failed

@flaky-bot flaky-bot bot reopened this Feb 21, 2025
@sai-sunder-s sai-sunder-s added priority: p3 Desirable enhancement or fix. May not be included in next release. and removed priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Apr 1, 2025
nolanleastin added a commit that referenced this issue Apr 1, 2025
tests.unit.test_flow.TestInstalledAppFlow attempts to create a new
server at the specified host:port. New server creation occasionally
results in 'address already in use' because the socket will be
unavailable for a period of time after the socket is closed, resulting
in flaky test failures.

Work around this in the tests by letting the OS pick an available port each time.

Fixes #381
nolanleastin added a commit that referenced this issue Apr 1, 2025
tests.unit.test_flow.TestInstalledAppFlow attempts to create a new
server at the specified host:port. New server creation occasionally
results in 'address already in use' because the socket will be
unavailable for a period of time after the socket is closed, resulting
in flaky test failures.

Work around this in the tests by letting the OS pick an available port each time.

Fixes #381
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
3 participants