On Thu, Sep 09, 2021 at 06:13:16PM +0200, Pavel Hrdina wrote: > Rebased on top of current master. > > I'm posting this as an RFC mainly because I'm not sure how to model > the new API. This patches introduce a new naive API that will change > only the APPID and nothing else. > > Currently there are no other known features related to Fibre Channel > resources so this non-extendable API will be sufficient, however the > appid lives in <resource> element in the XML where we currently have > root cgroup partition. Even though changing the partition will not be > supported and we don't know about anything else that could be placed > here it doesn't mean it will not happen in the future. In that case > we would have to add new API as well. I'm curious why we need to even support updates for the APPID in the first place. I understood it to be a unique identifier for the VM from POV of the storage, and unique identifier are not something you would typically change on the fly. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|