Skip to content

Revert "mgr/volumes: handle bad arguments during subvolume create"#63281

Merged
vshankar merged 1 commit intomainfrom
revert-53989-mds-handle-bad-arguments-during-subvolume-create-2
May 19, 2025
Merged

Revert "mgr/volumes: handle bad arguments during subvolume create"#63281
vshankar merged 1 commit intomainfrom
revert-53989-mds-handle-bad-arguments-during-subvolume-create-2

Conversation

@vshankar
Copy link
Contributor

@vshankar vshankar commented May 14, 2025

PR #53989 is causing failures in
fs:upgrade. Also, @VallariAg reported an issue with something
similar. I don't think adequate tests were run to qualify the PR
as mergeable. Reverting the change for now.

Signed-off-by: Venky Shankar vshankar@redhat.com

Reverts #53989

@vshankar vshankar requested a review from a team May 14, 2025 10:17
@vshankar vshankar requested a review from a team as a code owner May 14, 2025 10:17
@vshankar vshankar force-pushed the revert-53989-mds-handle-bad-arguments-during-subvolume-create-2 branch from 3298e14 to ba7db79 Compare May 14, 2025 10:31
@anthonyeleven
Copy link
Contributor

wai?

@vshankar
Copy link
Contributor Author

wai?

Apologies - I didn't quite get that.

@anthonyeleven
Copy link
Contributor

Suggesting adding some rationale for the reversion to the PR description.

@vshankar
Copy link
Contributor Author

Suggesting adding some rationale for the reversion to the PR description.

Fair enough. I created this revert via GitHub UI which does bare minimum if putting description. I'll add some...

PR #53989 is causing failures in
fs:upgrade. Also, @VallariAg reported an issue with something
similar. I don't think adequate tests were run to qualify the PR
as mergeable. Reverting the change for now.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
@vshankar vshankar force-pushed the revert-53989-mds-handle-bad-arguments-during-subvolume-create-2 branch from ba7db79 to ddb86b7 Compare May 14, 2025 10:57
@vshankar
Copy link
Contributor Author

Commit message and PR updated.

@vshankar
Copy link
Contributor Author

jenkins test make check

@vshankar
Copy link
Contributor Author

jenkins test make check arm64

2 similar comments
@nizamial09
Copy link
Member

jenkins test make check arm64

@vshankar
Copy link
Contributor Author

jenkins test make check arm64

Copy link
Contributor

@rishabh-d-dave rishabh-d-dave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rishabh-d-dave
Copy link
Contributor

jenkins test make check arm64

2 similar comments
@vshankar
Copy link
Contributor Author

jenkins test make check arm64

@vshankar
Copy link
Contributor Author

jenkins test make check arm64

@vshankar vshankar merged commit df7b755 into main May 19, 2025
18 checks passed
@vshankar vshankar deleted the revert-53989-mds-handle-bad-arguments-during-subvolume-create-2 branch May 19, 2025 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants