-
Notifications
You must be signed in to change notification settings - Fork 50
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
What do you think Dad ? #17
Open
clun
wants to merge
26
commits into
LukeTillman:master
Choose a base branch
from
KillrVideo:master
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
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
c3a7f58 Adding the web and generator components to the external compose yaml 065383c Removed dse container dependency from the headless-server script, it is no longer needed 9a421df Adding Studio support to the external script for use with our KillrVideo deployment 8e9f29b Properly removing the extras directory 2cfbbcd Simpified how we handle connecting to an external cluster in "headless" server mode Removed the extras directory entirely as the scripts within were essentially replicating the same type of yaml chaining done in the .env file Moved the docker-compose-server.yaml from the extras dir into the same area with all other yamls and renamed it to docker-compose-headless-server.yaml to make its function more obvious Updated docker-compose-external.yaml to rename the dse-config container to dse so it hooks up properly to other containers. This mimicks what happens in the non-external cluster case where a local DSE cluster is created within the dse container. Honestly, this only matters for cases where an external clsuter is being used. If you are using default settings you can ignore. a70c2f0 Updated docker-compose.yaml from killrvideo-dse-config 1.2.1 to 1.2.2 4ed12a9 updating to new version of killrvideo-dse-config, more reliable startup sequence 4258eab Updating to DSE 6 889f878 updating to new DSE docker images that create search schema using CQL 7f2d239 fixed issue with killrvideo-dse:4.0.2 image ca8e344 reverting back to killrvideo-dse:4.0.1 docker image, search core not loading correctly in 4.0.2 78f4014 add docker-compose configuration using killrvideo-dse-config / official DSE server image d17828b add docker-compose configuration for enabling DSE auth/auth 1d928f7 updating to DSE 5.1.6 732bd3a Modify loopback alias creation logic to work on both Linux and Darwin. c52f48b Merge pull request #7 from KillrVideo/memlock 9085233 updating to docker-compose version 3 format 0d7fff0 Restore IPC_LOCK / memlock options (closes #6) fec813c incrementing version of killrvideo-dse Docker image referenced in docker-compose scripts 6fd12c6 Adding additional docker-compose example files for using Studio, Ops Center, and volumes b141049 migrating to official DataStax docker images f5c25b5 Updated hardcoded entry for EXTERNAL_CLUSTER_IP to be an environ var reference I must have did this by accident on my last set of updates. This value should always read from the environment. 357e2af Merge pull request #5 from KillrVideo/add_server_dse_config 140c94b Moved "extra" configuration files and scripts into their own directory in order to keep things tidy 0ae6e7b Merge pull request #4 from KillrVideo/add_server_config 7f45352 Added separate docker yaml to handle using an external DSE cluster This is not meant for default use and is not used for the default, local environment setup. You should know what you are doing if you attempt to use this configuration. git-subtree-dir: lib/killrvideo-docker-common git-subtree-split: c3a7f58a5f840bc02ee4f959fc19bd8d2a294386
Dse graph : Merge PR live on twitch
closes #2
Refactored The Solution & Project Structure + Build Fixes
Upstream/remove etcd. Win!
…ly was never needed).
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.