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

feat(metricprovider): allow user to define metrics.provider.job.metadata #2762

Conversation

hidalgopl
Copy link
Contributor

@hidalgopl hidalgopl commented May 7, 2023

Labels and annotations specified in AnalysisTemplate .spec.metrics.provider.job.metadata will be copied to the created Jobs.
discussed with @zachaller on Slack: https://cloud-native.slack.com/archives/C01U781DW2E/p1683298745937339

@hidalgopl hidalgopl force-pushed the add-labels-and-annotation-from-job-metrics-provider-to-job branch from 74d2812 to a5e7cbc Compare May 7, 2023 19:07
…o created Jobs metadata

Signed-off-by: Paweł Bojanowski <pbojanowski@protonmail.com>
@hidalgopl hidalgopl force-pushed the add-labels-and-annotation-from-job-metrics-provider-to-job branch from a5e7cbc to 0006ae4 Compare May 7, 2023 19:13
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2023

Go Published Test Results

1 951 tests   1 951 ✔️  2m 36s ⏱️
   118 suites         0 💤
       1 files           0

Results for commit 3313e4e.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented May 7, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a7852ba) 81.64% compared to head (3313e4e) 81.64%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2762   +/-   ##
=======================================
  Coverage   81.64%   81.64%           
=======================================
  Files         133      133           
  Lines       20172    20178    +6     
=======================================
+ Hits        16469    16475    +6     
  Misses       2849     2849           
  Partials      854      854           
Impacted Files Coverage Δ
metricproviders/job/job.go 87.41% <100.00%> (+0.52%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2023

E2E Tests Published Test Results

    4 files      4 suites   3h 16m 54s ⏱️
  96 tests   87 ✔️   5 💤 4
388 runs  364 ✔️ 20 💤 4

For more details on these failures, see this check.

Results for commit 30a5bea.

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented May 9, 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

@zachaller
Copy link
Collaborator

Thanks for the contribution.

@zachaller zachaller merged commit 39dbb4c into argoproj:master May 9, 2023
20 checks passed
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

2 participants