[Fleet]: tsdb settings are not greyed out for system-metrics-cpu when the button is not editable by user.

[amolnater-qasource]

Kibana version: 8.7 BC4 kibana cloud environment

Host OS and Browser version: All, All

System Integration version: v1.20.9

Build details:

VERSION: 8.7 BC4
BUILD: 60949
COMMIT: de22cd9361a0dbf429f9648d3c7b7c45aa862e90

Preconditions:

1. 8.7 BC4 kibana cloud environment should be available.
2. Setup should be done as available under Added Time-series indexing (TSDB) to Integrations Experimental Indexing settings #144974

Steps to reproduce:

1. Navigate to Fleet and add System v1.20.9 to the agent policy.
2. Observe under System-Cpu-metrics, tsdb settings are already enabled and user is unable to make any changes.
3. Observe tsdb settings are not greyed out.

Screen Recording:

ec2-44-197-249-206.compute-1.amazonaws.com.-.Remote.Desktop.Connection.2023-03-07.11-52-05.mp4

Expected Result:
tsdb settings should be greyed out for system-metrics-cpu when the button is not editable by user.

[elasticmachine]

Pinging @elastic/fleet (Team:Fleet)

[amolnater-qasource]

@manishgupta-qasource Please review.

[manishgupta-qasource]

Secondary review for this ticket is Done

[juliaElastic]

I think it is expected that TSDB can be toggled off by the users with this change: #149591

[amolnater-qasource]

Hi @juliaElastic

Thank you for looking into this and sharing the related PR.

We are successfully enable and disable tsdb for System v1.20.5.

However, we are unable to disable the same for v1.20.9 System integration. That's why we have logged this issue specifically for this version.

Could you please confirm if it is expected?

Thanks!

[kpollich]

Hi @amolnater-qasource - this is expected in the newer version of system as it enables TSDB on the package side as of elastic/integrations#5160. e.g. the package is enabling TSDB for certain metrics, so the toggle will be "enabled" in the UI but not changeable by the user.

Interestingly, I do not see an entry for v1.20.9 in the integration's changelog, so this may have been a prerelease version: https://github.com/elastic/integrations/blob/main/packages/system/changelog.yml

It's clear that there is some effort to move system metrics over to TSDB as in elastic/integrations#5160, but I actually don't see v1.20.9 tracked in git anywhere.

https://epr.elastic.co/package/system/1.20.9/ is also a 404. Something we should investigate further. But to answer your original question this looks like expected behavior when a package defines index_mode: time_series for a given data stream.

[amolnater-qasource]

Hi @kpollich

Thank you for the updates.

the package is enabling TSDB for certain metrics, so the toggle will be "enabled" in the UI but not changeable by the user.

According to this we expected the button to be greyed out so that the non-editable fields will be easily differentiated.

Further, the System integration version system-1.20.9 is loaded on setting the registry-package available under section: Testing scenario of enabling TSDB in package spec by default at #144974 (comment)

Registery:
registry-packages.zip

Please let us know if anything else is required from our end.
Thanks!

[kpollich]

@amolnater-qasource - Thanks understood! Seems like the EuiSwitch state may be incorrectly set here. Should be checked=false and disabled=true. Will toss this on our bug board 👍