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

🐛 kairos register output is dirty #1251

Closed
mauromorales opened this issue Apr 5, 2023 · 2 comments · Fixed by kairos-io/go-nodepair#12
Closed

🐛 kairos register output is dirty #1251

mauromorales opened this issue Apr 5, 2023 · 2 comments · Fixed by kairos-io/go-nodepair#12
Assignees
@jimmykarily
Labels
bug Something isn't working

Comments

@mauromorales
Copy link
Member

mauromorales commented Apr 5, 2023 •
edited by jimmykarily
Loading

Dirty output (this is for the 2.0 rc release)

zeno% ./kairos-cli register --config ./config.yaml --device auto ~/Pictures/kairos3.png                              
Sending registration payload, please wait
2023/04/04 23:33:48 failed to sufficiently increase receive buffer size (was: 208 kiB, wanted: 2048 kiB, got: 416 kiB). See https://github.com/quic-go/quic-go/wiki/UDP-Receive-Buffer-Size for details.
Not enough nodes
map[]
Not enough nodes
map[healthcheck:map[12D3KooWPa3uoaFDVszwPrELAmriHtxEqfeGEqK2ox71gBuhwoQD:"2023-04-04T21:33:54Z" 12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:"2023-04-04T21:33:51Z"] pairing:map[12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:"ok" data:{"cc":"#cloud-config\nusers:\n  - name: kairos\n    passwd: kairos\n","device":"auto"}] presence:map[12D3KooWPa3uoaFDVszwPrELAmriHtxEqfeGEqK2ox71gBuhwoQD:"" 12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:""]]
Pairing in progress
map[healthcheck:map[12D3KooWPa3uoaFDVszwPrELAmriHtxEqfeGEqK2ox71gBuhwoQD:"2023-04-04T21:33:54Z" 12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:"2023-04-04T21:33:51Z"] pairing:map[12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:"ok" data:{"cc":"#cloud-config\nusers:\n  - name: kairos\n    passwd: kairos\n","device":"auto"}] presence:map[12D3KooWPa3uoaFDVszwPrELAmriHtxEqfeGEqK2ox71gBuhwoQD:"" 12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw:""]]
Active 12D3KooWPa3uoaFDVszwPrELAmriHtxEqfeGEqK2ox71gBuhwoQD
Active 12D3KooWRFhnDq5EeCzRiHKEKGSp2cAnkSCjXPxvnVw54KVmFnuw
Payload sent, installation will start on the machine briefly

Let's remove the map[] prints
@mauromorales mauromorales added the bug Something isn't working label Apr 5, 2023
@mauromorales mauromorales assigned mudler and unassigned mudler Apr 5, 2023
@mauromorales mauromorales moved this to In Progress 🏃 in 🧙Issue tracking board Apr 24, 2023
@mauromorales mauromorales moved this from In Progress 🏃 to Todo 🖊 in 🧙Issue tracking board Apr 24, 2023
@jimmykarily jimmykarily self-assigned this Apr 24, 2023
@jimmykarily jimmykarily moved this from Todo 🖊 to In Progress 🏃 in 🧙Issue tracking board Apr 24, 2023
jimmykarily added a commit to kairos-io/go-nodepair that referenced this issue Apr 24, 2023
@jimmykarily
Remove debugging print statement
d8f0757 
Partially fixes: kairos-io/kairos#1251
(needs other repos to bump to this commit)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
@jimmykarily jimmykarily mentioned this issue Apr 24, 2023
Merged
@jimmykarily
Copy link
Contributor

First PR: kairos-io/go-nodepair#12

@jimmykarily
Copy link
Contributor

jimmykarily commented Apr 24, 2023 •
edited
Loading

I'll bump go-nodepair in provider-kairos as part of this: kairos-io/provider-kairos#306
because I've already done the work to bump edgevpn and all.

Update: Done here

jimmykarily added a commit to kairos-io/provider-kairos that referenced this issue Apr 24, 2023
@jimmykarily
Bump go-nodepair to fix kairos-io/kairos#1251
0cfa204 
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
@github-project-automation github-project-automation bot moved this from In Progress 🏃 to Done ✅ in 🧙Issue tracking board Apr 25, 2023
jimmykarily added a commit to kairos-io/provider-kairos that referenced this issue May 3, 2023
@jimmykarily
Bump go-nodepair to fix kairos-io/kairos#1251
79dbbb2 
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jimmykarily jimmykarily

Labels
bug Something isn't working
Projects
🧙Issue tracking board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove debugging print statement kairos-io/go-nodepair
3 participants
@mauromorales @mudler @jimmykarily