fix(deps): update module github.com/equinix-labs/metal-go to v0.29.0 (#379)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[github.com/equinix-labs/metal-go](https://github.com/equinix-labs/metal-go)
| require | minor | `v0.26.0` -> `v0.29.0` |

---

### Release Notes

<details>
<summary>equinix-labs/metal-go
(github.com/equinix-labs/metal-go)</summary>

###
[`v0.29.0`](https://github.com/equinix-labs/metal-go/releases/tag/v0.29.0)

[Compare
Source](https://github.com/equinix-labs/metal-go/compare/v0.28.0...v0.29.0)

##### Features

- patch bgpsession spec to remove status enum
([#&#8203;178](https://github.com/equinix-labs/metal-go/issues/178))
([cd21e3a](https://github.com/equinix-labs/metal-go/commit/cd21e3ae50c62304b11e4a55a263f85f9ccbd8d4))

###
[`v0.28.0`](https://github.com/equinix-labs/metal-go/releases/tag/v0.28.0)

[Compare
Source](https://github.com/equinix-labs/metal-go/compare/v0.27.0...v0.28.0)

##### Features

- API Sync by GitHub Action (2023-11-08)
([#&#8203;171](https://github.com/equinix-labs/metal-go/issues/171))
([ad2dc45](https://github.com/equinix-labs/metal-go/commit/ad2dc4553f61a02d6021890ad52cf045ba6b0535))

##### Bug Fixes

- avoid nil pointer error when debugging requests with no body
([#&#8203;176](https://github.com/equinix-labs/metal-go/issues/176))
([d87441c](https://github.com/equinix-labs/metal-go/commit/d87441ceae54e4c83dfdba0b2950b7a50c64e884))

###
[`v0.27.0`](https://github.com/equinix-labs/metal-go/releases/tag/v0.27.0)

[Compare
Source](https://github.com/equinix-labs/metal-go/compare/v0.26.0...v0.27.0)

##### Features

- upgrade openapi-generator to v7.1.0
([#&#8203;172](https://github.com/equinix-labs/metal-go/issues/172))
([d09946e](https://github.com/equinix-labs/metal-go/commit/d09946e09f2c71d5eb261beee58f5ad061cb718f))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/equinix/metal-cli).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMS41IiwidXBkYXRlZEluVmVyIjoiMzcuODcuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Charles Treatman <[email protected]>

Author: renovate[bot]

go.mod

@@ -3,7 +3,7 @@ module github.com/equinix/metal-cli
 go 1.19
 
 require (
-	github.com/equinix-labs/metal-go v0.26.0
+	github.com/equinix-labs/metal-go v0.29.0
 	github.com/olekukonko/tablewriter v0.0.5
 	github.com/packethost/packngo v0.30.0
 	github.com/pkg/errors v0.9.1

go.sum

@@ -60,8 +60,8 @@ github.com/envoyproxy/go-control-plane v0.9.4/go.mod h1:6rpuAdCZL397s3pYoYcLgu1m
 github.com/envoyproxy/go-control-plane v0.9.7/go.mod h1:cwu0lG7PUMfa9snN8LXBig5ynNVH9qI8YYLbd1fK2po=
 github.com/envoyproxy/go-control-plane v0.9.9-0.20201210154907-fd9021fe5dad/go.mod h1:cXg6YxExXjJnVBQHBLXeUAgxn2UodCpnH306RInaBQk=
 github.com/envoyproxy/protoc-gen-validate v0.1.0/go.mod h1:iSmxcyjqTsJpI2R4NaDN7+kN2VEUnK/pcBlmesArF7c=
-github.com/equinix-labs/metal-go v0.26.0 h1:0rBTyjF8j58dg++kMFLRi9Jhs5gng5BFn5Y0bl5NPtM=
-github.com/equinix-labs/metal-go v0.26.0/go.mod h1:SmxCklxW+KjmBLVMdEXgtFO5gD5/b4N0VxcNgUYbOH4=
+github.com/equinix-labs/metal-go v0.29.0 h1:dTi48+ni4Xbey0VJjHrkoAOxJvVPl9ukA/6DoJv6o8s=
+github.com/equinix-labs/metal-go v0.29.0/go.mod h1:SmxCklxW+KjmBLVMdEXgtFO5gD5/b4N0VxcNgUYbOH4=
 github.com/frankban/quicktest v1.14.4 h1:g2rn0vABPOOXmZUj+vbmUp0lPoXEMuhTpIluN0XL9UY=
 github.com/fsnotify/fsnotify v1.6.0 h1:n+5WquG0fcWoWp6xPWfHdbskMCQaFnG6PfBrh1Ky4HY=
 github.com/fsnotify/fsnotify v1.6.0/go.mod h1:sl3t1tCWJFWoRz9R8WJCbQihKKwmorjAbSClcnxKAGw=

internal/organizations/create.go

@@ -51,6 +51,13 @@ func (c *Client) Create() *cobra.Command {
 		RunE: func(cmd *cobra.Command, args []string) error {
 			cmd.SilenceUsage = true
 
+			// API spec says organization address.address is required,
+			// but the address is not included by default
+			defaultIncludes := []string{"address", "billing_address"}
+
+			include := c.Servicer.Includes(defaultIncludes)
+			exclude := c.Servicer.Excludes(nil)
+
 			req := metal.NewOrganizationInput()
 			req.Name = &name
 
@@ -70,7 +77,7 @@ func (c *Client) Create() *cobra.Command {
 				req.Logo = &logo
 			}
 
-			org, _, err := c.Service.CreateOrganization(context.Background()).OrganizationInput(*req).Include(c.Servicer.Includes(nil)).Exclude(c.Servicer.Excludes(nil)).Execute()
+			org, _, err := c.Service.CreateOrganization(context.Background()).OrganizationInput(*req).Include(include).Exclude(exclude).Execute()
 			if err != nil {
 				return fmt.Errorf("Could not create Organization: %w", err)
 			}

internal/organizations/retrieve.go

@@ -44,7 +44,12 @@ func (c *Client) Retrieve() *cobra.Command {
 
 		RunE: func(cmd *cobra.Command, args []string) error {
 			cmd.SilenceUsage = true
-			include := c.Servicer.Includes(nil)
+
+			// API spec says organization address.address is required,
+			// but the address is not included by default
+			defaultIncludes := []string{"address", "billing_address"}
+
+			include := c.Servicer.Includes(defaultIncludes)
 			exclude := c.Servicer.Excludes(nil)
 
 			if organizationID == "" {

internal/organizations/update.go

@@ -42,6 +42,13 @@ func (c *Client) Update() *cobra.Command {
 			cmd.SilenceUsage = true
 			req := metal.NewOrganizationInput()
 
+			// API spec says organization address.address is required,
+			// but the address is not included by default
+			defaultIncludes := []string{"address", "billing_address"}
+
+			include := c.Servicer.Includes(defaultIncludes)
+			exclude := c.Servicer.Excludes(nil)
+
 			if name != "" {
 				req.Name = &name
 			}
@@ -62,7 +69,7 @@ func (c *Client) Update() *cobra.Command {
 				req.Website = &website
 			}
 
-			org, _, err := c.Service.UpdateOrganization(context.Background(), organizationID).OrganizationInput(*req).Execute()
+			org, _, err := c.Service.UpdateOrganization(context.Background(), organizationID).OrganizationInput(*req).Include(include).Exclude(exclude).Execute()
 			if err != nil {
 				return fmt.Errorf("Could not update Organization: %w", err)
 			}

test/e2e/gateways/create_test.go

@@ -76,6 +76,7 @@ func TestGateways_Create(t *testing.T) {
 				apiClient := helper.TestClient()
 				gateways, _, err := apiClient.MetalGatewaysApi.
 					FindMetalGatewaysByProject(context.Background(), projectId).
+					Include([]string{"ip_reservation"}).
 					Execute()
 				if err != nil {
 					t.Error(err)

test/e2e/gateways/delete_test.go

@@ -93,6 +93,7 @@ func TestGateways_Delete(t *testing.T) {
 				apiClient := helper.TestClient()
 				gateways, _, err := apiClient.MetalGatewaysApi.
 					FindMetalGatewayById(context.Background(), metalGateway.GetId()).
+					Include([]string{"ip_reservation"}).
 					Execute()
 				if err != nil && !strings.Contains(err.Error(), "Not Found") {
 					t.Error(err)

test/helper/helper.go

@@ -377,7 +377,11 @@ func CreateTestOrganization(name string) (string, error) {
 	addr.SetZipCode("02108")
 	organizationInput.Address = addr
 
-	resp, _, err := TestApiClient.OrganizationsApi.CreateOrganization(context.Background()).OrganizationInput(*organizationInput).Execute()
+	// API spec says organization address.address is required,
+	// but the address is not included by default
+	defaultIncludes := []string{"address", "billing_address"}
+
+	resp, _, err := TestApiClient.OrganizationsApi.CreateOrganization(context.Background()).OrganizationInput(*organizationInput).Include(defaultIncludes).Execute()
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "Error when calling `OrganizationsApi.CreateOrganization``: %v\n", err)
 		return "", err