-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[BUG] Upgrade minio image to more recent version #3539
Labels
Comments
lukas-vlcek
changed the title
[BUG] Update minio image to more recent version
[BUG] Upgrade minio image to more recent version
Jun 8, 2022
@lukas-vlcek assigned this to you as you've already picked it up. |
saratvemulapalli
added
dependencies
Pull requests that update a dependency file
and removed
untriaged
labels
Jun 8, 2022
lukas-vlcek
added a commit
to lukas-vlcek/OpenSearch
that referenced
this issue
Jun 27, 2022
On top of that addressing MinIO warning: - setup ulimits - use of static console port - do not use deprecated argument names With regard to static ports this is related to a change MinIO introduced in RELEASE.2021-07-08T01-15-01Z where the console was embedded into the server itself. For more details visit: https://docs.min.io/minio/baremetal/console/minio-console.html We could let the port be assigned dynamically, however, when we set it statically it can give user an option to make use of the console web UI for MinIO troubleshooting (though I think the ports still need to be exported in this case). Closes opensearch-project#3539 Signed-off-by: Lukáš Vlček <lukas.vlcek@aiven.io>
saratvemulapalli
pushed a commit
that referenced
this issue
Jul 12, 2022
On top of that addressing MinIO warning: - setup ulimits - use of static console port - do not use deprecated argument names With regard to static ports this is related to a change MinIO introduced in RELEASE.2021-07-08T01-15-01Z where the console was embedded into the server itself. For more details visit: https://docs.min.io/minio/baremetal/console/minio-console.html We could let the port be assigned dynamically, however, when we set it statically it can give user an option to make use of the console web UI for MinIO troubleshooting (though I think the ports still need to be exported in this case). Closes #3539 Signed-off-by: Lukáš Vlček <lukas.vlcek@aiven.io>
opensearch-trigger-bot bot
pushed a commit
that referenced
this issue
Jul 12, 2022
On top of that addressing MinIO warning: - setup ulimits - use of static console port - do not use deprecated argument names With regard to static ports this is related to a change MinIO introduced in RELEASE.2021-07-08T01-15-01Z where the console was embedded into the server itself. For more details visit: https://docs.min.io/minio/baremetal/console/minio-console.html We could let the port be assigned dynamically, however, when we set it statically it can give user an option to make use of the console web UI for MinIO troubleshooting (though I think the ports still need to be exported in this case). Closes #3539 Signed-off-by: Lukáš Vlček <lukas.vlcek@aiven.io> (cherry picked from commit 9af3407)
reta
pushed a commit
that referenced
this issue
Jul 13, 2022
On top of that addressing MinIO warning: - setup ulimits - use of static console port - do not use deprecated argument names With regard to static ports this is related to a change MinIO introduced in RELEASE.2021-07-08T01-15-01Z where the console was embedded into the server itself. For more details visit: https://docs.min.io/minio/baremetal/console/minio-console.html We could let the port be assigned dynamically, however, when we set it statically it can give user an option to make use of the console web UI for MinIO troubleshooting (though I think the ports still need to be exported in this case). Closes #3539 Signed-off-by: Lukáš Vlček <lukas.vlcek@aiven.io> (cherry picked from commit 9af3407) Co-authored-by: Lukáš Vlček <lukas.vlcek@aiven.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Describe the bug
tl;dr:
Shall we upgrade minio image version?
More details:
The minio image version used in fixtures is quite dated:
OpenSearch/test/fixtures/minio-fixture/Dockerfile
Line 1 in 81a77aa
As of writing when docker compose of the minio fixture is run we can see three containers being started like this:
If you check logs of any of these containers you can see this:
As you can see the container itself logs the recommendation to update the image to new version:
To Reproduce
Run any task that includes
:test:fixtures:minio-fixture:composeUp
and check container logs.
Expected behavior
Host/Environment (please complete the following information):
Additional context
I tried to update the Dockerfile to use more recent version of the minio image (as suggested by minio itself) like this:
After this change the
docker logs <containerID>
changes to:We are getting three WARNINGs:
ulimits
I was able to get rid of one of the WARNINGs right away by specifying
ulimits
for each service in the docker-compose.yml like this:I believe the remaining two WARNINGs are solvable too.
Feedback welcome.
The text was updated successfully, but these errors were encountered: