[jira] [Commented] (TC-223) Creating a DS through the API results in an invalid DNSSEC keypair

[ASF GitHub Bot (JIRA)]

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

ASF GitHub Bot commented on TC-223:
---

GitHub user mitchell852 opened a pull request:

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

[BACKPORT TC-223] - default regex is required on a ds before dnssec keys 
can be created (…

…if required)

(cherry picked from commit c3ad83792530159cef0daf0310c768a47e680888)

# Conflicts:
#   traffic_ops/app/lib/API/Deliveryservice.pm

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

$ git pull https://github.com/mitchell852/incubator-trafficcontrol 
2.0.x-tc-223

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

https://github.com/apache/incubator-trafficcontrol/pull/469.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 #469


commit e4337c05f94645e7d8eb016f1694522723721b97
Author: Jeremy Mitchell 
Date:   2017-04-13T02:45:42Z

default regex is required on a ds before dnssec keys can be created (if 
required)

(cherry picked from commit c3ad83792530159cef0daf0310c768a47e680888)

# Conflicts:
#   traffic_ops/app/lib/API/Deliveryservice.pm




> Creating a DS through the API results in an invalid DNSSEC keypair
> --
>
> Key: TC-223
> URL: https://issues.apache.org/jira/browse/TC-223
> Project: Traffic Control
>  Issue Type: Bug
>  Components: Traffic Ops
>Affects Versions: 2.0.0, 2.1.0
>Reporter: David Neuman
>Assignee: Jeremy Mitchell
>
> If you have DNSSEC enabled and you create a DS from the API, a DNSSEC keypair 
> with name = "" gets created (example below).  It looks like this happens 
> because when you create a DS through the API you don't create the DS with a 
> regex.  When the create method calls create_dnssec_keys that method uses the 
> regex to create the name field.
> {quote}
>  "Test-DS-1491489049" : {
>  "zsk" : [
> {
>"effectiveDate" : 1491489041,
>"public" : "...",
>"private" : "...",
>"expirationDate" : 1494081041,
>"name" : "",
>"status" : "new",
>"inceptionDate" : 1491489041,
>"ttl" : "60"
> }
>  ],
>  "ksk" : [
> {
>"status" : "new",
>"ttl" : "60",
>"inceptionDate" : 1491489041,
>"public" : "...",
>"effectiveDate" : 1491489041,
>"name" : "",
>"private" : "...",
>"expirationDate" : 1523025041
> }
>  ]
>   },
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] incubator-trafficcontrol pull request #469: [BACKPORT TC-223] - default rege...

[mitchell852]

GitHub user mitchell852 opened a pull request:

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

[BACKPORT TC-223] - default regex is required on a ds before dnssec keys 
can be created (…

…if required)

(cherry picked from commit c3ad83792530159cef0daf0310c768a47e680888)

# Conflicts:
#   traffic_ops/app/lib/API/Deliveryservice.pm

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

$ git pull https://github.com/mitchell852/incubator-trafficcontrol 
2.0.x-tc-223

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

https://github.com/apache/incubator-trafficcontrol/pull/469.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 #469


commit e4337c05f94645e7d8eb016f1694522723721b97
Author: Jeremy Mitchell 
Date:   2017-04-13T02:45:42Z

default regex is required on a ds before dnssec keys can be created (if 
required)

(cherry picked from commit c3ad83792530159cef0daf0310c768a47e680888)

# Conflicts:
#   traffic_ops/app/lib/API/Deliveryservice.pm




---
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] [Created] (TC-234) TO postinstall error moving coverage-zone.json

[Dan Kirkwood (JIRA)]

Dan Kirkwood created TC-234:
---

 Summary: TO postinstall error moving coverage-zone.json
 Key: TC-234
 URL: https://issues.apache.org/jira/browse/TC-234
 Project: Traffic Control
  Issue Type: Bug
  Components: Traffic Ops
Affects Versions: 2.0.0
Reporter: Dan Kirkwood
 Fix For: 2.0.0


This is a regression from 1.8.x -- if answering "no" to "Download MaxMind 
data", I get this message from postinstall:

Not downloading Maxmind data
Copying coverage zone file to public dir
/bin/mv: cannot stat ‘/opt/traffic_ops/app/public/coverage-zone.json’: No such 
file or directory



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] incubator-trafficcontrol pull request #409: Fix Traffic Monitor 2.0 TO polli...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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] [Commented] (TC-232) TO postinstall fails with 401 Unauthorized

[Dan Kirkwood (JIRA)]

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

Dan Kirkwood commented on TC-232:
-

Investigation shows postinstall is picking up WWW::Curl::Easy from the system 
where it was previously installed (from rpm package 
perl-WWW-Curl-4.15-12.el7.x86_64).   It needs the version installed by carton.  
 Fix is forthcoming...


> TO postinstall fails with 401 Unauthorized
> --
>
> Key: TC-232
> URL: https://issues.apache.org/jira/browse/TC-232
> Project: Traffic Control
>  Issue Type: Bug
>  Components: Traffic Ops
>Affects Versions: 2.0.0
>Reporter: Dan Kirkwood
> Fix For: 2.0.0
>
>
> Running postinstall immediately after initial  traffic_ops install,  I get 
> this error after traffic_ops is started:
> Checking profiles at https://localhost using username admin
> Error checking if profiles exist: 401 Unauthorized



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (TC-232) TO postinstall fails with 401 Unauthorized

[Dan Kirkwood (JIRA)]

Dan Kirkwood created TC-232:
---

 Summary: TO postinstall fails with 401 Unauthorized
 Key: TC-232
 URL: https://issues.apache.org/jira/browse/TC-232
 Project: Traffic Control
  Issue Type: Bug
  Components: Traffic Ops
Affects Versions: 2.0.0
Reporter: Dan Kirkwood
 Fix For: 2.0.0


Running postinstall immediately after initial  traffic_ops install,  I get this 
error after traffic_ops is started:

Checking profiles at https://localhost using username admin
Error checking if profiles exist: 401 Unauthorized






--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (TC-217) Self Service - CRUD Sub Tenant

[Ryan Durfey (JIRA)]

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

Ryan Durfey commented on TC-217:


I don't have a specific use case where we need the hierarchy yet, but I lean 
toward including it now so we don't have to build it in later.  I agree that a 
simple nested JSON would work for most any use case.

> Self Service - CRUD Sub Tenant
> --
>
> Key: TC-217
> URL: https://issues.apache.org/jira/browse/TC-217
> Project: Traffic Control
>  Issue Type: New Feature
>  Components: Traffic Ops, Traffic Ops API
>Reporter: Ashish Timilsina
>
> Allow users to CRUD sub tenants below their tenant level or below any 
> sub-tenant level they choose. Default would be directly below the current 
> tenant value.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] incubator-trafficcontrol pull request #467: Modified admin.pl to support the...

[dewrich]

GitHub user dewrich opened a pull request:

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

Modified admin.pl to support the postgres superuser



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

$ git pull https://github.com/dewrich/incubator-trafficcontrol 
postgres-superuser

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

https://github.com/apache/incubator-trafficcontrol/pull/467.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 #467


commit 792856424f12656994ca709a8fec519048fed975
Author: Dewayne Richardson 
Date:   2017-04-12T20:46:52Z

switched the goose user to be postgres

commit 5f43d7f042b65542ab8c37acfcd04bf8755c8ac7
Author: Dewayne Richardson 
Date:   2017-04-12T20:47:19Z

switched the postgres user to be the driver of the admin.pl user

commit 84545f2cfeb0283c9edaaf04fdaf1e0274dd6de3
Author: Dewayne Richardson 
Date:   2017-04-12T20:47:36Z

Merge branch 'master' into postgres-superuser

commit 718895cb78060ba985b5b0feeb7cb2a098aebe58
Author: Dewayne Richardson 
Date:   2017-04-13T15:01:18Z

added the ability to do a create_super_user




---
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] [Commented] (TC-228) TO: postinstall changes needed for postgres

[ASF GitHub Bot (JIRA)]

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

ASF GitHub Bot commented on TC-228:
---

GitHub user dangogh opened a pull request:

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

[TC-228] collective changes to postinstall to support postgres

this is a collection of all the changes to postinstall and related 
utilities to support postgres setup and to provide a non-interactive means to 
invoke postinstall.

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

$ git pull https://github.com/dangogh/incubator-trafficcontrol 
postinstall-2.0

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

https://github.com/apache/incubator-trafficcontrol/pull/466.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 #466


commit 0092660ea16d516c38b3c0f4af458e47a5167c24
Author: PeterRyder 
Date:   2017-02-06T15:51:43Z

Removed openssl file and now pass data directly to GenerateCert

(cherry picked from commit 9729d23677c3b8d2d4431d6e2e1335ff72fda7ff)

commit c9a0ca020591aa80badc1ad13a72073607dea56e
Author: PeterRyder 
Date:   2017-02-06T22:06:13Z

Check connection to database

(cherry picked from commit 6316bb33586b7b5e240bfc86ee307c076aec5023)

commit 06a1ccf62edec7e18e144a9f24eb9b3aa8b5e120
Author: PeterRyder 
Date:   2017-02-06T22:13:07Z

changed database driver to Pg

(cherry picked from commit bd79a22f4bcdf37ca202b643a20b7a502c32)

commit 6de0e4603dfb043df1d692feca70fac192f3a73d
Author: PeterRyder 
Date:   2017-02-07T16:14:23Z

Corrected database connection and removed hardcoded getDbDriver function

(cherry picked from commit 72f5b25bf81a4f3756b3763568cd091fbec0e696)

commit 4a3daa216ec58978ef8738216b14b24ccb369bc1
Author: PeterRyder 
Date:   2017-02-07T16:20:59Z

Changed default port to 5432, the postgres default

(cherry picked from commit 5fd9f4c56d0911a8f4ca12472f2ae0e0612a2e7e)

commit 1e8386cdedf67e46c3ef48db28bf18940b6dd558
Author: PeterRyder 
Date:   2017-02-07T18:44:00Z

Fixed yml file for admin.pl with correct inputs for psql

(cherry picked from commit e086d1f7613776bbe73498409f73a46f3d158ab1)

commit 57cfaf396e6c0f96fa657ecbe1eb0af204998365
Author: Jeff Martin 
Date:   2017-04-07T17:31:52Z

Term::ReadPassword fix: TC-185

(cherry picked from commit 290e4bc082df416a727490dfc4a8210b3ff5d778)

commit 9b76d6758c5ad5f7c1d4d28812de0fa6d0672ca0
Author: Jeff Martin 
Date:   2017-04-11T19:06:55Z

[TC-228] postinstall and INSTALL.md updates

INSTALL.md
- Changed manual installation approach for non-containered postgres

postinstall
- Set reconfigure to 1, always init database
- Set debug to 1, always output so the end user sees what is
 happening
- set the goose needed driver to "postgres" instead of "Pg"
- changed question default: "Database server root (admin) user"
 name from "root" to "postgres"
- set PGUSER and PGPASSWORD in env to login to the postgres
 superuser account
- update traffic_ops database table with the "admin" user name
 and password provided by user (after initdb)

(cherry picked from commit f08a1c2a20266d67ed1a0c778180b6da99426096)

commit f8f30bf584b992a52d655ec65945a689c6f868fe
Author: Jeff Martin 
Date:   2017-04-11T20:18:06Z

[TC-228] global dec fix

Global symbol "%dbconf" requires explicit package name at 
/opt/traffic_ops/install/bin/postinstall line 569.
Global symbol "%dbconf" requires explicit package name at 
/opt/traffic_ops/install/bin/postinstall line 570.

(cherry picked from commit 5cf6fae56e05eb467b3fb2e6be885d3975f7632e)

commit 1850cda251dfbb0c8dfe0ae68da42960d5e91c8e
Author: Jeff Martin 
Date:   2017-04-11T20:58:39Z

[TC-228] - bin and lib file updates
- remove install/bin/build_trafficops_perl_library
- remove mysql perl deps in install/lib/BuildPerlDeps.pm

(cherry picked from commit 7180ab5884e5b42663a073dd4750f67c8f89779c)

commit bef71f8dbd77d75e3248069137ca6c45627bbd89
Author: Dan Kirkwood 
Date:   2017-04-12T22:21:08Z

set perl5lib for sub-processes

(cherry picked from commit 670bb30f2f982d79ba90e9e6e7e8216a27ff7f7c)

commit a1eb80ec2a04c77be28f45c0d92dc2156f2601db
Author: Dan Kirkwood 
Date:   2017-04-12T23:51:55Z

chdir immediately before download_web_deps

(cherry picked from commit 65d1de01c6a9a25146e88540c7bb49ffc2d2ab5b)

commit