You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/03/11 21:50:20 UTC

[GitHub] [trafficcontrol] rawlinp opened a new pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

rawlinp opened a new pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630


   ## What does this PR (Pull Request) do?
   Change the supported Traffic Ops database version from PostgreSQL 9.6 to 13.2. Updates documentation, cdn-in-a-box, the Traffic Ops rpm, the traffic_ops_db container + tests, and the traffic_ops_ort integration tests to PostgreSQL 13.2.
   
   Additionally, it makes one minor change to `db/admin.go` to print `psql` stderr output in case the `createdb` or `create_user` commands fail.
   
   ## Which Traffic Control components are affected by this PR?
   - CDN in a Box
   - Documentation
   - CI tests
   
   ## What is the best way to verify this PR?
   Run cdn-in-a-box and the TO API tests, ensure they still pass. Build and run the traffic_ops_db container. Build and run the traffic_ops_db docker tests. Build and run the traffic_ops_ort integration tests. 
   
   ## The following criteria are ALL met by this PR
   - [x] Tests now use postgres 13.2
   - [x] This PR includes documentation
   - [x] This PR includes an update to CHANGELOG.md
   - [x] This PR includes any and all required license headers
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY** (see [the Apache Software Foundation's security guidelines](https://www.apache.org/security/) for details)
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r598847007



##########
File path: docs/source/admin/traffic_ops.rst
##########
@@ -237,6 +237,11 @@ To upgrade from older Traffic Ops versions, stop the service, use :manpage:`yum(
 
 After this completes, see Guide_ for instructions on running the :program:`postinstall` script. Once the :program:`postinstall` script, has finished, run the following command as the root user (or with :manpage:`sudo(8)`): ``systemctl start traffic_ops`` to start the service.
 
+Upgrading to 6.0
+----------------
+
+As of 6.0, Traffic Ops supports PostgreSQL version 13.2. In order to migrate from the prior PostgreSQL version 9.6, it is recommended to use the `pg_upgrade <https://www.postgresql.org/docs/13/pgupgrade.html>`_ tool.

Review comment:
       Might want to specify that `6.0` refers to Apache Traffic Control, since the API has its own versioning




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r599111654



##########
File path: traffic_ops_ort/testing/docker/traffic_ops/Dockerfile
##########
@@ -35,12 +35,16 @@ RUN set -o nounset -o errexit && \
 	if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
 		use_repo=''; \
 		enable_repo=''; \
+		# needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
+		dnf -y install centos-release-scl-rh; \

Review comment:
       The error mentioned above relating to `devtoolset-7-gcc` and `install_goose.sh` happens in this Dockerfile, too.

##########
File path: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
##########
@@ -35,12 +35,16 @@ RUN set -o nounset -o errexit && \
 	if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
 		use_repo=''; \
 		enable_repo=''; \
+		# needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
+		dnf -y install centos-release-scl-rh; \

Review comment:
       Because `centos-release-scl-rh` is installed, `devtoolset-7-gcc` ends up providing the `gcc` dependency instead of `gcc`, which ends up making `install_goose.sh` fail because `/opt/rh/devtoolset-7/root/usr/bin/gcc` is not in the PATH:
   
   ```
   #11 [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all
   #11 sha256:31d9b30b624f1d81c87461e7d564dc4d8d75ec98e04a912c21a9ca3b4a4848c5
   #11 1.086 Now installing goose
   #11 1.524 bitbucket.org/liamstask/goose (download)
   #11 2.710 github.com/go-sql-driver/mysql (download)
   #11 4.319 github.com/kylelemons/go-gypsy (download)
   #11 5.474 github.com/lib/pq (download)
   #11 6.978 github.com/mattn/go-sqlite3 (download)
   #11 9.613 github.com/ziutek/mymysql (download)
   #11 11.07 github.com/lib/pq/oid
   #11 11.09 github.com/kylelemons/go-gypsy/yaml
   #11 11.09 github.com/lib/pq/scram
   #11 11.09 github.com/ziutek/mymysql/mysql
   #11 11.10 github.com/go-sql-driver/mysql
   #11 11.12 github.com/lib/pq
   #11 11.13 github.com/mattn/go-sqlite3
   #11 11.14 # github.com/mattn/go-sqlite3
   #11 11.14 exec: "gcc": executable file not found in $PATH
   #11 11.18 github.com/ziutek/mymysql/native
   #11 11.33 github.com/ziutek/mymysql/godrv
   #11 11.41 Error on line 23 of ./install_goose.sh
   #11 ERROR: executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ------
    > [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all:
   ------
   executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r598871755



##########
File path: docs/source/admin/traffic_ops.rst
##########
@@ -237,6 +237,11 @@ To upgrade from older Traffic Ops versions, stop the service, use :manpage:`yum(
 
 After this completes, see Guide_ for instructions on running the :program:`postinstall` script. Once the :program:`postinstall` script, has finished, run the following command as the root user (or with :manpage:`sudo(8)`): ``systemctl start traffic_ops`` to start the service.
 
+Upgrading to 6.0
+----------------
+
+As of 6.0, Traffic Ops supports PostgreSQL version 13.2. In order to migrate from the prior PostgreSQL version 9.6, it is recommended to use the `pg_upgrade <https://www.postgresql.org/docs/13/pgupgrade.html>`_ tool.

Review comment:
       Done!




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman merged pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman merged pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804215848


   > Oops, I missed `traffic_ops/INSTALL.md`, but I purposely left the others alone since they are either experimental or largely unmaintained already.
   
   Makes sense for `experimental/graphql.sample`, but I know that @rob05c still uses `infrastructure/docker/traffic_ops/*` for working on ORT subcomponents.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804200373


   Oops, I missed `traffic_ops/INSTALL.md`, but I purposely left the others alone since they are either experimental or largely unmaintained already.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804245509


   Really? That is surprising, because that Dockerfile depends on something that was removed from the repo 2 years ago, so I assume it's been broken/unmaintained for at least the last 2 years. If it was maintained, I wouldn't mind updating it, but I don't want to waste time updating things that have been unused/unmaintained for 2 years.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-800446263


   Rebased to resolve merge conflicts.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804333758


   Ok, I got the infrastructure/docker/traffic_ops containers to build properly, and the db/db_init containers appear to run/complete successfully. However, the to_server container fails to run, and I don't want to go down that rabbit hole too far. That container needs some serious TLC, because it is still making requests to old Perl non-API routes like `/phys_location/create` which were disabled quite a long time ago. It's also hitting API 1.2 which is due to be removed soon as well.
   
   Anyways, I'd consider the DB upgrade side taken care of. Fixing all of that ☝️  is out of scope of this PR IMO.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804377491


   
   > When building CDN in a Box (and the RPMs) for CentOS 7, the `trafficops` service fails to build.
   > 
   > ```
   > #14 18.37 Error: Package: postgresql13-devel-13.2-1PGDG.rhel7.x86_64 (pgdg13)
   > #14 18.37            Requires: llvm-toolset-7-clang >= 4.0.1
   > ```
   
   The `to_server` service of `traffic_ops_ort/testing/docker` has the same issue when building with build arg `RHEL_VERSION=7`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rob05c commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rob05c commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804250699


   I use that Docker container almost daily. I don't often do a clean rebuild, I usually upgrade in-place. I'm pretty sure I've done a clean rebuild less than 2 years ago, but maybe not.
   
   Standalone Dockerfile containers for TO and other things seem really useful to me (and are). But I guess if I'm literally the only one using them, I might have to maintain them. 
   
   I'll try to find time to update it at some point.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r599111102



##########
File path: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
##########
@@ -35,12 +35,16 @@ RUN set -o nounset -o errexit && \
 	if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
 		use_repo=''; \
 		enable_repo=''; \
+		# needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
+		dnf -y install centos-release-scl-rh; \

Review comment:
       Because `centos-release-scl-rh` is installed, `devtoolset-7-gcc` ends up providing the `gcc` Traffic Ops dependency instead of `gcc`, which ends up making `install_goose.sh` fail because `/opt/rh/devtoolset-7/root/usr/bin/gcc` is not in the PATH:
   
   ```
   #11 [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all
   #11 sha256:31d9b30b624f1d81c87461e7d564dc4d8d75ec98e04a912c21a9ca3b4a4848c5
   #11 1.086 Now installing goose
   #11 1.524 bitbucket.org/liamstask/goose (download)
   #11 2.710 github.com/go-sql-driver/mysql (download)
   #11 4.319 github.com/kylelemons/go-gypsy (download)
   #11 5.474 github.com/lib/pq (download)
   #11 6.978 github.com/mattn/go-sqlite3 (download)
   #11 9.613 github.com/ziutek/mymysql (download)
   #11 11.07 github.com/lib/pq/oid
   #11 11.09 github.com/kylelemons/go-gypsy/yaml
   #11 11.09 github.com/lib/pq/scram
   #11 11.09 github.com/ziutek/mymysql/mysql
   #11 11.10 github.com/go-sql-driver/mysql
   #11 11.12 github.com/lib/pq
   #11 11.13 github.com/mattn/go-sqlite3
   #11 11.14 # github.com/mattn/go-sqlite3
   #11 11.14 exec: "gcc": executable file not found in $PATH
   #11 11.18 github.com/ziutek/mymysql/native
   #11 11.33 github.com/ziutek/mymysql/godrv
   #11 11.41 Error on line 23 of ./install_goose.sh
   #11 ERROR: executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ------
    > [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all:
   ------
   executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804413365


   Nice catch! Both builds should be fixed now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#issuecomment-804261958


   Oh, I must've had an old branch checked out. It looks like it was updated a year ago to remove the reference to a deleted file, so it was only broken for a year before anyone realized it. I'll see if it still builds cleanly and runs as-is.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r599111102



##########
File path: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
##########
@@ -35,12 +35,16 @@ RUN set -o nounset -o errexit && \
 	if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
 		use_repo=''; \
 		enable_repo=''; \
+		# needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
+		dnf -y install centos-release-scl-rh; \

Review comment:
       Because `centos-release-scl-rh` is installed, `devtoolset-7-gcc` ends up providing the `gcc` Traffic Ops dependency instead of `gcc` itself, which ends up making `install_goose.sh` fail because `/opt/rh/devtoolset-7/root/usr/bin/gcc` is not in the PATH:
   
   ```
   #11 [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all
   #11 sha256:31d9b30b624f1d81c87461e7d564dc4d8d75ec98e04a912c21a9ca3b4a4848c5
   #11 1.086 Now installing goose
   #11 1.524 bitbucket.org/liamstask/goose (download)
   #11 2.710 github.com/go-sql-driver/mysql (download)
   #11 4.319 github.com/kylelemons/go-gypsy (download)
   #11 5.474 github.com/lib/pq (download)
   #11 6.978 github.com/mattn/go-sqlite3 (download)
   #11 9.613 github.com/ziutek/mymysql (download)
   #11 11.07 github.com/lib/pq/oid
   #11 11.09 github.com/kylelemons/go-gypsy/yaml
   #11 11.09 github.com/lib/pq/scram
   #11 11.09 github.com/ziutek/mymysql/mysql
   #11 11.10 github.com/go-sql-driver/mysql
   #11 11.12 github.com/lib/pq
   #11 11.13 github.com/mattn/go-sqlite3
   #11 11.14 # github.com/mattn/go-sqlite3
   #11 11.14 exec: "gcc": executable file not found in $PATH
   #11 11.18 github.com/ziutek/mymysql/native
   #11 11.33 github.com/ziutek/mymysql/godrv
   #11 11.41 Error on line 23 of ./install_goose.sh
   #11 ERROR: executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ------
    > [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all:
   ------
   executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r598847007



##########
File path: docs/source/admin/traffic_ops.rst
##########
@@ -237,6 +237,11 @@ To upgrade from older Traffic Ops versions, stop the service, use :manpage:`yum(
 
 After this completes, see Guide_ for instructions on running the :program:`postinstall` script. Once the :program:`postinstall` script, has finished, run the following command as the root user (or with :manpage:`sudo(8)`): ``systemctl start traffic_ops`` to start the service.
 
+Upgrading to 6.0
+----------------
+
+As of 6.0, Traffic Ops supports PostgreSQL version 13.2. In order to migrate from the prior PostgreSQL version 9.6, it is recommended to use the `pg_upgrade <https://www.postgresql.org/docs/13/pgupgrade.html>`_ tool.

Review comment:
       Might want to specify that this refers to Apache Traffic Control, since the API has its own versioning




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman commented on a change in pull request #5630: Update Traffic Ops database from PostgreSQL 9.6 to 13.2

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on a change in pull request #5630:
URL: https://github.com/apache/trafficcontrol/pull/5630#discussion_r599111102



##########
File path: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
##########
@@ -35,12 +35,16 @@ RUN set -o nounset -o errexit && \
 	if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
 		use_repo=''; \
 		enable_repo=''; \
+		# needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
+		dnf -y install centos-release-scl-rh; \

Review comment:
       Because `centos-release-scl-rh` is installed, `devtoolset-7-gcc` ends up providing the `gcc` dependency of `golang-bin`, which ends up making `install_goose.sh` fail because `/opt/rh/devtoolset-7/root/usr/bin/gcc` is not in the PATH:
   
   ```
   #11 [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all
   #11 sha256:31d9b30b624f1d81c87461e7d564dc4d8d75ec98e04a912c21a9ca3b4a4848c5
   #11 1.086 Now installing goose
   #11 1.524 bitbucket.org/liamstask/goose (download)
   #11 2.710 github.com/go-sql-driver/mysql (download)
   #11 4.319 github.com/kylelemons/go-gypsy (download)
   #11 5.474 github.com/lib/pq (download)
   #11 6.978 github.com/mattn/go-sqlite3 (download)
   #11 9.613 github.com/ziutek/mymysql (download)
   #11 11.07 github.com/lib/pq/oid
   #11 11.09 github.com/kylelemons/go-gypsy/yaml
   #11 11.09 github.com/lib/pq/scram
   #11 11.09 github.com/ziutek/mymysql/mysql
   #11 11.10 github.com/go-sql-driver/mysql
   #11 11.12 github.com/lib/pq
   #11 11.13 github.com/mattn/go-sqlite3
   #11 11.14 # github.com/mattn/go-sqlite3
   #11 11.14 exec: "gcc": executable file not found in $PATH
   #11 11.18 github.com/ziutek/mymysql/native
   #11 11.33 github.com/ziutek/mymysql/godrv
   #11 11.41 Error on line 23 of ./install_goose.sh
   #11 ERROR: executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ------
    > [ 7/11] RUN ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all:
   ------
   executor failed running [/bin/sh -c ./install_goose.sh && rm ./install_goose.sh && dnf -y remove git && dnf clean all]: exit code: 1
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org