flagger icon indicating copy to clipboard operation
flagger copied to clipboard

add support for deploymentLabels

Open quintonm opened this issue 1 year ago • 2 comments

This PR adds support for configuring additional labels for the deployment using deploymentLabels in values.yaml.

quintonm avatar Sep 24 '24 16:09 quintonm

Could this be used to address the issues raised in #1312?

KrylixZA avatar Oct 08 '24 07:10 KrylixZA

@KrylixZA i'm investigating that. i'd say if that needs to be addressed, we shall do so in a different PR, since they're completely different things.

aryan9600 avatar Oct 08 '24 19:10 aryan9600

@aryan9600 README has been updated. Should I make a similar update to docs/gitbook/install/flagger-install-on-kubernetes.md?

quintonm avatar Nov 22 '24 01:11 quintonm

Sure, that'd be great. Could you also specify the podLabels and deploymentLabels values in the table in charts/flagger/README.md?

aryan9600 avatar Nov 23 '24 08:11 aryan9600

Sure, that'd be great. Could you also specify the podLabels and deploymentLabels values in the table in charts/flagger/README.md?

done

quintonm avatar Nov 28 '24 06:11 quintonm

@aryan9600 Any other changes needed?

quintonm avatar Dec 03 '24 06:12 quintonm

@aryan9600 @stefanprodan - Just following up on this PR. Are any other changes needed? Thanks for taking the time to review!

quintonm avatar Dec 18 '24 17:12 quintonm

@aryan9600 - How do I trigger a build? It looks like one is needed before the PR can be merged.

quintonm avatar Jan 12 '25 03:01 quintonm

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 39.30%. Comparing base (febc327) to head (03d4acc). Report is 24 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1707      +/-   ##
==========================================
+ Coverage   39.27%   39.30%   +0.03%     
==========================================
  Files         284      284              
  Lines       22379    22391      +12     
==========================================
+ Hits         8789     8801      +12     
  Misses      12643    12643              
  Partials      947      947              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Jan 13 '25 06:01 codecov-commenter

@aryan9600 done

quintonm avatar Jan 13 '25 13:01 quintonm

@aryan9600 Is there anything else that I need to do before build-flagger action can be executed?

quintonm avatar Jan 20 '25 12:01 quintonm

@stefanprodan Can you help me get my PR merged?

quintonm avatar Jan 25 '25 19:01 quintonm