+
Skip to content

Conversation

SammyOina
Copy link
Contributor

@SammyOina SammyOina commented Jan 14, 2025

What type of PR is this?

What does this do?

  • New Features

    • Added support for 9P networking protocol and filesystem
    • Enhanced VM creation with certificate and environment configuration
    • Introduced more flexible environment variable management
  • Configuration Changes

    • Updated systemd service to use external environment file
    • Modified Linux kernel configuration to support new networking features
  • Bug Fixes

    • Improved error handling in VM creation process
    • Enhanced cleanup procedures for virtual machine resources
  • Refactor

    • Updated gRPC service methods to accept more detailed request parameters
    • Restructured configuration management for agent and VM creation

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

Did you document any new/modified feature?

Notes

manager creates vms and is a grpc server. CLI can be used as a client for manager to create and remove vms
when a vm is launched, agent will start inside the vm, agent constains the cvms grpc client which connects to the cloud cvms service/ test/cvms. once cvms client on agent connects to cloud, it is ready to receive run requests and start agent sever. once agent server starts algo and data sets can be sent and results can then be received.

Copy link

codecov bot commented Jan 14, 2025

Codecov Report

Attention: Patch coverage is 16.10169% with 99 lines in your changes missing coverage. Please review.

Project coverage is 61.13%. Comparing base (1f32f51) to head (b6b8f0b).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
cli/manager.go 0.00% 42 Missing ⚠️
manager/service.go 32.75% 26 Missing and 13 partials ⚠️
manager/qemu/config.go 0.00% 6 Missing and 2 partials ⚠️
manager/qemu/vm.go 0.00% 6 Missing and 2 partials ⚠️
manager/api/grpc/server.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #355      +/-   ##
==========================================
- Coverage   62.28%   61.13%   -1.16%     
==========================================
  Files          54       54              
  Lines        4261     4371     +110     
==========================================
+ Hits         2654     2672      +18     
- Misses       1344     1419      +75     
- Partials      263      280      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SammyOina SammyOina marked this pull request as ready for review January 14, 2025 14:55
Copy link
Contributor

@WashingtonKK WashingtonKK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add instructions for running with this architecture.

@WashingtonKK
Copy link
Contributor

WashingtonKK commented Jan 19, 2025

@SammyOina please fix rebase, you have agent/cvm [introduced in this PR, cocos-344-c] and agent/cvms [was merged to main from cocos-344-a].

Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
Signed-off-by: Sammy Oina <sammyoina@gmail.com>
@SammyOina SammyOina requested a review from dborovcanin January 20, 2025 12:35
@dborovcanin dborovcanin merged commit 881aaaa into ultravioletrs:main Jan 20, 2025
2 of 4 checks passed
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

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载