Skip to content

Conversation

@hanzei
Copy link
Contributor

@hanzei hanzei commented Apr 10, 2025

Summary

This PR adds a GitHub Actions workflow for testing Docker deployment capabilities, focusing on:

  1. Docker Compose Validation: Validates all Docker Compose configuration files (base, with NGINX, without NGINX)
  2. Container Deployment Test: Tests actual deployment of Mattermost containers without NGINX
  3. API Accessibility: Verifies that the Mattermost API is accessible and responding

The workflow is designed to ensure our Docker configurations remain valid and functional with every change to the repository.

Changes

The PR includes these key improvements:

  • Added workflow with validation and deployment jobs
  • Created comprehensive container checks
  • Added API endpoint accessibility testing
  • Implemented proper logging for troubleshooting

Test plan

  • Verify that the GitHub Actions workflow runs successfully on PR
  • Confirm that all Docker Compose files are properly validated
  • Ensure containers start successfully and API responds correctly

🤖 Generated with Claude Code

hanzei and others added 8 commits April 10, 2025 12:09
- Remove CircleCI configuration
- Add GitHub Actions workflow for shellcheck

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix variable quoting issues to prevent globbing and word splitting
- Fix variable reference typo (POSTGRES_POSTGRES_NEW_VERSION → POSTGRES_NEW_VERSION)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Create GitHub Actions workflow that tests Docker deployment
- Test both with and without NGINX configurations
- Verify containers run properly and services are accessible
- Follow steps from official documentation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Remove the step that was checking logs for a service named 'app'
- Fix CI failure due to 'no such service: app' error

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add generation of self-signed certificates for NGINX
- Update log checking to use service names from docker-compose: mattermost, postgres, nginx
- Fix HTTPS test with -k flag for self-signed certificate

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@hanzei hanzei closed this Apr 10, 2025
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.

2 participants