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

Remove some repeated log in Wifi hal #29

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Remove some repeated log in Wifi hal #29

merged 1 commit into from
Oct 16, 2024

Conversation

zhaoye1
Copy link
Contributor

@zhaoye1 zhaoye1 commented Oct 15, 2024 •
edited
Loading

Change log level to error,
close below repeated debug log:
10-15 11:22:21.370 522 1348 E wifi hal: Out of event poll
10-15 11:22:21.370 522 1348 D wifi hal: Data on event socket 0
10-15 11:22:21.370 522 1348 D wifi hal: Drv Event 70 received

Tests:
above log disapeared

Tracked-On: OAM-126416

@sysopenci
Copy link

Improper Commit Message
Valid Commit Message
Improper Jira Status,
jira status not in ['In Progress','Implemented']

@sysopenci sysopenci added Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Engineering Build Not Started Engineering Build Not Started Developer Approved and removed Pending Developer Approval Pending Developer Approval labels Oct 15, 2024
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Not Started Engineering Build Not Started labels Oct 15, 2024
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@zhaoye1
Remove some repeated log in Wifi hal
3bee5af 
Change log level to error,
close below repeated debug log:
10-15 11:22:21.370   522  1348 E wifi hal: Out of event poll
10-15 11:22:21.370   522  1348 D wifi hal: Data on event socket 0
10-15 11:22:21.370   522  1348 D wifi hal: Drv Event 70 received

Tests:
above log disapeared

Tracked-On: OAM-126416
Signed-off-by: Ye, Zhao <[email protected]>
Signed-off-by: Zhao Ye <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

dyang23
dyang23 approved these changes Oct 16, 2024
View reviewed changes
Copy link

@dyang23 dyang23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sysopenci sysopenci added PR approved and removed Pending PR Review Pending PR Review labels Oct 16, 2024
@sysopenci
Copy link

FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Failed and removed Engineering Build Successful Engineering Build Successful labels Oct 16, 2024
@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Failed labels Oct 16, 2024
RanderWang
RanderWang approved these changes Oct 16, 2024
View reviewed changes
Copy link

@RanderWang RanderWang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

gowthambabu92
gowthambabu92 approved these changes Oct 16, 2024
View reviewed changes
Copy link
Contributor

@gowthambabu92 gowthambabu92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sysopenci
Copy link

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci merged commit e49a5dc into projectceladon:celadon/u/mr0/master Oct 16, 2024
17 checks passed
@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon_umr0_master-merge/181

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

@gowthambabu92 gowthambabu92 gowthambabu92 approved these changes

@dyang23 dyang23 dyang23 approved these changes

@RanderWang RanderWang RanderWang approved these changes

@JeevakaPrabu JeevakaPrabu JeevakaPrabu approved these changes

Assignees
No one assigned
Labels
Developer Approved Engineering Build Successful Engineering Build Successful PR approved Valid commit message
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@zhaoye1 @sysopenci @JeevakaPrabu @RanderWang @dyang23 @gowthambabu92