Commit Graph

184 Commits

Author SHA1 Message Date
a49704febc 🐛 fix: use relative paths instead of cd commands 2026-04-08 10:54:42 +02:00
3f718a7b9f 🐛 fix: correct working directory paths in CI workflow 2026-04-08 10:53:41 +02:00
db4922be05 🐛 fix: include Dockerfile.build in dependency hash calculation
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 3m27s
CI/CD Pipeline / CI Pipeline (push) Failing after 16s
2026-04-08 10:49:18 +02:00
b6b5a03942 🐛 fix: add Node.js to build cache for GitHub Actions compatibility
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 9s
CI/CD Pipeline / CI Pipeline (push) Failing after 15s
2026-04-08 10:47:31 +02:00
8ea380dfaa 🧪 test: trigger CI workflow
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 9s
CI/CD Pipeline / CI Pipeline (push) Failing after 27s
2026-04-08 10:43:43 +02:00
ac87850ea3 🐛 fix: use simple docker run with CACHE_IMAGE variable 2026-04-08 10:43:05 +02:00
dc2662e960 🐛 fix: use simple docker run with CACHE_IMAGE variable
- Bypass docker compose entirely

- Use CACHE_IMAGE variable for consistency

- Simpler volume mount approach
2026-04-08 10:27:51 +02:00
94e625b416 🐛 fix: use original docker run approach with proper volume mount
- Back to docker run instead of docker compose

- Use pwd for current directory volume mount

- Match original working approach from successful builds
2026-04-08 10:19:00 +02:00
76e3579f06 🐛 fix: try running go generate directly on host
- Bypass Docker entirely for Swagger generation

- Test if issue is Docker-specific or workspace-related

- Simplest possible approach to isolate the problem
2026-04-08 10:17:09 +02:00
fc410fdb48 🐛 fix: remove working directory flags to work with Gitea auto-mounting
- Remove -w flags that conflict with Gitea's auto-generated mounts

- Let Docker Compose use default working directory from docker-compose.yml

- Align with Gitea Actions volume mounting best practices
2026-04-08 10:16:03 +02:00
4e21f3b98f 🐛 fix: make both docker compose commands consistent
- Add cd command to both steps

- Use same working directory pattern

- Ensures consistent volume mount behavior
2026-04-08 09:53:33 +02:00
934a7d9eb5 🐛 fix: change working directory before docker compose
- Run docker compose from repository root

- Ensures volume mount . resolves to full repository

- Simpler approach than command-line volume override
2026-04-08 09:52:20 +02:00
52a933fe67 🐛 fix: use command-line volume mount for correct workspace path
- Override docker-compose.yml volume with explicit host path

- Mount full repository root to /workspace in container

- Simplifies debugging and ensures go.mod is accessible
2026-04-08 09:51:12 +02:00
641522c1c7 🐛 fix: add comprehensive debugging for go.mod issue
- Add detailed workspace structure debugging

- Search for go.mod file location

- Show directory contents before go generate
2026-04-08 09:37:45 +02:00
84d9ba2ff5 🐛 fix: use correct workspace path in Docker container
- Remove debug step

- Use /workspace instead of /workspace/arcodange/dance-lessons-coach

- Matches actual Docker volume mount structure
2026-04-08 09:33:40 +02:00
78f6bf393e 🐛 fix: add debugging to diagnose workspace structure
- Add debug step to check workspace directory structure

- Add pwd and ls commands to Swagger generation step

- Helps diagnose go.mod file not found issue
2026-04-08 09:32:27 +02:00
b5cc67642f 🐛 fix: simplify Swagger docs generation command
- Remove complex working directory and GOPATH settings

- Use simple cd and package path approach

- Matches local working command structure
2026-04-08 09:31:12 +02:00
b31c6e46f9 🐛 fix: use correct workspace path for Swagger docs generation
- Update working directory to full repository path

- Set GOPATH to match repository root

- Fixes go.mod file not found error in CI workflow
2026-04-08 09:30:08 +02:00
94f3704f0e 🐛 fix: set GOPATH correctly for Swagger docs generation
- Set GOPATH environment variable to workspace root

- Allow go generate to find go.mod from parent directories

- Fixes go.mod file not found error in CI workflow
2026-04-08 09:29:09 +02:00
689d1e2057 🐛 fix: correct working directory for Swagger docs generation
- Change working directory approach for go generate command

- Run from root workspace but cd to pkg/server for execution

- Fixes go.mod file not found error in CI workflow
2026-04-08 09:27:49 +02:00
b1d9dce29c 🔄 fix: trigger workflow manually
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 13s
CI/CD Pipeline / CI Pipeline (push) Failing after 13s
2026-04-08 09:24:09 +02:00
39bf083a23 🤖 ci: fix Docker Compose exec issue by using run instead
- Replace docker compose exec with docker compose run --rm

- Remove unnecessary container startup step

- Fixes 409 TCP upgrade error in CI workflow

- Maintains same functionality with better isolation
2026-04-08 09:23:04 +02:00
e333c69a71 🐛 fix: set DEPS_HASH as regular variable before export 2026-04-08 00:55:59 +02:00
841b23bb32 🐛 fix: debug DEPS_HASH value 2026-04-08 00:54:43 +02:00
58c8f09c77 🐛 fix: use needs output directly in docker compose step 2026-04-08 00:54:21 +02:00
5a2c896b4a 🐛 fix: add separate step to export DEPS_HASH 2026-04-08 00:53:58 +02:00
6eaca669cd 🐛 fix: use env var instead of needs output for DEPS_HASH 2026-04-08 00:53:21 +02:00
68a5cf7876 🐛 fix: set DEPS_HASH env var for ci-pipeline job 2026-04-08 00:52:47 +02:00
6f51a69854 🐛 fix: remove docker compose install from CI 2026-04-08 00:47:05 +02:00
a0d2bd6996 🐛 fix: correct database.ssl_mode environment variable binding
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 10s
CI/CD Pipeline / CI Pipeline (push) Failing after 17s
- Fix environment variable binding for SSL mode
- Use correct mapstructure tag name
- Match struct field binding

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:28:18 +02:00
07dc72d19e 🐛 fix: remove incorrect argument from LoadConfig call
- Fix compilation error
- LoadConfig takes no arguments

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:27:18 +02:00
076bc120e0 🐛 fix: make BDD test server use actual config with environment variables
- Modify createTestConfig to load real config
- Respect DLC_DATABASE_HOST and other environment variables
- Fallback to defaults if config loading fails
- Enable proper database connection configuration
- Fix BDD tests in Docker containers

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:26:27 +02:00
0026515237 🐛 fix: add missing database environment variable bindings
- Add BindEnv calls for database configuration
- DLC_DATABASE_HOST, DLC_DATABASE_PORT, DLC_DATABASE_USER
- DLC_DATABASE_PASSWORD, DLC_DATABASE_NAME, DLC_DATABASE_SSL_MODE
- Enable environment variable override for database config
- Fix BDD test database connection issues

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:25:01 +02:00
543e9e240a 🐛 fix: use shasum on macOS, sha256sum on Linux
- Detect which hash command is available
- Fix script compatibility across platforms
- Maintain same functionality

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:18:38 +02:00
5ffa60ef65 🐛 fix: use correct DLC_DATABASE_HOST for Docker vs native execution
- Use localhost for native execution (PostgreSQL port mapping)
- Use dance-lessons-coach-postgres for Docker container execution
- Set appropriate environment variables for each execution mode
- Fix container-to-container communication

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:15:57 +02:00
f18716d447 🔧 refactor: simplify BDD database configuration
- Remove unnecessary getPostgresHost() function
- Use direct localhost configuration
- PostgreSQL port is mapped to host, so localhost works everywhere
- Simplify code by removing unnecessary abstraction
- Use existing config system properly

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:10:31 +02:00
cf13b44e0d 🐛 fix: properly detect Docker container vs native execution
- Enhance getPostgresHost() to detect Docker environment
- Check multiple Docker-specific indicators
- Use container name when running inside Docker
- Use localhost when running natively
- Fix BDD test PostgreSQL connection issues

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-08 00:03:49 +02:00
f8474b69c1 🐛 fix: BDD tests PostgreSQL connection for native execution
- Simplify getPostgresHost() to always use localhost by default
- Use DLC_DATABASE_HOST=localhost for all local execution
- Container name only needed when running INSIDE Docker containers
- Fix native execution PostgreSQL connection issues
- Ensure BDD tests connect to localhost:5432 (mapped port)

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:52:57 +02:00
6c2399bea6 🔧 refactor: replace docker compose run with docker run --rm
- Use docker run --rm instead of docker compose run
- More explicit container cleanup with --rm flag
- Direct volume mounting and network configuration
- Maintain same functionality with cleaner approach
- Better matches CI/CD workflow style

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:51:54 +02:00
a1804d7aec 🐛 fix: add shared network configuration for Docker Compose services
- Add dance-lessons-coach-network to both compose files
- Ensure build-cache and postgres containers can communicate
- Fix container-to-container connectivity
- Maintain same network name for consistency

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:49:58 +02:00
22099d0ac6 🔧 refactor: simplify local CI/CD script by assuming Docker availability
- Remove conditional Docker availability checks
- Assume Docker is always available (required for workflow)
- Simplify logic flow
- Maintain same functionality but cleaner code
- Match CI/CD workflow assumptions

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:41:15 +02:00
18213f365d 🐛 fix: BDD tests PostgreSQL connection for native execution
- Enhance getPostgresHost() to detect Docker vs native execution
- Use localhost for native execution (PostgreSQL port mapping)
- Use container name for Docker execution
- Fix local CI/CD script to set correct DLC_DATABASE_HOST
- Ensure BDD tests can connect in both Docker and native modes

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:40:08 +02:00
9528b7d288 🐛 fix: BDD tests PostgreSQL connection using DLC_* environment variables
- Add getPostgresHost() function to use DLC_DATABASE_HOST env var
- Default to dance-lessons-coach-postgres container name
- Update local CI/CD script to set proper DLC_* environment variables
- Fix Docker Compose test execution to pass correct database config
- Ensure BDD tests can connect to PostgreSQL in both Docker and native modes

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:38:27 +02:00
f54c60c6a3 🔧 refactor: simplify CI/CD workflow by removing redundant cache checks
- Remove duplicate steps section
- Eliminate redundant cache availability checks
- Simplify logic since build-cache job guarantees cache availability
- Remove unnecessary conditional execution
- Streamline Docker Compose usage

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:28:47 +02:00
9213c97da6 🔧 fix: optimize Docker cache usage by removing redundant go mod tidy
- Remove unnecessary go mod tidy execution when using Docker cache
- Dependencies are already pre-installed in the cache image
- Only run go mod tidy when not using cache (native execution)
- This leverages the true purpose of the Docker build cache

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:22:30 +02:00
da0dc0e30f 🔧 chore: align local CI/CD script with Gitea workflow
- Add dependency hash calculation matching CI workflow
- Add Docker cache detection and usage
- Add PostgreSQL service with Docker Compose
- Execute commands in Docker Compose containers when cache available
- Update test summary to reflect new workflow matching

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:20:52 +02:00
2ca361ccd9 🔧 chore: align Gitea workflow with local CI/CD script
- Add go mod tidy step to match local script
- Execute build.sh script in Docker Compose container
- Maintain consistent build process between local and CI
- Ensure Docker cache compatibility

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:18:22 +02:00
02674dfb50 feat: migrate to Docker Compose for build cache management
- Replace docker run commands with docker compose
- Use separate docker-compose.build.yml for build cache
- Maintain PostgreSQL service with Docker Compose
- Ensure consistent container networking

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-07 23:15:26 +02:00
4848becbdc 🐛 fix: set working directory directly in Docker command instead of using cd 2026-04-07 22:30:05 +02:00
b562cad09d 🤖 ci: final trigger attempt
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 9s
CI/CD Pipeline / CI Pipeline (push) Failing after 39s
2026-04-07 19:33:27 +02:00