Skip to content

Update faqs-app-service-linux.yml #1860

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

btardif
Copy link

@btardif btardif commented Apr 15, 2025

We don't support private registries over HTTP

We don't support private registries over HTTP
Copy link

@btardif : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit e8e8443:

✅ Validation status: passed

File Status Preview URL Details
support/azure/app-service/faqs-app-service-linux.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

PRMerger Results

Issue Description
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.

@btardif
Copy link
Author

btardif commented Apr 15, 2025

@genlin

For LWASv1 / docker pull to support this, the registry needs to be added as an "insecure registry" on docker startup (dockerd | Docker Docs), which is something we only do for Internal Registry Roles where enabled.

For LWASv2 image pull to support this, a specific flag needs to be present (something akin to --plain-http=true), and this is again something that is only present when we pull platform images from the Internal Registry Roles.

I don't believe HTTP pull for private registries has ever been a supported scenario.

@stevenernst131
Copy link

@btardif hold up this might not even be correct to begin with, let me follow up with you offline.

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.

3 participants