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.

Sign up for GitHub

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

Jump to bottom

fix(argo-rollouts): Correct outdated URL for ingress #3093

Merged
merged 1 commit into from
Dec 30, 2024
Merged

fix(argo-rollouts): Correct outdated URL for ingress #3093

merged 1 commit into from
Dec 30, 2024

Conversation

yu-croco
Copy link
Collaborator

@yu-croco yu-croco commented Dec 30, 2024 •
edited
Loading

Aligned to #3079

Checklist:

  • I have bumped the chart version according to versioning
  • I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

@yu-croco
fix(argo-rollouts): Correct outdated URL for ingress
cdfa9ca 
Signed-off-by: yu-croco <yu.croco@gmail.com>
@yu-croco yu-croco marked this pull request as ready for review December 30, 2024 14:37
@yu-croco yu-croco requested a review from jmeridth as a code owner December 30, 2024 14:37
@jmeridth jmeridth merged commit 0877c58 into argoproj:main Dec 30, 2024
14 checks passed
@yu-croco yu-croco mentioned this pull request Dec 31, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbevc1 mbevc1 mbevc1 approved these changes

@jmeridth jmeridth jmeridth approved these changes

@mkilchhofer mkilchhofer Awaiting requested review from mkilchhofer

@tico24 tico24 Awaiting requested review from tico24

Assignees
No one assigned
Labels
argo-rollouts size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yu-croco @jmeridth @mbevc1