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

{Release} Fix CI issue for release branch #21277

Merged
merged 3 commits into from
Feb 14, 2022

Conversation

zhoxing-ms
Copy link
Contributor

Description

Cherry pick from dev branch:

  1. [Packaging] BREAKING CHANGE: Drop jmespath-terminal from docker image #21206
  2. {Packaging} Fix CI job "Test Yum Package" by using centos7 #21207
  3. {ADO} Pin version 2.1.17 for cred scan #21230

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@zhoxing-ms zhoxing-ms changed the base branch from dev to release February 14, 2022 02:00
@@ -43,8 +43,7 @@ RUN apk add --no-cache bash openssh ca-certificates jq curl openssl perl git zip
ARG JP_VERSION="0.1.3"

RUN curl -L https://github.com/jmespath/jp/releases/download/${JP_VERSION}/jp-linux-amd64 -o /usr/local/bin/jp \
&& chmod +x /usr/local/bin/jp \
&& pip install --no-cache-dir --upgrade jmespath-terminal
&& chmod +x /usr/local/bin/jp
Copy link
Member

Choose a reason for hiding this comment

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

#21206 is a breaking change and should be included in release notes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

@jiasli jiasli mentioned this pull request Feb 14, 2022
3 tasks
@zhoxing-ms zhoxing-ms merged commit e861f13 into Azure:release Feb 14, 2022
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.

None yet

7 participants