Skip to content

Conversation

@haofan-ms
Copy link
Contributor

Reason for Change:

Issue Fixed:

Credit Where Due:

Does this change contain code from or inspired by another project?

  • No
  • Yes

If "Yes," did you notify that project's maintainers and provide attribution?

  • No
  • Yes

Requirements:

Notes:

@haofan-ms haofan-ms requested review from christinalau0 and jadarsie and removed request for jadarsie March 18, 2025 17:42
@haofan-ms haofan-ms force-pushed the users/hafa/update-kubelet-flag branch from b32424b to e4191ed Compare March 19, 2025 02:15
Copy link
Member

@jadarsie jadarsie left a comment

Choose a reason for hiding this comment

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

I think we have to modify validate.go instead of upgrade.go.

@haofan-ms
Copy link
Contributor Author

I think we have to modify validate.go instead of upgrade.go.

I updated both places, but from the test result only the messages from upgrade.go shows up in the console. Also our earlier api model overwrites all happen in upgrade.go, so I was thinking it make more sense to change there.

@haofan-ms haofan-ms merged commit f947ec1 into master Mar 19, 2025
2 checks passed
@haofan-ms haofan-ms deleted the users/hafa/update-kubelet-flag branch March 19, 2025 19:53
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.

3 participants