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

error variable measurement #194

Closed
wants to merge 1 commit into from
Closed

error variable measurement #194

wants to merge 1 commit into from

Conversation

BatRaph
Copy link
Contributor

@BatRaph BatRaph commented Jan 20, 2022

When a run slo-generator compute i got two errors
it seems code in dynatrace.py to get slo measurement is not good

i have replace
conf = slo_config['backend']
measurement = conf['measurement']
by
measurement = slo_config['spec']['service_level_indicator']

(same as in datadog.py)

ERRORS

conf = slo_config['backend']
KeyError: 'backend'

then

Traceback (most recent call last):
File "/home/laurendeau/project/slo-generator-project/venv/lib/python3.8/site-packages/slo_generator/report.py", line 216, in run_backend
data = method(self.timestamp, self.window, config)
File "/home/laurendeau/project/slo-generator-project/venv/lib/python3.8/site-packages/slo_generator/backends/dynatrace.py", line 58, in query_sli
measurement = conf['measurement']

@ocervell
Copy link
Collaborator

Thanks for fixing this ! Could you please sign the CLA at https://cla.developers.google.com/ so that I can merge your PR ? Thank you.

@BatRaph
Copy link
Contributor Author

BatRaph commented Jan 25, 2022

Close to reopen it after

@BatRaph BatRaph closed this Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants