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.

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

Device failure and recovery dates can be misleading #5169

Closed
bmfmancini opened this issue Jan 16, 2023 · 8 comments
Closed

Device failure and recovery dates can be misleading #5169

bmfmancini opened this issue Jan 16, 2023 · 8 comments
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone

Comments

@bmfmancini
Copy link
Member

I have seen cases where the status_fail_date in the host table is showing before the status_rec_date which would make it appear the devices failed in the future

MariaDB [cacti]> select status_fail_date,status_rec_date   from host where description = 'host' \G;  
*************************** 1. row ***************************
status_fail_date: 2023-01-16 05:16:10
 status_rec_date: 2023-01-16 05:11:06
MariaDB [cacti]> SELECT @@system_time_zone as tz;
+------+
| tz   |
+------+
| EDT  |
+------+

Timezone is correct on the system and the DB

@bmfmancini bmfmancini added bug Undesired behaviour unverified Some days we don't have a clue labels Jan 16, 2023
@bmfmancini
Copy link
Member Author

MariaDB [cacti]>  SELECT host_template_id,poller_id,status_fail_date,status_rec_date FROM host
    -> WHERE status_fail_date > status_rec_date AND DATE(status_fail_date) = CURDATE() AND DATE(status_rec_date) = CURDATE();
+------------------+-----------+---------------------+---------------------+
| host_template_id | poller_id | status_fail_date    | status_rec_date     |
+------------------+-----------+---------------------+---------------------+
|                1 |         1 | 2023-01-16 09:49:08 | 2023-01-16 08:42:07 |
|                1 |         1 | 2023-01-16 05:16:10 | 2023-01-16 05:11:06 |
|                1 |         1 | 2023-01-16 09:56:06 | 2023-01-16 09:22:04 |
|                1 |         1 | 2023-01-16 05:16:10 | 2023-01-16 05:11:04 |
|                1 |         1 | 2023-01-16 05:15:11 | 2023-01-16 05:11:05 |
|                1 |         1 | 2023-01-16 05:16:11 | 2023-01-16 05:11:05 |
|                6 |         7 | 2023-01-16 06:18:08 | 2023-01-16 05:42:04 |
|                1 |         1 | 2023-01-16 05:15:08 | 2023-01-16 05:12:03 |
|                1 |         1 | 2023-01-16 05:16:10 | 2023-01-16 05:11:03 |
|                1 |         1 | 2023-01-16 05:15:08 | 2023-01-16 05:12:03 |
+------------------+-----------+---------------------+---------------------+

TheWitness added a commit that referenced this issue Jan 16, 2023
Host table status_fail_date and status_rec_date can appear wrong
@bmfmancini bmfmancini removed the unverified Some days we don't have a clue label Jan 16, 2023
TheWitness added a commit that referenced this issue Jan 16, 2023
Host table status_fail_date and status_rec_date can appear wrong
@TheWitness TheWitness added confirmed Bug is confirm by dev team resolved A fixed issue labels Jan 16, 2023
@TheWitness TheWitness added this to the v1.2.24 milestone Jan 16, 2023
@TheWitness
Copy link
Member

Report back once you have this all tested.

@bmfmancini
Copy link
Member Author

will do

@TheWitness
Copy link
Member

Any feedback yet?

@TheWitness
Copy link
Member

@bmfmancini, that was for you.

@bmfmancini
Copy link
Member Author

Yes all good

@TheWitness
Copy link
Member

TheWitness commented Jan 19, 2023

Need to keep open until I've ported the changes.

@netniV netniV closed this as completed Jan 20, 2023
@Cacti Cacti deleted a comment from TheWitness Jan 20, 2023
@TheWitness TheWitness added the porting required Requires porting to develop label Jan 22, 2023
TheWitness added a commit that referenced this issue Jan 22, 2023
Host table status_fail_date and Status_rec_date can appear wrong
@TheWitness
Copy link
Member

Porting complete

@TheWitness TheWitness removed the porting required Requires porting to develop label Jan 22, 2023
@netniV netniV changed the title Host table status_fail_date and Status_rec_date can appear wrong Device failure and recovery dates can be misleading Feb 15, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

3 participants