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

Explore alternative methods for unpacking catalog data #66

Closed
everettraven opened this issue May 17, 2023 · 2 comments · Fixed by #145
Closed

Explore alternative methods for unpacking catalog data #66

everettraven opened this issue May 17, 2023 · 2 comments · Fixed by #145
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. needs discussion

Comments

@everettraven
Copy link
Collaborator

Following #65 it is probably a good idea to explore what alternatives exist for unpacking methods separate from copying implementations from rukpak.

Some potential alternatives:

  • Make catalogd a rukpak provisioner
  • Make the rukpak unpacking implementation a library
@anik120
Copy link
Collaborator

anik120 commented Jun 20, 2023

There's also concerns from @ncdc about the fidelity of pod logs being read by the controller (specifically, logs being complete, and therefore being a reliable source of complete content). Instead the suggestion is that unpack jobs/pods should send the data back to catalogd using a service endpoint:port that catalogd makes available for these uploads, and the logs are then stored reliably in the filesystem/create confimap/s with the data, before they're consumed

@everettraven everettraven added kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. needs discussion labels Aug 21, 2023
@anik120
Copy link
Collaborator

anik120 commented Sep 14, 2023

Should this be turned into a discussion?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. needs discussion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants