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

Initial implementation #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

tjcelaya
Copy link

@tjcelaya tjcelaya commented Dec 1, 2017

Replication is working. JMX monitoring was working before, but no longer (and is probably not in scope).

Backups to Manta might be in scope? Still studying https://docs.datastax.com/en/cassandra/3.0/cassandra/operations/opsBackupRestore.html

Dockerfile Outdated

# Add Containerpilot and set its configuration
ENV CONTAINERPILOT /etc/containerpilot.json5
ENV CONTAINERPILOT_VERSION 3.3.3
Copy link

@jwreagor jwreagor Dec 8, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reason for using ContainerPilot 3.3.3?

You should at least include the series of fixes for memory leaks (#488 and #493) that were addressed with 3.4.2.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for calling that out, the majority of this Dockerfile was plundered from autopilotpattern/postgres but I hadn't gotten the change to review versions. Fixing this in the next commit.

# Start with a single host which will bootstrap the cluster.
# In production we'll want to use an HA cluster.
consul:
image: autopilotpattern/consul:0.7.2-r0.8
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've qualified Consul 1.0 support with the latest version of ContainerPilot and autopilotpattern/consul:1.0.0r43.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file has been neglected since I've been iterating on examples/compose/local-compose.yml while figuring stuff out.

I've actually been working against a locally built 1.0.0r47 that incorporates autopilotpattern/consul#46 and autopilotpattern/consul#47 in an effort to make this multi-DC capable from the start.

I will definitely revisit this file once local-compose.yml stabilizes

@jwreagor
Copy link

jwreagor commented Dec 8, 2017

So far this looks like a great addition to the autopilot library.

…e seeds lock at a time and all others will render their configs to include the seed node (even if it is still bootstrapping)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants