Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

converting from layer2 to layer3 bond with IPV4 and IPV6 shows some junk output #386

Closed
displague opened this issue Nov 9, 2023 · 1 comment · Fixed by #390
Closed

converting from layer2 to layer3 bond with IPV4 and IPV6 shows some junk output #386

displague opened this issue Nov 9, 2023 · 1 comment · Fixed by #390

Comments

@displague
Copy link
Member 

          why port convert from layer2 to layer3 bond with IPV4 and IPV6 shows some Junk data

Vasubabus-MacBook-Pro:bin vasubabu$ ./metal port convert -i d414a2ca-2e91-4e2d-a598-42180c354e9d -2=false -b -4 -6
2023/11/07 23:29:57 Converting port d414a2ca-2e91-4e2d-a598-42180c354e9d to layer-3 with addresses [{0x140002c2540 0x140002c2544 map[]} {0x140002c2540 0x140002c2544 map[]} {0x140002c2540 0x140002c2544 map[]}]

Originally posted by @Vasubabu in #355 (review)
@displague displague mentioned this issue Nov 9, 2023
Merged
@aayushrangwala
Copy link
Contributor

aayushrangwala commented Nov 15, 2023 •
edited
Loading 

go run main.go port convert -i 08c363e1-d61b-48ba-913a-b57e7f6ed479 -2=false -b -4 -6                                                                       

2023/11/15 18:14:22 Converting port 08c363e1-d61b-48ba-913a-b57e7f6ed479 to layer-3 with [public] addresses IPv[[4 6]]
+--------------------------------------+-------+-----------------+--------------+-----+--------+
|                  ID                  | NAME  |      TYPE       | NETWORK TYPE | MAC | BONDED |
+--------------------------------------+-------+-----------------+--------------+-----+--------+
| 08c363e1-d61b-48ba-913a-b57e7f6ed479 | bond0 | NetworkBondPort | layer3       |     | true   |
+--------------------------------------+-------+-----------------+--------------+-----+--------+

@aayushrangwala aayushrangwala mentioned this issue Nov 15, 2023
Merged
displague added a commit that referenced this issue Nov 16, 2023
@displague
fix: output for convert layer2 to layer 3 command (#390)
54a34f3 
metal-cli Convert subcommand from layer 2 to layer 3 logs pointers for
the addresses

Fixes: #386
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: output for convert layer2 to layer 3 command aayushrangwala/metal-cli
2 participants
@displague @aayushrangwala