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
OCPBUGS-32791,CONSOLE-4014: Consume telemetry CLUSTER_ID and use it together with orgId as segment identifier #13786
OCPBUGS-32791,CONSOLE-4014: Consume telemetry CLUSTER_ID and use it together with orgId as segment identifier #13786
Conversation
44eed0a
to
7ef516f
Compare
/retest |
1 similar comment
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jerolimov 👍
Adding couple of comments
frontend/packages/console-telemetry-plugin/src/listeners/segment.ts
Outdated
Show resolved
Hide resolved
@jerolimov: This pull request references CONSOLE-4014 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
7ef516f
to
bf92982
Compare
@jerolimov: This pull request references CONSOLE-4014 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
bf92982
to
cec1f61
Compare
cec1f61
to
843c3c9
Compare
@jerolimov: This pull request references Jira Issue OCPBUGS-32791, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. This pull request references CONSOLE-4014 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jerolimov 👍
/lgtm
/retest |
1 similar comment
/retest |
/hold Revision 843c3c9 was retested 3 times: holding |
/unhold |
/label qe-approved |
@jerolimov: This pull request references Jira Issue OCPBUGS-32791, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: This pull request references CONSOLE-4014 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
// With 4.15+ we can use the user object directly, but on older releases (<4.15) | ||
// we need to extract the user object from the metadata. | ||
// All properties are defined in the UserInfo interface and marked as optional. | ||
const uid = user?.uid || user?.metadata?.uid; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
down the road we should be having helpers for getting these so we dont need to care about the path
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jerolimov, jhadvig 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 |
Docs Approver: |
/label docs-approved |
Thank you for the nice video, it adds a lot of context for reviewers. /label px-approved |
/retest |
@jerolimov: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
[ART PR BUILD NOTIFIER] This PR has been included in build openshift-enterprise-console-container-v4.16.0-202404261114.p0.gdfe987f.assembly.stream.el9 for distgit openshift-enterprise-console. |
Fix included in accepted release 4.16.0-0.nightly-2024-04-29-154406 |
This is the frontend part related to openshift/console-operator#889
SERVER_FLAGS.telemetry.CLUSTER_ID
and pass it together with otherclusterProperties
to segment.clusterId
andorganizationId
to calculate the anonymized user identifier for segment.I've tested this together with 889 on a cluster bot:
CONSOLE-4014.mp4
/cc @jhadvig @TheRealJon