Skip to content
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

Documentation for telent in oai #149

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jain-ashish-sam
Copy link
Contributor

@jain-ashish-sam jain-ashish-sam commented Jul 1, 2024

The documentation include:

  1. Getting stats from DU over telnet
  2. Reconfiguring Bandwidth from 20 to 40 Mhz
  3. Deploying ue's (20 & 40 Mhz), and run an E2E ping test
    Note: For Bandwidth-Reconfiguration, we are not using Nephio capabilities. We are planning it to add in the future

@nephio-prow nephio-prow bot requested review from efiacor and tliron July 1, 2024 03:04
Copy link

netlify bot commented Jul 1, 2024

Deploy Preview for nephio ready!

Name Link
🔨 Latest commit 5752886
🔍 Latest deploy log https://app.netlify.com/sites/nephio/deploys/6684224c461d4e0008d6578b
😎 Deploy Preview https://deploy-preview-149--nephio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@arora-sagar arora-sagar requested review from liamfallon and arora-sagar and removed request for tliron and efiacor July 1, 2024 12:07
UPF_POD=$(kubectl get pods -n oai-core --context=edge-admin@edge -l workload.nephio.org/oai=upf -o jsonpath='{.items[*].metadata.name}')
UPF_tun0_IP_ADDR=$(kubectl exec -it $UPF_POD -n oai-core -c upf-edge --context edge-admin@edge -- ip -f inet addr show tun0 | sed -En -e 's/.*inet ([0-9.]+).*/\1/p')
kubectl exec -it $UE_POD -n oai-ue --context edge-admin@edge -- ping -I oaitun_ue1 -c 3 $UPF_tun0_IP_ADDR
kubectl exec -it $UE_POD -n oai-ue --context edge-admin@edge -- ping -c 3 $UPF_tun0_IP_ADDR
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jain-ashish-sam I would say leave -I oaitun_ue1 to force ping to use that interface. I know to ping UPF it is not needed but for internet it will be needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I have updated it!

@arora-sagar
Copy link
Contributor

/approve

Copy link
Contributor

nephio-prow bot commented Jul 1, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: arora-sagar

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:

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

@nephio-prow nephio-prow bot added the approved label Jul 1, 2024
@arora-sagar
Copy link
Contributor

@jain-ashish-sam You also need to update kubectl create -f test-infra/e2e/tests/oai/004-ran-network.yaml this part either you update the script or use EOF to add the files there. The current 004-ran-network.yaml is not deploying o1 enabled DU.

@arora-sagar
Copy link
Contributor

@jain-ashish-sam Can you update the title of the PR and remove O1 from there and just keep telnet?

Another thing can you explicitly mention in the PR description for the NF reconfiguration we are not using Nephio capabilities.

@@ -711,25 +711,106 @@ The output is similar to:
9496.571150 [GNB_APP] I [gNB 0] Received NGAP_REGISTER_GNB_CNF: associated AMF 1
```

### Check Stats via telnet O1 Module
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the term O1?

```

```bash
O1_IP=$(kubectl get svc oai-gnb-du-o1-telnet-lb -n oai-ran-du --context edge-admin@edge -o=jsonpath='{.status.loadBalancer.ingress[0].ip}')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There shouldn't be any reference O1 - this is simply a telnet interface to the DU; not O1. The file is peppered with this misinformation. Minimally we should note that this is not a standard O1 interface but a proprietary telnet interface.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have removed the O1 reference

@nephio-prow nephio-prow bot added the approved label Jul 1, 2024
@jain-ashish-sam jain-ashish-sam changed the title Documentation for o1_telent in oai Documentation for telent in oai Jul 2, 2024
@jain-ashish-sam
Copy link
Contributor Author

As far as " The current 004-ran-network.yaml is not deploying o1 enabled DU." is concerned, I will update the packageRevision in the "test-infra" once catalog repo is tagged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants