Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/289
I agree this might be useful, but I'm -1 on including this directly in the
trafficcontrol repo.
---
If your project is set up for it, you can reply to this email and have your
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/508
as we discussed this morning -- KEYS should be in dist.apache.org -- not
here in the repo:
https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/495
[TC-243] fix DBI->connect calls to allow connection to remote host
also removes default answers for cert generation
You can merge this pull request into a Git reposit
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/30
I think this should be closed -- postinstall now has `-defaults` option to
write out a cfg file with all default questions/answers. And `-a
-cfile=` to load all from the file
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/490
[BACKPORT TC-228] collective changes to postinstall to support postgres
All changes cherry-picked from master. Supports postgres initial setup as
well as postinstall batch-mode
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/489
[TC-233] no superuser for seeds
to load seeds.sql, needs to run as traffic_ops user and not postgres user
You can merge this pull request into a Git repository by running
Github user dangogh closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/466
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/340
we ended up taking a different direction for running these commands, so
this change isn't needed. Please close.
---
If your project is set up for it, you can reply
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/467#discussion_r111583751
--- Diff: traffic_ops/app/db/admin.pl ---
@@ -29,23 +29,44 @@
use YAML qw(LoadFile);
use DBIx::Class::Schema::Loader qw
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/467#discussion_r111583451
--- Diff: traffic_ops/app/db/admin.pl ---
@@ -29,23 +29,44 @@
use YAML qw(LoadFile);
use DBIx::Class::Schema::Loader qw
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/463
[TC-228] postinstall minor fixes
hopefully the final one to be backported to 2.0.x
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/459
[TC-228] more postinstall fixes
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol
postinstall
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/450
[TC-226] fixes inserts to work on initial setup
missed a couple of thing in seeds.sql -- this should fix them
You can merge this pull request into a Git repository by running
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/428
forgot to add that when I merged the change.. can you go ahead and close
it?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/340
no -- they should be able to make their own user able to connect as a db
admin or create postgres user with the same. dbconf.yml is for goose. don't
change that. I think
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/340
`[--admuser (Postgres Admin User)] [--admpwd (Postgres Admin Password)]` --
this requires a password on the cmd line -- not a good idea either, since it
ends up in the shell
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/340
can you change the title to mention admin.pl? I want to be able to find
this a year from now..
---
If your project is set up for it, you can reply to this email and have your
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106258947
--- Diff: INSTALL.md ---
@@ -0,0 +1,101 @@
+# From new CentOS 7 install:
+
+## Disable selinux:
+### change `/etc/selinux
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106258828
--- Diff: INSTALL.md ---
@@ -0,0 +1,101 @@
+# From new CentOS 7 install:
+
+## Disable selinux:
+### change `/etc/selinux
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106258688
--- Diff: INSTALL.md ---
@@ -0,0 +1,101 @@
+# From new CentOS 7 install:
+
+## Disable selinux:
+### change `/etc/selinux
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/344#discussion_r106258643
--- Diff: INSTALL.md ---
@@ -0,0 +1,101 @@
+# From new CentOS 7 install:
+
+## Disable selinux:
+### change `/etc/selinux
Github user dangogh closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/333
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/344
Install instructions
Add install instructions
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/327
adding those 2 things to BUILD.md shortly..For last point, I think
deal with it separately. Created issue
https://issues.apache.org/jira/browse/TC-180
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/338
[TC-72] BACKPORT Dockerfile build local
This should go in to 2.0 to better meet Apache build guidelines
You can merge this pull request into a Git repository by running
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/331
dang..never saw this...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/327
@alficles -- latest should address those clean concerns..
@limited -- please consider pulling in to 2.0
---
If your project is set up for it, you can reply to this email
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/333
[TC-176] fix minor typo in -- affects only 2.0.x branch
causes traffic_router to fail to build
You can merge this pull request into a Git repository by running:
$ git pull
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/327
@alficles -- copying the repo wouldn't be easier: chicken vs egg.
docker-compose operates from that directory, so would need to switch to the
copy mid-stream. Cleaning up
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/327
@limited -- is this what you had in mind? Perhaps should be considered
for merge into 2.0? (for Apache compliance)
---
If your project is set up for it, you can reply
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/327
@rob05c -- `traffic_monitor` and `traffic_monitor_golang` both produce the
same name rpm. Does the former need to be removed from the
`docker-compose.yml`?
---
If your project
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/322#discussion_r103572789
--- Diff: traffic_ops/app/t/api/1.2/tenant.t ---
@@ -0,0 +1,156 @@
+package main;
+#
+#
+# Licensed under the Apache
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/302
[TC-122] use uri form for pg_dump connection
uri form includes db name, user, passwd, host, port
You can merge this pull request into a Git repository by running:
$ git pull
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/300
fyi -- this can wait until 2.1
---
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
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/300
[TC-160] remove check for ip6 addr and gateway in same network
still allows user to enter, but does not check that address is in the same
network
You can merge this pull request
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/297
[TC-155] require user to run as db superuser -- not necessarily postgres
user
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/287
change master version back to 2.0.0
misc/release.pl changed VERSION file when it shouldn't have.. Changing it
back and I'll create an issue to fix the release script.
You can
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/273
Doesn't look like it...`git merge-base master cdn-table-1` shows me
this commit:
```commit 224eab5c10d8b5e0ebedf18edfb375e0f3c0cff0
Author: Jeff Elsloo <jeffrey_
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/255#discussion_r99711784
--- Diff: traffic_ops/install/bin/postinstall ---
@@ -568,8 +558,10 @@ sub getDefaults {
sub setupDatabase {
my $todbconf
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/240#discussion_r98947371
--- Diff: traffic_ops/app/lib/MojoPlugins/Health.pm ---
@@ -159,6 +159,9 @@ sub register {
cachegroups
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
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
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/209
Cleaned up LICENSE file and removed several 3rd party files that were not
used.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/209
Fix a couple of license issues for 1.8.0
- WebAppers Progress Bar removed from LICENSE
- normalize.css added to LICENSE (under bootstrap)
- year updated in NOTICE
You can
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/206
TO - remove duplicate license and repeated code
in lib/Schema/Result/ files
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/203
remove go program compiles from TO spec
previously removed go programs -- removing from spec file
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/201
decouple pgmigration from postgres container
changes from @dewrich -- also updated instructions in README
You can merge this pull request into a Git repository by running
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/198
TO - postgres update for db/admin.pl
simplifies process for creating roles/databases for dev modes (development,
test, ...). When using `db/admin.pl --env= createdb
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/192
traffic_ops docker build -- no need to compile git
git version on centos 7 is enough up-to-date -- don't compile latest version
You can merge this pull request into a Git
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/181#discussion_r96442294
--- Diff: traffic_ops/app/lib/API/Cdn.pm ---
@@ -264,7 +264,8 @@ sub get_traffic_monitor_config {
{ 'cdn.name' => $cdn_n
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/182
this was merged to master -- I will cherry-pick to 1.8.x
---
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
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/170
get dnssec from old-releases dir
for 1.8.0 -- dnssec made a new release 0.13. The version we have been
using is 0.12, which was moved to old-releases directory
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/168
merge postgres branch
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol
merge-postgres
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/126#discussion_r95027416
--- Diff: traffic_ops/install/bin/input.json ---
@@ -0,0 +1,169 @@
+{
+ "/opt/traffic_ops/app/conf/production/database
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/126#discussion_r95029992
--- Diff: traffic_ops/install/bin/postinstall-new ---
@@ -0,0 +1,781 @@
+#!/usr/bin/perl
+
+#
+# Licensed under the Apache
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/162
[TC-84] revert change to compile goose during rpm install
decided instead to add instructions for user to install goose and Perl
modules needed for postinstall. These will come
Github user dangogh closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/157
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/157
[TC-84] compiles goose from source; installs cpan modules during rpm install
This is needed for 1.8 -- apache compliance does not allow release of goose
binary
You can merge
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/147
[TC- 39] update LICENSE with details on most licenses
Could not find text for CC license for percentagebar.js, so that still
needs to be resolved..
You can merge this pull
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/139
merged
---
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
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/30
@limited this should be covered by #126, don't you think?
---
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
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/138
[TC-7] break down mysql-to-postgres conversion docker-compose setup
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh
Github user dangogh closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/137
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/137
[TC-7] break down mysql-to-postgres conversion docker-compose setup
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh
Github user dangogh closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/135
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/135
merged
---
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
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/135
Psql fix pgloader
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol
psql-fix-pgloader
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/130
[TC-65] remove build number from tarball name
minor change to build script..
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user dangogh commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/128#discussion_r91369713
--- Diff:
traffic_ops/app/lib/Schema/Result/DeliveryServiceInfoForCdnList.pm ---
@@ -0,0 +1,103 @@
+
+
+#
+# Copyright
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/123
add Apache license to psql docker files
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dangogh/incubator-trafficcontrol
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/122
shouldn't be on master... needs to be psql-rebase branch..
---
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
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/121
Rat and apache license
Added to .rat-excludes for examining for Apache license requirement.
Added required license to a bunch of files.
You can merge this pull request
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/117
already did this with cherry-pick -x to 1.8.x -- please close
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user dangogh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/118
[TC-62] explicitly define project name when creating the tarball
to be pulled in to 1.8.0
You can merge this pull request into a Git repository by running:
$ git pull https
Github user dangogh commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/113
merged.. ok to close
---
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
101 - 175 of 175 matches
Mail list logo