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

Refactor Idea #221

Closed
misilot opened this issue Feb 8, 2022 · 9 comments · Fixed by #223
Closed

Refactor Idea #221

misilot opened this issue Feb 8, 2022 · 9 comments · Fixed by #223

Comments

@misilot
Copy link
Contributor

misilot commented Feb 8, 2022

To move the docker-compose.*.yml templates used with make docker-compose.yml to a sub directory.

Doing this would clean up the root directory and have the singular docker-compose.yml that is compiled + a system docker-compose.override.yml

@kstapelfeldt
Copy link

Don thinks this is a good idea (from the tech call).

@kstapelfeldt
Copy link

@misilot would you be willing to submit a pull request for this?

@misilot
Copy link
Contributor Author

misilot commented Feb 9, 2022

@kstapelfeldt i can take a stab at it. It will probably once I get a working install that I can migrate Config to from an older "tag/branch" of isle-dc

@misilot
Copy link
Contributor Author

misilot commented Feb 9, 2022

@kstapelfeldt would it make sense to move some of the other things into a "build" type of directory as well?

misilot added a commit to misilot/isle-dc that referenced this issue Feb 9, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
@kstapelfeldt
Copy link

I must say I don't know, and the meeting is over, but there was general enthusiasm for the direction you are suggesting! You could try the slack channel for more feedback?

misilot added a commit to misilot/isle-dc that referenced this issue Feb 9, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Feb 10, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Feb 11, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Feb 23, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Feb 23, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Feb 23, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
@nigelgbanks
Copy link
Contributor

nigelgbanks commented Feb 24, 2022

Something to add perhaps we could remove the requirement to make the docker-compose.yml and instead use
COMPOSE_FILE=docker/docker-compose.cantaloup.yml:docker/docker-compose.database.yml:... in the .env file.

@misilot
Copy link
Contributor Author

misilot commented Feb 24, 2022

using the make -B docker-compose.yml command has it's advantages that it compiles a single docker-compose.yml that you can use an docker-compose.override.yml with fairly easily.

You would still have to use make <somecommand> if the docker-compose.yml isn't being created on the system to have access to the environment variable COMPOSE_FILE, and users wouldn't be able to just run arbitrary docker-compose commands, unless I am not following what you are suggesting.

@nigelgbanks
Copy link
Contributor

@misilot if you have a .env file with COMPOSE_FILE set you can use docker compose commands directly, there isn't a need for the docker-compose.yml file in such a case.

What we would lose is the way that some compose files are included dynamically though settings like INCLUDE_WATCHTOWER_SERVICE=true, instead users would have to add the docker file to the list of docker files in COMPOSE_FILE, like COMPOSE_FILE=...:docker-compose.watchtower.yml

@misilot
Copy link
Contributor Author

misilot commented Feb 24, 2022

Oh, I was thinking of docker-compose and not docker compose. I am still in docker-compose land of thinking.

Which there are a lot of customization based on flags and the way variables are set, including all the variable interpolations, but if it can be uncomplicated I think that would be great!

misilot added a commit to misilot/isle-dc that referenced this issue Mar 4, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Mar 11, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Mar 24, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Mar 30, 2022
This makes the root a little cleaner

Fixes Islandora-Devops#221
misilot added a commit to misilot/isle-dc that referenced this issue Apr 5, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue Apr 13, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue May 11, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue May 17, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue May 20, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue Jun 2, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue Jun 6, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue Jun 6, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
misilot added a commit to misilot/isle-dc that referenced this issue Jun 6, 2022
Fixes Islandora-Devops#221

 - Update context to the new locations in docker-compose.*.yml files
 - Move tls.yml to `build` directory and rename file to traefik-tls.yml.
   Since this file is in SCM, it isn't expected to be modified.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants