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

Jobs REST API Step edit, Trigger edit, multiple Is Running #3295

Merged
merged 3 commits into from
Apr 27, 2021

Conversation

lorthirk
Copy link

Three new Jobs REST APIs:

  • PUT ​/{scopeId}​/jobs​/{jobId}​/steps​/{stepId} - Update a single Job Step
  • PUT ​/{scopeId}​/jobs​/{jobId}​/triggers​/{triggerId} - Update a single Job Trigger
  • POST ​/{scopeId}​/jobs​/_isRunning - Check if multiple Jobs are running

Request:

{
  "jobIds": [
    "dX63xIXNstA",
    "TqcLqDS4Wj4",
    "AQ"
  ]
}

Response:

{
  "list": [
    {
      "jobId": "AQ"
    },
    {
      "jobId": "TqcLqDS4Wj4",
      "isRunning": false
    },
    {
      "jobId": "dX63xIXNstA",
      "isRunning": false
    }
  ]
}

Related Issue
No related issues

@lorthirk lorthirk added Enhancement This PR/Issue improves an part of Kapua REST API We hope to REST everyday labels Apr 20, 2021
@lorthirk lorthirk requested a review from Coduz April 20, 2021 13:29
@codecov
Copy link

codecov bot commented Apr 20, 2021

Codecov Report

Merging #3295 (b3bf1fa) into develop (58f80bf) will decrease coverage by 1.04%.
The diff coverage is 23.95%.

❗ Current head b3bf1fa differs from pull request most recent head 7e990a9. Consider uploading reports for the commit 7e990a9 to get more accurate results
Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #3295      +/-   ##
=============================================
- Coverage      53.23%   52.18%   -1.05%     
+ Complexity       788      785       -3     
=============================================
  Files           1475     1478       +3     
  Lines          30151    30232      +81     
  Branches        2520     2522       +2     
=============================================
- Hits           16050    15776     -274     
- Misses         13203    13562     +359     
+ Partials         898      894       -4     
Impacted Files Coverage Δ Complexity Δ
.../commons/configuration/ServiceConfigQueryImpl.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
...vice/event/store/internal/EventStoreQueryImpl.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
...va/org/eclipse/kapua/commons/util/xml/XmlUtil.java 75.47% <0.00%> (-0.72%) 0.00 <0.00> (ø)
...apua/job/engine/client/JobEngineServiceClient.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...apua/job/engine/jbatch/JobEngineServiceJbatch.java 45.00% <0.00%> (-6.86%) 8.00 <0.00> (ø)
...gine/queue/jbatch/QueuedJobExecutionQueryImpl.java 100.00% <ø> (ø) 1.00 <0.00> (ø)
...i/core/model/job/IsJobRunningMultipleResponse.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...a/app/api/core/model/job/IsJobRunningResponse.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...a/app/api/core/model/job/MultipleJobIdRequest.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...apua/app/api/resources/v1/resources/JobEngine.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
... and 121 more

Claudio Mezzasalma added 2 commits April 21, 2021 10:35
Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
@lorthirk lorthirk force-pushed the feature-newJobsApiFeatures branch from b537711 to 7e990a9 Compare April 21, 2021 08:35
@lorthirk lorthirk merged commit 99dde48 into eclipse:develop Apr 27, 2021
@lorthirk lorthirk deleted the feature-newJobsApiFeatures branch April 27, 2021 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement This PR/Issue improves an part of Kapua REST API We hope to REST everyday
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant