Skip to content

Update scripts to used datastream.dataset for dashboards#216

Closed
ruflin wants to merge 2 commits intoelastic:masterfrom
ruflin:adjust-datastream-fields
Closed

Update scripts to used datastream.dataset for dashboards#216
ruflin wants to merge 2 commits intoelastic:masterfrom
ruflin:adjust-datastream-fields

Conversation

@ruflin
Copy link
Copy Markdown
Contributor

@ruflin ruflin commented Aug 4, 2020

The agent will ship datastream.* fields. Because of this, the import scripts can be adjusted to only use these fields.

This PR doese not rename any Golang variables. This should be done in a follow up and in sync with potential changes to the registry.

The agent will ship datastream.* fields. Because of this, the import scripts can be adjusted to only use these fields.

This PR doese not rename any Golang variables. This should be done in a follow up and in sync with potential changes to the registry.
@ruflin ruflin requested a review from mtojek August 4, 2020 08:34
@ruflin ruflin self-assigned this Aug 4, 2020
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Aug 4, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #216 updated]

  • Start Time: 2020-08-04T09:09:17.683+0000

  • Duration: 3 min 42 sec

Copy link
Copy Markdown
Contributor

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

LGTM. I believe you could combine this one together with #213

@ruflin
Copy link
Copy Markdown
Contributor Author

ruflin commented Aug 4, 2020

Closing in favor of #213

@ruflin ruflin closed this Aug 4, 2020
@ruflin ruflin deleted the adjust-datastream-fields branch August 4, 2020 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants