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

Stream logs from TaskRuns and PipelineRuns #36

Merged
merged 5 commits into from
Sep 13, 2020

Conversation

waveywaves
Copy link
Member

fixes #22

@codecov
Copy link

codecov bot commented Sep 13, 2020

Codecov Report

Merging #36 into master will decrease coverage by 15.97%.
The diff coverage is 7.08%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #36       +/-   ##
===========================================
- Coverage   58.90%   42.92%   -15.98%     
===========================================
  Files           7        9        +2     
  Lines         292      417      +125     
  Branches       38       55       +17     
===========================================
+ Hits          172      179        +7     
- Misses         92      209      +117     
- Partials       28       29        +1     
Impacted Files Coverage Δ
...ekton/client/build/create/mock/CreateStepMock.java 77.77% <0.00%> (-22.23%) ⬇️
...ns/tekton/client/logwatch/PipelineRunLogWatch.java 0.00% <0.00%> (ø)
...lugins/tekton/client/logwatch/TaskRunLogWatch.java 0.00% <0.00%> (ø)
...plugins/tekton/client/build/create/CreateStep.java 45.37% <11.53%> (-9.47%) ⬇️
...ves/jenkins/plugins/tekton/client/TektonUtils.java 47.54% <40.00%> (-0.68%) ⬇️
...ugins/tekton/client/GlobalPluginConfiguration.java 50.00% <100.00%> (ø)
.../jenkins/plugins/tekton/client/build/BaseStep.java 100.00% <100.00%> (ø)

@waveywaves
Copy link
Member Author

waveywaves commented Sep 13, 2020

Considering the reduction in testing coverage, opened #37 to track adding tests

@waveywaves waveywaves merged commit 8c36d91 into jenkinsci:master Sep 13, 2020
@garethjevans garethjevans added the enhancement Related to improving any arbitary thing label Mar 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Related to improving any arbitary thing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stream output from TaskRun and PipelineRun to Jenkins Console
2 participants