Skip to content

Conversation

@DeltaMichael
Copy link
Contributor

@DeltaMichael DeltaMichael commented May 26, 2023

Why

Re-deploying the helm chart from a CI pipeline that is not related to control service, e.g. frontend, requires a way to track control service docker images other than commit hashes.

What

Introduce latest and stable tags for control service docker images. Each docker image is tagged with latest in the publish_artifacts stage. Once the image is deployed in the test environment and tests have run, it is tagged with stable.

How was this tested

Ran new and changed scripts locally.
CI https://gitlab.com/vmware-analytics/versatile-data-kit/-/pipelines/882279167

What type of change are you making

Breaking change to CI

@DeltaMichael DeltaMichael marked this pull request as draft May 26, 2023 13:58
@DeltaMichael DeltaMichael force-pushed the person/mdilyan/control-service-tags branch 5 times, most recently from 039332e to d49dd2d Compare May 29, 2023 16:03
Why

Re-deploying the helm chart from a CI pipeline
that is not related to control service, e.g. frontend,
requires a way to track control service docker images
other than commit hashes.

What

Introduce latest and stable tags for control service
docker images. Each docker image is tagged with latest
in the publish_artifacts stage. Once the image is
deployed in the test environment and tests have run,
it is tagged with stable.

How was this tested

Ran new and changed scripts locally.
CI

What type of change are you making

Breaking change to CI

Signed-off-by: Dilyan Marinov <mdilyan@vmware.com>
@DeltaMichael DeltaMichael force-pushed the person/mdilyan/control-service-tags branch 2 times, most recently from 474c265 to d0901e0 Compare May 30, 2023 07:13
@DeltaMichael DeltaMichael marked this pull request as ready for review May 30, 2023 07:13
@DeltaMichael DeltaMichael merged commit 2cbc6ec into main Jun 1, 2023
@DeltaMichael DeltaMichael deleted the person/mdilyan/control-service-tags branch June 1, 2023 07:15
DeltaMichael added a commit that referenced this pull request Jun 9, 2023
Stacked on top of #2138

## Why

The frontend is now part of quickstart-vdk.
On changes to the docker image, the helm chart needs to be redeployed.
We need to make sure that the frontend is operational once the helm
chart is deployed.

## What

Trigger deployment of test control service on frontend changes merged
into main
Ping the frontend in the test environment using heartbeat once control
service is deployed
Tag the frontend image as stable if the test is successful

## How has this been tested

[Test
pipeline](https://gitlab.com/vmware-analytics/versatile-data-kit/-/pipelines/894015566)

## What kind of change is this

feature, non-breaking

Signed-off-by: Dilyan Marinov <mdilyan@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants