Stream: containers

Topic: โœ” sunsetting docker-aio


view this post on Zulip Oliver Bertuch (Mar 17 2023 at 13:49):

I added some small notes about the future of docker-aio in #9349 at https://github.com/IQSS/dataverse/pull/9439/commits/7bcadabdd80fb89f0b7fdc0c51af10f77f464a10. Maybe that helps.

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 17 2023 at 14:40):

I'm excited about having fewer files to touch every time we upgrade Payara, Postgres, and Solr. :sweat_smile:

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 17 2023 at 14:48):

A tricky think about docker-aio is that it's re-used by docker-dcm.

I do want to simply delete docker-aio. But what about docker-dcm? Perhaps we can move that code to https://github.com/sbgrid/data-capture-module

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 17 2023 at 14:58):

I just suggested the idea here: https://github.com/sbgrid/data-capture-module/issues/39

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 18 2023 at 01:15):

The reply from Pete (in that issue I opened):

"If you think it's worth copying them here, feel free to make a PR. But I think your idea about deleting, and going to git history if necessary, may be the way to go. The main point of docker-dcm was "dataverse configured for a DCM/RSAL"; bringing it in to this repo might have a bunch of "dangling references" to the dataverse repo. I'd also guess there have been changes in how to configure dataverse that docker-dcm / DCM/RSAL in general haven't kept up with."

I think he's right. We should probably just delete both docker-aio and docker-dcm and simply go back to git history if we want those old scripts.

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 22 2023 at 13:59):

During tech hours yesterday we talked about this a bit. I believe the idea is to first get API tests running in the new containers (no issue for this yet) and then remove docker-aio (since that's its main purpose, to run the API tests).

view this post on Zulip Philip Durbin ๐Ÿš€ (Mar 28 2023 at 18:31):

So far so good on running API tests against containerized Dataverse: https://dataverse.zulipchat.com/#narrow/stream/375812-containers/topic/running.20API.20tests/near/345172089

view this post on Zulip Philip Durbin ๐Ÿš€ (Aug 11 2023 at 09:49):

docker-aio and more is being removed here: https://github.com/IQSS/dataverse/pull/9685/commits/b829ca7a7802441845ef0ea6d878bde7c3651a2e

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 07 2023 at 11:05):

PR #9685 was merged. docker-aio is gone. I didn't even mention it in the 6.0 release notes (unmerged). If you think I should, please let me know in the release notes PR at #9878.

view this post on Zulip Oliver Bertuch (Sep 07 2023 at 11:07):

As it is a breaking change, I'd say it should be noted.

view this post on Zulip Oliver Bertuch (Sep 07 2023 at 11:07):

People might still use it and should know where to go to now.

view this post on Zulip Oliver Bertuch (Sep 07 2023 at 11:08):

"let me know" = leave a review comment in the PR?

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 07 2023 at 11:16):

Yes, please! A ```suggestion would be nice! Thanks!

view this post on Zulip Oliver Bertuch (Sep 07 2023 at 12:00):

https://github.com/IQSS/dataverse/pull/9878/files#r1318500231

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 07 2023 at 13:18):

Thanks! Committed! Resolving!

view this post on Zulip Notification Bot (Sep 07 2023 at 13:18):

Philip Durbin has marked this topic as resolved.


Last updated: Oct 30 2025 at 05:14 UTC