Add shared state in lime-report#1242
Open
javierbrk wants to merge 3 commits into
Open
Conversation
Member
|
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. |
Member
|
@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. |
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. |
francoriba
added a commit
to fcefyn-testbed/lime-packages
that referenced
this pull request
May 20, 2026
lime-report.sh in this feed does not implement -m (PR libremesh#1242); invoking it with -m produced empty output and placeholder issue bodies. Run the script with default output, pass LG_PROXY/LG_ENV, and surface labgrid-client stderr on failure.
francoriba
added a commit
to fcefyn-testbed/lime-packages
that referenced
this pull request
May 23, 2026
lime-report.sh in this feed does not implement -m (PR libremesh#1242); invoking it with -m produced empty output and placeholder issue bodies. Run the script with default output, pass LG_PROXY/LG_ENV, and surface labgrid-client stderr on failure.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.
https://gist.github.com/javierbrk/1f8a45fdd47815100225f13eb57e28f4