Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Vibhu-gslab <109593615+Vibhu-gslab@users.noreply.github.com>
  • Loading branch information
MaheshGSLAB and Vibhu-gslab committed Aug 29, 2024
1 parent cf7e270 commit 8a2f54b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,4 @@ servers:
sflow: true

expected_error_message: >-
Unable to configure `sflow` because it is enabled on interface but `sflow_settings.destinations` is not configured.
`sflow_settings.destinations` is required to configure `sflow`.
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ def sflow(self) -> dict | None:

destinations = get(self._hostvars, "sflow_settings.destinations")
if destinations is None:
msg = "Unable to configure `sflow` because it is enabled on interface but `sflow_settings.destinations` is not configured."
msg = "`sflow_settings.destinations` is required to configure `sflow`."
raise AristaAvdMissingVariableError(msg)

sflow_settings_vrfs = get(self._hostvars, "sflow_settings.vrfs", default=[])
Expand Down

0 comments on commit 8a2f54b

Please sign in to comment.