-
Notifications
You must be signed in to change notification settings - Fork 7
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
Release 3.5.0 #17
Closed
Closed
Release 3.5.0 #17
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
…ill work + make unitTest default "test" task (pull request #114) Approved-by: Rick Herrick Approved-by: Bin Zhang
* Bump XNAT dependency back down to 1.8.10 * Remove deprecated and unused methods from ContainerControlApi/DockerControlApi * Minor docs fix: add <p> to empty lines * Minor: use isEmpty instead of size > 0 * Remove unused param on list images * Change a one-case switch statement to an if statement * Minor test log fix * DockerControlApiIntegrationTest: assorted refactors * Add test util method to get backend config * Migrate CommandLaunchIntegrationTest to use backend config * Rename JmsConsumerExceptionTest so that it isn't included in integration tests * Migrate all other integration tests to use backend config * Use slightly different assume method to skip swarm and kubernetes tests * Fix errors in DockerServiceIntegrationTest * Also check if we are running integration tests in all the client skip util methods * Don't close the client after each test * Fix CommandLaunchIntegrationTest failure from shared project mounting * Use mock k8s client on connection problems * Refactor integration tests to not use static docker clients * Store timestamp as a string (in anticipation of new client, which does not parse timestamps) * Reuse part of docker client caching that closes out the client * Update expected media type in rest api tests from "application/json;charset=UTF-8" to "application/json" * Stop asserting on the content type of values returned from the rest api * Only create command once in CommandRestApiTest#testGetAll Approved-by: Rick Herrick Approved-by: James Ransford Approved-by: Bin Zhang
…esGpu that requests a swarm service with a GPU and verifies the backend received that message
…request #119) * Migrate getContainerEvents * Migrate startDockerContainer and setSwarmServiceReplicasToOne * Migrate DockerControlApiIntegrationTest to use new client * Adjust docker http client timeouts * Add another getDockerClientNew method with no args, use default server * Massive change to migrate all integration test client operations use new client * Migrate create container and create service * Migrate delete image * Slight tweaks to BUSYBOX constants and versions * Migrate inspect image / get image by id * Standardize docker-java exception handling * Migrate list images * Migrate ping * Reuse RETURN_SELF answer in test mocks * Migrate container/service logs * Migrate getTaskForService * Migrate pull image (with auth) * Add support for generic resources (i.e. gpus) using a fork of docker-java * Migrate TASK_STATE_FAILED to TaskState.FAILED * Move client config into its own method * Migrate AuthConfig and getting registry URL for image * Complete removal of docker-client * Fix hub ping * Clean up some more fully qualified classes left over from transition * Changelog Approved-by: James Ransford Approved-by: Bin Zhang
* Add dependabot file * Add gradle build workflow * Clean up / modify jar manifest definition * Add some logging around manifest definition (for testing purposes) * Modify how manifest is built and exception is caught * Update version of git-details plugin * Remove maven settings plugin and mavenLocal repository * Remove maven publishing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.