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] Fix return code #8013

Merged
merged 2 commits into from
Feb 3, 2025
Merged

[8.0] Fix return code #8013

merged 2 commits into from
Feb 3, 2025

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Jan 28, 2025

Should also closes #8010

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Jan 28, 2025
@fstagni fstagni linked an issue Jan 28, 2025 that may be closed by this pull request
@chaen
Copy link
Contributor

chaen commented Feb 3, 2025

Do the change in the run module tests belong to this PR ?

@fstagni
Copy link
Contributor Author

fstagni commented Feb 3, 2025

Do the change in the run module tests belong to this PR ?

removed

@chaen chaen merged commit 059fe49 into DIRACGrid:rel-v8r0 Feb 3, 2025
26 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 3, 2025
@DIRACGridBot
Copy link

Sweep summary

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

Failed:

  • integration
    cherry-pick 059fe49 into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-059fe497a-integration
    git cherry-pick -x -m 1 059fe497a
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #8013 Fix return code' --author='Federico Stagni <[email protected]>'
    git push -u origin cherry-pick-2-059fe497a-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] Fix return code' \
         --body 'Sweep #8013 `Fix return code` to `integration`.
    
    Adding original author @fstagni as watcher.
    
    
    Closes #8023'

fstagni added a commit to fstagni/DIRAC that referenced this pull request Feb 3, 2025
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.

Non-zero exit code for proxy command
3 participants