-
Notifications
You must be signed in to change notification settings - Fork 81
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.
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
Added a fix for event handling of different errors #8891
Conversation
event_type: 'ERROR', | ||
scope: 'NODE', | ||
severity: 'ERROR', | ||
state: 'DEGRADED' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why state is DEGRADED?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per my understanding, by DEGRADED
state we are referring that the operation can still be performed by making some adjustments to the command(could be some permission issue). Please correct me if I am wrong
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it affects the state of the cluster, @naveenpaul1 can you explain when event's state should be DEGRADED and when it should be HEALTHY?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes @romayalon, We have added DEGRADED stat to those events that could be the reason for potential I/O failers and core Noobaa config dir-related issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@naveenpaul1 but does it do anything to the cluster? like ip movement etc? or it might happen only if health reported an error?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@romayalon I also think this event should not have DEGRADED
stat. For I/O events we added DEGRADED stat assuming could be because of some internal issue something related to cluster or storage itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I will update it to HEALTHY then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@naveenpaul1 @achouhan09 maybe you can add this definition of DEGRADED
stat in our docs so it would be clear to others as well.
event_type: 'ERROR', | ||
scope: 'NODE', | ||
severity: 'ERROR', | ||
state: 'DEGRADED' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes @romayalon, We have added DEGRADED stat to those events that could be the reason for potential I/O failers and core Noobaa config dir-related issue.
Signed-off-by: Aayush Chouhan <achouhan@redhat.com>
Describe the Problem
Previously, when an InternalError occurred during an operation, it was not recorded as an event.
Explain the Changes
Issues: Fixed #xxx / Gap #xxx
Testing Instructions:
make rpm
insidenoobaa-core
to create rpm, which will be found in dirbuild/rpm/
after creation.$ docker pull redhat/ubi9:latest
$ docker run --privileged -it --user root -d --platform=linux/amd64 redhat/ubi9 /usr/sbin/init
$ docker exec -it <container-id> bash
noobaa-core
(update rpm name):$ docker cp build/rpm/noobaa-core-5.19.0-20250326.el9.x86_64.rpm <container-id>:tmp/
$ yum install -y rsyslog wget make initscripts
$ systemctl start rsyslog
$ wget https://rpmfind.net/linux/centos-stream/9-stream/AppStream/x86_64/os/Packages/boost-system-1.75.0-8.el9.x86_64.rpm
$ wget https://rpmfind.net/linux/centos-stream/9-stream/AppStream/x86_64/os/Packages/boost-thread-1.75.0-8.el9.x86_64.rpm
$ rpm -i boost-system-1.75.0-8.el9.x86_64.rpm
$ rpm -i boost-thread-1.75.0-8.el9.x86_64.rpm
$ rpm -i tmp/noobaa-core-5.19.0-20250326.el9.x86_64.rpm
$ rpm -qa | grep noobaa
(check if noobaa installed or not)InternalError
, for example try adding thisthrow ManageCLIError.InternalError;
as first line of functiondelete_account()
inmanage_nsfs.js
. After that try creating and then deleting account using noobaa-cli and check the noobaa-events in the below steps --> new event for InternalError should be created.$ cat var/log/noobaa_events.log