Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/develop'
Browse files Browse the repository at this point in the history
  • Loading branch information
ddutt committed Jun 21, 2022
2 parents 7c010ed + cf74d48 commit 46eec6c
Show file tree
Hide file tree
Showing 3 changed files with 99 additions and 0 deletions.
75 changes: 75 additions & 0 deletions docs/rate-limiting-AAA.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,75 @@
# Rate Limiting AAA Server Requests

Many AAA servers (such as TACACS, Radius) cannot handle the rate at which SuzieQ can issue requests. This is especially true in larger installations, those with older AAA servers, or when command authorization is used in addition to authentication. The most common symptom of this problem is the authentication failed error message. To avoid user lockout, we stopped retrying on authentication failures. To fix this, we added three new parameters to the poller configuration. A consequence of throttling is of course that it takes longer to collect the data.

* **max-cmd-pipeline**: This is an integer value that ensures that no more than this number of requests are sent to a device in a second. Thus a value of 9 implies that we never have more than 9 outstanding commands or logins. If you use distributed pollers, you need to ensure that this number is a multiple of the number of pollers. Thus with a value of 9, you can use either 1 or 3 pollers. With 8, you can use 1, 2, or 4 pollers and so on. This is **specified in the suzieq-cfg.yml** file. The default is 0 i.e. no limits.

* **per-cmd-auth**: This is a boolean to specify whether need to throttle logins as well as commands sent to a device. This is required in installations where commands are authorized before execution. True means use it for commands as well as logins. This is specified in the devices section of the poller inventory file. The default is False.

* **retries-on-auth-fail**: Some older AAA servers fail even at low rates. In certain installations, a maximum of 3 authentication failures are tolerated before the user account is locked, and in some installations it can be anything more than a single failure. This parameter now enables us to support both types of installations. This is specified in the devices section of the poller inventory file. The default is 0, i.e. we never retry on authentication failure.

Here's a sample suzieq-cfg.yml file with the max-cmd-pipeline parameter (see the poller section).
```
data-directory: tests/data/parquet
coalescer:
period: 1h
archive-directory:
logging-level: DEBUG
rest:
# Uncomment these lines if you're using your own files for the REST server
# The certificates listed below are provided purely to get started, In any
# secure deployment, these must be generated specifically for the site and
# these lines uncommented and containing the location of the site-specific file.
# rest_certfile: /suzieq/cert.pem
# rest_keyfile: /suzieq/key.pem
#
API_KEY: 496157e6e869ef7f3d6ecb24a6f6d847b224ee4f
logging-level: WARNING
address: 127.0.0.1
port: 8000
# no-https: True
log-stdout: True
# rest-certfile: /secrets/cert.pem
# rest-keyfile: /secrets/key.pem
# logfile: /tmp/sq-rest-server.log
# log-stdout: True
poller:
connect-timeout: 60
period: 60
logsize: 10000000
logging-level: WARNING
log-stdout: True
max-cmd-pipeline: 4
ux:
engine: pandas
analyzer:
timezone: America/Los_Angeles
```

Here's a sample inventory file with per-cmd-auth and retries-on-auth-fail set:
```
---
sources:
- name: ans
type: ansible
path: /tmp/ansinv
devices:
- name: default
per-cmd-auth: False
retries-on-auth-fail: 2
ignore-known-hosts: true
auths:
- name: default
username: vagrant
password: vagrant
namespaces:
- name: demo
source: ans
device: default
```
23 changes: 23 additions & 0 deletions docs/release-notes.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,28 @@
# Release Notes

## 0.18.0 (June 21, 2022)

This 18th release of SuzieQ contains significant improvements to all the various NOS parsers, but especially IOSXE (more versions and more platforms supported now) and versions of NXOS older than 9.3.x. We've also added support for throttling the rate of requests SuzieQ poller issues to prevent overrunning AAA Servers. A more detailed list of features and critical bug fixes follows:

* **Improved support for IOSXE** (more versions, more platforms).
* **Support for all NXOS versions**, those before 9.3.x which had no ```json native``` support.
* **Improved parsing support for most NOS**.
* **Support for rate limiting requests to AAA Servers**. See the poller documentation for details.
* **Additional fields for device table added**. You can now get a distriubution of the common causes for device reboots.
* **Fixed bug in supporting multiple schema versions of a table**. This was only a bug in reading, writing was correct.
* **New command, namespace, to get information about namespaces**. This deprecates network show/summarize/unique/top. Endpoint tracker is still at network find.
* **Improved OSPF assert**, to handle non-p2p interfaces better.
* **Removed all commands that required use of Linux shell on NOS**. This is for NXOS and EOS.
* **Vastly improved Node discovery**. We wouldn't reattempt node discovery if it failed more than twice, improved log messages and more.
* **Improved Path Support**. We now handle recursive routes more effectively and so can be used in more than EVPN scenarios.
* **Vastly faster endpoint tracker**.
* Support accepting MAC in any format (such as 00-AA-BB-CC-DD-EE or 00AA:BBCC:DDEE etc.)
* Added tons of new tests including importantly to catch more parsing errors
* Updated libraries including those with security updates.

The one person to thank from the user community for this release is Andy Miller. Thanks Andy, this release wouldn't have been what it is, if it wasn't for you.


## 0.17.2 (Apr 28, 2022)

This is the second patch release of the 17th release. The main issues that are fixed in this release are:
Expand Down
1 change: 1 addition & 0 deletions mkdocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ nav:
- Gathering Data:
- Poller: poller.md
- Inventory File: inventory.md
- Rate Limiting AAA Server Requests: rate-limiting-AAA.md
- Tables: tables.md
- Transports: transports.md
- Polling From a Local Folder: simnode.md
Expand Down

0 comments on commit 46eec6c

Please sign in to comment.