Skip to content

fix sticky ADMIN_KW - #834

Open
d-w-moore wants to merge 5 commits into
irods:mainfrom
d-w-moore:833.m
Open

d-w-moore wants to merge 5 commits into
irods:mainfrom
d-w-moore:833.m

Conversation

@d-w-moore

@d-w-moore d-w-moore commented Sep 9, 2026 •

Copy link
Copy Markdown
Collaborator

Use of ADMIN_KW carried over into subsequent metadata calls even if unwanted.

Comment thread irods/manager/metadata_manager.py Outdated
Comment thread irods/test/meta_test.py Outdated
Comment thread irods/test/meta_test.py Outdated

@korydraughn korydraughn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does the updated test reproduce the bug reported in the issue?

Comment thread irods/manager/metadata_manager.py Outdated
Comment thread irods/test/meta_test.py Outdated
Comment thread irods/test/meta_test.py Outdated
Comment thread irods/test/meta_test.py Outdated
@d-w-moore

d-w-moore commented Sep 18, 2026 •

Copy link
Copy Markdown
Collaborator Author

Does the updated test reproduce the bug reported in the issue?

Yes. if you revert the changes, the test fails on the line d.metadata.set('a','b') with an INSUFFICIENT_PRIVILEGE_LEVEL error, agreeing with the issue report and diagnosis.

@korydraughn korydraughn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Squash it if everything is functioning correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants