[GitHub] incubator-trafficcontrol issue #768: Tenant parent - on delete cascade

2017-08-01 Thread MattMills
Github user MattMills commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/768 In that case, why would this be an ON DELETE CASCADE, rather than an ON DELETE RESTRICT? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-trafficcontrol pull request #344: TO Install instructions

2017-03-15 Thread MattMills
Github user MattMills commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106282322 --- Diff: traffic_ops/INSTALL.md --- @@ -88,14 +107,18 @@ We need to tell carton where it is so the `DBD::Pg` module is installed

[GitHub] incubator-trafficcontrol pull request #344: TO Install instructions

2017-03-15 Thread MattMills
Github user MattMills commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106281138 --- Diff: traffic_ops/INSTALL.md --- @@ -8,53 +8,71 @@ Instructions are here: https://yum.postgresql.org/ - grab

[GitHub] incubator-trafficcontrol pull request #344: TO Install instructions

2017-03-15 Thread MattMills
Github user MattMills commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106280949 --- Diff: traffic_ops/INSTALL.md --- @@ -8,53 +8,71 @@ Instructions are here: https://yum.postgresql.org/ - grab

[GitHub] incubator-trafficcontrol pull request #345: [TC-182] Add additional statuses...

2017-03-10 Thread MattMills
Github user MattMills closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/345 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafficcontrol issue #345: [TC-182] Add additional statuses into t...

2017-03-10 Thread MattMills
Github user MattMills commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/345 After looking at this more with Dave, there is a larger underlying problem here affecting more than just statuses. --- If your project is set up for it, you can reply

[GitHub] incubator-trafficcontrol pull request #345: [TC-182] Add additional statuses...

2017-03-10 Thread MattMills
GitHub user MattMills opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/345 [TC-182] Add additional statuses into traffic ops seeds.sql This pr adds additional required statuses into the traffic ops status table fixing TC-182. You can merge this pull