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

fix-for(E2E controller ignition over 1-hop RF sync link) #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

banswamy
Copy link
Contributor

@banswamy banswamy commented Dec 29, 2022

Signed-off-by: bmswamy [email protected]

Prerequisites

  • I have read the Contributing Guidelines.
  • I have read the Code of Conduct.
  • If this is a non-trivial change, I have already opened an accompanying Issue.
  • If applicable, I have included documentation updates alongside my code changes.

Description

E2E controller ignition over 1-hop RF sync link

Test Plan

  • Tested below command in e2e controller for link up
  • tg link up -i PUMA_2_DN_2 -r PUMA_3_CN_3
  • Tested below command for topology list
  • tg topology ls

bash-5.0# tg topology ls
NodeName MacAddr PopNode NodeType Status SiteName


PUMA_1_DN_1 34:ef:b6:8a:0e:0e True DN ONLINE_INITIATOR PUMA_1_DN
PUMA_2_DN_2 34:ef:b6:45:fc:18 False DN ONLINE PUMA_2_DN
PUMA_3_CN_3 34:ef:b6:8a:10:a8 False CN ONLINE PUMA_3_CN

LinkName ANodeName ZNodeName Alive LinkType LinkupAttempts


link-PUMA_1_DN_1-PUMA_2_DN_2 PUMA_1_DN_1 PUMA_2_DN_2 True WIRELESS 1
link-PUMA_2_DN_2-PUMA_3_CN_3 PUMA_2_DN_2 PUMA_3_CN_3 True WIRELESS 1

SiteName Latitude Longitude Altitude Accuracy


PUMA_1_DN 37.4862 -122.146 -3.4 24.13
PUMA_2_DN 37.4862 -122.146 -3.4 24.13
PUMA_3_CN 37.4862 -122.146 -3.4 24.13

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 29, 2022
Copy link
Contributor

@elludraon elludraon left a comment

Choose a reason for hiding this comment

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

I completely don't understand the reason for the POP_ADDR related changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants