Skip to content

Disabling macsec support on arista_7280r4(k)_32qf_32df gearbox phys#24761

Merged
rlhui merged 2 commits intosonic-net:masterfrom
arista-nwolfe:master-citrine-gb-macsec-disable
Mar 8, 2026
Merged

Disabling macsec support on arista_7280r4(k)_32qf_32df gearbox phys#24761
rlhui merged 2 commits intosonic-net:masterfrom
arista-nwolfe:master-citrine-gb-macsec-disable

Conversation

@arista-nwolfe
Copy link
Contributor

This change depends on sonic-net/sonic-swss#3926 merging first.
sonic-net/sonic-swss#3926 introduces the ability to indicate whether a gearbox phy supports macsec or not.
This allows the macsecorch to send the macsec SAI calls to the switching asic (syncd) instead of the gearbox phy (gbsyncd).

arista_7280r4(k)_32qf_32df is a platform that has this same behavior (GB doesn't support macsec).

The parent PR for this SKU is #24206

Which release branch to backport (provide reason below if selected)

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph
Copy link
Contributor

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe arista-nwolfe force-pushed the master-citrine-gb-macsec-disable branch from 142b352 to 9351fdd Compare January 13, 2026 00:50
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

This change is dependent on
sonic-net/sonic-swss#3926
merging first.

Signed-off-by: Nathan Wolfe <nwolfe@arista.com>
@arista-nwolfe arista-nwolfe force-pushed the master-citrine-gb-macsec-disable branch from 9351fdd to e3ba64b Compare January 23, 2026 00:49
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw ms_conflict

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui rlhui merged commit 09c3f2d into sonic-net:master Mar 8, 2026
27 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202511: #25960

jackson-micas pushed a commit to jackson-micas/sonic-buildimage that referenced this pull request Mar 10, 2026
sonic-net/sonic-swss#3926 introduces the ability to indicate whether a gearbox phy supports macsec or not.
This allows the macsecorch to send the macsec SAI calls to the switching asic (syncd) instead of the gearbox phy (gbsyncd).

arista_7280r4(k)_32qf_32df is a platform that has this same behavior (GB doesn't support macsec).

The parent PR for this SKU is sonic-net#24206

Signed-off-by: Nathan Wolfe <nwolfe@arista.com>
Co-authored-by: Arvindsrinivasan Lakshmi Narasimhan <55814491+arlakshm@users.noreply.github.com>
Signed-off-by: jackson <jackson@micasnetworks.com>
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.

6 participants