Skip to content

chore: v1: add --transform to v1 filter command.#7319

Merged
tejal29 merged 1 commit intoGoogleContainerTools:v1from
tejal29:add_transform_flag
Apr 21, 2022
Merged

chore: v1: add --transform to v1 filter command.#7319
tejal29 merged 1 commit intoGoogleContainerTools:v1from
tejal29:add_transform_flag

Conversation

@tejal29
Copy link
Copy Markdown
Contributor

@tejal29 tejal29 commented Apr 15, 2022

This is related to #6683 but for V1 branch

For v1 branch

  1. Adding --tranform flag to the filter command. This command will be used in helmv3.1Deployer to use helm post renderer functionality for performing image replacements.

@tejal29 tejal29 requested a review from a team as a code owner April 15, 2022 22:33
@tejal29 tejal29 requested review from gsquared94 and removed request for a team April 15, 2022 22:33
@tejal29 tejal29 force-pushed the add_transform_flag branch from 3724af8 to 931a095 Compare April 15, 2022 22:35
@tejal29 tejal29 changed the title feat: add tests chore: v1: add --transform to v1 filter command. Apr 15, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 15, 2022

Codecov Report

❗ No coverage uploaded for pull request base (v1@4ad93bb). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             v1    #7319   +/-   ##
=====================================
  Coverage      ?   68.50%           
=====================================
  Files         ?      560           
  Lines         ?    26535           
  Branches      ?        0           
=====================================
  Hits          ?    18178           
  Misses        ?     7096           
  Partials      ?     1261           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@tejal29 tejal29 force-pushed the add_transform_flag branch from 931a095 to 398d210 Compare April 20, 2022 02:03
WithFlags([]*Flag{
{Value: &renderFromBuildOutputFile, Name: "build-artifacts", Shorthand: "a", Usage: "File containing build result from a previous 'skaffold build --file-output'"},
{Value: &debuggingFilters, Name: "debugging", DefValue: false, Usage: `Apply debug transforms similar to "skaffold debug"`, IsEnum: true},
{Value: &transform, Name: "transform", DefValue: false, Usage: `(experimental) Replace images in manifest and set labels`, IsEnum: true},
Copy link
Copy Markdown
Contributor

@gsquared94 gsquared94 Apr 20, 2022

Choose a reason for hiding this comment

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

is there a reason for marking this flag experimental vs just hidden?

@tejal29 tejal29 merged commit 848305b into GoogleContainerTools:v1 Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants