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.

Sign up for GitHub

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

Jump to bottom

Ports conflict when running api server tests on Windows #6939

Closed
feloy opened this issue Jun 28, 2023 · 0 comments · Fixed by #6995
Closed

Ports conflict when running api server tests on Windows #6939

feloy opened this issue Jun 28, 2023 · 0 comments · Fixed by #6995
Assignees
@rm3l
Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering area/Windows Issues or PRs specific to Windows kind/bug Categorizes issue or PR as related to a bug.
Milestone
v3.13.0 🚀

Comments

@feloy
Copy link
Contributor

feloy commented Jun 28, 2023 •
edited
Loading

/kind bug

[FAILED] [137.279 seconds]
odo dev command with api server tests when the component is bootstrapped when odo dev is run with --api-server flag [BeforeEach] should serve endpoints
  [BeforeEach] C:/Users/Administrator.ANSIBLE-TEST-VS/4281/tests/integration/cmd_dev_api_server_test.go:87
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/4281/tests/integration/cmd_dev_api_server_test.go:101

[...]
  [odo] I0628 06:37:19.756636    2904 starterserver.go:78] Stopping the API server; encountered error: listen tcp :20000: bind: Only one usage of each socket address (protocol/network address/port) is normally permitted.

Examples:
@openshift-ci openshift-ci bot added the kind/bug Categorizes issue or PR as related to a bug. label Jun 28, 2023
@feloy feloy changed the title Ports conflict when running api server tests won Windows Ports conflict when running api server tests on Windows Jun 28, 2023
@github-actions github-actions bot added the needs-triage Indicates an issue or PR lacks a `triage/*` and requires one. label Jun 28, 2023
@rm3l rm3l added area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering area/Windows Issues or PRs specific to Windows and removed needs-triage Indicates an issue or PR lacks a `triage/*` and requires one. labels Jul 5, 2023
This was referenced Jul 5, 2023
Merged
Merged
Merged
This was referenced Jul 17, 2023
Merged
Merged
Merged
Merged
rm3l added a commit to feloy/odo that referenced this issue Jul 20, 2023
@rm3l
Disable --api-server by default in the tests
f442fd4 
They are enabled only if `options.StartAPIServer`
is explicitly enabled.
This is to avoid potential port conflicting issue,
especially on Windows - see [1]

[1] redhat-developer#6939
@rm3l rm3l mentioned this issue Jul 21, 2023
Merged
3 tasks
@rm3l rm3l self-assigned this Jul 21, 2023
@rm3l rm3l moved this to In Review 👀 in odo Project Jul 21, 2023
@rm3l rm3l added this to the v3.13.0 🚀 milestone Jul 21, 2023
@rm3l rm3l mentioned this issue Jul 21, 2023
Merged
3 tasks
@github-project-automation github-project-automation bot moved this from In Review 👀 to Done ✅ in odo Project Jul 21, 2023
rm3l added a commit to feloy/odo that referenced this issue Jul 21, 2023
@rm3l
Disable --api-server by default in the tests
022a990 
They are enabled only if `options.StartAPIServer`
is explicitly enabled.
This is to avoid potential port conflicting issue,
especially on Windows - see [1]

[1] redhat-developer#6939
rm3l added a commit to feloy/odo that referenced this issue Jul 21, 2023
@rm3l
Disable --api-server by default in the tests
b35407b 
They are enabled only if `options.StartAPIServer`
is explicitly enabled.
This is to avoid potential port conflicting issue,
especially on Windows - see [1]

[1] redhat-developer#6939
openshift-merge-robot pushed a commit that referenced this issue Jul 21, 2023
@feloy @rm3l
Remove API Server from experimental mode, set UI Server as experiment…
9605c92 
…al (#6985)

* Remove API Server from experimental mode, set UI Server as experimental

* Adapt api-server integration tests

* Disable --api-server by default in the tests

They are enabled only if `options.StartAPIServer`
is explicitly enabled.
This is to avoid potential port conflicting issue,
especially on Windows - see [1]

[1] #6939

* Error out if `--api-server-port` is set but `--api-server` is false

---------

Co-authored-by: Armel Soro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rm3l rm3l

Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering area/Windows Issues or PRs specific to Windows kind/bug Categorizes issue or PR as related to a bug.
Projects
odo Project
Archived in project
Milestone
v3.13.0 🚀
2 participants
@rm3l @feloy