Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

Setup docker build process on PR instead of on Merge #437

Open
alex-dukhno opened this issue Nov 22, 2020 · 0 comments
Open

Setup docker build process on PR instead of on Merge #437

alex-dukhno opened this issue Nov 22, 2020 · 0 comments
Labels
C-Infrastructure CI and other infrastructure to test functionality and correctness

Comments

@alex-dukhno
Copy link
Owner

Currently, docker build publish process is configured on PR merge into master. It is done because PR from contributors can't access secret tokens. That prevents of running compatibility tests against new code.

Additional info: https://docs.github.com/en/free-pro-team@latest/packages/getting-started-with-github-container-registry/migrating-to-github-container-registry-for-docker-images
@alex-dukhno alex-dukhno added the C-Infrastructure CI and other infrastructure to test functionality and correctness label Nov 22, 2020
@alex-dukhno alex-dukhno added this to the 2020.Q4.Backlog milestone Nov 22, 2020
@alex-dukhno alex-dukhno mentioned this issue Dec 6, 2020
Closed
@alex-dukhno alex-dukhno modified the milestones: 2020.Q4.Backlog, Backlog Dec 31, 2020
@alex-dukhno alex-dukhno removed this from the Backlog milestone Dec 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
C-Infrastructure CI and other infrastructure to test functionality and correctness
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alex-dukhno