[GitHub] incubator-trafficcontrol pull request #232: goTM logging cleanup

2017-01-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/232


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Assigned] (TC-114) Fail to restart Tomcat when TM start before config pulled

2017-01-26 Thread Jeremy Mitchell (JIRA)

 [ 
https://issues.apache.org/jira/browse/TC-114?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jeremy Mitchell reassigned TC-114:
--

Assignee: Zhilin Huang

> Fail to restart Tomcat when TM start before config pulled
> -
>
> Key: TC-114
> URL: https://issues.apache.org/jira/browse/TC-114
> Project: Traffic Control
>  Issue Type: Bug
>  Components: Traffic Monitor
>Affects Versions: 1.7.0
>Reporter: Zhilin Huang
>Assignee: Zhilin Huang
>
> Here are the steps to reproduce it:
> 1) Clean install traffic monitor
> 2) Start tomcat and check the tomcat process ID
> 3) Add traffic monitor to traffic ops
> 4) Run traffic monitor config init perl script
> 5) Restart traffic monitor and check the tomcat process ID
> Check the process id,  you will find the traffic monitor is not restarted at 
> all. And the traffic monitor will be always in error status except kill it by 
> "kill -9".



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (TC-114) Fail to restart Tomcat when TM start before config pulled

2017-01-26 Thread Jeremy Mitchell (JIRA)

[ 
https://issues.apache.org/jira/browse/TC-114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15840345#comment-15840345
 ] 

Jeremy Mitchell commented on TC-114:


ok, you were not a "contributor". you are now so I assigned it to you. thanks!

> Fail to restart Tomcat when TM start before config pulled
> -
>
> Key: TC-114
> URL: https://issues.apache.org/jira/browse/TC-114
> Project: Traffic Control
>  Issue Type: Bug
>  Components: Traffic Monitor
>Affects Versions: 1.7.0
>Reporter: Zhilin Huang
>Assignee: Zhilin Huang
>
> Here are the steps to reproduce it:
> 1) Clean install traffic monitor
> 2) Start tomcat and check the tomcat process ID
> 3) Add traffic monitor to traffic ops
> 4) Run traffic monitor config init perl script
> 5) Restart traffic monitor and check the tomcat process ID
> Check the process id,  you will find the traffic monitor is not restarted at 
> all. And the traffic monitor will be always in error status except kill it by 
> "kill -9".



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (TC-122) TO DBdump no longer works with Postgres

2017-01-26 Thread Jeremy Mitchell (JIRA)
Jeremy Mitchell created TC-122:
--

 Summary: TO DBdump no longer works with Postgres
 Key: TC-122
 URL: https://issues.apache.org/jira/browse/TC-122
 Project: Traffic Control
  Issue Type: Bug
  Components: Traffic Ops
Affects Versions: 2.0.0
Reporter: Jeremy Mitchell
 Fix For: 2.0.0


The following route produces an empty file:

$r->get('/dbdump')->over( authenticated => 1 )->to( 'GenDbDump#dbdump', 
namespace => $namespace );

I think pg_dump requires a password or something. maybe ~/.pgpass is the answer?






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] incubator-trafficcontrol pull request #233: refactored functionality to the ...

2017-01-26 Thread dewrich
GitHub user dewrich opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/233

refactored functionality to the migrate.sh for automation



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dewrich/incubator-trafficcontrol 
dewrich/enhancement/pg-migration

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/233.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #233


commit 911a81d0b0c36a08876625d1e3e9ad01c2b7d251
Author: Dewayne Richardson 
Date:   2017-01-26T21:43:47Z

refactored functionality to the migrate.sh for automation




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #234: Move TM2 to trafficcontrol/traff...

2017-01-26 Thread rob05c
GitHub user rob05c opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/234

Move TM2 to trafficcontrol/traffic_monitor_golang, add Docker build



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/rob05c/incubator-trafficcontrol tm2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/234.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #234


commit 87699e2decbdef0ff7775857367976b42f118878
Author: Robert Butts 
Date:   2017-01-26T21:31:39Z

Move TM2 to trafficcontrol/traffic_monitor_golang

commit 9512bf2a17e652fef036a7ad8feb2d082a63169e
Author: Robert Butts 
Date:   2017-01-26T21:34:40Z

Add TM2 docker build




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #233: refactored functionality to the ...

2017-01-26 Thread dangogh
Github user dangogh commented on a diff in the pull request:


https://github.com/apache/incubator-trafficcontrol/pull/233#discussion_r98099812
  
--- Diff: traffic_ops/app/db/pg-migration/README.md ---
@@ -1,17 +1,10 @@
-# Converting existing mysql `traffic_ops` database to postgres
+# Converts existing mysql `traffic_ops` database to postgres
 
 * Requires a recent ( 1.12 ) version of `docker-engine` and 
`docker-compose`.
 
-* Modify the mysql.env for your existing Mysql Database
-
-* Modify the postgres.env for your new Postgres Database
-  (NOTE: do not set the POSTGRES_HOST to 'localhost' it needs to be the IP 
address or DNS available hostname so that the container can reach out to 
Postgres)
-
+* Modify the mysql-to-postgres.env file for the parameters in your 
Migration 
 
 * Ensure that your new Postgres service is running (local or remote)
 
 * Run the Mysql to Postgres Migration Docker flow
-  * `$ docker-compose -f pgmigration.yml down -v && docker-compose -f 
pgmigration.yml build && TO_SERVER=https://traffic_ops.kabletown.com TO_USER=me 
TO_PASSWORD='my!passwd' docker-compose -f pgmigration.yml up`
-
-* Run the Postgres datatype conversion
-  * `$ docker-compose -f convert.yml up` 
+  * `sh migrate.sh
--- End diff --

minor -- missing closing backquote


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #233: refactored functionality to the ...

2017-01-26 Thread dangogh
Github user dangogh commented on a diff in the pull request:


https://github.com/apache/incubator-trafficcontrol/pull/233#discussion_r98100084
  
--- Diff: traffic_ops/app/db/pg-migration/migrate.sh ---
@@ -0,0 +1,90 @@
+#!/bin/bash 
+#
+#  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.
+#
+
+: ${TO_SERVER?"Please the TO_SERVER environment variable: ie: 
https://kabletown.net"}
+: ${TO_USER?"Please the TO_USER environment variable: ie: "}
+: ${TO_PASSWORD?"Please the TO_PASSWORD environment variable: ie: "}
+
--- End diff --

minor -- missing a word:   Please ->set<- the ...


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #233: refactored functionality to the ...

2017-01-26 Thread dangogh
Github user dangogh commented on a diff in the pull request:


https://github.com/apache/incubator-trafficcontrol/pull/233#discussion_r98100403
  
--- Diff: traffic_ops/app/db/pg-migration/mysql-to-postgres.env ---
@@ -0,0 +1,43 @@
+#!/bin/bash
+#
+#  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.
+#
+
+#Traffic Ops Settings
+# The following configs should be configured to point to the 
+# Traffic Ops instances that is connected to the MySQL that 
+# you want to convert
+TO_SERVER=to_url
+TO_USER=(your_user)
+TO_PASSWORD=(your_password)
+
+# Staging Mysql 
+# These variables are for the "staging" Docker MySQL instance that is used
+# to load the Traffic Ops "dbdump" into.  ONLY change the MYSQL_HOST 
variable
+# Leave the other variables.
+MYSQL_HOST=( the_ip_address_running_this_migration )
+MYSQL_PORT=3306
+MYSQL_DATABASE=traffic_ops_db
+MYSQL_USER=to_user
+MYSQL_PASSWORD=twelve
+MYSQL_RANDOM_ROOT_PASSWORD=yes
+
+# The following 
+# New Producion Postgres Settings, these variables should be changed for 
the Postgres 
--- End diff --

sp


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Resolved] (TC-103) GET /update/:host_name should return boolean values

2017-01-26 Thread Jeremy Mitchell (JIRA)

 [ 
https://issues.apache.org/jira/browse/TC-103?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jeremy Mitchell resolved TC-103.

Resolution: Fixed

> GET /update/:host_name should return boolean values
> ---
>
> Key: TC-103
> URL: https://issues.apache.org/jira/browse/TC-103
> Project: Traffic Control
>  Issue Type: Improvement
>  Components: Traffic Ops, Traffic Ops API, Traffic Ops ORT
>Affects Versions: 2.0.0
>Reporter: Jeremy Mitchell
>Assignee: Jeremy Mitchell
>Priority: Minor
>  Labels: postgres
> Fix For: 2.0.0
>
>
> GET /update/:host_name returns
> [ { "upd_pending": 0, "parent_pending": "0", "status": "REPORTED",  
> "host_id": 22,  "host_name": "foo-bar" } ]
> it should return
> [ { "upd_pending": true|false, "parent_pending": true|false, "status": 
> "REPORTED",  "host_id": 22,  "host_name": "foo-bar" } ]
> since TO ORT relies heavily on this endpoint, ensure that ORT can handle 
> booleans as opposed to int/string 0|1



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] incubator-trafficcontrol pull request #233: refactored functionality to the ...

2017-01-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/233


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---