Skip to content

Add volumes input for mounting host directories into the Firebird container#12

Merged
juarezr merged 2 commits into
juarezr:masterfrom
mariuz:master
Apr 9, 2026
Merged

Add volumes input for mounting host directories into the Firebird container#12
juarezr merged 2 commits into
juarezr:masterfrom
mariuz:master

Conversation

@mariuz
Copy link
Copy Markdown
Contributor

@mariuz mariuz commented Apr 9, 2026

Users needed a way to mount host filesystem paths into the Firebird Docker container to access .fdb files directly from the host — previously only achievable by running Docker manually.

Changes

  • action.yml — New volumes input accepting newline-separated host_path:container_path mappings
  • entrypoint.sh — Parses INPUT_VOLUMES and appends --volume flags to the docker run invocation
  • testing.sh — Added volume test loop: starts container with a mapped volume, queries the DB, asserts the .fdb file is visible on the host
  • testing_changes.yml — New ci-volumes job testing volume mounting across Firebird versions latest, 5, 4, 3; verifies the database file lands on the host path
  • README.md — Documents the new parameter and adds it to the usage example

Usage

- uses: juarezr/firebirdsql-github-action@v2
  with:
    version: 'latest'
    firebird_database: 'my_database.fdb'
    firebird_user: 'my_user'
    firebird_password: 'my_password'
    volumes: '/tmp/firebird-data:/var/lib/firebird/data'

The host directory must exist and be writable before the action runs (e.g. sudo mkdir -p /tmp/firebird-data && sudo chmod 777 /tmp/firebird-data). Multiple volumes are supported — one per line.

Copilot AI and others added 2 commits April 9, 2026 10:20
…ests

Add `volumes` input for mounting host directories into the Firebird container
@mariuz
Copy link
Copy Markdown
Contributor Author

mariuz commented Apr 9, 2026

related issue #10

@juarezr juarezr added the enhancement New feature or request label Apr 9, 2026
@juarezr juarezr linked an issue Apr 9, 2026 that may be closed by this pull request
@juarezr juarezr merged commit 4df0efe into juarezr:master Apr 9, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for using the database from a mapped volume

3 participants