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

feat(webui): improve performance of resource status update in webui #66

Merged
merged 9 commits into from
Dec 11, 2024

Conversation

Meng-20
Copy link
Collaborator

@Meng-20 Meng-20 commented Dec 11, 2024

Implement the API BladesGetResourceStatus in webui to improve the performance of ComposeMemory, FreeMemory, Assign/UnassignMemory.

scott-howe-1 and others added 9 commits December 4, 2024 09:10
…nd update (#57)

* refactor(cli): command usage and example string generation refactor and update

* doc(cli): add docker-specific note about service-related cli options
* feat: set the nodes width based on the device id dimensions

* feat: add ipaddress to the existed devices in dashboard

* feat: add new button to discover new devices and the related interfaces

* fix: fix the bug in addDiscoveredHosts interface

* fix: fix bug in addDiscoveredBlades interface

* feat: add function addDiscoveredDevices with output popup and waiting popup

* style:  separate lines for discovered blades and hosts

* feat: add waiting progress for device discovery

* feat: update the content after adding new discovered devices

* feat: distinguish node status by border color

* feat: prevent the user from manually adding blades to the CMA_Discovered_Blades appliance

* style: remove unnecessary stype setup
…e using privileged mode (#60)

* feat: update the docker running command to make D-bus available ay priviledged mode

* fix: update the docker running command to make D-bus available using privileged mode
Found a couple bugs in appliance.AddBlade() and manager.AddHost().
(1) corner case where an empty bladeId is being allowed, when it shouldn't
(2) Not enough information is being saved to new blade objects during error handling.  This was causing a panic during the delete because the backendOps variable was a nil point being used to call a backend function.
@Meng-20 Meng-20 requested a review from scott-howe-1 December 11, 2024 00:13
@Meng-20 Meng-20 self-assigned this Dec 11, 2024
@scott-howe-1 scott-howe-1 changed the title feat(webui): update resource status in webui feat(webui): improve performance of resource status update in webui Dec 11, 2024
@Meng-20 Meng-20 merged commit a0c0a1a into main Dec 11, 2024
5 checks passed
@Meng-20 Meng-20 deleted the feat/update_resource_status_in_webui branch December 11, 2024 00:26
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.

2 participants