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

[PW_SID:826405] Using l_notice for low level IWD state information #312

Open
wants to merge 11 commits into
base: workflow
Choose a base branch
from
Open

[PW_SID:826405] Using l_notice for low level IWD state information #312

wants to merge 11 commits into from

Conversation

IWDTestBot
Copy link
Owner

The --logger,-l flag can now be used to specify the logger type.
Unset (default) will set log output to stderr as it is today. The
other valid options are "syslog" and "journal".

src/main.c | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)

@jprestwo @IWDTestBot
Workflow's for syncing with upstream, build, unit test, and test-runner
8aaf3da
@jprestwo @IWDTestBot
workflow: use newer commit for hostapd
e1d185b
@jprestwo @IWDTestBot
ci: remove cache/ from tar file list
b716eb1 
This is taken care of by the individual cache items and
if none exist, tar fails.
@jprestwo @IWDTestBot
ci: use kernel 5.19
ac81fdb
@jprestwo @IWDTestBot
ci: use iwd-ci after renaming to remove -v2
72405eb
@jprestwo @IWDTestBot
ci: remove set-output use, now deprecated
46b3556
@jprestwo @IWDTestBot
main: add runtime flag for setting the logger
b39d5f3 
The --logger,-l flag can now be used to specify the logger type.
Unset (default) will set log output to stderr as it is today. The
other valid options are "syslog" and "journal".
@jprestwo @IWDTestBot
iwd: add iwd_notice for special event/state type of logging
6744d9a 
iwd_notice is being added so modules can communicate internal
state or event information via the NOTICE log level. This log
level will be reserved in IWD for only these type of messages.

The iwd_notice macro aims to help enforce some formatting
requirements for these type of log messages. The messages
should be one or more comma-separated "key: value" pairs starting
with "event: <name>" and followed by any additional info that
pertains to that event.

iwd_notice only enforces the initial event key/value format and
additional arguments are left to the caller to be formatted
correctly.
@jprestwo @IWDTestBot
doc: document use of l_log APIs
ef78b17 
With the introduction of l_notice in IWD some guidelines need to be
set for l_info, l_warn, l_error, l_debug and l_notice.
@jprestwo @IWDTestBot
station: use iwd_notice for state/event information
2413a7e 
For anyone debugging or trying to identify network infrastructure
problems the IWD DBus API isn't all that useful and ultimately
requires going through debug logs to figure out exactly what
happened. Having a concise set of debug logs containing only
relavent information would be very useful. In addition, having
some kind of syntax for these logs to be parsed by tooling could
automate these tasks.

This is being done, starting with station, by using iwd_notice
which internally uses l_notice. The use of the notice log level
(5) in IWD will be strictly for the type of messages described
above.
@jprestwo @IWDTestBot
netdev: add notice events for connection timeouts
410e6b8 
The information specific to auth/assoc/connect timeouts isn't
communicated to station so emit the notice events within netdev.
We could communicate this to station by adding separate netdev
events, but this does not seem worth it for this use case as
these notice events aren't strictly limited to station.
@IWDTestBot
Copy link
Owner Author

Fetch PR
Test ID: fetch
Desc: Fetch the PR commits for this CI run
Duration: 3.29 seconds
Result: PASS

Make Distcheck
Test ID: makedistcheck
Desc: Run distcheck to check the distribution
Duration: 55.44 seconds
Result: PASS

Build - Configure
Test ID: build
Desc: Configure the BlueZ source tree
Duration: 12.36 seconds
Result: PASS

Make Check
Test ID: makecheck
Desc: Run 'make check'
Duration: 2.14 seconds
Result: PASS

Make Check w/Valgrind
Test ID: makecheckvalgrind
Desc: Run 'make check' with Valgrind
Duration: 95.17 seconds
Result: PASS

Incremental Build with patches
Test ID: incremental_build
Desc: Incremental build per patch in the series
Duration: 265.91 seconds
Result: PASS

@IWDTestBot
Copy link
Owner Author

Fetch PR
Test ID: fetch
Desc: Fetch the PR commits for this CI run
Duration: 2.73 seconds
Result: PASS

GitLint
Test ID: gitlint
Desc: Run gitlint with rule in .gitlint
Duration: 2.29 seconds
Result: PASS

Make Distcheck
Test ID: makedistcheck
Desc: Run distcheck to check the distribution
Duration: 37.26 seconds
Result: PASS

Build - Configure
Test ID: build
Desc: Configure the BlueZ source tree
Duration: 9.66 seconds
Result: PASS

Make Check
Test ID: makecheck
Desc: Run 'make check'
Duration: 3.81 seconds
Result: PASS

Make Check w/Valgrind
Test ID: makecheckvalgrind
Desc: Run 'make check' with Valgrind
Duration: 77.62 seconds
Result: PASS

Incremental Build with patches
Test ID: incremental_build
Desc: Incremental build per patch in the series
Duration: 214.11 seconds
Result: PASS

Autotest Runner
Test ID: testrunner
Desc: Runs IWD's autotest framework
Duration: 12178.27 seconds
Result: PASS

Clang Build
Test ID: clang
Desc: Build IWD using clang compiler
Duration: 83.81 seconds
Result: PASS

@github-actions github-actions bot force-pushed the workflow branch 10 times, most recently from 9eef0d5 to d3b4175 Compare February 28, 2024 18:01
@github-actions github-actions bot force-pushed the workflow branch 4 times, most recently from 68c71d2 to 43f4327 Compare March 4, 2024 20:00
@github-actions github-actions bot force-pushed the workflow branch 2 times, most recently from 4170bb4 to c067bc7 Compare March 15, 2024 14:00
@github-actions github-actions bot force-pushed the workflow branch 5 times, most recently from 61cb7e5 to ede3c3a Compare August 23, 2024 17:32
@github-actions github-actions bot force-pushed the workflow branch 4 times, most recently from 8991f76 to ca0eb77 Compare September 3, 2024 15:26
@github-actions github-actions bot force-pushed the workflow branch 8 times, most recently from aada677 to 1163d8b Compare September 11, 2024 16:00
@github-actions github-actions bot force-pushed the workflow branch 2 times, most recently from cd38a46 to a84ea46 Compare September 25, 2024 15:00
@github-actions github-actions bot force-pushed the workflow branch 3 times, most recently from 9ef1a07 to b2ed861 Compare October 8, 2024 16:33
@github-actions github-actions bot force-pushed the workflow branch 5 times, most recently from 0375753 to 7e9a794 Compare October 24, 2024 21:25
@github-actions github-actions bot force-pushed the workflow branch 3 times, most recently from d961fcd to bd89ecb Compare November 13, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IWDTestBot @jprestwo