Skip to content

Followup edits to the scaling and performance networking info #4177

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

Merged
merged 1 commit into from
Feb 22, 2018

Conversation

bfallonf
Copy link

As per #4171

Some minor rewording, but @jeremyeder I had a question. I'll put it inline.

Thanks for the work on this!

If you would like to plan for a larger environment, here are some example values to consider adding to the `[OSE3:vars]` section in your
Ansible inventory file:
To plan for a larger environment, the following are suggested values to consider
adding to the `[OSE3:vars]` section in your Ansible inventory file:
Copy link
Author

Choose a reason for hiding this comment

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

@jeremyeder did you intentionally erase the other values for the inventory file as well? Also, is this supposed to happen at install?

Copy link
Contributor

Choose a reason for hiding this comment

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

Which other values are you referring to @bfallonf ?

Copy link
Author

Choose a reason for hiding this comment

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

@ahardin-rh
Copy link
Contributor

@bfallonf @jeremyeder I see this is still a WIP. Once this merges, I can knock out the work for: https://trello.com/c/eI6qJFRU/407-docs-break-routing-and-network-performance-into-different-sections-3

@bfallonf
Copy link
Author

bfallonf commented May 4, 2017

Hmm forgot about this. I guess @jeremyeder let me know what you think about my question above and @ahardin-rh or I can get this merged.

@ahardin-rh
Copy link
Contributor

@jeremyeder bump

@@ -134,4 +135,7 @@ osm_cluster_network_cidr=10.128.0.0/10

This will allow for 8192 nodes, each with 510 usable IPs.

Please refer to the supportability limits in the {product-title} documentation for node/pod limits for the version of software you are installing.
Please refer to the supportability limits in the {product-title} documentation
Copy link
Contributor

Choose a reason for hiding this comment

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

s/Please refer/Refer

@ncbaratta ncbaratta added the peer-review-done Signifies that the peer review team has reviewed this PR label Nov 9, 2017
@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 18, 2018
@bfallonf
Copy link
Author

@jeremyeder We all forgot about this one and the bot reminded us. I'll send you an email in case you're not up on github notifications, but we can probably close this.

@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 22, 2018
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 22, 2018
@bfallonf bfallonf added this to the Next Release milestone Feb 22, 2018
@bfallonf
Copy link
Author

Received ack over email. Merging.

@bfallonf bfallonf merged commit 5da471d into openshift:master Feb 22, 2018
@bfallonf
Copy link
Author

/cherrypick enterprise-3.6

@openshift-cherrypick-robot

@bfallonf: new pull request created: #7891

In response to this:

/cherrypick enterprise-3.6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bfallonf
Copy link
Author

/cherrypick enterprise-3.7

@openshift-cherrypick-robot

@bfallonf: new pull request created: #7892

In response to this:

/cherrypick enterprise-3.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bfallonf
Copy link
Author

/cherrypick enterprise-3.9

@openshift-cherrypick-robot

@bfallonf: new pull request created: #7893

In response to this:

/cherrypick enterprise-3.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bfallonf bfallonf deleted the followup_4171 branch February 22, 2018 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-3.6 branch/enterprise-3.7 branch/enterprise-3.9 peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants