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

operator: Add no hugepages test #2505

Conversation

sebrandon1
Copy link
Member

Adds an operator suite test for making sure they are not using hugepages in their pods that utilizes the existing pod.HasHugepages() function.

Note, I have left the test case as fully Optional for now until we discuss Mandatory or Optional as a group.

@dcibot
Copy link
Collaborator

dcibot commented Oct 8, 2024

tests/operator/suite.go Outdated Show resolved Hide resolved
@dcibot
Copy link
Collaborator

dcibot commented Oct 9, 2024

@dcibot
Copy link
Collaborator

dcibot commented Oct 10, 2024

tests/identifiers/identifiers.go Outdated Show resolved Hide resolved
@dcibot
Copy link
Collaborator

dcibot commented Oct 11, 2024

@sebrandon1 sebrandon1 force-pushed the operator_pods_no_hugepages branch 2 times, most recently from 0e45077 to f445b16 Compare October 15, 2024 16:43
@dcibot
Copy link
Collaborator

dcibot commented Oct 16, 2024

from change #2505:

  • ERROR no DCI job found

@greyerof
Copy link
Contributor

/dci-rerun

@dcibot
Copy link
Collaborator

dcibot commented Oct 16, 2024

@sebrandon1 sebrandon1 merged commit 8b7da6e into redhat-best-practices-for-k8s:main Oct 16, 2024
34 checks passed
@sebrandon1 sebrandon1 deleted the operator_pods_no_hugepages branch October 16, 2024 15:43
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.

4 participants