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

New API call to merge/update node data #466

Open
nuclearcat opened this issue Feb 7, 2024 · 1 comment
Open

New API call to merge/update node data #466

nuclearcat opened this issue Feb 7, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@nuclearcat
Copy link
Member

nuclearcat commented Feb 7, 2024

In some situations we might have minor updates for node data, for example we need only to add new data or update if field already exist, but we might now have up to date data. So if we will have such call, we can send only following:

{
"id": "65b218ce985d5bae000b0458",
"state": "done",
"result": "pass",
"data": {
    "new_field": { "new_subfield": "new_data" }
 }
}

Right now we have to fetch updated node data in kbuild for example, because job_id was added, and update node, while with this merge call we can just update state, result and add relevant test data.

@nuclearcat nuclearcat added the enhancement New feature or request label Feb 7, 2024
@nuclearcat
Copy link
Member Author

Relevant PR fixing issue related to outdated Node data: kernelci/kernelci-core#2344

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

1 participant