Skip to content
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

Properly declare analytics variable #9047

Merged
merged 1 commit into from
Apr 1, 2022
Merged

Properly declare analytics variable #9047

merged 1 commit into from
Apr 1, 2022

Conversation

ArthurSens
Copy link
Contributor

@ArthurSens ArthurSens commented Mar 31, 2022

Description

A small fix for a bug introduced in #8884 and identified at #9044

Release Notes

NONE

@ArthurSens
Copy link
Contributor Author

/werft run

@ArthurSens
Copy link
Contributor Author

ArthurSens commented Mar 31, 2022

/werft run

👍 started the job as gitpod-build-as-fix-analytics.5

@ArthurSens
Copy link
Contributor Author

Tagging Huiwen and Filip just for awareness :)

@ArthurSens
Copy link
Contributor Author

ArthurSens commented Mar 31, 2022

/hold

werft seems to be having problems connection to the pods? 🤔

Signed-off-by: ArthurSens <arthursens2005@gmail.com>
@ArthurSens
Copy link
Contributor Author

ArthurSens commented Apr 1, 2022

/werft run analytics=segment|TEZnsG4QbLSxLfHfNieLYGF4cDwyFWoe

👍 started the job as gitpod-build-as-fix-analytics.9

@ArthurSens ArthurSens requested a review from a team April 1, 2022 10:07
@ArthurSens
Copy link
Contributor Author

Jobs with and without analytics passed

Copy link
Member

@meysholdt meysholdt left a comment

Choose a reason for hiding this comment

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

Code changes LGTM

@ArthurSens
Copy link
Contributor Author

ArthurSens commented Apr 1, 2022

Not sure why this PR is not in tide's queue 🤔

@filiptronicek
Copy link
Member

filiptronicek commented Apr 1, 2022

@ArthurSens there's still the do-not-merge/hold label 👀

@ArthurSens
Copy link
Contributor Author

ArthurSens commented Apr 1, 2022

@ArthurSens there's still the do-not-merge/hold label 👀

🤦‍♂️

@roboquat roboquat merged commit c7194c9 into main Apr 1, 2022
@roboquat roboquat deleted the as/fix-analytics branch April 1, 2022 17:24
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