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

Bump Citus to 12.1.5 #7649

Merged
merged 6 commits into from
Jul 17, 2024
Merged

Bump Citus to 12.1.5 #7649

merged 6 commits into from
Jul 17, 2024

Conversation

hanefi
Copy link
Member

@hanefi hanefi commented Jul 10, 2024

Blocked on #7648

Copy link

codecov bot commented Jul 10, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Project coverage is 89.19%. Comparing base (4c110fa) to head (15ecc37).
Report is 37 commits behind head on release-12.1.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-12.1    #7649      +/-   ##
================================================
- Coverage         89.55%   89.19%   -0.37%     
================================================
  Files               274      274              
  Lines             59479    59575      +96     
  Branches           7415     7435      +20     
================================================
- Hits              53268    53135     -133     
- Misses             4082     4269     +187     
- Partials           2129     2171      +42     

@gurkanindibay
Copy link
Contributor

can you rerun the arbitary configs again to see the tests success? I think there is a download limit exceed here. It may be fixed with a rerun IMO

We move the CI images to the github container registry.

Given we mostly (if not solely) run these containers on github actions
infra it makes sense to have them hosted closer to where they are
needed.

Image changes: citusdata/the-process#157

(cherry picked from commit e776a7e)
gurkanindibay and others added 5 commits July 17, 2024 15:11
Removes el/7 and ol/7 as runners and update checkout action to v4

We use EL/7 and OL/7 runners to test packaging for these distributions.
However, for the past two weeks, we've encountered errors during the
checkout step in the pipelines. The error message is as follows:
```
/__e/node20/bin/node: /lib64/libm.so.6: version `GLIBC_2.27' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.21' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libc.so.6: version `GLIBC_2.28' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libc.so.6: version `GLIBC_2.25' not found (required by /__e/node20/bin/node)
```
The GCC version within the EL/7 and OL/7 Docker images is 2.17, and we
cannot upgrade it. Therefore, we need to remove these images from the
packaging test pipelines. Consequently, we will no longer verify if the
code builds for EL/7 and OL/7.

However, we are not using these packaging images as runners within the
packaging infrastructure, so we can continue to use these images for
packaging.

Additional Info: I learned that Marlin team fully dropped the el/7
support so we will drop in further releases as well

(cherry picked from commit c603c3e)
Upgrade postgres versions to:
 - 14.12
 -  15.7
 - 16.3

Depends on citusdata/the-process#158

(cherry picked from commit accb7d0)
This PR has following changes :
1. Enable MERGE command for single_shard_distributed targets.

(cherry picked from commit 3c467e6)
(cherry picked from commit 5c09786)
@hanefi hanefi merged commit 15ecc37 into release-12.1 Jul 17, 2024
120 of 121 checks passed
@hanefi hanefi deleted the release-12.1-hanefi branch July 17, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants