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

Update Postgres version to 13.16 #8079

Merged
merged 4 commits into from
Sep 27, 2024
Merged

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
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
34 checks passed
@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 join this conversation on GitHub. Already have an account? Sign in to comment
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