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

e2e: test topology-aware policy nodeResourceTopology exporting #465

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

e2e: test topology-aware policy nodeResourceTopology exporting #465

wants to merge 2 commits into from

Conversation

askervin
Copy link
Collaborator

No description provided.

@askervin askervin force-pushed the 5Z8_e2e_topology_aware_nrt branch from 7c61c18 to f719da3 Compare January 21, 2025 09:29
klihub
klihub reviewed Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@klihub klihub left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I only have a nano-nit (borderline besserwissering), which I'm fine with also ignoring. LGTM otherwise.

@askervin askervin force-pushed the 5Z8_e2e_topology_aware_nrt branch from f719da3 to 42ef846 Compare January 21, 2025 14:40
@klihub klihub requested a review from fmuyassarov January 22, 2025 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klihub klihub klihub approved these changes

@fmuyassarov fmuyassarov Awaiting requested review from fmuyassarov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@askervin @klihub