Merged
Conversation
This file contains hidden or 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
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.
This makes some changes related to topics, adds/updates some CLI commands, and makes some improvements/fixes to the Docker launcher.
Most of the 'get' API endpoints have been removed, in favour of using topics to fetch the data. New topics have been added for tokens and manifests, and for getting an asset. And a topic has been added to inspect the queue of executions for a workspace.
New CLI commands have been added for listing pool launches (including getting information about a specific launch, including the end of the logs for failed launches), listing sessions, getting a pool's details, and monitoring the workspace's queue.
The Docker launcher has been updated to improve how the server host is inferred, with support for explicitly configuring the server host on a pool. The end of log output is captured for failed launches.
The
COFLUX_BASE_DOMAINenvironment variable has been replaced withCOFLUX_PUBLIC_HOST, with "%" being used to indicate the project, for multi-project deployments - e.g.,%.example.com.