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

Monitoring.json doesn't list the delivery services #3874

Closed
3 of 16 tasks
lbathina opened this issue Aug 15, 2019 · 2 comments
Closed
3 of 16 tasks

Monitoring.json doesn't list the delivery services #3874

lbathina opened this issue Aug 15, 2019 · 2 comments
Labels
bug something isn't working as intended crconfig-snapshot related in some way to Snapshots (CRConfig) low impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops

Comments

@lbathina
Copy link

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:

DS list returned in the response of monitoring.json is empty.

Expected / new behavior:

should return a json with DS list same as the DS list in /api/1.2/cdns//snapshot

Minimal reproduction of the problem with instructions:

request for /api/1.2/cdns//configs/monitoring.json
compare the DS field with DS field in the response of endpoint /api/1.2/cdns//snapshot

Anything else:
@ocket8888 ocket8888 added bug something isn't working as intended crconfig-snapshot related in some way to Snapshots (CRConfig) low impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops labels Aug 15, 2019
@mitchell852
Copy link
Member

mitchell852 commented Aug 21, 2019 •
edited
Loading

i'm a bit confused, does monitoring.json need the ds field or not? if it DOES need it, it seems like this would be more than a minor bug but if it doesn't need it, is it really a bug? maybe @rob05c can help answer that question.

@ocket8888
Copy link
Contributor

Fixed by #5184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug something isn't working as intended crconfig-snapshot related in some way to Snapshots (CRConfig) low impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mitchell852 @ocket8888 @lbathina