Skip to content

fix(tag): throw an error when creating tag already existing - #41

Merged
brenot-a merged 2 commits into
5.0-mgdisfrom
fix/PLAID-152965-existing-tag
Oct 22, 2026
Merged

brenot-a merged 2 commits into
5.0-mgdisfrom
fix/PLAID-152965-existing-tag

Conversation

@brenot-a

@brenot-a brenot-a commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

SUMMARY

@brenot-a brenot-a changed the title Fix/plaid 152965 existing tag Fix: throw an error when creating tag already existing Oct 16, 2026
@brenot-a brenot-a changed the title Fix: throw an error when creating tag already existing Fix(tag): throw an error when creating tag already existing Oct 16, 2026
@brenot-a brenot-a changed the title Fix(tag): throw an error when creating tag already existing fix(tag): throw an error when creating tag already existing Oct 16, 2026
@brenot-a
brenot-a marked this pull request as ready for review October 19, 2026 12:11
@legal-s

legal-s commented Oct 21, 2026

Copy link
Copy Markdown
Collaborator

@brenot-a Can you reference the PRs from which you retrieved the code for these backports?

@brenot-a

Copy link
Copy Markdown
Collaborator Author

@legal-s the PR apache#43390

I did not port the exact changes, but rather the concept of what they did. I applied the same principle, not allowing a user to modify tags when they are not permitted to, to v5
Upstream changes could not be applied as-is on v5: daos/tag.py and commands/tag/utils.py differ from master

@brenot-a
brenot-a force-pushed the fix/PLAID-152965-existing-tag branch from 1096d1f to 694a2af Compare October 21, 2026 14:29
@brenot-a
brenot-a merged commit 5eccfa2 into 5.0-mgdis Oct 22, 2026
46 checks passed
@brenot-a
brenot-a deleted the fix/PLAID-152965-existing-tag branch October 22, 2026 07:18
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