Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

@zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Sep 27, 2024

#8079 reduces updating the Postgres version to updating the POSTGRES_VERSION environment variable in .env.


Which Traffic Control components are affected by this PR?

  • Documentation
  • Traffic Control Cache Config (t3c, formerly ORT) - tests
  • Traffic Control Health Client (tc-health-client) - tests
  • Traffic Ops
  • CDN in a Box for Developers
  • CDN in a Box
  • Traffic Ops DB tests
  • Automation - GHA workflows

What is the best way to verify this PR?

Verify that tests pass.

PR submission checklist

@zrhoffman zrhoffman added Traffic Ops related to Traffic Ops documentation related to documentation tests related to tests and/or testing infrastructure tech debt rework due to choosing easy/limited solution 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 improvement The functionality exists but it could be improved in some way. dependencies Pull requests that update a dependency file dev-ciab Related to CDN in a Box for Developers labels Sep 27, 2024
@zrhoffman zrhoffman changed the title Update Postgres version to 1.13 Update Postgres version to 13.16 Sep 27, 2024
@codecov
Copy link

codecov bot commented Sep 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.55%. Comparing base (096b2e2) to head (238e058).
Report is 3 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #8079       +/-   ##
=============================================
+ Coverage     29.27%   68.55%   +39.27%     
  Complexity       98       98               
=============================================
  Files           860      319      -541     
  Lines         91394    12313    -79081     
  Branches       1013     1013               
=============================================
- Hits          26758     8441    -18317     
+ Misses        62481     3516    -58965     
+ Partials       2155      356     -1799     
Flag Coverage Δ
golib_unit ?
grove_unit ?
t3c_unit ?
traffic_monitor_unit ?
traffic_ops_unit ?
traffic_portal_v2 74.33% <ø> (-0.02%) ⬇️
traffic_stats_unit ?
unit_tests 74.33% <ø> (+47.74%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rimashah25 rimashah25 merged commit ab2578c into apache:master Sep 27, 2024
@zrhoffman zrhoffman mentioned this pull request Sep 27, 2024
4 tasks
rimashah25 pushed a commit to rimashah25/trafficcontrol that referenced this pull request Sep 27, 2024
* 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
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]>
@zrhoffman zrhoffman deleted the postgres-13.16 branch September 28, 2024 08:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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 dependencies Pull requests that update a dependency file dev-ciab Related to CDN in a Box for Developers documentation related to documentation improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution tests related to tests and/or testing infrastructure Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants