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

[DO NOT MERGE] Release 2.31.0 for Mariner #20620

Closed
wants to merge 2 commits into from

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Dec 6, 2021

🛑 DO NOT MERGE

This PR is only used to build Mariner RPM.

It includes changes from

@jiasli jiasli changed the title [DO NOT MERGE] Release 2.31.0 for mariner [DO NOT MERGE] Release 2.31.0 for Mariner Dec 6, 2021
@jiasli jiasli force-pushed the release-2.31.0-mariner branch from 3b46d08 to 84d6bd2 Compare December 6, 2021 06:02
@jiasli jiasli requested review from mfrw and chalamalasetty December 6, 2021 07:04
@jiasli jiasli self-assigned this Dec 6, 2021
@@ -4,7 +4,7 @@ FROM cblmariner.azurecr.io/base/core:${tag} AS build-env
ARG cli_version=dev

RUN tdnf update -y
RUN tdnf install -y rpm-build gcc libffi-devel python3-devel openssl-devel make diffutils patch dos2unix python3-virtualenv perl
RUN tdnf install -y binutils file rpm-build gcc libffi-devel python3-devel openssl-devel make diffutils patch dos2unix python3-virtualenv perl
Copy link
Member

Choose a reason for hiding this comment

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

Did some local testing with these changes.
LGTM :)

image

@yonzhan yonzhan added this to the Backlog milestone Dec 22, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented Dec 22, 2021

build Mariner RPM

@jiasli jiasli closed this Mar 11, 2022
@jiasli jiasli deleted the release-2.31.0-mariner branch March 11, 2022 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants