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

Refresh AWS last used date #22

Closed
wants to merge 4 commits into from
Closed

Refresh AWS last used date #22

wants to merge 4 commits into from

Conversation

inkel
Copy link
Collaborator

@inkel inkel commented May 26, 2022

Note that AWS CloudTrail only allows looking up for events in the past 90 days, thus we have to make some assumptions.

This currently hangs the whole UI and it's a hack that works, but a hack nonetheless.

inkel added 4 commits May 24, 2022 15:56
This is currently unexported, but we'll add another function to
retrieve the last used date from querying AWS CloudTrail DetachVolume
events related with this disk, and update if found.

Signed-off-by: Leandro López (inkel) <leandro.lopez@grafana.com>
This will be useful later when trying to retrieve the disks last used date.

Signed-off-by: Leandro López (inkel) <leandro.lopez@grafana.com>
This is a bit of a hack and it only works on one volume at a time, yet
it is a valid solution nonetheless. Note that it seems LookupEvents is
only taking the first lookup attribute for filtering, so we need to
iterate over the results later.

Also, AWS only report events for the past 90 days, anything greater
than that it will be discarded.

Signed-off-by: Leandro López (inkel) <leandro.lopez@grafana.com>
Note that AWS CloudTrail only allows looking up for events in the past
90 days, thus we have to make some assumptions.

This currently hangs the whole UI and it's a hack that works, but a
hack nonetheless.

Signed-off-by: Leandro López (inkel) <leandro.lopez@grafana.com>
@inkel inkel self-assigned this May 26, 2022
@inkel inkel marked this pull request as ready for review May 26, 2022 18:44
@inkel
Copy link
Collaborator Author

inkel commented Nov 30, 2022

Closing this as it won't be needed for an MVP release. We might want to review this idea in the future.

@inkel inkel closed this Nov 30, 2022
@inkel inkel deleted the feature/aws-last-used-at branch November 30, 2022 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant