-
Notifications
You must be signed in to change notification settings - Fork 275
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
Add support for bare hostname as endpoint, fix unnecessary namespace param inclusion #5516
Comments
This was referenced Feb 23, 2024
Merged
Validated on master with 08699df / 1.29Environment DetailsInfrastructure
Node(s) CPU architecture, OS, and Version:
Cluster Configuration:
Config.yaml:
Additional files
Testing StepsNote:
Air-Gap Setup
Pull-Through Cache Configuration
Copying Files to Airgapped Instance [Do this for each pair of airgapped instances]
Certificates and RKE2 Setup
Replication Results:
Registries
Observations:
Validation Results:
Additional context / logs:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
An issue to track the work being done in rancher/wharfie#24
Internal Jira Ref: SURE-7640
Wharfie does not support a bare hostname as the mirror endpoint in registries.yaml. If the endpoint is a bare hostname, or lacks a URI scheme (http://, https://), the node will not use the endpoint. If the endpoint is necessary to pull the rke2-runtime image, rke2 will fail to start.
Wharfie unnecessarily adds the namespace to the end of the request URI, if it is added as an endpoint. For example, with
system-default-registry: registry.example.com
, and the following content in registries.yaml, the image will fail to pull if the registry is namespace-aware (for example, sonatype nexus).When rke2 is running with
debug: true
, it can be seen that the registry hostname is appended to the request as?ns=registry.example.com
. The namespace query parameter should only be set if the endpoint hostname is not the same as the registry hostname.The text was updated successfully, but these errors were encountered: