Tom Parrott

Results 1094 comments of Tom Parrott

I've tried that reproducer though, but using `lxc storage volume info` for step 4 and never seen the snapshot in 5.0, 5.21 or latest

@markylaing its this line that is incorrectly filtering out snapshots https://github.com/canonical/lxd/blob/main/lxd/storage_pools.go#L673 Please can you take a look

> Oh interesting, I am authenticated with TLS if that helps, I'd expect to see all used by resources in this case. Its happening with unix socket too.

@markylaing the error is coming from https://github.com/canonical/lxd/blob/c74a664d5716a712dae4afe37012e0cbcfbd8a07/lxd/project/permissions.go#L67-L69 ``` ERROR [2024-10-18T13:43:21Z] tomp4 err="Cannot check permissions for entity type \"storage_volume_snapshot\" and entitlement \"can_view\": No entitlements can be granted against entities of type...

@markylaing tracked the issue to https://github.com/canonical/lxd/blob/c74a664d5716a712dae4afe37012e0cbcfbd8a07/lxd/auth/entitlements_generated.go#L579-L589 There seems to be a missing section for entity type `storage_volume_snapshot` which the resource `1.0/storage-pools/default/volumes/custom/test-vol/snapshots/snap0` maps to.

@markylaing should we add a section for entity.TypeStorageVolumeSnapshot to that map, or instead should `entity.ParseURL(` not being parsing a snapshot URL into its own entity type, but rather that of...

Can you review that PR in full and see which commits are suitable (even if not directly related to resizing) ta.