Change network payload handling (#444)

* change network payload handling

* apply pr suggestions

Author: GokceGK

docs/stackit_beta_network_create.md

@@ -20,7 +20,7 @@ stackit beta network create [flags]
   $ stackit beta network create --name network-1  --ipv4-dns-name-servers "1.1.1.1,8.8.8.8,9.9.9.9" --ipv4-prefix-length 25
 
   Create an IPv6 network with name "network-1" with DNS name servers and a prefix length
-  $ stackit beta network create --name network-1  --ipv6-dns-name-servers "1.1.1.1,8.8.8.8,9.9.9.9" --ipv6-prefix-length 25
+  $ stackit beta network create --name network-1  --ipv6-dns-name-servers "2001:4860:4860::8888,2001:4860:4860::8844" --ipv6-prefix-length 56
 ```
 
 ### Options

docs/stackit_beta_network_update.md

@@ -20,7 +20,7 @@ stackit beta network update [flags]
   $ stackit beta network update xxx --name network-1-new --ipv4-dns-name-servers "2.2.2.2"
 
   Update IPv6 network with ID "xxx" with new name "network-1-new" and new DNS name servers
-  $ stackit beta network update xxx --name network-1-new --ipv6-dns-name-servers "2.2.2.2"
+  $ stackit beta network update xxx --name network-1-new --ipv6-dns-name-servers "2001:4860:4860::8888"
 ```
 
 ### Options

internal/cmd/beta/network/create/create.go

@@ -55,7 +55,7 @@ func NewCmd(p *print.Printer) *cobra.Command {
 			),
 			examples.NewExample(
 				`Create an IPv6 network with name "network-1" with DNS name servers and a prefix length`,
-				`$ stackit beta network create --name network-1  --ipv6-dns-name-servers "1.1.1.1,8.8.8.8,9.9.9.9" --ipv6-prefix-length 25`,
+				`$ stackit beta network create --name network-1  --ipv6-dns-name-servers "2001:4860:4860::8888,2001:4860:4860::8844" --ipv6-prefix-length 56`,
 			),
 		),
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -151,19 +151,25 @@ func parseInput(p *print.Printer, cmd *cobra.Command) (*inputModel, error) {
 
 func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APIClient) iaas.ApiCreateNetworkRequest {
 	req := apiClient.CreateNetwork(ctx, model.ProjectId)
+	addressFamily := &iaas.CreateNetworkAddressFamily{}
+
+	if model.IPv6DnsNameServers != nil {
+		addressFamily.Ipv6 = &iaas.V1CreateNetworkIPv6{
+			Nameservers:  model.IPv6DnsNameServers,
+			PrefixLength: model.IPv6PrefixLength,
+		}
+	}
+
+	if model.IPv4DnsNameServers != nil {
+		addressFamily.Ipv4 = &iaas.CreateNetworkIPv4{
+			Nameservers:  model.IPv4DnsNameServers,
+			PrefixLength: model.IPv4PrefixLength,
+		}
+	}
 
 	payload := iaas.CreateNetworkPayload{
-		Name: model.Name,
-		AddressFamily: &iaas.CreateNetworkAddressFamily{
-			Ipv4: &iaas.CreateNetworkIPv4{
-				Nameservers:  model.IPv4DnsNameServers,
-				PrefixLength: model.IPv4PrefixLength,
-			},
-			Ipv6: &iaas.V1CreateNetworkIPv6{
-				Nameservers:  model.IPv6DnsNameServers,
-				PrefixLength: model.IPv6PrefixLength,
-			},
-		},
+		Name:          model.Name,
+		AddressFamily: addressFamily,
 	}
 
 	return req.CreateNetworkPayload(payload)

internal/cmd/beta/network/update/update.go

@@ -54,7 +54,7 @@ func NewCmd(p *print.Printer) *cobra.Command {
 			),
 			examples.NewExample(
 				`Update IPv6 network with ID "xxx" with new name "network-1-new" and new DNS name servers`,
-				`$ stackit beta network update xxx --name network-1-new --ipv6-dns-name-servers "2.2.2.2"`,
+				`$ stackit beta network update xxx --name network-1-new --ipv6-dns-name-servers "2001:4860:4860::8888"`,
 			),
 		),
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -151,17 +151,23 @@ func parseInput(p *print.Printer, cmd *cobra.Command, inputArgs []string) (*inpu
 
 func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APIClient) iaas.ApiPartialUpdateNetworkRequest {
 	req := apiClient.PartialUpdateNetwork(ctx, model.ProjectId, model.NetworkId)
+	addressFamily := &iaas.UpdateNetworkAddressFamily{}
+
+	if model.IPv6DnsNameServers != nil {
+		addressFamily.Ipv6 = &iaas.V1UpdateNetworkIPv6{
+			Nameservers: model.IPv6DnsNameServers,
+		}
+	}
+
+	if model.IPv4DnsNameServers != nil {
+		addressFamily.Ipv4 = &iaas.UpdateNetworkIPv4{
+			Nameservers: model.IPv4DnsNameServers,
+		}
+	}
 
 	payload := iaas.PartialUpdateNetworkPayload{
-		Name: model.Name,
-		AddressFamily: &iaas.UpdateNetworkAddressFamily{
-			Ipv4: &iaas.UpdateNetworkIPv4{
-				Nameservers: model.IPv4DnsNameServers,
-			},
-			Ipv6: &iaas.V1UpdateNetworkIPv6{
-				Nameservers: model.IPv6DnsNameServers,
-			},
-		},
+		Name:          model.Name,
+		AddressFamily: addressFamily,
 	}
 
 	return req.PartialUpdateNetworkPayload(payload)