Skip to content
Snippets Groups Projects

Add PostgreSQL 14 builds

Merged Dustin Collins requested to merge add-postgres-14 into master

What does this MR do?

Adds postgres 14 to the builds in order to test our support of it.

Related issues

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion

Required

  • Merge Request Title, and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com
  • Pipeline is green on dev.gitlab.org if the change is touching anything besides documentation or internal cookbooks
  • trigger-package has a green pipeline running against latest commit

Expected (please provide an explanation if not completing)

  • Test plan indicating conditions for success has been posted and passes
  • Documentation created/updated
  • Tests added
  • Integration tests added to GitLab QA
  • Equivalent MR/issue for the GitLab Chart opened
Edited by Clemens Beck

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Dustin Collins added 3 commits

    added 3 commits

    Compare with previous version

  • Dustin Collins added 13 commits

    added 13 commits

    Compare with previous version

  • Dustin Collins resolved all threads

    resolved all threads

  • Dustin Collins added 6 commits

    added 6 commits

    • 3f9038ef - Add PostgreSQL 14 builds
    • 43c9f366 - Add PostgreSQL 14 builds
    • 7fc99207 - Merge branch 'add-postgres-14' of gitlab.com:gitlab-org/omnibus-gitlab into add-postgres-14
    • 0c5eb628 - fix postgres config names
    • 942b37e0 - fix postgresql patch
    • baa8d92d - Merge branch 'add-postgres-14' of dev.gitlab.org:gitlab/omnibus-gitlab into add-postgres-14

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    • c68dd66b - move file link logic to postgresql, update dependencies

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    Compare with previous version

  • DJ Mountney
  • Dustin Collins added 1 commit

    added 1 commit

    • bd60a222 - move libpq to postgresql_new

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    • 73358f00 - restore original dependencies

    Compare with previous version

  • Dustin Collins added 22 commits

    added 22 commits

    Compare with previous version

  • Dustin Collins resolved all threads

    resolved all threads

  • Initial testing looks good. I am able to run the omnibus Docker container with postgres 14.7 without issue. I used this compose file for testing:

    version: '3.6'
    services:
      gitlab:
        container_name: 'gitlab'
        image: 'registry.gitlab.com/gitlab-org/omnibus-gitlab/gitlab-ee:15.11.0-rfbranch.844895870.7888ea29-0'
        hostname: 'gitlab.dev'
        environment:
          GITLAB_OMNIBUS_CONFIG: |
            external_url 'http://gitlab.dev'
            letsencrypt['enable'] = false
            gitlab_rails['initial_root_password'] = 'rtrtrtrt'
        ports:
          - '8080:80'
          - '8443:443'
        shm_size: '256m'

    Screenshot_from_2023-04-21_14-03-13

    Edited by Dustin Collins
  • mentioned in issue #7396 (closed)

  • mentioned in issue #7702 (closed)

  • Dmytro Makovey changed the description

    changed the description

  • Dustin Collins changed the description

    changed the description

  • Setting this MR to close issue #7702 (closed). Updating this MR to match changes made for PG13 bump,!5390 (diffs).

  • Dustin Collins added 9 commits

    added 9 commits

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    Compare with previous version

  • This is ready for review.

  • Dustin Collins marked this merge request as ready

    marked this merge request as ready

  • Dmytro Makovey requested review from @dmakovey

    requested review from @dmakovey

  • Dmytro Makovey changed the description

    changed the description

  • Dustin Collins marked this merge request as draft

    marked this merge request as draft

  • added workflowin dev label and removed workflowin review label

  • Dustin Collins mentioned in merge request !6856 (merged)

    mentioned in merge request !6856 (merged)

  • Dustin Collins added 142 commits

    added 142 commits

    Compare with previous version

  • Contributor
    5 Warnings
    :warning: 66a71ae4: The commit subject must start with a capital letter. For more information, take a look at our Commit message guidelines.
    :warning: bf387f04: The commit subject must start with a capital letter. For more information, take a look at our Commit message guidelines.
    :warning: e1d58435: The commit subject must start with a capital letter. For more information, take a look at our Commit message guidelines.
    :warning: ab9d182a: The commit subject must start with a capital letter. For more information, take a look at our Commit message guidelines.
    :warning:

    You've made some changes to the software definitions.

    Any change in software definition requires:

    • package rebuild to verify that builds across the supported platforms are successful.
    • manual verification to confirm the package is installable
    • uses an approved license

    If the change does not affect existing functionality, for example when
    resolving rubocop warnings, adding comments, or fixing typos, add one of
    the following labels:

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • Dustin Collins added 1 commit

    added 1 commit

    • 3ef4ee96 - remove rest of libpq from postgres13 config

    Compare with previous version

  • Dustin Collins added 1 commit

    added 1 commit

    • a73de58d - Restore postgresql_old for now

    Compare with previous version

  • Dustin Collins added 16 commits

    added 16 commits

    Compare with previous version

  • Ben Prescott_ mentioned in merge request gitlab!121327 (merged)

    mentioned in merge request gitlab!121327 (merged)

  • Dustin Collins added 4 commits

    added 4 commits

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading