Skip to content

Conversation

@filiptronicek
Copy link
Member

@filiptronicek filiptronicek commented Oct 11, 2024

Description

When supervisor_check_ready_retry is disabled, don't report any supervisor_check_ready events.

Related Issue(s)

Fixes https://gitpod.slack.com/archives/C071G5TTS49/p1728468748822559

How to test

You can try opening a workspace on https://ft-limit-sb2f59260e6.preview.gitpod-dev.com/workspaces and inspecting the network tab for outgoing trackEvent requests.

/hold

@kylos101
Copy link
Contributor

Hey @filiptronicek , I observed trackEvent via a workwspace in this preview (with the flag on and off), but didn't find a trackEvent where the event was supervisor_check_ready. Maybe I did something wrong?

image

@filiptronicek
Copy link
Member Author

@kylos101 I found it useful to filter my network events for trackEvent. Seems to work on my end:
image

@kylos101
Copy link
Contributor

@kylos101 I found it useful to filter my network events for trackEvent. Seems to work on my end: image

Awesome, thank you for sharing test results.

I filtered by trackEvent, and saw other events, but not this one. Does it take a bit for the feature flag value to propagate to the preview?

@filiptronicek
Copy link
Member Author

@kylos101 usually takes about 3-10 minutes at most

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM!

@roboquat roboquat merged commit 19d8a73 into main Oct 14, 2024
58 of 59 checks passed
@roboquat roboquat deleted the ft/limit-supervisor-checkready-events branch October 14, 2024 16:11
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