Skip to content

[7.17] [APM] Disable fields when frozen property is set to true (#126497)#128797

Merged
cauemarcondes merged 4 commits intoelastic:7.17from
cauemarcondes:backport/7.17/pr-126497
Mar 30, 2022
Merged

[7.17] [APM] Disable fields when frozen property is set to true (#126497)#128797
cauemarcondes merged 4 commits intoelastic:7.17from
cauemarcondes:backport/7.17/pr-126497

Conversation

@cauemarcondes
Copy link
Copy Markdown
Contributor

@cauemarcondes cauemarcondes commented Mar 29, 2022

Backport

This will backport the following commits from main to 7.17:

Questions ?

Please refer to the Backport tool documentation

@cauemarcondes
[APM] Disable fields when frozen property is set to true (elastic#126497
32132c7 
)

* using frozen property to disable fields

* removing isCloud since frozen must be enough

(cherry picked from commit b1f9201)

# Conflicts:
#	x-pack/plugins/apm/public/components/fleet_integration/apm_policy_form/index.tsx
#	x-pack/plugins/apm/public/components/fleet_integration/apm_policy_form/settings_definition/agent_authorization_settings.test.ts
@cauemarcondes cauemarcondes added the backport This PR is a backport of another PR label Mar 29, 2022
@cauemarcondes cauemarcondes enabled auto-merge (squash) March 29, 2022 18:04
@cauemarcondes cauemarcondes disabled auto-merge March 29, 2022 18:05
@cauemarcondes cauemarcondes requested a review from ogupte March 30, 2022 18:38
ogupte
ogupte approved these changes Mar 30, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@ogupte ogupte left a comment

Choose a reason for hiding this comment

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

LGTM

@cauemarcondes cauemarcondes enabled auto-merge (squash) March 30, 2022 18:55
@cauemarcondes cauemarcondes merged commit aeba85a into elastic:7.17 Mar 30, 2022
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Mar 30, 2022

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 2.7MB 2.7MB -323.0B
Unknown metric groups

References to deprecated APIs

id before after diff
canvas 67 63 -4
cases 151 145 -6
dashboard 231 230 -1
data 465 457 -8
dataEnhanced 49 43 -6
dataVisualizer 242 232 -10
discover 1810 1500 -310
fleet 39 35 -4
indexPatternEditor 85 75 -10
indexPatternFieldEditor 156 145 -11
indexPatternManagement 498 434 -64
infra 266 162 -104
inputControlVis 325 295 -30
lens 353 277 -76
management 2 1 -1
maps 1667 1072 -595
ml 80 79 -1
monitoring 53 41 -12
security 71 63 -8
securitySolution 991 880 -111
stackAlerts 183 157 -26
visDefaultEditor 213 163 -50
visTypeTimeseries 337 322 -15
visTypeVega 35 34 -1
visualize 89 67 -22
total -1486

History

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

@cauemarcondes cauemarcondes deleted the backport/7.17/pr-126497 branch March 30, 2022 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ogupte ogupte ogupte approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backport This PR is a backport of another PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cauemarcondes @kibana-ci @ogupte