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

Krnlmon maintenance #171

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Krnlmon maintenance #171

merged 2 commits into from
Nov 4, 2024

Conversation

ffoulkes
Copy link
Contributor

@ffoulkes ffoulkes commented Nov 4, 2024

  • Modified cmake check scripts to specify -L krnlmon and --output-on-failure when running ctest.

  • Fixed a compile error in switchlink_link.c caused by the IFLA_LINKINFO refactoring. The create_lag_member must always be defined now, even if LAG_OPTION is undefined, because it's a parameter to process_linkinfo_attr().

  • Updated .gitignore file.

- Modified cmake check scripts to specify -L krnlmon and
  --output-on-failure when running ctest.

- Fixed a compile error in switchlink_link.c caused by the
  IFLA_LINKINFO refactoring. The create_lag_member must always
  be defined now, even if LAG_OPTION is undefined, because it's
  a parameter to process_linkinfo_attr().

- Updated .gitignore file.

Signed-off-by: Derek Foster <[email protected]>
Copy link
Collaborator

@5abeel 5abeel left a comment

Choose a reason for hiding this comment

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

LGTM

@ffoulkes ffoulkes merged commit e8d6ba8 into main Nov 4, 2024
4 checks passed
@ffoulkes ffoulkes deleted the krnlmon-maintenance branch November 4, 2024 19:35
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.

2 participants