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

Job Targets CSV Export and Jobs Registry readonly REST API #2789

Merged

Conversation

lorthirk
Copy link

@lorthirk lorthirk commented Dec 6, 2019

This PR introduces two features:

  • A console user can now export Targets for a given Job to a CSV file
  • Readonly REST APIs are now available to query all Jobs related informations. The following endpoints are now available:
    • Jobs:
      • GET {scopeId}/jobs
      • GET {scopeId}/jobs/{jobId}
      • POST {scopeId}/jobs/{jobId}/_query
      • POST {scopeId}/jobs/{jobId}/_count
    • Targets:
      • GET {scopeId}/jobs/{jobId}/targets
      • GET {scopeId}/jobs/{jobId}/targets/{targetId}
      • GET {scopeId}/jobs/{jobId}/targets/{targetId}/executions
      • POST {scopeId}/jobs/{jobId}/targets/_query
      • POST {scopeId}/jobs/{jobId}/targets/_count
    • Executions:
      • GET {scopeId}/jobs/{jobId}/executions
      • GET {scopeId}/jobs/{jobId}/executions/{executionId}
      • GET {scopeId}/jobs/{jobId}/executions/{executionId}/targets
      • POST {scopeId}/jobs/{jobId}/executions/_query
      • POST {scopeId}/jobs/{jobId}/executions/_count
    • Triggers:
      • GET {scopeId}/jobs/{jobId}/triggers
      • GET {scopeId}/jobs/{jobId}/triggers/{triggerId}
      • POST {scopeId}/jobs/{jobId}/triggers/_query
      • POST {scopeId}/jobs/{jobId}/triggers/_count

Related Issue
No related issues

Claudio Mezzasalma added 2 commits December 5, 2019 11:06
Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
@lorthirk lorthirk added Bug This is a bug or an unexpected behaviour. Fix it! Enhancement This PR/Issue improves an part of Kapua REST API We hope to REST everyday labels Dec 6, 2019
@lorthirk lorthirk requested a review from Coduz December 6, 2019 08:29
@codecov
Copy link

codecov bot commented Dec 6, 2019

Codecov Report

Merging #2789 into develop will increase coverage by 16.21%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             develop    #2789       +/-   ##
==============================================
+ Coverage      41.05%   57.27%   +16.21%     
- Complexity      2591     3163      +572     
==============================================
  Files           1074     1074               
  Lines          30474    30590      +116     
  Branches        2427     2455       +28     
==============================================
+ Hits           12512    17519     +5007     
+ Misses         17005    11899     -5106     
- Partials         957     1172      +215
Impacted Files Coverage Δ Complexity Δ
...lipse/kapua/service/scheduler/trigger/Trigger.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...pse/kapua/transport/mqtt/MqttClientErrorCodes.java 0% <0%> (-100%) 0% <0%> (-1%)
...ssage/kura/app/notification/KuraNotifyMetrics.java 69.69% <0%> (-30.31%) 3% <0%> (ø)
...ipse/kapua/transport/mqtt/MqttClientException.java 0% <0%> (-28.58%) 0% <0%> (-1%)
...s/message/notification/KapuaNotifyChannelImpl.java 75% <0%> (-25%) 0% <0%> (ø)
...anager/DeviceManagementRegistryManagerService.java 54.46% <0%> (-19.87%) 0% <0%> (ø)
.../DeviceManagementNotificationMessageProcessor.java 54.38% <0%> (-13.91%) 0% <0%> (ø)
...ator/kura/kapua/TranslatorLifeNotifyKuraKapua.java 75.75% <0%> (-13.14%) 10% <0%> (+1%)
...ssage/kura/app/notification/KuraNotifyPayload.java 46.66% <0%> (-11.67%) 9% <0%> (ø)
...ssage/kura/app/notification/KuraNotifyChannel.java 44.44% <0%> (-11.12%) 2% <0%> (+1%)
... and 248 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb446c9...aa3079b. Read the comment docs.

@Coduz Coduz merged commit 1fec25c into eclipse-kapua:develop Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it! 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.

2 participants