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
Bug 1868748: baremetal: rename JSON field for ClusterProvisioningIP #4053
Bug 1868748: baremetal: rename JSON field for ClusterProvisioningIP #4053
Conversation
/label platform/baremetal |
@stbenjam: This pull request references Bugzilla bug 1868748, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
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. |
@stbenjam: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
Also IMO this is not a change that we should be doing in code freeze-reduce bugs cycle. The attached bug is not a bug rather a feature request. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya 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 |
Sure, we can certainly wait for this to go in 4.7, it's just a minor annoyance. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Hey @stbenjam this needs a rebase, could you update then we can hopefully land it in time for 4.7? |
b98d29a
to
eaf6f66
Compare
/retest |
I've updated this PR /assign @hardys Could you take a look? Thanks! |
/bugzilla refresh |
@stbenjam: This pull request references Bugzilla bug 1868748, which is valid. 3 validation(s) were run on this bug
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. |
/lgtm |
The ClusterProvisioningIP field in the baremetal platform had an incorrect JSON annotation for serialization/deserialization, using "provisioningHostIP" instead of "clusterProvisioningIP." This name is incorrect and doesn't adequately reflect it's purpose. This field is the IP used in the cluster for provisioning. This commit deprecates the old JSON field, and uses the correct JSON annotation for ClusterProvisioningIP.
eaf6f66
to
038e8b9
Compare
#4429 introduced a conflict. Rebased. |
/lgtm |
/test e2e-aws |
/test e2e-aws |
/test e2e-aws |
@stbenjam: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
/test e2e-aws 🙏 |
@stbenjam: All pull requests linked via external trackers have merged: Bugzilla bug 1868748 has been moved to the MODIFIED state. 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. |
The ClusterProvisioningIP field in the baremetal platform had an
incorrect JSON annotation for serialization/deserialization, using
"provisioningHostIP" instead of "clusterProvisioningIP." This name is
incorrect and doesn't adequately reflect it's purpose. This field is
the IP used in the cluster for provisioning.
This commit deprecates the old JSON field, and uses the correct JSON
annotation for ClusterProvisioningIP.