Skip to content

COCOS-346 - Explore cloud init for Cloud setup#357

Merged
dborovcanin merged 10 commits into
ultravioletrs:mainfrom
smithjilks:cocos-346-explore-cloud-init
Jan 31, 2025
Merged

COCOS-346 - Explore cloud init for Cloud setup#357
dborovcanin merged 10 commits into
ultravioletrs:mainfrom
smithjilks:cocos-346-explore-cloud-init

Conversation

@smithjilks

Copy link
Copy Markdown
Contributor

What type of PR is this?

This is a feature because it adds cloud init config for EOS for cocos.

What does this do?

It sdds cloud init config for EOS for cocos.

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

Resolves #346

  • Related Issue #
  • Resolves #

Have you included tests for your changes?

No

Did you document any new/modified feature?

No.

Notes

@smithjilks smithjilks self-assigned this Jan 21, 2025
@smithjilks smithjilks requested a review from SammyOina January 21, 2025 07:49
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from e3365c9 to 7873580 Compare January 23, 2025 23:40
@codecov

codecov Bot commented Jan 23, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.13%. Comparing base (881aaaa) to head (b348602).
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #357   +/-   ##
=======================================
  Coverage   61.13%   61.13%           
=======================================
  Files          54       54           
  Lines        4371     4371           
=======================================
  Hits         2672     2672           
  Misses       1419     1419           
  Partials      280      280           

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

Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from 7873580 to 24ebef1 Compare January 24, 2025 13:26
Comment thread hal/cloud/meta-data Outdated
@@ -0,0 +1,2 @@
instance-id: iid-cocos-vm
local-hostname: cocos-vm No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
local-hostname: cocos-vm
local-hostname: cocos-vm

Comment thread hal/cloud/qemu.sh Outdated
-device vhost-vsock-pci,id=vhost-vsock-pci0,guest-cid=198 \
-object sev-snp-guest,id=sev0,cbitpos=51,reduced-phys-bits=1 \
-drive if=pflash,format=raw,unit=0,file=$OVMF_CODE,readonly=on \
-drive if=pflash,format=raw,unit=1,file=$OVMF_VARS No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
-drive if=pflash,format=raw,unit=1,file=$OVMF_VARS
-drive if=pflash,format=raw,unit=1,file=$OVMF_VARS

Comment thread hal/cloud/user-data Outdated
- systemctl enable cocos-agent.service
- systemctl start cocos-agent.service

final_message: "Cocos agent setup complete and service started successfully." No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
final_message: "Cocos agent setup complete and service started successfully."
final_message: "Cocos agent setup complete and service started successfully."

Comment thread hal/cloud/user-data Outdated
- git clone https://github.com/ultravioletrs/cocos.git /home/cocos_user/cocos

# Download and Install the agent binary
- wget -q https://github.com/ultravioletrs/cocos/releases/download/v0.3.1/cocos-agent

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use some way to set this variable

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the version

Comment thread hal/cloud/user-data Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also include docker installation, also set mtls cert files and env files

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from dd60c71 to 5091671 Compare January 28, 2025 01:08
Signed-off-by: Jilks Smith <smithjilks@gmail.com>
Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from 5091671 to dff9f1c Compare January 28, 2025 09:49
…to cocos-346-explore-cloud-init

# Conflicts:
#	hal/cloud/qemu.sh
Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@drasko drasko changed the title COCO-346 - Explore cloud init for EOS COCOS-346 - Explore cloud init for EOS Jan 28, 2025
Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from 251448a to b68819c Compare January 30, 2025 07:45

@dborovcanin dborovcanin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we have a basic README here with an explanation on how to use these scripts?

Signed-off-by: Jilks Smith <smithjilks@gmail.com>
Signed-off-by: Jilks Smith <smithjilks@gmail.com>
@smithjilks smithjilks force-pushed the cocos-346-explore-cloud-init branch from 8af5b07 to b348602 Compare January 30, 2025 13:52
Comment thread .github/workflows/hal.yml
buildroot/output/images/bzImage
buildroot/output/images/rootfs.cpio.gz

cocos/build/cocos-agent

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is there a step to build these binaries?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. There is.

There is a make run.

Comment thread .github/workflows/hal.yml
make BR2_EXTERNAL=../cocos/hal/linux cocos_defconfig
make

- name: Build cocos

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Here is the run.

@SammyOina SammyOina changed the title COCOS-346 - Explore cloud init for EOS COCOS-346 - Explore cloud init for Cloud setup Jan 31, 2025
@dborovcanin dborovcanin merged commit da88fe1 into ultravioletrs:main Jan 31, 2025
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.

Explore cloud-init for building EOS

3 participants