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

Move and adjust the s3 inventory #25

Open
5 of 6 tasks
stijnvanhoey opened this issue Jan 23, 2023 · 4 comments
Open
5 of 6 tasks

Move and adjust the s3 inventory #25

stijnvanhoey opened this issue Jan 23, 2023 · 4 comments
Assignees

Comments

@stijnvanhoey
Copy link
Collaborator

stijnvanhoey commented Jan 23, 2023

  • create new bucket "aloft-inventory" and move inventory their
  • use the CSV inventory (not the parquet one)
  • add the functionality to remove old inventory files (from manifest -> remove files)
  • inventory need to be applied for all 'sources' (baltrad, ecol,...)
  • the file name is used as single source of truth and the file name <-> s3-path logic is a function that can be passed (injected) to the routine.
  • assign ec2 engine rights to access aloft-inventory bucket
@stijnvanhoey stijnvanhoey self-assigned this Jan 23, 2023
@stijnvanhoey
Copy link
Collaborator Author

New bucket is created and inventory configuration switched.

inventory need to be applied for all 'sources' (baltrad, ecol,...)

The setup is done that it looks for */hdf5 as prefix of the path; need to verify that this effectively works.

@peterdesmet
Copy link
Member

peterdesmet commented Feb 6, 2023

@stijnvanhoey there isn't necessarily an issue to include daily and monthly in that inventory. Those could also be included in the coverage file. It's fairly easy for the user to filter them out

@stijnvanhoey
Copy link
Collaborator Author

As the inventory is not working with the *-wildcard, w"ll manage without the daily/monthly and filter them on the data routine.

@peterdesmet peterdesmet added this to the AWS deployment milestone May 8, 2023
@stijnvanhoey
Copy link
Collaborator Author

add the functionality to remove old inventory files (from manifest -> remove files)

Will be tackled by @TheJenne18 as part of the updated deployment

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

No branches or pull requests

2 participants