-
Notifications
You must be signed in to change notification settings - Fork 244
fix: fixing repodepot support for azlinux3 and ubuntu2404 as well as PMC on Ubuntu #7565
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
base: main
Are you sure you want to change the base?
Conversation
| cloud-init status --wait | ||
|
|
||
| function init_mariner_repo_depot { | ||
| local repodepot_endpoint=$1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this copied from somewhere? Can you please add the source?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which part are you asking if it's copied ?
| IS_MARINER=1 | ||
| elif [[ $NAME == *"Microsoft Azure Linux"* ]]; then | ||
| IS_AZURELINUX=1 | ||
| else |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i see that below you have mentioned
no customization
Can we keep the behaviour consistent, like log a error message/warning that we cannot determine the distribution and customization may not be applied. i feel exit 1 is a bit extreme
… and all AzLinux version.
What type of PR is this?
/kind bug
What this PR does / why we need it:
fixing repodepot support for azlinux3 and ubuntu2404 as well as PMC on Ubuntu
Requirements:
Special notes for your reviewer:
Release note: