[GitHub] [trafficcontrol] shamrickus opened a new pull request, #7075: Add Last Updated field to Traffic Portal

2022-09-19 Thread GitBox
shamrickus opened a new pull request, #7075: URL: https://github.com/apache/trafficcontrol/pull/7075 This PR adds the Last Updated field to every DS form in Traffic Portal (tpv2 already has them). ## Which Traffic Control components are affected by this PR? - Traffic

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
ocket commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974713602 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -96,6 +96,15 @@ func GetServersEligible(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] rimashah25 opened a new pull request, #7074: Fixed server table's quick search and filter for multiple profiles

2022-09-19 Thread GitBox
rimashah25 opened a new pull request, #7074: URL: https://github.com/apache/trafficcontrol/pull/7074 Closes: #7049 and #7052 ## Which Traffic Control components are affected by this PR? - Traffic Portal ## What is the best way to verify this PR? Add

[GitHub] [trafficcontrol] zrhoffman merged pull request #7069: Fix t3c for large number of added files

2022-09-19 Thread GitBox
zrhoffman merged PR #7069: URL: https://github.com/apache/trafficcontrol/pull/7069 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] rob05c commented on a diff in pull request #7069: Fix t3c for large number of added files

2022-09-19 Thread GitBox
rob05c commented on code in PR #7069: URL: https://github.com/apache/trafficcontrol/pull/7069#discussion_r974619046 ## cache-config/t3c-check-refs/config/config.go: ## @@ -106,13 +112,12 @@ func InitConfig(appVersion string, gitRevision string) (Cfg, error) {

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #7069: Fix t3c for large number of added files

2022-09-19 Thread GitBox
zrhoffman commented on code in PR #7069: URL: https://github.com/apache/trafficcontrol/pull/7069#discussion_r974613948 ## cache-config/t3c-check-refs/config/config.go: ## @@ -106,13 +112,12 @@ func InitConfig(appVersion string, gitRevision string) (Cfg, error) {

[GitHub] [trafficcontrol] zrhoffman merged pull request #7043: Fix t3c non-topo edge-origin DS params

2022-09-19 Thread GitBox
zrhoffman merged PR #7043: URL: https://github.com/apache/trafficcontrol/pull/7043 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
zrhoffman commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974574516 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -147,7 +156,8 @@ t.name as server_type, s.type as server_type_id, s.config_update_time >

[GitHub] [trafficcontrol] rob05c commented on a diff in pull request #7043: Fix t3c non-topo edge-origin DS params

2022-09-19 Thread GitBox
rob05c commented on code in PR #7043: URL: https://github.com/apache/trafficcontrol/pull/7043#discussion_r974596651 ## lib/go-atscfg/remapdotconfig.go: ## @@ -1002,11 +1002,53 @@ func serverIsLastCacheForDS(server *Server, ds *DeliveryService, topologies map[

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #7043: Fix t3c non-topo edge-origin DS params

2022-09-19 Thread GitBox
zrhoffman commented on code in PR #7043: URL: https://github.com/apache/trafficcontrol/pull/7043#discussion_r974592855 ## lib/go-atscfg/remapdotconfig.go: ## @@ -1002,11 +1002,53 @@ func serverIsLastCacheForDS(server *Server, ds *DeliveryService, topologies map[

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
srijeet0406 commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974586783 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -96,6 +96,15 @@ func GetServersEligible(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
srijeet0406 commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974575739 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -96,6 +96,15 @@ func GetServersEligible(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
srijeet0406 commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974575739 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -96,6 +96,15 @@ func GetServersEligible(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
ocket commented on code in PR #7073: URL: https://github.com/apache/trafficcontrol/pull/7073#discussion_r974569641 ## traffic_ops/traffic_ops_golang/deliveryservice/eligible.go: ## @@ -96,6 +96,15 @@ func GetServersEligible(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] rimashah25 commented on a diff in pull request #7068: Renamed rascal to tm/traffic_monitor

2022-09-19 Thread GitBox
rimashah25 commented on code in PR #7068: URL: https://github.com/apache/trafficcontrol/pull/7068#discussion_r974551757 ## docs/source/admin/traffic_router.rst: ## @@ -195,7 +195,7 @@ Much of a Traffic Router's configuration can be obtained through the :term:`Para

[GitHub] [trafficcontrol] srijeet0406 opened a new pull request, #7073: Adding the `asn` field to all `GET` endpoints that return a `server` object

2022-09-19 Thread GitBox
srijeet0406 opened a new pull request, #7073: URL: https://github.com/apache/trafficcontrol/pull/7073 This PR closes #7064 ## Which Traffic Control components are affected by this PR? - Traffic Ops ## What is the best way to verify this PR? Run

[GitHub] [trafficcontrol] ocket8888 opened a new issue, #7072: Incorrect documentation for `/deliveryservices/sslkeys/generate`

2022-09-19 Thread GitBox
ocket opened a new issue, #7072: URL: https://github.com/apache/trafficcontrol/issues/7072 ## This Bug Report affects these Traffic Control components: - Documentation ## Current behavior: The documentation describes the fields - `businessUnit` - `city` -

[GitHub] [trafficcontrol] zrhoffman commented on pull request #7071: Add Kotlin configuration for JDK 11

2022-09-15 Thread GitBox
zrhoffman commented on PR #7071: URL: https://github.com/apache/trafficcontrol/pull/7071#issuecomment-1248507863 Rebased due to a changelog merge conflict with #7065 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [trafficcontrol] srijeet0406 merged pull request #7065: Allow update time params on Servers when CDN is locked.

2022-09-15 Thread GitBox
srijeet0406 merged PR #7065: URL: https://github.com/apache/trafficcontrol/pull/7065 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] srijeet0406 closed issue #7047: TO /api/4.0/servers/:hostname/update should allow setting `config_apply_time` and `revalidate_apply_time` even if user doesn't have the lock

2022-09-15 Thread GitBox
srijeet0406 closed issue #7047: TO /api/4.0/servers/:hostname/update should allow setting `config_apply_time` and `revalidate_apply_time` even if user doesn't have the lock URL: https://github.com/apache/trafficcontrol/issues/7047 -- This is an automated message from the Apache Git Service.

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7065: Allow update time params on Servers when CDN is locked.

2022-09-15 Thread GitBox
srijeet0406 commented on code in PR #7065: URL: https://github.com/apache/trafficcontrol/pull/7065#discussion_r972184353 ## traffic_ops/testing/api/v4/serverupdatestatus_test.go: ## @@ -29,6 +29,119 @@ import ( client

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7065: Allow update time params on Servers when CDN is locked.

2022-09-13 Thread GitBox
srijeet0406 commented on code in PR #7065: URL: https://github.com/apache/trafficcontrol/pull/7065#discussion_r969976940 ## traffic_ops/traffic_ops_golang/server/update.go: ## @@ -296,9 +296,16 @@ func UpdateHandlerV4(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] zrhoffman commented on pull request #7071: Add Kotlin configuration for JDK 11

2022-09-13 Thread GitBox
zrhoffman commented on PR #7071: URL: https://github.com/apache/trafficcontrol/pull/7071#issuecomment-1245709900 Rebased due to a changelog merge conflict with #7063 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [trafficcontrol] srijeet0406 merged pull request #7063: Add TO API v5

2022-09-13 Thread GitBox
srijeet0406 merged PR #7063: URL: https://github.com/apache/trafficcontrol/pull/7063 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman merged pull request #7070: Add t3-tail to .gitignore

2022-09-13 Thread GitBox
zrhoffman merged PR #7070: URL: https://github.com/apache/trafficcontrol/pull/7070 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman opened a new pull request, #7071: Add Kotlin configuration for JDK 11

2022-09-13 Thread GitBox
zrhoffman opened a new pull request, #7071: URL: https://github.com/apache/trafficcontrol/pull/7071 - [x] This PR adds Kotlin support to Traffic Router *without* converting any classes to Kotlin. ## Which Traffic Control components are affected by this PR? -

[GitHub] [trafficcontrol] ocket8888 opened a new pull request, #7070: Add t3-tail to .gitignore

2022-09-13 Thread GitBox
ocket opened a new pull request, #7070: URL: https://github.com/apache/trafficcontrol/pull/7070 Adds the built `t3c-tail` binary to the list of git-ignored files. ## Which Traffic Control components are affected by this PR? - Traffic Control Cache Config (`t3c`,

[GitHub] [trafficcontrol] rob05c opened a new pull request, #7069: Fix t3c for large number of added files

2022-09-12 Thread GitBox
rob05c opened a new pull request, #7069: URL: https://github.com/apache/trafficcontrol/pull/7069 Fixes t3c-check-refs to accept the list of added files over stdin or the input file, instead of an arg. Also changes the corresponding t3c-apply call of check-refs to pass the list as

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7063: Add TO API v5

2022-09-09 Thread GitBox
ocket commented on code in PR #7063: URL: https://github.com/apache/trafficcontrol/pull/7063#discussion_r967572263 ## docs/source/api/v4/servers.rst: ## @@ -40,7 +40,7 @@ Request Structure | cachegroupName | no | Return only those servers within the

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7063: Add TO API v5

2022-09-09 Thread GitBox
ocket commented on code in PR #7063: URL: https://github.com/apache/trafficcontrol/pull/7063#discussion_r967572156 ## traffic_ops/v5-client/README.md: ## @@ -0,0 +1,53 @@ +# Traffic Ops Go Client + Review Comment: No, TO APIv5 is not released, so it doesn't need that

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7063: Add TO API v5

2022-09-09 Thread GitBox
ocket commented on code in PR #7063: URL: https://github.com/apache/trafficcontrol/pull/7063#discussion_r967572090 ## docs/source/api/v4/deliveryserviceserver.rst: ## @@ -58,7 +58,7 @@ Response Structure -- Unlike most API endpoints, this will return a

[GitHub] [trafficcontrol] srijeet0406 commented on a diff in pull request #7063: Add TO API v5

2022-09-09 Thread GitBox
srijeet0406 commented on code in PR #7063: URL: https://github.com/apache/trafficcontrol/pull/7063#discussion_r967546168 ## docs/source/api/v4/servers.rst: ## @@ -40,7 +40,7 @@ Request Structure | cachegroupName | no | Return only those servers within the

[GitHub] [trafficcontrol] moltzaum closed pull request #3580: TO: Added ReadBack function to fix "joined fields" issue

2022-09-09 Thread GitBox
moltzaum closed pull request #3580: TO: Added ReadBack function to fix "joined fields" issue URL: https://github.com/apache/trafficcontrol/pull/3580 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [trafficcontrol] rimashah25 commented on a diff in pull request #7068: Renamed rascal to tm/traffic_monitor

2022-09-09 Thread GitBox
rimashah25 commented on code in PR #7068: URL: https://github.com/apache/trafficcontrol/pull/7068#discussion_r967282731 ## traffic_ops/traffic_ops_golang/monitoring/monitoring.go: ## @@ -36,11 +36,11 @@ import ( "github.com/lib/pq" ) -const CacheMonitorConfigFile =

[GitHub] [trafficcontrol] rawlinp commented on a diff in pull request #7068: Renamed rascal to tm/traffic_monitor

2022-09-09 Thread GitBox
rawlinp commented on code in PR #7068: URL: https://github.com/apache/trafficcontrol/pull/7068#discussion_r967257497 ## traffic_ops/traffic_ops_golang/monitoring/monitoring.go: ## @@ -36,11 +36,11 @@ import ( "github.com/lib/pq" ) -const CacheMonitorConfigFile =

[GitHub] [trafficcontrol] ocket8888 merged pull request #7067: TP fix some DS rewrite fields not working

2022-09-08 Thread GitBox
ocket merged PR #7067: URL: https://github.com/apache/trafficcontrol/pull/7067 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] rimashah25 opened a new pull request, #7068: Renamed rascal to tm/traffic_monitor

2022-09-08 Thread GitBox
rimashah25 opened a new pull request, #7068: URL: https://github.com/apache/trafficcontrol/pull/7068 ## Which Traffic Control components are affected by this PR? - Documentation - Traffic Control Cache Config (`t3c`, formerly ORT) - Traffic Control Health

[GitHub] [trafficcontrol] rimashah25 commented on issue #7049: Traffic Portal (v1) - Servers do not display for layered/multi profiles

2022-09-08 Thread GitBox
rimashah25 commented on issue #7049: URL: https://github.com/apache/trafficcontrol/issues/7049#issuecomment-1240973564 I'll resolve this issue. Similar to https://github.com/apache/trafficcontrol/issues/7052 -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] rimashah25 commented on issue #7052: Traffic Portal v1 - Quick Search does not work for second/layered profiles

2022-09-08 Thread GitBox
rimashah25 commented on issue #7052: URL: https://github.com/apache/trafficcontrol/issues/7052#issuecomment-1240973361 I'll resolve this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [trafficcontrol] shamrickus commented on pull request #7067: TP fix some DS rewrite fields not working

2022-09-08 Thread GitBox
shamrickus commented on PR #7067: URL: https://github.com/apache/trafficcontrol/pull/7067#issuecomment-1240968268 It was related to the obscure fields changes so it's not in 7. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] mitchell852 commented on pull request #7067: TP fix some DS rewrite fields not working

2022-09-08 Thread GitBox
mitchell852 commented on PR #7067: URL: https://github.com/apache/trafficcontrol/pull/7067#issuecomment-1240967340 @shamrickus this feels like something that might warrant a 7.0.1? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [trafficcontrol] shamrickus opened a new pull request, #7067: TP fix some DS rewrite fields not working

2022-09-08 Thread GitBox
shamrickus opened a new pull request, #7067: URL: https://github.com/apache/trafficcontrol/pull/7067 This PR fixes an issue where certain header rewrite fields on a DS were not working (displaying data or sending data to the APII). ## Which Traffic Control components are

[GitHub] [trafficcontrol] ocket8888 merged pull request #7066: TP don't autocomplete on user change password field.

2022-09-08 Thread GitBox
ocket merged PR #7066: URL: https://github.com/apache/trafficcontrol/pull/7066 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] shamrickus opened a new pull request, #7066: TP don't autocomplete on user change password field.

2022-09-07 Thread GitBox
shamrickus opened a new pull request, #7066: URL: https://github.com/apache/trafficcontrol/pull/7066 This PR fixes another Chrome autofill password issue similar to #7053 ## Which Traffic Control components are affected by this PR? - Traffic Portal ## What is

[GitHub] [trafficcontrol] mitchell852 closed pull request #7057: TP: Adds the ability to turn ACME on/off for generating SSL keys for a delivery service

2022-09-07 Thread GitBox
mitchell852 closed pull request #7057: TP: Adds the ability to turn ACME on/off for generating SSL keys for a delivery service URL: https://github.com/apache/trafficcontrol/pull/7057 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [trafficcontrol] shamrickus opened a new pull request, #7065: Allow update time params on Servers when CDN is locked.

2022-09-07 Thread GitBox
shamrickus opened a new pull request, #7065: URL: https://github.com/apache/trafficcontrol/pull/7065 This PR fixes #7047 ## Which Traffic Control components are affected by this PR? - Traffic Ops ## What is the best way to verify this PR? Run the tests and

[GitHub] [trafficcontrol] srijeet0406 opened a new issue, #7064: All endpoints need to account for the ASN field in servers

2022-09-07 Thread GitBox
srijeet0406 opened a new issue, #7064: URL: https://github.com/apache/trafficcontrol/issues/7064 ## This Bug Report affects these Traffic Control components: - Traffic Ops ## Current behavior: After https://github.com/apache/trafficcontrol/pull/7023 was

[GitHub] [trafficcontrol] ocket8888 merged pull request #7023: Add asn field to server

2022-09-07 Thread GitBox
ocket merged PR #7023: URL: https://github.com/apache/trafficcontrol/pull/7023 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] ocket8888 opened a new pull request, #7063: Add TO API v5

2022-09-07 Thread GitBox
ocket opened a new pull request, #7063: URL: https://github.com/apache/trafficcontrol/pull/7063 Adds version 5.0 of the Traffic Ops API. Everything still uses v4 (which doesn't matter yet as there are no differences between v4 and v5 as of this PR) ## Which Traffic

[GitHub] [trafficcontrol] ocket8888 merged pull request #7036: TP: Add indicator to Delivery Services if they are a steering target

2022-09-07 Thread GitBox
ocket merged PR #7036: URL: https://github.com/apache/trafficcontrol/pull/7036 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] ocket8888 closed issue #2101: Traffic Portal feature request: the ability to identify if a DS is used as a steering target

2022-09-07 Thread GitBox
ocket closed issue #2101: Traffic Portal feature request: the ability to identify if a DS is used as a steering target URL: https://github.com/apache/trafficcontrol/issues/2101 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] zrhoffman merged pull request #7062: Update Go version to 1.19.1

2022-09-06 Thread GitBox
zrhoffman merged PR #7062: URL: https://github.com/apache/trafficcontrol/pull/7062 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] asf-ci opened a new pull request, #7062: Update Go version to 1.19.1

2022-09-06 Thread GitBox
asf-ci opened a new pull request, #7062: URL: https://github.com/apache/trafficcontrol/pull/7062 ## What does this PR (Pull Request) do? This PR makes the Go components of Traffic Control build using Go version 1.19.1 and updates the `golang.org/x/` dependencies. See the Go

[GitHub] [trafficcontrol] zrhoffman closed pull request #7060: Update Go version to 1.19.1

2022-09-06 Thread GitBox
zrhoffman closed pull request #7060: Update Go version to 1.19.1 URL: https://github.com/apache/trafficcontrol/pull/7060 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [trafficcontrol] zrhoffman commented on pull request #7060: Update Go version to 1.19.1

2022-09-06 Thread GitBox
zrhoffman commented on PR #7060: URL: https://github.com/apache/trafficcontrol/pull/7060#issuecomment-1238487179 deleting this branch to generate a new one off the tip of master -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] zrhoffman merged pull request #7061: Fixes GetProfileID functions

2022-09-06 Thread GitBox
zrhoffman merged PR #7061: URL: https://github.com/apache/trafficcontrol/pull/7061 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] ericholguin opened a new pull request, #7061: Fixes GetProfileID functions

2022-09-06 Thread GitBox
ericholguin opened a new pull request, #7061: URL: https://github.com/apache/trafficcontrol/pull/7061 Fixes issue where test is using the wrong GetProfileID function. ## Which Traffic Control components are affected by this PR? - Traffic Ops Tests ##

[GitHub] [trafficcontrol] asf-ci opened a new pull request, #7060: Update Go version to 1.19.1

2022-09-06 Thread GitBox
asf-ci opened a new pull request, #7060: URL: https://github.com/apache/trafficcontrol/pull/7060 ## What does this PR (Pull Request) do? This PR makes the Go components of Traffic Control build using Go version 1.19.1 and updates the `golang.org/x/` dependencies. See the Go

[GitHub] [trafficcontrol] rawlinp merged pull request #7058: TP: adds documentation link for content routing type

2022-09-06 Thread GitBox
rawlinp merged PR #7058: URL: https://github.com/apache/trafficcontrol/pull/7058 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman merged pull request #7009: Refactor Profiles and Profile Parameters Tests

2022-09-06 Thread GitBox
zrhoffman merged PR #7009: URL: https://github.com/apache/trafficcontrol/pull/7009 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman opened a new pull request, #7059: Update `github.com/go-acme/lego` dependency to v3

2022-09-06 Thread GitBox
zrhoffman opened a new pull request, #7059: URL: https://github.com/apache/trafficcontrol/pull/7059 This PR resolves #4835 by updating the `github.com/go-acme/lego` dependency to `github.com/go-acme/lego/v3`. ## Which Traffic Control components are affected by this PR?

[GitHub] [trafficcontrol] mitchell852 opened a new pull request, #7058: TP: adds documentation link for content routing type

2022-09-05 Thread GitBox
mitchell852 opened a new pull request, #7058: URL: https://github.com/apache/trafficcontrol/pull/7058 ## Which Traffic Control components are affected by this PR? - Traffic Portal ## What is the best way to verify this PR? In TP, create a new delivery service and when asked to

[GitHub] [trafficcontrol] mitchell852 opened a new pull request, #7057: Adds the ability to turn ACME on/off for generating SSL keys for a delivery service

2022-09-04 Thread GitBox
mitchell852 opened a new pull request, #7057: URL: https://github.com/apache/trafficcontrol/pull/7057 ## Which Traffic Control components are affected by this PR? - Traffic Portal ## What is the best way to verify this PR? In TP, navigate to an HTTPS delivery service

[GitHub] [trafficcontrol] ericholguin opened a new pull request, #7056: Refactor Topologies Tests

2022-09-02 Thread GitBox
ericholguin opened a new pull request, #7056: URL: https://github.com/apache/trafficcontrol/pull/7056 This PR refactors **v4/topologies_test.go**, **v3/topologies_test.go**, in order to make tests more granular, providing transparency on what is being tested and reducing the need

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7036: TP: Add indicator to Delivery Services if they are a steering target

2022-09-02 Thread GitBox
ocket commented on code in PR #7036: URL: https://github.com/apache/trafficcontrol/pull/7036#discussion_r961318010 ## traffic_portal/app/src/common/service/utils/DeliveryServiceUtils.js: ## @@ -17,45 +17,61 @@ * under the License. */ -var DeliveryServiceUtils =

[GitHub] [trafficcontrol] zrhoffman opened a new issue, #7055: TP "Clear Table Filters" action does not have a button

2022-09-01 Thread GitBox
zrhoffman opened a new issue, #7055: URL: https://github.com/apache/trafficcontrol/issues/7055 ## This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components: - Traffic Portal ## Current behavior: In orer to

[GitHub] [trafficcontrol] shamrickus commented on a diff in pull request #7036: TP: Add indicator to Delivery Services if they are a steering target

2022-09-01 Thread GitBox
shamrickus commented on code in PR #7036: URL: https://github.com/apache/trafficcontrol/pull/7036#discussion_r961034826 ## traffic_portal/app/src/common/service/utils/DeliveryServiceUtils.js: ## @@ -17,45 +17,61 @@ * under the License. */ -var DeliveryServiceUtils =

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7036: TP: Add indicator to Delivery Services if they are a steering target

2022-09-01 Thread GitBox
ocket commented on code in PR #7036: URL: https://github.com/apache/trafficcontrol/pull/7036#discussion_r961021870 ## experimental/traffic-portal/src/app/api/delivery-service.service.ts: ## @@ -158,6 +159,16 @@ export class DeliveryServiceService extends APIService {

[GitHub] [trafficcontrol] rawlinp merged pull request #7053: TP do not autofill password fields on non-login pages

2022-09-01 Thread GitBox
rawlinp merged PR #7053: URL: https://github.com/apache/trafficcontrol/pull/7053 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] zrhoffman merged pull request #7054: ATC Collaborators for September 2022

2022-09-01 Thread GitBox
zrhoffman merged PR #7054: URL: https://github.com/apache/trafficcontrol/pull/7054 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] asf-ci opened a new pull request, #7054: ATC Collaborators for September 2022

2022-09-01 Thread GitBox
asf-ci opened a new pull request, #7054: URL: https://github.com/apache/trafficcontrol/pull/7054 This PR uses [`.asf.yaml`](https://s.apache.org/asfyamltriage) to assign the GitHub [Triage

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED) and makes it optional

2022-08-31 Thread GitBox
ocket commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r960146800 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r960038082 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] shamrickus opened a new pull request, #7053: TP do not autofill password fields on non-login pages

2022-08-31 Thread GitBox
shamrickus opened a new pull request, #7053: URL: https://github.com/apache/trafficcontrol/pull/7053 This PR fixes an issue where ilo passwords on a server were being automatically filled with the users login/password in both TPv1 and 2. This should only be an issue with Chromium

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
ocket commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959826133 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
ocket commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959826133 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
ocket commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959826133 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959820465 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959820465 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959819927 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959819046 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959819046 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #7042: Enables the use of offline reason for all "offline" statuses (those that are not ONLINE/REPORTED)

2022-08-31 Thread GitBox
ocket commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r959818308 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of "offline reason" for all statuses not ONLINE/REPORTED

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r956834910 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] mitchell852 commented on a diff in pull request #7042: Enables the use of "offline reason" for all statuses not ONLINE/REPORTED

2022-08-31 Thread GitBox
mitchell852 commented on code in PR #7042: URL: https://github.com/apache/trafficcontrol/pull/7042#discussion_r956834910 ## traffic_ops/traffic_ops_golang/server/put_status.go: ## @@ -121,7 +121,7 @@ func UpdateStatusHandler(w http.ResponseWriter, r *http.Request) {

[GitHub] [trafficcontrol] ocket8888 commented on issue #7052: Traffic Portal v1 - Quick Search does not work for second/layered profiles

2022-08-31 Thread GitBox
ocket commented on issue #7052: URL: https://github.com/apache/trafficcontrol/issues/7052#issuecomment-1233129049 The simplest fix would be to update the display value to include all Profile names, probably separated by `, `. Then a filter on the table would check all of the profiles

[GitHub] [trafficcontrol] TaylorCFrey opened a new issue, #7052: Traffic Portal v1 - Quick Search does not work for second/layered profiles

2022-08-31 Thread GitBox
TaylorCFrey opened a new issue, #7052: URL: https://github.com/apache/trafficcontrol/issues/7052 ## This Bug Report affects these Traffic Control components: - Traffic Portal ## Current behavior: Open the server table in Traffic Portal. At the top of the

[GitHub] [trafficcontrol] shamrickus merged pull request #7051: Bump moment-timezone from 0.5.34 to 0.5.37 in /traffic_portal/app/src

2022-08-31 Thread GitBox
shamrickus merged PR #7051: URL: https://github.com/apache/trafficcontrol/pull/7051 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [trafficcontrol] dependabot[bot] opened a new pull request, #7051: Bump moment-timezone from 0.5.34 to 0.5.37 in /traffic_portal/app/src

2022-08-30 Thread GitBox
dependabot[bot] opened a new pull request, #7051: URL: https://github.com/apache/trafficcontrol/pull/7051 Bumps [moment-timezone](https://github.com/moment/moment-timezone) from 0.5.34 to 0.5.37. Changelog Sourced from

[GitHub] [trafficcontrol] TaylorCFrey opened a new pull request, #7050: Add configuration setting for TO client request timeout

2022-08-30 Thread GitBox
TaylorCFrey opened a new pull request, #7050: URL: https://github.com/apache/trafficcontrol/pull/7050 Closes: #7048 ## Which Traffic Control components are affected by this PR? - Documentation - Traffic Stats ## What is the best way to verify this

[GitHub] [trafficcontrol] TaylorCFrey opened a new issue, #7049: Traffic Portal (v1) - Servers do not display layered/multi profiles

2022-08-30 Thread GitBox
TaylorCFrey opened a new issue, #7049: URL: https://github.com/apache/trafficcontrol/issues/7049 ## This Bug Report affects these Traffic Control components: - Traffic Portal ## Current behavior: The second (+) profile is not filterable on the server table

[GitHub] [trafficcontrol] TaylorCFrey opened a new issue, #7048: Traffic Stats - Add Traffic Ops client timeout configuration

2022-08-30 Thread GitBox
TaylorCFrey opened a new issue, #7048: URL: https://github.com/apache/trafficcontrol/issues/7048 ## This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components: - Traffic Stats ## Current behavior: The

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
zrhoffman commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r958745529 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently serve

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
zrhoffman commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r958745529 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently serve

[GitHub] [trafficcontrol] rawlinp opened a new issue, #7047: TO /api/4.0/servers/:hostname/update should allow setting `config_apply_time` and `revalidate_apply_time` even if user doesn't have the loc

2022-08-30 Thread GitBox
rawlinp opened a new issue, #7047: URL: https://github.com/apache/trafficcontrol/issues/7047 ## This Bug Report affects these Traffic Control components: - Traffic Ops ## Current behavior: If a CDN is locked and a user tries to call `POST

[GitHub] [trafficcontrol] alficles commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
alficles commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r958729468 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently serve

[GitHub] [trafficcontrol] alficles commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
alficles commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r958724270 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently serve

[GitHub] [trafficcontrol] zrhoffman commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
zrhoffman commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r946004662 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently serve

[GitHub] [trafficcontrol] zrhoffman opened a new issue, #7046: POST /deliveryservices/sslkeys/add accepts unrelated certificates

2022-08-30 Thread GitBox
zrhoffman opened a new issue, #7046: URL: https://github.com/apache/trafficcontrol/issues/7046 ## This Bug Report affects these Traffic Control components: - Traffic Ops ## Current behavior: `POST /api/4.1/deliveryservices/sslkeys/add` accepts unrelated

[GitHub] [trafficcontrol] TaylorCFrey commented on a diff in pull request #6622: Added Client Certificate Auth blueprint.

2022-08-30 Thread GitBox
TaylorCFrey commented on code in PR #6622: URL: https://github.com/apache/trafficcontrol/pull/6622#discussion_r958560648 ## blueprints/client-certificate-auth.md: ## @@ -0,0 +1,240 @@ + + +# Client Certificate Authentication + +## Problem Description + +Passwords currently

<    7   8   9   10   11   12   13   14   15   16   >