Skip to content

Adminer docker compose dev#1323

Open
valentinRyckaert wants to merge 6 commits into
ElixirTeSS:masterfrom
valentinRyckaert:adminer-docker-compose-dev
Open

Adminer docker compose dev#1323
valentinRyckaert wants to merge 6 commits into
ElixirTeSS:masterfrom
valentinRyckaert:adminer-docker-compose-dev

Conversation

@valentinRyckaert

Copy link
Copy Markdown

Summary of changes

  • PostgreSQL DB was not accessible by the host in Docker development mode (no binding with host port)
  • Added an Adminer instance on port 8080 to facilitate DB management/observations

Motivation and context

  • No possibility of having access to the database using Docker in development.

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree to license it to the TeSS codebase under the BSD license.

@kennethrioja kennethrioja left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comments, else it works on my side! Thanks Valentin, this is very useful!

Comment thread docs/docker.md

<http://localhost:3000>


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this line

Comment thread docs/docker.md Outdated

### Access Adminer

An Adminer instance is accessible at the following URL:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add the link to Adminer https://www.adminer.org/
and add something like 'An Adminer instance to connect to and visualize your local database...'

Comment thread docker-compose.yml Outdated
volumes:
- db-data:/var/lib/postgresql/data
adminer:
image: adminer

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add the container name and a specific release tag of adminer, e.g.:

    container_name: ${PREFIX}-adminer
    image: adminer:5.4.2

Comment thread docker-compose.yml Outdated
image: adminer:5.4.2
restart: always
ports:
- 8080:8080

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- 8080:8080
- 8080:8080
depends_on:
- db

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is one last suggestion I have before merging the PR

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Docker-based development setup to make local database management easier by adding an Adminer service and documenting how to access it.

Changes:

  • Adds an Adminer container to the development docker-compose.yml, exposed on port 8080.
  • Documents Adminer access details in docs/docker.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
docs/docker.md Adds a short section describing how to access Adminer in dev.
docker-compose.yml Introduces an adminer service alongside existing dev containers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker-compose.yml
Comment on lines +40 to +42
adminer:
container_name: ${PREFIX}-adminer
image: adminer:5.4.2
Comment thread docker-compose.yml Outdated
Comment on lines +44 to +45
ports:
- 8080:8080
Comment thread docker-compose.yml Outdated
adminer:
container_name: ${PREFIX}-adminer
image: adminer:5.4.2
restart: always
Comment thread docs/docker.md Outdated
valentinRyckaert and others added 2 commits June 17, 2026 14:09
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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 this pull request may close these issues.

3 participants