Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fixed docker workflow #20

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Conversation

AntonioRodriguezRuiz
Copy link
Contributor

This pull request includes updates to the Docker deployment workflow to streamline the image building and pushing process. The most important changes involve modifying environment variables, updating job steps, and switching to a new action for multi-stage Docker builds.

Improvements to environment variables:

  • .github/workflows/docker-deploy.yml: Updated GHCR_IMAGE to dynamically format the image name based on the GitHub event type and branch reference. Introduced IMAGE_NAME for better readability and maintainability.

Changes to job steps:

  • .github/workflows/docker-deploy.yml: Removed steps for configuring QEMU, Docker Buildx, and logging into DockerHub, as they are no longer necessary.
  • .github/workflows/docker-deploy.yml: Updated the Build and Push Ghcr Image step to use the firehed/multistage-docker-build-action for a more efficient multi-stage Docker build process.For now only working on ghcr

@AntonioRodriguezRuiz AntonioRodriguezRuiz self-assigned this Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added the 🐳 Dockerize Indicates that the pull request is going to be dockerized label Nov 18, 2024
@github-actions github-actions bot added 🚀 CI/CD Indicates the change is related to CI/CD workflows 🐳 Build labels Nov 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 2 changed files in this pull request and generated no suggestions.

Files not reviewed (1)
  • Dockerfile: Language not supported
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added the 🐳 Dockerize Indicates that the pull request is going to be dockerized label Nov 18, 2024
@alvarobernal2412 alvarobernal2412 added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@alvarobernal2412 alvarobernal2412 force-pushed the fix/docker_deploy_workflow branch from 160f6b2 to 50e254e Compare November 18, 2024 15:38
@alvarobernal2412 alvarobernal2412 added 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Build and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized labels Nov 18, 2024
@github-actions github-actions bot added size/xs and removed size/s labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added 🐳 Dockerize Indicates that the pull request is going to be dockerized size/s and removed size/xs labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz added size/s 🐳 Dockerize Indicates that the pull request is going to be dockerized and removed 🐳 Dockerize Indicates that the pull request is going to be dockerized size/xs labels Nov 18, 2024
@github-actions github-actions bot added size/xs and removed size/s labels Nov 18, 2024
@AntonioRodriguezRuiz AntonioRodriguezRuiz merged commit a04c44e into main Nov 18, 2024
3 checks passed
@AntonioRodriguezRuiz AntonioRodriguezRuiz deleted the fix/docker_deploy_workflow branch November 18, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 CI/CD Indicates the change is related to CI/CD workflows 🐳 Dockerize Indicates that the pull request is going to be dockerized
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants