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.

Sign up for GitHub

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

Jump to bottom

feat(root): Add Postgres 17 #4552

Merged
merged 12 commits into from
Nov 13, 2024
Merged

feat(root): Add Postgres 17 #4552

merged 12 commits into from
Nov 13, 2024

Conversation

tasso94
Copy link
Member

@tasso94 tasso94 commented Aug 21, 2024 •
edited by psavidis
Loading

Context

This PR contains the following changes:

  • Reference to the PR changes on the camunda-bpm-jenkins-shared-library
  • Add Postgres 17 Support
    • Executed by Podspecs

Related-to: #4165

Notes to DRI

Screenshots

  • Main pipeline builds
    • non postgres images will run with labels e.g sqlserver_200 (screenshot)
    • postgres 17 runs with combo-podspec (screenshot)

@tasso94 tasso94 self-assigned this Aug 21, 2024
@tasso94 tasso94 added ci:postgresql Runs the builds for the PostgreSQL database. ci:skipTests Skips tests execution in Assembly and Distro-ee stages. labels Aug 21, 2024
@tasso94 tasso94 mentioned this pull request Oct 15, 2024
Closed
7 tasks
@tasso94 tasso94 force-pushed the postgresql-17-declarative-pod-spec branch from 5fe3a0d to 41fb1f2 Compare October 15, 2024 11:18
@psavidis psavidis force-pushed the postgresql-17-declarative-pod-spec branch from 41fb1f2 to f8a404e Compare October 24, 2024 09:02
psavidis
psavidis reviewed Oct 24, 2024
View reviewed changes
.ci/daily/Jenkinsfile Outdated Show resolved Hide resolved
psavidis
psavidis reviewed Oct 24, 2024
View reviewed changes
.ci/sidetrack/Jenkinsfile Outdated Show resolved Hide resolved
psavidis
psavidis reviewed Oct 24, 2024
View reviewed changes
Jenkinsfile Outdated Show resolved Hide resolved
psavidis
psavidis reviewed Oct 24, 2024
View reviewed changes
.ci/config/matrices.yaml Outdated Show resolved Hide resolved
@psavidis psavidis force-pushed the postgresql-17-declarative-pod-spec branch 5 times, most recently from 7cd880e to 2a02229 Compare October 31, 2024 10:11
@psavidis psavidis added ci:migration Runs the process instance migration builds. ci:rolling-update Runs the rolling update builds. labels Oct 31, 2024
@psavidis psavidis force-pushed the postgresql-17-declarative-pod-spec branch 3 times, most recently from 5218f09 to ae01c7b Compare November 1, 2024 10:22
@psavidis psavidis added ci:all-db Runs the builds for all databases. ci:default-build Runs the builds that have no explicit trigger (e.g. different history levels). and removed ci:postgresql Runs the builds for the PostgreSQL database. labels Nov 1, 2024
@yanavasileva yanavasileva added ci:e2e Runs the frontend end-to-end tests. ci:run Runs the integration tests for the Run distribution. and removed ci:e2e Runs the frontend end-to-end tests. labels Nov 1, 2024
@psavidis psavidis changed the title Postgresql 17 declarative pod spec chore(.ci): Add Postgres 17 Nov 8, 2024
psavidis
psavidis reviewed Nov 8, 2024
View reviewed changes
.ci/config/matrices.yaml Outdated Show resolved Hide resolved
@psavidis psavidis removed the ci:default-build Runs the builds that have no explicit trigger (e.g. different history levels). label Nov 8, 2024
@psavidis psavidis force-pushed the postgresql-17-declarative-pod-spec branch 2 times, most recently from bbdeb98 to 2db018f Compare November 8, 2024 14:39
@psavidis
Copy link
Contributor

psavidis commented Nov 8, 2024 •
edited
Loading

TODO after collecting build proof for large-data-tests-postgresql-170, rolling-update-postgresql-170 remaining stages.

Update: Removed postgres 17 from stages (commit1, commit2)

@psavidis psavidis assigned psavidis and unassigned tasso94 Nov 10, 2024
@psavidis psavidis requested review from mboskamp and removed request for mboskamp November 10, 2024 15:30
@psavidis psavidis changed the title chore(.ci): Add Postgres 17 chore(root): Add Postgres 17 Nov 11, 2024
@psavidis psavidis changed the title chore(root): Add Postgres 17 feat(root): Add Postgres 17 Nov 11, 2024
@psavidis psavidis requested a review from mboskamp November 11, 2024 14:48
mboskamp
mboskamp approved these changes Nov 12, 2024
View reviewed changes
Copy link
Member

@mboskamp mboskamp left a comment

Choose a reason for hiding this comment

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

👍 Looks good, don't forget to remove the test branches.

@psavidis
Copy link
Contributor

👍 Looks good, don't forget to remove the test branches.

Test branch is removed. Merging can proceed.

@psavidis psavidis force-pushed the postgresql-17-declarative-pod-spec branch from 0dcea6a to baf0b80 Compare November 13, 2024 10:24
@psavidis psavidis merged commit 5592755 into master Nov 13, 2024
2 of 3 checks passed
@psavidis psavidis deleted the postgresql-17-declarative-pod-spec branch November 13, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psavidis psavidis psavidis left review comments

@mboskamp mboskamp mboskamp approved these changes

Assignees

@psavidis psavidis

Labels
ci:migration Runs the process instance migration builds. ci:postgresql Runs the builds for the PostgreSQL database. ci:rolling-update Runs the rolling update builds. ci:skipTests Skips tests execution in Assembly and Distro-ee stages.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tasso94 @psavidis @mboskamp @yanavasileva