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

Docs for data plane registration #1332

Merged
merged 2 commits into from
May 28, 2024
Merged

Conversation

Miles-Garnsey
Copy link
Member

What this PR does:

Document data plane registration using k8ssandra-client.

Which issue(s) this PR fixes:
Fixes #1327

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

@Miles-Garnsey Miles-Garnsey requested a review from a team as a code owner May 28, 2024 02:42
@Miles-Garnsey Miles-Garnsey force-pushed the docs/data-plane-registration branch from 9ce6375 to 853d1be Compare May 28, 2024 03:44
@Miles-Garnsey Miles-Garnsey force-pushed the docs/data-plane-registration branch from 853d1be to 451e602 Compare May 28, 2024 03:46
README.md Outdated Show resolved Hide resolved
docs/content/en/install/local/_index.md Show resolved Hide resolved
docs/content/en/install/local/multi-cluster-helm/_index.md Outdated Show resolved Hide resolved
@@ -192,16 +191,6 @@ Here is a summary of what the script does:
Create a `ClientConfig` in the `kind-k8ssandra-0` cluster using the service account
token and CA cert from `kind-k8ssandra-1`:
Copy link
Contributor

Choose a reason for hiding this comment

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

Todo: I think lines 182 to 192 should be removed. Right?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think so actually. k8ssandra-client is doing exactly the same thing, and I personally wouldn't run it unless I had some idea of what resources it was going to manipulate.

We probably shouldn't call it a script though, maybe line 182 does need to change.

docs/content/en/reference/multi-cluster/_index.md Outdated Show resolved Hide resolved
docs/content/en/tasks/data-plane-registration/_index.md Outdated Show resolved Hide resolved
docs/content/en/tasks/data-plane-registration/_index.md Outdated Show resolved Hide resolved
docs/content/en/tasks/data-plane-registration/_index.md Outdated Show resolved Hide resolved
Co-authored-by: Alexander Dejanovski <[email protected]>
@Miles-Garnsey Miles-Garnsey force-pushed the docs/data-plane-registration branch from 89025e6 to 1f21c13 Compare May 28, 2024 06:49
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@adejanovski adejanovski merged commit da96a4f into main May 28, 2024
10 checks passed
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.

Document k8ssandra-client data plane registration usage
2 participants