Skip to content

Migrate old "Fabric3" -> "fabric" version 3.2 #296

Open
brainexe wants to merge 13 commits into
masterfrom
fabric
Open

Migrate old "Fabric3" -> "fabric" version 3.2 #296
brainexe wants to merge 13 commits into
masterfrom
fabric

Conversation

@brainexe

Copy link
Copy Markdown
Member

Our current Fabric3 is quite outdated and causing more and more compatibility issues (like python3.13)
-> use the maintained "fabric" version

already tested: basic VM start/stop/build/delete/sync/info/restart/disk-set/mem-set
todo: migration (offline + online) / AWS / rename / vcpu-set

Comment thread igvm/host.py Outdated
Comment thread igvm/host.py
Comment thread igvm/vm.py Outdated
@brainexe
brainexe force-pushed the fabric branch 2 times, most recently from 89b439e to 660f8e1 Compare February 13, 2026 07:38
@brainexe brainexe changed the title DRAFT: Migrate old "Fabric3" -> "fabric" version 3.2 Migrate old "Fabric3" -> "fabric" version 3.2 Feb 24, 2026
@brainexe
brainexe requested a review from lamaral February 24, 2026 08:39
@brainexe
brainexe force-pushed the fabric branch 2 times, most recently from 13bb332 to fc1e10b Compare February 26, 2026 05:57

@lamaral lamaral 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.

I did some limited testing and couldn't break it, so LGTM.

@brainexe
brainexe force-pushed the fabric branch 2 times, most recently from dccb019 to 5d853e5 Compare March 12, 2026 08:49
brainexe added 2 commits June 11, 2026 09:44
VM.put() runs mv/chmod on the HV without chroot when the VM is mounted,
so the target must be wrapped in vm_path(); otherwise plain paths like
'/etc/hostname' hit the HV's own filesystem. Wrap remote_path once inside
put() and drop the redundant wrap in upload_template().
- .deb: require python3-fabric | python3-fabric3
- fixed integration test
- proper retries in puppet command
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