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

chore: Fix cassandra deploy scripts #995

Merged
merged 4 commits into from
Nov 14, 2023

Conversation

michaelkro
Copy link
Contributor

@michaelkro michaelkro commented Nov 9, 2023

Deployer scripts

  • Enable JMX authentication and add user with credentials. NOTE: Try as I might, I could not figure out how to escape the ! that is usually included in our standard test credentials.
    user: newrelic
    pw: Virtuoso4all
    
  • AL2 quirks
    • When installing cassandra with yum, configuration files are located at /etc/cassandra/conf/, instead of the default /etc/cassandra

Testing

NOTES:

  • The current rhel recipe is broken. You will need to reference the fixed recipe in this branch when testing.
  • To fully validate that the CLI install was successful, ensure that you see a new cassandra entity show up in the entity explorer.
  1. If you run demo-deployer in a docker container with open-install-library as a mounted volume, you can reference local deploy scripts in the deployer config (e.g. apache-cassandra-debian10.json) by replacing "source_repository" with "local_source_path". For example:
    {
      ...
      "local_source_path": "/mnt/deployer/open-install-library",
      "deploy_script_path": "/test/deploy/linux/cassandra/install/debian/roles",
      ...
    }
    
  2. Cassandra install via CLI using the creds above should succeed
  3. Cassandra install via CLI should fail when using incorrect creds

@michaelkro michaelkro marked this pull request as draft November 10, 2023 15:59
@michaelkro michaelkro force-pushed the mro/NR-178908/fix-cassandra-test-def branch 3 times, most recently from f4dd7df to 5923109 Compare November 14, 2023 17:49
@michaelkro michaelkro force-pushed the mro/NR-178908/fix-cassandra-test-def branch from 5923109 to 243378c Compare November 14, 2023 21:09
@michaelkro michaelkro marked this pull request as ready for review November 14, 2023 21:10
@michaelkro michaelkro changed the title chore: Fix cassandra resources chore: Fix cassandra deploy scripts Nov 14, 2023
@michaelkro michaelkro force-pushed the mro/NR-178908/fix-cassandra-test-def branch from 708cb30 to eaffdf1 Compare November 14, 2023 21:46
@michaelkro michaelkro force-pushed the mro/NR-178908/fix-cassandra-test-def branch from 2da5667 to aee609e Compare November 14, 2023 23:30
@michaelkro michaelkro merged commit e4df5b7 into main Nov 14, 2023
27 of 44 checks passed
@michaelkro michaelkro deleted the mro/NR-178908/fix-cassandra-test-def branch November 14, 2023 23:54
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