This repository has been archived by the owner on Sep 21, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1
Dropwizard082 basiccallbak #1
Open
sverhoeven
wants to merge
80
commits into
sverhoeven:master
Choose a base branch
from
xenon-middleware:dropwizard082-basiccallbak
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Dropwizard082 basiccallbak #1
sverhoeven
wants to merge
80
commits into
sverhoeven:master
from
xenon-middleware:dropwizard082-basiccallbak
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add test for toString of JobLauncherConfiguration.
The url of the second job was the url of the first job with the second job identifier appended. Because path() method of uri builder keeps appending. Fixed by moving path() outside loop with a template variable.
As output like stdout/stderr could contain details about exception.
See xenon-middleware/xenon#73 Now `octopus.files().cancelJob(job)` method returns status so we use that.
See xenon-middleware/xenon#106 So use the Ocotopus job identifier as key. Replaced usage of uuid with simple string, to prevent expectations that key will be an UUID.
…at key will be an UUID.
…nding/running jobs. Fixes #1. Document and test the different preStage/postStage variants. When job times out then don't download the sandbox. Only download sandbox for jobs that are done. After job cancelation remove the sandbox.
…topus(octopus). Octopus job is more strict so add a JobDescription during testing.
…n expected job description. Job can be cancelled before the job status has been fetched, added test for it.
…Octopus and runs shell commands. The test to verify the http client can handle MAC authentication has been refactored. Instead of using an external http server to test against in now uses an internal one. The internal http server is part of the apache http client tests. The integration config file is not needed anymore.
Octopus Sandox requires a Files object instead of an Octopus object.
…e using files engine instead of another path.
They are seen as errors by Netbeans
Fixes #8
Get Osmium to compile cleanly
Upped version as Osmium is java 8 compliant.
environment now comes after stderr and before the optional callback.
Feature: environment variables
This would be relevant to compute on multiple different types of clusters. The configuration is backwards compatible, so joblauncher.yml can state: ``` xenon: scheduler: scheme: local queue: multi ... ``` or: ``` xenon: schedulers: mycomputer: scheme: local queue: multi mycluster: scheme: ssh queue: single defaultScheduler: mycomputer ... ``` The property `scheduler` is mutually exclusive with `schedulers` and `defaultScheduler`.
The configuration now takes multiple launchers, each with its own sandbox and scheduler. This way both SSH and local file systems can be supported, along with the scheduling system.
The mac auth for callback can not be implemented in latest http client, switching to basic auth for callbacks.
…ropwizard. Upgraded cobertura-maven-plugin so fix error.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.