Closed alculquicondor closed 5 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from alculquicondor. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
@alculquicondor Could you follow this DCO step? https://github.com/kubeflow/mpi-operator/pull/636/checks?check_run_id=23927150703
I missed make generate
.
Fixed.
I missed
make generate
.Fixed.
@alculquicondor I guess that you missed make crd
.
I missed
make generate
. Fixed.@alculquicondor I guess that you missed
make crd
.
NVM. There are any other reasons...
Yeah, I don't know what I missed
Yeah, I don't know what I missed
I found that your PR dropped some fields from swagger.json.
master
branch: https://github.com/kubeflow/mpi-operator/blob/1b33fec7e3596fb45e843d44544692e1423f7489/pkg/apis/kubeflow/v2beta1/swagger.json#L159-L161
this PR: https://github.com/alculquicondor/mpi-operator/blob/973702775818aac7452963f594b43da5e732089a/pkg/apis/kubeflow/v2beta1/swagger.json#L159-L161
@alculquicondor Did make generate
succeed in your local?
Something is hexed in my local repo
/close
@alculquicondor: Closed this PR.
To prepare for release.