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.

Sign up for GitHub

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

Jump to bottom

Add Traffic Ops API Testing Monitor Support #3924

Open
2 of 16 tasks
rob05c opened this issue Sep 16, 2019 · 1 comment
Open
2 of 16 tasks

Add Traffic Ops API Testing Monitor Support #3924

rob05c opened this issue Sep 16, 2019 · 1 comment
Labels
high impact impacts the basic function, deployment, or operation of a CDN low difficulty the estimated level of effort to resolve this issue is low tech debt rework due to choosing easy/limited solution tests related to tests and/or testing infrastructure Traffic Monitor related to Traffic Monitor Traffic Ops related to Traffic Ops

Comments

@rob05c
Copy link
Member

rob05c commented Sep 16, 2019 •
edited by mitchell852
Loading

Currently, the TO API Tests don't support Traffic Monitors. This makes it impossible to write integration tests for endpoints that call out to Monitors.

We need to add Monitor support, so we can test the Monitor endpoints.

This may be as simple as adding a few fixtures and configs, or it may be rather difficult. To begin with, someone needs to investigate or prototype it, to find out how much work it's going to be.

See also #3348

I'm submitting a ...

  • bug report
  • new feature / enhancement request
  • improvement request (usability, performance, tech debt, etc.)
  • other

Traffic Control components affected ...

  • CDN in a Box
  • Documentation
  • Grove
  • Traffic Control Client
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • unknown

Current behavior:

API Tests don't support Monitors.

Expected / new behavior:

API Tests support Monitors, and support tests that make requests to them.

Minimal reproduction of the problem with instructions:

Write an API Test that requests a Monitor endpoint.

Anything else:
@rob05c rob05c added tests related to tests and/or testing infrastructure tech debt rework due to choosing easy/limited solution labels Sep 16, 2019
This was referenced Sep 16, 2019
Closed
Merged
@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops Traffic Monitor related to Traffic Monitor labels Nov 18, 2019
@mitchell852 mitchell852 added high impact impacts the basic function, deployment, or operation of a CDN low difficulty the estimated level of effort to resolve this issue is low labels Jan 18, 2022
@mitchell852
Copy link
Member

mitchell852 commented Jan 18, 2022 •
edited
Loading

added high impact label as this blocks any testing for certain TO API endpoints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
high impact impacts the basic function, deployment, or operation of a CDN low difficulty the estimated level of effort to resolve this issue is low tech debt rework due to choosing easy/limited solution tests related to tests and/or testing infrastructure Traffic Monitor related to Traffic Monitor Traffic Ops related to Traffic Ops
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mitchell852 @rob05c