Fix cgroup_skb/* get sk_storage failed

[hzxuzhonghu]

What type of PR is this?
/kind bug

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #1348

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

[hzxuzhonghu]

FYI, kmesh simulate 0.0.0.2:930 or 0.0.0.2:931 when it need to manage the pod

[hzxuzhonghu]

cc @yp969803

@Kuromesi

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.26%. Comparing base (bd40bbe) to head (44253ce).
Report is 23 commits behind head on main.

see 2 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 8a07412...44253ce. Read the comment docs.

🚀 New features to boost your workflow:

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

[yp969803]

have u checked it locally, using bpf logs, it is better if we check the stuffs locally before merging to prevent bugs.
you can use this manifest to simulate long connection, deploy in default ns

apiVersion: v1
kind: Pod
metadata:
  name: ws-server
  labels:
    app: ws-server
spec:
  containers:
  - name: ws-server
    image: node:18
    command: ["/bin/sh", "-c"]
    args:
      - |
        mkdir server 
        cd server 
        npm init -y
        npm install ws
        cat << EOF > server.js
        const WebSocket = require('ws');
        const server = new WebSocket.Server({ port: 8080 });
        server.on('listening', () => {
          console.log('🚀 WebSocket server started on ws://localhost:8080');
        });

        server.on('connection', socket => {
          console.log('Client connected');
          
          // Send data every 1 seconds
          setInterval(() => {
            socket.send(`Hello world`);
            console.log('Sent message to client');
          }, 1000);

          socket.on('message', message => {
            console.log(`Received`);
          });

          socket.on('close', () => {
            console.log('Client disconnected');
          });
        });
        EOF
        node server.js

    ports:
    - containerPort: 8080

---
apiVersion: v1
kind: Service
metadata:
  name: ws-server-service
  labels:
    app: ws-server
spec:
  selector:
    app: ws-server
  ports:
  - protocol: TCP
    port: 8080        # Port exposed by the service
    targetPort: 8080  # Port the server container listens on

--- 

apiVersion: v1
kind: Pod
metadata:
  name: ws-client
  labels:
    app: ws-client
spec:
  containers:
  - name: ws-client
    image: node:18
    command: ["/bin/sh", "-c"]
    args:
      - |
        mkdir client
        cd client
        npm init -y
        npm install ws 
        cat <<EOF > client.js
        const WebSocket = require('ws');
        const socket = new WebSocket('ws://ws-server-service.default.svc.cluster.local:8080');

        socket.on('open', () => {
          console.log('Connected to server');
          socket.send('Hello from client');
          console.log('Sent message to server');
        });

        socket.on('message', data => {
          console.log(`Received`);

        });

        socket.on('close', () => {
          console.log('Connection closed');
        });
        EOF
        node client.js

[hzxuzhonghu]

what do you mean?

/retest

[yp969803]

i was just asking if u have checked it locally, does the normal packets reaching to the observed_data function

[hzxuzhonghu]

I am not sure if kmesh restart would influence the long connection metric. For this patch, it does not bring regression for long connection

[nlgwcy]

Suggested change

	if (bpf_ntohs(dst_port) != enable_port && bpf_ntohs(dst_port) != disable_port)
	if (bpf_ntohs(dst_port) != ENABLE_KMESH_PORT && bpf_ntohs(dst_port) != DISABLE_KMESH_PORT)

[hzxuzhonghu]

/retest

[hzxuzhonghu]

@nlgwcy Another look

[nlgwcy]

/lgtm
/approve

[kmesh-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nlgwcy

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 [nlgwcy]

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