Make status.addresses JSON values

[andrewrynhard]

I would like to use kubectl get machine X -o jsonpath... to parse out addresses assigned to a node. Unfortunately the values in the status.addresses array are formatted golang data structures. Making these JSON would allow me to parse out whatever I needed from the entries.

status:
  addresses:
  ...
  - address: packngo.IPAddressAssignment{IpAddressCommon:packngo.IpAddressCommon{ID:"REDACTED",
      Address:"10.100.2.17", Gateway:"10.100.2.16", Network:"10.100.2.16", AddressFamily:4,
      Netmask:"255.255.255.254", Public:false, CIDR:31, Created:"2020-07-25T14:54:10Z",
      Updated:"", Href:"/ips/REDACTED", Management:true,
      Manageable:true, Project:packngo.Href{Href:"/projects/REDACTED"},
      Tags:[], CustomData:map[]}, AssignedTo:packngo.Href{Href:"/devices/REDACTED"}}
    type: InternalIP

[deitch]

Interesting that it is leaking through the packngo.IPAddressAssignment. Is the above output from a currently set up cluster via CAPP, @andrewrynhard ?

I think it is set here which calls here, but that is setting a corev1.NodeAddress on it.

Do you know why this is @gianarb ?

[andrewrynhard]

@deitch This is with the latest release of CAPP.

[gianarb]

In theory it should have a string: 10.100.2.17.
I will have a look at it very soon @andrewrynhard thank you for opening the issue

I think the bug is here:

https://github.com/kubernetes-sigs/cluster-api-provider-packet/blob/8f54262d2b2171e676e2922ce8e48414040b42c6/pkg/cloud/packet/client.go#L139

addr.String() is wrong, it should be something like addr.Address. I will try and cut a release by tomorrow (CET)