dragonflyoss / Dragonfly2

Dragonfly is an open source P2P-based file distribution and image acceleration system. It is hosted by the Cloud Native Computing Foundation (CNCF) as an Incubating Level Project.
https://d7y.io
Apache License 2.0
2.26k stars 287 forks source link

No traces when pulling images from ACK containerd #3052

Open sigboom opened 9 months ago

sigboom commented 9 months ago

Bug report:

k8s version is 1.26, containerd version is 1.6.20, dragonfly helm package verison is 1.1.29

  1. The Dragonfly has been deployed on Alibaba Cloud ACK and two registries have been set up
    containerd:
    enable: true
    injectConfigPath: true
    registries:
      - "https://ghcr.io"
      - "https://lthub-registry.cn-beijing.cr.aliyuncs.com"
  2. All services have started logging normally, referencing dfdaemon.
    dfdaemon:
    enable: true
    config:
    console: true
    verbose: true
    pprofPort: 18066
    jaeger: ""
    jaeger:
    enable: true
  3. The storage is configured with 20Gi to comply with the minimum quota requirement of Alibaba Cloud's cloud disk.
  4. Confirm that all local services have been successfully started. image
  5. Look at backend services, Confirm All Are Accessible Normally image image
  6. Based on official use cases for testing, corresponding links https://d7y.io/docs/getting-started/quick-start/kubernetes/#containerd-pull-image-back-to-source-for-the-first-time-through-dragonfly image
  7. Check that the configuration for containerd on the instance is generated correctly image
  8. The correct request records for the official website use case were not found on Jaeger. image

Expected behavior:

image

How to reproduce it:

Using Helm to Deploy on ACK Cluster

Environment:

sigboom commented 9 months ago

one dfdaemon console log file dfdaemon.log What logs do I need to provide?

gaius-qi commented 9 months ago

Check whether the config_path in the containerd configuration is correct, refer to https://github.com/containerd/containerd/blob/main/docs/cri/config.md#registry-configuration.

If correct, please check the containerd log. Why is mirror not used?

sigboom commented 9 months ago

Check whether the config_path in the containerd configuration is correct, refer to https://github.com/containerd/containerd/blob/main/docs/cri/config.md#registry-configuration.

If correct, please check the containerd log. Why is mirror not used?

I found it!This is default config in ACK /etc/containerd/config.toml. image The default configuration of ACK is missing an "s"

sigboom commented 9 months ago

What happens next is up to the community to handle the situation. dragonfly/templates/dfdaemon/dfdaemon-daemonset.yaml It is recommended to read config_path from the current node and not hardcode it in the deployment script.

gaius-qi commented 9 months ago

What happens next is up to the community to handle the situation. dragonfly/templates/dfdaemon/dfdaemon-daemonset.yaml It is recommended to read config_path from the current node and not hardcode it in the deployment script.

@sigboom Agree!