Skip to content

Don't report supervisor_check_ready unless the FF is enabled #20292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 14, 2024

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.

4 participants