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

Undeprecate upload_catalogs option #359

Open
mtneug opened this issue Dec 16, 2021 · 1 comment
Open

Undeprecate upload_catalogs option #359

mtneug opened this issue Dec 16, 2021 · 1 comment

Comments

@mtneug
Copy link
Member

mtneug commented Dec 16, 2021

The upload_catalogs option has been deprecated for future removal. Could this decision be reverted?

  • When reingesting (e.g. because the recording was deleted from Opencast by mistake) the new recording has no metadata making it hard to manage.
  • When deleting the video while it is recorded the ingest will still happen again with no metadata.
@lkiesow
Copy link
Member

lkiesow commented Jan 19, 2022

When deleting the video while it is recorded the ingest will still happen again with no metadata.

I think that it would make more sense for pyCA to not ingest the media in such a case. If someone deleted the scheduled event in Opencast, it shouldn't magically re-appear.

When reingesting (e.g. because the recording was deleted from Opencast by mistake) the new recording has no metadata making it hard to manage.

That's also only a problem if it has been completely removed, in which case it actually makes sense for it to not magically re-appear. If it's just that the ingest broke or there was a processing problem, a re-ingest works just fine without these catalogs. Still, we could offer an option to add the title (which we should always have) in these cases, making the events easier to handle. We are talking about manually retrieving events in these cases anyway.

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