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.

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

fix: list generator elements should be optional #17424

Conversation

blakepettersson
Copy link
Member

@blakepettersson blakepettersson commented Mar 6, 2024

A bug was reported, where an applicationset with an empty elements array, when created with argocd appset create <filename>.yaml gets a ...list.elements: Required value error.

My hypothesis is that when calling the K8s API, golang JSON marshalling mangles the empty elements array to nil, rather than creating an empty array when submitting the POST.

Still need to figure out why the same setup seemingly works fine when the same appset is in an app-of-apps.

Fixes #17450

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
  • The title of the PR states what changed and the related issues number (used for the release note).
  • The title of the PR conforms to the Toolchain Guide
  • I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
  • I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have signed off all my commits as required by DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My build is green (troubleshooting builds).
  • My new feature complies with the feature status guidelines.
  • I have added a brief description of why this PR is necessary and/or what this PR solves.
  • Optional. My organization is added to USERS.md.
  • Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
@blakepettersson blakepettersson marked this pull request as ready for review March 8, 2024 11:33
@blakepettersson blakepettersson requested a review from a team as a code owner March 8, 2024 11:33
@blakepettersson
Copy link
Member Author

So, looking at this further the discrepancy between how this works in an app-of-apps setting and when invoked using argocd appset create seems to be the following:

  1. When invoked from appset create .. the ApplicationSet gets marshaled as an ApplicationSet, without any indirection, causing elements to be nil.
  2. When synced in SyncAppState, the ApplicationSet gets marshaled as an Unstructured, where there is a bunch of weird marshaling logic going on. This causes the marshaled JSON string to have an empty array of elements.

Thoughts?

@blakepettersson
Copy link
Member Author

An example appset, which without this patch would fail with appset create .... If this appset is in an app-of-apps, it'd instead fail at a later stage (since there's no repo with that name, but that's another story).

apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
  annotations:
    argocd.argoproj.io/sync-wave: "0"
  name: some-random-app
  namespace: argocd
spec:
  goTemplate: true
  generators:
    - matrix:
        generators:
          - git:
              repoURL: [email protected]:blakepettersson/some-random-repo.git
              revision: main
              files:
                - path: some-random-file.yaml
          - list:
              elements: []
              elementsYaml: "{{ .clusters | toJson }}"

  syncPolicy: {}
  template:
    metadata:
      name: "some-random-app-{{.env}}"
    spec:
      project: default
      source:
        repoURL: [email protected]:blakepettersson/some-random-repo.git
        path: "some-random-path/{{.env}}"
        targetRevision: main
      destination:
        name: '{{.name}}'
        namespace: 'some-random-app-{{.env}}'
      ignoreDifferences:
        - group: argoproj.io
          jsonPointers:
            - /status
          kind: Application
      syncPolicy:
        automated:
          allowEmpty: true
          prune: true
          selfHeal: true
        syncOptions:
          - CreateNamespace=true
          - ApplyOutOfSyncOnly=true

@jannfis
Copy link
Member

jannfis commented Mar 8, 2024

A bug was reported

Reported where? Care to link it, please?

@blakepettersson
Copy link
Member Author

Reported where?

Internally by an Akuity customer - I cannot link to the reported bug sadly 😞

@jannfis
Copy link
Member

jannfis commented Mar 8, 2024

I cannot link to the reported bug sadly

Care to open a bug on GitHub describing this problem? It would help others in the community who happen to stumble over it.

@blakepettersson
Copy link
Member Author

Sure thing, here's the linked issue: #17450

Copy link
Collaborator

@alexmt alexmt left a comment

Choose a reason for hiding this comment

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

LGTM!

@alexmt alexmt merged commit 51cfd50 into argoproj:master Mar 13, 2024
29 checks passed
@blakepettersson
Copy link
Member Author

/cherry-pick release-2.9

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Mar 13, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
@blakepettersson
Copy link
Member Author

/cherry-pick release-2.10

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Mar 13, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
@blakepettersson
Copy link
Member Author

/cherry-pick release-2.8

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Mar 13, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Mangaal pushed a commit to Mangaal/argo-cd that referenced this pull request Mar 14, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Mangaal <[email protected]>
Mangaal pushed a commit to Mangaal/argo-cd that referenced this pull request Mar 14, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Mangaal <[email protected]>
Mangaal pushed a commit to Mangaal/argo-cd that referenced this pull request Mar 14, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Mangaal <[email protected]>
Mangaal pushed a commit to Mangaal/argo-cd that referenced this pull request Mar 14, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Mangaal <[email protected]>
ishitasequeira pushed a commit that referenced this pull request Mar 28, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
ishitasequeira pushed a commit that referenced this pull request Mar 28, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
lyda pushed a commit to lyda/argo-cd that referenced this pull request Mar 28, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Kevin Lyda <[email protected]>
ishitasequeira added a commit that referenced this pull request May 8, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>
pasha-codefresh added a commit to codefresh-io/argo-cd that referenced this pull request May 14, 2024
* fix: elements should be optional (argoproj#17424) (argoproj#17510)

A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>

* Merge pull request from GHSA-jhwx-mhww-rgc3

* sec: limit helm index max size

Signed-off-by: pashakostohrys <[email protected]>

* sec: limit helm index max size

Signed-off-by: pashakostohrys <[email protected]>

* feat: fix tests and linter

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* Bump version to 2.10.5 (argoproj#17654)

Signed-off-by: GitHub <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix cosign (argoproj#17656)

Signed-off-by: Justin Marquis <[email protected]>

* chore(deps): bump webpack-dev-middleware from 5.3.1 to 5.3.4 in /ui (argoproj#17598) (argoproj#17686)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 5.3.1 to 5.3.4.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v5.3.4/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v5.3.1...v5.3.4)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
...

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

* fix(ui): Fix color generation for pod name in logs viewer. Fixes argoproj#17704 (argoproj#17706) (argoproj#17710)

* Fix color generation for pod name in logs viewer



* Add rebuy to users.md



---------

Signed-off-by: Philipp Trulson <[email protected]>
Co-authored-by: Philipp Trulson <[email protected]>

* fix: fix calculating patch for respect ignore diff feature (argoproj#17693)

* test: unit test for respectIgnoreDifferences bug

Signed-off-by: Jesse Suen <[email protected]>

* test: simplify unit test

Signed-off-by: Jesse Suen <[email protected]>

* fix: fix calculating patch for respect ignore diff feature

Signed-off-by: Alexander Matyushentsev <[email protected]>

---------

Signed-off-by: Jesse Suen <[email protected]>
Signed-off-by: Alexander Matyushentsev <[email protected]>
Co-authored-by: Jesse Suen <[email protected]>

* fix(security): use Chainguard fork of git-urls (argoproj#17732) (argoproj#17735)

Signed-off-by: Michael Crenshaw <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>

* Bump version to 2.10.6 (argoproj#17744)

Signed-off-by: GitHub <[email protected]>
Co-authored-by: alexmt <[email protected]>

* Merge pull request from GHSA-2gvw-w6fj-7m3c

Signed-off-by: pashakostohrys <[email protected]>

* Bump version to 2.10.7 (argoproj#17831)

Signed-off-by: GitHub <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix: docker build fails due to "The repository 'http://deb.debian.org/debian buster-backports Release' does not have a Release file."

Signed-off-by: pashakostohrys <[email protected]>

* fix: codegen and e2e tests in release-2.10 (argoproj#17844)

* fix: codegen and e2e tests

Signed-off-by: pashakostohrys <[email protected]>

* fix: codegen and e2e tests

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* chore: upgrade redis to 7.0.15 (argoproj#17666)

Upgrade to latest stable 7.0.x version to fix CVEs:

CVE-2023-41056

Signed-off-by: Tais P. Hansen <[email protected]>

* Merge pull request from GHSA-9m6p-x4h2-6frq

* feat: limit jq.Run with timeout

Signed-off-by: pashakostohrys <[email protected]>

* feat: ignore normalizer jq execution timeout as env variable

Signed-off-by: pashakostohrys <[email protected]>

* feat: customize error message and add doc section

Signed-off-by: pashakostohrys <[email protected]>

* feat: improve log and change a way how to get variable

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import`s order

Signed-off-by: pashakostohrys <[email protected]>

* chore: rename variable inside sts

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import order

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import`s order

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* Merge pull request from GHSA-9m6p-x4h2-6frq

* feat: limit jq.Run with timeout

Signed-off-by: pashakostohrys <[email protected]>

* feat: ignore normalizer jq execution timeout as env variable

Signed-off-by: pashakostohrys <[email protected]>

* feat: customize error message and add doc section

Signed-off-by: pashakostohrys <[email protected]>

* feat: improve log and change a way how to get variable

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import`s order

Signed-off-by: pashakostohrys <[email protected]>

* chore: rename variable inside sts

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import order

Signed-off-by: pashakostohrys <[email protected]>

* chore: fix import`s order

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* fix: codegen after security fix - 2.10 (argoproj#17985)

* fix: codegen after security fix

Signed-off-by: pashakostohrys <[email protected]>

* fix: codegen after security fix

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* Bump version to 2.10.8 (argoproj#17990)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: pasha-codefresh <[email protected]>

* fix: enable sha256 and sha512 for git ssh (argoproj#18028) (argoproj#18029)

* fix: bumping the knownhosts to v1.2.2 since this contains a fix that allows for sha256 and sha512 algorithms when using git ssh




* chore: remove older version of module from go sum



---------

Signed-off-by: Marc Arndt <[email protected]>
Signed-off-by: Marc Arndt <[email protected]>
Co-authored-by: Marc Arndt <[email protected]>
Co-authored-by: Marc Arndt <[email protected]>

* Bump version to 2.10.9 (argoproj#18033)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: jannfis <[email protected]>

* fix: status.sync.comparedTo should use replace patch strategy (argoproj#18061) (argoproj#18075)

* fix: status.sync.comparedTo should use replace patch strategy



* add e2e tests



---------

Signed-off-by: Alexander Matyushentsev <[email protected]>

* chore: bump go-jose from 3.0.1 to 3.0.3 (argoproj#18102)

Signed-off-by: Jayendra Parsai <[email protected]>
Co-authored-by: Jayendra Parsai <[email protected]>

* docs: fix 404 styling (argoproj#18094) (argoproj#18105)

* docs: fix 404 styling



* hack around custom tag destruction



---------

Signed-off-by: Michael Crenshaw <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>

* chore: update gitops engine for force sync option (argoproj#5882) - 2.10 (argoproj#18123)

Signed-off-by: pashakostohrys <[email protected]>

* fix: Enable Redis authentication in the default installation

* fix: linter issue

* fix: linter issue

---------

Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: pashakostohrys <[email protected]>
Signed-off-by: GitHub <[email protected]>
Signed-off-by: Justin Marquis <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Philipp Trulson <[email protected]>
Signed-off-by: Jesse Suen <[email protected]>
Signed-off-by: Alexander Matyushentsev <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Tais P. Hansen <[email protected]>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Marc Arndt <[email protected]>
Signed-off-by: Marc Arndt <[email protected]>
Signed-off-by: Jayendra Parsai <[email protected]>
Co-authored-by: gcp-cherry-pick-bot[bot] <98988430+gcp-cherry-pick-bot[bot]@users.noreply.github.com>
Co-authored-by: Blake Pettersson <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: pasha-codefresh <[email protected]>
Co-authored-by: Justin Marquis <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Philipp Trulson <[email protected]>
Co-authored-by: Alexander Matyushentsev <[email protected]>
Co-authored-by: Jesse Suen <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Co-authored-by: alexmt <[email protected]>
Co-authored-by: Tais P. Hansen <[email protected]>
Co-authored-by: Marc Arndt <[email protected]>
Co-authored-by: Marc Arndt <[email protected]>
Co-authored-by: jannfis <[email protected]>
Co-authored-by: Jayendra Parsai <[email protected]>
Co-authored-by: Jayendra Parsai <[email protected]>
Co-authored-by: May Zhang <[email protected]>
mkieweg pushed a commit to mkieweg/argo-cd that referenced this pull request Jun 11, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
Hariharasuthan99 pushed a commit to AmadeusITGroup/argo-cd that referenced this pull request Jun 16, 2024
A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>
oleksandr-codefresh added a commit to codefresh-io/argo-cd that referenced this pull request Jun 18, 2024
* docs(webhook): use real cm name instead of placeholder (#17002)

The document says I should registed configMap named argocd-notifications-cm but then uses placeholder in examples.

Signed-off-by: Petr Studeny <[email protected]>

* docs: Update Okta OIDC SSO docs (#13811)

* Update the Okta SSO docs

* fill out the OIDC section with step-by-step instructions on using Okta
  with custom authorization servers
* adjust outdated docs about updating the docs

Signed-off-by: Jonas Courteau <[email protected]>

* Add the Okta version that these docs are written against

Signed-off-by: Jonas Courteau <[email protected]>

---------

Signed-off-by: Jonas Courteau <[email protected]>
Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* Add a description for using contour httpproxy CRD (#14614)

Which allows you to reuse the same hostname.

Co-authored-by: Boris Smidt <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix: ci failures (#17107)

Signed-off-by: Soumya Ghosh Dastidar <[email protected]>

* chore(deps): bump library/golang from 1.21.3 to 1.22.0 in /test/remote (#17111)

Bumps library/golang from 1.21.3 to 1.22.0.

---
updated-dependencies:
- dependency-name: library/golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* chore(deps-dev): bump yarn from 1.22.10 to 1.22.13 in /ui-test (#17092)

Bumps [yarn](https://github.com/yarnpkg/yarn) from 1.22.10 to 1.22.13.
- [Release notes](https://github.com/yarnpkg/yarn/releases)
- [Changelog](https://github.com/yarnpkg/yarn/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yarnpkg/yarn/compare/1.22.10...v1.22.13)

---
updated-dependencies:
- dependency-name: yarn
  dependency-type: direct:development
...

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

* chore(deps): bump library/redis from 7.0.11 to 7.2.4 in /test/container (#16806)

Bumps library/redis from 7.0.11 to 7.2.4.

---
updated-dependencies:
- dependency-name: library/redis
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* chore(deps-dev): bump yarn from 1.22.10 to 1.22.21 in /ui (#17096)

Bumps [yarn](https://github.com/yarnpkg/yarn) from 1.22.10 to 1.22.21.
- [Release notes](https://github.com/yarnpkg/yarn/releases)
- [Changelog](https://github.com/yarnpkg/yarn/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yarnpkg/yarn/compare/1.22.10...v1.22.21)

---
updated-dependencies:
- dependency-name: yarn
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Blake Pettersson <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix the typo (#17116)

* chore(deps): bump library/node from 20.6.1 to 21.6.1 (#17053)

Bumps library/node from 20.6.1 to 21.6.1.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

* chore(deps): bump library/node from 20.7.0 to 21.6.1 (#17065)

Signed-off-by: fengshunli <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix(kustomize): set build dir (#15057) #16229 #16652 (#16653)

* use repo root, not app path

Signed-off-by: Prune <[email protected]>

correct patch

Signed-off-by: Prune <[email protected]>

* use Getwd to find the root path for diff commands

Signed-off-by: Prune <[email protected]>

* set dot a default for argo app commands

Signed-off-by: Prune <[email protected]>

* revert default values

Signed-off-by: Prune <[email protected]>

* patch diff in TestNamespacedResourceDiffing

Signed-off-by: Prune <[email protected]>

* patching some diff and sync

Signed-off-by: Prune <[email protected]>

* patch remaining diff in error

Signed-off-by: Prune <[email protected]>

---------

Signed-off-by: Prune <[email protected]>

* chore(ci): tweak backend filters (#17134)

The existing backend filters get triggered even on frontend-only or
docs-only changes, which should not be the case. The reason for this
seems to be the fact that each filter line is ORed rather than ANDed.
To remedy this, we put all the filters on the same line.

I tried the filter out in a REPL
(https://runkit.com/blakepettersson/65c3daba99653f0008c74eda). This is
a filter using picomatch (the same library `dorny/paths-filter` uses).

Signed-off-by: Blake Pettersson <[email protected]>

* docs: fix error in toolchain setup (#17154)

Signed-off-by: Alexandre Gaudreault <[email protected]>

* chore(dex): 2.37.0 to 2.38.0 (#17157)

Signed-off-by: asingh51 <[email protected]>
Co-authored-by: asingh51 <[email protected]>

* feat: Add support for passing Redis Sentinel username(ACL) and password (#17168)

* Add support for passing Sentinel username and password

Signed-off-by: ShlomiTubul <[email protected]>

* fix align with var naming

Signed-off-by: ShlomiTubul <[email protected]>

* fix align with var naming

Signed-off-by: ShlomiTubul <[email protected]>

---------

Signed-off-by: ShlomiTubul <[email protected]>
Co-authored-by: ShlomiTubul <[email protected]>

* fix: stop initializing deployment informer if dynamic sharding is disabled (#17097)

* fix: stop initializing deployment informer if dynamic sharding is disabled

Signed-off-by: Soumya Ghosh Dastidar <[email protected]>

* feat: updated sharding cache getter func

Signed-off-by: Soumya Ghosh Dastidar <[email protected]>

---------

Signed-off-by: Soumya Ghosh Dastidar <[email protected]>

* feat: query escape function for notifications (#16343)

Signed-off-by: Jan Schumann <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* Update triggers doc to fix typo (#17185)

Signed-off-by: David Grizzanti <[email protected]>

* docs: fixes Template.md targetRevision typo (#17190)

* Template.md targetRevision typo fixed

Signed-off-by: Ajay Chidambaram <[email protected]>

* retrigger checks

Signed-off-by: Ajay Chidambaram <[email protected]>

* sign off

Signed-off-by: chidambaram27 <[email protected]>
Signed-off-by: Ajay Chidambaram <[email protected]>

* sign off

Signed-off-by: Ajay Chidambaram <[email protected]>

---------

Signed-off-by: Ajay Chidambaram <[email protected]>
Signed-off-by: chidambaram27 <[email protected]>

* docs: Private-helm-repo section target added to helm.md (#16697)

* helm-repo

Signed-off-by: Surajyadav <[email protected]>

* Update docs/user-guide/helm.md

Co-authored-by: Blake Pettersson <[email protected]>
Signed-off-by: Suraj yadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>
Signed-off-by: Suraj yadav <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>

* fix: infer correct shard in statefulset setup (#17124, #17016)  (#17167)

* fix: infer correct shard in statefulset setup

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix the case if only a single replica

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: resolving pointer on shard compare

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: add readlock for cluster accessor

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: use defer to protect access of 'shard'

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: revert locking in getclusteraccessor

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: handle nil shard case

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: handle any nil shard value as false

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: handle nil case and fix another missing pointer dereference 

Signed-off-by: Lukas Wöhrl <[email protected]>

* revert

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: added tests and fixed some behaviour bugs

Signed-off-by: Lukas Wöhrl <[email protected]>

* test: add test to validate that Shard value is not overriden

Signed-off-by: Lukas Wöhrl <[email protected]>

* fix: added tests and fixe the case when server is changed inside a secret

Signed-off-by: Lukas Wöhrl <[email protected]>

* tests: add test cases for infering the shard logic

Signed-off-by: Lukas Wöhrl <[email protected]>

---------

Signed-off-by: Lukas Wöhrl <[email protected]>

* feat: wait until resources are deleted #6085 (#16733)

* feat: wait until resources are deleted

Signed-off-by: MichaelMorris <[email protected]>

* Added unit and e2e test

Signed-off-by: MichaelMorris <[email protected]>

---------

Signed-off-by: MichaelMorris <[email protected]>

* fix(controller): add missing workqueue metrics (#16315) (#17013)

* fix(controller): add missing kubernetes metrics

Signed-off-by: Alexandre Gaudreault <[email protected]>

* validate workqueue metrics are present

Signed-off-by: Alexandre Gaudreault <[email protected]>

* use newer metrics registry

Signed-off-by: Alexandre Gaudreault <[email protected]>

* fix duplicated

Signed-off-by: Alexandre Gaudreault <[email protected]>

* init runtime controller in test to have correct metrics

Signed-off-by: Alexandre Gaudreault <[email protected]>

* fix lint error

Signed-off-by: Alexandre Gaudreault <[email protected]>

* update controller-runtime to remove metrics with high cardinality

Signed-off-by: Alexandre Gaudreault <[email protected]>

---------

Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>

* chore(deps): upgrade helm to 3.14.1 (#17213)

* chore(deps): upgrade helm to 3.14.1

Signed-off-by: Alexandre Gaudreault <[email protected]>

* move files to folder......

Signed-off-by: Alexandre Gaudreault <[email protected]>

---------

Signed-off-by: Alexandre Gaudreault <[email protected]>

* docs: Add PayIt to USERS.md (#17215)

* docs: Add PayIt to USERS.md

Signed-off-by: Matt Menzenski <[email protected]>

* docs: Add PayIt to USERS.md

Signed-off-by: Matt Menzenski <[email protected]>

---------

Signed-off-by: Matt Menzenski <[email protected]>

* docs: Add Nextbasket to USERS.md (#17228)

Signed-off-by: NextBasket-Petyo <[email protected]>

* fix: Permission Denied error when calling GetAppDetails API (#17221) (#17229)

Signed-off-by: Keith Chong <[email protected]>

* Count git checkout failures (#15657)

Signed-off-by: Mikołaj Przybysz <[email protected]>
Co-authored-by: Mikołaj Przybysz <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>

* chore(deps): bump library/node from 21.6.1 to 21.6.2 in /ui-test (#17226)

Bumps library/node from 21.6.1 to 21.6.2.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/node from 21.6.1 to 21.6.2 (#17223)

Bumps library/node from 21.6.1 to 21.6.2.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/golang in /test/remote (#17138)

Bumps library/golang from `094e47e` to `ef61a20`.

---
updated-dependencies:
- dependency-name: library/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* feat(grafana-dashboard): Update example dashboard, add AppSet Telemetry (#17232)

Signed-off-by: lukepatrick <[email protected]>

* chore(deps): bump library/node from `6fb1883` to `65998e3` in /ui-test (#17245)

Bumps library/node from `6fb1883` to `65998e3`.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/golang in /test/remote (#17244)

Bumps library/golang from `ef61a20` to `7b297d9`.

---
updated-dependencies:
- dependency-name: library/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* Add `AppName` to the RepoServerAppDetailsQuery for notification-controller (#17233)

Signed-off-by: Dong Wang <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* chore(deps): bump library/node from `50703e6` to `65998e3` (#17243)

Bumps library/node from `50703e6` to `65998e3`.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* docs: Update USERS.md (#17248)

* Update USERS.md

Signed-off-by: itayvolo <[email protected]>

* Update USERS.md

Signed-off-by: itayvolo <[email protected]>

* Update USERS.md

Signed-off-by: itayvolo <[email protected]>

---------

Signed-off-by: itayvolo <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* docs(metrics): add release label to haproxy (#17264)

Add missing `release` label to `argocd-redis-haproxy-metrics`
ServiceMonitor example.

Signed-off-by: Gaston Festari <[email protected]>

* docs: Fix typo in notifications example (#17250)

* Fix typo in notifications example

I'm not too sure what the example should look like, so I'm taking a guess here.

Signed-off-by: Adrian Moisey <[email protected]>

* Update docs/operator-manual/notifications/troubleshooting.md

Co-authored-by: Soumya Ghosh Dastidar <[email protected]>
Signed-off-by: Adrian Moisey <[email protected]>

---------

Signed-off-by: Adrian Moisey <[email protected]>
Co-authored-by: Soumya Ghosh Dastidar <[email protected]>

* Update contributors-quickstart.md (#17266)

Signed-off-by: Dan Garfield <[email protected]>

* reduce unnecessary unmarshal (#17187)

Signed-off-by: Wilson Wang <[email protected]>

* fix typo (#17272)

Signed-off-by: Oscar Wieman <[email protected]>

* docs: Added examples for alternate EKS cluster authentication methods (#17270)

* Added examples for alternate EKS cluster authentication methods
Signed-off-by: Damon Edstrom <[email protected]>

* Update docs/operator-manual/declarative-setup.md

Signed-off-by: Dan Garfield <[email protected]>

Signed-off-by: Dan Garfield <[email protected]>

---------

Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* docs for PR #9791 (#16021)

Signed-off-by: mzain <[email protected]>

* chore(ci): use changed files action (#17180)

`dorny/paths-filter` doesn't seem to handle (multiple) negations well.
Therefore, this PR switches to `tj-actions/changed-files`, since it is
already successfully used in argo-workflows.

Signed-off-by: Blake Pettersson <[email protected]>

* fix(ui): Include application name in status badge (#17126)

* Added application name to badge

Signed-off-by: sshenoy6 <[email protected]>

* Rever svg change

Signed-off-by: sshenoy6 <[email protected]>

* Doc for disabling application name

Signed-off-by: sshenoy6 <[email protected]>

* Flag to not display application name

Signed-off-by: sshenoy6 <[email protected]>

* Added tests

Signed-off-by: sshenoy6 <[email protected]>

* Make no app name the default

Signed-off-by: sshenoy6 <[email protected]>

* Have enable app name as a query parameter

Signed-off-by: sshenoy6 <[email protected]>

* Have enable app name as a query parameter

Signed-off-by: sshenoy6 <[email protected]>

* argocd to original

Signed-off-by: sshenoy6 <[email protected]>

* Update docs/user-guide/status-badge.md

Signed-off-by: Dan Garfield <[email protected]>

Signed-off-by: Dan Garfield <[email protected]>

---------

Signed-off-by: sshenoy6 <[email protected]>
Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: sshenoy6 <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* fix: Update test image ssl/crypto libs (#17303)

Signed-off-by: Carlos Santana <[email protected]>

* chore(deps-dev): bump postcss from 8.2.13 to 8.4.35 in /ui (#17140)

Bumps [postcss](https://github.com/postcss/postcss) from 8.2.13 to 8.4.35.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.2.13...8.4.35)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

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

* chore(deps): bump library/redis in /test/container (#17137)

Bumps library/redis from `cc8b0b8` to `11c3e41`.

---
updated-dependencies:
- dependency-name: library/redis
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump react-helmet and @types/react-helmet in /ui (#11556)

Bumps [react-helmet](https://github.com/nfl/react-helmet) and [@types/react-helmet](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react-helmet). These dependencies needed to be updated together.

Updates `react-helmet` from 5.2.1 to 6.1.0
- [Release notes](https://github.com/nfl/react-helmet/releases)
- [Changelog](https://github.com/nfl/react-helmet/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nfl/react-helmet/commits/6.1.0)

Updates `@types/react-helmet` from 5.0.19 to 6.1.6
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react-helmet)

---
updated-dependencies:
- dependency-name: react-helmet
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: "@types/react-helmet"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

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

* chore(deps): bump bitnami/kubectl in /test/container (#14220)

Bumps bitnami/kubectl from `670fe3f` to `14ab746`.

---
updated-dependencies:
- dependency-name: bitnami/kubectl
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore: add Rocket.Chat to users (#17306)

Signed-off-by: Debdut Chakraborty <[email protected]>

* chore(deps): bump library/ubuntu in /test/container (#13409)

Bumps library/ubuntu from `9a0bdde` to `67211c1`.

---
updated-dependencies:
- dependency-name: library/ubuntu
  dependency-type: direct:production
...

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

* chore(deps): bump library/redis in /test/container (#17309)

Bumps library/redis from `11c3e41` to `e647cfe`.

---
updated-dependencies:
- dependency-name: library/redis
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/registry in /test/container (#13050)

Bumps library/registry from `41f413c` to `b209a41`.

---
updated-dependencies:
- dependency-name: library/registry
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/busybox in /test/e2e/multiarch-container (#14592)

Bumps library/busybox from `2376a0c` to `3fbc632`.

---
updated-dependencies:
- dependency-name: library/busybox
  dependency-type: direct:production
...

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

* fix: use simple python image to build mkdocs (#17313)

* fix: use simple python image to build mkdocs

Signed-off-by: Carlos Santana <[email protected]>

* use python 3.7

Signed-off-by: Carlos Santana <[email protected]>

---------

Signed-off-by: Carlos Santana <[email protected]>

* chore(deps): bump library/registry in /test/container (#17317)

Bumps library/registry from `b209a41` to `f4e1b87`.

---
updated-dependencies:
- dependency-name: library/registry
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* docs: fixed name of environment variable and config map property enabling scm providers for applicationsets (#17326)

Signed-off-by: Juliusz <[email protected]>
Co-authored-by: Juliusz <[email protected]>

* fix(ui): Added dashed line between collapsed sections in Compact-diff (#17173)

* dashed-line-breaker

Signed-off-by: Surajyadav <[email protected]>

* dark-mode

Signed-off-by: Surajyadav <[email protected]>

* dark-mode-text-fix

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>

* fix: multi-source app breaks application parameters UI (#16910) (#17033)

Signed-off-by: Keith Chong <[email protected]>

* chore(deps): upgrade helm to 3.14.2 (#17330)

* chore(deps): upgrade helm to 3.14.2

Signed-off-by: David Bunn <[email protected]>

* Signing commit

Signed-off-by: David Bunn <[email protected]>

---------

Signed-off-by: David Bunn <[email protected]>

* chore(deps): bump library/node from 21.6.1 to 21.6.2 in /test/container (#17316)

Bumps library/node from 21.6.1 to 21.6.2.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* fix: The argocd server api-content-type flag does not allow empty content-type header (#17331)

Signed-off-by: Alexander Matyushentsev <[email protected]>

* fix(ui): The tiles in Applications List are too wide #17220 (#17340)

* fix(ui): The tiles in Applications List are too wide

Signed-off-by: Rafal Pelczar <[email protected]>

* change min width of app tiles

Signed-off-by: Rafal Pelczar <[email protected]>

---------

Signed-off-by: Rafal Pelczar <[email protected]>

* chore(notifications): remove unneeded operations from templates (#17307)

* chore(deps): bump library/ubuntu in /test/container (#13409)

Bumps library/ubuntu from `9a0bdde` to `67211c1`.

---
updated-dependencies:
- dependency-name: library/ubuntu
  dependency-type: direct:production
...

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

* rm unneeded operations from templates

Signed-off-by: Michael Firestone <[email protected]>
Signed-off-by: Michael Firestone <[email protected]>

* rm more unneeded ops

Signed-off-by: Michael Firestone <[email protected]>

---------

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

* feat: allow webhook settings to be referenced by external secret (#16262)

Signed-off-by: Arthur Outhenin-Chalandre <[email protected]>

* feat: Add app data to sharding cache to allow sharding by apps (#17014)

* Adding app list to sharding cache

Signed-off-by: Andrew Lee <[email protected]>

* Add shard by apps test

Signed-off-by: Andrew Lee <[email protected]>

* Fix lint

Signed-off-by: Andrew Lee <[email protected]>

* Add coverage to test

Signed-off-by: Andrew Lee <[email protected]>

* Fix lint

Signed-off-by: Andrew Lee <[email protected]>

* Converted cluster/app accesors to private, add apps-in-any-namespace suport in shardingcache init, added read lock to GetAppDistribution

Signed-off-by: Andrew Lee <[email protected]>

* Fix tests

Signed-off-by: Andrew Lee <[email protected]>

---------

Signed-off-by: Andrew Lee <[email protected]>

* docs: Update USERS.md (#17371)

Hi, I added IABAI as using officially ArgoCD

Signed-off-by: Andrea Sannuto <[email protected]>

* feat: add cli commands to add/delete sourceNamespaces from AppProject (#17337)

* Add cli to add/delete sourceNamespaces

Signed-off-by: Raghavi Shirur <[email protected]>

* update command/comments

Signed-off-by: Raghavi Shirur <[email protected]>

* update command/comments(1)

Signed-off-by: Raghavi Shirur <[email protected]>

* update user-guide docs

Signed-off-by: Raghavi Shirur <[email protected]>

* Retrigger CI pipeline

Signed-off-by: Raghavi Shirur <[email protected]>

* add check for '*' & rename command to remove-source-namespace

Signed-off-by: Raghavi Shirur <[email protected]>

* update command/comments(2)

Signed-off-by: Raghavi Shirur <[email protected]>

* update command/comments(3)

Signed-off-by: Raghavi Shirur <[email protected]>

* Retrigger CI pipeline

Signed-off-by: Raghavi Shirur <[email protected]>

---------

Signed-off-by: Raghavi Shirur <[email protected]>

* feat: add ability to auto label clusters from k8s clusterinfo (#17289)

* feat: add ability to auto label clusters

This gives the ability to automatically label cluster secrets on a cluster-by-cluster basis. If `enableClusterInfoLabels` is set on a cluster secret, the controller will (eventually) label the cluster secret with the current k8s version detected by the cluster info.

This needs documentation, e2e tests, as well as CLI/UI additions.

Signed-off-by: Blake Pettersson <[email protected]>

* refactor: use labels instead of secret data

This is easier to work with, especially in the context where we need
this feature.

Signed-off-by: Blake Pettersson <[email protected]>

* docs: add description on how to use dynamic labeling

Signed-off-by: Blake Pettersson <[email protected]>

---------

Signed-off-by: Blake Pettersson <[email protected]>

* chore(ci): free up disk space for goreleaser (#17373)

Signed-off-by: Michael Crenshaw <[email protected]>

* fix(ui): Dark-mode-enhancements (#17241)

* dark-mode-enhancements

Signed-off-by: Surajyadav <[email protected]>

* monaco

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>
Signed-off-by: Suraj yadav <[email protected]>

* feat: add cli commands to add/remove sources for multi-source applications (#17310)

* Initial commit

Signed-off-by: ishitasequeira <[email protected]>

* add cli commands to add/remove sources for multi-source app

Signed-off-by: ishitasequeira <[email protected]>

* add checks

Signed-off-by: ishitasequeira <[email protected]>

* add docs

Signed-off-by: ishitasequeira <[email protected]>

* refactor code and update tests

Signed-off-by: ishitasequeira <[email protected]>

* add removed additional switch case

Signed-off-by: ishitasequeira <[email protected]>

* fix suggested nits

Signed-off-by: ishitasequeira <[email protected]>

---------

Signed-off-by: ishitasequeira <[email protected]>

* feat: enable users to run commands related to Argo Applications in any namespace (#17360)

* enable --app-namespace falg for application get command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application diff command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application wait command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application rollback command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application patch command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application edit command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application history command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application sync  command

Signed-off-by: Mangaal <[email protected]>

* enable --app-namespace falg for application delete  command

Signed-off-by: Mangaal <[email protected]>

* cli doc generated

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>

* fix: preserve escape codes in repo url of git webhook payload (#17376)

Signed-off-by: Eric Bissonnette <[email protected]>

* fix: disable rate limiting sompletely by default (#17355)

Signed-off-by: Soumya Ghosh Dastidar <[email protected]>

* docs: use service-name instead of service-id (#17389)

* use service-name instead of service-id

Signed-off-by: DongHo Jung <[email protected]>

* trigger CICD

Signed-off-by: DongHo Jung <[email protected]>

---------

Signed-off-by: DongHo Jung <[email protected]>

* feat: Allow Kustomize common labels to not apply to selectors (#17329)

* modify crds

Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>

* cmd opts and test

Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>

* kustomize build and test

Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>

* fix option order and add ancestry to users

Signed-off-by: lets-call-n-walk <[email protected]>

* fix users format

Signed-off-by: lets-call-n-walk <[email protected]>

* generated files

Signed-off-by: lets-call-n-walk <[email protected]>

* set flag value

Signed-off-by: lets-call-n-walk <[email protected]>

* modify crds

Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>

* chore(deps): bump library/registry in /test/container (#17317)

Bumps library/registry from `b209a41` to `f4e1b87`.

---
updated-dependencies:
- dependency-name: library/registry
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>

* generated files

Signed-off-by: lets-call-n-walk <[email protected]>

* add docs

Signed-off-by: lets-call-n-walk <[email protected]>

* fix doc

Signed-off-by: lets-call-n-walk <[email protected]>

* remove debug prints

Signed-off-by: lets-call-n-walk <[email protected]>

* fix autogen docs

Signed-off-by: lets-call-n-walk <[email protected]>

---------

Signed-off-by: Collin <[email protected]>
Signed-off-by: lets-call-n-walk <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Collin Walker <[email protected]>
Co-authored-by: Collin <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(ui): Highlight failing containers in the UI (#17143)

* failing container icon added

Signed-off-by: Surajyadav <[email protected]>

* lint-fix

Signed-off-by: Surajyadav <[email protected]>

* ..

Signed-off-by: Surajyadav <[email protected]>

* tried yarn lint-fix

Signed-off-by: Surajyadav <[email protected]>

* margin

Signed-off-by: Surajyadav <[email protected]>

* running

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>

* fix(ui): application-summary-help-icon fix (#17385)

* Update application-summary.tsx

revision-history icon fix

Signed-off-by: Suraj yadav <[email protected]>

* icons

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Suraj yadav <[email protected]>
Signed-off-by: Surajyadav <[email protected]>

* chore: add more unit tests around useDiffCache function (#17404)

* chore: add more unit tests around useDiffCache function

Signed-off-by: Leonardo Luz Almeida <[email protected]>

* fix doc

Signed-off-by: Leonardo Luz Almeida <[email protected]>

---------

Signed-off-by: Leonardo Luz Almeida <[email protected]>

* fix(grpcproxy): parse headers contain colon (#13274) (#14294)

* fix(grpcproxy): parse headers contain colon

Signed-off-by: bogay <[email protected]>

* test(apiclient): headers in wrong format

Signed-off-by: bogay <[email protected]>

---------

Signed-off-by: bogay <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>

* docs: Fix some awkward phrasing in `core.md` (#17412)

Signed-off-by: Nate Douglas <[email protected]>

* docs (aks cluster): update workloadidentity documentation (#17401)

Signed-off-by: duncan485 <[email protected]>

* docs: remove repetitive words (#17430)

Signed-off-by: hishope <[email protected]>

* feat(cli): enable --app-namespace flag for argocd app subcommand (#17437)

* add --app-namespace flag for set/unset command

Signed-off-by: Mangaal <[email protected]>

* add --app-namespace flag for add-source/remove-source command

Signed-off-by: Mangaal <[email protected]>

* fix bug, handle array out of-bound when --source-index=len(source)

Signed-off-by: Mangaal <[email protected]>

* add documentation

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>

* docs: Clarify for `valueFiles` behaviour with `path` field (#17431)

This commit adds clarification by explaining that `valueFiles` must be a relative path to the root of sources, even if the `path` field is set.

Signed-off-by: Takahiro Suzuki <[email protected]>
Signed-off-by: tkasuz <[email protected]>

* docs: Small edits to `docs/user-guide/application-set.md` (#17434)

* Update application-set.md

Signed-off-by: Nate Douglas <[email protected]>

* Make example code consistent with examples elsewhere.

Signed-off-by: Nathan Douglas <[email protected]>

---------

Signed-off-by: Nate Douglas <[email protected]>
Signed-off-by: Nathan Douglas <[email protected]>

* fix: add retry condition with kube-apiserver sent GOAWAY (#17422)

Signed-off-by: penglongli <[email protected]>

* fix(ui): align resource nodegroup (#17427)

Signed-off-by: Caio Paiva <[email protected]>

* docs: proposal to implement sync timeout and termination settings (#16630)

Signed-off-by: Alexander Matyushentsev <[email protected]>

* feat: Allow mkdocs to livereload when using docker (#17383)

* feat: Allow mkdocs to livereload when using docker

Signed-off-by: Carlos Santana <[email protected]>

* update the docs

Signed-off-by: Carlos Santana <[email protected]>

---------

Signed-off-by: Carlos Santana <[email protected]>

* chore(deps): bump github.com/go-jose/go-jose/v3 from 3.0.1 to 3.0.3 (#17442)

Bumps [github.com/go-jose/go-jose/v3](https://github.com/go-jose/go-jose) from 3.0.1 to 3.0.3.
- [Release notes](https://github.com/go-jose/go-jose/releases)
- [Changelog](https://github.com/go-jose/go-jose/blob/v3.0.3/CHANGELOG.md)
- [Commits](https://github.com/go-jose/go-jose/compare/v3.0.1...v3.0.3)

---
updated-dependencies:
- dependency-name: github.com/go-jose/go-jose/v3
  dependency-type: direct:production
...

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

* chore: Fix Helm Installation Breaking on Mac (#17426)

* fix helm installation to work with mac

Signed-off-by: lets-call-n-walk <[email protected]>

* fix checksums

Signed-off-by: lets-call-n-walk <[email protected]>

* change install filename and makefile

Signed-off-by: lets-call-n-walk <[email protected]>

* change name to just helm - fix dockerfile

Signed-off-by: lets-call-n-walk <[email protected]>

---------

Signed-off-by: lets-call-n-walk <[email protected]>

* docs: re-worded updated a link in release-process-and-cadance.md (#17438)

Signed-off-by: AlbinB97 <[email protected]>

* Fixed `project` parameter docs for Gitlab pull request generator (#17429)

Signed-off-by: Nguyen Thai <[email protected]>

* chore: update cosign and version (#17441)

* chore: update cosign and version

Signed-off-by: Justin Marquis <[email protected]>

* fix typo

Signed-off-by: Justin Marquis <[email protected]>

---------

Signed-off-by: Justin Marquis <[email protected]>

* chore: update slsa3 generate (#17451)

Signed-off-by: Justin Marquis <[email protected]>

* docs: dex google oidc: add note on dex connector type (#17453)

Signed-off-by: Aiman Ismail <[email protected]>

* docs: Update site.md (#17454)

erros -> errors

Signed-off-by: Ikko Eltociear Ashimine <[email protected]>

* add Oncourse Home (#17457)

* chore(deps): bump library/busybox in /test/e2e/multiarch-container (#17445)

Bumps library/busybox from `3fbc632` to `650fd57`.

---
updated-dependencies:
- dependency-name: library/busybox
  dependency-type: direct:production
...

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

* chore(deps): bump library/node from 21.6.2 to 21.7.0 in /ui-test (#17444)

Bumps library/node from 21.6.2 to 21.7.0.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* make CGO_ENABLED configurable (#17462)

Signed-off-by: Mark McCormick <[email protected]>

* chore(deps): bump library/ubuntu in /test/container (#17414)

Bumps library/ubuntu from `f9d633f` to `77906da`.

---
updated-dependencies:
- dependency-name: library/ubuntu
  dependency-type: direct:production
...

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

* chore: remove repetitive words (#17464)

Signed-off-by: avoidalone <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* feat: add option `manual` to the `--sync-policy` flag of the `app create` command (#17459)

* feat: add sync policy option 'manual' for app create command

Signed-off-by: Anirudh Sudhir <[email protected]>

* chore: Update tests to reflect sync policy option changes

Signed-off-by: Anirudh Sudhir <[email protected]>

---------

Signed-off-by: Anirudh Sudhir <[email protected]>

* [Bot] docs: Update Snyk reports (#17458)

Signed-off-by: CI <[email protected]>
Co-authored-by: CI <[email protected]>

* docs: fix kustomize example resources url (#17468)

Signed-off-by: Jonas Bakken <[email protected]>

* chore(deps): bump library/node from 21.6.2 to 21.7.1 in /test/container (#17475)

Bumps library/node from 21.6.2 to 21.7.1.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* chore(deps): bump library/golang in /test/container (#17415)

Bumps library/golang from 1.21.3 to 1.22.1.

---
updated-dependencies:
- dependency-name: library/golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* fix(notifications): Helm.GetParameterValueByName should take helm.parametes first (#17472)

* fix: Helm.GetParameterValueByName should take helm.parametes first

Signed-off-by: pashakostohrys <[email protected]>

* fix linters

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>

* fix: registry argument to be only the host instead full URL (#17381)

Signed-off-by: Pablo Aguilar <[email protected]>

* chore: Fix containerized toolchain (#17480)

Signed-off-by: jannfis <[email protected]>

* chore(deps): bump docker/build-push-action from 5.1.0 to 5.2.0 (#17463)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5.1.0 to 5.2.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](https://github.com/docker/build-push-action/compare/4a13e500e55cf31b7a5d59a38ab2040ab0f42f56...af5a7ed5ba88268d5278f7203fb52cd833f66d6e)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* docs: remove `configManagementPlugins` from argocd-cm.yaml (#17486)

That field is no longer supported.

Signed-off-by: Michael Crenshaw <[email protected]>

* chore: Support running and testing locally using podman instead of docker (#17481)

* chore: Support rootless podman for run/test

Signed-off-by: jannfis <[email protected]>

* Note DOCKER env in docs

Signed-off-by: jannfis <[email protected]>

---------

Signed-off-by: jannfis <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>

* fix(appset): keep reconciling even when params error occurred (#17062)

* fix(appset): keep reconcile even when params error occurred

Signed-off-by: Or Koren <[email protected]>

* requeue on generator rendering error

Signed-off-by: Or Koren <[email protected]>

* test ignoring partial rendering errors

Signed-off-by: Or Koren <[email protected]>

* e2e test create app with param error

Signed-off-by: Or Koren <[email protected]>

---------

Signed-off-by: Or Koren <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>

* docs(EKS): Fix ArgoCD management role AssumeRole policy for IRSA (#17455)

Signed-off-by: Xavier Krantz <[email protected]>

* chore(deps): bump library/node from 21.7.0 to 21.7.1 in /ui-test (#17498)

Bumps library/node from 21.7.0 to 21.7.1.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/golang in /test/container (#17495)

Bumps library/golang from `34ce21a` to `0b55ab8`.

---
updated-dependencies:
- dependency-name: library/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/redis in /test/container (#17496)

Bumps library/redis from `e647cfe` to `7dd7070`.

---
updated-dependencies:
- dependency-name: library/redis
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/node in /test/container (#17494)

Bumps library/node from `f358dfc` to `b9ccc4a`.

---
updated-dependencies:
- dependency-name: library/node
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* Make evergreen (#17507)

This example uses an old revision `v1.0.1` instead it should just use stable.

Signed-off-by: Dan Garfield <[email protected]>

* docs: added all available fields for applicationset.yaml #16095 (#16104)

* adding all available filds for generators

Signed-off-by: Harshvir Potpose <[email protected]>

* add remaining fields in applicationset.yml

Signed-off-by: Harshvir Potpose <[email protected]>

* Update docs/operator-manual/applicationset.yaml

Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: Harshvir Potpose <[email protected]>

* Update docs/operator-manual/applicationset.yaml

Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: Harshvir Potpose <[email protected]>

* Update docs/operator-manual/applicationset.yaml

Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: Harshvir Potpose <[email protected]>

* Update docs/operator-manual/applicationset.yaml

Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: Harshvir Potpose <[email protected]>

* fix

Signed-off-by: Harshvir Potpose <[email protected]>

---------

Signed-off-by: Harshvir Potpose <[email protected]>
Signed-off-by: Harshvir Potpose <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>

* docs: Add note in installation step 1 with argocd cli (#8030)

* Add note in installation stesps

Added notes in installation step 1: 
kubectl config set-context --current --namespace=argocd

* Updated doc

* Update docs/getting_started.md

Signed-off-by: Dan Garfield <[email protected]>

---------

Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* Add documentation for google transitive groups (#9487)

Signed-off-by: Trung <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>

* fix: elements should be optional (#17424)

A bug was reported, where an applicationset with an empty elements
array, when created with `argocd appset create <filename>.yaml` gets a
`...list.elements: Required value` error.

My hypothesis is that when calling the K8s API, golang JSON marshalling
mangles the empty `elements` array to `nil`, rather than creating an
empty array when submitting the `POST`.

Still need to figure out why the same setup seemingly works fine when
the same appset is in an app-of-apps.

Signed-off-by: Blake Pettersson <[email protected]>

* Merge pull request from GHSA-jwv5-8mqv-g387

* fix: Validate external URLs for applicatins

Signed-off-by: Ry0taK <[email protected]>

* fix(ui): remove invalid external-link

Signed-off-by: Alexandre Gaudreault <[email protected]>

* linting

Signed-off-by: Alexandre Gaudreault <[email protected]>

---------

Signed-off-by: Ry0taK <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Co-authored-by: Alexandre Gaudreault <[email protected]>

* Merge pull request from GHSA-g623-jcgg-mhmm

Signed-off-by: Michael Crenshaw <[email protected]>

* feat: Add support to enable FullTimeStamp in logging (#15127)

* Add support to enable FullTimeStamp in logging
Signed-off-by: skalinov <[email protected]>

* fix: Fix go linter file exist issue
Signed-off-by: skalinov <[email protected]>

* fix: Remove --skip-pkg-cache
Signed-off-by: skalinov <[email protected]>

* Update util/log/logrus_test.go

Use custom set env for prevent linter to be failed

Signed-off-by: pasha-codefresh <[email protected]>

* Update common/common.go

Signed-off-by: Dan Garfield <[email protected]>

* Update util/log/logrus_test.go

Signed-off-by: pasha-codefresh <[email protected]>

* Update util/log/logrus_test.go

remove os import

Signed-off-by: pasha-codefresh <[email protected]>

* Update util/log/logrus_test.go

sort dependencies

Signed-off-by: pasha-codefresh <[email protected]>

* fix formatting

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pasha-codefresh <[email protected]>
Signed-off-by: Dan Garfield <[email protected]>
Signed-off-by: pashakostohrys <[email protected]>
Co-authored-by: skalinov <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* docs: add attestation of SBOM to release assets in signed-release-assets.md (#17520)

Closes https://github.com/argoproj/argo-cd/issues/17200.

Signed-off-by: Andreas Hunkeler <[email protected]>

* fix: Argo CD unnecessary enforce sequential helm manifest generation for one chart (#17518)

* fix: Argo CD unnecessary enforce sequential helm manifest generation for one chart

Signed-off-by: Alexander Matyushentsev <[email protected]>

* Update docs/operator-manual/high_availability.md

Signed-off-by: Dan Garfield <[email protected]>

---------

Signed-off-by: Alexander Matyushentsev <[email protected]>
Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* chore(deps): bump actions/checkout (#17493)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3df4ab11eba7bda6032a0b82a6bb43b11571feac to 8410ad0602e1e429cee44a835ae9f77f654a6694.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/3df4ab11eba7bda6032a0b82a6bb43b11571feac...8410ad0602e1e429cee44a835ae9f77f654a6694)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
...

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

* chore: Add error messages to return statements in BlockingDial function (#17521)

* Add error messages to return statements in BlockingDial function

Signed-off-by: danqixu <[email protected]>

* amend error wrapping from %s to %w

Signed-off-by: danqixu <[email protected]>

---------

Signed-off-by: danqixu <[email protected]>

* chore(deps): upgrade helm to 3.14.3 (#17531)

* chore(deps): upgrade helm to 3.14.3

Signed-off-by: David Bunn <[email protected]>

* chore(deps): upgrade helm to 3.14.3

Signed-off-by: David Bunn <[email protected]>

---------

Signed-off-by: David Bunn <[email protected]>

* chore(deps): bump docker/build-push-action from 5.2.0 to 5.3.0 (#17537)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5.2.0 to 5.3.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](https://github.com/docker/build-push-action/compare/af5a7ed5ba88268d5278f7203fb52cd833f66d6e...2cdde995de11925a030ce8070c3d77a52ffcf1c0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* fix(ui): add confirmation box in resource summary delete action (#17485)

Signed-off-by: ashutosh16 <[email protected]>

* docs: Update the status of the feature, app in any namespace, from beta to stable (#17529)

* Update the status of the feature, app in any namespace, from beta to stable

Signed-off-by: Mangaal <[email protected]>

* adding warning and removing **Current feature state**

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>

* Merge pull request from GHSA-2vgg-9h6w-m454

* feat: pick random user and exclude admin user and current user from deletion candidates

Signed-off-by: pashakostohrys <[email protected]>

* feat: increase default max cache size

Signed-off-by: pashakostohrys <[email protected]>

* add nil protection

Signed-off-by: pashakostohrys <[email protected]>

* Update util/session/sessionmanager.go

Signed-off-by: Dan Garfield <[email protected]>

Signed-off-by: Dan Garfield <[email protected]>

* chore: fix linter issues

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>
Signed-off-by: Dan Garfield <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>

* Merge pull request from GHSA-6v85-wr92-q4p7

* fix: Fix concurrency issue in session manager

Signed-off-by: jannfis <[email protected]>

* Add note that modification to the map must be done in a thread safe manner

* chore: fix linter issues

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: jannfis <[email protected]>
Signed-off-by: pashakostohrys <[email protected]>
Co-authored-by: Dan Garfield <[email protected]>
Co-authored-by: pashakostohrys <[email protected]>

* chore(deps-dev): bump postcss from 8.4.35 to 8.4.36 in /ui (#17555)

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.35 to 8.4.36.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.4.35...8.4.36)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

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

* chore(deps): bump library/registry in /test/container (#17554)

Bumps library/registry from `f4e1b87` to `fb9c9ae`.

---
updated-dependencies:
- dependency-name: library/registry
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

* docs: mention that the argocd diff command does not include secrets (#15950)

* doc(cli): update argocd_app_diff

Adding the fact that secrets are ignored by the diff (not really stated in the doc so far)

Signed-off-by: ario0 <[email protected]>

* doc: remove whitespace

Signed-off-by: ario0 <[email protected]>

---------

Signed-off-by: ario0 <[email protected]>
Signed-off-by: Alexis Renard <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>

* feat: add cli commands to create/set/unset/edit sources for multi-source application (#17425)

* "feat:add cli commands to create/set/unset/edit sources for multi-source app"

Signed-off-by: ishitasequeira <[email protected]>

* fixed the ci failure

Signed-off-by: ishitasequeira <[email protected]>

* update commands

Signed-off-by: ishitasequeira <[email protected]>

* error out if source-index not specified for multi-source applications

Signed-off-by: ishitasequeira <[email protected]>

* fixed the ci failure

Signed-off-by: ishitasequeira <[email protected]>

* fix tests

Signed-off-by: ishitasequeira <[email protected]>

* set 0 as default source index for app create

Signed-off-by: ishitasequeira <[email protected]>

* add index to ParameterOverrides function

Signed-off-by: ishitasequeira <[email protected]>

* do not allow overrides for applications with multiple sources

Signed-off-by: ishitasequeira <[email protected]>

* update tests

Signed-off-by: ishitasequeira <[email protected]>

* remove create with override example

Signed-off-by: ishitasequeira <[email protected]>

* address comments

Signed-off-by: ishitasequeira <[email protected]>

* update tests

Signed-off-by: ishitasequeira <[email protected]>

* update examples in docs

Signed-off-by: ishitasequeira <[email protected]>

* update logs

Signed-off-by: ishitasequeira <[email protected]>

* Add test and update docs

Signed-off-by: ishitasequeira <[email protected]>

---------

Signed-off-by: ishitasequeira <[email protected]>

* feat(UI): Added link for docs to add clusters  (#17395)

* cluster-add-link

Signed-off-by: Surajyadav <[email protected]>

* docs

Signed-off-by: Surajyadav <[email protected]>

* docs-panel

Signed-off-by: Surajyadav <[email protected]>

* added

Signed-off-by: Surajyadav <[email protected]>

* fixed

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>

* chore(deps-dev): bump postcss from 8.4.36 to 8.4.37 in /ui (#17573)

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.36 to 8.4.37.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.4.36...8.4.37)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

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

* chore: Fix linter issue with session manager (#17597)

Signed-off-by: jannfis <[email protected]>

* chore(deps): bump slsa-framework/slsa-github-generator (#17593)

Bumps [slsa-framework/slsa-github-generator](https://github.com/slsa-framework/slsa-github-generator) from 1.9.0 to 1.10.0.
- [Release notes](https://github.com/slsa-framework/slsa-github-generator/releases)
- [Changelog](https://github.com/slsa-framework/slsa-github-generator/blob/main/CHANGELOG.md)
- [Commits](https://github.com/slsa-framework/slsa-github-generator/compare/v1.9.0...v1.10.0)

---
updated-dependencies:
- dependency-name: slsa-framework/slsa-github-generator
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

* chore(deps): bump webpack-dev-middleware from 5.3.1 to 5.3.4 in /ui (#17598)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 5.3.1 to 5.3.4.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v5.3.4/CHANGELOG.md)
- [Commits](https://github.com/webpack/webpack-dev-middleware/compare/v5.3.1...v5.3.4)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
...

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

* chore(deps-dev): bump postcss from 8.4.37 to 8.4.38 in /ui (#17582)

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.37 to 8.4.38.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.c…
@blakepettersson blakepettersson deleted the fix/list-generator-elements-needs-to-be-optional branch September 4, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Argo CD CLI prevents appset from being created when using list generator with empty array
3 participants