Skip to content

Conversation

@tcaenen
Copy link
Contributor

@tcaenen tcaenen commented Jul 2, 2025

Adds support for overriding the namespace used in the Argo Rollouts Helm chart via a new value: namespaceOverride.

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.
  • I have created a separate pull request for each chart according to pull requests
  • My build is green (troubleshooting builds).

Co-authored-by: Aikawa <[email protected]>
Signed-off-by: tcaenen <[email protected]>
yu-croco
yu-croco previously approved these changes Jul 5, 2025
Copy link
Collaborator

@yu-croco yu-croco left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution. LGTM.

@yu-croco yu-croco requested review from mbevc1, mkilchhofer and tico24 July 7, 2025 09:17
Copy link
Member

@mkilchhofer mkilchhofer left a comment

Choose a reason for hiding this comment

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

Almost LGTM. Found 2 search/replace issues

Signed-off-by: Marco Maurer (-Kilchhofer) <[email protected]>
Signed-off-by: Marco Maurer (-Kilchhofer) <[email protected]>
Copy link
Member

@mkilchhofer mkilchhofer left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @tcaenen

@mkilchhofer mkilchhofer requested a review from yu-croco July 8, 2025 21:15
@yu-croco yu-croco merged commit 85e7577 into argoproj:main Jul 9, 2025
7 checks passed
yaringol pushed a commit to yaringol/argo-helm that referenced this pull request Jul 12, 2025
mkilchhofer added a commit that referenced this pull request Sep 16, 2025
…er (#3365)

* chore(deps): update renovatebot/github-action action to v43 (#3363)

Signed-off-by: Yarin Golzar <[email protected]>

* added VerticalPodAutoscaler support for application controller #3232

Signed-off-by: Yarin Golzar <[email protected]>

* removed trailing spaces

Signed-off-by: Yarin Golzar <[email protected]>

* chore(deps): update renovatebot/github-action action to v43.0.1 (#3366)

Signed-off-by: Yarin Golzar <[email protected]>

* fix(argo-cd): do not use hardcoded commit server URL (#3367)

* fix(argo-cd): do not use hardcoded commit server URL

This commit adds the required template logic to the Argo CD Helm chart
to use the proper, release name dependent URL for the commit server. The current
implementation uses the default hardcoded URL `argocd-commit-server:8086`
from <https://github.com/argoproj/argo-cd/blob/v3.0.9/common/common.go#L31>.

Signed-off-by: Alexej Disterhoft <[email protected]>

* chore(argo-cd): update version to 8.1.2

Signed-off-by: Alexej Disterhoft <[email protected]>

* docs(argo-cd): add commit server service port and port name to README

Signed-off-by: Alexej Disterhoft <[email protected]>

---------

Signed-off-by: Alexej Disterhoft <[email protected]>

* chore(deps): bump github/codeql-action from 3.29.0 to 3.29.1 in the dependencies group (#3370)

chore(deps): bump github/codeql-action in the dependencies group

Bumps the dependencies group with 1 update: [github/codeql-action](https://github.com/github/codeql-action).

Updates `github/codeql-action` from 3.29.0 to 3.29.1
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@ce28f5b...39edc49)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 3.29.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Yarin Golzar <[email protected]>

* chore(deps): update renovatebot/github-action action to v43.0.2 (#3371)

Signed-off-by: Yarin Golzar <[email protected]>

* chore(deps): update ghcr.io/renovatebot/renovate docker tag to v41 (#3373)

Signed-off-by: argoproj-renovate[bot] <161757507+argoproj-renovate[bot]@users.noreply.github.com>
Co-authored-by: argoproj-renovate[bot] <161757507+argoproj-renovate[bot]@users.noreply.github.com>
Signed-off-by: Yarin Golzar <[email protected]>

* chore(deps): bump github/codeql-action from 3.29.1 to 3.29.2 in the dependencies group (#3378)

chore(deps): bump github/codeql-action in the dependencies group

Bumps the dependencies group with 1 update: [github/codeql-action](https://github.com/github/codeql-action).

Updates `github/codeql-action` from 3.29.1 to 3.29.2
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@39edc49...181d5ee)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 3.29.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Yarin Golzar <[email protected]>

* feat(argo-rollouts): Support ability to set .Values.namespaceOverride (#3375)

Signed-off-by: Yarin Golzar <[email protected]>

* updated docs to match vpa default values settings

Signed-off-by: Yarin Golzar <[email protected]>

* chore(argo-cd): Update dependency argoproj/argo-cd to v3.0.11 (#3380)

---------

Signed-off-by: Yarin Golzar <[email protected]>
Signed-off-by: Alexej Disterhoft <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: argoproj-renovate[bot] <161757507+argoproj-renovate[bot]@users.noreply.github.com>
Signed-off-by: Yarin Golzar <[email protected]>
Signed-off-by: Marco Maurer (-Kilchhofer) <[email protected]>
Co-authored-by: argoproj-renovate[bot] <161757507+argoproj-renovate[bot]@users.noreply.github.com>
Co-authored-by: Yarin Golzar <[email protected]>
Co-authored-by: Alexej Disterhoft <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: tcaenen <[email protected]>
Co-authored-by: Marco Maurer (-Kilchhofer) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants