forked from argoproj-labs/argocd-operator
-
Notifications
You must be signed in to change notification settings - Fork 0
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
merge upstream master #1
Open
rishabh625
wants to merge
309
commits into
rishabh625:update-go-version
Choose a base branch
from
argoproj-labs:master
base: update-go-version
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
* fix: remove default help chat url and text Signed-off-by: saumeya <[email protected]> * fix test Signed-off-by: saumeya <[email protected]> Signed-off-by: saumeya <[email protected]>
* update event emission logic for sso deprecation notices * address review comments
Fix health resource documentation to PVC from PV as example
* doc: add controller env variable examples Signed-off-by: saumeya <[email protected]> * review comments Signed-off-by: saumeya <[email protected]> * Empty-Commit * review comments Signed-off-by: saumeya <[email protected]>
…s it (#796) * fix only first problematic instance gets deprecation notice event * update comment * retrigger CI Signed-off-by: Jaideep Rao <[email protected]> Signed-off-by: Jaideep Rao <[email protected]>
"and it's components" -> "and its components"
Fixing discrepancy between Applicationset Controller Pod ContainerPort (8000) and Service TargetPort (8080) for metrics
* made changes to support the feature apps in any namespaces Signed-off-by: ishitasequeira <[email protected]> * address PR comments Signed-off-by: ishitasequeira <[email protected]> * add documentation for apps in any namespace Signed-off-by: ishitasequeira <[email protected]> * add "managed-by-cluster-argocd" label to sourceNamespaces Signed-off-by: ishitasequeira <[email protected]> * fix e2e tests Signed-off-by: ishitasequeira <[email protected]> * Address comments Signed-off-by: ishitasequeira <[email protected]> * Address the comments on the PR Signed-off-by: ishitasequeira <[email protected]> * update value for managed-by-cluster-argocd label in rolebinding.go Signed-off-by: ishitasequeira <[email protected]> * remove extra check and update documentation Signed-off-by: ishitasequeira <[email protected]> * update check for managed-by label having any value Signed-off-by: ishitasequeira <[email protected]> * rename SupportedNamespace to ApplicationSourceNamespace Signed-off-by: ishitasequeira <[email protected]> * cleanup resources when managed-by label is added later Signed-off-by: ishitasequeira <[email protected]> * update cleanup workflow Signed-off-by: ishitasequeira <[email protected]> * update docs and e2e test Signed-off-by: ishitasequeira <[email protected]> * add missing error.yaml for e2e test Signed-off-by: ishitasequeira <[email protected]> * Update e2e test case Signed-off-by: ishitasequeira <[email protected]> Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]> Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: ishitasequeira <[email protected]>
* feat: secure connection with dex, use https Signed-off-by: iam-veeramalla <[email protected]>
* fix: host url for Ingress in the .status feild on OpenShift Signed-off-by: iam-veeramalla <[email protected]>
Signed-off-by: Nicholas Morey <[email protected]>
* add support for new subkeys in resource customizations Signed-off-by: Regina Scott <[email protected]>
Signed-off-by: ishitasequeira <[email protected]>
* fix: Remove Argo CD as a base Image from the Export Dockerfile Signed-off-by: iam-veeramalla <[email protected]> * fix: address review comments Signed-off-by: iam-veeramalla <[email protected]> * fix: address review comments Signed-off-by: iam-veeramalla <[email protected]> Signed-off-by: iam-veeramalla <[email protected]>
Signed-off-by: iam-veeramalla <[email protected]> Signed-off-by: iam-veeramalla <[email protected]>
Signed-off-by: iam-veeramalla <[email protected]> Signed-off-by: iam-veeramalla <[email protected]>
Signed-off-by: Regina Scott <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]> Signed-off-by: Jaideep Rao <[email protected]>
#831) * chore: prepare release manifests (#827) Signed-off-by: iam-veeramalla <[email protected]>
Adds the missing Github.com keys added after https://github.blog/2021-09-01-improving-git-protocol-security-github/ Related to argoproj/argo-cd#7722
* chore: update version of the project to 0.6.0 and generate manifests Signed-off-by: iam-veeramalla <[email protected]>
* fix appset status not reflected in CR, add unit and e2e tests Signed-off-by: Jaideep Rao <[email protected]>
* fix appset status not reflected in CR, add unit and e2e tests Signed-off-by: Jaideep Rao <[email protected]> * expose workload status metrics Signed-off-by: Jaideep Rao <[email protected]> * fix go mod/sum drift Signed-off-by: Jaideep Rao <[email protected]> * address review comment Signed-off-by: Jaideep Rao <[email protected]> * fix merge conflicts Signed-off-by: Jaideep Rao <[email protected]> * add tracker for failed state Signed-off-by: Jaideep Rao <[email protected]> * fix merge conflicts Signed-off-by: Jaideep Rao <[email protected]> * fix merge conflicts Signed-off-by: Jaideep Rao <[email protected]> * undo changes to 0.5 bundle Signed-off-by: Jaideep Rao <[email protected]> * update 0.6 bundle manifests Signed-off-by: Jaideep Rao <[email protected]> Signed-off-by: Jaideep Rao <[email protected]>
* allow users to enable/disable monitoring for CR Signed-off-by: Jaideep Rao <[email protected]>
…olangci-lint (#1537) Signed-off-by: Cheng Fang <[email protected]>
…s instead of quote (#1538) Signed-off-by: Cheng Fang <[email protected]>
…figmaps is not updated (#1542) Signed-off-by: Jayendra Parsai <[email protected]> Co-authored-by: Jayendra Parsai <[email protected]>
…e exists (#1546) * Add Server Service check to apply the changes in case the service exists Signed-off-by: nmirasch <[email protected]> * Add changes to existing service and update Signed-off-by: nmirasch <[email protected]> * Add testing Signed-off-by: nmirasch <[email protected]> --------- Signed-off-by: nmirasch <[email protected]>
Signed-off-by: Jayendra Parsai <[email protected]> Co-authored-by: Jayendra Parsai <[email protected]>
* Use secret for route tls Signed-off-by: Siddhesh Ghadi <[email protected]> * Add e2e test Signed-off-by: Siddhesh Ghadi <[email protected]> * Fix lint and add gitleak ignore file Signed-off-by: Siddhesh Ghadi <[email protected]> * Update openshift api library to latest of 4.14 Signed-off-by: Siddhesh Ghadi <[email protected]> * Implement custom logic for externalCertificate from latest Route API schema Signed-off-by: Siddhesh Ghadi <[email protected]> * Update comment Signed-off-by: Siddhesh Ghadi <[email protected]> --------- Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: saumeya <[email protected]>
…1545) * fix: delete/dont create deployemnt resources if using .remote field Signed-off-by: Anand Kumar Singh <[email protected]> * fix: skip service creation if route is enabled Signed-off-by: Anand Kumar Singh <[email protected]> * update documentation Signed-off-by: Anand Kumar Singh <[email protected]> * add unit-tests Signed-off-by: Anand Kumar Singh <[email protected]> * add IsRemote for redis and repo-server component Signed-off-by: Anand Kumar Singh <[email protected]> * fix docs Signed-off-by: Anand Kumar Singh <[email protected]> * fix test Signed-off-by: Anand Kumar Singh <[email protected]> * rebase Signed-off-by: Anand Kumar Singh <[email protected]> * fix: goimports error Signed-off-by: Anand Kumar Singh <[email protected]> * fix: isRemote enabled should delete Signed-off-by: Anand Kumar Singh <[email protected]> --------- Signed-off-by: Anand Kumar Singh <[email protected]>
Signed-off-by: nmirasch <[email protected]>
…epo, application set controller, and application controller (#1532) * Added custom label and annotations Signed-off-by: enriqueav99 <[email protected]> * Added custom label and annotations Signed-off-by: enriqueav99 <[email protected]> * missunderstanding Signed-off-by: enriqueav99 <[email protected]> * missunderstanding Signed-off-by: enriqueav99 <[email protected]> * test Signed-off-by: enriqueav99 <[email protected]> * deleted from alpha Signed-off-by: enriqueav99 <[email protected]> * added the applicationset and fix test Signed-off-by: enriqueav99 <[email protected]> * make bundle Signed-off-by: enriqueav99 <[email protected]> * fixs Signed-off-by: enriqueav99 <[email protected]> * Deleted the test Signed-off-by: enriqueav99 <[email protected]> * Added the test again simplificated to check the CI Signed-off-by: enriqueav99 <[email protected]> * original makefile version Signed-off-by: enriqueav99 <[email protected]> * resolve conflicts Signed-off-by: enriqueav99 <[email protected]> * bundle Signed-off-by: enriqueav99 <[email protected]> * bundle Signed-off-by: enriqueav99 <[email protected]> * bundle Signed-off-by: enriqueav99 <[email protected]> * resolve problems Signed-off-by: enriqueav99 <[email protected]> * EOF Signed-off-by: enriqueav99 <[email protected]> * refactor Signed-off-by: enriqueav99 <[email protected]> * resolve conflicts Signed-off-by: enriqueav99 <[email protected]> * resolve conflicts Signed-off-by: enriqueav99 <[email protected]> * resolve conflicts Signed-off-by: enriqueav99 <[email protected]> * deleted olm 0.12.0 changes and added doc Signed-off-by: enriqueav99 <[email protected]> * deleted olm 0.12.0 changes and added doc Signed-off-by: enriqueav99 <[email protected]> * deleted olm 0.12.0 changes and added doc Signed-off-by: enriqueav99 <[email protected]> * deleted olm 0.12.0 changes and added doc Signed-off-by: enriqueav99 <[email protected]> * EOF Signed-off-by: enriqueav99 <[email protected]> * EOF Signed-off-by: enriqueav99 <[email protected]> * assert test Signed-off-by: enriqueav99 <[email protected]> * delete labels and annotations test Signed-off-by: enriqueav99 <[email protected]> * delete labels and annotations test Signed-off-by: enriqueav99 <[email protected]> * EOF Signed-off-by: enriqueav99 <[email protected]> --------- Signed-off-by: enriqueav99 <[email protected]>
* feat: add option to enable argo-rollout ui extension Signed-off-by: saumeya <[email protected]> * review comments and fixes Signed-off-by: saumeya <[email protected]> * add unit test Signed-off-by: saumeya <[email protected]> * doc update and e2e test Signed-off-by: saumeya <[email protected]> * review comments Signed-off-by: saumeya <[email protected]> * fix Signed-off-by: saumeya <[email protected]> * review changes, e2e fix Signed-off-by: saumeya <[email protected]> * review changes Signed-off-by: saumeya <[email protected]> * fix Signed-off-by: saumeya <[email protected]> --------- Signed-off-by: saumeya <[email protected]>
* Adds volumes and volumeMounts to argocd types * Adds volumes ans volumeMounts to appset deployment and pod * Updates generated files * Updates docs * Adds unit tests * Adds e2e-test Signed-off-by: appiepollo14 <[email protected]>
…r dynamicScaling (#1574) * fix the setting of environment variable ARGOCD_CONTROLLER_REPLICAS for dynamicScaling Signed-off-by: Ishita Sequeira <[email protected]> * fix lint issues Signed-off-by: Ishita Sequeira <[email protected]> --------- Signed-off-by: Ishita Sequeira <[email protected]>
Signed-off-by: Ishita Sequeira <[email protected]>
… configuration (#1580) * create events if tls.certificate and tls.key are used Signed-off-by: Anand Kumar Singh <[email protected]> * add warning and event if certificate and key fields used in route configuration Signed-off-by: Anand Kumar Singh <[email protected]> * update failing test Signed-off-by: Anand Kumar Singh <[email protected]> * Send event once for each component Signed-off-by: Anand Kumar Singh <[email protected]> * update util.go Signed-off-by: Anand Kumar Singh <[email protected]> * nit Signed-off-by: Anand Kumar Singh <[email protected]> * Modify event message Signed-off-by: Siddhesh Ghadi <[email protected]> --------- Signed-off-by: Anand Kumar Singh <[email protected]> Signed-off-by: Siddhesh Ghadi <[email protected]> Co-authored-by: Siddhesh Ghadi <[email protected]>
* add support for image override for argocd-extension Signed-off-by: Anand Kumar Singh <[email protected]> * fix unit test failure Signed-off-by: Anand Kumar Singh <[email protected]> --------- Signed-off-by: Anand Kumar Singh <[email protected]>
…k8s cluster (#1586) * update development.md file Signed-off-by: Atif Ali <[email protected]> * resolved issues Signed-off-by: Atif Ali <[email protected]> --------- Signed-off-by: Atif Ali <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
#1596) * Add applicationset controller policy configuration documentation Signed-off-by: nmirasch <[email protected]> --------- Signed-off-by: nmirasch <[email protected]>
Signed-off-by: Anand Kumar Singh <[email protected]>
* Fix code block in custom label doc Signed-off-by: Siddhesh Ghadi <[email protected]> * Add missing fields in table Signed-off-by: Siddhesh Ghadi <[email protected]> --------- Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
…1610) Subscription supposed to lowercase and no pipe is need at the end Signed-off-by: Vikas Pogu <[email protected]>
) Signed-off-by: Jayendra Parsai <[email protected]> Co-authored-by: Jayendra Parsai <[email protected]>
* feat: Add support for respectRBAC configuration. Signed-off-by: Jayendra Parsai <[email protected]> * docs: add Respect RBAC page to TOC Signed-off-by: Jonathan West <[email protected]> --------- Signed-off-by: Jayendra Parsai <[email protected]> Signed-off-by: Jonathan West <[email protected]> Co-authored-by: Jayendra Parsai <[email protected]> Co-authored-by: Jonathan West <[email protected]>
* Fix update logic to preserve k8s specific labels & annotations Operator overwrites k8s added labels or annotations. This sometimes results into unexpected behaviour. Eg. A kubectl rollout restart on deployment adds an annotation "kubectl.kubernetes.io/restartedAt" with timestamp. However operator detects this as a config drift and removes the annotations resulting into k8s terminating the rollout due to change in config. This commit fixes the issue by preserving such annotations & labels added onto live object during comparison. Signed-off-by: Siddhesh Ghadi <[email protected]>
…#1604) * feat: make specifying sidecar plugin image optional Signed-off-by: John Pitman <[email protected]> * update comment and docs Signed-off-by: John Pitman <[email protected]> * update bundle Signed-off-by: John Pitman <[email protected]> * add comment to v1alpha1 Signed-off-by: John Pitman <[email protected]> --------- Signed-off-by: John Pitman <[email protected]>
* feat: log k8s create/update/delete actions Signed-off-by: John Pitman <[email protected]> * fix linter errors Signed-off-by: John Pitman <[email protected]> * add nil check to LogResourceAction Signed-off-by: John Pitman <[email protected]> --------- Signed-off-by: John Pitman <[email protected]>
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.23.0 to 0.31.0. - [Commits](golang/crypto@v0.23.0...v0.31.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.
What type of PR is this?
What does this PR do / why we need it:
Have you updated the necessary documentation?
Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer: