Skip to content

fix(neutron): undersync mechanism operate only on baremetal ports PUC-1713#2058

Merged
nidzrai merged 1 commit into
mainfrom
undersync_mech_baremetal_only
Jun 2, 2026
Merged

fix(neutron): undersync mechanism operate only on baremetal ports PUC-1713#2058
nidzrai merged 1 commit into
mainfrom
undersync_mech_baremetal_only

Conversation

@nidzrai
Copy link
Copy Markdown
Contributor

@nidzrai nidzrai commented Jun 2, 2026

No description provided.

@nidzrai nidzrai force-pushed the undersync_mech_baremetal_only branch 2 times, most recently from 667b49c to c9aa243 Compare June 2, 2026 09:46
@nidzrai nidzrai changed the title fix(neutron): undersync mechanism would operate only on baremetal ports PUC-1713 fix(neutron): undersync mechanism operate only on baremetal ports PUC-1713 Jun 2, 2026
Copy link
Copy Markdown
Contributor

@stevekeay stevekeay left a comment

Choose a reason for hiding this comment

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

LGTM although I would vastly prefer if the commit message were amended to add a paragraph describing why this change was made.

Copy link
Copy Markdown
Collaborator

@skrobul skrobul left a comment

Choose a reason for hiding this comment

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

can you provide bit more context for this PR ?

What's the rationale behind excluding 'normal' ports? I may be wrong here, but I think Undersync still requires visibility of those port types which are commonly applied on trunk subports and router interfaces.

Copy link
Copy Markdown
Contributor

@stevekeay stevekeay left a comment

Choose a reason for hiding this comment

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

There appears to be a merge commit in this PR?

@nidzrai nidzrai force-pushed the undersync_mech_baremetal_only branch from 1dd6303 to 2d67d15 Compare June 2, 2026 11:45
@nidzrai nidzrai force-pushed the undersync_mech_baremetal_only branch from 2d67d15 to 61e5335 Compare June 2, 2026 11:47
@nidzrai nidzrai requested review from skrobul and stevekeay June 2, 2026 11:49
@nidzrai nidzrai added this pull request to the merge queue Jun 2, 2026
Merged via the queue into main with commit e72e5ea Jun 2, 2026
61 checks passed
@nidzrai nidzrai deleted the undersync_mech_baremetal_only branch June 2, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants