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

[CI] Failure due to hard coded minio port #32701

Closed
tvernum opened this issue Aug 8, 2018 · 7 comments
Closed

[CI] Failure due to hard coded minio port #32701

tvernum opened this issue Aug 8, 2018 · 7 comments
Assignees
Labels
:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI

Comments

@tvernum
Copy link
Contributor

tvernum commented Aug 8, 2018

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.4+multijob-unix-compatibility/os=fedora/20/console

20:21:44 > Task :plugins:repository-s3:startMinio FAILED
20:21:44 Task ':plugins:repository-s3:startMinio' is not up-to-date because:
20:21:44   Task has not declared any outputs despite executing actions.
20:21:44 �7ERROR �8�[6CUnable to start the server: Port :60920 is already in use.
20:21:44 �7      �8�[6C> Please ensure no other program uses the same address/port.
20:21:44 :plugins:repository-s3:startMinio (Thread[Task worker for ':' Thread 13,5,main]) completed. Took 0.018 secs.
20:21:44 * Where:
20:21:44 :plugins:repository-s3:integTestMinioCluster#stop (Thread[Task worker for ':' Thread 13,5,main]) started.
20:21:44 Build file '/var/lib/jenkins/workspace/elastic+elasticsearch+6.4+multijob-unix-compatibility/os/fedora/plugins/repository-s3/build.gradle' line: 245

The tests for minio use a hard-coded port number (60920), which caused a CI failure.

final String minioAddress = "127.0.0.1:60920"

@tvernum tvernum added :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI labels Aug 8, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@tvernum
Copy link
Contributor Author

tvernum commented Aug 8, 2018

Another:
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+intake/2487/console

05:17:04 > Task :plugins:repository-s3:startMinio FAILED
...
05:17:04 �7ERROR �8�[6CUnable to start the server: Port :60920 is already in use.
05:17:04 �7      �8�[6C> Please ensure no other program uses the same address/port.

@bleskes
Copy link
Contributor

bleskes commented Aug 8, 2018

@tlrx can you take a look when you have a moment? Thanks.

@jasontedor
Copy link
Member

@ywelsch Might be interested too. If I recall from a past discussion this is a limitation at this time?

spinscale added a commit to spinscale/elasticsearch that referenced this issue Aug 10, 2018
spinscale added a commit that referenced this issue Aug 10, 2018
spinscale added a commit that referenced this issue Aug 10, 2018
@spinscale
Copy link
Contributor

another one: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+matrix-java-periodic/ES_BUILD_JAVA=java10,ES_RUNTIME_JAVA=java10,nodes=virtual&&linux/228/console

had a quick look and minio does not allow for binding to any port and report back, you have to specify a port between 1 and 65535 yourself

@ywelsch
Copy link
Contributor

ywelsch commented Aug 10, 2018

We can have it retry on a higher port. I'm astonished though that this is causing issues on CI as we have other places in our build with hard-coded ports.

@dnhatn
Copy link
Member

dnhatn commented Aug 13, 2018

Relates #32208

ywelsch added a commit that referenced this issue Aug 14, 2018
Minio does not support dynamic ports. The workaround here is to scan for a free port first. This is
not foolproof, but as we don't expect too many of these builds to run at once on the same machine,
this should do the trick.

Closes #32701
Closes #32208
ywelsch added a commit that referenced this issue Aug 14, 2018
Minio does not support dynamic ports. The workaround here is to scan for a free port first. This is
not foolproof, but as we don't expect too many of these builds to run at once on the same machine,
this should do the trick.

Closes #32701
Closes #32208
ywelsch added a commit that referenced this issue Aug 14, 2018
Minio does not support dynamic ports. The workaround here is to scan for a free port first. This is
not foolproof, but as we don't expect too many of these builds to run at once on the same machine,
this should do the trick.

Closes #32701
Closes #32208
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

8 participants