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
Renamed image "white lists" to pre-pull image lists in test #92711
Renamed image "white lists" to pre-pull image lists in test #92711
Conversation
Signed-off-by: alejandrox1 <alarcj137@gmail.com>
Continuation of #92318 (because I messed up the commit history in the other PR 😅 ) |
/sig testing |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alejandrox1 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@@ -61,16 +61,16 @@ var NodePrePullImageList = sets.NewString( | |||
"gcr.io/kubernetes-e2e-test-images/node-perf/tf-wide-deep-amd64:1.0", | |||
) | |||
|
|||
// updateImageWhiteList updates the framework.ImageWhiteList with | |||
// updateImageAllowList updates the framework.ImagePrePullList with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why the names mismatch? should it be updateImagePrePullList
?
/retest Review the full test history for this PR. Silence the bot with an |
1 similar comment
/retest Review the full test history for this PR. Silence the bot with an |
Signed-off-by: alejandrox1 alarcj137@gmail.com
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
In the spirit of #90277
xref #92317
I wanted to propose this change to the e2e node code to get the conversation started and try to improve the naming conventions we use throughout the codebase.
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: