Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@zrhoffman
Copy link
Member

#8056 fixes #8055 removing the version key from compose files and making the project use docker compose instead of docker-compose.


Which Traffic Control components are affected by this PR?

  • Traffic Control Cache Config (t3c, formerly ORT)
  • Traffic Control Health Client (t3c-health-client)
  • Traffic Monitor
  • Traffic Ops
  • Traffic Portal
  • Documentation
  • CDN in a Box
  • Automation

What is the best way to verify this PR?

Verify that the GitHub Actions pass now.

PR submission checklist

Excludes Dockerfile-tsb and the vendor directory.

    sed -i 's/docker-compose /docker compose /g' $(git grep -l 'docker-compose ')
    pattern="$(<<'PATTERN' cat
    docker-compose\($\|['`"*,)]\)
    PATTERN
    )"
    sed -i "s/${pattern}/docker compose\1/g" -- $(git grep -l "$pattern")
    sed -i 's/"docker compose"/"docker", "compose"/g' $(git grep -Fl '"docker compose"')
sed -i '/^version:/d' $(git grep -il '^version:' '*docker-compose*.y*ml')
This is necessary for the plugin version of docker compose to query
exited containers.
@zrhoffman zrhoffman added bug something isn't working as intended Traffic Ops related to Traffic Ops Traffic Monitor related to Traffic Monitor Traffic Portal v1 related to Traffic Portal version 1 documentation related to documentation cdn-in-a-box related to the Docker-based CDN-in-a-Box system build related to the build process database relating to setup/installation/structure of the Traffic Ops database cache-config Cache config generation automation related to automated testing/deployment/packaging etc. tc-health-client Traffic Control Health Client dev-ciab Related to CDN in a Box for Developers labels Aug 7, 2024
@rawlinp rawlinp merged commit 8a84c50 into apache:master Aug 8, 2024
@zrhoffman zrhoffman deleted the docker-compose-plugin branch August 8, 2024 17:29
Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

zrhoffman added a commit to zrhoffman/trafficcontrol that referenced this pull request Sep 27, 2024
* docker-compose -> docker compose

Excludes Dockerfile-tsb and the vendor directory.

    sed -i 's/docker-compose /docker compose /g' $(git grep -l 'docker-compose ')
    pattern="$(<<'PATTERN' cat
    docker-compose\($\|['`"*,)]\)
    PATTERN
    )"
    sed -i "s/${pattern}/docker compose\1/g" -- $(git grep -l "$pattern")
    sed -i 's/"docker compose"/"docker", "compose"/g' $(git grep -Fl '"docker compose"')

* Remove version from compose files.

sed -i '/^version:/d' $(git grep -il '^version:' '*docker-compose*.y*ml')

* Include `--all` in docker-compose ps command.

This is necessary for the plugin version of docker compose to query
exited containers.

* Changelog entry

* Support docker compose fallback for older docker versions that do not
have the compose plugin
zrhoffman added a commit to zrhoffman/trafficcontrol that referenced this pull request Sep 27, 2024
* docker-compose -> docker compose

Excludes Dockerfile-tsb and the vendor directory.

    sed -i 's/docker-compose /docker compose /g' $(git grep -l 'docker-compose ')
    pattern="$(<<'PATTERN' cat
    docker-compose\($\|['`"*,)]\)
    PATTERN
    )"
    sed -i "s/${pattern}/docker compose\1/g" -- $(git grep -l "$pattern")
    sed -i 's/"docker compose"/"docker", "compose"/g' $(git grep -Fl '"docker compose"')

* Remove version from compose files.

sed -i '/^version:/d' $(git grep -il '^version:' '*docker-compose*.y*ml')

* Include `--all` in docker-compose ps command.

This is necessary for the plugin version of docker compose to query
exited containers.

* Changelog entry

* Support docker compose fallback for older docker versions that do not
have the compose plugin
rimashah25 pushed a commit to rimashah25/trafficcontrol that referenced this pull request Sep 27, 2024
rimashah25 pushed a commit to rimashah25/trafficcontrol that referenced this pull request Sep 27, 2024
rimashah25 pushed a commit to rimashah25/trafficcontrol that referenced this pull request Sep 27, 2024
* docker-compose -> docker compose

Excludes Dockerfile-tsb and the vendor directory.

    sed -i 's/docker-compose /docker compose /g' $(git grep -l 'docker-compose ')
    pattern="$(<<'PATTERN' cat
    docker-compose\($\|['`"*,)]\)
    PATTERN
    )"
    sed -i "s/${pattern}/docker compose\1/g" -- $(git grep -l "$pattern")
    sed -i 's/"docker compose"/"docker", "compose"/g' $(git grep -Fl '"docker compose"')

* Remove version from compose files.

sed -i '/^version:/d' $(git grep -il '^version:' '*docker-compose*.y*ml')

* Include `--all` in docker-compose ps command.

This is necessary for the plugin version of docker compose to query
exited containers.

* Changelog entry

* Support docker compose fallback for older docker versions that do not
have the compose plugin
zrhoffman added a commit that referenced this pull request Sep 27, 2024
* Updated version

* Updated changelog

* Use `docker compose` plugin (#8056)

* docker-compose -> docker compose

Excludes Dockerfile-tsb and the vendor directory.

    sed -i 's/docker-compose /docker compose /g' $(git grep -l 'docker-compose ')
    pattern="$(<<'PATTERN' cat
    docker-compose\($\|['`"*,)]\)
    PATTERN
    )"
    sed -i "s/${pattern}/docker compose\1/g" -- $(git grep -l "$pattern")
    sed -i 's/"docker compose"/"docker", "compose"/g' $(git grep -Fl '"docker compose"')

* Remove version from compose files.

sed -i '/^version:/d' $(git grep -il '^version:' '*docker-compose*.y*ml')

* Include `--all` in docker-compose ps command.

This is necessary for the plugin version of docker compose to query
exited containers.

* Changelog entry

* Support docker compose fallback for older docker versions that do not
have the compose plugin

* Improve PUT /deliveryservice_request_comments id (#8071)

* Update actions/upload-artifact and actions/download-artifact to v4 (#8081)

Fixes message:
This request has been automatically failed because it uses a deprecated
version of `actions/upload-artifact: v2`. Learn more:
https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

* Update Postgres version to 13.16 (#8079)

* Use :atc-postgres-version: role in documentation

* Get postgres version from .env in Traffic Ops spec file

* Get POSTGRES_VERSION from .env in Dockerfiles and docker-compose files

* Update Postgres version to 13.16

* Update Trafficserver Alpine image to 9.2.4 (#7994)

* No substitution in the name

* Download ATS from the Apache archive

download.apache.org only contains currently-supported releases.

* Use Alpine main repo version 3.16 for adding package libexecinfo-dev

Alpine versions above 3.16 do not contain package libexecinfo-dev, which
ATS needs.

Alpine 3.19 still fails for unknown reasons, but Alpine 3.18 builds with
this fix.

* Set default for GITHUB_REPOSITORY

* Update Apache Traffic Server to 9.2.4

(cherry picked from commit 14007e3)

* Update Go version to 1.23.1 (#8073)

Co-authored-by: asf-ci-trafficcontrol <[email protected]>

(cherry picked from commit f647d58)

* Update Chromedriver Versions (#8084)

Update chromedriver

(cherry picked from commit 289c2c3)

* Update Go version to 1.22.5 (#8034)

* Update Go version to 1.22.5

* Update some images to Rocky Linux 8

---------

Co-authored-by: asf-ci-trafficcontrol <[email protected]>
Co-authored-by: Zach Hoffman <[email protected]>

(cherry picked from commit 0b21b43)

* Update changelog-1 and ciab.yml

* Update Go version to 1.23.1 (#8073)

Co-authored-by: asf-ci-trafficcontrol <[email protected]>

(cherry picked from commit f647d58)

---------

Co-authored-by: Zach Hoffman <[email protected]>
Co-authored-by: asf-ci-trafficcontrol <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automation related to automated testing/deployment/packaging etc. bug something isn't working as intended build related to the build process cache-config Cache config generation cdn-in-a-box related to the Docker-based CDN-in-a-Box system database relating to setup/installation/structure of the Traffic Ops database dev-ciab Related to CDN in a Box for Developers documentation related to documentation tc-health-client Traffic Control Health Client Traffic Monitor related to Traffic Monitor Traffic Ops related to Traffic Ops Traffic Portal v1 related to Traffic Portal version 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Version in "././infrastructure/docker/build/docker-compose.yml" is unsupported

3 participants