kubevirt / community

Community content
https://kubevirt.io
46 stars 101 forks source link

sig-storage: extend charter with sig responsabilities #274

Closed alicefr closed 2 months ago

alicefr commented 3 months ago

Extend sig-storage charter to include the group responsibilities and scope.

Release note:

NONE
alicefr commented 3 months ago

/cc @mhenriks @aglitke I tried to add some draft description for sig-storage. We are definitely missing more information about CDI. Would you mind taking a look and add what is missing

alicefr commented 3 months ago

Do you think memory dump falls under storage sig purview? Other than the fact that it writes to a PVC and integrates with vmexport I'm not sure it is very storage focused.

Maybe from the user point of view not much, but the implementation is very close to hotplug, I think we can add it here

fabiand commented 2 months ago

Looks good to me!

Thanks

/approve

kubevirt-bot commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabiand

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubevirt/community/blob/main/OWNERS)~~ [fabiand] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
fabiand commented 2 months ago

Is an update to https://github.com/kubevirt/community/blob/main/sigs.yaml needed?

alicefr commented 2 months ago

@fabiand do you mean for the list of the OWNER files?

fabiand commented 2 months ago

@alicefr I'm asking if the file needs an update. Does the subproject section (aka OWNERS files) need an update, or meetings, or … ?

alicefr commented 2 months ago

@fabiand I don't think so. I have already added myself and Mike was already there. Otherwise, the meeting notes are correct

alicefr commented 2 months ago

@mhenriks can you please take another look?

mhenriks commented 2 months ago

/lgtm