Skip to content

Add shared state in lime-report#1242

Open
javierbrk wants to merge 3 commits intolibremesh:masterfrom
javierbrk:add_shared_state_in_report
Open

Add shared state in lime-report#1242
javierbrk wants to merge 3 commits intolibremesh:masterfrom
javierbrk:add_shared_state_in_report

Conversation

@javierbrk
Copy link
Copy Markdown
Collaborator

Lime report has a lot of interesting commands to debug node status in a network.

This pull request adds shared-state information to the report.

This pull also adds the -m option to generate a very simple "md" to improve readability.

lime-report.sh -m

https://gist.github.com/javierbrk/1f8a45fdd47815100225f13eb57e28f4

@ilario
Copy link
Copy Markdown
Member

ilario commented Feb 21, 2026

Thanks!!!

Adding the output of shared-state is very useful!

And the cirrent output format can be completely replaced by the Markdown version, in my opinion. I mean: there is no need to have an option for switching from legacy output format to Markdown output: we can directly discard the legacy output format and use only the Markdown formatted one.
Do you agree @javierbrk?

@ilario
Copy link
Copy Markdown
Member

ilario commented Apr 22, 2026

@javierbrk @javierajorge if you prefer I can add a commit to this pull request simplifying the code and removing the legacy output. Let me know.

@javierbrk
Copy link
Copy Markdown
Collaborator Author

I think it's important to maintain compatibility in case some tool uses this output, but couldn't find any. I agree to discard the old format. Please unlock the task when you can, I can work on it by the weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants