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

Report generation fails while extracting server metrics #136

Closed
alexafshar opened this issue Mar 19, 2024 · 1 comment
Closed

Report generation fails while extracting server metrics #136

alexafshar opened this issue Mar 19, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@alexafshar
Copy link
Collaborator

Due to garbled characters that can end up as machineId string over time, parsing of API payload can fail. This is similar to #109 accept the failure happens even further up the chain. Need to re-examine and broaden the exception handler scope so to catch all case.

stack trace

2023-10-26 16:25:38,767 [INFO] xxxxx.xxxx.xxxxx.com - Extracting Servers 2023-10-26 16:25:38,826 [ERROR] Traceback (most recent call last): File "/backend/core/Engine.py", line 205, in run await self.process() File "/backend/core/Engine.py", line 340, in process await jobStep.extract(self.controllerData) File "/backend/extractionSteps/general/ControllerLevelDetails.py", line 38, in extract hostInfo["servers"] = (await controller.getServers()).data File "/backend/api/appd/AppDService.py", line 755, in getServers machineIds = [serverKey["machineId"] for serverKey in serverKeys.data["machineKeys"]] TypeError: list indices must be integers or slices, not str

2023-10-26 16:25:59,951 [INFO] Running from source/docker, using / as root directory
2023-10-26 16:25:59,952 [INFO]

@alexafshar alexafshar added the bug Something isn't working label Mar 19, 2024
@alexafshar alexafshar self-assigned this Mar 19, 2024
@alexafshar alexafshar added this to In progress in config-assessment-tool Mar 19, 2024
@alexafshar alexafshar moved this from In progress to In Master in config-assessment-tool Apr 26, 2024
@alexafshar
Copy link
Collaborator Author

fix released with v1.6.1

@alexafshar alexafshar moved this from In Master to Relased in config-assessment-tool May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

1 participant