Skip to content

Commit

Permalink
frontend: fix docker push
Browse files Browse the repository at this point in the history
frontend_publish_ui_image extends
`.images:dind:docker-push-to-vdk-repos` but since both provide
`before_script` the frontend_publish_ui_image overwrites the one
provided by .images:dind:docker-push-to-vdk-repos which adds the
docker_push_vdk.sh script to the PATH.

This caused failures in the pipeline like this
https://gitlab.com/vmware-analytics/versatile-data-kit/-/pipelines/947073454

The fix is to remove  before_script from the `frontend_publish_ui_image`
job and move all to `script`
  • Loading branch information
antoniivanov committed Jul 27, 2023
1 parent c3567a9 commit dc2aece
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions projects/frontend/cicd/.gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -102,10 +102,10 @@ frontend-data-pipelines-release:
expire_in: 1 week

frontend_publish_ui_image:
extends: .images:dind:docker-push-to-vdk-repos
stage: pre_release_image
before_script:
- cd projects/frontend/data-pipelines/gui
script:
- cd projects/frontend/data-pipelines/gui
- apk --no-cache add bash git
- docker login --username "${VDK_DOCKER_REGISTRY_USERNAME}" --password "${VDK_DOCKER_REGISTRY_PASSWORD}" "${VDK_DOCKER_REGISTRY_URL}"
- ../../cicd/publish_image_docker.sh vdk-operations-ui . $CI_PIPELINE_ID
Expand All @@ -116,7 +116,6 @@ frontend_publish_ui_image:
changes: *frontend_shared_components_locations
- if: '$CI_COMMIT_BRANCH == "main"'
changes: *frontend_data_pipelines_locations
extends: .images:dind:docker-push-to-vdk-repos

frontend_tag_ui_image_stable:
stage: release_image
Expand Down

0 comments on commit dc2aece

Please sign in to comment.