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

DTSAM-317 Update processing of REFRESH_JOB env var #1923

Merged
merged 6 commits into from
May 30, 2024

Conversation

mattnayler
Copy link
Contributor

Jira link (if applicable)

DTSAM-263 "Updates to allow AM Refreshes to be run unmanned"
DTSAM-317 "AM Refresh Updates: ORM: Registering Refresh Job Configuration"

Change description

Update processing of REFRESH_JOB environment variable on start up:

  • Allow it to generate multiple refresh jobs at a time.
  • Prevent unexpected override of job config during startup.

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change NO

mattnayler and others added 3 commits May 15, 2024 11:54
   [DTSAM-263](https://tools.hmcts.net/jira/browse/DTSAM-263)
   [AM-2902](https://tools.hmcts.net/jira/browse/AM-2902)

Import some elements of AM-2902:
* Update the processing of the REFRESH_JOB env variable on start up to
  allow it to generate multiple refresh jobs at a time.
Copy link
Contributor

@mikebrownccd mikebrownccd left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@sumansreenivasan sumansreenivasan left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebrownccd mikebrownccd merged commit 4f676b7 into master May 30, 2024
7 checks passed
@mikebrownccd mikebrownccd deleted the DTSAM-317__Refresh_Job_Config branch May 30, 2024 13:30
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