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

Simplify OpenQA::Worker::Job::_stop_step_4_upload with turned around if-condition #3493

Merged
merged 3 commits into from
Nov 9, 2020

Conversation

okurz
Copy link
Member

@okurz okurz commented Oct 28, 2020

No description provided.

lib/OpenQA/Worker/Job.pm Outdated Show resolved Hide resolved
lib/OpenQA/Worker/Job.pm Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Nov 9, 2020

Codecov Report

Merging #3493 (f66c6ee) into master (b61ccfb) will decrease coverage by 0.00%.
The diff coverage is 55.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3493      +/-   ##
==========================================
- Coverage   95.61%   95.61%   -0.01%     
==========================================
  Files         366      366              
  Lines       31749    31745       -4     
==========================================
- Hits        30356    30352       -4     
  Misses       1393     1393              
Impacted Files Coverage Δ
lib/OpenQA/Worker.pm 94.51% <ø> (ø)
t/24-worker-jobs.t 97.81% <ø> (-0.01%) ⬇️
lib/OpenQA/Worker/Job.pm 74.88% <31.70%> (ø)
t/24-worker-overall.t 99.12% <100.00%> (+<0.01%) ⬆️
t/25-downloader.t 98.29% <100.00%> (-0.06%) ⬇️

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 7bfaab8...f66c6ee. Read the comment docs.

@okurz
Copy link
Member Author

okurz commented Nov 9, 2020

the OBS check failed due to an unrelated problem about chromedriver and some changed dependencies in Factory, probably an incomplete Factory rebuild caused this or something.

crossing fingers

@okurz okurz merged commit c341c41 into os-autoinst:master Nov 9, 2020
@okurz okurz deleted the enhance/worker_job branch November 9, 2020 20:44
openqabot pushed a commit to openqabot/openQA that referenced this pull request Nov 10, 2020
commit c341c41
Merge: 7bfaab8 f66c6ee
Author:     Oliver Kurz <okurz@suse.de>
AuthorDate: Mon Nov 9 21:44:51 2020 +0100
Commit:     GitHub <noreply@github.com>
CommitDate: Mon Nov 9 21:44:51 2020 +0100

    Merge pull request os-autoinst#3493 from okurz/enhance/worker_job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants