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

[APM] Add cloud attributes to data telemetry #71008

Merged
merged 2 commits into from
Jul 7, 2020

Conversation

smith
Copy link
Contributor

@smith smith commented Jul 7, 2020

Add cloud provider, region, and availability zone to the data telemetry mappings and queries.

Fixes #70465

Add cloud provider, region, and availability zone to the data telemetry mappings and queries.

Fixes elastic#70465
@smith smith added release_note:skip Skip the PR/issue when compiling release notes v7.9.0 labels Jul 7, 2020
@smith smith requested a review from a team as a code owner July 7, 2020 19:57
@botelastic botelastic bot added the Team:APM All issues that need APM UI Team support label Jul 7, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

}
: {}),
: { ssl: { rejectUnauthorized: false } }),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Needed to add these to run against the observability cluster. Not proud of it but it's for running against a known cluster for testing purposes so not going to worry too much about lax SSL.

Copy link
Member

Choose a reason for hiding this comment

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

IIRC (big if), you can also set NODE_TLS_REJECT_UNAUTHORIZED

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It works. I'll take this part out and add that to the docs.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@smith smith merged commit 2190f2a into elastic:master Jul 7, 2020
smith added a commit to smith/kibana that referenced this pull request Jul 7, 2020
Add cloud provider, region, and availability zone to the data telemetry mappings and queries.

Fixes elastic#70465
@smith smith deleted the nls/actual-cloud-telemetry branch July 8, 2020 03:06
smith added a commit that referenced this pull request Jul 8, 2020
Add cloud provider, region, and availability zone to the data telemetry mappings and queries.

Fixes #70465
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jul 8, 2020
* master: (36 commits)
  fixed api url in example plugin (elastic#70934)
  [data.search.aggs]: Remove remaining client dependencies (elastic#70251)
  [Security Solution][Endpoint] Fix base64 download bug and adopt new user artifact/manifest format (elastic#70998)
  [Security Solution][Exceptions] - Exception Modal Part I (elastic#70639)
  [SIEM][Detection Engine][Lists] Adds additional data types to value based lists
  [SIEM][Detection Engine][Lists] Removes feature flag for lists
  [APM] Show license callout in ML settings (elastic#70959)
  Migrate service settings test to jest (elastic#70992)
  [APM] Add cloud attributes to data telemetry (elastic#71008)
  Fix breadcrumb on panels for visibility / round corners (elastic#71010)
  Improve search typescript (elastic#69333)
  [savedObjects field count] run in baseline job (elastic#70999)
  [Security Solution] [Timeline] Timeline manager tweaks (elastic#69988)
  [Endpoint] Support redirect from Policy Details to Ingest when user initiates Edit Policy from Datasource Edit page (elastic#70874)
  [APM] Add API tests (elastic#70740)
  [Security Solution][Exceptions] - Tie server and client code together (elastic#70918)
  [Audit Logging] Add AuditTrail service (elastic#69278)
  [Usage Collection] Ensure no type duplicates (elastic#70946)
  [Security Solution] [Timeline] Bugfix for timeline row actions disappear sometimes (elastic#70958)
  [CI] Add pipeline task queue framework and merge workers into one (elastic#64011)
  ...
@sorenlouv sorenlouv added the apm:test-plan-7.9.0 Test plan for 7.9 release label Jul 16, 2020
@cauemarcondes cauemarcondes self-assigned this Aug 3, 2020
@cauemarcondes cauemarcondes added the apm:test-plan-done Pull request that was successfully tested during the test plan label Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:test-plan-7.9.0 Test plan for 7.9 release apm:test-plan-done Pull request that was successfully tested during the test plan release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v7.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add APM cloud identification metadata to telemetry
6 participants