Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Sebastian Luna-Valero <[email protected]>
  • Loading branch information
enolfc and sebastian-luna-valero authored Dec 4, 2024
1 parent ce2c443 commit 551dfe8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ Getting VMs information [####################################] 100%
## fedcloud-sla-monitor
`fedcloud-sla-monitor` checks the configuration of sites supporting SLAs. It
compares the reported usage in the accoutnting portal and the information
compares the reported usage in the accounting portal and the information
retrieved from the cloud-info-provider and reports any deviations.
### Requirements
Expand Down
4 changes: 2 additions & 2 deletions fedcloud_vm_monitoring/sla_monitor_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,11 @@ def check_site_slas(site, site_slas, goc, acct, appdb):
accounted_vos = sla["vos"].intersection(acct.site_vos(site))
if accounted_vos:
click.echo(
f"[OK] SITE {site} has accouting info for SLA {sla_name} ({accounted_vos})"
f"[OK] SITE {site} has accounting info for SLA {sla_name} ({accounted_vos})"
)
else:
click.echo(
f"[ERR] SITE {site} has no accouting info for SLA {sla_name}"
f"[ERR] SITE {site} has no accounting info for SLA {sla_name}"
)
info_vos = sla["vos"].intersection(appdb_vos)
if info_vos:
Expand Down

0 comments on commit 551dfe8

Please sign in to comment.