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

[Docs][Maps] Update the asset tracking tutorial #173999

Merged
merged 5 commits into from Jan 3, 2024

Conversation

jsanz
Copy link
Member

@jsanz jsanz commented Dec 27, 2023

Fixes #167484

Updates the asset tracking tutorial with the following improvements:

  • From [Maps][Docs] Improve the asset tracking tutorial to work with Elastic Agent #167484, @Danouchka contributed improvements to the way the Elasticsearch assets are defined to work with Elastic Agent
  • Moved some content to use the Kibana Console to generate assets, instead of going through all the UI elements
  • Updated references to the data that tracks not only Portland buses but also trains and light trains
  • Replaces the alert to use the Index connector
  • Adds a new section on visualizing the alerts
  • Images updated to Kibana 8.11 UI

Copy link

Documentation preview:

@jsanz jsanz added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting docs Feature:Maps labels Dec 27, 2023
@jsanz jsanz marked this pull request as ready for review December 28, 2023 10:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@jsanz
Copy link
Member Author

jsanz commented Dec 28, 2023

Hi @nickpeihl @Danouchka I volunteered you to review this update to the Asset Tracking tutorial 😃 No rush about this at all!

The preview is available at

https://kibana_173999.docs-preview.app.elstc.co/guide/en/kibana/master/asset-tracking-tutorial.html

@Danouchka
Copy link

Hello I have followed the tutorial on a brand new 8.11.3 in Elastic Cloud and I get this in the Discover (despite the indice is growing and the agent is healthy)
Capture d’écran 2024-01-02 à 16 24 27

@Danouchka
Copy link

Seems there's an issue with the ingest pipeline as the remove processor is not executed but the issue might be somewhere else

Copy link
Member

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm! Thanks for updating the docs!

Added a comment where calling kbn APIs from DevTools in development returns an error. But it seems to work in production.

[source,yaml]
[source,js]
----------------------------------
POST kbn:/api/fleet/agent_policies?sys_monitoring=true
Copy link
Member

Choose a reason for hiding this comment

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

Testing this on main returns an error. But it looks like this only happens in development environments right now. Tested a similar API on a 8.12 pre-production instance which works fine. Here is an existing issue about this.

I think we can keep this as it is as long as production environments are not affected.

{
  "statusCode": 400,
  "error": "Bad Request",
  "message": "Please specify a version via elastic-api-version header. Available versions: [2023-10-31]"
}

@jsanz
Copy link
Member Author

jsanz commented Jan 3, 2024

Hello I have followed the tutorial on a brand new 8.11.3 in Elastic Cloud and I get this in the Discover (despite the indice is growing and the agent is healthy)

Hi, the issue I see is that your index is not correctly defined, leaving a mapping of the trimet.time field as long, and that clashes with the Data View definition that expects that file to be a date (as far as I can see).

The root cause may be in the ingest process, where the index is not getting the correct template and ingest pipeline.

image

@Danouchka
Copy link

OK I see the issue, I missed the component template definition and the index template definition while going through the tutorial

@Danouchka
Copy link

Thanks @jsanz ! Now it's ok , I have corrected

@jsanz
Copy link
Member Author

jsanz commented Jan 3, 2024

Thanks @jsanz ! Now it's ok , I have corrected

No worries, in d274496 I've set up the snippets as opened so they are easier to follow. It was my fault to put them collapsed making them harder to spot.

@jsanz jsanz merged commit f91df9c into elastic:main Jan 3, 2024
7 checks passed
@jsanz jsanz deleted the docs/maps/asset-tutorial-update branch January 3, 2024 13:29
amyjtechwriter added a commit that referenced this pull request Jan 31, 2024
…175973)

Adding the updates made to the asset tracking tutorial to 8.12. 

Closes: #175522. <-- User feedback that steps might have been missing.
Myself and the user were looking at the previous version of the
tutorial.

Copy of: #173999 <-- Jorge's updates to the tutorial.
jsanz added a commit that referenced this pull request Feb 2, 2024
…6061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of #173999 at
#175973.
WafaaNasr pushed a commit to WafaaNasr/kibana that referenced this pull request Feb 6, 2024
…stic#176061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of elastic#173999 at
elastic#175973.
fkanout pushed a commit to fkanout/kibana that referenced this pull request Feb 7, 2024
…stic#176061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of elastic#173999 at
elastic#175973.
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
…stic#176061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of elastic#173999 at
elastic#175973.
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
…stic#176061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of elastic#173999 at
elastic#175973.
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
…stic#176061)

This PR incorporates some linting fixes to the asset tracking tutorial
that @amyjtechwriter added to the backport done to 8.12 of elastic#173999 at
elastic#175973.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting docs Feature:Maps release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Maps][Docs] Improve the asset tracking tutorial to work with Elastic Agent
5 participants