Skip to content

Conversation

@jiasli
Copy link
Member

@jiasli jiasli commented Apr 3, 2023

Description
Revert #26005

According to https://eng.ms/docs/products/mariner-linux/gettingstarted/containers/marinercontainerimage

(Mariner) By default containers include a limited set of root certificates specifically for trusting Microsoft specific web sites like packages.microsoft.com. More complete sets of root certificates may be installed if necessary. Click here to read more.

Now we have to manually install ca-certificates to allow Azure CLI on Mariner to access addresses such as

download_url = f"https://downloads.bicep.azure.com/{release_tag}/{{}}"

base_url = 'https://azcopyvnext.azureedge.net/release20211027/azcopy_{}_{}_{}.{}'

@azure-client-tools-bot-prd
Copy link

Validation for Azure CLI Full Test Starting...

Thanks for your contribution!

@ghost ghost added Auto-Assign Auto assign by bot CI CI labels Apr 3, 2023
@ghost ghost requested a review from yonzhan April 3, 2023 06:43
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 3, 2023

CI

@ghost ghost requested review from bebound and jsntcy April 3, 2023 06:44
@ghost ghost assigned wangzelin007 and jiasli Apr 3, 2023
@ghost ghost added the Installation label Apr 3, 2023
#!/usr/bin/env bash

# This script should be run in a centos7 docker.
# This script should be run in a Mariner 2.0 docker container.
Copy link
Contributor

Choose a reason for hiding this comment

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

How about change centos7 to ubi or fedora in test_rpm_in_docker.sh

Copy link
Member Author

Choose a reason for hiding this comment

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

That‘s irrelevant to this PR. Let's keep this PR clean.

@jiasli
Copy link
Member Author

jiasli commented Apr 14, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@jiasli jiasli merged commit 10aa576 into Azure:dev Apr 21, 2023
@jiasli jiasli deleted the ca-certificates branch April 21, 2023 06:26
StrawnSC pushed a commit to StrawnSC/azure-cli that referenced this pull request Apr 23, 2023
kairu-ms pushed a commit to kairu-ms/azure-cli that referenced this pull request Apr 28, 2023
yash-nisar pushed a commit to yash-nisar/azure-cli that referenced this pull request May 16, 2023
avgale pushed a commit to avgale/azure-cli that referenced this pull request Aug 24, 2023
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.

4 participants