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

Conversation

@serDrem
Copy link
Contributor

@serDrem serDrem commented Jun 20, 2017

Enables TM to poll health information over IPV6, and optionally combine it with IPV4 information for traffic router.

fixes #2735

@alficles
Copy link
Contributor

alficles commented Jul 6, 2017

This is pretty cool. It looks like there are two separate PRs in here, and the second one could really use a squash. The astats changes can stand on their own, so they should probably get their own PR. And the IPv6 for TM should be squashed. There are some confusing reverts of inadvertent changes that probably shouldn't go in, even as intermediate changesets.

It'll be a lot easier to review it that way, I think.

@asfgit
Copy link
Contributor

asfgit commented Aug 6, 2017

Can one of the admins verify this patch?

@dneuman64
Copy link
Contributor

+1 to @alficles comments. @serDrem can you please make this two PRs? One for astats and one for TM?

@dneuman64
Copy link
Contributor

This should also be linked to this issue: https://issues.apache.org/jira/browse/TC-230

@asfgit
Copy link
Contributor

asfgit commented Aug 14, 2017

Can one of the admins verify this patch?

@asfgit
Copy link
Contributor

asfgit commented Aug 15, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/18/

@serDrem
Copy link
Contributor Author

serDrem commented Aug 21, 2018

been absorbed by another PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Traffic Monitor related to Traffic Monitor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Traffic Monitor Polling over ipv6

6 participants