Fix the problem that the file ‘oncn-mda.conf’ has no permissions.

kmesh-net / kmesh

High Performance ServiceMesh Data Plane Based on Programmable Kernel

https://kmesh.net

Apache License 2.0

464 stars 70 forks source link

Fix the problem that the file ‘oncn-mda.conf’ has no permissions. #750

Open jiayoukun opened 3 months ago

jiayoukun commented 3 months ago

fix err : time="2024-08-19T03:28:28Z" level=error msg="the config you input not unique to root, please modify permission of the config enable serviceMesh accelerating failed! " subsys=pkg/bpf

kmesh-bot commented 3 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign hzxuzhonghu for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/kmesh-net/kmesh/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.28%. Comparing base (f10e507) to head (975ceec). Report is 5 commits behind head on main.

see 3 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 33ed729...975ceec. Read the comment docs.

bitcoffeeiux commented 2 months ago

This issue can be discussed in the community this week. In the container, there seems to be no reason to limit the permission to 600. The value is set to 600 because kmesh does not need to run in the container before.