Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/858
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/858
manually closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/864
TPv2 - hides ds/user assign if use_tenancy is turned on
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/863
[TC-551] - adds the ability to filter parameters by name or config_file
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/860
[TC-535] - 2.1.x url sig tenancy checks
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/858
[BACKPORT] [TC-441] - refactored how tokin login works to ensure tokin
login is complete beâ¦
â¦fore attempting to resolve other dependencies that need the
authentication
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/857
[TC-441] - refactored how tokin login works to ensure tokin login is
complete beâ¦
â¦fore attempting to resolve other dependencies that need the
authentication cookie
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/847
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/847
manually closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/831#discussion_r134509030
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -69,14 +86,17 @@ sub copy_url_sig_keys {
else
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/850
[TC-483] - TPv2 - marks tenant as required for users and delivery services
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/848
[TC-520] - TPv2 - adds a table to view all delivery services for a given
tenant
as well as adding a tenant filter to the ds api
You can merge this pull request into a Git
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/847
[BACKPORT] [TC-441] - corrects reset password link to match format of TP
(cherry picked from commit 404b4a0362a38c2656b53cbe056b7675b902c162)
You can merge this pull request
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/844
[TC-441] - corrects reset password link to match format of TP
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134332821
--- Diff: traffic_ops/app/lib/TrafficOpsRoutes.pm ---
@@ -571,6 +571,11 @@ sub api_routes {
->to( 'KeysUrl
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134331397
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/deliveryServiceUrlSigKeys.tpl.html
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324897
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/DeliveryServiceUrlSigKeysController.js
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324139
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/DeliveryServiceUrlSigKeysController.js
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134323600
--- Diff:
traffic_portal/app/src/common/api/DeliveryServiceUrlSigKeysService.js ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324566
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/DeliveryServiceUrlSigKeysController.js
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324990
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/index.js
---
@@ -0,0 +1,43
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134325053
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/index.js
---
@@ -0,0 +1,43
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134322151
--- Diff:
traffic_portal/app/src/common/modules/form/deliveryService/FormDeliveryServiceController.js
---
@@ -169,9 +169,9 @@ var
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324745
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/DeliveryServiceUrlSigKeysController.js
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134324451
--- Diff:
traffic_portal/app/src/modules/private/configure/deliveryServices/urlSigKeys/DeliveryServiceUrlSigKeysController.js
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134323487
--- Diff:
traffic_portal/app/src/common/api/DeliveryServiceUrlSigKeysService.js ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/790#discussion_r134322763
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -26,6 +26,32 @@ use constant URL_SIG_KEYS_BUCKET
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/835
manually closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/835
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/840
manually closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/840
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/840
[BACKPORT] [TC-491] [TC-492]deletes applicable staticdnsentries and
jobs when a ds is deleted
cherry pick of b7b293718c0ca8174dba5759c01301208d5c6a78
You can merge
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/833
@dneuman64 will be looking at this one
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/838
[BACKPORT] [TC-441] - hooks up password reset to TP
(cherry picked from commit 421d390de80d5fa75a0d17488cf3a24bbe2fcc27)
You can merge this pull request into a Git repository
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/814
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/814
manually closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/834
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/834
[BACKPORT] [TC-533] - ssl_keys - tenancy checks
(cherry picked from commit 01452c99a0fc4d459742a67bad3bab9f20fcf4e6)
You can merge this pull request into a Git repository
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/831#discussion_r134007307
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -134,17 +154,22 @@ sub generate {
my $current_user
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/831#discussion_r134007130
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -69,14 +86,17 @@ sub copy_url_sig_keys {
else
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/831#discussion_r134007236
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -98,7 +118,7 @@ sub copy_url_sig_keys {
if ( defined
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/831#discussion_r134007263
--- Diff: traffic_ops/app/lib/API/DeliveryService/KeysUrlSig.pm ---
@@ -98,7 +118,7 @@ sub copy_url_sig_keys {
if ( defined
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/833
[TC-541] - TPv2 - turning on saveState for all tables
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/783
The ISO created was verified in this manner:
sudo mkdir /mnt/iso
sudo mount -t iso9660 -o loop this-is-the-iso-created.iso /mnt/iso/
df -h
cd /mnt/iso
Github user mitchell852 closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/821
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/821
manually closing this
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/818#discussion_r133571450
--- Diff: traffic_portal/app/src/common/modules/header/header.tpl.html ---
@@ -61,11 +61,11
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/825
[TC-441] - hooks up password reset to TP
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/824
[TC-491] [TC-492] - deletes applicable staticdnsentries and jobs when a ds
is deleted
You can merge this pull request into a Git repository by running:
$ git pull https
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/818
@dneuman64 will be looking at this one
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/822
[TC-82] - removes dead API routes that must have resurfaced due to a bad
conflict râ¦
â¦esolution
You can merge this pull request into a Git repository by running
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/754
and was this discussed on the ML because i'm a pretty hard -1 on more DS
table columns (if possible). That table is way out of control imo (50+ columns
already?). @knutsel did
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/821
[BACKPORT] [TC-521] TPv2 - accounts for missing dependency that centos 7.3
doesn't provide
⦠when centos 7.4 is available this dependency will be provided with the
distro
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/818
[TC-515] - TPv2 - adds human readable ds protocols to ds table
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/817
[TC-524] - hooks popout button to open new window/tab
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/815
[TC-521] - TPv2 - accounts for missing dependency that centos 7.3 doesn't
provide. whenâ¦
⦠centos 7.4 is available this dependency will be provided with the
distro.
You
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/814
[BACKPORT] [TC-519] - adds back missing tenants menu item
(cherry picked from commit a049589041ece51d6069cae09e2fdf23440d4a91)
You can merge this pull request into a Git
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/812
[TC-517] TP - displays users filtered by tenant. API adds tenant filter to
useâ¦
â¦rs api.
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/811
[TC-519] - adds back missing tenants menu item
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/790
you might want to consider adding a confirm dialog to the generate button.
i.e. "Are you sure you want to generate url sig keys?" - search for
dialog.confirm to
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/792
[Backport] [TC-504] - fixes number validation on global max settings
(cherry picked from commit 8603bc83dfe9b667af54f9101028740940bac86f)
You can merge this pull request
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/791
[TC-504] - fixes number validation on global max settings
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/787
TPv2 - adds unique query param value on build to force invalidation of
assets
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/784
[TC-423] - adds clone profile to TPv2 and adds a few missing change log
statements
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/783
[TC-437] - adds the ability to generate an ISO to the api / traffic portal
v2
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/782
updates tp dev install doc
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator-trafficcontrol
updates
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/777
[TC-498] - case mismatch was causing no validation on fullName when
updating current user
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/776
fixes tenant test
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator-trafficcontrol
fixes-tenant-test
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/766#discussion_r131236009
--- Diff: traffic_ops/app/lib/API/DeliveryService/SteeringTarget.pm ---
@@ -29,6 +30,20 @@ sub index {
my $steering_id = $self
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/753#discussion_r129939021
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -949,6 +959,8 @@ sub get_deliveryservices_by_userId {
my @data
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/753#discussion_r129940638
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -408,16 +423,24 @@ sub assign_deliveryservices {
if ( !defined($user
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/753#discussion_r129937967
--- Diff: traffic_ops/app/lib/API/DeliveryServiceUser.pm ---
@@ -31,6 +32,18 @@ sub delete {
return $self->forbid
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/753#discussion_r129937865
--- Diff: traffic_ops/app/lib/API/DeliveryServiceUser.pm ---
@@ -31,6 +32,18 @@ sub delete {
return $self->forbid
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/753#discussion_r129940738
--- Diff: traffic_ops/app/lib/Utils/Tenant.pm ---
@@ -224,6 +224,15 @@ sub is_ds_resource_accessible {
return $self
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/758#discussion_r131186253
--- Diff: traffic_ops/app/lib/API/Parameter.pm ---
@@ -317,7 +317,7 @@ sub update {
my $name = $params->{name} || $f
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/758#discussion_r131001497
--- Diff: traffic_ops/app/lib/API/Parameter.pm ---
@@ -317,7 +317,7 @@ sub update {
my $name = $params->{name} || $f
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/751#discussion_r130975670
--- Diff: traffic_ops/app/lib/API/DeliveryServiceRegexes.pm ---
@@ -222,6 +262,18 @@ sub delete {
return $self
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/770
[TC-486] - adds steering role to steering apis
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mitchell852/incubator
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r130410539
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -569,10 +575,17 @@ sub get_unassigned_servers_by_dsid {
$ds_server_criteria
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r130410587
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -726,41 +746,26 @@ sub get_servers_by_type {
my $orderby = $self
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r130407887
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -400,40 +401,25 @@ sub get_servers_by_status {
my $orderby
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r130407783
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -223,8 +240,19 @@ sub remove_server_from_ds {
my $ds_id
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r130407547
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -80,15 +86,21 @@ sub assign_servers_to_ds {
return
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/751#discussion_r130406581
--- Diff: traffic_ops/app/lib/API/DeliveryServiceRegexes.pm ---
@@ -222,6 +262,18 @@ sub delete {
return $self
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/751#discussion_r130405550
--- Diff: traffic_ops/app/lib/API/DeliveryServiceRegexes.pm ---
@@ -27,12 +28,18 @@ use Validate::Tiny ':all';
sub all
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/751#discussion_r130405449
--- Diff: traffic_ops/app/lib/API/DeliveryServiceMatches.pm ---
@@ -28,11 +29,19 @@ sub index {
my $format = $self->pa
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/767
TPv2 - disables target on update. a steering target is comprised of a
sourceâ¦
⦠and a target. changing the target represents a creation of a new target
rather than
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/761#discussion_r130380695
--- Diff: traffic_ops/client/delivery_service_endpoints.go ---
@@ -22,6 +22,10 @@ func deliveryServicesEp() string {
return
GitHub user mitchell852 opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/764
[TC-480] - adds exampleurls (aka edge fqdns) to ds view/edit form
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129934310
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -1021,6 +1030,10 @@ sub details
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129934238
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -947,6 +952,10 @@ sub details_v11 {
}
my
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129933537
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -569,10 +575,17 @@ sub get_unassigned_servers_by_dsid {
$ds_server_criteria
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129913844
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -27,6 +28,11 @@ sub index {
my @data;
my $orderby
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129913397
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -27,6 +28,11 @@ sub index {
my @data;
my $orderby
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129930025
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -223,8 +240,19 @@ sub remove_server_from_ds {
my $ds_id
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129914745
--- Diff: traffic_ops/app/lib/API/DeliveryServiceServer.pm ---
@@ -80,15 +86,21 @@ sub assign_servers_to_ds {
return
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129933912
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -726,41 +746,26 @@ sub get_servers_by_type {
my $orderby = $self
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129933611
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -569,10 +575,17 @@ sub get_unassigned_servers_by_dsid {
$ds_server_criteria
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129931534
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -445,18 +431,27 @@ sub get_servers_by_dsid {
my $orderby_snakecase
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/752#discussion_r129931600
--- Diff: traffic_ops/app/lib/API/Server.pm ---
@@ -489,11 +484,22 @@ sub get_edge_servers_by_dsid {
my $self= shift
1 - 100 of 417 matches
Mail list logo