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

Fixing Wolverine LC's incorrect symlinks #21779

Merged
merged 2 commits into from
Feb 20, 2025

Conversation

arista-nwolfe
Copy link
Contributor

In #21512 we created separate directories for the different Wolverine SKUs but the pmon_daemon_control.json symlink was moved to the non-LC file in x86_64-arista_common

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • 202405
  • 202411

In sonic-net#21512 we created
separate directories for the different Wolverine SKUs but the
pmon_daemon_control.json symlink was moved to the non-LC file in
x86_64-arista_common
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

arlakshm
arlakshm previously approved these changes Feb 19, 2025
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arista-nwolfe arista-nwolfe changed the title Fixing LC's pmon_daemon_control.json symlink Fixing Wolverine LC's incorrect symlinks Feb 19, 2025
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202405: Azure/sonic-buildimage-msft#663

ram25794 pushed a commit to ram25794/sonic-buildimage that referenced this pull request Feb 21, 2025
* Fixing LC's pmon_daemon_control.json symlink

In sonic-net#21512 we created
separate directories for the different Wolverine SKUs but the
pmon_daemon_control.json symlink was moved to the non-LC file in
x86_64-arista_common

* Fixing a few more LC symlinks that were pointing to the wrong files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants