Jenkins build is back to normal : trafficcontrol-PR #4630

See

[GitHub] [trafficcontrol] asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE

asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE URL: https://github.com/apache/trafficcontrol/pull/4015#issuecomment-545702039 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] lbathina commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself.

lbathina commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself. URL: https://github.com/apache/trafficcontrol/issues/4019#issuecomment-545678320 Goal is not to allow comparison of profile with itself. I agree with oclet. That is best and

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338303676 ## File path: docs/source/api/deliveryservice_server_capabilities.rst ## @@ -0,0 +1,222 @@ +..

[GitHub] [trafficcontrol] ocket8888 commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and…

ocket commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and… URL: https://github.com/apache/trafficcontrol/pull/3534#issuecomment-545655710 I wouldn't worry about that, the CI is all but worthless these days.

[GitHub] [trafficcontrol] mitchell852 merged pull request #3604: Removed restangular from CacheGroupService

mitchell852 merged pull request #3604: Removed restangular from CacheGroupService URL: https://github.com/apache/trafficcontrol/pull/3604 This is an automated message from the Apache Git Service. To respond to the message,

Jenkins build is back to normal : trafficcontrol-master-build #1557

See

[GitHub] [trafficcontrol] rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler

rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler URL: https://github.com/apache/trafficcontrol/pull/4011#issuecomment-545647906 > Hm. I thought that was fixed a while back, though. A holistic solution was never merged AFAIK.

[GitHub] [trafficcontrol] ocket8888 closed issue #3820: Rewrite /roles to Go

ocket closed issue #3820: Rewrite /roles to Go URL: https://github.com/apache/trafficcontrol/issues/3820 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [trafficcontrol] ocket8888 commented on issue #4011: Implement GET /api/1.1/roles handler

ocket commented on issue #4011: Implement GET /api/1.1/roles handler URL: https://github.com/apache/trafficcontrol/pull/4011#issuecomment-545645851 Hm. I thought that was fixed a while back, though. This is an automated

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338288856 ## File path: docs/source/api/deliveryservice_server_capabilities.rst ## @@ -0,0 +1,222 @@ +..

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338288655 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] asf-ci commented on issue #3998: TP: Ability to manager capabilities of servers

asf-ci commented on issue #3998: TP: Ability to manager capabilities of servers URL: https://github.com/apache/trafficcontrol/pull/3998#issuecomment-545644412 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4629

See Changes: [ocket] Add top-level vendor dir to TO-Go and TM unit test containers (#4009) [ocket] accounts for null arrays (#4008) [mitchell852] adds server capbility management to a server

Jenkins build is back to normal : trafficcontrol-traffic_ops-test #1600

See

[GitHub] [trafficcontrol] asf-ci commented on issue #4020: Fix TO API test failure when starting from a fresh `db/admin reset`

asf-ci commented on issue #4020: Fix TO API test failure when starting from a fresh `db/admin reset` URL: https://github.com/apache/trafficcontrol/pull/4020#issuecomment-545628056 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4628

See Changes: [rawlin_peters] Fix TO API test failure when starting from a fresh `db/admin reset` -- [...truncated 2.58 MB...] traffic_portal_build_1 |

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created

mitchell852 edited a comment on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created URL: https://github.com/apache/trafficcontrol/issues/4016#issuecomment-545621196 yeah, every time a ds is updated, the consistent hash

[GitHub] [trafficcontrol] mitchell852 commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created

mitchell852 commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created URL: https://github.com/apache/trafficcontrol/issues/4016#issuecomment-545621196 yeah, every time a ds is updated, the consistent hash query

[GitHub] [trafficcontrol] mitchell852 commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself.

mitchell852 commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself. URL: https://github.com/apache/trafficcontrol/issues/4019#issuecomment-545620043 > I think you'll need to add a form validation that disables pressing the "Submit" ("Compare"?)

[GitHub] [trafficcontrol] mattjackson220 removed a comment on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and…

mattjackson220 removed a comment on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and… URL: https://github.com/apache/trafficcontrol/pull/3534#issuecomment-545610056 retest this please This

[GitHub] [trafficcontrol] mattjackson220 commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and…

mattjackson220 commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and… URL: https://github.com/apache/trafficcontrol/pull/3534#issuecomment-545610056 retest this please This is an

[GitHub] [trafficcontrol] rawlinp opened a new pull request #4020: Fix TO API test failure when starting from a fresh `db/admin reset`

rawlinp opened a new pull request #4020: Fix TO API test failure when starting from a fresh `db/admin reset` URL: https://github.com/apache/trafficcontrol/pull/4020 ## What does this PR (Pull Request) do? This PR fixes an issue that causes the TO API tests to fail when being run

[GitHub] [trafficcontrol] ocket8888 commented on issue #4011: Implement GET /api/1.1/roles handler

ocket commented on issue #4011: Implement GET /api/1.1/roles handler URL: https://github.com/apache/trafficcontrol/pull/4011#issuecomment-545602942 Forgot this was only a change for `GET` for a second there. But just so I'm clear, you're saying the issue is in `api.GenericUpdate`/some

[GitHub] [trafficcontrol] rob05c commented on a change in pull request #4018: Add deliveryservice server capability docs

rob05c commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338235115 ## File path: docs/source/api/server_server_capabilities.rst ## @@ -222,4 +222,4 @@ Response

[GitHub] [trafficcontrol] ChrisHines commented on a change in pull request #4018: Add deliveryservice server capability docs

ChrisHines commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338234536 ## File path: docs/source/api/server_server_capabilities.rst ## @@ -222,4 +222,4 @@ Response

[GitHub] [trafficcontrol] rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler

rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler URL: https://github.com/apache/trafficcontrol/pull/4011#issuecomment-545597371 Also, it seems intentional that `PUT` doesn't actually update the priv_level, since that would go away with roles and capabilities. The

[GitHub] [trafficcontrol] rob05c commented on a change in pull request #4018: Add deliveryservice server capability docs

rob05c commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338233186 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] mhoppa commented on a change in pull request #4018: Add deliveryservice server capability docs

mhoppa commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338232252 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler

rawlinp commented on issue #4011: Implement GET /api/1.1/roles handler URL: https://github.com/apache/trafficcontrol/pull/4011#issuecomment-545595699 I didn't write the PUT/POST/1.3+GET handlers. IMO we shouldn't even be able to create/update `roles` today since they're built-in, but I

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338230415 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] ChrisHines commented on a change in pull request #4018: Add deliveryservice server capability docs

ChrisHines commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338229265 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] ocket8888 commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself.

ocket commented on issue #4019: TP: Compare profiles - should not allow to compare a profile to itself. URL: https://github.com/apache/trafficcontrol/issues/4019#issuecomment-545580299 > _"The best is to remove the item selected in first dropdown from the other so they are not allowed

[GitHub] [trafficcontrol] lbathina opened a new issue #4019: TP: Compare profiles - should not allow to compare a profile to itself.

lbathina opened a new issue #4019: TP: Compare profiles - should not allow to compare a profile to itself. URL: https://github.com/apache/trafficcontrol/issues/4019 ## I'm submitting a ... - [X] bug report - [ ] new feature / enhancement request - [ ]

[GitHub] [trafficcontrol] ocket8888 commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created

ocket commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created URL: https://github.com/apache/trafficcontrol/issues/4016#issuecomment-545578477 Thanks. Seems to be more of an improvement than a bug;

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338208151 ## File path: .gitignore ## @@ -11,6 +11,7 @@ traffic_ops/app/public/CRConfig-Snapshots/*

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338209082 ## File path: docs/source/api/deliveryservice_server_capabilities.rst ## @@ -0,0 +1,222 @@ +..

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338209592 ## File path: docs/source/api/deliveryservice_server_capabilities.rst ## @@ -0,0 +1,222 @@ +..

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338210487 ## File path: docs/source/api/server_server_capabilities.rst ## @@ -222,4 +222,4 @@ Response

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4018: Add deliveryservice server capability docs

ocket commented on a change in pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#discussion_r338208580 ## File path: docs/source/api/deliveryservice_server_capabilities.rst ## @@ -0,0 +1,222 @@ +..

[GitHub] [trafficcontrol] asf-ci commented on issue #4018: Add deliveryservice server capability docs

asf-ci commented on issue #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018#issuecomment-545575429 Can one of the admins verify this patch? This is an automated message

[GitHub] [trafficcontrol] ChrisHines opened a new pull request #4018: Add deliveryservice server capability docs

ChrisHines opened a new pull request #4018: Add deliveryservice server capability docs URL: https://github.com/apache/trafficcontrol/pull/4018 ## What does this PR (Pull Request) do? - [ ] This PR adds documentation for the deliveryservice_server_capabilities endpoint added by

[GitHub] [trafficcontrol] ocket8888 opened a new issue #4017: Go client/API integration tests are a bit too verbose

ocket opened a new issue #4017: Go client/API integration tests are a bit too verbose URL: https://github.com/apache/trafficcontrol/issues/4017 ## I'm submitting a ... - improvement request (usability, performance, tech debt, etc.) ## Traffic Control components affected ...

[GitHub] [trafficcontrol] asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE

asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE URL: https://github.com/apache/trafficcontrol/pull/4015#issuecomment-545562964 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4627

See Changes: [ocket] go fmt -- [...truncated 2.56 MB...] traffic_portal_build_1 | | | +-- json-schema-traverse@0.4.1 traffic_portal_build_1

[GitHub] [trafficcontrol] asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE

asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE URL: https://github.com/apache/trafficcontrol/pull/4015#issuecomment-545560955 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4626

See Changes: [ocket] Add top-level vendor dir to TO-Go and TM unit test containers (#4009) [ocket] accounts for null arrays (#4008) [mitchell852] Rewrote /user/reset_password to Go (#3929)

[GitHub] [trafficcontrol] ocket8888 commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created

ocket commented on issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created URL: https://github.com/apache/trafficcontrol/issues/4016#issuecomment-545558789 > _"... the following change log entries are created"_

Build failed in Jenkins: trafficcontrol-PR #4625

See Changes: [rawlin_peters] Add missing query params to API docs -- GitHub pull request #4013 of commit c0055ec2889d793dddf73fa9ca8f932acdbc5a23, no merge conflicts.

[GitHub] [trafficcontrol] asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler

asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler URL: https://github.com/apache/trafficcontrol/pull/4013#issuecomment-545549041 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] lbathina opened a new issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created

lbathina opened a new issue #4016: TP: everytime a ds is updated with consistent hash query params the following change log entries are created URL: https://github.com/apache/trafficcontrol/issues/4016 ## I'm submitting a ... - [X] bug report - [ ] new feature

Build failed in Jenkins: trafficcontrol-PR #4624

See Changes: [rawlin_peters] Add changelog entry -- GitHub pull request #4013 of commit 0a1c2c77eb848564d5410ab7581a1fcea300e7ae, no merge conflicts. Running as SYSTEM

[GitHub] [trafficcontrol] asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler

asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler URL: https://github.com/apache/trafficcontrol/pull/4013#issuecomment-545545068 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler

asf-ci commented on issue #4013: Add TO-Go /api/1.1/servers/status GET handler URL: https://github.com/apache/trafficcontrol/pull/4013#issuecomment-545543835 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4623

See Changes: [mitchell852] Add TP test for DNSSEC keys (#3735) [mitchell852] Change redirect location after creating a parameter or profile (#3737) [mitchell852] Fix bug that causes DS requests to fulfill

Build failed in Jenkins: trafficcontrol-traffic_ops-test #1599

See Changes: [mitchell852] Removed restangular from ChangeLogService (#3608) -- Started by an SCM change Running as SYSTEM [EnvInject] - Loading node

Build failed in Jenkins: trafficcontrol-master-build #1556

See Changes: [mitchell852] Removed restangular from ChangeLogService (#3608) -- Started by an SCM change Running as SYSTEM [EnvInject] - Loading node

[GitHub] [trafficcontrol] mitchell852 merged pull request #3608: Removed restangular from ChangeLogService

mitchell852 merged pull request #3608: Removed restangular from ChangeLogService URL: https://github.com/apache/trafficcontrol/pull/3608 This is an automated message from the Apache Git Service. To respond to the message,

Build failed in Jenkins: trafficcontrol-PR #4622

See Changes: [ocket] Removed restangular from ./ChangeLogService.js [ocket] Fixed some service methods not throwing in error handlers [ocket] fixes bugs with restangular removal

[GitHub] [trafficcontrol] asf-ci commented on issue #3608: Removed restangular from ChangeLogService

asf-ci commented on issue #3608: Removed restangular from ChangeLogService URL: https://github.com/apache/trafficcontrol/pull/3608#issuecomment-545514334 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4622/

[GitHub] [trafficcontrol] asf-ci commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and…

asf-ci commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and… URL: https://github.com/apache/trafficcontrol/pull/3534#issuecomment-545501967 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4621

See Changes: [matthew_jackson] TP Delivery Service Generate SSL update, new letsencrypt generate and [matthew_jackson] Added API for dns challenges and setup DNS Challenge calls to LE [matthew_jackson] DNS

Jenkins build is back to normal : trafficcontrol-PR #4620

See

[GitHub] [trafficcontrol] asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE

asf-ci commented on issue #4015: Rewrite /federations to Go - POST/PUT/DELETE URL: https://github.com/apache/trafficcontrol/pull/4015#issuecomment-545499542 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-master-build #1555

See Changes: [mitchell852] Rewrote /user/reset_password to Go (#3929) -- Started by an SCM change Running as SYSTEM [EnvInject] - Loading node environment

[GitHub] [trafficcontrol] asf-ci commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and…

asf-ci commented on issue #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and… URL: https://github.com/apache/trafficcontrol/pull/3534#issuecomment-545480049 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #4619

See Changes: [matthew_jackson] TP Delivery Service Generate SSL update, new letsencrypt generate and [matthew_jackson] Added API for dns challenges and setup DNS Challenge calls to LE [matthew_jackson] DNS

[GitHub] [trafficcontrol] mitchell852 merged pull request #3929: Rewrote /user/reset_password to Go

mitchell852 merged pull request #3929: Rewrote /user/reset_password to Go URL: https://github.com/apache/trafficcontrol/pull/3929 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [trafficcontrol] mitchell852 closed issue #3842: Rewrite /user/reset_password to Go

mitchell852 closed issue #3842: Rewrite /user/reset_password to Go URL: https://github.com/apache/trafficcontrol/issues/3842 This is an automated message from the Apache Git Service. To respond to the message, please log on