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

reverting changes to allow schedule jobs to respect the remote policy #5033

Merged
merged 1 commit into from Jul 10, 2019

Conversation

@ltamaster
Copy link
Contributor

commented Jul 3, 2019

Is this a bugfix, or an enhancement? Please describe.
Bug:
reverting changes to allow schedule jobs to respect the remote policy execution if exists
#4799

Jobs scheduled can be removed from quartz, so the scheduled job won't be executed on any cluster member

Describe the solution you've implemented
the severNodeUUID cannot be changed, it just can be done when the job is saved.

reverting changes to allow schedule jobs to respect the cluster remot…
…e execution policy when it exists

#4799

Jobs scheduled can be remove from quartz, so the scheduled job won't be executed on any cluster member

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

@gschueler

This comment has been minimized.

Copy link
Member

commented Jul 8, 2019

@ltamaster does this revert all of the behavior of that previous change?

@ltamaster

This comment has been minimized.

Copy link
Contributor Author

commented Jul 8, 2019

@gschueler
​I think it will revert it completely​ because the goal of the previous feature was respected the remote execution policy. We need to double check how to reach that.

@gschueler gschueler merged commit da33831 into master Jul 10, 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

@gschueler gschueler added this to the 3.1.0-RC2 milestone Jul 10, 2019

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.