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

API to search build details #544

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

API to search build details #544

wants to merge 11 commits into from

Conversation

munishchouhan
Copy link
Member

@munishchouhan munishchouhan commented Jun 25, 2024 •
edited
Loading

This PR will add an API to get builds for a specific image or user

API: /v1alpha1/builds
query parameters: imageName and user

@munishchouhan munishchouhan linked an issue Jun 25, 2024 that may be closed by this pull request
Endpoint to look up build id by container name #541
Open
@munishchouhan munishchouhan marked this pull request as draft June 25, 2024 14:19
@munishchouhan munishchouhan self-assigned this Jun 25, 2024
@munishchouhan munishchouhan mentioned this pull request Jun 25, 2024
Open
@munishchouhan
Copy link
Member Author

@ewels In this PR I am fetching records only from database.
Now, when a build starts, Wave creates a record in the cache first, and when the build completes, it saves the record in the database and also updates it in the cache. So, do you need records from the cache, too?

@munishchouhan munishchouhan requested a review from ewels June 26, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ewels ewels Awaiting requested review from ewels

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

Assignees

@munishchouhan munishchouhan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Endpoint to look up build id by container name
1 participant
@munishchouhan