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.

Sign up for GitHub

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

Jump to bottom

Muxponders' configuration constraints #120

Closed
italobusi opened this issue Jan 12, 2023 · 3 comments · Fixed by #154
Closed

Muxponders' configuration constraints #120

italobusi opened this issue Jan 12, 2023 · 3 comments · Fixed by #154
Labels
agreed Resolution has been agreed: need for a PR to be merged before closing the issue IETF-118 update for IETF-118 YANG model update

Comments

@italobusi
Copy link
Member

See ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang#56
@italobusi italobusi added enhancement New feature or request question Further information is requested and removed enhancement New feature or request labels Jan 12, 2023
@sergiobelotti
Copy link
Collaborator

sergiobelotti commented Feb 14, 2023 •
edited

This issue will be discussed in the context of flexible grid meeting. We will take it open till there are no feedback form the discussion in the flexible grid context.

@sergiobelotti
Copy link
Collaborator

Call on September 12:
There is already a proposal in the context of flexible grid. We need to discuss in the flexible grid call on Thursday if it would be better to add the modification here in Optical Impairment to make the updated feasible for both WSON and Flexgrid NW.

@danielkinguk danielkinguk mentioned this issue Sep 14, 2023
Closed
@sergiobelotti
Copy link
Collaborator

Call on September 09-26-23:

Aihua presented some slides describing a proposal to solve the issue. https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/files/11530989/Muxponder.Constraints.v3.pptx

No comments during the presentation apart request for clarifications but the proposal seems a good way to solve the issue. No concerns were reaised.
The agreement is to review the slides for next week and raise question if something appears not clear.

italobusi added a commit that referenced this issue Oct 4, 2023
@italobusi @sergiobelotti
Added muxponder constraints: fix #120
7139578 
Co-authored-by: sergio belotti <sergio.belotti@nokia.com>
@italobusi italobusi mentioned this issue Oct 4, 2023
Merged
@sergiobelotti sergiobelotti added agreed Resolution has been agreed: need for a PR to be merged before closing the issue YANG model update IETF-118 update for IETF-118 and removed question Further information is requested labels Oct 9, 2023
italobusi added a commit that referenced this issue Oct 23, 2023
@italobusi @sergiobelotti
YANG updates for IETF 118 (#154)
ed0ef67 
Fixed relative paths in grouping media-channel-groups: see comment from Tom Petch in #144

Added muxponder constraints: fix #120

Added DGE attributes to amplifier element: see #153 (comment)

Changed the type of delta-power

Aligned with ietf-ccamp-wg/ietf-ccamp-layer0-types-ext-RFC9093-bis#78

---------

Co-authored-by: sergio belotti <sergio.belotti@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
agreed Resolution has been agreed: need for a PR to be merged before closing the issue IETF-118 update for IETF-118 YANG model update
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

YANG updates for IETF 118 ietf-ccamp-wg/draft-ietf-ccamp-optical-impairment-topology-yang
2 participants
@italobusi @sergiobelotti