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

[analytics] group rendered events by session #5220

Merged
merged 1 commit into from
Aug 17, 2021
Merged

Conversation

akosyakov
Copy link
Member

@akosyakov akosyakov commented Aug 16, 2021

  • /werft analytics=segment|TEZnsG4QbLSxLfHfNieLYGF4cDwyFWoe

More analytics to track down #5220

We need to group events by window session id, workspace and instance ids are not helpful because there can be multiple windows for the same instance and reloading the page bring a lot of confusion to data.

@roboquat roboquat requested review from csweichel and geropl August 16, 2021 12:45
@akosyakov akosyakov requested review from jakobhero and removed request for csweichel and geropl August 16, 2021 12:45
@akosyakov akosyakov force-pushed the ak/rendered_by_session branch from 5806e81 to 968dde3 Compare August 16, 2021 12:55
@akosyakov akosyakov requested a review from geropl August 16, 2021 13:06
@gitpod-io gitpod-io deleted a comment from jakobhero Aug 17, 2021
@akosyakov
Copy link
Member Author

akosyakov commented Aug 17, 2021

/werft run

👍 started the job as gitpod-build-ak-rendered-by-session.5

@jakobhero
Copy link
Contributor

jakobhero commented Aug 17, 2021

/werft run

👍 started the job as gitpod-build-ak-rendered-by-session.6

@roboquat
Copy link
Contributor

LGTM label has been added.

Git tree hash: 566e5489aaaa1ad8e494e12e1806fe81a73b0e90

@akosyakov
Copy link
Member Author

/assign @csweichel

@akosyakov
Copy link
Member Author

/approve no-issue

1 similar comment
@JanKoehnlein
Copy link
Contributor

/approve no-issue

@roboquat
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akosyakov, jakobhero, JanKoehnlein

Associated issue requirement bypassed by: akosyakov, JanKoehnlein

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@roboquat roboquat merged commit e0f5b36 into main Aug 17, 2021
@roboquat roboquat deleted the ak/rendered_by_session branch August 17, 2021 12:44
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