Sync with argoproj/argo-cd #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* chore(action): add newlines at eof Signed-off-by: Josh Soref <[email protected]> * chore(action): fix whitespace indentation Signed-off-by: Josh Soref <[email protected]> * chore(action): use local annotations Signed-off-by: Josh Soref <[email protected]> --------- Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
…rting API-Server (#15574) * feat: auto configure extensions Signed-off-by: Leonardo Luz Almeida <[email protected]> * feat: auto-reload extension configs without restarting api-server Signed-off-by: Leonardo Luz Almeida <[email protected]> * clean unused gorilla mux Signed-off-by: Leonardo Luz Almeida <[email protected]> * update docs Signed-off-by: Leonardo Luz Almeida <[email protected]> * Address review comments Signed-off-by: Leonardo Luz Almeida <[email protected]> * Add more test cases Signed-off-by: Leonardo Luz Almeida <[email protected]> * refactoring to reduce unnecessary function Signed-off-by: Leonardo Luz Almeida <[email protected]> * Add log Signed-off-by: Leonardo Luz Almeida <[email protected]> * fix bugs found during manual tests Signed-off-by: Leonardo Luz Almeida <[email protected]> --------- Signed-off-by: Leonardo Luz Almeida <[email protected]>
…oc (#15619) The supported version policy mentioned in the operator manual installation document diverges from the official policy that is mentioned in the security policy and the release cadence. This change brings the version policy in line by referring the readers to the release and cadence documentation to check the specified policy. Signed-off-by: Muhammad Mooneeb Hussain <[email protected]>
Signed-off-by: Timoses <[email protected]>
…rollback for tree view (#15572) * app sync and app wait tree view changes Signed-off-by: schakrad <[email protected]> * documentation changes Signed-off-by: schakrad <[email protected]> * included the json,yaml and wide formats and removed the value assignment to the flag Signed-off-by: schakrad <[email protected]> * Reoved extra spaces Signed-off-by: schakrad <[email protected]> * removed extra spaces Signed-off-by: schakrad <[email protected]> * refactored the code Signed-off-by: schakrad <[email protected]> * better log statements Signed-off-by: schakrad <[email protected]> --------- Signed-off-by: schakrad <[email protected]>
* Migrate Application Controller from Statefulset to Deployment Signed-off-by: ishitasequeira <[email protected]> * Add sharding deployment logic Signed-off-by: ishitasequeira <[email protected]> * Update sharding logic and add comments Signed-off-by: ishitasequeira <[email protected]> * Add heartbeat as an environment variable Signed-off-by: ishitasequeira <[email protected]> * Add retry logic, heartbeat timeout environment variable Signed-off-by: ishitasequeira <[email protected]> * use the logic of pre-specified shard number on application controller pod Signed-off-by: ishitasequeira <[email protected]> * fix manifests Signed-off-by: ishitasequeira <[email protected]> * fix lint and e2e tests Signed-off-by: ishitasequeira <[email protected]> * comment out failing e2e test Signed-off-by: ishitasequeira <[email protected]> * increase readiness probe interval period Signed-off-by: ishitasequeira <[email protected]> * "comment out readiness probe to see if e2e tests succeed" Signed-off-by: ishitasequeira <[email protected]> * revert commented readiness probe Signed-off-by: ishitasequeira <[email protected]> * revert commented test case Signed-off-by: ishitasequeira <[email protected]> * read environment variable for application controller deployment name Signed-off-by: ishitasequeira <[email protected]> * Add nil check on replica count for deployment of application controller Signed-off-by: ishitasequeira <[email protected]> * Address comments Signed-off-by: ishitasequeira <[email protected]> * Add Informer, Update documentation, add unit tests Signed-off-by: ishitasequeira <[email protected]> * update godoc Signed-off-by: ishitasequeira <[email protected]> * remove unwanted code and logs Signed-off-by: ishitasequeira <[email protected]> * Add more documentation Signed-off-by: ishitasequeira <[email protected]> * revert ApplicationController manifest to StatefulSet Signed-off-by: ishitasequeira <[email protected]> * reverting updated docs Signed-off-by: ishitasequeira <[email protected]> * Add documentation for the new dynamic distribution feature Signed-off-by: ishitasequeira <[email protected]> * update documentation Signed-off-by: ishitasequeira <[email protected]> * Add an overlay for application controller deployment and update documentation Signed-off-by: ishitasequeira <[email protected]> * fix nit Signed-off-by: ishitasequeira <[email protected]> * Marking the feature as alpha Signed-off-by: ishitasequeira <[email protected]> * Add feature status link Signed-off-by: ishitasequeira <[email protected]> * revert go,mod changes Signed-off-by: ishitasequeira <[email protected]> * update docs to avoid focusing on StatefulSet/Deployment (#26) * update docs to avoid focusing on StatefulSet/Deployment Signed-off-by: Michael Crenshaw <[email protected]> --------- Signed-off-by: Michael Crenshaw <[email protected]> * minor update to the doc Signed-off-by: ishitasequeira <[email protected]> --------- Signed-off-by: ishitasequeira <[email protected]> Signed-off-by: Michael Crenshaw <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
* chore(deps): bump node version Signed-off-by: Michael Crenshaw <[email protected]> * fix version Signed-off-by: Michael Crenshaw <[email protected]> * update lockfile Signed-off-by: Michael Crenshaw <[email protected]> --------- Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Robin Lieb <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3.8.0 to 3.8.1. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@bea5baf...5e21ff4) --- updated-dependencies: - dependency-name: actions/setup-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: Justin Marquis <[email protected]> Co-authored-by: pasha-codefresh <[email protected]>
Signed-off-by: Mike Ng <[email protected]>
) * feat(health): add `PushSecret` health status Signed-off-by: Alexandre Gaudreault <[email protected]> * add status healthy Signed-off-by: Alexandre Gaudreault <[email protected]> * Push action Signed-off-by: Alexandre Gaudreault <[email protected]> * fix test Signed-off-by: Alexandre Gaudreault <[email protected]> --------- Signed-off-by: Alexandre Gaudreault <[email protected]>
#15654) Signed-off-by: Michael Crenshaw <[email protected]>
* feat(appset): ignoreDifferences (#9101) Signed-off-by: Michael Crenshaw <[email protected]> * better error messages Signed-off-by: Michael Crenshaw <[email protected]> * do better Signed-off-by: Michael Crenshaw <[email protected]> * docs Signed-off-by: Michael Crenshaw <[email protected]> * more tests, update docs Signed-off-by: Michael Crenshaw <[email protected]> * e2e test Signed-off-by: Michael Crenshaw <[email protected]> * expect auto-added fields Signed-off-by: Michael Crenshaw <[email protected]> * correct label Signed-off-by: Michael Crenshaw <[email protected]> * better Signed-off-by: Michael Crenshaw <[email protected]> * remove line that was reverted Signed-off-by: Michael Crenshaw <[email protected]> * Update docs/operator-manual/applicationset.yaml Signed-off-by: Michael Crenshaw <[email protected]> * remove line that mysteriously causes applicationset/utils unit tests to fail Signed-off-by: Michael Crenshaw <[email protected]> * login to fix test Signed-off-by: Michael Crenshaw <[email protected]> * maybe this will work, who knows Signed-off-by: Michael Crenshaw <[email protected]> * burn it all down Signed-off-by: Michael Crenshaw <[email protected]> * works on my machine Signed-off-by: Michael Crenshaw <[email protected]> --------- Signed-off-by: Michael Crenshaw <[email protected]>
…9298) (#15026) * fix(applicationset): cannot validate inherited project permissions Signed-off-by: Alexandre Gaudreault <[email protected]> * update tests to reflect behavior Signed-off-by: Alexandre Gaudreault <[email protected]> --------- Signed-off-by: Alexandre Gaudreault <[email protected]>
Bumps [semver](https://github.com/npm/node-semver) from 5.7.1 to 5.7.2. - [Release notes](https://github.com/npm/node-semver/releases) - [Changelog](https://github.com/npm/node-semver/blob/v5.7.2/CHANGELOG.md) - [Commits](npm/node-semver@v5.7.1...v5.7.2) --- updated-dependencies: - dependency-name: semver dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <[email protected]>
) Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: ymktmk <[email protected]> Co-authored-by: Blake Pettersson <[email protected]>
* fix(ci): free up disk space Signed-off-by: Michael Crenshaw <[email protected]> * Update .github/workflows/image-reuse.yaml Co-authored-by: Justin Marquis <[email protected]> Signed-off-by: Michael Crenshaw <[email protected]> * Update .github/workflows/image-reuse.yaml Signed-off-by: Michael Crenshaw <[email protected]> --------- Signed-off-by: Michael Crenshaw <[email protected]> Co-authored-by: Justin Marquis <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
…15659) (#15660) * fix(appset): don't emit k8s events for unchanged application events and move that scenario to debug logging level Signed-off-by: Eric Blackburn <[email protected]> * Retrigger CI pipeline Signed-off-by: Eric Blackburn <[email protected]> * Retrigger CI pipeline Signed-off-by: Eric Blackburn <[email protected]> --------- Signed-off-by: Eric Blackburn <[email protected]> Co-authored-by: Alexander Matyushentsev <[email protected]>
In the Projects section there are two words 'what' and 'where' are written in italic(in the first two sentences) but in the third sentence 'what' is written in normal text. So i changed the two words written in italic to normal text to maintain the uniformity of writing style of documentation. See this issue in the Projects section : "https://argo-cd.readthedocs.io/en/stable/user-guide/projects/" Signed-off-by: PranitRout07 <[email protected]>
To see the issue , go to this link "https://argo-cd.readthedocs.io/en/stable/operator-manual/app-any-namespace/" In Prerequisites section ,the word 'not' is unnecessarily written in italic. It should be written in normal text to maintain the uniformity. Signed-off-by: PranitRout07 <[email protected]> Co-authored-by: Dan Garfield <[email protected]>
* fix: Tooltips point in wrong direction#11935 Signed-off-by: Teng, Jessie <[email protected]> * fix: Tooltips point in wrong direction#11935 Signed-off-by: Teng <[email protected]> --------- Signed-off-by: Teng, Jessie <[email protected]> Signed-off-by: Teng <[email protected]> Co-authored-by: Teng, Jessie <[email protected]>
* fix(11164): Advanced templating using patchTemplate Signed-off-by: gmuselli <[email protected]> * small changes Signed-off-by: Michael Crenshaw <[email protected]> --------- Signed-off-by: gmuselli <[email protected]> Signed-off-by: Michael Crenshaw <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
…sInAnyNamespaceEnabled flag is set (#16249) Signed-off-by: Eilers, Jonas <[email protected]>
…16062) (#16241) * fix(appset): store sha from webhook to get latest change during reconcile (#16062) Signed-off-by: dhruvang1 <[email protected]> * fix(appset): Don't use revision cache when reconciling after webhook(#16062) Signed-off-by: dhruvang1 <[email protected]> --------- Signed-off-by: dhruvang1 <[email protected]>
Signed-off-by: gmuselli <[email protected]>
…ssue #16523) (#16520) * Update cert-manager.opcertificate health.lua Signed-off-by: Chris Murray <[email protected]> * adding test case for cert issuing Signed-off-by: Chris Murray <[email protected]> * fixing typo Signed-off-by: Chris Murray <[email protected]> --------- Signed-off-by: Chris Murray <[email protected]>
…t platforms (#16065) Signed-off-by: Ondrej Sika <[email protected]>
Signed-off-by: Josh Soref <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
Wrongly placed horizontal line (`----`) was formatting code-block as a header. Fixed it with a necessary line break Signed-off-by: Elouan Keryell-Even <[email protected]>
Signed-off-by: Phil Nichol <[email protected]>
Signed-off-by: Jesse Suen <[email protected]>
Signed-off-by: Rotem Tamir <[email protected]>
… allow app's deletion (#12172) (#16506) * fix(appset): remove unnecessary condition Signed-off-by: mikutas <[email protected]> * docs: update explanation about policy Signed-off-by: mikutas <[email protected]> --------- Signed-off-by: mikutas <[email protected]>
Signed-off-by: Jay Shah <[email protected]>
* chore: upgrade kubernetes dependencies from 0.26.4 to 0.26.11 Fixes some vulnerabilities trivy is reporting on (not necessarily vulnerabe, trivy tends to have a lot of false positives when it comes to golang projects): * CVE-2023-3676 * CVE-2023-3955 * CVE-2023-5528 * CVE-2023-2431 * CVE-2023-2727 * CVE-2023-2728 Signed-off-by: Zoltán Reegn <[email protected]> * go mod tidy Signed-off-by: Zoltán Reegn <[email protected]> * Add go mod tidy to kubernetes updater script Signed-off-by: Zoltán Reegn <[email protected]> --------- Signed-off-by: Zoltán Reegn <[email protected]>
Signed-off-by: Elouan Keryell-Even <[email protected]>
…ns (#16534) Signed-off-by: dmpe <[email protected]>
Signed-off-by: Jason Wang <[email protected]>
The preserveFileMode isn't under parameters, it should be right under spec. It's correct in the other example here. Signed-off-by: yedayak <[email protected]>
…s ref only (#16501) * fix(repo-server): excess git requests part 2, short-circuit GenerateManifests ref only Signed-off-by: nromriell <[email protected]> * chore(logging): pr feedback, add debug log to short circuit Signed-off-by: nromriell <[email protected]> * chore: pr feedback, add ref to debug statement Signed-off-by: nromriell <[email protected]> --------- Signed-off-by: nromriell <[email protected]>
…nore wildard customizations (#16461) Signed-off-by: Ricky McMillen <[email protected]>
So that this can be used for custom resources. The motivation for this is issue #6008: with this change one should be able to use the known type to ensure equal durations don't present a diff, e,g. via adding to `argocd-cm`: data: resource.customizations.knownTypeFields.cert-manager.io_Certificate: | - field: spec.duration type: meta/v1/Duration This change is based on 5b464c9, though I've included the API version in the type path to be consistent with the generated type (i.e. `meta/v1/Duration` and not `meta/Duration`). For documentation I've just manually listed the extra types that aren't auto-generated, though this requires having to keep this list and the code in-sync but this is probably not a big issue since the number of extra types is not frequently changed. In the tests I've used `require.*` methods since I find this simpler than `if !assert.Blah(...) { return }` which the other tests in this file are using. Signed-off-by: Matthew Hughes <[email protected]>
…on (#16607) Signed-off-by: Arron Francis <[email protected]>
…6610) Signed-off-by: Dennis Pan <[email protected]>
Signed-off-by: Robin Lieb <[email protected]>
* docs: add documentation for the argocd cluster commands This adds initial non-reference documentation for the `argocd cluster` commands as part of #14531. The main goal here is to bootstrap that page and point users wanting to remove the `in-cluster` cluster to the right setting. Signed-off-by: Maurice Meyer <[email protected]> * fixup! docs: add documentation for the argocd cluster commands Signed-off-by: Morre <[email protected]> * fixup! docs: add documentation for the argocd cluster commands Signed-off-by: Maurice Meyer <[email protected]> --------- Signed-off-by: Maurice Meyer <[email protected]> Signed-off-by: Morre <[email protected]>
…t webhook handler (#16503) (#16504) * fix: case insensitive comparison of repo details in appset webhook handler Signed-off-by: Sam Greening <[email protected]> * chore(appset test): test case-insensitivity of appset github webhooks Signed-off-by: Sam Greening <[email protected]> * chore(appset test): test case for PR labeled event And fix a couple of spelling errors Signed-off-by: Sam Greening <[email protected]> --------- Signed-off-by: Sam Greening <[email protected]> Co-authored-by: Soumya Ghosh Dastidar <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Please see Contribution FAQs if you have questions about your pull-request.