ceph / ceph/go-ceph

Deprecate `PinSubVolumeGroup` in favour of `PinSubVolumeInGroup`

Open
#1,306 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
691
Forks
296
Avg merge
2d 16h
Merged PRs (30d)
11

Description

> Should we deprecate PinSubVolumeGroup since this call should be able to do everything PinSubVolumeGroup does (right)?

Good point.

> Even if yes, we don't have to do it right away.

I'll create an issue to be addressed once `PinSubVolumeInGroup` becomes stable.

_Originally posted by @anoopcs9 in https://github.com/ceph/go-ceph/issues/1305#issuecomment-5168921208_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the PinSubVolumeGroup and PinSubVolumeInGroup API definitions and checking whether PinSubVolumeInGroup is stable. Review their usages and related tests or documentation; done means the project has agreed that the older call can be deprecated in favor of the stable replacement.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.