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

docker build - bump base image to centos stream 9 #7571

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

alphaprinz
Copy link
Contributor

Explain the changes

  1. Upgrade base image to centos stream 9.
  2. Adapt build.

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

  1. make tests, github actions.
  • Doc added/updated
  • Tests added

src/deploy/NVA_build/NooBaa.Dockerfile Outdated Show resolved Hide resolved
src/deploy/NVA_build/NooBaa.Dockerfile Outdated Show resolved Hide resolved
src/deploy/NVA_build/builder.Dockerfile Outdated Show resolved Hide resolved
@alphaprinz alphaprinz marked this pull request as ready for review November 9, 2023 07:59
@@ -13,7 +13,7 @@
's3select_napi.cpp'
],
'link_settings': {
'libraries': ['/lib64/libboost_thread.so.1.66.0']
'libraries': ['/lib64/libboost_thread.so.1.75.0']
Copy link
Contributor

Choose a reason for hiding this comment

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

@alphaprinz @liranmauda, does this affect downstream builds?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, I've asked Boris.
I think we need to work with a softlink so that specific version won't matter?

Copy link
Contributor

Choose a reason for hiding this comment

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

Downstream is based on RHEL 9 since 5.13
@b-ranto Will this change effect Downstream? maybe we can have a private build to test is. WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

It won't, we use a hack that manually update the version to the one in RHEL 9:

https://pkgs.devel.redhat.com/cgit/containers/mcg-core/tree/Dockerfile?h=rhodf-4.13-rhel-9#n29

i.e. we have had this patch applied downstream for a while now. :-)

Signed-off-by: Amit Prinz Setter <alphaprinz@gmail.com>
@alphaprinz alphaprinz merged commit 41aa04b into noobaa:master Nov 13, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants