Skip to content

Fix GitHub Actions workflows and documentation after Dockerfile consolidation#130

Merged
jcpunk merged 2 commits intomodernizefrom
copilot/sub-pr-129
Jan 2, 2026
Merged

Fix GitHub Actions workflows and documentation after Dockerfile consolidation#130
jcpunk merged 2 commits intomodernizefrom
copilot/sub-pr-129

Conversation

Copy link

Copilot AI commented Jan 2, 2026

The Dockerfiles were moved to dockerfiles/ directory in commit 95d3a09, but the GitHub Actions workflows still referenced the old locations without specifying the file parameter, causing builds to fail.

Changes

  • GitHub Actions workflows: Added file parameter to docker/build-push-action to explicitly specify Dockerfile paths:

    • build_docker_image.yml: dockerfiles/Dockerfile
    • build_docker_dependencies_image.yml: dockerfiles/Dockerfile.dependencies
  • Documentation: Updated docs/README.md:

    • Removed references to obsolete microservices (config-service, ers-dbwriter, opmon-dbwriter)
    • Fixed docker build context path from /dockerfiles to ./dockerfiles

Example

- name: Build and push Docker images
  uses: docker/build-push-action@v3
  with:
    context: ${{ github.workspace }}/microservices/
    file: ${{ github.workspace }}/microservices/dockerfiles/Dockerfile  # Added
    push: true
    tags: ${{ steps.meta.outputs.tags }}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ocumentation

Co-authored-by: jcpunk <3534830+jcpunk@users.noreply.github.com>
Copilot AI changed the title [WIP] Modernize, minimize, and cleanup containers Fix GitHub Actions workflows and documentation after Dockerfile consolidation Jan 2, 2026
Copilot AI requested a review from jcpunk January 2, 2026 19:00
@jcpunk jcpunk marked this pull request as ready for review January 2, 2026 19:02
@jcpunk jcpunk merged commit 4b13d45 into modernize Jan 2, 2026
@jcpunk jcpunk deleted the copilot/sub-pr-129 branch January 2, 2026 19:02
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