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

FullStory: Skip setup for Elasticians #171041

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

afharo
Copy link
Member

@afharo afharo commented Nov 10, 2023

Summary

To save some sessions coming from Elasticians, we want to disable FullStory for any deployments owned by Elastic employees.

cc @shahinakmal @bollinic

Planning to backport this to the previous minor so we have it live asap.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@afharo afharo added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes telemetry Issues related to the addition of telemetry to a feature backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Team:Journey/Onboarding Platform Journey Onboarding team labels Nov 10, 2023
@afharo afharo requested a review from a team as a code owner November 10, 2023 16:58
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@elasticmachine
Copy link
Contributor

Pinging @elastic/platform-onboarding (Team:Journey/Onboarding)

@afharo afharo self-assigned this Nov 10, 2023
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Investigations - Security Solution Cypress Tests #3 / Ransomware Detection Alerts Ransomware in Timelines Renders ransomware entries in timelines table Renders ransomware entries in timelines table

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cloudFullStory 3.5KB 3.7KB +143.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @afharo

@afharo afharo merged commit 19da2d5 into elastic:main Nov 13, 2023
@afharo afharo deleted the fullstory/skip-setup-for-elasticians branch November 13, 2023 14:14
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 13, 2023
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.11

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 13, 2023
# Backport

This will backport the following commits from `main` to `8.11`:
- [FullStory: Skip setup for Elasticians
(#171041)](#171041)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Alejandro Fernández
Haro","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-11-13T14:14:23Z","message":"FullStory:
Skip setup for Elasticians
(#171041)","sha":"19da2d527686b329b53aba0320ff8b1b1c0cd7fe","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","Feature:Telemetry","release_note:skip","telemetry","backport:prev-minor","Team:Journey/Onboarding","v8.12.0"],"number":171041,"url":"https://github.com/elastic/kibana/pull/171041","mergeCommit":{"message":"FullStory:
Skip setup for Elasticians
(#171041)","sha":"19da2d527686b329b53aba0320ff8b1b1c0cd7fe"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/171041","number":171041,"mergeCommit":{"message":"FullStory:
Skip setup for Elasticians
(#171041)","sha":"19da2d527686b329b53aba0320ff8b1b1c0cd7fe"}}]}]
BACKPORT-->

Co-authored-by: Alejandro Fernández Haro <[email protected]>
@mistic
Copy link
Member

mistic commented Nov 14, 2023

This PR haven't made it into the latest BC of 8.11.1. Updating the labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Journey/Onboarding Platform Journey Onboarding team telemetry Issues related to the addition of telemetry to a feature v8.11.2 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants