[GitHub] nir-sopher commented on issue #1859: cr-config: calculating the hash-id if needed

2018-02-08 Thread GitBox
nir-sopher commented on issue #1859: cr-config: calculating the hash-id if needed URL: https://github.com/apache/incubator-trafficcontrol/pull/1859#issuecomment-364213554 As far as I tested, empty string is "false". Nir On Feb 8, 2018 7:40 PM, "Jeremy Mitchell"

[GitHub] mitchell852 commented on issue #1863: Traffic Portal - ORG servers should show what MSO DSs are using them

2018-02-08 Thread GitBox
mitchell852 commented on issue #1863: Traffic Portal - ORG servers should show what MSO DSs are using them URL: https://github.com/apache/incubator-trafficcontrol/issues/1863#issuecomment-364156563 and also, on the ds / server assignment page

[GitHub] dewrich commented on issue #1844: DeepCachingType -- "NEVER" is now default

2018-02-08 Thread GitBox
dewrich commented on issue #1844: DeepCachingType -- "NEVER" is now default URL: https://github.com/apache/incubator-trafficcontrol/pull/1844#issuecomment-364190181 @rob05c Gonna leave this one up to you to merge for 2.2

[GitHub] mitchell852 commented on a change in pull request #1859: cr-config: calculating the hash-id if needed

2018-02-08 Thread GitBox
mitchell852 commented on a change in pull request #1859: cr-config: calculating the hash-id if needed URL: https://github.com/apache/incubator-trafficcontrol/pull/1859#discussion_r167013045 ## File path: traffic_ops/app/lib/UI/Topology.pm ## @@ -264,7 +264,7 @@ sub

[GitHub] dneuman64 commented on a change in pull request #1825: updated docs to add some missing data

2018-02-08 Thread GitBox
dneuman64 commented on a change in pull request #1825: updated docs to add some missing data URL: https://github.com/apache/incubator-trafficcontrol/pull/1825#discussion_r167021831 ## File path: docs/source/admin/traffic_ops/using.rst ## @@ -293,135 +293,136 @@ The

[GitHub] dneuman64 commented on a change in pull request #1825: updated docs to add some missing data

2018-02-08 Thread GitBox
dneuman64 commented on a change in pull request #1825: updated docs to add some missing data URL: https://github.com/apache/incubator-trafficcontrol/pull/1825#discussion_r167021865 ## File path: docs/source/admin/traffic_ops/using.rst ## @@ -293,135 +293,136 @@ The

[GitHub] dneuman64 commented on issue #1825: updated docs to add some missing data

2018-02-08 Thread GitBox
dneuman64 commented on issue #1825: updated docs to add some missing data URL: https://github.com/apache/incubator-trafficcontrol/pull/1825#issuecomment-364200438 @limited can you take a look again? This is an automated

[GitHub] asfgit commented on issue #1865: Added CHANGELOG.md file with per-DS routing names upgrade notes

2018-02-08 Thread GitBox
asfgit commented on issue #1865: Added CHANGELOG.md file with per-DS routing names upgrade notes URL: https://github.com/apache/incubator-trafficcontrol/pull/1865#issuecomment-364259965 Refer to this link for build results (access rights to CI server needed):

[GitHub] rivasj opened a new pull request #1866: Added geolocation.default.override parameter to CRConfig to handle maxmind default locations

2018-02-08 Thread GitBox
rivasj opened a new pull request #1866: Added geolocation.default.override parameter to CRConfig to handle maxmind default locations URL: https://github.com/apache/incubator-trafficcontrol/pull/1866 Maintain a list of country code/coordinate pairs as ;, in CRConfig.json to use instead of

[GitHub] DylanVolz opened a new pull request #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-08 Thread GitBox
DylanVolz opened a new pull request #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864 This is an automated message from the Apache Git Service. To

[GitHub] asfgit commented on issue #1866: Added geolocation.default.override parameter to CRConfig to handle maxmind default locations

2018-02-08 Thread GitBox
asfgit commented on issue #1866: Added geolocation.default.override parameter to CRConfig to handle maxmind default locations URL: https://github.com/apache/incubator-trafficcontrol/pull/1866#issuecomment-364269428 Can one of the admins verify this patch?

[GitHub] rawlinp opened a new pull request #1865: Added CHANGELOG.md file with per-DS routing names upgrade notes

2018-02-08 Thread GitBox
rawlinp opened a new pull request #1865: Added CHANGELOG.md file with per-DS routing names upgrade notes URL: https://github.com/apache/incubator-trafficcontrol/pull/1865 This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-08 Thread GitBox
asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864#issuecomment-364255176 Refer to this link for build results (access rights to CI server needed):

[GitHub] DylanVolz opened a new pull request #1861: implement servers CRUD endpoints in go

2018-02-07 Thread GitBox
DylanVolz opened a new pull request #1861: implement servers CRUD endpoints in go URL: https://github.com/apache/incubator-trafficcontrol/pull/1861 fixes #1620 should be (rebased and) merged after #1860 This is an

[GitHub] asfgit commented on issue #1861: implement servers CRUD endpoints in go

2018-02-07 Thread GitBox
asfgit commented on issue #1861: implement servers CRUD endpoints in go URL: https://github.com/apache/incubator-trafficcontrol/pull/1861#issuecomment-363984348 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 commented on issue #1855: SSL cert page uses wrong hostname on update.

2018-02-07 Thread GitBox
mitchell852 commented on issue #1855: SSL cert page uses wrong hostname on update. URL: https://github.com/apache/incubator-trafficcontrol/issues/1855#issuecomment-363988455 @jpappa200 - i can't seem to recreate this bug. I'll reach out to you and we can try in our staging environment.

[GitHub] nir-sopher opened a new pull request #1859: cr-config: calculating the hash-id if needed

2018-02-07 Thread GitBox
nir-sopher opened a new pull request #1859: cr-config: calculating the hash-id if needed URL: https://github.com/apache/incubator-trafficcontrol/pull/1859 Hi, When adding a server via the old traffic ops, the xmpp_id field is saved to be the host_name. This field is used later on as

[GitHub] asfgit commented on issue #1860: normalize url query parameters handling and where and orderby clause construction

2018-02-07 Thread GitBox
asfgit commented on issue #1860: normalize url query parameters handling and where and orderby clause construction URL: https://github.com/apache/incubator-trafficcontrol/pull/1860#issuecomment-363948971 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 opened a new pull request #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy.

2018-02-07 Thread GitBox
mitchell852 opened a new pull request #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy. URL: https://github.com/apache/incubator-trafficcontrol/pull/1857 This is an automated message

[GitHub] asfgit commented on issue #1860: normalize url query parameters handling and where and orderby clause construction

2018-02-07 Thread GitBox
asfgit commented on issue #1860: normalize url query parameters handling and where and orderby clause construction URL: https://github.com/apache/incubator-trafficcontrol/pull/1860#issuecomment-363947809 Refer to this link for build results (access rights to CI server needed):

[GitHub] dneuman64 opened a new pull request #1856: Removed a zip file containing logos.

2018-02-07 Thread GitBox
dneuman64 opened a new pull request #1856: Removed a zip file containing logos. URL: https://github.com/apache/incubator-trafficcontrol/pull/1856 They are not allowed by apache and caused RAT errors. This is an automated

[GitHub] jpappa200 commented on issue #1855: SSL cert page uses wrong hostname on update.

2018-02-07 Thread GitBox
jpappa200 commented on issue #1855: SSL cert page uses wrong hostname on update. URL: https://github.com/apache/incubator-trafficcontrol/issues/1855#issuecomment-363905448 yes if multiple are configured it's using the wrong one.

[GitHub] knutsel closed pull request #1856: Removed a zip file containing logos.

2018-02-07 Thread GitBox
knutsel closed pull request #1856: Removed a zip file containing logos. URL: https://github.com/apache/incubator-trafficcontrol/pull/1856 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] dangogh commented on issue #1858: url parameters for last_updated are not parsed correctly

2018-02-07 Thread GitBox
dangogh commented on issue #1858: url parameters for last_updated are not parsed correctly URL: https://github.com/apache/incubator-trafficcontrol/issues/1858#issuecomment-363930089 this is a regression from 2.1, so should be fixed for 2.2

[GitHub] asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy.

2018-02-07 Thread GitBox
asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy. URL: https://github.com/apache/incubator-trafficcontrol/pull/1857#issuecomment-363915166 Refer to this link for build results (access rights to CI server needed):

[GitHub] DylanVolz opened a new issue #1858: url parameters for last_updated are not parsed correctly

2018-02-07 Thread GitBox
DylanVolz opened a new issue #1858: url parameters for last_updated are not parsed correctly URL: https://github.com/apache/incubator-trafficcontrol/issues/1858 url encoding the last_updated field of an hwinfo or parameters object and supplying it in the url parameters to create where

[GitHub] asfgit commented on issue #1859: cr-config: calculating the hash-id if needed

2018-02-07 Thread GitBox
asfgit commented on issue #1859: cr-config: calculating the hash-id if needed URL: https://github.com/apache/incubator-trafficcontrol/pull/1859#issuecomment-363931897 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy.

2018-02-07 Thread GitBox
asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy. URL: https://github.com/apache/incubator-trafficcontrol/pull/1857#issuecomment-363922168 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy.

2018-02-07 Thread GitBox
asfgit commented on issue #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy. URL: https://github.com/apache/incubator-trafficcontrol/pull/1857#issuecomment-363928865 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 opened a new pull request #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd?

2018-02-07 Thread GitBox
mitchell852 opened a new pull request #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd? URL: https://github.com/apache/incubator-trafficcontrol/pull/1862 ?ated fixes #1855

[GitHub] asfgit commented on issue #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd?

2018-02-07 Thread GitBox
asfgit commented on issue #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd? URL: https://github.com/apache/incubator-trafficcontrol/pull/1862#issuecomment-363999489 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd?

2018-02-07 Thread GitBox
asfgit commented on issue #1862: [Issue-1855] - messing with the hostname causes mismatches when the ssl keys are upd? URL: https://github.com/apache/incubator-trafficcontrol/pull/1862#issuecomment-364000335 Refer to this link for build results (access rights to CI server needed):

[GitHub] nir-sopher opened a new pull request #1867: Job deletion API

2018-02-08 Thread GitBox
nir-sopher opened a new pull request #1867: Job deletion API URL: https://github.com/apache/incubator-trafficcontrol/pull/1867 Allowing the deletion of a Job This is an automated message from the Apache Git Service. To

[GitHub] asfgit commented on issue #1867: Job deletion API

2018-02-08 Thread GitBox
asfgit commented on issue #1867: Job deletion API URL: https://github.com/apache/incubator-trafficcontrol/pull/1867#issuecomment-364289631 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1001/

[GitHub] dangogh commented on a change in pull request #1867: Job deletion API

2018-02-08 Thread GitBox
dangogh commented on a change in pull request #1867: Job deletion API URL: https://github.com/apache/incubator-trafficcontrol/pull/1867#discussion_r167097104 ## File path: traffic_ops/app/lib/TrafficOpsRoutes.pm ## @@ -531,7 +531,7 @@ sub api_routes {

[GitHub] dangogh commented on issue #1842: Added traffic portal using.rst to admin docs

2018-02-06 Thread GitBox
dangogh commented on issue #1842: Added traffic portal using.rst to admin docs URL: https://github.com/apache/incubator-trafficcontrol/pull/1842#issuecomment-363517595 I don't know why we'd care about paper size, but looks like Sphinx changed the method to use it:

[GitHub] alficles opened a new issue #1851: browser-icons.png should be removed

2018-02-06 Thread GitBox
alficles opened a new issue #1851: browser-icons.png should be removed URL: https://github.com/apache/incubator-trafficcontrol/issues/1851 `incubator-trafficcontrol/traffic_ops/app/public/images/browser-icons.png` should be removed. It's only reference (as far as I can tell) is on the

[GitHub] rob05c commented on issue #916: [TC-197] File descriptor leak caused by NIO protocol connector

2018-02-06 Thread GitBox
rob05c commented on issue #916: [TC-197] File descriptor leak caused by NIO protocol connector URL: https://github.com/apache/incubator-trafficcontrol/issues/916#issuecomment-363514686 Workaround was merged in #1848. The underlying problem will be fixed by upgrading Tomcat, but the

[GitHub] dangogh commented on issue #1842: Added traffic portal using.rst to admin docs

2018-02-06 Thread GitBox
dangogh commented on issue #1842: Added traffic portal using.rst to admin docs URL: https://github.com/apache/incubator-trafficcontrol/pull/1842#issuecomment-363521541 @trevorhays this really needs to be rebased and maybe squashed at least partially.. Let me know if I can help

[GitHub] dneuman64 closed pull request #1850: Added Traffic Control logos

2018-02-06 Thread GitBox
dneuman64 closed pull request #1850: Added Traffic Control logos URL: https://github.com/apache/incubator-trafficcontrol/pull/1850 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] RSK14 commented on issue #1783: AWS Traffic Sever

2018-02-06 Thread GitBox
RSK14 commented on issue #1783: AWS Traffic Sever URL: https://github.com/apache/incubator-trafficcontrol/issues/1783#issuecomment-363476904 Cool, thank you so much Regret, again I won?t mistake Satheesh On Tue, 6 Feb 2018 at 9:45 PM, David Neuman

[GitHub] dneuman64 commented on issue #1452: Need to be able to define default lat/lon for CLIENT_STEERING DSs

2018-02-06 Thread GitBox
dneuman64 commented on issue #1452: Need to be able to define default lat/lon for CLIENT_STEERING DSs URL: https://github.com/apache/incubator-trafficcontrol/issues/1452#issuecomment-363478887 Yeah, I think I figured that out after I submitted this issue and then forgot to close it. I

[GitHub] knutsel commented on issue #1842: Added traffic portal using.rst to admin docs

2018-02-06 Thread GitBox
knutsel commented on issue #1842: Added traffic portal using.rst to admin docs URL: https://github.com/apache/incubator-trafficcontrol/pull/1842#issuecomment-363520779 @trevorhays - Can you please double check the docs on http://traffic-control-cdn.readthedocs.io/en/latest/ after this

[GitHub] RSK14 commented on issue #1783: AWS Traffic Sever

2018-02-06 Thread GitBox
RSK14 commented on issue #1783: AWS Traffic Sever URL: https://github.com/apache/incubator-trafficcontrol/issues/1783#issuecomment-363408329 can you please response any one thank you This is an automated message from the

[GitHub] dangogh closed pull request #1800: Fix deep_caching_type validator regex

2018-02-06 Thread GitBox
dangogh closed pull request #1800: Fix deep_caching_type validator regex URL: https://github.com/apache/incubator-trafficcontrol/pull/1800 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] trevorhays commented on issue #1842: Added traffic portal using.rst to admin docs

2018-02-06 Thread GitBox
trevorhays commented on issue #1842: Added traffic portal using.rst to admin docs URL: https://github.com/apache/incubator-trafficcontrol/pull/1842#issuecomment-363608996 @knutsel The docs were updated. This is an automated

[GitHub] dg4prez commented on issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1

2018-02-06 Thread GitBox
dg4prez commented on issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1 URL: https://github.com/apache/incubator-trafficcontrol/issues/1173#issuecomment-363614647 I'll update this within the next couple of days or so.

[GitHub] rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-06 Thread GitBox
rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130#issuecomment-363568623 @ezelkow1 Is it all automatic, though? Or are there any config changes or parameters or anything that have to be set to use ATS 7,

[GitHub] mitchell852 closed pull request #1842: Added traffic portal using.rst to admin docs

2018-02-06 Thread GitBox
mitchell852 closed pull request #1842: Added traffic portal using.rst to admin docs URL: https://github.com/apache/incubator-trafficcontrol/pull/1842 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] asfgit commented on issue #809: Automatic License Checking with Weasel

2018-02-06 Thread GitBox
asfgit commented on issue #809: Automatic License Checking with Weasel URL: https://github.com/apache/incubator-trafficcontrol/pull/809#issuecomment-363551026 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-06 Thread GitBox
rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130#issuecomment-363571583 Ok, we should probably put that under a heading for "Using ATS 6+", with a short paragraph explaining, in

[GitHub] asfgit commented on issue #809: Automatic License Checking with Weasel

2018-02-06 Thread GitBox
asfgit commented on issue #809: Automatic License Checking with Weasel URL: https://github.com/apache/incubator-trafficcontrol/pull/809#issuecomment-363547403 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new issue #1852: Document spinner-small.gif license

2018-02-06 Thread GitBox
rob05c opened a new issue #1852: Document spinner-small.gif license URL: https://github.com/apache/incubator-trafficcontrol/issues/1852 spinner-small.gif's binary includes the ASCII text `NETSCAPE2.0- This is an automated message from the Apache Git Service.

[GitHub] asfgit commented on issue #1853: Implemented default.lat/long.to.override configs to avoid using maxmind default coordinates

2018-02-06 Thread GitBox
asfgit commented on issue #1853: Implemented default.lat/long.to.override configs to avoid using maxmind default coordinates URL: https://github.com/apache/incubator-trafficcontrol/pull/1853#issuecomment-363552384 Can one of the admins verify this patch?

[GitHub] rivasj opened a new pull request #1853: Implemented default.lat/long.to.override configs to avoid using maxmind default coordinates

2018-02-06 Thread GitBox
rivasj opened a new pull request #1853: Implemented default.lat/long.to.override configs to avoid using maxmind default coordinates URL: https://github.com/apache/incubator-trafficcontrol/pull/1853 Maxmind default coordinates for the US (37.751,-97.822) direct a large amount of traffic to

[GitHub] ezelkow1 commented on issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-06 Thread GitBox
ezelkow1 commented on issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130#issuecomment-363568079 This should be handled by https://github.com/apache/incubator-trafficcontrol/pull/1602 and

[GitHub] ezelkow1 commented on issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-06 Thread GitBox
ezelkow1 commented on issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130#issuecomment-363570061 Its not a completely automatic transition, users will have to convert their cacheurls to cachekeys if they want to move to ats7

[GitHub] dneuman64 commented on issue #1783: AWS Traffic Sever

2018-02-06 Thread GitBox
dneuman64 commented on issue #1783: AWS Traffic Sever URL: https://github.com/apache/incubator-trafficcontrol/issues/1783#issuecomment-363474609 It's ok, if our emails don't work, this will. It depends upon the use case but we round-robin, consistent hash, and "weighted" algorithms.

[GitHub] asfgit commented on issue #1850: Added Traffic Control logos

2018-02-06 Thread GitBox
asfgit commented on issue #1850: Added Traffic Control logos URL: https://github.com/apache/incubator-trafficcontrol/pull/1850#issuecomment-363495617 Can one of the admins verify this patch? This is an automated message from

[GitHub] dneuman64 closed issue #1452: Need to be able to define default lat/lon for CLIENT_STEERING DSs

2018-02-06 Thread GitBox
dneuman64 closed issue #1452: Need to be able to define default lat/lon for CLIENT_STEERING DSs URL: https://github.com/apache/incubator-trafficcontrol/issues/1452 This is an automated message from the Apache Git Service.

[GitHub] dneuman64 commented on issue #1783: AWS Traffic Sever

2018-02-06 Thread GitBox
dneuman64 commented on issue #1783: AWS Traffic Sever URL: https://github.com/apache/incubator-trafficcontrol/issues/1783#issuecomment-363456628 Hi Satheesh, I don't know why but our mailing list always seems to want to reject your emails. I have tried to approve them over and over, but

[GitHub] RSK14 commented on issue #1783: AWS Traffic Sever

2018-02-06 Thread GitBox
RSK14 commented on issue #1783: AWS Traffic Sever URL: https://github.com/apache/incubator-trafficcontrol/issues/1783#issuecomment-363464964 just asked for my knowledge,thanks for your timing reply. I won?t send again. Satheesh R On Tue, 6 Feb 2018 at 8:55 PM, David

[GitHub] chadgilloth opened a new pull request #1850: added some logos

2018-02-06 Thread GitBox
chadgilloth opened a new pull request #1850: added some logos URL: https://github.com/apache/incubator-trafficcontrol/pull/1850 ATC logo files in various formats. Page header logo files. This is an automated message from the

[GitHub] rawlinp opened a new pull request #1849: TR: Log WARN rather than FATAL when a polling URL param isn't set

2018-02-05 Thread GitBox
rawlinp opened a new pull request #1849: TR: Log WARN rather than FATAL when a polling URL param isn't set URL: https://github.com/apache/incubator-trafficcontrol/pull/1849 For example, deepcoveragezone.polling.url can remain unset if the CDN operator doesn't want to use the Deep

[GitHub] dneuman64 closed pull request #1848: TC-197 work around for Filer descriptor leak Issue #916

2018-02-05 Thread GitBox
dneuman64 closed pull request #1848: TC-197 work around for Filer descriptor leak Issue #916 URL: https://github.com/apache/incubator-trafficcontrol/pull/1848 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rawlinp commented on issue #1843: WIP: Fix steering race condition in TR

2018-02-05 Thread GitBox
rawlinp commented on issue #1843: WIP: Fix steering race condition in TR URL: https://github.com/apache/incubator-trafficcontrol/pull/1843#issuecomment-363237457 retest this please This is an automated message from the

[GitHub] rob05c commented on issue #1870: Remove old references to mso algorithm

2018-02-09 Thread GitBox
rob05c commented on issue #1870: Remove old references to mso algorithm URL: https://github.com/apache/incubator-trafficcontrol/pull/1870#issuecomment-364478887 Fixes #1173 This is an automated message from the Apache Git

[GitHub] dewrich closed issue #1841: Loosen permissions on fetching/uploading/generating SSL keys for a delivery service

2018-02-09 Thread GitBox
dewrich closed issue #1841: Loosen permissions on fetching/uploading/generating SSL keys for a delivery service URL: https://github.com/apache/incubator-trafficcontrol/issues/1841 This is an automated message from the

[GitHub] DylanVolz commented on issue #1858: url parameters for last_updated are not parsed correctly

2018-02-09 Thread GitBox
DylanVolz commented on issue #1858: url parameters for last_updated are not parsed correctly URL: https://github.com/apache/incubator-trafficcontrol/issues/1858#issuecomment-364511675 This is was due to user error while testing. Upon retesting last_updated filtering is working fine.

[GitHub] DylanVolz closed issue #1858: url parameters for last_updated are not parsed correctly

2018-02-09 Thread GitBox
DylanVolz closed issue #1858: url parameters for last_updated are not parsed correctly URL: https://github.com/apache/incubator-trafficcontrol/issues/1858 This is an automated message from the Apache Git Service. To respond

[GitHub] asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-09 Thread GitBox
asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864#issuecomment-364514967 Refer to this link for build results (access rights to CI server needed):

[GitHub] dangogh commented on a change in pull request #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-09 Thread GitBox
dangogh commented on a change in pull request #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864#discussion_r167304285 ## File path: traffic_ops/traffic_ops_golang/cdn/cdns.go ## @@ -128,16 +128,17

[GitHub] asfgit commented on issue #1818: DS Requests UI flow

2018-02-09 Thread GitBox
asfgit commented on issue #1818: DS Requests UI flow URL: https://github.com/apache/incubator-trafficcontrol/pull/1818#issuecomment-364518118 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1007/

[GitHub] dewrich closed pull request #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy.

2018-02-09 Thread GitBox
dewrich closed pull request #1857: [Issue-1841] - loosens permissions on ds ssl key management. relies on tenancy. URL: https://github.com/apache/incubator-trafficcontrol/pull/1857 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-09 Thread GitBox
asfgit commented on issue #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864#issuecomment-364520526 Refer to this link for build results (access rights to CI server needed):

[GitHub] dangogh closed pull request #1864: fix unclosed rows and error overwriting in cdns.go

2018-02-09 Thread GitBox
dangogh closed pull request #1864: fix unclosed rows and error overwriting in cdns.go URL: https://github.com/apache/incubator-trafficcontrol/pull/1864 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mitchell852 closed pull request #1870: Remove old references to mso algorithm

2018-02-12 Thread GitBox
mitchell852 closed pull request #1870: Remove old references to mso algorithm URL: https://github.com/apache/incubator-trafficcontrol/pull/1870 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] mitchell852 commented on issue #1872: Immutable ds cdn master

2018-02-12 Thread GitBox
mitchell852 commented on issue #1872: Immutable ds cdn master URL: https://github.com/apache/incubator-trafficcontrol/pull/1872#issuecomment-364945968 @nir-sopher - your explanation makes sense to me but maybe this is something you want to take to the dev mailing list for more discussion?

[GitHub] dewrich closed pull request #1398: Get ats_uid after packages are installed

2018-02-12 Thread GitBox
dewrich closed pull request #1398: Get ats_uid after packages are installed URL: https://github.com/apache/incubator-trafficcontrol/pull/1398 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] dewrich closed issue #1397: ORT tries to get ats_uid before installing trafficserver

2018-02-12 Thread GitBox
dewrich closed issue #1397: ORT tries to get ats_uid before installing trafficserver URL: https://github.com/apache/incubator-trafficcontrol/issues/1397 This is an automated message from the Apache Git Service. To respond

[GitHub] mtorluemke closed pull request #1768: Add Traffic Monitor Plugin System for Cache Stats Formats

2018-02-12 Thread GitBox
mtorluemke closed pull request #1768: Add Traffic Monitor Plugin System for Cache Stats Formats URL: https://github.com/apache/incubator-trafficcontrol/pull/1768 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] rob05c commented on issue #1620: API: Server API sets xmpp_id to null

2018-02-12 Thread GitBox
rob05c commented on issue #1620: API: Server API sets xmpp_id to null URL: https://github.com/apache/incubator-trafficcontrol/issues/1620#issuecomment-364980150 Fixed in #1859 This is an automated message from the Apache Git

[GitHub] rob05c closed issue #1620: API: Server API sets xmpp_id to null

2018-02-12 Thread GitBox
rob05c closed issue #1620: API: Server API sets xmpp_id to null URL: https://github.com/apache/incubator-trafficcontrol/issues/1620 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rob05c commented on issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1

2018-02-12 Thread GitBox
rob05c commented on issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1 URL: https://github.com/apache/incubator-trafficcontrol/issues/1173#issuecomment-364974928 Fixed in #1870

[GitHub] rob05c closed issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1

2018-02-12 Thread GitBox
rob05c closed issue #1173: 'Multi Site Origin Algorithm' is removed in delivery service UI in traffic_ops in TC-2.1 URL: https://github.com/apache/incubator-trafficcontrol/issues/1173 This is an automated message from the

[GitHub] elsloo closed pull request #1720: Adding TXT Records support

2018-02-12 Thread GitBox
elsloo closed pull request #1720: Adding TXT Records support URL: https://github.com/apache/incubator-trafficcontrol/pull/1720 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] elsloo commented on issue #1480: Experimental support for Client Subnet in DNS Queries RFC7871

2018-02-12 Thread GitBox
elsloo commented on issue #1480: Experimental support for Client Subnet in DNS Queries RFC7871 URL: https://github.com/apache/incubator-trafficcontrol/pull/1480#issuecomment-365007920 @limited can you or someone else update this PR to use the Jackson JSON libraries, using the examples

[GitHub] mitchell852 closed pull request #1807: Replacing ccr during config file generation

2018-02-12 Thread GitBox
mitchell852 closed pull request #1807: Replacing ccr during config file generation URL: https://github.com/apache/incubator-trafficcontrol/pull/1807 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] mitchell852 closed issue #990: [TC-353] Can't use CNAME hostname with "ccr" prefix

2018-02-12 Thread GitBox
mitchell852 closed issue #990: [TC-353] Can't use CNAME hostname with "ccr" prefix URL: https://github.com/apache/incubator-trafficcontrol/issues/990 This is an automated message from the Apache Git Service. To respond to

[GitHub] rob05c closed pull request #1844: DeepCachingType -- "NEVER" is now default

2018-02-12 Thread GitBox
rob05c closed pull request #1844: DeepCachingType -- "NEVER" is now default URL: https://github.com/apache/incubator-trafficcontrol/pull/1844 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] dangogh opened a new pull request #1873: License fixes

2018-02-12 Thread GitBox
dangogh opened a new pull request #1873: License fixes URL: https://github.com/apache/incubator-trafficcontrol/pull/1873 separated license fixes for 2.2 release This is an automated message from the Apache Git Service. To

[GitHub] asfgit commented on issue #1873: License fixes

2018-02-12 Thread GitBox
asfgit commented on issue #1873: License fixes URL: https://github.com/apache/incubator-trafficcontrol/pull/1873#issuecomment-365088770 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1012/ Test

[GitHub] rob05c closed issue #1852: Document spinner-small.gif license

2018-02-12 Thread GitBox
rob05c closed issue #1852: Document spinner-small.gif license URL: https://github.com/apache/incubator-trafficcontrol/issues/1852 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rob05c closed pull request #1873: License fixes

2018-02-12 Thread GitBox
rob05c closed pull request #1873: License fixes URL: https://github.com/apache/incubator-trafficcontrol/pull/1873 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] asfgit commented on issue #1769: Monitor Stats Format Plugin For Delivery Service Names

2018-02-12 Thread GitBox
asfgit commented on issue #1769: Monitor Stats Format Plugin For Delivery Service Names URL: https://github.com/apache/incubator-trafficcontrol/pull/1769#issuecomment-365058642 Refer to this link for build results (access rights to CI server needed):

[GitHub] mtorluemke closed pull request #1769: Monitor Stats Format Plugin For Delivery Service Names

2018-02-12 Thread GitBox
mtorluemke closed pull request #1769: Monitor Stats Format Plugin For Delivery Service Names URL: https://github.com/apache/incubator-trafficcontrol/pull/1769 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rob05c closed pull request #1854: Added 2.0 to 2.2 migration document

2018-02-12 Thread GitBox
rob05c closed pull request #1854: Added 2.0 to 2.2 migration document URL: https://github.com/apache/incubator-trafficcontrol/pull/1854 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-12 Thread GitBox
rob05c commented on issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130#issuecomment-365072673 TC now supports Cachekey, and documentation on migrating has been merged with #1854, therefore I'm resolving this. Any new Cachekey

[GitHub] rob05c closed issue #1130: cacheurl is deprecated in ATS 7.x

2018-02-12 Thread GitBox
rob05c closed issue #1130: cacheurl is deprecated in ATS 7.x URL: https://github.com/apache/incubator-trafficcontrol/issues/1130 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] dangogh closed pull request #1874: Change unknown images to known licensed images

2018-02-12 Thread GitBox
dangogh closed pull request #1874: Change unknown images to known licensed images URL: https://github.com/apache/incubator-trafficcontrol/pull/1874 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

<    5   6   7   8   9   10   11   12   13   14   >