Build failed in Jenkins: trafficcontrol-PR #1980

2018-07-02 Thread Apache Jenkins Server
See 

--
GitHub pull request #2475 of commit 383439429e3da5b1249afbda7935821ee1f88875, 
no merge conflicts.
Setting status of 383439429e3da5b1249afbda7935821ee1f88875 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1980/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 383439429e3da5b1249afbda7935821ee1f88875^{commit} # timeout=10
Checking out Revision 383439429e3da5b1249afbda7935821ee1f88875 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 383439429e3da5b1249afbda7935821ee1f88875
Commit message: "fix up places where IsTenancyEnabledTx now returns an error"
 > git rev-list --no-walk 383439429e3da5b1249afbda7935821ee1f88875 # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins62023509147454539.sh
++ echo jenkins-trafficcontrol-PR-1980
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1980
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-mZm7
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-zKh7
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-mZm7 -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 
0100   6170   6170 0   2037  0 --:--:-- --:--:-- --:--:--  2043
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
3 8079k3  322k0 0   222k  0  0:00:36  0:00:01  0:00:35  328k 70 
8079k   70 5718k0 0  2335k  0  0:00:03  0:00:02  0:00:01 2887k100 
8079k  100 8079k0 0  3106k  0  0:00:02  0:00:02 --:--:-- 3789k
+ chmod +x /tmp/docker-compose-mZm7
+ rm -rf dist
+ /tmp/docker-compose-mZm7 -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1980 up
Creating network "trafficcontrolpr1980_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401969608
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1980/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DylanVolz commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
DylanVolz commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401969216
 
 
   retest this please
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401968834
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1979/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1979

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[friede] remove DISCLAIMER -- no longer an incubating project

[friede] remove -incubating from source tarball name

[friede] remove remnants of "incubator"

[dewrich] Add TO Go delete deliveryservice_server

[dewrich] Add TO Go delete ds_servers client funcs, tests

[neuman] Fix TO CRConfig perms back to ops/readonly

[Dylan_Volz] convert all implementations of CRUD interfaces to use Keys methods

[Dylan_Volz] all packages through origins (alphabetically) refactored

[Dylan_Volz] converted all packages to use db-less interfaces

[Dylan_Volz] fix cachegroup where clause info to match select statement

[Dylan_Volz] remove nested tenancy check from deliveryservice request read 
method

[Dylan_Volz] fix bug with insert query introduced by conflict

[Dylan_Volz] fix tenants test by resetting id_seq and fixing error message

[Dylan_Volz] fix profiles by removing nested query with rows.Next loop and fix 
routes

[Dylan_Volz] run goimports on all files

[Dylan_Volz] convert tenancy package to new interfaces

[Dylan_Volz] define CRUDFactory function type to cleanup signatures

[Dylan_Volz] convert Validate functions to return a single error with all 
validation

[Dylan_Volz] documentation cleanup and code simplifications

[Dylan_Volz] fix tests broken by validation error change

[Dylan_Volz] fix up places where IsTenancyEnabledTx now returns an error

--
GitHub pull request #2475 of commit 383439429e3da5b1249afbda7935821ee1f88875, 
no merge conflicts.
Setting status of 383439429e3da5b1249afbda7935821ee1f88875 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1979/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 383439429e3da5b1249afbda7935821ee1f88875^{commit} # timeout=10
Checking out Revision 383439429e3da5b1249afbda7935821ee1f88875 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 383439429e3da5b1249afbda7935821ee1f88875
Commit message: "fix up places where IsTenancyEnabledTx now returns an error"
 > git rev-list --no-walk 3d169dc02c6ad3136e9228320c7c53c00a8839b3 # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins1270329895526359635.sh
++ echo jenkins-trafficcontrol-PR-1979
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1979
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-bcCU
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-bnDj
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-bcCU -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0100 
  6170   6170 0   1829  0 --:--:-- --:--:-- --:--:--  1825
  1 8079k1  134k0 0   103k  0  0:01:18  0:00:01  0:01:17  103k 
38 8079k   38 3075k0 0  1346k  0  0:00:06  0:00:02  0:00:04 
2988k100 8079k  100 8079k0 0  3012k  0  0:00:02  0:00:02 --:--:-- 
5757k
+ chmod +x /tmp/docker-compose-bcCU
+ rm -rf dist
+ /tmp/docker-compose-bcCU -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1979 up
Creating network "trafficcontrolpr1979_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] rawlinp opened a new issue #2496: TO Go PUT ASNs, regions, and tenants endpoints now require "id" field in request (possibly breaking clients)

2018-07-02 Thread GitBox
rawlinp opened a new issue #2496: TO Go PUT ASNs, regions, and tenants 
endpoints now require "id" field in request (possibly breaking clients)
URL: https://github.com/apache/trafficcontrol/issues/2496
 
 
   The following TO Go PUT endpoints now appear to require `"id": ` in the 
request body even though the Perl API did not require this (`id` is already a 
required path parameter):
   
   PUT /asns/{id} ("id" key/value omitted from request body):
   ```
   {
   "alerts": [
   {
   "level": "error",
   "text": "unable to parse required keys from request body"
   }
   ]
   }
   ```
   
   PUT /regions/{id} ("id" key/value omitted from request body):
   ```
   {
   "alerts": [
   {
   "level": "error",
   "text": "key in body does not match key in params"
   }
   ]
   }
   ```
   
   PUT /tenants/{id} ("id" key/value omitted from request body):
   ```
   {
   "alerts": [
   {
   "level": "error",
   "text": "key in body does not match key in params"
   }
   ]
   }
   ```
   
   Since `"id"` is now erroneously a required field, some clients might be 
broken; however, Traffic Portal is unaffected because it appears to always send 
the `"id"` field in PUT requests.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2018-07-02 Thread Apache Jenkins Server
See 




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

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[neuman] Fix TO CRConfig perms back to ops/readonly

--
[...truncated 18.35 MB...]
traffic_portal_build_1   | | +-- path-is-absolute@1.0.1 
traffic_portal_build_1   | | +-- prettyjson@1.2.1 
traffic_portal_build_1   | | | +-- colors@1.3.0 
traffic_portal_build_1   | | | `-- minimist@1.2.0 
traffic_portal_build_1   | | +-- shush@1.0.0 
traffic_portal_build_1   | | | +-- caller@0.0.1 
traffic_portal_build_1   | | | | `-- tape@2.3.3 
traffic_portal_build_1   | | | |   +-- deep-equal@0.1.2 
traffic_portal_build_1   | | | |   +-- defined@0.0.0 
traffic_portal_build_1   | | | |   +-- jsonify@0.0.0 
traffic_portal_build_1   | | | |   +-- resumer@0.0.0 
traffic_portal_build_1   | | | |   `-- through@2.3.8 
traffic_portal_build_1   | | | `-- strip-json-comments@0.1.3 
traffic_portal_build_1   | | +-- timespan@2.3.0 
traffic_portal_build_1   | | +-- utile@0.2.1 
traffic_portal_build_1   | | | +-- async@0.2.10 
traffic_portal_build_1   | | | +-- deep-equal@1.0.1 
traffic_portal_build_1   | | | +-- i@0.3.6 
traffic_portal_build_1   | | | +-- mkdirp@0.5.1 
traffic_portal_build_1   | | | | `-- minimist@0.0.8 
traffic_portal_build_1   | | | +-- ncp@0.4.2 
traffic_portal_build_1   | | | `-- rimraf@2.6.2 
traffic_portal_build_1   | | |   `-- glob@7.1.2 
traffic_portal_build_1   | | | +-- fs.realpath@1.0.0 
traffic_portal_build_1   | | | +-- inflight@1.0.6 
traffic_portal_build_1   | | | | `-- wrappy@1.0.2 
traffic_portal_build_1   | | | `-- once@1.4.0 
traffic_portal_build_1   | | `-- winston@0.8.3 
traffic_portal_build_1   | |   +-- cycle@1.0.3 
traffic_portal_build_1   | |   +-- isstream@0.1.2 
traffic_portal_build_1   | |   +-- pkginfo@0.3.1 
traffic_portal_build_1   | |   `-- stack-trace@0.0.10 
traffic_portal_build_1   | +-- morgan@1.7.0 
traffic_portal_build_1   | | `-- basic-auth@1.0.4 
traffic_portal_build_1   | +-- requirejs@2.1.22 
traffic_portal_build_1   | `-- ssl-root-cas@1.1.10 
traffic_portal_build_1   |   +-- es6-promise@2.3.0 
traffic_portal_build_1   |   `-- request@2.87.0 
traffic_portal_build_1   | +-- aws-sign2@0.7.0 
traffic_portal_build_1   | +-- aws4@1.7.0 
traffic_portal_build_1   | +-- caseless@0.12.0 
traffic_portal_build_1   | +-- combined-stream@1.0.6 
traffic_portal_build_1   | | `-- delayed-stream@1.0.0 
traffic_portal_build_1   | +-- extend@3.0.1 
traffic_portal_build_1   | +-- forever-agent@0.6.1 
traffic_portal_build_1   | +-- form-data@2.3.2 
traffic_portal_build_1   | | `-- asynckit@0.4.0 
traffic_portal_build_1   | +-- har-validator@5.0.3 
traffic_portal_build_1   | | +-- ajv@5.5.2 
traffic_portal_build_1   | | | +-- co@4.6.0 
traffic_portal_build_1   | | | +-- fast-deep-equal@1.1.0 
traffic_portal_build_1   | | | +-- fast-json-stable-stringify@2.0.0 
traffic_portal_build_1   | | | `-- json-schema-traverse@0.3.1 
traffic_portal_build_1   | | `-- har-schema@2.0.0 
traffic_portal_build_1   | +-- http-signature@1.2.0 
traffic_portal_build_1   | | +-- assert-plus@1.0.0 
traffic_portal_build_1   | | +-- jsprim@1.4.1 
traffic_portal_build_1   | | | +-- extsprintf@1.3.0 
traffic_portal_build_1   | | | +-- json-schema@0.2.3 
traffic_portal_build_1   | | | `-- verror@1.10.0 
traffic_portal_build_1   | | `-- sshpk@1.14.2 
traffic_portal_build_1   | |   +-- asn1@0.2.3 
traffic_portal_build_1   | |   +-- bcrypt-pbkdf@1.0.2 
traffic_portal_build_1   | |   +-- dashdash@1.14.1 
traffic_portal_build_1   | |   +-- ecc-jsbn@0.1.1 
traffic_portal_build_1   | |   +-- getpass@0.1.7 
traffic_portal_build_1   | |   +-- jsbn@0.1.1 
traffic_portal_build_1   | |   +-- safer-buffer@2.1.2 
traffic_portal_build_1   | |   `-- tweetnacl@0.14.5 
traffic_portal_build_1   | +-- is-typedarray@1.0.0 
traffic_portal_build_1   | +-- json-stringify-safe@5.0.1 
traffic_portal_build_1   | +-- oauth-sign@0.8.2 
traffic_portal_build_1   | +-- performance-now@2.1.0 
traffic_portal_build_1   | +-- qs@6.5.2 
traffic_portal_build_1   | +-- safe-buffer@5.1.2 
traffic_portal_build_1   | +-- tough-cookie@2.3.4 
traffic_portal_build_1   | | `-- punycode@1.4.1 
traffic_portal_build_1   | +-- tunnel-agent@0.6.0 
traffic_portal_build_1   | `-- uuid@3.3.2 
traffic_portal_build_1   | 
traffic_portal_build_1   | npm WARN optional SKIPPING OPTIONAL DEPENDENCY: 
fsevents@^1.0.0 (node_modules/chokidar/node_modules/fsevents):
traffic_portal_build_1   | npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: 
Unsupported platform for fsevents@1.2.4: wanted {"os":"darwin","arch":"any"} 
(current: {"os":"linux","arch":"x64"})
traffic_portal_build_1   | npm WARN traffic_portal@ No description
traffic_portal_build_1   | npm WARN traffic_portal@ No repository field.
traffic_portal_build_1   | npm WARN traffic_portal@ No license field.
tra

[GitHub] asfgit commented on issue #2353: Add TO Go regions/name

2018-07-02 Thread GitBox
asfgit commented on issue #2353: Add TO Go regions/name
URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401956095
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1978/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rob05c commented on issue #2353: Add TO Go regions/name

2018-07-02 Thread GitBox
rob05c commented on issue #2353: Add TO Go regions/name
URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401955221
 
 
   @dewrich Fixed


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dewrich edited a comment on issue #2353: Add TO Go regions/name

2018-07-02 Thread GitBox
dewrich edited a comment on issue #2353: Add TO Go regions/name
URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401954484
 
 
   @rob05c Doesn't compile either due to project rename changes: 
   
   `rob05c-to-go-regions-name-2353 ᐅ go build && ./traffic_ops_golang -cfg 
~/cdn.conf -dbcfg ~/database-traffic_ops.conf
   region/name.go:27:2: cannot find package 
"github.com/apache/incubator-trafficcontrol/lib/go-tc" in any of:

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (vendor tree)

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc

/usr/local/Cellar/go@1.9/1.9.7/libexec/src/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (from $GOROOT)

/Users/dricha209/projects/go/src/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (from $GOPATH)
   region/name.go:28:2: cannot find package 
"github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api" 
in any of:

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (vendor tree)

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api

/usr/local/Cellar/go@1.9/1.9.7/libexec/src/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (from $GOROOT)

/Users/dricha209/projects/go/src/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (from $GOPATH)
   `


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dewrich commented on issue #2353: Add TO Go regions/name

2018-07-02 Thread GitBox
dewrich commented on issue #2353: Add TO Go regions/name
URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401954484
 
 
   @rob05c Doesn't compile either due to package changes: 
```rob05c-to-go-regions-name-2353 ᐅ go build && ./traffic_ops_golang -cfg 
~/cdn.conf -dbcfg ~/database-traffic_ops.conf
   region/name.go:27:2: cannot find package 
"github.com/apache/incubator-trafficcontrol/lib/go-tc" in any of:

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (vendor tree)

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/vendor/github.com/apache/incubator-trafficcontrol/lib/go-tc

/usr/local/Cellar/go@1.9/1.9.7/libexec/src/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (from $GOROOT)

/Users/dricha209/projects/go/src/github.com/apache/incubator-trafficcontrol/lib/go-tc
 (from $GOPATH)
   region/name.go:28:2: cannot find package 
"github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api" 
in any of:

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (vendor tree)

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/traffic_ops/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api

/Users/dricha209/projects/go/src/github.com/apache/trafficcontrol/vendor/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api

/usr/local/Cellar/go@1.9/1.9.7/libexec/src/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (from $GOROOT)

/Users/dricha209/projects/go/src/github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/api
 (from $GOPATH)
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #737

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[neuman] Fix TO CRConfig perms back to ops/readonly

--
[...truncated 1.90 MB...]
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.0/health.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.0/ort.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/asn.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cachegroup.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cachegroupparameter.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cdn.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryservice/keys_url_sig.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryservice/ssl_keys.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryserviceserver.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/hwinfo.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/job.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/keys.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/log.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/metrics.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/parameter.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/phys_location.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/profile.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/region.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/roles.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/server.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/staticdns.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse err

[GitHub] dewrich edited a comment on issue #2339: Add TO Go users/id/deliveryservices

2018-07-02 Thread GitBox
dewrich edited a comment on issue #2339: Add TO Go users/id/deliveryservices
URL: https://github.com/apache/trafficcontrol/pull/2339#issuecomment-401953317
 
 
   API Tests are broken: ```Dewayne Richardson [4:09 PM]
   2339 the API tests are broken
   
   ```DEBUG: userdeliveryservices_test.go:33: 2018-07-02T16:09:12.882921-06:00: 
CreateTestDeliveryServices
   DEBUG: userdeliveryservices_test.go:35: 2018-07-02T16:09:12.9655-06:00: 
CreateTestUsersDeliveryServices
   --- FAIL: TestUsersDeliveryServices (0.85s)
   userdeliveryservices_test.go:65: cannot GET users: json: cannot 
unmarshal string into Go struct field User.id of type int
   FAIL
   exit status 1
   FAILgithub.com/apache/trafficcontrol/traffic_ops/testing/api/v13
1.038s```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dewrich commented on issue #2339: Add TO Go users/id/deliveryservices

2018-07-02 Thread GitBox
dewrich commented on issue #2339: Add TO Go users/id/deliveryservices
URL: https://github.com/apache/trafficcontrol/pull/2339#issuecomment-401953317
 
 
   API Tests are broken: ```Dewayne Richardson [4:09 PM]
   2339 the API tests are broken
   
   ```DEBUG: userdeliveryservices_test.go:33: 2018-07-02T16:09:12.882921-06:00: 
CreateTestDeliveryServices
   DEBUG: userdeliveryservices_test.go:35: 2018-07-02T16:09:12.9655-06:00: 
CreateTestUsersDeliveryServices
   --- FAIL: TestUsersDeliveryServices (0.85s)
   userdeliveryservices_test.go:65: cannot GET users: json: cannot 
unmarshal string into Go struct field User.id of type int
   FAIL
   exit status 1
   FAILgithub.com/apache/trafficcontrol/traffic_ops/testing/api/v13
1.038s``


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] jrushford commented on issue #2485: Modify the grove rpm version format

2018-07-02 Thread GitBox
jrushford commented on issue #2485: Modify the grove rpm version format
URL: https://github.com/apache/trafficcontrol/pull/2485#issuecomment-401953221
 
 
   @dangogh  I re-worked the build_rpm.sh to use the functions.sh and build the 
rpm like the others are built.  The grove version number is different however 
and that should not be a problem.  the build number and git hash follows the 
same convention as other tc rpmbuilds.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2485: Modify the grove rpm version format

2018-07-02 Thread GitBox
asfgit commented on issue #2485: Modify the grove rpm version format
URL: https://github.com/apache/trafficcontrol/pull/2485#issuecomment-401952657
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1977/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1977

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[dewrich] Add TO Go delete deliveryservice_server

[dewrich] Add TO Go delete ds_servers client funcs, tests

[jrushford] Modify the grove/build/build_rpm.sh script to incorporate the last 9

--
GitHub pull request #2485 of commit e4d1b6833778672fc520fb67c95366f087306ee3, 
no merge conflicts.
Setting status of e4d1b6833778672fc520fb67c95366f087306ee3 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1977/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse e4d1b6833778672fc520fb67c95366f087306ee3^{commit} # timeout=10
Checking out Revision e4d1b6833778672fc520fb67c95366f087306ee3 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f e4d1b6833778672fc520fb67c95366f087306ee3
Commit message: "Modify the grove/build/build_rpm.sh script to incorporate the 
last 9 digit commit hash on the grove/ directory as the release when building 
the grove rpm."
 > git rev-list --no-walk e6ca7a5ecd152c56efc2c8bc36c10fe9bbb293fe # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins6514448189385948403.sh
++ echo jenkins-trafficcontrol-PR-1977
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1977
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-ZYd7
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-gp6l
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-ZYd7 -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0100 
  6170   6170 0   1888  0 --:--:-- --:--:-- --:--:--  1886
  1 8079k1 855460 0  79528  0  0:01:44  0:00:01  0:01:43 79528 
27 8079k   27 2191k0 0  1068k  0  0:00:07  0:00:02  0:00:05 
2159k100 8079k  100 8079k0 0  3140k  0  0:00:02  0:00:02 --:--:-- 
5344k
+ chmod +x /tmp/docker-compose-ZYd7
+ rm -rf dist
+ /tmp/docker-compose-ZYd7 -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1977 up
Creating network "trafficcontrolpr1977_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] dneuman64 closed pull request #2495: Fix TO CRConfig perms back to ops/readonly

2018-07-02 Thread GitBox
dneuman64 closed pull request #2495: Fix TO CRConfig perms back to ops/readonly
URL: https://github.com/apache/trafficcontrol/pull/2495
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/traffic_ops/traffic_ops_golang/crconfig/handler.go 
b/traffic_ops/traffic_ops_golang/crconfig/handler.go
index d225bfe45..b1cafa19e 100644
--- a/traffic_ops/traffic_ops_golang/crconfig/handler.go
+++ b/traffic_ops/traffic_ops_golang/crconfig/handler.go
@@ -36,8 +36,6 @@ import (
"github.com/jmoiron/sqlx"
 )
 
-const PrivLevel = auth.PrivLevelAdmin
-
 // Handler creates and serves the CRConfig from the raw SQL data.
 // This MUST only be used for debugging or previewing, the raw un-snapshotted 
data MUST NOT be used by any component of the CDN.
 func Handler(db *sqlx.DB, cfg config.Config) http.HandlerFunc {
diff --git a/traffic_ops/traffic_ops_golang/routes.go 
b/traffic_ops/traffic_ops_golang/routes.go
index 91d847c96..5650f9de5 100644
--- a/traffic_ops/traffic_ops_golang/routes.go
+++ b/traffic_ops/traffic_ops_golang/routes.go
@@ -291,10 +291,10 @@ func Routes(d ServerData) ([]Route, []RawRoute, 
http.Handler, error) {
{1.1, http.MethodDelete, `tenants/{id}$`, 
api.DeleteHandler(tenant.GetRefType(), d.DB), auth.PrivLevelOperations, 
Authenticated, nil},
 
//CRConfig
-   {1.1, http.MethodGet, `cdns/{cdn}/snapshot/?$`, 
crconfig.SnapshotGetHandler(d.DB, d.Config), crconfig.PrivLevel, Authenticated, 
nil},
-   {1.1, http.MethodGet, `cdns/{cdn}/snapshot/new/?$`, 
crconfig.Handler(d.DB, d.Config), crconfig.PrivLevel, Authenticated, nil},
-   {1.1, http.MethodPut, `cdns/{id}/snapshot/?$`, 
crconfig.SnapshotHandler(d.DB, d.Config), crconfig.PrivLevel, Authenticated, 
nil},
-   {1.1, http.MethodPut, `snapshot/{cdn}/?$`, 
crconfig.SnapshotHandler(d.DB, d.Config), crconfig.PrivLevel, Authenticated, 
nil},
+   {1.1, http.MethodGet, `cdns/{cdn}/snapshot/?$`, 
crconfig.SnapshotGetHandler(d.DB, d.Config), auth.PrivLevelReadOnly, 
Authenticated, nil},
+   {1.1, http.MethodGet, `cdns/{cdn}/snapshot/new/?$`, 
crconfig.Handler(d.DB, d.Config), auth.PrivLevelReadOnly, Authenticated, nil},
+   {1.1, http.MethodPut, `cdns/{id}/snapshot/?$`, 
crconfig.SnapshotHandler(d.DB, d.Config), auth.PrivLevelOperations, 
Authenticated, nil},
+   {1.1, http.MethodPut, `snapshot/{cdn}/?$`, 
crconfig.SnapshotHandler(d.DB, d.Config), auth.PrivLevelOperations, 
Authenticated, nil},
 
//SSLKeys deliveryservice endpoints here that are marked  
marked as '-wip' need to have tenancy checks added
{1.3, http.MethodGet, 
`deliveryservices-wip/xmlId/{xmlID}/sslkeys$`, 
getDeliveryServiceSSLKeysByXMLIDHandler, auth.PrivLevelAdmin, Authenticated, 
nil},
@@ -320,9 +320,9 @@ func Routes(d ServerData) ([]Route, []RawRoute, 
http.Handler, error) {
// rawRoutes are served at the root path. These should be almost 
exclusively old Perl pre-API routes, which have yet to be converted in all 
clients. New routes should be in the versioned API path.
rawRoutes := []RawRoute{
// DEPRECATED - use PUT /api/1.2/snapshot/{cdn}
-   {http.MethodGet, `tools/write_crconfig/{cdn}/?$`, 
crconfig.SnapshotOldGUIHandler(d.DB, d.Config), crconfig.PrivLevel, 
Authenticated, nil},
+   {http.MethodGet, `tools/write_crconfig/{cdn}/?$`, 
crconfig.SnapshotOldGUIHandler(d.DB, d.Config), auth.PrivLevelOperations, 
Authenticated, nil},
// DEPRECATED - use GET /api/1.2/cdns/{cdn}/snapshot
-   {http.MethodGet, `CRConfig-Snapshots/{cdn}/CRConfig.json?$`, 
crconfig.SnapshotOldGetHandler(d.DB, d.Config), crconfig.PrivLevel, 
Authenticated, nil},
+   {http.MethodGet, `CRConfig-Snapshots/{cdn}/CRConfig.json?$`, 
crconfig.SnapshotOldGetHandler(d.DB, d.Config), auth.PrivLevelReadOnly, 
Authenticated, nil},
}
 
return routes, rawRoutes, proxyHandler, nil


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1976

2018-07-02 Thread Apache Jenkins Server
See 

--
GitHub pull request #2495 of commit d4298d654280fa8fc4702d8910c2f69e3e33b9ce, 
no merge conflicts.
Setting status of d4298d654280fa8fc4702d8910c2f69e3e33b9ce to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1976/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
 > git rev-parse origin/d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # 
 > timeout=10
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Retrying after 10 seconds
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
 > git rev-parse origin/d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # 
 > timeout=10
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Retrying after 10 seconds
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
 > git rev-parse origin/d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # 
 > timeout=10
 > git rev-parse d4298d654280fa8fc4702d8910c2f69e3e33b9ce^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Archiving artifacts


[GitHub] asfgit commented on issue #2495: Fix TO CRConfig perms back to ops/readonly

2018-07-02 Thread GitBox
asfgit commented on issue #2495: Fix TO CRConfig perms back to ops/readonly
URL: https://github.com/apache/trafficcontrol/pull/2495#issuecomment-401952561
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1976/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rob05c opened a new pull request #2495: Fix TO CRConfig perms back to ops/readonly

2018-07-02 Thread GitBox
rob05c opened a new pull request #2495: Fix TO CRConfig perms back to 
ops/readonly
URL: https://github.com/apache/trafficcontrol/pull/2495
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rawlinp opened a new issue #2494: Multiple Go API endpoints don't return "joined" fields on PUT/POST

2018-07-02 Thread GitBox
rawlinp opened a new issue #2494: Multiple Go API endpoints don't return 
"joined" fields on PUT/POST
URL: https://github.com/apache/trafficcontrol/issues/2494
 
 
   This is a regression from the Perl API, where keys like 
`parentCachegroupName` are returned with the name of the cachegroup rather than 
`null`. This requires following up `INSERT` or `UPDATE` SQL statements with a 
`SELECT` that returns the proper fields. This is essentially what the Perl API 
did through the DBIx ORM for these endpoints, but currently these APIs don't 
follow up with a `SELECT` to return the same data as the Perl API and return a 
`null` value in the response:
   
   - POST, PUT /api/1.1/cachegroups
 - parentCachegroupName, secondaryParentCachegroupName, typeName
   - POST, PUT /api/1.1/asns
 - cachegroup
   - POST, PUT /api/1.3/deliveryservices
 - cdnName, profileName, profileDescription
   - POST, PUT /api/1.1/regions
 - divisionName
   - POST, PUT /api/1.1/phys_locations
 - region
   - POST, PUT /api/1.1/servers
 - cachegroup, cdnName, physLocation, profile, profileDesc, status, type
   - POST, PUT /api/1.1/tenants
 - parentName
   
   Example PUT /cachegroups/{id} response from traffic_ops_golang:
   ```
   {
   "alerts": [
   {
   "level": "success",
   "text": "cachegroup was updated."
   }
   ],
   "response": {
   "fallbackToClosest": null,
   "id": 531,
   "lastUpdated": "2018-07-02 17:07:59+00",
   "latitude": 39.739167,
   "longitude": -104.984722,
   "name": "rawl-cg",
   "parentCachegroupId": 3,
   "parentCachegroupName": null,
   "secondaryParentCachegroupId": null,
   "secondaryParentCachegroupName": null,
   "shortName": "rawl-cg",
   "typeId": 6,
   "typeName": null
   }
   }
   ```
   Example PUT /cachegroups/{id} response from Perl TO with the same request 
payload:
   ```
   {
   "alerts": [
   {
   "level": "success",
   "text": "Cachegroup update was successful."
   }
   ],
   "response": {
   "fallbackToClosest": 1,
   "id": 532,
   "lastUpdated": "2018-07-02 21:42:40.062963+00",
   "latitude": 39.739167,
   "longitude": -104.984722,
   "name": "rawl-cg",
   "parentCachegroupId": 3,
   "parentCachegroupName": "mid-west",
   "secondaryParentCachegroupId": null,
   "secondaryParentCachegroupName": null,
   "shortName": "rawl-cg",
   "typeId": 6,
   "typeName": "EDGE_LOC"
   }
   }
   ```
   Notice how the Perl API returns all the `fooName` fields whereas the Go API 
returns `null`.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401941178
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1975/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2018-07-02 Thread Apache Jenkins Server
See 




[GitHub] mitchell852 commented on a change in pull request #2307: Add TO Go cdns/routing

2018-07-02 Thread GitBox
mitchell852 commented on a change in pull request #2307: Add TO Go cdns/routing
URL: https://github.com/apache/trafficcontrol/pull/2307#discussion_r199622126
 
 

 ##
 File path: traffic_ops/traffic_ops_golang/routes.go
 ##
 @@ -104,7 +104,7 @@ func Routes(d ServerData) ([]Route, []RawRoute, 
http.Handler, error) {
{1.1, http.MethodGet, `cdns/configs$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
{1.1, http.MethodGet, `cdns/domains$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
{1.1, http.MethodGet, `cdns/health$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
-   {1.1, http.MethodGet, `cdns/routing$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
 
 Review comment:
   yes, because TO_golang has no concept of to_extensions (at the moment), 
merging this PR  would kill the perl to_extension that is currently being used 
by Comcast in production.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2339: Add TO Go users/id/deliveryservices

2018-07-02 Thread GitBox
asfgit commented on issue #2339: Add TO Go users/id/deliveryservices
URL: https://github.com/apache/trafficcontrol/pull/2339#issuecomment-401934781
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1974/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1974

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[rob] Add TO Go users/id/deliveryservices

[rob] Add TO Go deliveryservice/users tests

--
GitHub pull request #2339 of commit d477e6ae483272900cbd9952c3b69f152ca5d91e, 
no merge conflicts.
Setting status of d477e6ae483272900cbd9952c3b69f152ca5d91e to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1974/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse d477e6ae483272900cbd9952c3b69f152ca5d91e^{commit} # timeout=10
Checking out Revision d477e6ae483272900cbd9952c3b69f152ca5d91e (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f d477e6ae483272900cbd9952c3b69f152ca5d91e
Commit message: "Add TO Go deliveryservice/users tests"
 > git rev-list --no-walk 9336d7863413112986b8cab44027bfab25141d88 # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins4871585532748801842.sh
++ echo jenkins-trafficcontrol-PR-1974
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1974
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-EJnC
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-NKhl
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-EJnC -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0100 
  6170   6170 0   1791  0 --:--:-- --:--:-- --:--:--  1788
  2 8079k2  220k0 0   157k  0  0:00:51  0:00:01  0:00:50  157k 
47 8079k   47 3831k0 0  1622k  0  0:00:04  0:00:02  0:00:02 
3741k100 8079k  100 8079k0 0  3041k  0  0:00:02  0:00:02 --:--:-- 
6242k
+ chmod +x /tmp/docker-compose-EJnC
+ rm -rf dist
+ /tmp/docker-compose-EJnC -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1974 up
Creating network "trafficcontrolpr1974_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] dewrich commented on a change in pull request #2307: Add TO Go cdns/routing

2018-07-02 Thread GitBox
dewrich commented on a change in pull request #2307: Add TO Go cdns/routing
URL: https://github.com/apache/trafficcontrol/pull/2307#discussion_r199619450
 
 

 ##
 File path: traffic_ops/traffic_ops_golang/routes.go
 ##
 @@ -104,7 +104,7 @@ func Routes(d ServerData) ([]Route, []RawRoute, 
http.Handler, error) {
{1.1, http.MethodGet, `cdns/configs$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
{1.1, http.MethodGet, `cdns/domains$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
{1.1, http.MethodGet, `cdns/health$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
-   {1.1, http.MethodGet, `cdns/routing$`, 
handlerToFunc(proxyHandler), 0, NoAuth, []Middleware{}},
 
 Review comment:
   We need to solve custom TO extensions or the ability to override for custom 
project's needs


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1973

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[rob] Add TO Go cdns/capacity

--
GitHub pull request #2306 of commit 9336d7863413112986b8cab44027bfab25141d88, 
no merge conflicts.
Setting status of 9336d7863413112986b8cab44027bfab25141d88 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1973/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 9336d7863413112986b8cab44027bfab25141d88^{commit} # timeout=10
Checking out Revision 9336d7863413112986b8cab44027bfab25141d88 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 9336d7863413112986b8cab44027bfab25141d88
Commit message: "Add TO Go cdns/capacity"
 > git rev-list --no-walk 23c733dea48f77ddb925df9044997f253e33a395 # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins2035395800076010643.sh
++ echo jenkins-trafficcontrol-PR-1973
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1973
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-9nCK
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-tpEB
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-9nCK -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0100 
  6170   6170 0   1446  0 --:--:-- --:--:-- --:--:--  1444
  2 8079k2  203k0 0   141k  0  0:00:57  0:00:01  0:00:56  141k 
51 8079k   51 4146k0 0  1713k  0  0:00:04  0:00:02  0:00:02 
4023k100 8079k  100 8079k0 0  2981k  0  0:00:02  0:00:02 --:--:-- 
6206k
+ chmod +x /tmp/docker-compose-9nCK
+ rm -rf dist
+ /tmp/docker-compose-9nCK -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1973 up
Creating network "trafficcontrolpr1973_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2306: Add TO Go cdns/capacity

2018-07-02 Thread GitBox
asfgit commented on issue #2306: Add TO Go cdns/capacity
URL: https://github.com/apache/trafficcontrol/pull/2306#issuecomment-401929962
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1973/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1972

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[rob05c] Conversion Utility to modify Traffic Ops profile based on conversion

[friede] remove DISCLAIMER -- no longer an incubating project

[friede] remove -incubating from source tarball name

[friede] remove remnants of "incubator"

[dewrich] Add TO Go delete deliveryservice_server

[dewrich] Add TO Go delete ds_servers client funcs, tests

[rawlin_peters] Refactor cachegroups to use coordinates table

--
GitHub pull request #2492 of commit 23c733dea48f77ddb925df9044997f253e33a395, 
no merge conflicts.
Setting status of 23c733dea48f77ddb925df9044997f253e33a395 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1972/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 23c733dea48f77ddb925df9044997f253e33a395^{commit} # timeout=10
Checking out Revision 23c733dea48f77ddb925df9044997f253e33a395 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 23c733dea48f77ddb925df9044997f253e33a395
Commit message: "Refactor cachegroups to use coordinates table"
 > git rev-list --no-walk cd9aca15982e6fc4ea47c39638b39248dcd93b7a # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins7725802179526070543.sh
++ echo jenkins-trafficcontrol-PR-1972
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1972
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-y8hX
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-X9Uc
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-y8hX -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 
0100   6170   6170 0   1754  0 --:--:-- --:--:-- --:--:--  1757
  0 8079k0 00 0  0  0 --:--:-- --:--:-- --:--:-- 0 
14 8079k   14 1188k0 0   654k  0  0:00:12  0:00:01  0:00:11 
1236k100 8079k  100 8079k0 0  3225k  0  0:00:02  0:00:02 --:--:-- 
4893k
+ chmod +x /tmp/docker-compose-y8hX
+ rm -rf dist
+ /tmp/docker-compose-y8hX -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1972 up
Creating network "trafficcontrolpr1972_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2492: Refactor cachegroups to use coordinates table

2018-07-02 Thread GitBox
asfgit commented on issue #2492: Refactor cachegroups to use coordinates table
URL: https://github.com/apache/trafficcontrol/pull/2492#issuecomment-401929841
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1972/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #736

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[dewrich] Add TO Go delete deliveryservice_server

[dewrich] Add TO Go delete ds_servers client funcs, tests

--
[...truncated 1.91 MB...]
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.0/health.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.0/ort.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/asn.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cachegroup.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cachegroupparameter.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/cdn.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryservice/keys_url_sig.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryservice/ssl_keys.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/deliveryserviceserver.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/hwinfo.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/job.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/keys.t  
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/log.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/metrics.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/parameter.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/phys_location.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/profile.t   
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/region.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/roles.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/server.t
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |   Non-zero exit status: 255
unit_1 |   Parse errors: No plan found in TAP output
unit_1 | /opt/traffic_ops/app/t/api/1.1/staticdns.t 
(Wstat: 65280 Tests: 0 Failed: 0)
unit_1 |

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

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[dewrich] Add TO Go delete deliveryservice_server

[dewrich] Add TO Go delete ds_servers client funcs, tests

--
[...truncated 19.27 MB...]
traffic_portal_build_1   | | +-- path-is-absolute@1.0.1 
traffic_portal_build_1   | | +-- prettyjson@1.2.1 
traffic_portal_build_1   | | | +-- colors@1.3.0 
traffic_portal_build_1   | | | `-- minimist@1.2.0 
traffic_portal_build_1   | | +-- shush@1.0.0 
traffic_portal_build_1   | | | +-- caller@0.0.1 
traffic_portal_build_1   | | | | `-- tape@2.3.3 
traffic_portal_build_1   | | | |   +-- deep-equal@0.1.2 
traffic_portal_build_1   | | | |   +-- defined@0.0.0 
traffic_portal_build_1   | | | |   +-- jsonify@0.0.0 
traffic_portal_build_1   | | | |   +-- resumer@0.0.0 
traffic_portal_build_1   | | | |   `-- through@2.3.8 
traffic_portal_build_1   | | | `-- strip-json-comments@0.1.3 
traffic_portal_build_1   | | +-- timespan@2.3.0 
traffic_portal_build_1   | | +-- utile@0.2.1 
traffic_portal_build_1   | | | +-- async@0.2.10 
traffic_portal_build_1   | | | +-- deep-equal@1.0.1 
traffic_portal_build_1   | | | +-- i@0.3.6 
traffic_portal_build_1   | | | +-- mkdirp@0.5.1 
traffic_portal_build_1   | | | | `-- minimist@0.0.8 
traffic_portal_build_1   | | | +-- ncp@0.4.2 
traffic_portal_build_1   | | | `-- rimraf@2.6.2 
traffic_portal_build_1   | | |   `-- glob@7.1.2 
traffic_portal_build_1   | | | +-- fs.realpath@1.0.0 
traffic_portal_build_1   | | | +-- inflight@1.0.6 
traffic_portal_build_1   | | | | `-- wrappy@1.0.2 
traffic_portal_build_1   | | | `-- once@1.4.0 
traffic_portal_build_1   | | `-- winston@0.8.3 
traffic_portal_build_1   | |   +-- cycle@1.0.3 
traffic_portal_build_1   | |   +-- isstream@0.1.2 
traffic_portal_build_1   | |   +-- pkginfo@0.3.1 
traffic_portal_build_1   | |   `-- stack-trace@0.0.10 
traffic_portal_build_1   | +-- morgan@1.7.0 
traffic_portal_build_1   | | `-- basic-auth@1.0.4 
traffic_portal_build_1   | +-- requirejs@2.1.22 
traffic_portal_build_1   | `-- ssl-root-cas@1.1.10 
traffic_portal_build_1   |   +-- es6-promise@2.3.0 
traffic_portal_build_1   |   `-- request@2.87.0 
traffic_portal_build_1   | +-- aws-sign2@0.7.0 
traffic_portal_build_1   | +-- aws4@1.7.0 
traffic_portal_build_1   | +-- caseless@0.12.0 
traffic_portal_build_1   | +-- combined-stream@1.0.6 
traffic_portal_build_1   | | `-- delayed-stream@1.0.0 
traffic_portal_build_1   | +-- extend@3.0.1 
traffic_portal_build_1   | +-- forever-agent@0.6.1 
traffic_portal_build_1   | +-- form-data@2.3.2 
traffic_portal_build_1   | | `-- asynckit@0.4.0 
traffic_portal_build_1   | +-- har-validator@5.0.3 
traffic_portal_build_1   | | +-- ajv@5.5.2 
traffic_portal_build_1   | | | +-- co@4.6.0 
traffic_portal_build_1   | | | +-- fast-deep-equal@1.1.0 
traffic_portal_build_1   | | | +-- fast-json-stable-stringify@2.0.0 
traffic_portal_build_1   | | | `-- json-schema-traverse@0.3.1 
traffic_portal_build_1   | | `-- har-schema@2.0.0 
traffic_portal_build_1   | +-- http-signature@1.2.0 
traffic_portal_build_1   | | +-- assert-plus@1.0.0 
traffic_portal_build_1   | | +-- jsprim@1.4.1 
traffic_portal_build_1   | | | +-- extsprintf@1.3.0 
traffic_portal_build_1   | | | +-- json-schema@0.2.3 
traffic_portal_build_1   | | | `-- verror@1.10.0 
traffic_portal_build_1   | | `-- sshpk@1.14.2 
traffic_portal_build_1   | |   +-- asn1@0.2.3 
traffic_portal_build_1   | |   +-- bcrypt-pbkdf@1.0.2 
traffic_portal_build_1   | |   +-- dashdash@1.14.1 
traffic_portal_build_1   | |   +-- ecc-jsbn@0.1.1 
traffic_portal_build_1   | |   +-- getpass@0.1.7 
traffic_portal_build_1   | |   +-- jsbn@0.1.1 
traffic_portal_build_1   | |   +-- safer-buffer@2.1.2 
traffic_portal_build_1   | |   `-- tweetnacl@0.14.5 
traffic_portal_build_1   | +-- is-typedarray@1.0.0 
traffic_portal_build_1   | +-- json-stringify-safe@5.0.1 
traffic_portal_build_1   | +-- oauth-sign@0.8.2 
traffic_portal_build_1   | +-- performance-now@2.1.0 
traffic_portal_build_1   | +-- qs@6.5.2 
traffic_portal_build_1   | +-- safe-buffer@5.1.2 
traffic_portal_build_1   | +-- tough-cookie@2.3.4 
traffic_portal_build_1   | | `-- punycode@1.4.1 
traffic_portal_build_1   | +-- tunnel-agent@0.6.0 
traffic_portal_build_1   | `-- uuid@3.3.2 
traffic_portal_build_1   | 
traffic_portal_build_1   | npm WARN optional SKIPPING OPTIONAL DEPENDENCY: 
fsevents@^1.0.0 (node_modules/chokidar/node_modules/fsevents):
traffic_portal_build_1   | npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: 
Unsupported platform for fsevents@1.2.4: wanted {"os":"darwin","arch":"any"} 
(current: {"os":"linux","arch":"x64"})
traffic_portal_build_1   | npm WARN traffic_portal@ No description
traffic_portal_build_1   | npm WARN traffic_portal@ No repository field.
traffic_portal_b

[GitHub] dewrich closed pull request #2441: Add TO Go delete deliveryservice_server

2018-07-02 Thread GitBox
dewrich closed pull request #2441: Add TO Go delete deliveryservice_server
URL: https://github.com/apache/trafficcontrol/pull/2441
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/lib/go-tc/deliveryservices.go b/lib/go-tc/deliveryservices.go
index 9f8f56e0e..6f6093f59 100644
--- a/lib/go-tc/deliveryservices.go
+++ b/lib/go-tc/deliveryservices.go
@@ -543,3 +543,9 @@ type DeliveryServiceRouting struct {
RegionalAlternate int `json:"regionalAlternate"`
RegionalDeniedint `json:"regionalDenied"`
 }
+
+type DSServerIDs struct {
+   DeliveryServiceID *int  `json:"dsId", db:"deliveryservice"`
+   ServerIDs []int `json:"servers"`
+   Replace   *bool `json:"replace"`
+}
diff --git a/traffic_ops/client/v13/deliveryserviceserver.go 
b/traffic_ops/client/v13/deliveryserviceserver.go
new file mode 100644
index 0..75ef91222
--- /dev/null
+++ b/traffic_ops/client/v13/deliveryserviceserver.go
@@ -0,0 +1,67 @@
+/*
+
+   Licensed under the Apache License, Version 2.0 (the "License");
+   you may not use this file except in compliance with the License.
+   You may obtain a copy of the License at
+
+   http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+*/
+
+package v13
+
+import (
+   "encoding/json"
+   "strconv"
+
+   "github.com/apache/trafficcontrol/lib/go-tc"
+   "github.com/apache/trafficcontrol/lib/go-util"
+)
+
+func (to *Session) GetDeliveryServiceServers() ([]tc.DeliveryServiceServer, 
ReqInf, error) {
+   path := apiBase + `/deliveryserviceserver`
+   // deliveryService
+   resp := tc.DeliveryServiceServerResponse{}
+   reqInf, err := get(to, path, &resp)
+   if err != nil {
+   return nil, reqInf, err
+   }
+   return resp.Response, reqInf, nil
+}
+
+// CreateDeliveryServiceServers associates the given servers with the given 
delivery services. If replace is true, it deletes any existing associations for 
the given delivery service.
+func (to *Session) CreateDeliveryServiceServers(dsID int, serverIDs []int, 
replace bool) (*tc.DSServerIDs, error) {
+   path := apiBase + `/deliveryserviceserver`
+   req := tc.DSServerIDs{
+   DeliveryServiceID: util.IntPtr(dsID),
+   ServerIDs: serverIDs,
+   Replace:   util.BoolPtr(replace),
+   }
+   jsonReq, err := json.Marshal(&req)
+   if err != nil {
+   return nil, err
+   }
+   resp := struct {
+   Response tc.DSServerIDs `json:"response"`
+   }{}
+   if err := post(to, path, jsonReq, &resp); err != nil {
+   return nil, err
+   }
+   return &resp.Response, nil
+}
+
+// DeleteDeliveryService deletes the given delivery service server association.
+func (to *Session) DeleteDeliveryServiceServer(dsID int, serverID int) error {
+   path := apiBase + `/deliveryservice_server/` + strconv.Itoa(dsID) + `/` 
+ strconv.Itoa(serverID)
+   resp := struct{ tc.Alerts }{}
+   err := del(to, path, &resp)
+   if err != nil {
+   return err
+   }
+   return nil
+}
diff --git a/traffic_ops/testing/api/v13/deliveryserviceservers_test.go 
b/traffic_ops/testing/api/v13/deliveryserviceservers_test.go
new file mode 100644
index 0..2c60ae15d
--- /dev/null
+++ b/traffic_ops/testing/api/v13/deliveryserviceservers_test.go
@@ -0,0 +1,111 @@
+package v13
+
+/*
+
+   Licensed under the Apache License, Version 2.0 (the "License");
+   you may not use this file except in compliance with the License.
+   You may obtain a copy of the License at
+
+   http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+*/
+
+import (
+   "testing"
+
+   "github.com/apache/trafficcontrol/lib/go-log"
+)
+
+func TestDeliveryServiceServers(t *testing.T) {
+   CreateTestCDNs(t)
+   CreateTestTypes(t)
+   CreateTestProfiles(t)
+   CreateTestStatuses(t)
+   CreateTestDivisions(t)
+   CreateTestRegions(t)
+   CreateTestPhysLocations(t)
+   CreateTestCacheGroups(t)
+   CreateTestServers(t)
+   CreateTestDeliveryServices(t)
+
+  

[GitHub] asfgit commented on issue #2493: testing passing contents of a file from the server to TP and promptin…

2018-07-02 Thread GitBox
asfgit commented on issue #2493: testing passing contents of a file from the 
server to TP and promptin…
URL: https://github.com/apache/trafficcontrol/pull/2493#issuecomment-401914171
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1971/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1971

2018-07-02 Thread Apache Jenkins Server
See 


Changes:

[mitchell852] testing passing contents of a file from the server to TP and 
prompting a

--
GitHub pull request #2493 of commit cd9aca15982e6fc4ea47c39638b39248dcd93b7a, 
no merge conflicts.
Setting status of cd9aca15982e6fc4ea47c39638b39248dcd93b7a to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1971/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse cd9aca15982e6fc4ea47c39638b39248dcd93b7a^{commit} # timeout=10
Checking out Revision cd9aca15982e6fc4ea47c39638b39248dcd93b7a (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f cd9aca15982e6fc4ea47c39638b39248dcd93b7a
Commit message: "testing passing contents of a file from the server to TP and 
prompting a download dialog"
 > git rev-list --no-walk 134bb62e4f865a77435592f602146e05576afabd # timeout=10
[trafficcontrol-PR] $ /bin/bash /tmp/jenkins4451844422701584350.sh
++ echo jenkins-trafficcontrol-PR-1971
++ sed s/-//g
++ sed s/jenkins//
+ proj=trafficcontrolPR1971
+ yml=infrastructure/docker/build/docker-compose.yml
++ mktemp /tmp/docker-compose-
+ dc=/tmp/docker-compose-XtVK
++ mktemp /tmp/tc-status-
+ st=/tmp/tc-status-sziB
++ uname -s
++ uname -m
+ curl -o /tmp/docker-compose-XtVK -L 
https://github.com/docker/compose/releases/download/1.13.0/docker-compose-Linux-x86_64
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
  0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0  
0 00 00 0  0  0 --:--:-- --:--:-- --:--:-- 0100 
  6170   6170 0   1878  0 --:--:-- --:--:-- --:--:--  1875
  1 8079k1  100k0 0  88796  0  0:01:33  0:00:01  0:01:32 88796 
20 8079k   20 1664k0 0   782k  0  0:00:10  0:00:02  0:00:08 
1614k100 8079k  100 8079k0 0  2975k  0  0:00:02  0:00:02 --:--:-- 
5131k
+ chmod +x /tmp/docker-compose-XtVK
+ rm -rf dist
+ /tmp/docker-compose-XtVK -f infrastructure/docker/build/docker-compose.yml -p 
trafficcontrolPR1971 up
Creating network "trafficcontrolpr1971_default" with the default driver
could not find an available, non-overlapping IPv4 address pool among the 
defaults to assign to the network
+ exit 1
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] mitchell852 opened a new pull request #2493: testing passing contents of a file from the server to TP and promptin…

2018-07-02 Thread GitBox
mitchell852 opened a new pull request #2493: testing passing contents of a file 
from the server to TP and promptin…
URL: https://github.com/apache/trafficcontrol/pull/2493
 
 
   …g a download dialog


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Jenkins build is back to normal : trafficcontrol-PR-rat #1241

2018-07-02 Thread Apache Jenkins Server
See 



Build failed in Jenkins: trafficcontrol-PR-rat #1240

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1970
originally caused by:
 GitHub pull request #2338 of commit 134bb62e4f865a77435592f602146e05576afabd, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins7132964584335891177.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2473789484598596047.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2338: Add TO Go users

2018-07-02 Thread GitBox
asfgit commented on issue #2338: Add TO Go users
URL: https://github.com/apache/trafficcontrol/pull/2338#issuecomment-401892293
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1970/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

2018-07-02 Thread GitBox
rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table 
Endpoints in Go
URL: https://github.com/apache/trafficcontrol/issues/2232#issuecomment-398070296
 
 
   # Table Usage
   
   ```
   get /api/1.1/cdns/domains
   - DB: Profile, CDN
   
   get /api/1.1/servers/:id/configfiles/ats
   get /api/1.1/profiles/:id/configfiles/ats/:filename
   get /api/1.1/servers/:id/configfiles/ats/:filename
   get /api/1.1/cdns/:id/configfiles/ats/:filename
   
   get /api/1.1/dbdump
   - Ext: pg_dump
   
   get /api/1.1/deliveryservices/:id/routing
   - DB: Tenant, DeliveryService, DeliveryServiceRegex, Regex, Cdn, Type
   - Ext: Router
   
   get /api/1.1/deliveryservices/:id/state
   - DB: Tenant, DeliveryService, CDN
   - Ext: Monitor
   
   post /api/1.1/deliveryservices/request
   - Ext: Email
   
   get /api/1.1/steering/:id/targets
   - DB: Tenant, DeliveryService, SteeringTarget, Type
   
   get /api/1.1/steering/:id/targets/:target_id
   - DB: Tenant, DeliveryService, SteeringTarget, Type
   
   post /api/1.1/steering/:id/targets
   - DB: Tenant, DeliveryService, SteeringTarget, Type
   
   put /api/1.1/steering/:id/targets/:target_id
   - DB: Tenant, DeliveryService, SteeringTarget, Type
   
   delete /api/1.1/steering/:id/targets/:target_id
   - DB: Tenant, DeliveryService, SteeringTarget
   
   delete /api/1.1/divisions/name/:name
   - DB: Division, Region
   
   get /api/1.1/federations
   - DB: TmUser, FederationTmUser, FederationDeliveryService, DeliveryService
   
   post /api/1.1/federations
   put /api/1.1/federations
   delete /api/1.1/federations
   get /api/1.1/cdns/:name/federations
   get /api/1.1/cdns/:name/federations/:fedId
   post /api/1.1/cdns/:name/federations
   put /api/1.1/cdns/:name/federations/:fedId
   delete /api/1.1/cdns/:name/federations/:fedId
   get /api/1.1/federations/:fedId/users
   post /api/1.1/federations/:fedId/users
   delete /api/1.1/federations/:fedId/users/:userId
   get /api/1.1/federations/:fedId/deliveryservices
   post /api/1.1/federations/:fedId/deliveryservices
   delete /api/1.1/federations/:fedId/deliveryservices/:dsId
   get /api/1.1/federations/:fedId/federation_resolvers
   post /api/1.1/federations/:fedId/federation_resolvers
   post /api/1.1/federation_resolvers
   delete /api/1.1/federation_resolvers/:id
   - DB: TmUser, FederationTmUser, FederationDeliveryService, DeliveryService, 
FederationResolver, FederationFederationResolver
   
   get /api/1.1/hwinfo/dtdata
   - DB: HwInfo, Server
   
   get /api/1.1/osversions
   - DB: Parameter, Server
   - Ext: file:osversions.cfg
   
   post /api/1.1/isos
   - DB: Parameter
   - Ext: mkisofs
   
   get /api/1.1/jobs
   - DB: DeliveryService, Tenant, Job
   
   get /api/1.1/jobs/:id
   - DB: DeliveryService, Job
   
   get /api/1.1/user/current/jobs
   - DB: Tenant, DeliveryService, Job, TmUser
   
   post /api/1.1/user/current/jobs
   - DB: Tenant, DeliveryService, Job, TmUser
   
   post /api/1.1/parameters/validate
   - DB: Parameter
   
   get /api/1.1/cachegroups/:id/parameters
   - DB: Parameter, CachegroupParameter
   
   get /api/1.1/cachegroups/:id/unassigned_parameters
   - DB: Parameter, CachegroupParameter
   
   get /api/1.1/cachegroup/:parameter_id/parameter
   - DB: Cachegroup, CachegroupParameter
   
   get /api/1.1/cachegroupparameters
   - DB: CachegroupParameter
   
   post /api/1.1/cachegroupparameters
   - DB: Cachegroup, Parameter, ProfileParameter, CachegroupParameter
   
   delete /api/1.1/cachegroupparameters/:cachegroup_id/:parameter_id
   - DB: Cachegroup, Parameter, CachegroupParameter
   
   get /api/1.1/cachegroups/:parameter_id/parameter/available
   - DB: Cachegroup, CachegroupParameter
   
   post /api/1.1/regions/:region_name/phys_locations
   - DB: PhysLocation, Region
   
   post /api/1.1/profiles/name/:profile_name/copy/:profile_copy_from
   - DB: Profile, ProfileParameter
   
   get /api/1.1/profiles/:id/export
   - DB: Profile, ProfileParameter, Parameter
   
   post /api/1.1/profiles/import
   - DB: Cdn, Profile, ProfileTypeValue, Parameter, ProfileParameter
   
   get /api/1.1/parameters/:id/profiles
   - DB: Profile, ProfileParameter
   
   get /api/1.1/parameters/:id/unassigned_profiles
   - DB: Profile, ProfileParameter
   
   post /api/1.1/divisions/:division_name/regions
   - DB: Region, Division
   
   delete /api/1.1/regions/name/:name
   - DB: Region
   
   get /api/1.1/roles
   - DB: Role
   
   get /api/1.1/capabilities
   - DB: Capability
   
   get /api/1.1/capabilities/:name
   - DB: Capability
   
   put /api/1.1/capabilities/:name
   - DB: Capability
   
   post /api/1.1/capabilities
   - DB: Capability
   
   delete /api/1.1/capabilities/:name
   - DB: Capability, ApiCapability
   
   get /api/1.1/api_capabilities
   - DB: ApiCapability
   
   get /api/1.1/api_capabilities/:id
   - DB: ApiCapability
   
   put /api/1.1/api_capabilities/:id
   - DB: ApiCapability
   
   post /api/1.1/api_capabilities
   - DB: ApiCapability
   
   delete /api

Build failed in Jenkins: trafficcontrol-PR-rat #1239

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1969
originally caused by:
 GitHub pull request #2369 of commit 31eadad712e5278b5175cc005b796e87f6771c9e, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2335266777141640387.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins9036669168131875759.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

2018-07-02 Thread GitBox
asfgit commented on issue #2369: To go deletednsseckeys
URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401879116
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1969/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1238

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1968
originally caused by:
 GitHub pull request #2432 of commit e7454eb22b8fe2c3592052b9f67ef9da27ba36af, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2935368061194698817.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins7106919251081041355.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2432: Add TO Go deliveryservices urlkeys

2018-07-02 Thread GitBox
asfgit commented on issue #2432: Add TO Go deliveryservices urlkeys
URL: https://github.com/apache/trafficcontrol/pull/2432#issuecomment-401878332
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1968/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1237

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1967
originally caused by:
 GitHub pull request #2439 of commit 7bea2ac5d253c4661b2d1c86b78be750f8cf06dc, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins477060975504732821.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins4607754636244738625.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2439: Add TO Go riak/bucket

2018-07-02 Thread GitBox
asfgit commented on issue #2439: Add TO Go riak/bucket
URL: https://github.com/apache/trafficcontrol/pull/2439#issuecomment-401877410
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1967/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rob05c commented on a change in pull request #2369: To go deletednsseckeys

2018-07-02 Thread GitBox
rob05c commented on a change in pull request #2369: To go deletednsseckeys
URL: https://github.com/apache/trafficcontrol/pull/2369#discussion_r199567779
 
 

 ##
 File path: traffic_ops/traffic_ops_golang/cdn/dnssec.go
 ##
 @@ -0,0 +1,58 @@
+package cdn
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import (
+   "errors"
+   "net/http"
+
+   "github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/api"
+   
"github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/riaksvc"
+)
+
+const CDNDNSSECKeyType = "dnssec"
+
+func DeleteDNSSECKeys(w http.ResponseWriter, r *http.Request) {
+   inf, userErr, sysErr, errCode := api.NewInfo(r, []string{"name"}, nil)
+   if userErr != nil || sysErr != nil {
+   api.HandleErr(w, r, errCode, userErr, sysErr)
+   return
+   }
+   defer inf.Close()
+
+   key := inf.Params["name"]
+
+   riakCluster, err := riaksvc.GetRiakClusterTx(inf.Tx.Tx, 
inf.Config.RiakAuthOptions)
+   if err != nil {
+   api.HandleErr(w, r, http.StatusInternalServerError, nil, 
errors.New("getting riak cluster: "+err.Error()))
+   return
+   }
+   if err := riakCluster.Start(); err != nil {
 
 Review comment:
   Fixed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1236

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1966
originally caused by:
 GitHub pull request #2421 of commit b6f6be49d5759a811ba7737cdc3855523a742a69, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2606438759620001417.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins1145735282243011400.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete

2018-07-02 Thread GitBox
asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete
URL: https://github.com/apache/trafficcontrol/pull/2421#issuecomment-401876049
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1966/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] alficles commented on a change in pull request #2369: To go deletednsseckeys

2018-07-02 Thread GitBox
alficles commented on a change in pull request #2369: To go deletednsseckeys
URL: https://github.com/apache/trafficcontrol/pull/2369#discussion_r199563530
 
 

 ##
 File path: traffic_ops/traffic_ops_golang/cdn/dnssec.go
 ##
 @@ -0,0 +1,58 @@
+package cdn
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import (
+   "errors"
+   "net/http"
+
+   "github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/api"
+   
"github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/riaksvc"
+)
+
+const CDNDNSSECKeyType = "dnssec"
+
+func DeleteDNSSECKeys(w http.ResponseWriter, r *http.Request) {
+   inf, userErr, sysErr, errCode := api.NewInfo(r, []string{"name"}, nil)
+   if userErr != nil || sysErr != nil {
+   api.HandleErr(w, r, errCode, userErr, sysErr)
+   return
+   }
+   defer inf.Close()
+
+   key := inf.Params["name"]
+
+   riakCluster, err := riaksvc.GetRiakClusterTx(inf.Tx.Tx, 
inf.Config.RiakAuthOptions)
+   if err != nil {
+   api.HandleErr(w, r, http.StatusInternalServerError, nil, 
errors.New("getting riak cluster: "+err.Error()))
+   return
+   }
+   if err := riakCluster.Start(); err != nil {
 
 Review comment:
   I was expecting to see this paired with a `.Stop()` somewhere. Is it missing?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1235

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1965
originally caused by:
 GitHub pull request #2395 of commit b94fe3534a13f7677997f6bdc4bc6cd4a38b620d, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2284763385564702935.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins6426032763568220208.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2395: To go ds sslkeys generate

2018-07-02 Thread GitBox
asfgit commented on issue #2395: To go ds sslkeys generate
URL: https://github.com/apache/trafficcontrol/pull/2395#issuecomment-401870784
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1965/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1234

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1964
originally caused by:
 GitHub pull request #2400 of commit 7e93215fbb2637b979a57768c4facaee2fdcb46e, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins6805997868297323322.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins3070091397333466683.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2400: To go cdns sslkeys

2018-07-02 Thread GitBox
asfgit commented on issue #2400: To go cdns sslkeys
URL: https://github.com/apache/trafficcontrol/pull/2400#issuecomment-401869906
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1964/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1233

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1963
originally caused by:
 GitHub pull request #2369 of commit 2476da071419ecd136bfd0e5c388b8c495720799, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins3442613112725640411.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins3028357792341667421.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

2018-07-02 Thread GitBox
asfgit commented on issue #2369: To go deletednsseckeys
URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401869032
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1963/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1232

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1962
originally caused by:
 GitHub pull request #2382 of commit 8257ecf8fb2501f6ed5a002e5a7662b8dba11965, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H29 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins5585672339786718701.sh
+ rm -rf 

 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins7225082611849055452.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


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

2018-07-02 Thread Apache Jenkins Server
See 




[GitHub] asfgit commented on issue #2382: To go dnsseckeys generate

2018-07-02 Thread GitBox
asfgit commented on issue #2382: To go dnsseckeys generate
URL: https://github.com/apache/trafficcontrol/pull/2382#issuecomment-401868131
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1962/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

2018-07-02 Thread GitBox
asfgit commented on issue #2369: To go deletednsseckeys
URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401867312
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1961/
   Test FAILed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR #1961

2018-07-02 Thread Apache Jenkins Server
See 

--
GitHub pull request #2369 of commit 03752ab61205d6422d1314188bd09f9708c66205, 
no merge conflicts.
Setting status of 03752ab61205d6422d1314188bd09f9708c66205 to PENDING with url 
https://builds.apache.org/job/trafficcontrol-PR/1961/ and message: 'Build 
started for merge commit.'
Using context: default
[EnvInject] - Loading node environment variables.
Building remotely on H27 (ubuntu xenial) in workspace 

Cloning the remote Git repository
Cloning repository git://github.com/apache/trafficcontrol.git
 > git init  # timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # 
 > timeout=10
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
 > git rev-parse origin/03752ab61205d6422d1314188bd09f9708c66205^{commit} # 
 > timeout=10
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Retrying after 10 seconds
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
 > git rev-parse origin/03752ab61205d6422d1314188bd09f9708c66205^{commit} # 
 > timeout=10
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Retrying after 10 seconds
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url git://github.com/apache/trafficcontrol.git # 
 > timeout=10
Fetching upstream changes from git://github.com/apache/trafficcontrol.git
 > git --version # timeout=10
using GIT_SSH to set credentials 
 > git fetch --tags --progress git://github.com/apache/trafficcontrol.git 
 > +refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
 > git rev-parse origin/03752ab61205d6422d1314188bd09f9708c66205^{commit} # 
 > timeout=10
 > git rev-parse 03752ab61205d6422d1314188bd09f9708c66205^{commit} # timeout=10
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Archiving artifacts


[GitHub] asfgit commented on issue #2441: Add TO Go delete deliveryservice_server

2018-07-02 Thread GitBox
asfgit commented on issue #2441: Add TO Go delete deliveryservice_server
URL: https://github.com/apache/trafficcontrol/pull/2441#issuecomment-401867134
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1960/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Jenkins build is back to normal : trafficcontrol-PR-rat #1229

2018-07-02 Thread Apache Jenkins Server
See 



[GitHub] asfgit commented on issue #2492: Refactor cachegroups to use coordinates table

2018-07-02 Thread GitBox
asfgit commented on issue #2492: Refactor cachegroups to use coordinates table
URL: https://github.com/apache/trafficcontrol/pull/2492#issuecomment-401866456
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1959/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rawlinp opened a new pull request #2492: Refactor cachegroups to use coordinates table

2018-07-02 Thread GitBox
rawlinp opened a new pull request #2492: Refactor cachegroups to use 
coordinates table
URL: https://github.com/apache/trafficcontrol/pull/2492
 
 
   Replace cachegroup lat/long columns with a coordinate FK. On cg create,
   create a coordinate and set the coordinate ID in the cg. On cg update,
   update the referenced coordinate too. On delete, delete the cg and its
   reference coordinate.
   
   Keep the API unchanged; only refactor the backend.
   
   Keep the TO UI functional for cachegroup CRUD.
   
   Update the Perl API backend enough to keep the TO UI and API tests
   working (for Perl APIs yet to be rewritten).


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1228

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1958
originally caused by:
 GitHub pull request #2491 of commit 822566b7316e2ffbdd99cab08ee5601b58bb6035, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H33 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins5115237019330996337.sh
+ rm -rf 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins262776956742970730.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang

2018-07-02 Thread GitBox
asfgit commented on issue #2491: add cpu profiling process, and memory 
profiling and db stats endpoints to traffic_ops_golang
URL: https://github.com/apache/trafficcontrol/pull/2491#issuecomment-401860049
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1958/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] DylanVolz opened a new pull request #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang

2018-07-02 Thread GitBox
DylanVolz opened a new pull request #2491: add cpu profiling process, and 
memory profiling and db stats endpoints to traffic_ops_golang
URL: https://github.com/apache/trafficcontrol/pull/2491
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1227

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1957
originally caused by:
 GitHub pull request #2358 of commit 92083f6ad0189a85a272c3fc861e64a5186fa757, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H20 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins7793172366195962907.sh
+ rm -rf 

 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins197240422127173307.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2358: Created the static dns entry API

2018-07-02 Thread GitBox
asfgit commented on issue #2358: Created the static dns entry API
URL: https://github.com/apache/trafficcontrol/pull/2358#issuecomment-401848624
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1957/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP]

2018-07-02 Thread GitBox
asfgit commented on issue #2490: Update gen_iso endpoint to have option to 
stream bits instead of writing them to disk [WIP]
URL: https://github.com/apache/trafficcontrol/pull/2490#issuecomment-401847027
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rivasj opened a new pull request #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP]

2018-07-02 Thread GitBox
rivasj opened a new pull request #2490: Update gen_iso endpoint to have option 
to stream bits instead of writing them to disk [WIP]
URL: https://github.com/apache/trafficcontrol/pull/2490
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dewrich closed pull request #2233: Fixed the Go import package alignment because the github project moved

2018-07-02 Thread GitBox
dewrich closed pull request #2233: Fixed the Go import package alignment 
because the github project moved
URL: https://github.com/apache/trafficcontrol/pull/2233
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/traffic_ops/traffic_ops_golang/urisigning.go 
b/traffic_ops/traffic_ops_golang/urisigning.go
index 50679fde0..91cca2954 100644
--- a/traffic_ops/traffic_ops_golang/urisigning.go
+++ b/traffic_ops/traffic_ops_golang/urisigning.go
@@ -36,7 +36,7 @@ import (

"github.com/apache/incubator-trafficcontrol/traffic_ops/traffic_ops_golang/tenant"
"github.com/basho/riak-go-client"
"github.com/jmoiron/sqlx"
-   "github.com/lestrrat/go-jwx/jwk"
+   "github.com/lestrrat/go-jwx/jwe"
 )
 
 // CDNURIKeysBucket is the namespace or bucket used for CDN URI signing keys.
@@ -45,7 +45,7 @@ const CDNURIKeysBucket = "cdn_uri_sig_keys"
 // URISignerKeyset is the container for the CDN URI signing keys
 type URISignerKeyset struct {
RenewalKid *string   `json:"renewal_kid"`
-   Keys   []jwk.EssentialHeader `json:"keys"`
+   Keys   []jwe.EssentialHeader `json:"keys"`
 }
 
 // endpoint handler for fetching uri signing keys from riak


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dangogh commented on a change in pull request #2485: Modify the grove rpm version format

2018-07-02 Thread GitBox
dangogh commented on a change in pull request #2485: Modify the grove rpm 
version format
URL: https://github.com/apache/trafficcontrol/pull/2485#discussion_r199207507
 
 

 ##
 File path: grove/build/build_rpm.sh
 ##
 @@ -19,6 +19,7 @@ cd "$ROOTDIR/grove"
 
 BUILDDIR="$ROOTDIR/grove/rpmbuild"
 VERSION=`cat ./VERSION`.`git rev-list --all --count`
+RELEASE=`git log -1 | awk '/^commit/ {print substr($2,1,9)}'`
 
 Review comment:
   This doesn't get the same form as the other components, so the release id 
won't match.In `trafficcontrol/build/README.md` describes how this is done 
for the other components -- there are functions in 
`trafficcontrol/build/functions.sh` that do this for you -- I recommend 
following the recipe in the other components to get the same result.
   
   I do understand wanting to have the grove main version number be different 
from the others (e.g. 0.1.x vs 2.3.x),  but the commit count and commit hash 
should be the same.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Build failed in Jenkins: trafficcontrol-PR-rat #1226

2018-07-02 Thread Apache Jenkins Server
See 

--
Started by upstream project "trafficcontrol-PR" build number 1956
originally caused by:
 GitHub pull request #2475 of commit 97de0f73d95cdee2fe5ca6b5eb05b58accd2e230, 
no merge conflicts.
[EnvInject] - Loading node environment variables.
Building remotely on H24 (ubuntu xenial) in workspace 

[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins2855873847108918656.sh
+ rm -rf 

 

 

 '
[operations-center-context] Requesting copy of artifacts matching '**/*.tar.gz' 
from 'Last successful build (either stable or unstable)' of trafficcontrol-PR
[operations-center-context] Copied 2 artifacts matching '**/*.tar.gz' from 
'Last successful build (either stable or unstable)' of trafficcontrol-PR:
[operations-center-context]   apache-trafficcontrol-2.3.0-docs.tar.gz
[operations-center-context]   apache-trafficcontrol-2.3.0.tar.gz
[trafficcontrol-PR-rat] $ /bin/bash -xe /tmp/jenkins6726114281257442708.sh
+ set -exu
++ ls 'apache-trafficcontrol-*-incubating.tar.gz'
ls: cannot access 'apache-trafficcontrol-*-incubating.tar.gz': No such file or 
directory
+ tarball=
Build step 'Execute shell' marked build as failure
Archiving artifacts


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401813626
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1956/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401807580
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1955/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #2475: Remove db from interface

2018-07-02 Thread GitBox
asfgit commented on issue #2475: Remove db from interface
URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401804023
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/trafficcontrol-PR/1954/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services