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

Language Clean-up #2089

Merged
merged 9 commits into from
Jan 26, 2024
Merged

Language Clean-up #2089

merged 9 commits into from
Jan 26, 2024

Conversation

karl-cardenas-coding
Copy link
Contributor

@karl-cardenas-coding karl-cardenas-coding commented Jan 22, 2024

Describe the Change

This PR updates the documentation code base by removing all instances of master/slave. There is one instance that we cannot remove yet and that is due to PEM-4430

❗ Dependent on PR 34

Review Changes

💻 Preview URL

🎫 DOC-1033

@karl-cardenas-coding karl-cardenas-coding added backport-version-4-0 Backport change to version 4.0 auto-backport Enable backport backport-version-3-4 Backport change to version 3.4 backport-version-4-1 Backport change to version 4.1 backport-version-4-2 Backport change to version 4.2 labels Jan 22, 2024
Copy link

netlify bot commented Jan 22, 2024

Deploy Preview for docs-spectrocloud ready!

Name Link
🔨 Latest commit 8ee2f07
🔍 Latest deploy log https://app.netlify.com/sites/docs-spectrocloud/deploys/65b305d8c470fd0008c4af5d
😎 Deploy Preview https://deploy-preview-2089--docs-spectrocloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@karl-cardenas-coding karl-cardenas-coding added the documentation Improvements or additions to documentation label Jan 22, 2024
@karl-cardenas-coding karl-cardenas-coding marked this pull request as ready for review January 25, 2024 17:44
@karl-cardenas-coding karl-cardenas-coding requested a review from a team as a code owner January 25, 2024 17:44
- all - pods are scheduled on both master and worker nodes
Kubernetes provides a way to schedule the pods on the control plane and worker nodes. Pack Constraints framework must
know where the pods are scheduled because the resource validation validates only the control plane machine pool when the
pods are scheduled on control plane nodes. Similarily, if the pods are scheduled on worker nodes, then only the worker
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Similarily'?

[instance cost type](architecture.md#spot-instances), disk size, and the number of nodes. Click on **Next** after
you have completed configuring the node pool. The minimum number of CPUs and amount of memory depend on your cluster
profile, but in general you need at least 4 CPUs and 4 GB of memory both in the master pool and across all worker
pools.
profile, but in general you need at least 4 CPUs and 4 GB of memory both in the control plane pool and across all
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'CPUs'?

least 4 CPUs and 4 GB of memory both in the master pool and across all worker pools.
number of nodes. Configure the control plane and worker node pools. A control plane and a worker node pool are
configured by default. The minimum number of CPUs and amount of memory depend on your cluster profile, but in
general you need at least 4 CPUs and 4 GB of memory both in the control plane pool and across all worker pools.
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'CPUs'?

@@ -68,8 +68,8 @@ Ensure the following requirements are met before you attempt to deploy a cluster

11. The Node configuration page is where you can specify the availability zones (AZ), instance types, disk size, and the
number of nodes. Configure the worker node pool. The minimum number of CPUs and amount of memory depend on your
cluster profile, but in general you need at least 4 CPUs and 4 GB of memory both in the master pool and across all
worker pools.
cluster profile, but in general you need at least 4 CPUs and 4 GB of memory both in the control plane pool and
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'CPUs'?

| WORKER POOL | 3 | AWS t2.large($0.0992/hour) | 60GB - gp2($0.00014/GB/hour) |
| MACHINE POOL | SIZE | INSTANCE TYPE WITH COST | ROOT DISK WITH COST |
| ------------- | ---- | --------------------------- | ---------------------------- |
| Control Plane | 3 | AWS t2.medium($0.0496/hour) | 60GB - gp2($0.00014/GB/hour) |
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Google.Units] Put a nonbreaking space between the number and the unit in '60GB'.

| MACHINE POOL | SIZE | INSTANCE TYPE WITH COST | ROOT DISK WITH COST |
| ------------- | ---- | --------------------------- | ---------------------------- |
| Control Plane | 3 | AWS t2.medium($0.0496/hour) | 60GB - gp2($0.00014/GB/hour) |
| Worker Pool | 3 | AWS t2.large($0.0992/hour) | 60GB - gp2($0.00014/GB/hour) |
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Google.Units] Put a nonbreaking space between the number and the unit in '60GB'.

tool called KubeBench from Aqua Security to perform this scan. Scans are run against master and worker nodes of the
Kubernetes cluster, and a combined report is made available on the UI. Users can filter the report to view only the
master or worker results if required.
tool called KubeBench from Aqua Security to perform this scan. Scans are run against control plane and worker nodes of
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[spectrocloud.ableism] Avoid using ableism terms. Use 'issue' instead of 'run'.

@@ -65,14 +65,14 @@ available to the users to apply to their existing clusters at a time convenient
### Kubernetes

Kubernetes components and configuration are hardened in accordance with the Kubernetes CIS Benchmark. Palette executes
Kubebench, a CIS Benchmark scanner by Aqua Security, for every Kubernetes pack to ensure the master and worker nodes are
configured securely.
Kubebench, a CIS Benchmark scanner by Aqua Security, for every Kubernetes pack to ensure the control plane and worker
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Kubebench'?

@karl-cardenas-coding karl-cardenas-coding merged commit dbf675d into master Jan 26, 2024
9 checks passed
@karl-cardenas-coding karl-cardenas-coding deleted the word-removal branch January 26, 2024 01:25
@vault-token-factory-spectrocloud
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
version-4-0 Backport failed because of merge conflicts
version-3-4 Backport failed because of merge conflicts

You might need to backport the following PRs to version-3-4:
- Refactor/restructure cluster & app profiles sections (#1551)
- docs: refactor for self-hosted Palette DOC-465 (#1597)
- docs: updated URLs to use file path context
version-4-1 Backport failed because of merge conflicts
version-4-2

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

backport --pr 2089

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@vault-token-factory-spectrocloud
Copy link
Contributor

🎉 This PR is included in version 4.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@vault-token-factory-spectrocloud
Copy link
Contributor

🎉 This PR is included in version 4.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Enable backport backport-version-3-4 Backport change to version 3.4 backport-version-4-0 Backport change to version 4.0 backport-version-4-1 Backport change to version 4.1 backport-version-4-2 Backport change to version 4.2 documentation Improvements or additions to documentation released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants