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

Fr 162 perf monitor core device health in topology discovery view #1543

Conversation

MarcoMruz
Copy link
Contributor

@MarcoMruz MarcoMruz commented May 13, 2024

Q A
Fixed Issue? FR-162
Tests Added + Pass? Yes
Any Dependency Changes?
  • added new subscription into topology discovery to show live data about device performance and resources usage
  • data are shown in device info panel - currently only available for LLDP topology

use FRINXio/frinx-inventory-server#432 for testing

add WS_INVENTORY_URL to graphql proxy env file

soson and others added 10 commits April 30, 2024 09:34
* created kafka healthcheck toolbar

* add useffect to close automatically when kafka is healthy and restyling of toolbar
* changed base to 6.x-stable

* fixed ports for other topology layers
* stream list added (inventory client)

* run pr check on 6.X-stable
* add additional properties into workflow import and refactor workflow input from grid into vstack

* remove bad URLs used when redirecting

* create new function in shared to omit deeply nested objects

* add info comment about test framework limitations

* handle sent json in inputParams

* support multiple properties to be omitted in omitDeep function

* add possibility to export workflow definition

* generate graphql types
* Fix env inventory ws url (#1520)

* inventory ws schema fix in env

* fixed ivnentory ws path env fix

* removed console logs

* env fix

* disable nx cloud (#1526)

* fixed search button (#1523)

Co-authored-by: Martin Sottnik <[email protected]>

* fix redirecting url (#1521)

Co-authored-by: Martin Sottnik <[email protected]>

* use execute workflow by name instead of rerun (#1522)

Co-authored-by: Martin Sottnik <[email protected]>

* fixed disabled edit button redirecting when it should not (when installed) (#1527)

* fixed disabled link redirecting when it should not

* refactoring

* Wm fixing (#1528)

* Docker publish github action compatible with new release process (#1532)

Co-authored-by: Jozef Volak <[email protected]>

* bump version to 6.0.1 (#1534)

* Add topology net diff (#1535)

* initial setup for bgpls-diff

* displaying device diff color

* setup for version diff in BGP-LS

* reverted to previous commit

* merging main into branch

* UI fix

* removed console.logs

* sending same query twice fix (maybe?)

* fixed lint issue

* lint adjustments

* refactored helper function

---------

Co-authored-by: Martin Sottnik <[email protected]>

* upgrade gamma to 6.0.1

---------

Co-authored-by: plehocky <[email protected]>
Co-authored-by: Marco <[email protected]>
Co-authored-by: Jozef Volák <[email protected]>
Co-authored-by: Jozef Volak <[email protected]>
@MarcoMruz MarcoMruz requested review from soson and plehocky May 13, 2024 07:40
@MarcoMruz MarcoMruz marked this pull request as draft May 13, 2024 07:40
@MarcoMruz MarcoMruz changed the base branch from 6.X-stable to main May 13, 2024 09:03
@MarcoMruz MarcoMruz marked this pull request as ready for review May 17, 2024 09:33
@MarcoMruz MarcoMruz changed the base branch from main to story-performance-monitoring May 21, 2024 10:03
…ore-Device-health-in-Topology-Discovery-View
@soson soson merged commit f73bd61 into story-performance-monitoring May 22, 2024
@soson soson deleted the FR-162-PerfMonitor-core-Device-health-in-Topology-Discovery-View branch May 22, 2024 12:14
soson added a commit that referenced this pull request Jun 21, 2024
* Fr 245 perf monitor tier 1 add software column to device inventory view (#1551)

* add new model/version column to devices table

* grayish unknown device names

* Fr 162 perf monitor core device health in topology discovery view (#1543)

* ptp interface status added (#1533)

* Add kafka notification (#1537)

* created kafka healthcheck toolbar

* add useffect to close automatically when kafka is healthy and restyling of toolbar

* add graphql ws support and add device health subscription into LLDP topology

* remove logs

* pause subscription when no node is selected

* remove refresh every second

* fix-topology displaying redundant ports (#1538)

* changed base to 6.x-stable

* fixed ports for other topology layers

* FR-231 stream list (#1539)

* stream list added (inventory client)

* run pr check on 6.X-stable

* Fd 674 wm (#1540)

* add additional properties into workflow import and refactor workflow input from grid into vstack

* remove bad URLs used when redirecting

* create new function in shared to omit deeply nested objects

* add info comment about test framework limitations

* handle sent json in inputParams

* support multiple properties to be omitted in omitDeep function

* add possibility to export workflow definition

* generate graphql types

* Release 6.0.2 (#1542)

* Fix env inventory ws url (#1520)

* inventory ws schema fix in env

* fixed ivnentory ws path env fix

* removed console logs

* env fix

* disable nx cloud (#1526)

* fixed search button (#1523)

Co-authored-by: Martin Sottnik <[email protected]>

* fix redirecting url (#1521)

Co-authored-by: Martin Sottnik <[email protected]>

* use execute workflow by name instead of rerun (#1522)

Co-authored-by: Martin Sottnik <[email protected]>

* fixed disabled edit button redirecting when it should not (when installed) (#1527)

* fixed disabled link redirecting when it should not

* refactoring

* Wm fixing (#1528)

* Docker publish github action compatible with new release process (#1532)

Co-authored-by: Jozef Volak <[email protected]>

* bump version to 6.0.1 (#1534)

* Add topology net diff (#1535)

* initial setup for bgpls-diff

* displaying device diff color

* setup for version diff in BGP-LS

* reverted to previous commit

* merging main into branch

* UI fix

* removed console.logs

* sending same query twice fix (maybe?)

* fixed lint issue

* lint adjustments

* refactored helper function

---------

Co-authored-by: Martin Sottnik <[email protected]>

* upgrade gamma to 6.0.1

---------

Co-authored-by: plehocky <[email protected]>
Co-authored-by: Marco <[email protected]>
Co-authored-by: Jozef Volák <[email protected]>
Co-authored-by: Jozef Volak <[email protected]>

* change graphql-ws version to make it same as in the rest of the app

* remove ^ from graphql-ws version

* change graphql ws version inside device topology

---------

Co-authored-by: Martin Sottnik <[email protected]>
Co-authored-by: plehocky <[email protected]>
Co-authored-by: Jozef Volák <[email protected]>
Co-authored-by: Jozef Volak <[email protected]>

* Fr 163 perf monitor core device health in device inventory view resource usage (#1544)

* ptp interface status added (#1533)

* Add kafka notification (#1537)

* created kafka healthcheck toolbar

* add useffect to close automatically when kafka is healthy and restyling of toolbar

* add device load status into device inventory table

* remove service state column from device list

* show UNKNOWN label when value is not provided

* check story branches

---------

Co-authored-by: Martin Sottnik <[email protected]>

* Make perf optional (#1559)

* implement new context provider to make perf monit optional

* add perf monit provider to topology discovery to make it optional in device info panel

* Delete packages/frinx-inventory-client/src/pages/device-list/device-status-column.tsx

---------

Co-authored-by: plehocky <[email protected]>

* Add device health status perf monitor (#1568)

* added devicesConnection subscription and implemented into device status(health)

* device status is now visible even if perf monitor is turned off

* formatting

---------

Co-authored-by: Marco <[email protected]>
Co-authored-by: plehocky <[email protected]>
Co-authored-by: Jozef Volák <[email protected]>
Co-authored-by: Jozef Volak <[email protected]>
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