Fix: cgroup_skb pogs should respect enable_monitoring flag

[hzxuzhonghu]

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

[hzxuzhonghu]

Fix #1310

[yp969803]

kmesh/bpf/kmesh/probes/probe.h

Line 99 in ae81023

return;

Also you can Log the error, stating storage not found, i havnt done this because previously i was not checking is_monitoring_enable, hence can result to unecessary logs

[yp969803]

if (!is_monitoring_enable()) {
        return;

this check can be done at

kmesh/bpf/kmesh/probes/probe.h

Line 87 in ae81023

static inline void observe_on_data(struct bpf_sock *sk)

function, resulting in non repeating code

[yp969803]

Btw, thanku for taking this issue made my work easy :)

[hzxuzhonghu]

@yp969803 addressed all your comments

[codecov]

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 45.43%. Comparing base (b1ea0b3) to head (df56fd8).
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
pkg/bpf/bpf.go	16.66%	3 Missing and 2 partials ⚠️
pkg/controller/controller.go	0.00%	1 Missing ⚠️

Files with missing lines	Coverage Δ
pkg/controller/controller.go	0.00% <0.00%> (ø)
pkg/bpf/bpf.go	41.44% <16.66%> (-0.34%)	⬇️

---

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 f216fd3...df56fd8. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[yp969803]

/lgtm

[kmesh-bot]

@yp969803: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[hzxuzhonghu]

@yp969803 You can apply for mamebership via https://github.com/kmesh-net/kmesh/issues/new?assignees=&labels=&projects=&template=membership-request.md&title=REQUEST%3A+New+membership+for+%3Cyour+name%3E

[hzxuzhonghu]

/assign @LiZhenCheng9527

[LiZhenCheng9527]

/lgtm
/approve

[kmesh-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LiZhenCheng9527

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [LiZhenCheng9527]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment