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 getting ES configuration #8037

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

atsareg
Copy link
Contributor

@atsareg atsareg commented Feb 10, 2025

BEGINRELEASENOTES

*WorkloadManagement

FIX: get ElasticJobParametersDB index prefix from the configuration

*Configuration

FIX: evaluate useCRT flag as boolean in Utilities.getElasticDBParameters()

ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Feb 10, 2025
@atsareg atsareg added alsoTargeting:integration Cherry pick this PR to integration after merge and removed alsoTargeting:integration Cherry pick this PR to integration after merge labels Feb 10, 2025
@fstagni fstagni merged commit 554b37c into DIRACGrid:rel-v8r0 Feb 10, 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 10, 2025
@DIRACGridBot
Copy link

Sweep summary

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

Failed:

  • integration
    cherry-pick 554b37c into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-554b37c6e-integration
    git cherry-pick -x -m 1 554b37c6e
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #8037 Fix getting ES configuration' --author='Andrei Tsaregorodtsev <atsareg@in2p3.fr>'
    git push -u origin cherry-pick-2-554b37c6e-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 getting ES configuration' \
         --body 'Sweep #8037 `Fix getting ES configuration` to `integration`.
    
    Adding original author @atsareg as watcher.
    
    BEGINRELEASENOTES
    
    *WorkloadManagement
    
    FIX: get ElasticJobParametersDB index prefix from the configuration
    
    *Configuration
    
    FIX: evaluate useCRT flag as boolean in Utilities.getElasticDBParameters()
    
    ENDRELEASENOTES
    Closes #8038'

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

3 participants