Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use docker compose plugin #8056

Merged
merged 5 commits into from
Aug 8, 2024
Merged

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
37 checks passed
@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 <asf-ci-trafficcontrol@users.noreply.github.com>

(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 <asf-ci-trafficcontrol@users.noreply.github.com>
Co-authored-by: Zach Hoffman <zrhoffman@apache.org>

(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 <asf-ci-trafficcontrol@users.noreply.github.com>

(cherry picked from commit f647d58)

---------

Co-authored-by: Zach Hoffman <zrhoffman@apache.org>
Co-authored-by: asf-ci-trafficcontrol <127552316+asf-ci-trafficcontrol@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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