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

To suggest new description #231

Closed
tsaad-dev opened this issue Jun 16, 2023 · 6 comments
Closed

To suggest new description #231

tsaad-dev opened this issue Jun 16, 2023 · 6 comments
Assignees

Comments

@tsaad-dev
Copy link
Owner

48ade70#r118388324

@italobusi
Copy link
Collaborator

Proposal:

              description
                "A reference to the reverse secondary path when 
                co-routed with the secondary path.";

@tsaad-dev tsaad-dev assigned tsaad-dev and unassigned italobusi Jun 23, 2023
@italobusi
Copy link
Collaborator

2023-06-23

Agreed

tsaad-dev pushed a commit that referenced this issue Jun 23, 2023
@italobusi italobusi reopened this Jun 27, 2023
@italobusi
Copy link
Collaborator

We have just discovered that the change has been applied on the wrong line of the code

  1. This description applies to the list of candidate secondary paths and should be reverted back to the previous description

    OLD

    te/ietf-te.yang

    Lines 1028 to 1030 in ce52203

    description
    "A reference to the reverse secondary path when
    co-routed with the secondary path.";

    NEW

                        description
                          "A reference to the secondary reverse path that
                           may be utilised when the containing primary
                           reverse path is in use.";
    
    
  2. Instead this is the description that has to be updated (within the container secondary-paths):

    OLD

    te/ietf-te.yang

    Lines 1107 to 1109 in ce52203

    description
    "A reference to the secondary reverse path that
    may be utilised when the secondary path is in use.";

    NEW

               description
                 "A reference to the reverse secondary path when 
                 co-routed with the secondary path.";
    

Co-authored-by: sergio belotti sergio.belotti@nokia.com

@italobusi
Copy link
Collaborator

italobusi commented Nov 13, 2023

@tsaad-dev
Copy link
Owner Author

OK.

tsaad-dev pushed a commit that referenced this issue Dec 15, 2023
@italobusi
Copy link
Collaborator

Fixed in draft-ietf-teas-yang-te-35

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

No branches or pull requests

2 participants