mirror of
https://github.com/neondatabase/neon.git
synced 2025-12-22 21:59:59 +00:00
61b6c4cf30
## Fixing GitHub workflow issue related to build and push images ## Summary of changes Followup of PR#608[move docker file from build repo to neon to solve issue some issues The build started failing because it missed a validation in logic that determines changes in the docker file Also, all the dependent jobs were skipped because of the build and push of the image job. To address the above issue following changes were made - we are adding validation to generate image tag even if it's a merge to repo. - All the dependent jobs won't skip even if the build and push image job is skipped. - We have moved the logic to generate a tag in the sub-workflow. As the tag name was necessary to be passed to the sub-workflow it made sense to abstract that away where it was needed and then store it as an output variable so that downward dependent jobs could access the value. - This made the dependency logic easy and we don't need complex expressions to check the condition on which it will run - An earlier PR was closed that tried solving a similar problem that has some feedback and context before creating this PR https://github.com/neondatabase/neon/pull/6175 ## Checklist before requesting a review - [x] Move the tag generation logic from the main workflow to the sub-workflow of build and push the image - [x] Add a condition to generate an image tag for a non-PR-related run - [x] remove complex if the condition from the job if conditions --------- Co-authored-by: Alexander Bayandin <alexander@neon.tech> Co-authored-by: Abhijeet Patil <abhijeet@neon.tech>
25 lines
220 B
Plaintext
25 lines
220 B
Plaintext
/pg_install
|
|
/target
|
|
/tmp_check
|
|
/tmp_check_cli
|
|
__pycache__/
|
|
test_output/
|
|
.vscode
|
|
.idea
|
|
neon.iml
|
|
/.neon
|
|
/integration_tests/.neon
|
|
|
|
# Coverage
|
|
*.profraw
|
|
*.profdata
|
|
|
|
*.key
|
|
*.crt
|
|
*.o
|
|
*.so
|
|
*.Po
|
|
|
|
# pgindent typedef lists
|
|
*.list
|