Helder Ardachnikoff comments

Results 9 comments of


                                            Helder Ardachnikoff

Allow overriding instance properties during backup restore (similar to `incus copy`)

Hi, I play around with import/export and it seems easy to add these two overrides! Can I be assigned on this issue?

Allow overriding instance properties during backup restore (similar to `incus copy`)

CR: https://github.com/lxc/incus/pull/2475

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

I'd like to take this one! @stgraber may I ask for help about which components I should digging deep to understand how to make those changes

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

I spent some time understanding the snapshot code in Incus and came to the following conclusion regarding this change: - Need to add new version on update.go to add instances_snapshots_properties,...

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

@stgraber Thank you for your help! It's clear for me now 👍

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

@stgraber I'm checking if would really make sense move stateful from instances_snapshot to properties' table, since there's stateful flag on snapshot creation ```makefile Examples: incus snapshot create u1 snap0 Create...

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

ok, thanks

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

Hi @stgraber, I need a help again rsrs - quite challenge this change :-) I notice that DROP A COLUMN is not recognized by **generate-database**, I had check on incus...

Re-organize snapshot handling, allow for snapshot-specific properties (description, expiry, ...)

Hi @stgraber, I'm almost done with this change! I'd like to ask for help! Basically I'm planning add **InstanceProperties** struct nested on **InstanceSnapshot** struct in order to handle in a...