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.

Sign up for GitHub

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

Jump to bottom

Restore default for PBS max job name len. #5386

Merged
merged 7 commits into from
Mar 8, 2023
Merged

Restore default for PBS max job name len. #5386

merged 7 commits into from
Mar 8, 2023

Commits on Mar 2, 2023

  1. Restore default for PBS max job name len. 

    This needs to be used if it is not set in the site config and
was mistakenly removed.
    @wxtim
    wxtim committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    ffaa102 View commit details
    Browse the repository at this point in the history

  2. Update tests/unit/job_runner_handlers/test_pbs.py 

    Co-authored-by: Oliver Sanders <oliver.sanders@metoffice.gov.uk>
    @wxtim @oliver-sanders
    wxtim and oliver-sanders committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    bf7dca8 View commit details
    Browse the repository at this point in the history

  3. response to review

    @wxtim
    wxtim committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    616c57f View commit details
    Browse the repository at this point in the history

  4. make names of tests safer

    @wxtim
    wxtim committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    37fde34 View commit details
    Browse the repository at this point in the history

  5. fixed conflict in changes

    @wxtim
    wxtim committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    3399adb View commit details
    Browse the repository at this point in the history

  6. Update tests/unit/job_runner_handlers/test_pbs.py 

    Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
    @wxtim @MetRonnie
    wxtim and MetRonnie authored Mar 2, 2023
    Configuration menu
    Copy the full SHA
    3781eb0 View commit details
    Browse the repository at this point in the history

  7. fix test

    @wxtim
    wxtim committed Mar 2, 2023
    Configuration menu
    Copy the full SHA
    f334937 View commit details
    Browse the repository at this point in the history