Skip to content

♻️ Api server: improve job creation speed #4053

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

Conversation

sanderegg
Copy link
Member

@sanderegg sanderegg commented Apr 3, 2023

What do these changes do?

Increase job creation speed by 30%, removing unnecessary call to director-v2

Related issue/s

How to test

@sanderegg sanderegg added the a:apiserver api-server service label Apr 3, 2023
@sanderegg sanderegg added this to the Jelly Beans milestone Apr 3, 2023
@sanderegg sanderegg requested a review from pcrespov as a code owner April 3, 2023 09:48
@sanderegg sanderegg self-assigned this Apr 3, 2023
@codecov
Copy link

codecov bot commented Apr 3, 2023

Codecov Report

Merging #4053 (bd93de7) into master (4255546) will decrease coverage by 18.4%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #4053      +/-   ##
=========================================
- Coverage    85.0%   66.6%   -18.4%     
=========================================
  Files         799     397     -402     
  Lines       37152   20573   -16579     
  Branches      950     174     -776     
=========================================
- Hits        31591   13721   -17870     
- Misses       5342    6803    +1461     
+ Partials      219      49     -170     
Flag Coverage Δ
integrationtests 66.6% <ø> (ø)
unittests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 620 files with indirect coverage changes

@sanderegg sanderegg force-pushed the api-server/improve_job_creation_speed branch from 380aa86 to bd93de7 Compare April 3, 2023 14:15
@codeclimate
Copy link

codeclimate bot commented Apr 3, 2023

Code Climate has analyzed commit bd93de7 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Security 1

View more on Code Climate.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sanderegg sanderegg merged commit 661a727 into ITISFoundation:master Apr 3, 2023
@sanderegg sanderegg deleted the api-server/improve_job_creation_speed branch April 3, 2023 15:29
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Apr 20, 2023
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:apiserver api-server service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AppTeam Std Simulations on S4L/AWS M1-12 Maintenance and DevOps
3 participants