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

Use cockroach workload in deployment tutorials #9834

Merged
merged 1 commit into from
Feb 25, 2021

Conversation

jseldess
Copy link
Contributor

Instead of the distinct workload binary.

Fixes #5359.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jseldess jseldess force-pushed the deployment-built-in-workloads branch from b9127bc to cd6e153 Compare February 18, 2021 22:57
@cockroach-teamcity
Copy link
Member

@cockroach-teamcity
Copy link
Member

Copy link
Contributor

@taroface taroface left a comment

Choose a reason for hiding this comment

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

LGTM with one wording suggestion! Thank you for this update.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @jseldess)


_includes/v20.2/prod-deployment/insecure-test-load-balancing.md, line 4 at r1 (raw file):

{{site.data.alerts.callout_info}}
Be sure that you have set up an inbound rule that allows traffic from the application to the load balancer. In this case, you will run the sample workload on one of your machines. The traffic source for your inbound rule should therefore be the **internal (private)** IP address of that machine.

The phrase "set up an inbound rule" seems specific to AWS. Perhaps this should be a more generic "configured your network to allow traffic...". (Unfortunately, the "Configure your network" anchor is different on AWS vs. the other platforms, so we can't link back to it here.)


_includes/v20.2/prod-deployment/secure-test-load-balancing.md, line 4 at r1 (raw file):

{{site.data.alerts.callout_info}}
Be sure that you have set up an inbound rule that allows traffic from the application to the load balancer. In this case, you will run the sample workload on one of your machines. The traffic source for your inbound rule should therefore be the **internal (private)** IP address of that machine.

Same comment as above.


_includes/v21.1/prod-deployment/insecure-test-load-balancing.md, line 4 at r1 (raw file):

{{site.data.alerts.callout_info}}
Be sure that you have set up an inbound rule that allows traffic from the application to the load balancer. In this case, you will run the sample workload on one of your machines. The traffic source for your inbound rule should therefore be the **internal (private)** IP address of that machine.

Same comment as above.


_includes/v21.1/prod-deployment/secure-test-load-balancing.md, line 4 at r1 (raw file):

{{site.data.alerts.callout_info}}
Be sure that you have set up an inbound rule that allows traffic from the application to the load balancer. In this case, you will run the sample workload on one of your machines. The traffic source for your inbound rule should therefore be the **internal (private)** IP address of that machine.

Same comment as above.

Instead of the distinct workload binary.

Fixes #5359.
@jseldess jseldess force-pushed the deployment-built-in-workloads branch from cd6e153 to c1adbf6 Compare February 25, 2021 18:25
Copy link
Contributor Author

@jseldess jseldess left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @taroface)


_includes/v20.2/prod-deployment/insecure-test-load-balancing.md, line 4 at r1 (raw file):

Previously, taroface (Ryan Kuo) wrote…

The phrase "set up an inbound rule" seems specific to AWS. Perhaps this should be a more generic "configured your network to allow traffic...". (Unfortunately, the "Configure your network" anchor is different on AWS vs. the other platforms, so we can't link back to it here.)

Done.


_includes/v20.2/prod-deployment/secure-test-load-balancing.md, line 4 at r1 (raw file):

Previously, taroface (Ryan Kuo) wrote…

Same comment as above.

Done.


_includes/v21.1/prod-deployment/insecure-test-load-balancing.md, line 4 at r1 (raw file):

Previously, taroface (Ryan Kuo) wrote…

Same comment as above.

Done.


_includes/v21.1/prod-deployment/secure-test-load-balancing.md, line 4 at r1 (raw file):

Previously, taroface (Ryan Kuo) wrote…

Same comment as above.

Done.

@cockroach-teamcity
Copy link
Member

@jseldess jseldess merged commit 64d938b into master Feb 25, 2021
@jseldess jseldess deleted the deployment-built-in-workloads branch February 25, 2021 18:32
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.

Update deployment tutorials to use built-in workloads
3 participants