Skip to content

feat: E-Series initial MEL alert rules and runbook - #4498

Open
seolson wants to merge 2 commits into
NetApp:mainfrom
seolson:osean-eseries-alerts
Open

feat: E-Series initial MEL alert rules and runbook#4498
seolson wants to merge 2 commits into
NetApp:mainfrom
seolson:osean-eseries-alerts

Conversation

@seolson

@seolson seolson commented Oct 5, 2026 •
edited
Loading

Copy link
Copy Markdown

resolves #4497

@cla-bot

cla-bot Bot commented Oct 5, 2026

Copy link
Copy Markdown

Thank you for your interest in contributing to the Harvest project! We require contributors to sign our Corporate contributor license agreement (CCLA), and we don"t have the user(s) @seolson on file. In order for us to review and merge your code, please follow the instructions in step 6 of creating a pull request.
After signing the CCLA, you can ask us to recheck this PR by posting @cla-bot check as a comment to the PR.

@seolson

seolson commented Oct 5, 2026

Copy link
Copy Markdown
Author

@cla-bot check

@cla-bot

cla-bot Bot commented Oct 5, 2026

Copy link
Copy Markdown

Thank you for your interest in contributing to the Harvest project! We require contributors to sign our Corporate contributor license agreement (CCLA), and we don"t have the user(s) @seolson on file. In order for us to review and merge your code, please follow the instructions in step 6 of creating a pull request.
After signing the CCLA, you can ask us to recheck this PR by posting @cla-bot check as a comment to the PR.

@cla-bot

cla-bot Bot commented Oct 5, 2026

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@cgrinds

cgrinds commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

@seolson thanks for the contribution. We'll review it.

You can ignore the cla-bot. I've included you in this PR #4499

rahulguptajss
rahulguptajss reviewed Oct 6, 2026
View reviewed changes
Comment thread docs/resources/eseries-alert-runbook.md Outdated

**Impact**: Protection

**MEL Event**: `MEL_EV_PI_DRIVE_LOCKED_OUT` (0x1020)

@rahulguptajss rahulguptajss Oct 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we change the hex values to integer values so that they match the alert rules?

@seolson seolson Oct 6, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added integer values to all events.

Comment thread docs/resources/eseries-alert-runbook.md

Open SANtricity System Manager and go to the Recovery Guru. If this problem is listed, follow the steps shown there. If it is not listed, the condition has already cleared.

### Cont Redundancy Loss

@rahulguptajss rahulguptajss Oct 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missin Remediation

@seolson seolson Oct 6, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added.

Comment thread docs/resources/eseries-alert-runbook.md

Open SANtricity System Manager and go to the Recovery Guru. If this problem is listed, follow the steps shown there. If it is not listed, the condition has already cleared.

### Vol Xfer Alert

@rahulguptajss rahulguptajss Oct 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing Remediation

@seolson seolson Oct 6, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added.

Comment thread docs/resources/eseries-alert-runbook.md

Volume not on preferred path due to AVT/RDAC failover

### Set Controller Failed

@rahulguptajss rahulguptajss Oct 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing Remediation

@seolson seolson Oct 6, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added.

Comment thread docs/resources/eseries-alert-runbook.md

Place controller offline

### SYMbol Cont Service Mode

@rahulguptajss rahulguptajss Oct 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing Remediation

@seolson seolson Oct 6, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added.

@cla-bot cla-bot Bot added the cla-signed label Oct 6, 2026

This branch has not been deployed

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

Reviewers

@rahulguptajss rahulguptajss rahulguptajss left review comments
@cgrinds cgrinds Awaiting requested review from cgrinds cgrinds is a code owner
@Hardikl Hardikl Awaiting requested review from Hardikl Hardikl is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

E-Series initial MEL alert rules and runbook

3 participants

@seolson @cgrinds @rahulguptajss