Skip to content

Conversation

pasha-codefresh
Copy link
Member

@pasha-codefresh pasha-codefresh commented Sep 28, 2021

Continuation for #7251

In case if tracking method defined as annotation resource id format should be "application-name;group/kind/namespace/name"

In case if tracking method is "label" or not defined nothing should be changed

Signed-off-by: pashavictorovich <pavel@codefresh.io>
Signed-off-by: pashavictorovich <pavel@codefresh.io>
Signed-off-by: pashavictorovich <pavel@codefresh.io>
@pasha-codefresh pasha-codefresh added the enhancement New feature or request label Sep 28, 2021
@pasha-codefresh pasha-codefresh added this to the v2.2 milestone Sep 28, 2021
@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #7316 (c7e7797) into master (a731997) will increase coverage by 0.07%.
The diff coverage is 96.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7316      +/-   ##
==========================================
+ Coverage   41.15%   41.23%   +0.07%     
==========================================
  Files         161      161              
  Lines       21553    21581      +28     
==========================================
+ Hits         8871     8899      +28     
  Misses      11415    11415              
  Partials     1267     1267              
Impacted Files Coverage Δ
reposerver/repository/repository.go 60.94% <0.00%> (ø)
util/argo/resource_tracking.go 87.23% <100.00%> (+18.81%) ⬆️

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 a731997...c7e7797. Read the comment docs.

Signed-off-by: pashavictorovich <pavel@codefresh.io>
…ue-convention-e2e

Tracking method app id value convention e2e
Signed-off-by: pashavictorovich <pavel@codefresh.io>
Copy link
Collaborator

@alexmt alexmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: pashavictorovich <pavel@codefresh.io>
@alexmt alexmt merged commit 0d68194 into argoproj:master Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants