-
Notifications
You must be signed in to change notification settings - Fork 39
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
Create event when chassis or module is removed manually #2998
Create event when chassis or module is removed manually #2998
Conversation
f9636d2
to
10065b5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2998 +/- ##
==========================================
- Coverage 60.48% 60.47% -0.01%
==========================================
Files 604 604
Lines 43801 43816 +15
Branches 48 48
==========================================
+ Hits 26491 26497 +6
- Misses 17298 17307 +9
Partials 12 12 ☔ View full report in Codecov by Sentry. |
10065b5
to
5b70fc4
Compare
61460f3
to
3118c78
Compare
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.
looks great, only nitpicking, as usual :)
-- Add new deviceState alert | ||
INSERT INTO alerttype (eventtypeid,alerttype,alerttypedesc) VALUES | ||
('deviceState','deviceDeletedChassis','The device has been removed as a chassis.'); | ||
INSERT INTO alerttype (eventtypeid,alerttype,alerttypedesc) VALUES | ||
('deviceState','deviceDeletedModule','The device has been removed as a module.'); |
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.
Come to think of it, I think I forgot to look at this in your previous PR. 5.10 is the current stable release, this will only hit 5.11, so the migration script should be named differently...
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 just found out that I was unable to merge 5.10.x back into master because the OUI changescript and a bugfix from 5.10 had conflicting changescript names. I had to forcibly rename the OUI script on master, so there may be breakage for devs that do not wipe and re-create their postgres databases.
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.
yeah fuked up the filename quite hard for PowerSupplyAndFan aswell
0e42d37
to
44ee21d
Compare
44ee21d
to
73739ee
Compare
Quality Gate passedIssues Measures |
Fixes #2983