Skip to content

Commit

Permalink
[Infra UI] Fix process charts after refactoring (#168159)
Browse files Browse the repository at this point in the history
Closes #168156 
## Summary

This PR fixes process charts error: 

![image](https://github.com/elastic/kibana/assets/14139027/4d907c77-07a3-40da-96fd-2340bce2efc8)

I assume that after the merge of the refactoring PR, those files were
recreated by another PR or most likely when `main` was merged to [the
refactoring PR](#166965) with them
as resolved conflicts. Deleting the files again fixed the issue.

## Testing
I found the issue on serverless but it probably can be reproduced on
on-prem as well
On Serverless: 
- Start a local es instance: `yarn es serverless --kill --clean
--license trial --ssl`
- Enable infra in the `serverless.oblt.dev.yml` file:
   - `xpack.infra.enabled: true`
- Start a local kibana instance: `yarn serverless-oblt --ssl` and see if
the side nav contains the Infrastructure item
- Navigate to https://0.0.0.0:5601/ftw/app/hosts and open Asset details
flyout or page
- Click on the process tab and expand one of the processes
- The charts (CPU and Memory) should be visible
  • Loading branch information
jennypavlova authored Oct 6, 2023
1 parent baff0eb commit 000b70c
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 139 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,7 @@ import { parseSearchString } from './parse_search_string';
import { ProcessesTable } from './processes_table';
import { STATE_NAMES } from './states';
import { SummaryTable } from './summary_table';
import {
SortBy,
useProcessList,
ProcessListContextProvider,
} from '../../../../pages/metrics/inventory_view/hooks/use_process_list';
import { SortBy, useProcessList, ProcessListContextProvider } from '../../hooks/use_process_list';
import { getFieldByType } from '../../../../../common/inventory_models';
import { useAssetDetailsRenderPropsContext } from '../../hooks/use_asset_details_render_props';
import { useDateRangeProviderContext } from '../../hooks/use_date_range';
Expand Down

This file was deleted.

This file was deleted.

0 comments on commit 000b70c

Please sign in to comment.