Skip to content

AppWorld - Doc Fixes #539

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.

Sign up for GitHub

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 2 commits into from
Feb 26, 2025
Merged

AppWorld - Doc Fixes #539

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7216c39
class8: updates addressing feedback from appworld day 1
mtbChef Feb 26, 2025
89184b2
Merge pull request #538 from mtbChef/mvernik-class8-updates
mtbChef Feb 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions docs/class8/module3/module3.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,10 @@ Increase rlimit to 4096, by uncommenting line 4

* worker_rlimit_nofile 4096;

Increase worker_connections from 1024 to 4096

* worker_connections 4096; (line 11 or 12)

Publish the new configuration.

.. image:: /class8/images/nim-rlimit-4096.png
Expand Down
3 changes: 0 additions & 3 deletions docs/class8/module4/module4.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,6 @@ Host: http://10.1.1.9/

Advanced Options, Run time: 30s

.. image:: /class8/images/locus-500-50-30.png
:width: 200 px

.. note:: Where you do see the performance improvement in the Locust chart?

.. note:: Review NGINX Dashboard cache section. How much bandwidth was saved from going to upstream server? (For a hint, refer to the HINTS section at the bottom of this page.)
Expand Down
4 changes: 3 additions & 1 deletion docs/class8/module5/module5.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@ Review the following lines in the nginx.conf file. Uncommenting these directives
* `limit_conn_zone $binary_remote_addr zone=addr:10m;`
* `limit_conn addr 400;`

.. note:: Both connection and rate limits are advised as best-practices for any service running in production. These parameters can help defend against some Denial-of-Service attacks, as well as, prevent back-end application servers from being overwhelmed with rouge traffic.
Both connection and rate limits are advised as best-practices for any service running in production. These parameters can help defend against some Denial-of-Service attacks, as well as, prevent back-end application servers from being overwhelmed with rouge traffic.

.. note:: If you attempted to enable both connection and request limiting did you notice any errors when publishing the update? What could be a good solution to this? Ask for help if you need any assistance.

6. **Review upstream keepalive parameters**

Expand Down
Loading