Skip to content

Commit

Permalink
Fix mtu requirement bug
Browse files Browse the repository at this point in the history
There a bug with the MTU field in terraform where ExactlyOneOf
requires Jumbo or MTU to be set, but the intent was to allow
at most one of those fields to be set, but are optional.
  • Loading branch information
ismirlia authored and hkantare committed Jan 10, 2024
1 parent 8d929f3 commit dfb2339
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions ibm/service/power/resource_ibm_pi_network.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,19 +75,19 @@ func ResourceIBMPINetwork() *schema.Resource {
Description: "PI network gateway",
},
helpers.PINetworkJumbo: {
Type: schema.TypeBool,
Optional: true,
Computed: true,
Deprecated: "deprecated use pi_network_mtu instead",
ExactlyOneOf: []string{helpers.PINetworkMtu, helpers.PINetworkJumbo},
Description: "PI network enable MTU Jumbo option",
Type: schema.TypeBool,
Optional: true,
Computed: true,
Deprecated: "deprecated use pi_network_mtu instead",
ConflictsWith: []string{helpers.PINetworkMtu},
Description: "PI network enable MTU Jumbo option",
},
helpers.PINetworkMtu: {
Type: schema.TypeInt,
Optional: true,
Computed: true,
ExactlyOneOf: []string{helpers.PINetworkMtu, helpers.PINetworkJumbo},
Description: "PI Maximum Transmission Unit",
Type: schema.TypeInt,
Optional: true,
Computed: true,
ConflictsWith: []string{helpers.PINetworkJumbo},
Description: "PI Maximum Transmission Unit",
},
helpers.PINetworkAccessConfig: {
Type: schema.TypeString,
Expand Down

0 comments on commit dfb2339

Please sign in to comment.