[GitHub] [trafficcontrol] riupie commented on issue #4111: DS domain can't accessed. DNS wont update.

riupie commented on issue #4111: DS domain can't accessed. DNS wont update. URL: https://github.com/apache/trafficcontrol/issues/4111#issuecomment-554194422 Ok, thank you! This is an automated message from the Apache Git

[GitHub] [trafficcontrol] riupie closed issue #4111: DS domain can't accessed. DNS wont update.

riupie closed issue #4111: DS domain can't accessed. DNS wont update. URL: https://github.com/apache/trafficcontrol/issues/4111 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [trafficcontrol] jhg03a commented on issue #3585: Initial contribution of Ansible-based lab deployment efforts

jhg03a commented on issue #3585: Initial contribution of Ansible-based lab deployment efforts URL: https://github.com/apache/trafficcontrol/pull/3585#issuecomment-554184601 FYI, do not use Ansible 2.9.0 with the dataset loader role. There is a regression that was

[GitHub] [trafficcontrol] ocket8888 commented on issue #4111: DS domain can't accessed. DNS wont update.

ocket commented on issue #4111: DS domain can't accessed. DNS wont update. URL: https://github.com/apache/trafficcontrol/issues/4111#issuecomment-554173683 yeah, you have to do the server assignment as well. And you have to queue server updates or the caches won't actually serve the

[GitHub] [trafficcontrol] riupie commented on issue #4111: DS domain can't accessed. DNS wont update.

riupie commented on issue #4111: DS domain can't accessed. DNS wont update. URL: https://github.com/apache/trafficcontrol/issues/4111#issuecomment-554145700 > So I think maybe I see what's going on here: you have it set to serve "HTTP and HTTPS" protocols, but did you generate SSL Keys for

[GitHub] [trafficcontrol] mitchell852 commented on issue #2017: TP, TO API should allow for 'delete' of users

mitchell852 commented on issue #2017: TP, TO API should allow for 'delete' of users URL: https://github.com/apache/trafficcontrol/issues/2017#issuecomment-554130858 ^^ I guess the "delete" was never addressed in that PR #2834

[GitHub] [trafficcontrol] ocket8888 commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error

ocket commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error URL: https://github.com/apache/trafficcontrol/issues/4115#issuecomment-554125754 :+1: got it

[GitHub] [trafficcontrol] ocket8888 closed issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error

ocket closed issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error URL: https://github.com/apache/trafficcontrol/issues/4115 This is an automated message from the

[GitHub] [trafficcontrol] ZugNZwang commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error

ZugNZwang commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error URL: https://github.com/apache/trafficcontrol/issues/4115#issuecomment-554123109 Sorry. I just did some more testing. This only happens when using a

[GitHub] [trafficcontrol] ocket8888 commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error

ocket commented on issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error URL: https://github.com/apache/trafficcontrol/issues/4115#issuecomment-554121574 I can't replicate this. I have no Server Capabilities created in my

[GitHub] [trafficcontrol] ocket8888 commented on issue #3312: Disallow underscore in xmlId

ocket commented on issue #3312: Disallow underscore in xmlId URL: https://github.com/apache/trafficcontrol/issues/3312#issuecomment-554116640 That's a good point but I'm not sure that the definition of a DNS label actually has any overlap with forbidden characters in filenames.

[GitHub] [trafficcontrol] ocket8888 commented on issue #4007: Server Capability get by name - returns blank response array when the requested item doesn't exist

ocket commented on issue #4007: Server Capability get by name - returns blank response array when the requested item doesn't exist URL: https://github.com/apache/trafficcontrol/issues/4007#issuecomment-554114957 It's not a bug but it can stay open as an enhancement request

[GitHub] [trafficcontrol] ocket8888 commented on issue #3761: Improvements to Server-to-Delivery Service assignments

ocket commented on issue #3761: Improvements to Server-to-Delivery Service assignments URL: https://github.com/apache/trafficcontrol/pull/3761#issuecomment-554113397 Well, this does still fix the reported problem with "cloning DS assignments" in Traffic Portal, so instead I've opened

[GitHub] [trafficcontrol] ocket8888 opened a new issue #4116: Delivery Services can be assigned servers from a different CDN

ocket opened a new issue #4116: Delivery Services can be assigned servers from a different CDN URL: https://github.com/apache/trafficcontrol/issues/4116 ## I'm submitting a ... - bug report ## Traffic Control components affected ... - Traffic Ops ## Current

[GitHub] [trafficcontrol] ocket8888 edited a comment on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

ocket edited a comment on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-554105562 This PR changes Perl files that have been rewritten, and modifies the old, dead, TO UI code.

[GitHub] [trafficcontrol] ocket8888 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

ocket commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-554105562 This PR changes Perl files that have been rewritten, and modifies the old, dead, TO UI code. If you

[GitHub] [trafficcontrol] ocket8888 closed pull request #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

ocket closed pull request #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407 This is an automated message from the Apache Git

[GitHub] [trafficcontrol] ZugNZwang opened a new issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error

ZugNZwang opened a new issue #4115: TO: Deleting a Delivery Service Required Capability that does not exist returns incorrect error URL: https://github.com/apache/trafficcontrol/issues/4115 ## I'm submitting a ... - [X] bug report - [ ] new feature /

[GitHub] [trafficcontrol] rob05c commented on issue #3312: Disallow underscore in xmlId

rob05c commented on issue #3312: Disallow underscore in xmlId URL: https://github.com/apache/trafficcontrol/issues/3312#issuecomment-554080819 We should also disallow characters not allowed in POSIX filenames (possibly also Windows filenames). Because Cache Configs put the xml_id in some

[GitHub] [trafficcontrol] mhoppa commented on issue #3761: Improvements to Server-to-Delivery Service assignments

mhoppa commented on issue #3761: Improvements to Server-to-Delivery Service assignments URL: https://github.com/apache/trafficcontrol/pull/3761#issuecomment-554065993 thats fine as long as we dont close the issue then this looks good to me 

[GitHub] [trafficcontrol] ocket8888 commented on issue #3761: Improvements to Server-to-Delivery Service assignments

ocket commented on issue #3761: Improvements to Server-to-Delivery Service assignments URL: https://github.com/apache/trafficcontrol/pull/3761#issuecomment-554065338 I think I'd like to leave those others for a different PR. They don't look to hard themselves, but the handlers could

[GitHub] [trafficcontrol] mitchell852 commented on issue #4007: Server Capability get by name - returns blank response array when the requested item doesn't exist

mitchell852 commented on issue #4007: Server Capability get by name - returns blank response array when the requested item doesn't exist URL: https://github.com/apache/trafficcontrol/issues/4007#issuecomment-554064996 IMO this should simply be closed as this is expected behavior.

[GitHub] [trafficcontrol] mitchell852 closed pull request #3035: Added a new form field, "Description" to Servers db/configuration page

mitchell852 closed pull request #3035: Added a new form field, "Description" to Servers db/configuration page URL: https://github.com/apache/trafficcontrol/pull/3035 This is an automated message from the Apache Git Service.

[GitHub] [trafficcontrol] mitchell852 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

mitchell852 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-554054929 related to #3537 This is an

[GitHub] [trafficcontrol] mitchell852 closed pull request #1627: enhance traffic monitor to support configurable polling over ipv6

mitchell852 closed pull request #1627: enhance traffic monitor to support configurable polling over ipv6 URL: https://github.com/apache/trafficcontrol/pull/1627 This is an automated message from the Apache Git Service. To

[GitHub] [trafficcontrol] mitchell852 commented on issue #1627: enhance traffic monitor to support configurable polling over ipv6

mitchell852 commented on issue #1627: enhance traffic monitor to support configurable polling over ipv6 URL: https://github.com/apache/trafficcontrol/pull/1627#issuecomment-554053942 looks like #4012 will take the place of this PR. closing.

[GitHub] [trafficcontrol] ZugNZwang commented on issue #3312: Disallow underscore in xmlId

ZugNZwang commented on issue #3312: Disallow underscore in xmlId URL: https://github.com/apache/trafficcontrol/issues/3312#issuecomment-554027711 When making a `POST` request to `https://{{TO_BASE_URL}}/api/{{api_version}}/deliveryservices` with an `xmlId` key containing any of these:

Build failed in Jenkins: trafficcontrol-PR #4782

See Changes: [rob05c] TM: Report time for full body download for _astats (#3914) [hank.beatty] added ability to select TLS version in TR (#3961) [rob] Add DS Snapshot endpoint [rob] Add DS Snapshot table

[GitHub] [trafficcontrol] asf-ci commented on issue #3175: WIP Add Delivery Service Snapshots

asf-ci commented on issue #3175: WIP Add Delivery Service Snapshots URL: https://github.com/apache/trafficcontrol/pull/3175#issuecomment-554027158 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4782/

[GitHub] [trafficcontrol] ocket8888 commented on issue #4114: Rewrite current stats from Perl to Golang

ocket commented on issue #4114: Rewrite current stats from Perl to Golang URL: https://github.com/apache/trafficcontrol/pull/4114#issuecomment-554022564 So then is a work in progress or ready for review? This is an

[GitHub] [trafficcontrol] ocket8888 commented on issue #4111: DS domain can't accessed. DNS wont update.

ocket commented on issue #4111: DS domain can't accessed. DNS wont update. URL: https://github.com/apache/trafficcontrol/issues/4111#issuecomment-554014737 So I think maybe I see what's going on here: you have it set to serve "HTTP and HTTPS" protocols, but did you generate SSL Keys

[GitHub] [trafficcontrol] ocket8888 merged pull request #4108: adds deep coverage zone routed percentage to TP's routing widget

ocket merged pull request #4108: adds deep coverage zone routed percentage to TP's routing widget URL: https://github.com/apache/trafficcontrol/pull/4108 This is an automated message from the Apache Git Service. To

[GitHub] [trafficcontrol] mhoppa edited a comment on issue #4114: Rewrite current stats from Perl to Golang

mhoppa edited a comment on issue #4114: Rewrite current stats from Perl to Golang URL: https://github.com/apache/trafficcontrol/pull/4114#issuecomment-553974687 Still need to look into why Perl implementation did

[GitHub] [trafficcontrol] asf-ci commented on issue #3862: Add Traffic Ops /deliveryservices, /servers If-Modified-Since support

asf-ci commented on issue #3862: Add Traffic Ops /deliveryservices, /servers If-Modified-Since support URL: https://github.com/apache/trafficcontrol/pull/3862#issuecomment-553981039 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4781

See Changes: [rob] Add TO DS, Server If-Modified-Since support -- [...truncated 2.64 MB...] traffic_portal_build_1 | | | +-- json-schema-traverse@0.4.1

[GitHub] [trafficcontrol] asf-ci commented on issue #4114: Rewrite current stats from Perl to Golang

asf-ci commented on issue #4114: Rewrite current stats from Perl to Golang URL: https://github.com/apache/trafficcontrol/pull/4114#issuecomment-553976835 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4780/

Build failed in Jenkins: trafficcontrol-PR #4780

See Changes: [rob05c] TM: Report time for full body download for _astats (#3914) [hank.beatty] added ability to select TLS version in TR (#3961) [Michael_Hoppal] Rewrite current stats from Perl to Golang

[GitHub] [trafficcontrol] mhoppa commented on issue #4114: Rewrite current stats from Perl to Golang

mhoppa commented on issue #4114: Rewrite current stats from Perl to Golang URL: https://github.com/apache/trafficcontrol/pull/4114#issuecomment-553974687 Still need to look into why Perl implementation did

[GitHub] [trafficcontrol] mhoppa opened a new pull request #4114: Rewrite current stats from Perl to Golang

mhoppa opened a new pull request #4114: Rewrite current stats from Perl to Golang URL: https://github.com/apache/trafficcontrol/pull/4114 ## What does this PR (Pull Request) do? - [x] This PR fixes #3853 This PR rewrites current_stats from Perl to Golang and adds API