Skip to content

Add hardcoded pause to docker build

Ievgen Chernikov requested to merge update-feature-flag-product-analytics into main

What does this MR do and why?

We're seeing a lot of flakiness in E2E tests where it seems like the build is closed before the GDK has finished updating the flags. So sometimes we will get a situation where the flags haven't been set properly, and the image is set, meaning the PA tests fail.

This MR adds a default sleep to get around this limitation, although we should probably look into why this is happening in the first place when we have engineering time (I've tried, my local docker has broken in so many ways 😅).

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

You can follow the E2E testing docs to run them locally, although there have been issues lately with getting E2E tests to run within Mac environments.

Alternatively, the pipelines should pass.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Robert Hunt

Merge request reports

Loading