8000 [BACKPORT][v1.7.3][IMPROVEMENT] Change confusing error message to warning level · Issue #9917 · longhorn/longhorn · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[BACKPORT][v1.7.3][IMPROVEMENT] Change confusing error message to warning level #9917

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
Closed
github-actions bot opened this issue Dec 6, 2024 · 1 comment
Closed

[BACKPORT][v1.7.3][IMPROVEMENT] Change confusing error message to warning level #9917

github-actions bot opened this issue Dec 6, 2024 · 1 comment
Assignees
@yangchiu @derekbit
Labels
area/troubleshoot Troubleshoot related component/longhorn-manager Longhorn manager (control plane) kind/backport Backport request kind/improvement Request for improvement of existing function priority/1 Highly recommended to implement or fix in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Milestone
v1.7.3

Comments

@github-actions
Copy link
github-actions bot commented Dec 6, 2024

backport #9916
@github-actions github-actions bot added area/troubleshoot Troubleshoot related component/longhorn-manager Longhorn manager (control plane) kind/backport Backport request kind/improvement Request for improvement of existing function priority/1 Highly recommended to implement or fix in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated labels Dec 6, 2024
@github-actions github-actions bot added this to the v1.7.3 milestone Dec 6, 2024
@github-project-automation github-project-automation bot moved this to New Issues in Longhorn Sprint Dec 6, 2024
@derekbit derekbit moved this from New Issues to Ready For Testing in Longhorn Sprint Dec 6, 2024
@yangchiu
Copy link
Member

Verification passed on v1.7.x-head (longhorn-manager 8469e60) following the test steps:

$ kubectl logs longhorn-manager-q99w9 -n longhorn-system | grep "Failed to check if replica"
Defaulted container "longhorn-manager" out of: longhorn-manager, pre-pull-share-manager-image
time="2024-12-11T01:56:54Z" level=warning msg="Failed to check if replica test-1-r-4c86657a transitioned to mode RW after it was last healthy" func="controller.(*VolumeController).ReconcileEngineReplicaState" file="volume_controller.go:752" accessMode=rwo controller=longhorn-volume currentEngine=test-1-e-0 error="cannot parse timestamp : parsing time \"\" as \"2006-01-02T15:04:05Z07:00\": cannot parse \"\" as \"2006\"" frontend=blockdev migratable=false node=ip-10-0-1-169 owner=ip-10-0-1-169 state=attached volume=test-1
time="2024-12-11T01:56:54Z" level=warning msg="Failed to check if replica test-1-r-d750820f transitioned to mode RW after it was last healthy" func="controller.(*VolumeController).ReconcileEngineReplicaState" file="volume_controller.go:752" accessMode=rwo controller=longhorn-volume currentEngine=test-1-e-0 error="cannot parse timestamp : parsing time \"\" as \"2006-01-02T15:04:05Z07:00\": cannot parse \"\" as \"2006\"" frontend=blockdev migratable=false node=ip-10-0-1-169 owner=ip-10-0-1-169 state=attached volume=test-1
time="2024-12-11T01:56:54Z" level=warning msg="Failed to check if replica test-1-r-35503a49 transitioned to mode RW after it was last healthy" func="controller.(*VolumeController).ReconcileEngineReplicaState" file="volume_controller.go:752" accessMode=rwo controller=longhorn-volume currentEngine=test-1-e-0 error="cannot parse timestamp : parsing time \"\" as \"2006-01-02T15:04:05Z07:00\": cannot parse \"\" as \"2006\"" frontend=blockdev migratable=false node=ip-10-0-1-169 owner=ip-10-0-1-169 state=attached volume=test-1

@github-project-automation github-project-automation bot moved this from Ready For Testing to Closed in Longhorn Sprint Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yangchiu yangchiu

@derekbit derekbit

Labels
area/troubleshoot Troubleshoot related component/longhorn-manager Longhorn manager (control plane) kind/backport Backport request kind/improvement Request for improvement of existing function priority/1 Highly recommended to implement or fix in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Projects
Longhorn Sprint
Status: Closed
Milestone
v1.7.3
Development

No branches or pull requests
2 participants
@yangchiu @derekbit
0