generator-jhipster icon indicating copy to clipboard operation
generator-jhipster copied to clipboard

migrate to docker-compose v2.

Open mshima opened this issue 3 years ago • 2 comments

Closes https://github.com/jhipster/generator-jhipster/issues/19462


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

mshima avatar Jun 10 '22 01:06 mshima

@mshima Can you fix conflicts?

mraible avatar Aug 17 '22 08:08 mraible

@mraible the PR is open so I will not forget about it, but I should work on it for v8 and refactor the files generation. There is a lot of work to be done.

mshima avatar Aug 17 '22 11:08 mshima

@DanielFran those files are test samples, which should be generated when the generator templates changes. But they are not updated, and I can’t blame because I myself never remember it’s existence. They should be removed.

If you look at the PR I’ve removed it’s use at docker-compose generator. We should do the same for others generators. But it’s a big task to check kurbernetes related generators and others related to workspaces and docker. We should implement https://github.com/jhipster/generator-jhipster/issues/19980 and then a similar task for base-docker.

mshima avatar Oct 13 '22 16:10 mshima

@mshima Yes I know. When we update image versions we sometimes forget to update those files... wanted to fix that issue. But if they are not used anymore, ok, we can then remove them after. Thanks

DanielFran avatar Oct 13 '22 16:10 DanielFran

For image versions we should add a version map for docker and Java like we have for node. When running tests, we should replace the versions with placeholders.

mshima avatar Oct 13 '22 16:10 mshima