-
Notifications
You must be signed in to change notification settings - Fork 413
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
Change type of FailureReason
for AzureMachinePoolStatus
from MachineStatusError
to MachinePoolStatusFailure
#3825
Comments
@Jont828 would this be a good first issue? It seems like it could be one but I'm not sure if it's more complicated than it seems. |
I think so, it would just involve changing the API type and where it gets called in /good-first-issue |
@Jont828: GuidelinesPlease ensure that the issue body includes answers to the following questions:
For more details on the requirements of such an issue, please see here and ensure that they are met. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @Tea4menono |
@nawazkh: GitHub didn't allow me to assign the following users: Tea4menono. Note that only kubernetes-sigs members with read permissions, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
hi, it seems to be a straight change as @Jont828 mentioned. i was wondering since the field is on the API, there might be some client replying on the failure reason type (🤞not)? could you please assign this one to me? |
/assign |
What do you mean by client replying on the failure reason type? |
/kind bug
[Before submitting an issue, have you checked the Troubleshooting Guide?]
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
In the status field for AzureMachinePool, the failure reason is of type
FailureReason *errors.MachineStatusError
. However, there is another type defined callederrors.MachinePoolStatusFailure
that would be a better fit.What did you expect to happen:
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:
kubectl version
):/etc/os-release
):The text was updated successfully, but these errors were encountered: