Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0] dmeta - Error message in case failure due to permission denied #7465

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

atsareg
Copy link
Contributor

@atsareg atsareg commented Feb 11, 2024

Make error message in case of metadata manipulation permission failre, e.g.:

$ dmeta add testfile Year="1997"
Error: /vo.formation.idgrilles.fr/user/a/atsareg/testmeta/testfile Permission denied ( 13 : Permission denied)

Otherwise, no message at all

BEGINRELEASENOTES

*Interfaces
CHANGE: dfind - more explicit failure report

ENDRELEASENOTES

@atsareg atsareg added the alsoTargeting:integration Cherry pick this PR to integration after merge label Feb 11, 2024
@fstagni fstagni requested a review from sfayer February 12, 2024 17:20
Copy link
Member

@sfayer sfayer left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

Regards,
Simon

@fstagni fstagni merged commit 3081d48 into DIRACGrid:rel-v8r0 Feb 14, 2024
25 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels Feb 14, 2024
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/7898744875

Failed:

  • integration
    cherry-pick 3081d48 into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-3081d4813-integration
    git cherry-pick -x -m 1 3081d4813
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #7465 dmeta - Error message in case failure due to permission denied' --author='Andrei Tsaregorodtsev <[email protected]>'
    git push -u origin cherry-pick-2-3081d4813-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] dmeta - Error message in case failure due to permission denied' \
         --body 'Sweep #7465 `dmeta - Error message in case failure due to permission denied` to `integration`.
    
    Adding original author @atsareg as watcher.
    
    BEGINRELEASENOTES
    
    *Interfaces
    CHANGE: dfind - more explicit failure report
    
    ENDRELEASENOTES
    Closes #7469'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants