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

on cluster environment , the job scheduled should respect the remote exec policies #5089

Merged
merged 3 commits into from Jul 30, 2019

Conversation

@ltamaster
Copy link
Contributor

commented Jul 25, 2019

Is this a bugfix, or an enhancement? Please describe.
Enhancement:

  • on cluster environment, the job scheduled should respect the remote execution policies (if it is enabled).
  • When the job is created, it will be checked if the job will be scheduled on another cluster member or locally.
  • When a job is updated, that was already considered. But, it was added a flag to avoid registering on the local quartz when the job is owned by another node.
  • for takeover API, it will be forced to run locally

Describe the solution you've implemented
A new method was added to JobScheduleManager (scheduleRemoteJob) which will check if the remote policies are available on cluster env (and it will send the takeover message if it is enabled)

Describe alternatives you've considered

Additional context

on cluster environment , the job scheduled should respect the remote …
…execution policies

A new method was added to JobScheduleManager (scheduleRemoteJob) which will check if the remote policies are available on cluster env (and it will send the takeover message if it is enabled)

@ltamaster ltamaster requested a review from gschueler Jul 25, 2019

@ltamaster ltamaster added this to the 3.1.0 milestone Jul 30, 2019

@gschueler gschueler merged commit 3ba6c42 into master Jul 30, 2019

20 checks passed

Mergeable Mergeable Run has been Completed!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
security/snyk - build.gradle (rundeck) No manifest changes detected
security/snyk - core/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/copyfile-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/flow-control-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/git-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/jasypt-encryption-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/job-state-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/localexec-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/orchestrator-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/script-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/source-refresh-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/stub-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - plugins/upvar-plugin/build.gradle (rundeck) No manifest changes detected
security/snyk - rundeck-storage/build.gradle (rundeck) No manifest changes detected
security/snyk - rundeckapp/build.gradle (rundeck) No manifest changes detected
security/snyk - rundeckapp/grails-spa/package.json (rundeck) No manifest changes detected
security/snyk - rundeckapp/metricsweb/build.gradle (rundeck) No manifest changes detected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.