Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Mark topology-based delivery services as available in Traffic Monitor#4922

Merged
ocket8888 merged 2 commits intoapache:masterfrom
zrhoffman:tm-populate-ds-servers
Aug 4, 2020
Merged

Mark topology-based delivery services as available in Traffic Monitor#4922
ocket8888 merged 2 commits intoapache:masterfrom
zrhoffman:tm-populate-ds-servers

Conversation

@zrhoffman
Copy link
Member

What does this PR (Pull Request) do?

Which Traffic Control components are affected by this PR?

  • Documentation
  • Traffic Monitor

What is the best way to verify this PR?

  1. Make topology-based delivery service
  2. Perform CDN snapshot
  3. Look at Traffic Monitor logs. Lines to look for:
trafficmonitor_1            | 1595954995.290 host="demo1", type=Delivery Service, available=true, msg="available caches"
trafficmonitor_1            | 1595954995.290 host="demo1", type=DELIVERYSERVICE, available=true, msg="REPORTED - available"

If this is a bug fix, what versions of Traffic Control are affected?

The following criteria are ALL met by this PR

  • This PR includes tests
  • This PR includes documentation
  • This PR includes an update to CHANGELOG.md
  • This PR includes any and all required license headers
  • This PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

@mitchell852 mitchell852 added Traffic Monitor related to Traffic Monitor bug something isn't working as intended labels Aug 3, 2020
Copy link
Member

@shamrickus shamrickus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

  • Docs build
  • TM tests pass
  • Topology based delivery services are correctly reported in TM

@ocket8888 ocket8888 merged commit 732c3f8 into apache:master Aug 4, 2020
@zrhoffman zrhoffman added this to the Flexible Topologies milestone Sep 18, 2020
@zrhoffman zrhoffman removed this from the Flexible Topologies milestone Oct 16, 2020
@zrhoffman zrhoffman deleted the tm-populate-ds-servers branch October 18, 2020 00:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended Traffic Monitor related to Traffic Monitor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Traffic Monitor marks topology-based delivery services as unavailable

4 participants