Skip to content

Strengthen CI with shell linting, multi-arch smoke tests, and gated image publishing#71

Draft
Copilot wants to merge 8 commits intomainfrom
copilot/improve-ci-tests-alpine-php-webserver
Draft

Strengthen CI with shell linting, multi-arch smoke tests, and gated image publishing#71
Copilot wants to merge 8 commits intomainfrom
copilot/improve-ci-tests-alpine-php-webserver

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 19, 2026

  • Revisar el workflow actual, el smoke test y la integración de ZAP
  • Ejecutar lint y validaciones existentes antes de cambiar nada
  • Sustituir el job de ZAP por una comprobación web/PHP más simple y mantenible
  • Mantener el gate de CI para que publish dependa de esa validación
  • Revalidar YAML, scripts y diff final
  • Ejecutar code review y CodeQL, corregir hallazgos y cerrar
Original prompt

This section details on the original issue you should resolve

<issue_title>Improve Automatic Tests for CI on alpine-php-webserver Using GitHub Actions</issue_title>
<issue_description>The current setup for Continuous Integration (CI) in the alpine-php-webserver repository could be enhanced to provide more robust and comprehensive automated testing for the Docker image. Improving the testing process will help ensure higher reliability and prevent regressions when introducing new features or updates.


Proposed Improvements

  1. Add Multi-Platform Build and Tests:

    • Ensure the image is tested on multiple architectures (amd64, arm64, etc.) using Docker Buildx.
    • Validate compatibility and functionality across platforms.
  2. Lint Dockerfile and Shell Scripts:

  3. Run Containerized Tests:

    • Test the Docker image by running it and executing basic PHP scripts.
    • Verify Nginx and PHP-FPM functionality by simulating web requests using tools like curl or PHPUnit tests for web servers.
  4. Check for Vulnerabilities:

  5. Cache Optimization:

    • Implement caching strategies for Docker layers and Composer dependencies during the CI process to speed up builds.
  6. Automate Publishing on Successful Tests:

    • Automatically publish the Docker image to the registry only if all tests pass and on specific triggers (e.g., tags or branches like main).

Acceptance Criteria

  • CI workflow runs tests for multi-platform builds and passes without errors.
  • Dockerfile and scripts are linted during the CI workflow.
  • Tests verify PHP and Nginx configurations work as expected.
  • Vulnerability scan integrates into the CI pipeline with clear reporting.
  • Workflow outputs are clear and informative for debugging.

Additional Context

Implementing these improvements will align the repository with modern CI/CD practices and increase confidence in the image's stability and security. It also provides contributors with immediate feedback on their changes.
</issue_description>

Comments on the Issue (you are @copilot in this section)


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Copilot AI and others added 2 commits March 19, 2026 16:35
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve automatic tests for CI on alpine-php-webserver using GitHub Actions Strengthen CI with shell linting, multi-arch smoke tests, and gated image publishing Mar 19, 2026
Copilot AI requested a review from erseco March 19, 2026 16:38
Copilot AI and others added 3 commits March 19, 2026 17:56
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Copilot AI and others added 2 commits March 19, 2026 20:32
Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>
Co-authored-by: erseco <1876752+erseco@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.

Improve Automatic Tests for CI on alpine-php-webserver Using GitHub Actions

2 participants