Skip to content

Get extensions and packages versions to CI artifacts#2553

Merged
pablomartinezbernardo merged 33 commits intomasterfrom
pmartinez/version-reporting
Mar 18, 2024
Merged

Get extensions and packages versions to CI artifacts#2553
pablomartinezbernardo merged 33 commits intomasterfrom
pmartinez/version-reporting

Conversation

@pablomartinezbernardo
Copy link
Copy Markdown
Contributor

@pablomartinezbernardo pablomartinezbernardo commented Mar 4, 2024

Description

Use get_loaded_extensions() and composer show in CI to write dependency versions to a CSV as an artifact in CI, so it can be later retrieved to populate support data in metabase.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 4, 2024

Codecov Report

Merging #2553 (5a538da) into master (2e1ed12) will decrease coverage by 0.68%.
Report is 12 commits behind head on master.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2553      +/-   ##
============================================
- Coverage     75.91%   75.24%   -0.68%     
  Complexity     2574     2574              
============================================
  Files           240      240              
  Lines         27029    27029              
  Branches        976      976              
============================================
- Hits          20519    20337     -182     
- Misses         5990     6172     +182     
  Partials        520      520              
Flag Coverage Δ
appsec-extension 69.13% <ø> (ø)
tracer-extension 78.70% <100.00%> (ø)
tracer-php 73.31% <ø> (-1.77%) ⬇️

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

Files Coverage Δ
ext/integrations/integrations.c 97.88% <100.00%> (ø)

... and 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Mar 6, 2024

Benchmarks

Benchmark execution time: 2024-03-18 14:42:00

Comparing candidate commit 5a538da in PR branch pmartinez/version-reporting with baseline commit 2e1ed12 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 182 metrics, 0 unstable metrics.

@pablomartinezbernardo pablomartinezbernardo changed the title [Test] Try writing csv to artifacts folder Get extensions and packages versions to CI artifacts Mar 18, 2024
@pablomartinezbernardo pablomartinezbernardo marked this pull request as ready for review March 18, 2024 09:22
@pablomartinezbernardo pablomartinezbernardo requested a review from a team as a code owner March 18, 2024 09:22
Comment thread Makefile Outdated
@pablomartinezbernardo pablomartinezbernardo merged commit 91227e5 into master Mar 18, 2024
@pablomartinezbernardo pablomartinezbernardo deleted the pmartinez/version-reporting branch March 18, 2024 14:44
@github-actions github-actions Bot added this to the 0.99.0 milestone Mar 18, 2024
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.

4 participants