IIS_2617: Migrating tile_map to map in dashboards

[ishleenk17]

• Enhancement

What does this PR do?

This PR replaces the tile_map object with map in IIS Logs dashboard

Checklist

• I have reviewed tips for building integrations and this pull request is aligned with them.
• I have verified that all data streams collect metrics or logs.
• I have added an entry to my package's changelog.yml file.
• I have verified that Kibana version constraints are current according to guidelines.

How to test this PR locally

Load the newly created dashboard and we should see map as the object

• Closes Migrate tile_map objects to map objects #2617

Screenshots

Below screenshot shows map object updated in IIS

[elasticmachine]

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

• Start Time: 2022-06-01T19:01:26.803+0000

• Duration: 12 min 11 sec

Test stats 🧪

Test	Results
Failed	0
Passed	83
Skipped	0
Total	83

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

• /test : Re-trigger the build.

[elasticmachine]

🌐 Coverage report

Name	Metrics % (covered/total)	Diff
Packages	100.0% (2/2)	💚
Files	100.0% (2/2)	💚 3.538
Classes	100.0% (2/2)	💚 3.538
Methods	85.714% (30/35)	👎 -2.996
Lines	91.398% (255/279)	👍 1.677
Conditionals	100.0% (0/0)	💚

[ishleenk17]

/test

[ishleenk17]

/test

[ishleenk17]

/test

[andrewkroh]

The conversion changes LGTM.

I think this would be a good opportunity to "unlink" the visualizations such that they are embedded into the dashboard.

[ishleenk17]

The conversion changes LGTM.

I think this would be a good opportunity to "unlink" the visualisations such that they are embedded into the dashboard.

Andrew, since this issue involves multiple integrations. Would it be better if we track the unlinking the visualisations as part of separate issue for all of these integrations ?

[ishleenk17]

@andrewkroh @jsoriano @flash1293 : I am doing this migration using the latest stack 8.2.
Hence we see the visualisations/dashboards/maps in 8.2. As a result of this we have to drop all the previous kibana versions supported (7.14 an 8.0).

As per the issue the editor for tile map is removed in 8.0 and map was added.
Shall we do the change with older 8.0 ELK stack ? In that case we would have to drop support for only 7.14.
Please share your inputs here ?

cc: @lalit-satapathy

[jsoriano]

@ishleenk17 yes, it would be nice to build the dashboards with the oldest supported version if possible, so compatibility with more versions is maintained.

Maybe this can be done with 7.x, tile_maps were removed in 8.0, but I think that the map objects were already available in previous versions. So maybe you can build them with 7.17 if you want to continue maintaining the same package for both majors.

[andrewkroh]

I am doing this migration using the latest stack 8.2.
Hence we see the visualisations/dashboards/maps in 8.2. As a result of this we have to drop all the previous kibana versions supported (7.14 an 8.0).

I agree with @jsoriano that it would be best to use a 7.17 so that we can maintain compatibility for users still on 7.x, but AFAIK the "View in Maps" button that converts the tile_map for you only exists in 8.0+. You can check if it exists in 7.17, and if not then it will need to drop the 7.x compatibility or find a different way to do the conversion.

[andrewkroh]

Assuming the "View in Maps" conversion action only exists in 8.x then I am good with these changes.

[ishleenk17]

I am doing this migration using the latest stack 8.2.
Hence we see the visualisations/dashboards/maps in 8.2. As a result of this we have to drop all the previous kibana versions supported (7.14 an 8.0).

I agree with @jsoriano that it would be best to use a 7.17 so that we can maintain compatibility for users still on 7.x, but AFAIK the "View in Maps" button that converts the tile_map for you only exists in 8.0+. You can check if it exists in 7.17, and if not then it will need to drop the 7.x compatibility or find a different way to do the conversion.

Yes, just checked. The view maps works for map objects from 8.0 stack. So, I am using that stack for the dashboards.
This will result in dropping support for 7.14, but still supporting ^8.0.0

[muthu-mps]

LGTM!