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.

Sign up for GitHub

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

Jump to bottom

List cloudapi compose jobs #4425

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

List cloudapi compose jobs #4425

wants to merge 4 commits into from

Conversation

bcl
Copy link
Contributor

@bcl bcl commented Oct 22, 2024

This pull request includes:

  • adequate testing for the new functionality or fixed issue
  • adequate documentation informing people about the change such as
    • submit a PR for the READMEs listed here
    • submit a PR for the osbuild.org website repository if this PR changed any behavior not covered by the automatically updated READMEs

@bcl
Copy link
Contributor Author

bcl commented Oct 22, 2024

This is just the compose job listing part of #4154 with the output using ComposeStatus instead of just the ids. I'll rewrite the other PR to just do the delete part after this is merged and the questions over there are closed.

@bcl
jobqueue: Add AllRootJobIDs function to jobqueue
a97ce68 
This lists the root job UUIDs (the jobs with no dependants).
Currently only implemented by fsjobqueue. The function for
dbjobqueue currently returns nil.

Related: RHEL-60120
@bcl
cloudapi: Add /composes route to list root job UUIDs
f46bb1d 
This will be used to list the top level job UUIDs.

Related: RHEL-60120
@bcl
cloudapi: Move most of getComposeStatusImpl into a function
7fbfca8 
This will make it easier to use the status in responses.

Related: RHEL-60120
@bcl
cloudapi: Return the compose status of all root compose jobs
ccd9b90 
This returns the status using the same structure as it does for
requesting individual statuses for the jobs.

Related: RHEL-60120
@achilleas-k achilleas-k self-requested a review November 7, 2024 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@achilleas-k achilleas-k Awaiting requested review from achilleas-k

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bcl