Using pod network type as default when type is missing in the resource mapping #303
Using pod network type as default when type is missing in the resource mapping #303
Conversation
64e7229
to
3899e56
Compare
3899e56
to
355d82c
Compare
355d82c
to
b5466c3
Compare
…evirt#301 Signed-off-by: Jakub Dzon <jdzon@redhat.com>
b5466c3
to
36a92d6
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jakub-dzon, machacekondra, pkliczewski The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@@ -11,4 +11,6 @@ var ( | |||
PodType = "pod" | |||
// MultusType defines `pod` resource mapping network type | |||
MultusType = "multus" | |||
// UnsupportedType defines non-existing, unsupported `unsupported resource mapping network type |
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.
@jakub-dzon should the second "`unsupported" be removed from the description ? also there is a missing comma.
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.
The second "unsupported" is the the value used as unsupported type:) I will fix the missing punctuation mark.
Fixes #301.
Fixes:
Signed-off-by: Jakub Dzon jdzon@redhat.com