Skip to content

Conversation

lucacome
Copy link

@lucacome lucacome commented Apr 26, 2023

Updates the name to use nginx-ingress.controller.fullname consistently for GlobalConfiguration.

Fixes #3753

@lucacome lucacome requested a review from a team as a code owner April 26, 2023 00:51
@lucacome lucacome self-assigned this Apr 26, 2023
@github-actions github-actions bot added bug An issue reporting a potential bug helm_chart Pull requests that update the Helm Chart labels Apr 26, 2023
@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Merging #3815 (f608f4d) into main (38ae409) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3815      +/-   ##
==========================================
+ Coverage   52.37%   52.40%   +0.02%     
==========================================
  Files          59       59              
  Lines       16898    16898              
==========================================
+ Hits         8851     8855       +4     
+ Misses       7750     7748       -2     
+ Partials      297      295       -2     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lucacome lucacome added this to the v3.1.1 milestone Apr 26, 2023
@lucacome lucacome merged commit 45586c0 into main Apr 26, 2023
@lucacome lucacome deleted the fix/gc-helm-chart branch April 26, 2023 20:28
lucacome added a commit that referenced this pull request May 4, 2023
lucacome added a commit that referenced this pull request May 4, 2023
@lucacome lucacome removed the bug An issue reporting a potential bug label Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm_chart Pull requests that update the Helm Chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Helm GlobalConfiguration Argument
3 participants