Skip to content

Allow admin to override $GALAXY_MEMORY_MB#21700

Merged
mvdbeek merged 2 commits intogalaxyproject:devfrom
natefoo:override-galaxy-memory-mb
Jan 31, 2026
Merged

Allow admin to override $GALAXY_MEMORY_MB#21700
mvdbeek merged 2 commits intogalaxyproject:devfrom
natefoo:override-galaxy-memory-mb

Conversation

@natefoo
Copy link
Member

@natefoo natefoo commented Jan 29, 2026

Otherwise, $GALAXY_MEMORY_MB is unconditionally overridden.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@github-actions github-actions bot added this to the 26.1 milestone Jan 29, 2026
@natefoo natefoo marked this pull request as draft January 29, 2026 22:03
@natefoo
Copy link
Member Author

natefoo commented Jan 29, 2026

Draft until I test on Main

@natefoo
Copy link
Member Author

natefoo commented Jan 30, 2026

Works on Main.

@natefoo natefoo marked this pull request as ready for review January 30, 2026 19:12
@mvdbeek mvdbeek merged commit e0ece84 into galaxyproject:dev Jan 31, 2026
56 checks passed
@mvdbeek
Copy link
Member

mvdbeek commented Jan 31, 2026

I suppose you'll also want to sync that back to pulsar (https://github.com/galaxyproject/pulsar/blob/master/pulsar/managers/util/job_script/MEMORY_STATEMENT.sh)?

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.

3 participants