Skip to content

Conversation

@nesitor
Copy link
Member

@nesitor nesitor commented Jul 12, 2023

Added support for BTRFS to execute a VM instance.

@nesitor nesitor requested review from hoh and odesenfans July 12, 2023 12:03
@nesitor nesitor self-assigned this Jul 12, 2023
Copy link
Contributor

@odesenfans odesenfans left a comment

Choose a reason for hiding this comment

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

Minor comments.

@nesitor nesitor requested a review from odesenfans July 12, 2023 15:42
odesenfans and others added 2 commits July 13, 2023 12:27
@odesenfans odesenfans merged commit 84ed7d6 into main Jul 14, 2023
@odesenfans odesenfans deleted the andres-fix-support-btrfs-filesystem branch July 14, 2023 13:43
nesitor added a commit that referenced this pull request Jul 14, 2023
Added support for BTRFS to execute a VM instance.
nesitor added a commit that referenced this pull request Jul 14, 2023
* Problem: Instances loose data between nodes, so we need a way to handle and share this data.

Feature: Implement VM snapshots that will be managed independently.

* Fix: Added some refactor for snapshots and handled some errors.

* Fix: Split snapshot class in two.

* Fix: Make it work well with threads and asyncio.

* Fix: Added code improvements with threading and fixed PR comments.

* Fix: Fixed check_disk method error.

* Fix: Fixed more snapshot errors with multithreading

* Fix: CI for Ubuntu was failing too often because of apt/dpkg locks (#380)

Problem: the CI is looking for only one lock file, but apt/dpkg use several. Furthermore, `lslocks --json` seems to cut its output to a specific width, and configuration options (`--notruncate`) do not appear to have an impact.

Solution: repeat calls to apt-get update until we get the lock. This is not perfect but increases the rate of success.

* Create Ubuntu rootfs (#370)

Updated creation script to create a BTRFS image of Ubuntu filesystem.

* Fix: Use SDK version 0.7.0 (#371)

* Support BTRFS filesystem for instances (#373)

Added support for BTRFS to execute a VM instance.

* Feature: Delete old snapshots after do a new one.

* add dependency to python3-schedule

* Fix latest_snapshot attribute error

* default snapshot frequency 1 hour

---------

Co-authored-by: Olivier Desenfans <[email protected]>
Co-authored-by: Hugo Herter <[email protected]>
Co-authored-by: Andres D. Molins <[email protected]>
hoh pushed a commit that referenced this pull request Mar 12, 2024
Added support for BTRFS to execute a VM instance.
hoh added a commit that referenced this pull request Mar 12, 2024
* Problem: Instances loose data between nodes, so we need a way to handle and share this data.

Feature: Implement VM snapshots that will be managed independently.

* Fix: Added some refactor for snapshots and handled some errors.

* Fix: Split snapshot class in two.

* Fix: Make it work well with threads and asyncio.

* Fix: Added code improvements with threading and fixed PR comments.

* Fix: Fixed check_disk method error.

* Fix: Fixed more snapshot errors with multithreading

* Fix: CI for Ubuntu was failing too often because of apt/dpkg locks (#380)

Problem: the CI is looking for only one lock file, but apt/dpkg use several. Furthermore, `lslocks --json` seems to cut its output to a specific width, and configuration options (`--notruncate`) do not appear to have an impact.

Solution: repeat calls to apt-get update until we get the lock. This is not perfect but increases the rate of success.

* Create Ubuntu rootfs (#370)

Updated creation script to create a BTRFS image of Ubuntu filesystem.

* Fix: Use SDK version 0.7.0 (#371)

* Support BTRFS filesystem for instances (#373)

Added support for BTRFS to execute a VM instance.

* Feature: Delete old snapshots after do a new one.

* add dependency to python3-schedule

* Fix latest_snapshot attribute error

* default snapshot frequency 1 hour

---------

Co-authored-by: Olivier Desenfans <[email protected]>
Co-authored-by: Hugo Herter <[email protected]>
Co-authored-by: Andres D. Molins <[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