-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Telemetry] Fix cloud tests (NP version of #59040) #61271
Conversation
Pinging @elastic/pulse (Team:Pulse) |
Q: Should we still aim this PR to the label |
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
💚 Build SucceededTo update your PR or re-run it, just comment with: |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Apparently the backported build never finished so no email about the build done was sent and I forgot to proactively check. Updated the backport branch and rebuilding... |
Pinging @elastic/kibana-core (Team:Core) |
Summary
Fix
allowChangingOptInStatus
savedObject attribute not being able to get updated. Fixes test case for opt in api failing on cloud withallowChangingOptInStatus: false
config set.Adds test cases for opt in route with respect to
allowChangingOptInStatus
flag.Closes #51522
This is basically a reimplementation of #59040 for the NP. I thought it was easier than merging
master
into that one.Checklist
Delete any items that are not applicable to this PR.
For maintainers