Commit Graph

193 Commits

Author SHA1 Message Date
f013a1771d fixed bdd tests 2026-04-08 12:05:11 +02:00
f34a41a1c7 feat: enhance gitea-client with trigger-workflow command 2026-04-08 11:37:39 +02:00
3f84c91495 🔧 refactor: add fallback to cache check for reliability 2026-04-08 11:29:11 +02:00
48cb881af0 perf: optimize cache check using docker manifest inspect 2026-04-08 11:21:24 +02:00
ec869d6d5b 📝 docs: update CI/CD documentation for container/services approach 2026-04-08 11:19:25 +02:00
CI Bot
db60de80ad 🤖 chore: update coverage badge to 8.4% [skip ci] 2026-04-08 09:10:55 +00:00
cc3e254292 🐛 fix: add database environment variables for CI 2026-04-08 11:07:11 +02:00
00c5d26db0 🐛 fix: add PostgreSQL client to build cache
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 3m15s
CI/CD Pipeline / CI Pipeline (push) Failing after 2m59s
2026-04-08 10:59:42 +02:00
d4bd11a3df 🐛 fix: remove unnecessary set up build environment step 2026-04-08 10:56:24 +02:00
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