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

[cassandra] Add YAML for Cassandra version > 2.2 #2142

Merged
merged 1 commit into from
Dec 22, 2015

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Dec 10, 2015

@yannmh yannmh added the checks label Dec 10, 2015
@yannmh yannmh self-assigned this Dec 10, 2015
@yannmh yannmh added this to the 5.7.0 milestone Dec 10, 2015
- system
- system_auth
- system_distributed
- system_traces
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add OpsCenter to this list? It's a very common tool and I don't think anyone cares about metrics on it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for your feedback on it @sethrosenblum. I am adding it to the list of exclusion.

[[email protected]] Rebased to comply with contribution guidelines.
Also added `keyspace=OpsCenter` to the list of exclusions.
@yannmh yannmh force-pushed the yann/cassandra-2.2-yaml branch from 2df7566 to 2466541 Compare December 14, 2015 18:30
@yannmh yannmh removed their assignment Dec 22, 2015
@yannmh
Copy link
Member Author

yannmh commented Dec 22, 2015

We are entirely dropping the old Cassandra YAML configuration file. @remh can you take a pass on this PR and merge it please ?

remh pushed a commit that referenced this pull request Dec 22, 2015
[cassandra] Add YAML for Cassandra version > 2.2
@remh remh merged commit fa3e9e7 into master Dec 22, 2015
@remh
Copy link

remh commented Dec 22, 2015

Thanks!

@yannmh yannmh deleted the yann/cassandra-2.2-yaml branch December 23, 2015 16:20
yannmh added a commit to DataDog/chef-datadog that referenced this pull request Jan 19, 2016
* Enhance `DatadogMonitor` resource with a `version` attribute.
* Use `version` attribute in `cassandra` recipe to version, i.e.
  select the appriopriate YAML configuration file.
  Two versions are currently available
  * `1` (Default): Legacy YAML configuration file, compatible with
    Cassandra < 2.2.
  * `2`: Required for Cassandra > 2.2, use the YAML configuration file
  * introduced with
    DataDog/dd-agent@2df7566
* Enable `cassandra_aliasing` option to comply with
  DataDog/dd-agent#2035

More information:
DataDog/dd-agent#2142
sethrosenblum pushed a commit to DataDog/chef-datadog that referenced this pull request Apr 22, 2016
* Enhance `DatadogMonitor` resource with a `version` attribute.
* Use `version` attribute in `cassandra` recipe to version, i.e.
  select the appriopriate YAML configuration file.
  Two versions are currently available
  * `1` (Default): Legacy YAML configuration file, compatible with
    Cassandra < 2.2.
  * `2`: Required for Cassandra > 2.2, use the YAML configuration file
  * introduced with
    DataDog/dd-agent@2df7566
* Enable `cassandra_aliasing` option to comply with
  DataDog/dd-agent#2035

More information:
DataDog/dd-agent#2142
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants