TrueCloudLab/frostfs-node

cli: Introduce flag `--all` for `remove-rule` #976

New issue

Closed

opened 2024-02-12 12:35:09 +00:00 by aarifullin · 1 comment

aarifullin commented

2024-02-12 12:35:09 +00:00

Member

It would be better to introduce a method to drop all local overrides. So, we could introduce a flag -all for remove-rule omitting chain-ids. That definitely will make user experience better.
To implement this flag we can list targets, list all chains for targets and remove them.

NOTE: Control API does not let to recieve a batch of chains in a request: so, one chain per one request.

It would be better to introduce a method to drop all local overrides. So, we could introduce a flag `-all` for `remove-rule` omitting chain-ids. That definitely will make user experience better. To implement this flag we can list targets, list all chains for targets and remove them. NOTE: Control API does not let to recieve a batch of chains in a request: so, one chain per one request.

aarifullin added the

good first issue

labels 2024-02-12 12:35:18 +00:00

acid-ant self-assigned this 2024-02-13 06:52:56 +00:00

acid-ant changed title from ~~Introduce flag --all for remove-rule in frostfs-cli~~ to cli: Introduce flag --all for remove-rule

2024-02-13 08:02:44 +00:00

fyrchik added this to the v0.39.0 milestone 2024-02-14 06:30:45 +00:00

fyrchik modified the milestone from v0.39.0 to v0.38.0

2024-02-14 06:30:50 +00:00

acid-ant referenced this issue from a pull request that will close it,

2024-03-07 11:37:50 +00:00

ape: Remove all chains by target #1031

acid-ant closed this issue

2024-03-13 18:35:23 +00:00

acid-ant referenced this issue from a commit

2024-03-13 18:35:24 +00:00

[#976] morph: Implement missing methods for `SwitchRPCGuardedActor`

acid-ant referenced this issue from a commit

2024-03-13 18:35:24 +00:00

[#976] ape: Update dependency

acid-ant referenced this issue from a commit

2024-03-13 18:35:25 +00:00

[#976] control: Introduce new method `RemoveChainLocalOverridesByTarget`

acid-ant referenced this issue from a commit

2024-03-13 18:35:25 +00:00

[#976] cli: Allow to remove all chains by target

acid-ant referenced this issue from a commit

2024-03-13 18:35:25 +00:00

[#976] adm: Allow to remove all chains by target

acid-ant referenced this issue from a commit

2024-03-13 18:35:25 +00:00

[#976] Fix trailing whitespace and end of file

acid-ant commented

2024-03-13 19:15:00 +00:00

Member

Introduced flag --all for cli and adm for commands:

frostfs-cli control remove-rule --endpoint ... -c cfg.yml --address ... --target-type namespace --target-name nsname --all
frostfs-adm morph ape rm-rule-chain -c cfg.yml -r ... --target-type container --target-name "TEST0000" --chain-name s3 --all

Introduced flag `--all` for `cli` and `adm` for commands: - `frostfs-cli control remove-rule --endpoint ... -c cfg.yml --address ... --target-type namespace --target-name nsname --all` - `frostfs-adm morph ape rm-rule-chain -c cfg.yml -r ... --target-type container --target-name "TEST0000" --chain-name s3 --all`

Sign in to join this conversation.

No labels

good first issue

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#976

No description provided.