Skip to content

Agregando env de credenciales de dockerhub#3

Merged
christopherBinaural merged 1 commit intoreleasefrom
rls_pr_check_tests
Nov 24, 2025
Merged

Agregando env de credenciales de dockerhub#3
christopherBinaural merged 1 commit intoreleasefrom
rls_pr_check_tests

Conversation

@christopherBinaural
Copy link
Contributor

No description provided.

@christopherBinaural christopherBinaural self-assigned this Nov 24, 2025
Copilot AI review requested due to automatic review settings November 24, 2025 14:54
@christopherBinaural christopherBinaural merged commit 88cb3a2 into release Nov 24, 2025
1 check passed
@christopherBinaural christopherBinaural deleted the rls_pr_check_tests branch November 24, 2025 14:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Este PR agrega las credenciales de Docker Hub como variables de entorno al workflow de GitHub Actions. El cambio renombra las variables de entorno de Docker registry de ORG_DOCKER_REGISTRY_* a DOCKER_REGISTRY_*, siguiendo el mismo patrón de nomenclatura usado para las variables S3 en el mismo workflow.

  • Renombra tres variables de entorno de Docker registry para seguir el patrón de nomenclatura consistente
  • Los secretos de GitHub mantienen el prefijo ORG_ mientras que las variables de entorno pasadas a la acción usan nombres simplificados
  • Permite que el action mount-cicd-tool tenga acceso a las credenciales del registro de Docker

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +103 to +105
DOCKER_REGISTRY_URL: ${{ secrets.ORG_DOCKER_REGISTRY_URL }}
DOCKER_REGISTRY_USERNAME: ${{ secrets.ORG_DOCKER_REGISTRY_USERNAME }}
DOCKER_REGISTRY_PASSWORD: ${{ secrets.ORG_DOCKER_REGISTRY_PASSWORD }}
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

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

La documentación al inicio del archivo debe actualizarse para incluir los secretos de Docker registry que ahora se están utilizando. Agregar:

  • ORG_DOCKER_REGISTRY_URL
  • ORG_DOCKER_REGISTRY_USERNAME
  • ORG_DOCKER_REGISTRY_PASSWORD

a la sección de secretos requeridos (líneas 3-10).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants