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
Update Ixia for client api #249
Update Ixia for client api #249
Conversation
Pull Request Test Coverage Report for Build 3340298137
💛 - Coveralls |
topo/node/ixia/ixia.go
Outdated
_, err = c.IxiaTG(n.Namespace).Create(ctx, crd) | ||
if err != nil { | ||
return nil, err | ||
} | ||
|
||
if err != nil { |
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.
this is now redundant
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.
Removed those checks.
Do(ctx). | ||
Error() | ||
log.Infof("Updating ixia CRD (desiredState=%s) ...", desiredState) | ||
_, err = c.IxiaTG(n.Namespace).Update(ctx, unStrCRD, metav1.UpdateOptions{}) |
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.
can you add a comment here on why update is being used instead of create?
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.
Added comment for TopologySpecs and Create operations.
/gcbrun |
We had a breakage in the presubmit (that was fixed but is not included on this branch). Can you cherrypick that change or resync and merge it in? 6a700bf sorry about that |
I have resynced and merged. |
/gcbrun |
Looks like there are some merge conflicts, once those are fixed I can merge |
Resolved the go.sum conflict. |
there are still merge conflicts unfortunately |
okay I merged the other PR, ready to run the presubmit on this as well after the merge conflicts are resolved |
/gcbrun |
* Update Ixia for client api * Incorporated review comments * Cleaned go.sum
Make use of go client API for IxiaTG CRD instead of REST API.