Merge lwaftr to wingo-next #1328
Merged
Conversation
This file contains 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
Merge v2017.11
Merge config refactor
We'll also start moving utilities from lib.ptree.ptree into lib.yang.path_data.
* src/program/config/listen/listen.lua: Default path to '/'.
The trace log is in a JSON format that "snabb config listen" can play back.
The get-config RPC would want to create an array of Lua strings for the returned config. For large configs (e.g. the 1M-entry binding table that's already 122 MB raw), this is too much memory overhead. Instead change to incrementally write bytes to an output device held off-heap.
Improve performance of "snabb config get" on big configurations
2017.11.01 changelog
Fix property-based test stall
The conversion of localtime to ISO8601 format didn't include timezone information. For instance, timestamp 0 (UNIX epoch) was converted to in GMT +01:00: 1970-01-01T00:00:00Z While it should be: 1970-01-01T00:00:00Z+01:00 So now all alarms date information works using localtime and converting it to ISO8601 including timezone info. When an ISO8601 date is converted to seconds it takes into account timezone information. That makes possible to compare local timestamps and ISO8601 dates.
Rework alarms timestamping logic
* src/lib/yang/data.lua (data_grammar_from_schema): Prune cases with no members. Can be the case when we're asking for state data, for example, and a case has no state data.
The path selector and value generator now use the grammar instead of the schema. Invalid path and value generation are now parameterized by a function argument instead of being a global variable.
Property based tests redux
This extends loadtest's find-limit command so that multiple sets of traffic and NICs can be configured. The support is similar to how the loadtest transient command accepts multiple NICs.
Extend find-limit to support multiple NICs.
This commit fixes two bugs:
* It was impossible to distinguish between a nonempty batch of dirents
that contains all deleted entries (d.ino == 0) and a terminal batch
of dirents (size == 0). To fix this, we modify the getdents /
getdirentries interfaces to return nil, nil in the terminal case.
This is an incompatible change, but most users are probably using
the util.ls or util.dirtable methods.
* The util.ls method left the directory FD open, to be later closed by
GC. This can lead to running out of FDs if GC doesn't happen soon
enough. The fix is to close the directory fd after the iterator
terminates normally, in addition to closing the fd in error cases.
There's no need to use this when ljsyscall's "ls" and "dirtable" exist.
Avoid fd leaks in "snabb config get-state"
Merge v2018.04 prerelease from next to lwaftr
XPath formatter fixes
Implement alarm shelving
Fix undefined variable
Otherwise, we would never claim the name when starting with an empty config.
This replaces "sleep" calls with deterministic waits.
Make lwAFTR selftests less flaky
|
merging my own pr to my own branch, la la la |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This commit merges in the following Snabb lwAFTR releases:
2017.11.01
2018.01.2.1
2018.04.01 (pending)
See the linked CHANGELOG entries for full details.