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

Helm chart + values files from Git #2789

Closed
evgf opened this issue Dec 2, 2019 · 222 comments · Fixed by #10432
Closed

Helm chart + values files from Git #2789

evgf opened this issue Dec 2, 2019 · 222 comments · Fixed by #10432
Assignees
Labels
component:core Syncing, diffing, cluster state cache enhancement New feature or request type:usability Enhancement of an existing feature workaround There's a workaround, might not be great, but exists
Milestone

Comments

@evgf
Copy link

evgf commented Dec 2, 2019

Hello everyone,

we are trying to migrate our CD processes to ArgoCD and faced an issue where I really need some input. (Checked the official documentation and threads here but seems like there is no answer.)

Our typical scenario is:

  • Helm charts available via Git or as a remote Hem Repo
  • values-file or files, specific to the service & stage, available in Git.

What we want to achieve:

  • When the values-file gets updated, ArgoCD synchronises the environment using the Helm chart + this values files.

Should be pretty common scenario, I believe.

Here are the issues that we're facing when implementing it using ArgoCD:

  1. There is no way (or at least we did not find any) to use two Repos - one for the Chart and one for the values file. Although we only need to monitor the changes in the values-file(s) and not in the chart.

  2. If we fully rely on Git and store the charts & values for all stages in a single repo, there is no way to separate the chart versioning per stage. So I cannot deploy the "UAT" stage using the chart v. 1.1.1 while retaining the "Prod" stage using the chart v. 1.0.0.

  3. If we use separate Git branches to store charts + values for individual stages, there is no single Repo for the chart (all charts are stored per-stage) so no process of testing one chart version in Stage and then moving it to Prod.

  4. If we rely on Helm support in ArgoCD 1.3 (looks promising) and pack the values files inside the Helm Repo, we need to re-index it every time the values file changes (Git push + WebHook + CI/CD re-index - exactly what we want to avoid when moving to ArgoCD). Also, the repository server seems not re-indexing the Helm-Repo with the existing "targetRevision" and we need to re-start it to get the latest Helm Repo downloaded. If we increment the Helm Repo version for every change of the values-file, we need to adjust the targetVersion field in the Application + this is sort of against the Helm philosophy to include & version the values-files in the Helm Repo.

Am I missing something in this picture?
How would / do you implement such a scenario?

Kind Regards,

@OmerKahani
Copy link
Contributor

Just as an input, we are doing number 2. Most of the time staging and prod are the same so we only use one branch.

If we need to change the chart, we change the application in argocd to use a branch. That way staging will point to a branch. and once the change is good we merge back to master (and deploy to production)

@alexec
Copy link
Contributor

alexec commented Dec 2, 2019

You should be able to configure a values file from the UI that is a URL itself. Have you tried that?

It's unlikely we'll be able to support both Git+Helm repo, but you could also use a requirements.yaml in a Git repo to do this.

@evgf
Copy link
Author

evgf commented Dec 4, 2019

Hi @alexec, thank you for your answer.
Yes, I've seen this opportunity but it cannot solve this problem. There is no real need to support two Git Repos - let me explain the most typical Helm scenario:

Normally, the CI solution builds a new image and manages the values-file that includes a tag for this newly built image. It then uses Helm to retrieve the Helm Chart from a remote Helm Repo and provides this values file to install / upgrade the application.

Moving to CD, we need to observe the values-file in Git for changes and apply it together with the remote Helm Repo in case of any change.
Monitoring of the Helm-chart itself is not necessary as soon as we rely on a particular version which is configured in the Application CRD (as you do it in 1.3).

How would you implement this scenario using ArgoCD?

@alexec
Copy link
Contributor

alexec commented Dec 4, 2019

For you use case I’d try creating a Git repo with a Helm chart in it. That would have a values file you could update, and import the main Helm chart using requirements.yaml.

Alternatively, you CI could use the Argo CD CLI (or API) to update the image.

Another way, store the App manifest in Git (aka “declarative”) and your CI modifies that.

@evgf
Copy link
Author

evgf commented Dec 6, 2019

I'm currently implementing the option nr. 1 although the problem is that we have a Helm Chart which is used in 30+ Services and each of them is customized for 3 possible environments.

Replicating this Chart 30 times without a centralized Repo looks dirty. Can be a show stopper for the whole migration.

Modifying the Application definition is not an option since the whole goal is to reduce the rights that the CI-solution has. Giving it the right to update all Application-definitions from various teams in the argocd namespace is a a hard thing to convince people with.
Or you mean something different?

@evgf
Copy link
Author

evgf commented Dec 6, 2019

The actual plan is to separately pull the Helm Chart Repo and the values Repo on the CI-solution and then push both of them as a consolidated Chart to the "Stage" Repo / Branch.
Not ideal but should resolve the major issue.

@alexec alexec added enhancement New feature or request workaround There's a workaround, might not be great, but exists labels Dec 6, 2019
@cronik
Copy link

cronik commented Jan 15, 2020

I have a similar use case in trying to adopt Argo in our organization. We have many application (100's) using a common chart that is published to Artifactory which serves as our helm repository. The values files are stored in each app's repo. Having each app add a chart to their repo is not really practical at scale.

I was going to attempt to work around this using a configManagementPlugin along the lines of:

# NOTE: this hasn't been tested and almost certainly doesn't work
configManagementPlugins: |
  - name: myrepo-charts
    init:
      command: ["/bin/sh", "-c"]
      args: 
      - |
        helm init --client-only --skip-refresh \
        && helm repo rm stable \
        && helm repo add myrepo https://artifactory.example.com/artifactory/helm \
        && helm repo update \
        && helm fetch --untar --untardir charts --version $CHART_VERSION myrepo/$CHART_NAME

    generate:
      command: [sh, -c]
      args: ["helm template charts/$CHART_NAME -f $CHART_VALUES"]

Ideally there is a native solution for this case where we could do something like this:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: myapp-dit
spec:
  project: default
  source:
    repoURL: https://bitbucket.example.com/scm/foo/myapp.git
    targetRevision: HEAD
    path: ""
    helm:
      repoURL: https://artifactory.example.com/artifactory/helm
      chart: "web-service"
      chartVersion: "1.0.0"
      valueFiles: ['values-dit.yaml'] # resolved against source.repoURL and source.path

@rbkaspr
Copy link

rbkaspr commented Mar 26, 2020

@cronik Your solution actually works just fine, with some modifications. Here's a generalized version of my solution:

Added this plugin to argocd-cm

data:
  configManagementPlugins: |
    - name: helm-resolver
      init:
        command: [bash, -c]
        args: ["helm repo add <repoName> <repoURL>
      generate:
        command: [bash, -c]
        args: ["helm template -n $ARGOCD_APP_NAMESPACE --values values.yaml $ARGOCD_APP_NAME <chart> --version CHART-VERSION"]

And then created a Git repo that contains only the values file for overrides. My application spec for one of these apps would look like so:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: sample-app
  namespace: argocd
spec:
  destination:
    namespace: sample-namespace
    server: https://kubernetets.default.svc
  project: default
  source:
    path: .
    plugin:
      env:
        - name: CHART_VERSION
          value: 1.0.0
      name: helm-resolver
    repoURL: [email protected]/sample-app

Now I'd still say it'd be nice to have this sort of option more natively available in ArgoCD, but this appears to be a valid workaround at this time.

Tested and verified on v1.5.0-rc1

@matus-m
Copy link

matus-m commented Apr 2, 2020

The inability to pass external values.yaml files to helm charts is a real pain. Maybe it could also be solved by propagating values from app-of-apps(which is goign to have git as source) down to the ,,children" argo crds(whose source is helm rep). We can do it via templates, but it is very error prone and laborious to keep in sync if you have lot of apps deployed.

@franklevering
Copy link

Would this be a feature that might get supported by the ArgoCD team? We are facing a similar issue where we are trying to install an external Helm chart, but having our values.yaml in our Git repository. The version of the Helm Chart we are using won't be updated a lot, however, the values that we are passing to the Helm chart might change more often. Therefore we are versioning the values file within our repository and we prefer ArgoCD to update the Helm chart whenever the version of the values changes.

I'm happy to start working on a PR for this. However, I am not too familiar with the internals of ArgoCD so I would need some guidance on that. Also, it's good to know if this feature will make it to a stable release before I start working on anything.

@jannfis
Copy link
Member

jannfis commented Apr 6, 2020

@franklevering Sourcing an application from two (or more) repositories would be a major design change to many components in ArgoCD. It would also introduce a lot of challenges to be solved, and a lot of questions to be answered.

Having just read the thread in this issue, and considering many of the cons outspoken for original solution number 2, I would at least consider the following scenario:

  • Get the Chart.yaml in a Git repository
  • Get the values.yaml in another, application specific Git repository
  • Connect both repositories by including the repository containing Chart.yaml as a Git sub-module to the application specific Git repository
  • Create the ArgoCD application from the application specific repository

This way, both assets - Chart.yaml and application specific values.yaml can have their own life-cycles. By smart tagging or branching of Chart.yaml repository and appropriately including it as a sub-module, most if not all use-cases could be covered.

Working with Git submodules is supported since ArgoCD v1.4 (see https://argoproj.github.io/argo-cd/user-guide/private-repositories/#git-submodules).

What do you guys think about this?

@franklevering
Copy link

franklevering commented Apr 6, 2020

@franklevering Sourcing an application from two (or more) repositories would be a major design change to many components in ArgoCD. It would also introduce a lot of challenges to be solved, and a lot of questions to be answered.

Having just read the thread in this issue, and considering many of the cons outspoken for original solution number 2, I would at least consider the following scenario:

  • Get the Chart.yaml in a Git repository
  • Get the values.yaml in another, application specific Git repository
  • Connect both repositories by including the repository containing Chart.yaml as a Git sub-module to the application specific Git repository
  • Create the ArgoCD application from the application specific repository

This way, both assets - Chart.yaml and application specific values.yaml can have their own life-cycles. By smart tagging or branching of Chart.yaml repository and appropriately including it as a sub-module, most if not all use-cases could be covered.

Working with Git submodules is supported since ArgoCD v1.4 (see https://argoproj.github.io/argo-cd/user-guide/private-repositories/#git-submodules).

What do you guys think about this?

I understand your approach with Git submodules. However, after browsing through the ArgoCD examples we found another way to achieve what we want (https://github.com/argoproj/argocd-example-apps/tree/master/helm-dependency).

Instead of using git submodules, we can just create a Chart.yaml in our application specific Git repository. This Chart.yaml only has a dependency, which is the actual Helm chart we want to install. That way, we can monitor the application specific Git repository for changes and update the Helm chart. This would work for both external as well as self-created Helm charts.

@jannfis
Copy link
Member

jannfis commented Apr 6, 2020

Awesome, @franklevering - this is actually even simpler! Thanks for sharing this solution.

@bnutt
Copy link

bnutt commented Apr 7, 2020

@franklevering Were you able to get this case to work? Trying to pass my own valuesFile in git and its in the same directory as where the Chart.yaml is specificed. The Chart.yaml also has a dependencies section in it to reference the chart to be deployed and its repo. Currently it templates the chart, but keeps falling back to the default values, even if the app shows that it loaded the custom values file into Argo CD.

@amkartashov
Copy link

amkartashov commented Apr 9, 2020

helm dependency is w/a, and it's not complex, but

  • it feels awkward
  • values file differs from manual deployment of the same chart (and not only by moving dependency values under chart-name: , because tags and conditions are always taken from the top chart)

I like proposal for extending helm source type here #2789 (comment) - in addition it would be good to have option to change chartVersion as a parameter, and also it might be needed to modify source type detection so ArgoCD sees that this is an external helm package.

@laurentzoz
Copy link

laurentzoz commented Apr 14, 2020

Hello, this is my use case:

  • argocd 1.5.1
  • Google Kubernetes Engine cluster
  • I host a helm repo on a private GCS bucket, with a publicly accessible index.yaml
  • helm API v2
  • I made a custom image containing https://github.com/hayorov/helm-gcs, to access the private repo
  • I use the helm dependency model: all my charts are in a github monorepo, master branch only. Chart versioning is done by the helm repo. The final values files are on github using the following hierarchy:
deployments
-- dev
---- app1
------ Chart.yaml
------ values.yaml
-- prod
---- app1
------ Chart.yaml
------ values.yaml

This is where things become weird: this setup works in local and even in the repo-server container with kubectl exec + helm dependency build. helm template also works.

But impossible to make it work going through the regular way, I get the error
level=error msg="`helm dependency build` failed exit status 1: Error: could not find : chart frontend-react not found in gs://my-helm-chart-repo"
At first I was thinking about a basic permission issue between GCS and the helm plugin but doing the commands manually works like a charm 🤔
Is it because I open a shell or something ? I'm not really sure it's argocd related at this point.

EDIT:
I finally got it to work ! I read #1105 and #1153 which pointed me to the environment variable HELM_PLUGINS, the directory where the plugins should live. Using helm3 and my custom install, it's /home/argocd/.local/share/helm/plugins/ but YMMV
I simply added this env var directly in the repo-server deployment, nothing more !

@psionikangel
Copy link

@franklevering Were you able to get this case to work? Trying to pass my own valuesFile in git and its in the same directory as where the Chart.yaml is specificed. The Chart.yaml also has a dependencies section in it to reference the chart to be deployed and its repo. Currently it templates the chart, but keeps falling back to the default values, even if the app shows that it loaded the custom values file into Argo CD.

I also tried the same thing and altough my app was deployed, the custom values were not taken into account.

@test-account-0
Copy link

test-account-0 commented May 5, 2020

Also You can embed values in application:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: fluent-bit-stable
  namespace: ops
spec:
  project: default
  source:
    repoURL: https://kubernetes-charts.storage.googleapis.com
    chart: fluent-bit
    targetRevision: 2.8.12
    helm:
      values: |
        image:
          fluent_bit:
            repository: fluent/fluent-bit
            tag: 1.3.8
            pullPolicy: Always
        metrics:
          enabled: false
  destination:
    server: https://kubernetes.default.svc
    namespace: ops
  syncPolicy:
    automated:
      selfHeal: false

but option to have separate file in the same directory as application file would be better.

Getting values file from helm repo is something almost never used.

@jannfis jannfis added component:core Syncing, diffing, cluster state cache type:usability Enhancement of an existing feature labels May 14, 2020
@musabmasood
Copy link

musabmasood commented May 19, 2020

In the same boat. Our situation is something like this.

We followed the best practices of keeping k8s config separate from the app repo, so we have these (hypothetical) repos:

argocd-repo
This repo contains argocd declarative setup manifests, secrets etc. Basically to install argocd on our argocd cluster. Apart from that, this repo also contains a special custom helm chart helm-bootstrap-project which I will talk about later.

myapp-repo
This is the application repo for our "myapp". It contains the actual code and Dockerfile. The CI for this repo builds/tags/pushes the image and creates a commit in the myproject-repowhere it updates helm-myapp chart's values.yaml with the new tag.

myproject-k8s-repo
This repo contains the chart helm-myapp and multiple value files for its deployment on different clusters.

Process:

  • Developers develop on the myapp-repo and everytime there is a new merge-to-master myproject-repo/charts/helm-myapp should get updated with the latest version
  • SREs maintain myproject-repo with helm charts and their value files for different environments
  • SREs also maintain the argocd-repo and the common helm-bootstrap-project chart inside it

Apart from that, developers can occasionally create PRs on myproject-repo but they need SRE approvals.

The helm-bootstrap-project chart

When we have a new project or tenant, we wanted a simpler process to onboard it into argocd. This chart templates all the Applications, AppProjects and clusters etc for a new tenant. In the case above, our tenant is myproject so we deploy this chart with a values file something like this:

projectName: myproject
clusters: {...}
mainRepo: myproject-repo.git
deployments:
  staging:
   # by convention, looks for the helm chart in myproject-repo.git/charts/
   # by convention, uses the value file myproject-repo.git/env/staging/helm-myapp.yaml
    - helm-myapp
  prod:
   ...
...

So ideally for the myproject tenant, we just want to create a file argocd-bootstrap.yaml with the above configuration and keep it in the root of myproject-repo.git. We are struggling to find the best solution as well. Some options:

  1. myproject-repo.git/argocd-bootstrap.yaml can be an ArgoCD Application CRD with values embedded inside it, and uses the remote helm-bootstrap-project chart. Not really a fan of embedding values.
  2. Create a helm chart myproject-repo.git/charts/argocd-bootstrap which uses helm-bootstrap-project as a dependency. Again, it looks awkward.
  3. Clone the helm-bootstrap-project chart into myproject-repo.git, which defeats the whole purpose of the common chart
  4. Create tenant value files within the argocd-repo, this way they live with the helm-bootstrap-project chart
  5. Do something like Helm chart + values files from Git #2789 (comment)
  6. Use remote values file in the valueFiles parameter

Currently we are doing (4) but the value files change more frequently than the common chart, and we dont want developers to create PRs on the argocd-repo.

The thing with (1), (2) and (3) is that we would like our rendered manifests to auto-update when we make a change to the common chart. I guess with (2) we can use semantic versioning or something.

(5) and (6) is also interesting but we haven't tried it yet.

By the way ArgoCD is awesome, great work guys! 💯


Update: We tried remote value files (6) with raw github URL. Its not helpful as the Application is not able to track new changes if they are only on the values file.

@Romiko
Copy link

Romiko commented Jun 2, 2020

You should be able to configure a values file from the UI that is a URL itself. Have you tried that?

It's unlikely we'll be able to support both Git+Helm repo, but you could also use a requirements.yaml in a Git repo to do this.

Thanks, we got this working nicely with the prometheus operator :)

Awesome work on this project!

@OmerKahani
Copy link
Contributor

OmerKahani commented Jun 2, 2020

It's unlikely we'll be able to support both Git+Helm repo, but you could also use a requirements.yaml in a Git repo to do this.

we use requirements.yaml in all of our charts, the charts that ArgoCD uses are just value files and requirements.yaml that reference to the "real" chart, this also helps when we want to add objects as we can just add a templates folder to our chart.

@Romiko
Copy link

Romiko commented Jun 3, 2020

Hi,

I think this issue can be closed. Here is the solution.

  • A requirments.yaml in your components folder./components/mycomponent/requirements.yaml
  • The root element of all your values files e.g ./components/mycomponent/values.yaml must contain the remote chart name

Examples

./components/nginx-ingress/values-tools.yaml
nginx-ingress: controller:

./components/nginx-ingress/requirements.yaml
`
dependencies:

Sample ScreenShot

@Romiko
Copy link

Romiko commented Jun 3, 2020

It's unlikely we'll be able to support both Git+Helm repo, but you could also use a requirements.yaml in a Git repo to do this.

we use requirements.yaml in all of our charts, the charts that ArgoCD uses are just value files and requirements.yaml that reference to the "real" chart, this also helps when we want to add objects as we can just add a templates folder to our chart.

Yeah, we just did this now, the only catch was adding the chart name as a root element to all value files. my last post for an example.

@jetersen
Copy link
Contributor

jetersen commented Jun 3, 2020

@Romiko please copy paste the screenshot to github, it will take care of markdown and uploading it 👍
Also use of triple backticks would be appreciated

```yaml
nginx-ingress:
  controller:
```
```yaml
dependencies:
- name: nginx-ingress
  version: "1.39.0"
  repository: https://kubernetes-charts.storage.googleapis.com/
```

(Hint four ```` backticks allow you do the above)

This produces:

nginx-ingress:
  controller:
dependencies:
- name: nginx-ingress
  version: "1.39.0"
  repository: https://kubernetes-charts.storage.googleapis.com/

@test-account-0
Copy link

@Romiko I might not understand something. What is new in Your approach?

@sdelrio
Copy link

sdelrio commented Jan 3, 2023

@sdelrio have you installed the 2.6.0-rc1 ApplicationSet CRD? All the necessary changes should already be in place.

I just put this into helm values, you're right, the CRDs wont be updated since helm chart doesn't have the new ones

argo-cd: 
  global:
    image:
      tag: v2.6.0-rc1

This won't upgrade the CRDs.

Will manually update with https://github.com/argoproj/argo-cd/tree/master/manifests/crds and test again. Thanks!

@sdelrio
Copy link

sdelrio commented Jan 7, 2023

@crenshaw-dev all went fine with with ApplicationSet CRD thanks for help.

Also updated to v2.6.0-rc2 and after that just had to add the CRDs an make a workaround until #11796 issue is fixed, adding pathParamPrefix: in the generators git.

@dllegru
Copy link

dllegru commented Jan 12, 2023

@crenshaw-dev

Using this thread to post a series of issues we've seen when evaluating this FEAT in v2.6.0-rc2.
Issues are somewhat similar in some cases, so probably are happening by the same root case, just stating them anyway to cover all cases. Also one of the issues has already been mentioned in this thread.

When using the following Application definition:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: infrastructure-labs-exercise
  namespace: argo-cd
  finalizers:
    - resources-finalizer.argocd.argoproj.io
spec:
  project: default
  sources:
    - repoURL: https://github.com/xxxx/infrastructure-labs
      targetRevision: SPIKE/tests
      ref: myRepo
    - repoURL: https://charts.bitnami.com/bitnami
      targetRevision: 13.2.21
      chart: nginx
      helm:
        valueFiles:
          - $myRepo/argo-cd-exercises/nginx-values-10/values.yaml
  destination:
    name: in-cluster
    namespace: daniel-tests
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
      allowEmpty: false

Doing the following evaluations:


1 - Deploying from scratch

We will evaluate doing a deployment from a scratch as a new Application into the destination cluster.

  • Evaluation:
    • Status: ✅⚠️ (works with inconsistency state errors until refreshed)
    • It works as expected, deploying the chart defined in the sources: with the configuration defined in the values.yaml
    • An issue happens within Argo CD showing inconsistency as the Application it is indeed deployed into the cluster but being shown as status status: OutOfSync & health: missing in Argo CD.
  • Issue - inconsistency:
    • Application gets created, but the state is shown as status: OutOfSync & health: missing both in Argo GUI and object status
  • Fix issue - inconsistency:
    • It gets fixed by clicking in Refresh button in the Argo CD application GUI or by cli argocd app get infrastructure-labs-exercise --refresh

2 - Modify parameters of values.yaml after the Application is deployed & running

We will evaluate modifying parameters of the sourced values.yaml when the Application is already deployed, running and in Healthy state.

The repository containing the sourced values.yaml has a webhook configured towards Argo CD which should automatically trigger the Application sync with the new value changes.

  • Evaluation:

    • Status:  (non functional)
    • Changes pushed at values.yaml are not picked up by Argo CD automatically.
    • Manual intervention has to be done in order to pick up the new values.
  • Issue - not syncing with modified values.yaml:

    • After committing and pushing changes into the defined & tracked $myRepo values.yaml branch the Application does not get synced with the new values. Argo CD does not detect it and does not trigger any change in the Application.
      • We changed replicaSet: 2 for replicaSet: 5
    • It does not matter if we even have webhooks enabled or waiting for the default sync timer.
    • Doing an Application SYNC or REFRESH does not update the values.yaml into the Application deployed.
    • Application it is being shown as Healthy and Synced when is not true.
  • Fix - not syncing with modified values.yaml:

    • Gets fixed running a --hard-refresh either by GUI or CLI
    • After running the —hard-refresh it does fix the issue as now the Application has updated its values and increasing pods to 5 replicas as defined in the values.yaml, this also being reflected in the GUI.
    • Although, a new sub-issue happens after this, which is that the Application is being now shown as OutOfSync in both gui and cli.
    • This sub-issue gets fixed running a --refresh into the Application, seems related to the Issue - inconsistency mentioned in the first case.

3 - Modify the Application definition after it is deployed & running

We will evaluate modifying definition of the resource Application itself after is already deployed, running and in Healthy state.

We tested modifying both parameters at sources: either using a different Helm Chart version (targetRevision) and/or changing valueFiles repository folder/file.

We modified the Application updating a local yaml definition file then updating it in the cluster running kubectl apply -f application.yaml

  • Evaluation:

    • Status: ⚠️ (works with inconsistency state errors until refreshed)
    • It works as expected, after applying changes into the defined Application cluster resource, Argo CD does apply the new values.
    • An issue happens within Argo CD showing inconsistency as the Application it is indeed updated and applied with the new definition into the cluster, but being shown as status: OutOfSync & health: missing.
  • Issue - inconsistency:

    • Application gets updated and new changes applied, but the state is shown as status: OutOfSync & health: missing both in Argo GUI and object status
  • Fix issue - inconsistency:

    • It gets fixed by clicking in Refresh button in the Argo CD application gui or by cli argocd app get infrastructure-labs-exercise --refresh

4 - Delete the Application , change values.yaml and recreate Application

We will evaluate delete a deployed & running Application, modify parameters in the original source values.yaml and create again the Application

  • Evaluation:
    • Status:  (non functional)
    • When the Application gets created again, it is created with the previous values.yaml parameters.
    • Application being shown as status status: OutOfSync & health: missing in Argo CD.
    • Manual intervention has to be done in order to pick up the new values.
  • Issue - inconsistency & old values.yaml parameters deployed:
    • Application gets deployed with previous values.yaml
    • State is shown as status: OutOfSync & health: missing both in Argo GUI and object status
  • Fix issue - inconsistency:
    • Gets fixed running a --hard-refresh either by GUI or CLI
    • Similar sub-issue as case 2 happens, we have to run a --refresh again to have a fully synced and healthy Application

emirot pushed a commit to emirot/argo-cd that referenced this issue Jan 27, 2023
* feat: support multiple sources for application

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

remove debug logging and unwanted code

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

fix lint and unit test errors

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

fix lint and unit test errors

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

* fix bug introduced after rebase

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

executed make codegen

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

* rebase with master

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

remove unwanted logging

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

fix ci failures

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

* fix index out of bounds error

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

* ui fixes

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

* add revisions to SyncOperation for rollback

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

* change Source to *ApplicationSource in ApplicationSpec

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

* fix env variable read logic for ValueFiles

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

* update multiple sources doc

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

generate mock reposerverclient

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (argoproj#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (argoproj#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* path resolution tests (argoproj#12)

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

do things in better ways

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

consolidate

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

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

* add regex check for value of source.ref

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

* add webhook tests

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

change Source to *ApplicationSource in ApplicationSpec

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

address PR comments

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

rebase with master

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

Retrigger CI pipeline

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

rebased with master

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

* fix env variable read logic for ValueFiles

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

Address PR comments

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

* generate mock reposerverclient

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

* address comments

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (argoproj#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (argoproj#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* add regex check for value of source.ref

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

* Rebase with master

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

* Added unit tests (argoproj#15)

* add unit tests 1
* fix lint

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

* fix application parameters tab, rebased UI changes, tests

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

* More tests (argoproj#16)

* more tests

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

fix lint error

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

Test get ref sources (argoproj#17)

* test GetRefSources

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

* fix lint

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

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

GenerateManifests test (argoproj#18)

* GenerateManifests test

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

Fix broken tests (argoproj#19)

* fix broken tests

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

Symlink test (argoproj#20)

* check referenced sources for out-of-bounds symlinks

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

* unlock the values file repo before doing a symlink check (argoproj#22)

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

* multi source docs (argoproj#21)

* multi source docs

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

* fix warning title

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

* clarify

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

* clarify

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

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

* add e2e tests for multiple sources and fix UI lint (argoproj#23)

* add e2e tests for multiple sources and fix UI lint

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

* add auto-sync and hard refresh to e2e tests

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

* change refresh type to RefreshTypeNormal for e2e

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

* update e2e testcase with helm data

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

* add TestMultiSourceAppWithSourceOverride

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

* add missing yaml file

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

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

* rebase with master

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

* rebase with master

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

* fix lint

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

Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: emirot <[email protected]>
Aman1994 pushed a commit to Aman1994/k8id that referenced this issue Feb 1, 2023
Aman1994 pushed a commit to Obmondo/kubeaid that referenced this issue Feb 1, 2023
KlavsKlavsen pushed a commit to Obmondo/kubeaid that referenced this issue Feb 2, 2023
@lsoica
Copy link
Contributor

lsoica commented Feb 13, 2023

I've encountered the second issue mentioned here #2789 (comment) as well.
Are these issues tracked anywhere ?

@driosalido
Copy link

driosalido commented Feb 13, 2023

This is my experience using 2.6.1

Using #2789 as reference this is what I see:

1 - Same Issue. The application is deployed but no change of status until refresh
2 - Not exactly the same problem. ArgoCD pick the change and apply it, but not status change until refresh (no hard refresh needed)
3- Same behaviour
4 - Same issues with status and health until refresh , but the application was created with the the right values

@adrianmiron
Copy link

My main issue is it's not picking up changes from all the sources, a manual sync is needed.

@APemberton1
Copy link

Now we have multiple sources is there a way to use build env vars from the first source as part of the second. E.g.

  sources:
    - repoURL: https://github.com/xxxx/repo-1
      targetRevision: main
      ref: repo-1
    - repoURL: https://somecharturl.com
      targetRevision: 1.0.0
      chart: somechart
      helm:
        valueFiles:
          - $repo-1/values.yaml
         parameters:
         - name: image.tag
            value: ${ARGOCD_APP_REVISION}

ARGOCD_APP_REVISION resolves to 1.0.0 whereas I want it to resolve to the revision from repo-1 which I manage. Is there a way to do this currently? Would be very useful as part of automated image version tagging

schakrad pushed a commit to schakrad/argo-cd that referenced this issue Mar 14, 2023
* feat: support multiple sources for application

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

remove debug logging and unwanted code

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

fix lint and unit test errors

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

fix lint and unit test errors

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

* fix bug introduced after rebase

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

executed make codegen

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

* rebase with master

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

remove unwanted logging

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

fix ci failures

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

* fix index out of bounds error

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

* ui fixes

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

* add revisions to SyncOperation for rollback

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

* change Source to *ApplicationSource in ApplicationSpec

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

* fix env variable read logic for ValueFiles

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

* update multiple sources doc

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

generate mock reposerverclient

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (argoproj#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (argoproj#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* path resolution tests (argoproj#12)

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

do things in better ways

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

consolidate

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

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

* add regex check for value of source.ref

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

* add webhook tests

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

change Source to *ApplicationSource in ApplicationSpec

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

address PR comments

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

rebase with master

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

Retrigger CI pipeline

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

rebased with master

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

* fix env variable read logic for ValueFiles

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

Address PR comments

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

* generate mock reposerverclient

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

* address comments

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (argoproj#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (argoproj#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* add regex check for value of source.ref

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

* Rebase with master

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

* Added unit tests (argoproj#15)

* add unit tests 1
* fix lint

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

* fix application parameters tab, rebased UI changes, tests

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

* More tests (argoproj#16)

* more tests

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

fix lint error

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

Test get ref sources (argoproj#17)

* test GetRefSources

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

* fix lint

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

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

GenerateManifests test (argoproj#18)

* GenerateManifests test

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

Fix broken tests (argoproj#19)

* fix broken tests

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

Symlink test (argoproj#20)

* check referenced sources for out-of-bounds symlinks

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

* unlock the values file repo before doing a symlink check (argoproj#22)

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

* multi source docs (argoproj#21)

* multi source docs

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

* fix warning title

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

* clarify

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

* clarify

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

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

* add e2e tests for multiple sources and fix UI lint (argoproj#23)

* add e2e tests for multiple sources and fix UI lint

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

* add auto-sync and hard refresh to e2e tests

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

* change refresh type to RefreshTypeNormal for e2e

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

* update e2e testcase with helm data

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

* add TestMultiSourceAppWithSourceOverride

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

* add missing yaml file

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

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

* rebase with master

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

* rebase with master

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

* fix lint

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

Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>
crenshaw-dev added a commit that referenced this issue Mar 31, 2023
…ooltip for pod on resource tree (#11513)

* docs: Improve Keycloak documentation for command line sign-in (#8758)

Documenting what is discussed in #2932

Signed-off-by: Antoine Pultier <[email protected]>

Signed-off-by: Antoine Pultier <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Add CSI secret driver to the secret management options (#10900)

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

Signed-off-by: Duncan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump decode-uri-component from 0.2.0 to 0.2.2 in /ui (#11533)

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

* chore(deps-dev): bump @babel/preset-react from 7.7.0 to 7.18.6 in /ui (#11489)

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

* chore(deps-dev): bump babel-jest from 24.9.0 to 26.6.3 in /ui (#11483)

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

* docs: Added ARGOCD_ENV_ prefix to FOO (#11545)

Signed-off-by: Philip Haberkern <[email protected]>

Signed-off-by: Philip Haberkern <[email protected]>
Signed-off-by: schakrad <[email protected]>

* [Hack] Add concurrency to cluster creation in resource generator. (#11266)

* add threading to cluster creation

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

* Add default values

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

* Cleanup

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

* Move external dependency to internal

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

* ability to run cluster generation in parallel

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

* fix linter

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

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

* chore: Update notifications-engine dependency (#11479)

* chore: update `notifications-engine`

Signed-off-by: Jocelyn Thode <[email protected]>

* chore: generate notifications docs

Signed-off-by: Jocelyn Thode <[email protected]>

Signed-off-by: Jocelyn Thode <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/valyala/fasttemplate from 1.2.1 to 1.2.2 (#11552)

Bumps [github.com/valyala/fasttemplate](https://github.com/valyala/fasttemplate) from 1.2.1 to 1.2.2.
- [Release notes](https://github.com/valyala/fasttemplate/releases)
- [Commits](https://github.com/valyala/fasttemplate/compare/v1.2.1...v1.2.2)

---
updated-dependencies:
- dependency-name: github.com/valyala/fasttemplate
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/ktrysmt/go-bitbucket from 0.9.40 to 0.9.54 (#11554)

Bumps [github.com/ktrysmt/go-bitbucket](https://github.com/ktrysmt/go-bitbucket) from 0.9.40 to 0.9.54.
- [Release notes](https://github.com/ktrysmt/go-bitbucket/releases)
- [Commits](https://github.com/ktrysmt/go-bitbucket/compare/v0.9.40...v0.9.54)

---
updated-dependencies:
- dependency-name: github.com/ktrysmt/go-bitbucket
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/imdario/mergo from 0.3.12 to 0.3.13 (#11555)

Bumps [github.com/imdario/mergo](https://github.com/imdario/mergo) from 0.3.12 to 0.3.13.
- [Release notes](https://github.com/imdario/mergo/releases)
- [Commits](https://github.com/imdario/mergo/compare/0.3.12...v0.3.13)

---
updated-dependencies:
- dependency-name: github.com/imdario/mergo
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: update how to access arrays in Go templates (#11562)

Signed-off-by: Dieter Bocklandt <[email protected]>

Signed-off-by: Dieter Bocklandt <[email protected]>
Signed-off-by: schakrad <[email protected]>

* add otel interceptor (#11561)

Signed-off-by: minquan.chen <[email protected]>

Signed-off-by: minquan.chen <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: use repository GithubAppCreds proxy if set (#11422)

Signed-off-by: Nathanael Liechti <[email protected]>

Signed-off-by: Nathanael Liechti <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: show app age in application list view (#11209) (#11502)

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: sidebar css (#11531)

Signed-off-by: ashutosh16 <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Allow proxy to be saved when creating repoCreds (#11351) (#11425)

* fix: allow proxy to be saved in repoCreds (https + github app)

Signed-off-by: Nathanael Liechti <[email protected]>

* chore: changes from codegen

Signed-off-by: Nathanael Liechti <[email protected]>

* chore: add unit test for CreateRepoCreds

Signed-off-by: Nathanael Liechti <[email protected]>

Signed-off-by: Nathanael Liechti <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/go-openapi/runtime from 0.19.4 to 0.25.0 (#11568)

Bumps [github.com/go-openapi/runtime](https://github.com/go-openapi/runtime) from 0.19.4 to 0.25.0.
- [Release notes](https://github.com/go-openapi/runtime/releases)
- [Commits](https://github.com/go-openapi/runtime/compare/v0.19.4...v0.25.0)

---
updated-dependencies:
- dependency-name: github.com/go-openapi/runtime
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: Add skipCrds and ignoreMissingValueFiles to application.yaml example (#11565)

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

* chore: add Vinted to users list (#11214)

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

Signed-off-by: Edgaras <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump sigs.k8s.io/structured-merge-diff/v4 (#11580)

Bumps [sigs.k8s.io/structured-merge-diff/v4](https://github.com/kubernetes-sigs/structured-merge-diff) from 4.2.1 to 4.2.3.
- [Release notes](https://github.com/kubernetes-sigs/structured-merge-diff/releases)
- [Changelog](https://github.com/kubernetes-sigs/structured-merge-diff/blob/master/RELEASE.md)
- [Commits](https://github.com/kubernetes-sigs/structured-merge-diff/compare/v4.2.1...v4.2.3)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/structured-merge-diff/v4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/go-openapi/loads from 0.19.4 to 0.21.2 (#11569)

Bumps [github.com/go-openapi/loads](https://github.com/go-openapi/loads) from 0.19.4 to 0.21.2.
- [Release notes](https://github.com/go-openapi/loads/releases)
- [Commits](https://github.com/go-openapi/loads/compare/v0.19.4...v0.21.2)

---
updated-dependencies:
- dependency-name: github.com/go-openapi/loads
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* fix(helm): helm v3 doesn't have these flags (#11100) (#11540)

* fix: helm v3 doesn't have these flags

Signed-off-by: Alex Eftimie <[email protected]>

* Revert repoAdd change. Was to greedy, ca-file is needed there

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix(helm): login OCI Helm dependencies correctly (#8563) (#11327)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: misc css fixes for mobile (#5705) (#11508)

* Misc css fixes for mobile

Signed-off-by: Alex Eftimie <[email protected]>

* More fixes for mobile

Signed-off-by: Alex Eftimie <[email protected]>

* fix ui tests. bring back application status labels

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: added deep links backend changes (#11401)

* feat: added deep links backend changes

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

* fix: add rbac check to list links services

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

* feat: removed project param and updated sample config

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

* feat: update sample config

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

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

* chore(deps): bump github.com/Masterminds/semver/v3 from 3.1.1 to 3.2.0 (#11605)

Bumps [github.com/Masterminds/semver/v3](https://github.com/Masterminds/semver) from 3.1.1 to 3.2.0.
- [Release notes](https://github.com/Masterminds/semver/releases)
- [Changelog](https://github.com/Masterminds/semver/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Masterminds/semver/compare/v3.1.1...v3.2.0)

---
updated-dependencies:
- dependency-name: github.com/Masterminds/semver/v3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/go-redis/redis/v8 from 8.11.3 to 8.11.5 (#11606)

Bumps [github.com/go-redis/redis/v8](https://github.com/go-redis/redis) from 8.11.3 to 8.11.5.
- [Release notes](https://github.com/go-redis/redis/releases)
- [Changelog](https://github.com/go-redis/redis/blob/v8.11.5/CHANGELOG.md)
- [Commits](https://github.com/go-redis/redis/compare/v8.11.3...v8.11.5)

---
updated-dependencies:
- dependency-name: github.com/go-redis/redis/v8
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/casbin/casbin/v2 from 2.57.1 to 2.59.0 (#11607)

Bumps [github.com/casbin/casbin/v2](https://github.com/casbin/casbin) from 2.57.1 to 2.59.0.
- [Release notes](https://github.com/casbin/casbin/releases)
- [Changelog](https://github.com/casbin/casbin/blob/master/.releaserc.json)
- [Commits](https://github.com/casbin/casbin/compare/v2.57.1...v2.59.0)

---
updated-dependencies:
- dependency-name: github.com/casbin/casbin/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add Trusting Social users list (#11584)

* Add Trusting Social user

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

* fixing other

Signed-off-by: Cuong Nguyen Duc <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: set baseURL in github_app client (#11613)

Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: only add baseURL in github_app client for GHE client (#11622)

fixes #11613

Co-authored-by: crenshaw-dev <[email protected]>
Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Co-authored-by: crenshaw-dev <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.44.129 to 1.44.156 (#11629)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.129 to 1.44.156.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.44.129...v1.44.156)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* feat: Add support for proxy extensions (#11307)

* feat: Add support for proxy extensions

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

* return list of extensions

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

* add service set in argocd server struct

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

* implements cluster name lookup

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

* add cli docs

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

* apply connection config defaults

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

* add unit tests

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

* fix lint

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

* fix test

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

* Address review comments

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

* Add test for invalid extension name

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

* Address review comments

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

* Fix deadcode lint

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

* Fix unused lint

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

* Fix deadcode lint

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

* Better error message

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

* Fix deadcode lint

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

* Fix empty branch

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

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

* includeKinds for APIVersions in cluster info cache (#11241)

Signed-off-by: Roger Rumao <[email protected]>

Signed-off-by: Roger Rumao <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: add warning for user when using replace sync option (#11566)

* docs: adding warning to 'replace' sync option

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

* Update sync-options.md

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

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

* chore: Bump version in master to 2.6.0 (#11641)

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

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

* fix: Unknown sync operation state on app list page (#11621)

fix: Unknown sync operation state on app list page (#11621)

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

* docs: fix web terminal step list numbering (#11590)

docs: fix web terminal step list numbering (#11590)
Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add deny sources (#11639) (#11646)

This commit adds the ability to deny a source when it is prefixed with
`!`, in the same manner as with the "deny destinations" feature.

Fixes #11639.

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

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

* chore(deps): bump react-paginate from 6.5.0 to 8.1.4 in /ui (#11558)

Bumps [react-paginate](https://github.com/AdeleD/react-paginate) from 6.5.0 to 8.1.4.
- [Release notes](https://github.com/AdeleD/react-paginate/releases)
- [Changelog](https://github.com/AdeleD/react-paginate/blob/master/CHANGELOG.md)
- [Commits](https://github.com/AdeleD/react-paginate/compare/v6.5.0...v8.1.4)

---
updated-dependencies:
- dependency-name: react-paginate
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps-dev): bump tslint-react from 3.6.0 to 5.0.0 in /ui (#11559)

Bumps [tslint-react](https://github.com/palantir/tslint-react) from 3.6.0 to 5.0.0.
- [Release notes](https://github.com/palantir/tslint-react/releases)
- [Commits](https://github.com/palantir/tslint-react/compare/3.6.0...5.0.0)

---
updated-dependencies:
- dependency-name: tslint-react
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump express from 4.17.1 to 4.18.2 in /ui (#11591)

Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.18.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](https://github.com/expressjs/express/compare/4.17.1...4.18.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: kustomize has access to build environment (#11643)

Current docs reflect that the build environment is not available to kustomize. Since https://github.com/argoproj/argo-cd/pull/8096 it is now exposed for kustomize. This updates the kustomize section of the docs to reflect that.

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: pull request labels exposed in Pull Request generator template (#10204) (#11397)

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

Signed-off-by: maheshbaliga <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/casbin/casbin/v2 from 2.59.0 to 2.60.0 (#11656)

Bumps [github.com/casbin/casbin/v2](https://github.com/casbin/casbin) from 2.59.0 to 2.60.0.
- [Release notes](https://github.com/casbin/casbin/releases)
- [Changelog](https://github.com/casbin/casbin/blob/master/.releaserc.json)
- [Commits](https://github.com/casbin/casbin/compare/v2.59.0...v2.60.0)

---
updated-dependencies:
- dependency-name: github.com/casbin/casbin/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: fix flaky e2e test (#11509) (#11654)

* chore: fix flaky e2e test (#11509)

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

* don't centralize mock response - tests should be independent

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

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

* chore: downgrade React to v17 (#11653)

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

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

* chore(deps-dev): bump @types/cookie from 0.3.3 to 0.5.1 in /ui (#11659)

Bumps [@types/cookie](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/cookie) from 0.3.3 to 0.5.1.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/cookie)

---
updated-dependencies:
- dependency-name: "@types/cookie"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: fix flaky e2e test (#11670)

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

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

* [Bot] Update Snyk reports (#11649)

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

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

* feat: show cmpv2 plugins in create app wizard (#11615)

* feat: show cmpv2 plugins in create app wizard

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

* fix: fixed doc formatting and sidecar plugin info

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

* feat: updated plugin info

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

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

* chore(deps): bump github.com/ktrysmt/go-bitbucket from 0.9.54 to 0.9.55 (#11678)

Bumps [github.com/ktrysmt/go-bitbucket](https://github.com/ktrysmt/go-bitbucket) from 0.9.54 to 0.9.55.
- [Release notes](https://github.com/ktrysmt/go-bitbucket/releases)
- [Commits](https://github.com/ktrysmt/go-bitbucket/compare/v0.9.54...v0.9.55)

---
updated-dependencies:
- dependency-name: github.com/ktrysmt/go-bitbucket
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump actions/checkout from 3.1.0 to 3.2.0 (#11679)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3.1.0 to 3.2.0.
- [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/93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8...755da8c3cf115ac066823e79a1e1788f8940201b)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add Mercedes-Benz Tech Innovation to user list (#11682)

Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: appname in searchbar (#11493)

* fix: appname in searchbar

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

* fix: appname in searchbar

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

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

* fix: add pod age icon details in tooltip (#10290) (#11170)

* pod-age-icon details added in tooltip

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

* Tooltip change

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

Signed-off-by: schakradari <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: fix flaky e2e test for immutable fields (#11685)

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

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

* feat: expose deep links in UI (#11680)

Signed-off-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: Add Getyourguide to USERS.md (#11704)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: rollback react to known working version (#11703)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: pin kubectl version (#11726)

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

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

* docs: Clarification of the create namespace feature (#11723)

* docs: Clarification of the create namespace feature

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

* Address review suggestion

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

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

* chore: Add Cloud Scale to USERS.md (#11731)

* Update USERS.md

Add cloud scale

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

* Update USERS.md

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

* Update USERS.md

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

* Update USERS.md

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

* Update USERS.md

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

Signed-off-by: Nandita <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Allow Matrix generator to have two Git child generators without conflict (#10522) (#10523)

* misc: NoError instead of Error in repository test
Signed-off-by: Lobstrosity <[email protected]>

* misc: Extend lint timeout
Signed-off-by: Lobstrosity <[email protected]>

* feat: Add GitGenerator.PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* auto: Results of codegen
Signed-off-by: Lobstrosity <[email protected]>

* test: Add tests for PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* docs: Add notes to Matrix/Git generator docs about PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* misc: Undo unrelated test change
Signed-off-by: Lobstrosity <[email protected]>

* auto: Results of codegen
Signed-off-by: Lobstrosity <[email protected]>

* docs: Add detailed example
Signed-off-by: Lobstrosity <[email protected]>

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

* feat: ApplicationSet add `create-delete` policy #9101 (#11107)

* feat: ApplicationSet add create-delete policy

Signed-off-by: 久米 拓馬 <[email protected]>

* test for applicationSet policies

Signed-off-by: 久米 拓馬 <[email protected]>

* Update docs/operator-manual/applicationset/Controlling-Resource-Modification.md

Co-authored-by: Mubarak Jama <[email protected]>
Signed-off-by: Takuma Kume <[email protected]>

Signed-off-by: 久米 拓馬 <[email protected]>
Signed-off-by: Takuma Kume <[email protected]>
Co-authored-by: Mubarak Jama <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: upgrade helm to most recent version (v3.10.3) (#11725)

* chore: upgrade helm to most recent version (v3.10.3)

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

* Retrigger CI pipeline

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

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

* fix: Unbreak termination of operation with apps in other namespaces (#11239) (#11724)

* fix: Unbreak operation termination

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

* Revert change to Dockerfile

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

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

* fix: support relative links in OCI tags query response (#11708)

* fix: support relative links in OCI tags query response

Pagination for OCI tags retrieval is not supported when the
Link header URI is relative.
According to https://docs.docker.com/registry/spec/api/#pagination
and the therein referenced RFC
https://www.rfc-editor.org/rfc/rfc5988#section-5
relative links should be resolved to the initial request URL

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

* chore: clean up unused prints & assert errors

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

* fix: stop double-escaping repoURL

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

* chore: CodeQL CWE-117 log sanitizing

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

* chore: remove unnecessary error

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

Signed-off-by: detvdl <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: add Voyager Digital to USERS.md (#11735)

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

Signed-off-by: hopisaurus <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add google cloud source repo support (#7534) (#11618)

* feat: Add support for cloning Google Cloud Source repos (#7534)

* Google Cloud service account auth

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

* fix: Fill missing struct field (GCP SA key) in cli cmd

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

* fix(ui): Add proxy option when configuring Google Cloud Source repo

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

* fix: Remove secret (GCP SA key) in Get server req

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

* refactor: Do not use context.WithTimeout for Google creds

As the context is used in the background to refresh credentials, it
should not be cancelled.

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

* fix: Use proxy setting only in repo-service, not repocreds-service

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

* test: Create tests for GoogleCloudCreds

This commit refactors the implementation of GoogleCloudCreds in order to
make its methods testable.

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

* fix: Linting issues

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

* chore: Fix typo in docs.

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

* chore: Adjust url-allow-list for lint-docs action

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

* chore: Incorporate suggested refactorings

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

* Delete url-allow-list

Signed-off-by: Alex Eftimie <[email protected]>

* wrap errors

Signed-off-by: Alex Eftimie <[email protected]>

* More UI goodies and codegen

Signed-off-by: Alex Eftimie <[email protected]>

* Update docs screenshots

Signed-off-by: Alex Eftimie <[email protected]>

* move interface up next to other interfaces

Signed-off-by: Alex Eftimie <[email protected]>

* Reduce png size

Signed-off-by: Alex Eftimie <[email protected]>

* update generated

Signed-off-by: Alex Eftimie <[email protected]>

* fix whitespace from codegen

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: David Becher <[email protected]>
Signed-off-by: Alex Eftimie <[email protected]>
Co-authored-by: David Becher <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Document applications in any namespace (#10678)

* docs: Document applications in any namespace

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

* Fix some code blocks

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

* Fix link

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

* docs: Document applications in any namespace

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

* Fix some code blocks

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

* Fix link

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

* Apply reviewer comments

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

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

* feat: Multiple sources for applications (#2789) (#10432)

* feat: support multiple sources for application

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

remove debug logging and unwanted code

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

fix lint and unit test errors

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

fix lint and unit test errors

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

* fix bug introduced after rebase

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

executed make codegen

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

* rebase with master

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

remove unwanted logging

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

fix ci failures

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

* fix index out of bounds error

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

* ui fixes

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

* add revisions to SyncOperation for rollback

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

* change Source to *ApplicationSource in ApplicationSpec

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

* fix env variable read logic for ValueFiles

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

* update multiple sources doc

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

generate mock reposerverclient

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* path resolution tests (#12)

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

do things in better ways

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

consolidate

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

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

* add regex check for value of source.ref

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

* add webhook tests

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

change Source to *ApplicationSource in ApplicationSpec

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

address PR comments

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

rebase with master

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

Retrigger CI pipeline

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

rebased with master

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

* fix env variable read logic for ValueFiles

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

Address PR comments

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

* generate mock reposerverclient

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

* address comments

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* add regex check for value of source.ref

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

* Rebase with master

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

* Added unit tests (#15)

* add unit tests 1
* fix lint

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

* fix application parameters tab, rebased UI changes, tests

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

* More tests (#16)

* more tests

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

fix lint error

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

Test get ref sources (#17)

* test GetRefSources

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

* fix lint

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

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

GenerateManifests test (#18)

* GenerateManifests test

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

Fix broken tests (#19)

* fix broken tests

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

Symlink test (#20)

* check referenced sources for out-of-bounds symlinks

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

* unlock the values file repo before doing a symlink check (#22)

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

* multi source docs (#21)

* multi source docs

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

* fix warning title

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

* clarify

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

* clarify

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

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

* add e2e tests for multiple sources and fix UI lint (#23)

* add e2e tests for multiple sources and fix UI lint

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

* add auto-sync and hard refresh to e2e tests

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

* change refresh type to RefreshTypeNormal for e2e

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

* update e2e testcase with helm data

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

* add TestMultiSourceAppWithSourceOverride

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

* add missing yaml file

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

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

* rebase with master

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

* rebase with master

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

* fix lint

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

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

* feat: AppSet Progressive Rollouts with RollingSync (#9437) (#10048)

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

Signed-off-by: Matt Groot <[email protected]>
Co-authored-by: Matt Groot <[email protected]>
Signed-off-by: schakrad <[email protected]>

* [Bot] Update Snyk reports (#11739)

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

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

* feat: support Knative Serving 1.5 Custom Health Checks (#9719)

* fix: Update account.proto annotaion for gen grpc gateway

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

* fix: Changes from codegen

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

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* fix: Changes from codegen

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

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* Retrigger CI pipeline

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

* feat: support Knative Serving 1.5 Custom Health Checks

Signed-off-by: wei.wan <[email protected]>

* fix: fix test fail

Signed-off-by: wei.wan <[email protected]>

* feat: support Knative Serving 1.5 Custom Health Checks

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

Signed-off-by: wei840222 <[email protected]>
Signed-off-by: wei.wan <[email protected]>
Signed-off-by: wei <[email protected]>
Co-authored-by: wei.wan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Extend Cluster.cluster.x-k8s.io health check (#11705)

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

Signed-off-by: Jellyfrog <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add health checks for keptn resources (#11716)

* feat: add keptnappversion

Signed-off-by: Thomas Schuetz <[email protected]>

* feat: added keptn resource tests

Signed-off-by: Thomas Schuetz <[email protected]>

* feat: fix test for KeptnWorkloadInstance

Signed-off-by: Thomas Schuetz <[email protected]>

* fix: apiVersion and quotes

Signed-off-by: Thomas Schuetz <[email protected]>

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

* [Bot] Update Snyk reports (#11748)

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

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

* docs: add appset progressive rollout strategy proposal (#9979)

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

Signed-off-by: wmgroot <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: update cosign docs (#11749)

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

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

* chore: upgrade git-url-parse to avoid CVE-2022-2900 (#11744)

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

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

* chore: upgrade minimatch to avoid CVE-2022-3517 (#11745)

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

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

* chore: add kustomize project for testing param CMP locally (#11265)

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

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

* chore: upgrade go-oidc (#11579)

* chore: upgrade go-oidc

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

* take advantage of new error type

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

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

* fix: sign container images by digest (#11151)

* chore: sign container images by digest

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

* use sha hash

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

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

* doc: correct kustomize demo path (#11762)

Signed-off-by: Yixing Yan <[email protected]>

Signed-off-by: Yixing Yan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump actions/setup-go from 3.4.0 to 3.5.0 (#11697)

Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3.4.0 to 3.5.0.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](https://github.com/actions/setup-go/compare/d0a58c1c4d2b25278816e339b944508c875f3613...6edd4406fa81c3da01a34fa6f6343087c207a568)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add OpsMx to USERS.md (#11765)

adding our company name to Argo CD users.

Signed-off-by: Balaji Siva <[email protected]>

Signed-off-by: Balaji Siva <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: clarify that all labels must exist (#11693)

It's unclear if all or any of the labels need to exist. This clarifies that all of the labels must exist.

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: ssa e2e tests failing after updating to kubectl 1.26 (#11753)

* fix: ssa e2e test failing after updating to kubectl 1.26

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

* Remove pinned kubectl version

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

* Cleaner approach to fix e2e test

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

* Fix

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

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

* docs: Update example dockerfile (#11721)

The latest tag hasn't been updated in almost a year, and as a result, the ubuntu repositories are out of date and are throwing errors. This updates the example to use a fixed version, which are updated much more frequently.

Signed-off-by: Phil Wright- Christie <[email protected]>

Signed-off-by: Phil Wright- Christie <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: correct SSO configuration URL in example configmap (#11720)

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

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

* chore(deps): bump github.com/itchyny/gojq from 0.12.9 to 0.12.10 (#11677)

Bumps [github.com/itchyny/gojq](https://github.com/itchyny/gojq) from 0.12.9 to 0.12.10.
- [Release notes](https://github.com/itchyny/gojq/releases)
- [Changelog](https://github.com/itchyny/gojq/blob/main/CHANGELOG.md)
- [Commits](https://github.com/itchyny/gojq/compare/v0.12.9...v0.12.10)

---
updated-dependencies:
- dependency-name: github.com/itchyny/gojq
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: bump elasticsearch version to 8.5.1 (#11771)

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

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

* chore: change logging level to Debug (#11773)

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

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

* fix: support for enabling progressive rollouts from `argocd-cmd-params-cm` (#11776)

* fix(applicationset): use consistent syntax for env vars

Signed-off-by: Nicholas Morey <[email protected]>

* fix(manifests): add new appset env var from configmap

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: sources.ref allow hyphen and underscore (#11775)

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

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

* docs: clarify project destination possibilities (#11706)

Clarify that it's possible to reference clusters by `cluster` or by `name`.

Signed-off-by: Gaël Jourdan-Weil <[email protected]>

Signed-off-by: Gaël Jourdan-Weil <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/fsnotify/fsnotify from 1.5.1 to 1.6.0 (#11553)

Bumps [github.com/fsnotify/fsnotify](https://github.com/fsnotify/fsnotify) from 1.5.1 to 1.6.0.
- [Release notes](https://github.com/fsnotify/fsnotify/releases)
- [Changelog](https://github.com/fsnotify/fsnotify/blob/main/CHANGELOG.md)
- [Commits](https://github.com/fsnotify/fsnotify/compare/v1.5.1...v1.6.0)

---
updated-dependencies:
- dependency-name: github.com/fsnotify/fsnotify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: get image digest in seperate step (#11778)

* chore: get image digest in seperate step

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

* Retrigger CI pipeline

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

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

* chore: fix lint error (#11788)

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

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

* chore(deps): bump golang.org/x/term from 0.1.0 to 0.3.0 (#11792)

Bumps [golang.org/x/term](https://github.com/golang/term) from 0.1.0 to 0.3.0.
- [Release notes](https://github.com/golang/term/releases)
- [Commits](https://github.com/golang/term/compare/v0.1.0...v0.3.0)

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

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/aws/aws-sdk-go from 1.44.156 to 1.44.164 (#11791)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.156 to 1.44.164.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.44.156...v1.44.164)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* fix: set Path as empty if path is not specified for a source in multiple sources (#11756) (#11774)

* set Path as '' if path is not specified for a source in multiple sources

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

* update check for not setting value of path

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

* cleanup

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

* address comments

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

* fix lint

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

* fix lint

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

* Update ui/src/app/shared/components/revision.tsx

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

Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: Ishita Sequeira <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Update bullet formatting on Progressive Rollouts.md (#11777)

The bullet list in the example format was rendering inline in the paragraph on the doc site rather than showing a bulleted list. This also makes the rest of the doc follow the same convention.

Signed-off-by: Chris Reilly <[email protected]>

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

* fix: disable rollback button for apps with multiple sources (#11785)

* disble rollback button for apps with multiple sources

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

* fix lint errors

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

* disble rollback button for apps with multiple sources

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

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

* ci: enforce semantic PR title (#11779)

* ci: enforce semantic PR title

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

* concurrency limit

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

* remove scopes

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

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

* fix: web terminal outside argocd namespace (#11166) (#11400)

* fix: web terminal outside argocd namespace (#11166)

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

* reorganize

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

* fix reference

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

* move things around, fix stuff maybe

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

* tests

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

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

* chore(deps): bump actions/cache from 3.0.11 to 3.2.0 (#11809)

Bumps [actions/cache](https://github.com/actions/cache) from 3.0.11 to 3.2.0.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/9b0c1fce7a93df8e3bb8926b0d6e9d89e92f20a7...c17f4bf4666a8001b1a45c09eb7a485c41aa64c3)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

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

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

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

* fix: Applications with suspended jobs now marked "Suspended" instead of "Progressing" (#11603) (#11626)

* fix: add suspended condition

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

* fix: add suspended condition

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

* Update go.sum

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

* fix: add suspended condition

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

* fix: add suspended condition

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

* Update go.sum

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

* upgrade notes for 2.6

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

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

* fix: Application's own link in UI (#11123) (#11124)

Signed-off-by: Alex Eftimie <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: ui cluster server url overlaps (#11873)

Signed-off-by: Jiwon Kim <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: set cluster command (#9996)

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

Signed-off-by: maheshbaliga <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: inversion selection support for the reso…
@vl-kp
Copy link

vl-kp commented Apr 15, 2023

Now we have multiple sources is there a way to use build env vars from the first source as part of the second. E.g.

  sources:
    - repoURL: https://github.com/xxxx/repo-1
      targetRevision: main
      ref: repo-1
    - repoURL: https://somecharturl.com
      targetRevision: 1.0.0
      chart: somechart
      helm:
        valueFiles:
          - $repo-1/values.yaml
         parameters:
         - name: image.tag
            value: ${ARGOCD_APP_REVISION}

ARGOCD_APP_REVISION resolves to 1.0.0 whereas I want it to resolve to the revision from repo-1 which I manage. Is there a way to do this currently? Would be very useful as part of automated image version tagging

anyway to ref the key/values for the parameters

@crenshaw-dev
Copy link
Member

Accessing the build environment of another source is currently not supported. Would you mind opening a new enhancement request?

@APemberton1
Copy link

Raised enhaccement request: #14855

yyzxw pushed a commit to yyzxw/argo-cd that referenced this issue Aug 9, 2023
…ooltip for pod on resource tree (argoproj#11513)

* docs: Improve Keycloak documentation for command line sign-in (#8758)

Documenting what is discussed in #2932

Signed-off-by: Antoine Pultier <[email protected]>

Signed-off-by: Antoine Pultier <[email protected]>
Co-authored-by: pasha-codefresh <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Add CSI secret driver to the secret management options (#10900)

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

Signed-off-by: Duncan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump decode-uri-component from 0.2.0 to 0.2.2 in /ui (#11533)

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

* chore(deps-dev): bump @babel/preset-react from 7.7.0 to 7.18.6 in /ui (#11489)

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

* chore(deps-dev): bump babel-jest from 24.9.0 to 26.6.3 in /ui (#11483)

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

* docs: Added ARGOCD_ENV_ prefix to FOO (#11545)

Signed-off-by: Philip Haberkern <[email protected]>

Signed-off-by: Philip Haberkern <[email protected]>
Signed-off-by: schakrad <[email protected]>

* [Hack] Add concurrency to cluster creation in resource generator. (#11266)

* add threading to cluster creation

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

* Add default values

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

* Cleanup

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

* Move external dependency to internal

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

* ability to run cluster generation in parallel

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

* fix linter

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

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

* chore: Update notifications-engine dependency (#11479)

* chore: update `notifications-engine`

Signed-off-by: Jocelyn Thode <[email protected]>

* chore: generate notifications docs

Signed-off-by: Jocelyn Thode <[email protected]>

Signed-off-by: Jocelyn Thode <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/valyala/fasttemplate from 1.2.1 to 1.2.2 (#11552)

Bumps [github.com/valyala/fasttemplate](https://github.com/valyala/fasttemplate) from 1.2.1 to 1.2.2.
- [Release notes](https://github.com/valyala/fasttemplate/releases)
- [Commits](https://github.com/valyala/fasttemplate/compare/v1.2.1...v1.2.2)

---
updated-dependencies:
- dependency-name: github.com/valyala/fasttemplate
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/ktrysmt/go-bitbucket from 0.9.40 to 0.9.54 (#11554)

Bumps [github.com/ktrysmt/go-bitbucket](https://github.com/ktrysmt/go-bitbucket) from 0.9.40 to 0.9.54.
- [Release notes](https://github.com/ktrysmt/go-bitbucket/releases)
- [Commits](https://github.com/ktrysmt/go-bitbucket/compare/v0.9.40...v0.9.54)

---
updated-dependencies:
- dependency-name: github.com/ktrysmt/go-bitbucket
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/imdario/mergo from 0.3.12 to 0.3.13 (#11555)

Bumps [github.com/imdario/mergo](https://github.com/imdario/mergo) from 0.3.12 to 0.3.13.
- [Release notes](https://github.com/imdario/mergo/releases)
- [Commits](https://github.com/imdario/mergo/compare/0.3.12...v0.3.13)

---
updated-dependencies:
- dependency-name: github.com/imdario/mergo
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: update how to access arrays in Go templates (#11562)

Signed-off-by: Dieter Bocklandt <[email protected]>

Signed-off-by: Dieter Bocklandt <[email protected]>
Signed-off-by: schakrad <[email protected]>

* add otel interceptor (#11561)

Signed-off-by: minquan.chen <[email protected]>

Signed-off-by: minquan.chen <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: use repository GithubAppCreds proxy if set (#11422)

Signed-off-by: Nathanael Liechti <[email protected]>

Signed-off-by: Nathanael Liechti <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: show app age in application list view (#11209) (#11502)

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: sidebar css (#11531)

Signed-off-by: ashutosh16 <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Allow proxy to be saved when creating repoCreds (#11351) (#11425)

* fix: allow proxy to be saved in repoCreds (https + github app)

Signed-off-by: Nathanael Liechti <[email protected]>

* chore: changes from codegen

Signed-off-by: Nathanael Liechti <[email protected]>

* chore: add unit test for CreateRepoCreds

Signed-off-by: Nathanael Liechti <[email protected]>

Signed-off-by: Nathanael Liechti <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/go-openapi/runtime from 0.19.4 to 0.25.0 (#11568)

Bumps [github.com/go-openapi/runtime](https://github.com/go-openapi/runtime) from 0.19.4 to 0.25.0.
- [Release notes](https://github.com/go-openapi/runtime/releases)
- [Commits](https://github.com/go-openapi/runtime/compare/v0.19.4...v0.25.0)

---
updated-dependencies:
- dependency-name: github.com/go-openapi/runtime
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: Add skipCrds and ignoreMissingValueFiles to application.yaml example (#11565)

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

* chore: add Vinted to users list (#11214)

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

Signed-off-by: Edgaras <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump sigs.k8s.io/structured-merge-diff/v4 (#11580)

Bumps [sigs.k8s.io/structured-merge-diff/v4](https://github.com/kubernetes-sigs/structured-merge-diff) from 4.2.1 to 4.2.3.
- [Release notes](https://github.com/kubernetes-sigs/structured-merge-diff/releases)
- [Changelog](https://github.com/kubernetes-sigs/structured-merge-diff/blob/master/RELEASE.md)
- [Commits](https://github.com/kubernetes-sigs/structured-merge-diff/compare/v4.2.1...v4.2.3)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/structured-merge-diff/v4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/go-openapi/loads from 0.19.4 to 0.21.2 (#11569)

Bumps [github.com/go-openapi/loads](https://github.com/go-openapi/loads) from 0.19.4 to 0.21.2.
- [Release notes](https://github.com/go-openapi/loads/releases)
- [Commits](https://github.com/go-openapi/loads/compare/v0.19.4...v0.21.2)

---
updated-dependencies:
- dependency-name: github.com/go-openapi/loads
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* fix(helm): helm v3 doesn't have these flags (#11100) (#11540)

* fix: helm v3 doesn't have these flags

Signed-off-by: Alex Eftimie <[email protected]>

* Revert repoAdd change. Was to greedy, ca-file is needed there

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix(helm): login OCI Helm dependencies correctly (#8563) (#11327)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: misc css fixes for mobile (#5705) (#11508)

* Misc css fixes for mobile

Signed-off-by: Alex Eftimie <[email protected]>

* More fixes for mobile

Signed-off-by: Alex Eftimie <[email protected]>

* fix ui tests. bring back application status labels

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: added deep links backend changes (#11401)

* feat: added deep links backend changes

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

* fix: add rbac check to list links services

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

* feat: removed project param and updated sample config

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

* feat: update sample config

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

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

* chore(deps): bump github.com/Masterminds/semver/v3 from 3.1.1 to 3.2.0 (#11605)

Bumps [github.com/Masterminds/semver/v3](https://github.com/Masterminds/semver) from 3.1.1 to 3.2.0.
- [Release notes](https://github.com/Masterminds/semver/releases)
- [Changelog](https://github.com/Masterminds/semver/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Masterminds/semver/compare/v3.1.1...v3.2.0)

---
updated-dependencies:
- dependency-name: github.com/Masterminds/semver/v3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/go-redis/redis/v8 from 8.11.3 to 8.11.5 (#11606)

Bumps [github.com/go-redis/redis/v8](https://github.com/go-redis/redis) from 8.11.3 to 8.11.5.
- [Release notes](https://github.com/go-redis/redis/releases)
- [Changelog](https://github.com/go-redis/redis/blob/v8.11.5/CHANGELOG.md)
- [Commits](https://github.com/go-redis/redis/compare/v8.11.3...v8.11.5)

---
updated-dependencies:
- dependency-name: github.com/go-redis/redis/v8
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/casbin/casbin/v2 from 2.57.1 to 2.59.0 (#11607)

Bumps [github.com/casbin/casbin/v2](https://github.com/casbin/casbin) from 2.57.1 to 2.59.0.
- [Release notes](https://github.com/casbin/casbin/releases)
- [Changelog](https://github.com/casbin/casbin/blob/master/.releaserc.json)
- [Commits](https://github.com/casbin/casbin/compare/v2.57.1...v2.59.0)

---
updated-dependencies:
- dependency-name: github.com/casbin/casbin/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add Trusting Social users list (#11584)

* Add Trusting Social user

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

* fixing other

Signed-off-by: Cuong Nguyen Duc <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: set baseURL in github_app client (#11613)

Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: only add baseURL in github_app client for GHE client (#11622)

fixes #11613

Co-authored-by: crenshaw-dev <[email protected]>
Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Co-authored-by: crenshaw-dev <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.44.129 to 1.44.156 (#11629)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.129 to 1.44.156.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.44.129...v1.44.156)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* feat: Add support for proxy extensions (#11307)

* feat: Add support for proxy extensions

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

* return list of extensions

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

* add service set in argocd server struct

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

* implements cluster name lookup

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

* add cli docs

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

* apply connection config defaults

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

* add unit tests

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

* fix lint

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

* fix test

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

* Address review comments

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

* Add test for invalid extension name

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

* Address review comments

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

* Fix deadcode lint

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

* Fix unused lint

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

* Fix deadcode lint

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

* Better error message

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

* Fix deadcode lint

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

* Fix empty branch

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

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

* includeKinds for APIVersions in cluster info cache (#11241)

Signed-off-by: Roger Rumao <[email protected]>

Signed-off-by: Roger Rumao <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: add warning for user when using replace sync option (#11566)

* docs: adding warning to 'replace' sync option

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

* Update sync-options.md

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

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

* chore: Bump version in master to 2.6.0 (#11641)

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

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

* fix: Unknown sync operation state on app list page (#11621)

fix: Unknown sync operation state on app list page (#11621)

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

* docs: fix web terminal step list numbering (#11590)

docs: fix web terminal step list numbering (#11590)
Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add deny sources (#11639) (#11646)

This commit adds the ability to deny a source when it is prefixed with
`!`, in the same manner as with the "deny destinations" feature.

Fixes #11639.

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

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

* chore(deps): bump react-paginate from 6.5.0 to 8.1.4 in /ui (#11558)

Bumps [react-paginate](https://github.com/AdeleD/react-paginate) from 6.5.0 to 8.1.4.
- [Release notes](https://github.com/AdeleD/react-paginate/releases)
- [Changelog](https://github.com/AdeleD/react-paginate/blob/master/CHANGELOG.md)
- [Commits](https://github.com/AdeleD/react-paginate/compare/v6.5.0...v8.1.4)

---
updated-dependencies:
- dependency-name: react-paginate
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps-dev): bump tslint-react from 3.6.0 to 5.0.0 in /ui (#11559)

Bumps [tslint-react](https://github.com/palantir/tslint-react) from 3.6.0 to 5.0.0.
- [Release notes](https://github.com/palantir/tslint-react/releases)
- [Commits](https://github.com/palantir/tslint-react/compare/3.6.0...5.0.0)

---
updated-dependencies:
- dependency-name: tslint-react
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump express from 4.17.1 to 4.18.2 in /ui (#11591)

Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.18.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](https://github.com/expressjs/express/compare/4.17.1...4.18.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: kustomize has access to build environment (#11643)

Current docs reflect that the build environment is not available to kustomize. Since https://github.com/argoproj/argo-cd/pull/8096 it is now exposed for kustomize. This updates the kustomize section of the docs to reflect that.

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: pull request labels exposed in Pull Request generator template (#10204) (#11397)

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

Signed-off-by: maheshbaliga <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/casbin/casbin/v2 from 2.59.0 to 2.60.0 (#11656)

Bumps [github.com/casbin/casbin/v2](https://github.com/casbin/casbin) from 2.59.0 to 2.60.0.
- [Release notes](https://github.com/casbin/casbin/releases)
- [Changelog](https://github.com/casbin/casbin/blob/master/.releaserc.json)
- [Commits](https://github.com/casbin/casbin/compare/v2.59.0...v2.60.0)

---
updated-dependencies:
- dependency-name: github.com/casbin/casbin/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: fix flaky e2e test (#11509) (#11654)

* chore: fix flaky e2e test (#11509)

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

* don't centralize mock response - tests should be independent

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

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

* chore: downgrade React to v17 (#11653)

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

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

* chore(deps-dev): bump @types/cookie from 0.3.3 to 0.5.1 in /ui (#11659)

Bumps [@types/cookie](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/cookie) from 0.3.3 to 0.5.1.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/cookie)

---
updated-dependencies:
- dependency-name: "@types/cookie"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: fix flaky e2e test (#11670)

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

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

* [Bot] Update Snyk reports (#11649)

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

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

* feat: show cmpv2 plugins in create app wizard (#11615)

* feat: show cmpv2 plugins in create app wizard

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

* fix: fixed doc formatting and sidecar plugin info

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

* feat: updated plugin info

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

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

* chore(deps): bump github.com/ktrysmt/go-bitbucket from 0.9.54 to 0.9.55 (#11678)

Bumps [github.com/ktrysmt/go-bitbucket](https://github.com/ktrysmt/go-bitbucket) from 0.9.54 to 0.9.55.
- [Release notes](https://github.com/ktrysmt/go-bitbucket/releases)
- [Commits](https://github.com/ktrysmt/go-bitbucket/compare/v0.9.54...v0.9.55)

---
updated-dependencies:
- dependency-name: github.com/ktrysmt/go-bitbucket
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump actions/checkout from 3.1.0 to 3.2.0 (#11679)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3.1.0 to 3.2.0.
- [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/93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8...755da8c3cf115ac066823e79a1e1788f8940201b)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add Mercedes-Benz Tech Innovation to user list (#11682)

Signed-off-by: Marco Lecheler <[email protected]>

Signed-off-by: Marco Lecheler <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: appname in searchbar (#11493)

* fix: appname in searchbar

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

* fix: appname in searchbar

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

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

* fix: add pod age icon details in tooltip (#10290) (#11170)

* pod-age-icon details added in tooltip

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

* Tooltip change

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

Signed-off-by: schakradari <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: fix flaky e2e test for immutable fields (#11685)

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

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

* feat: expose deep links in UI (#11680)

Signed-off-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: Add Getyourguide to USERS.md (#11704)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: rollback react to known working version (#11703)

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: Alex Eftimie <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: pin kubectl version (#11726)

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

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

* docs: Clarification of the create namespace feature (#11723)

* docs: Clarification of the create namespace feature

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

* Address review suggestion

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

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

* chore: Add Cloud Scale to USERS.md (#11731)

* Update USERS.md

Add cloud scale

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

* Update USERS.md

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

* Update USERS.md

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

* Update USERS.md

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

* Update USERS.md

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

Signed-off-by: Nandita <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Allow Matrix generator to have two Git child generators without conflict (#10522) (#10523)

* misc: NoError instead of Error in repository test
Signed-off-by: Lobstrosity <[email protected]>

* misc: Extend lint timeout
Signed-off-by: Lobstrosity <[email protected]>

* feat: Add GitGenerator.PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* auto: Results of codegen
Signed-off-by: Lobstrosity <[email protected]>

* test: Add tests for PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* docs: Add notes to Matrix/Git generator docs about PathParamPrefix
Signed-off-by: Lobstrosity <[email protected]>

* misc: Undo unrelated test change
Signed-off-by: Lobstrosity <[email protected]>

* auto: Results of codegen
Signed-off-by: Lobstrosity <[email protected]>

* docs: Add detailed example
Signed-off-by: Lobstrosity <[email protected]>

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

* feat: ApplicationSet add `create-delete` policy #9101 (#11107)

* feat: ApplicationSet add create-delete policy

Signed-off-by: 久米 拓馬 <[email protected]>

* test for applicationSet policies

Signed-off-by: 久米 拓馬 <[email protected]>

* Update docs/operator-manual/applicationset/Controlling-Resource-Modification.md

Co-authored-by: Mubarak Jama <[email protected]>
Signed-off-by: Takuma Kume <[email protected]>

Signed-off-by: 久米 拓馬 <[email protected]>
Signed-off-by: Takuma Kume <[email protected]>
Co-authored-by: Mubarak Jama <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: upgrade helm to most recent version (v3.10.3) (#11725)

* chore: upgrade helm to most recent version (v3.10.3)

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

* Retrigger CI pipeline

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

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

* fix: Unbreak termination of operation with apps in other namespaces (#11239) (#11724)

* fix: Unbreak operation termination

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

* Revert change to Dockerfile

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

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

* fix: support relative links in OCI tags query response (#11708)

* fix: support relative links in OCI tags query response

Pagination for OCI tags retrieval is not supported when the
Link header URI is relative.
According to https://docs.docker.com/registry/spec/api/#pagination
and the therein referenced RFC
https://www.rfc-editor.org/rfc/rfc5988#section-5
relative links should be resolved to the initial request URL

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

* chore: clean up unused prints & assert errors

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

* fix: stop double-escaping repoURL

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

* chore: CodeQL CWE-117 log sanitizing

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

* chore: remove unnecessary error

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

Signed-off-by: detvdl <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore: add Voyager Digital to USERS.md (#11735)

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

Signed-off-by: hopisaurus <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add google cloud source repo support (#7534) (#11618)

* feat: Add support for cloning Google Cloud Source repos (#7534)

* Google Cloud service account auth

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

* fix: Fill missing struct field (GCP SA key) in cli cmd

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

* fix(ui): Add proxy option when configuring Google Cloud Source repo

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

* fix: Remove secret (GCP SA key) in Get server req

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

* refactor: Do not use context.WithTimeout for Google creds

As the context is used in the background to refresh credentials, it
should not be cancelled.

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

* fix: Use proxy setting only in repo-service, not repocreds-service

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

* test: Create tests for GoogleCloudCreds

This commit refactors the implementation of GoogleCloudCreds in order to
make its methods testable.

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

* fix: Linting issues

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

* chore: Fix typo in docs.

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

* chore: Adjust url-allow-list for lint-docs action

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

* chore: Incorporate suggested refactorings

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

* Delete url-allow-list

Signed-off-by: Alex Eftimie <[email protected]>

* wrap errors

Signed-off-by: Alex Eftimie <[email protected]>

* More UI goodies and codegen

Signed-off-by: Alex Eftimie <[email protected]>

* Update docs screenshots

Signed-off-by: Alex Eftimie <[email protected]>

* move interface up next to other interfaces

Signed-off-by: Alex Eftimie <[email protected]>

* Reduce png size

Signed-off-by: Alex Eftimie <[email protected]>

* update generated

Signed-off-by: Alex Eftimie <[email protected]>

* fix whitespace from codegen

Signed-off-by: Alex Eftimie <[email protected]>

Signed-off-by: David Becher <[email protected]>
Signed-off-by: Alex Eftimie <[email protected]>
Co-authored-by: David Becher <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Document applications in any namespace (#10678)

* docs: Document applications in any namespace

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

* Fix some code blocks

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

* Fix link

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

* docs: Document applications in any namespace

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

* Fix some code blocks

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

* Fix link

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

* Apply reviewer comments

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

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

* feat: Multiple sources for applications (#2789) (#10432)

* feat: support multiple sources for application

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

remove debug logging and unwanted code

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

fix lint and unit test errors

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

fix lint and unit test errors

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

* fix bug introduced after rebase

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

executed make codegen

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

* rebase with master

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

remove unwanted logging

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

fix ci failures

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

* fix index out of bounds error

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

* ui fixes

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

* add revisions to SyncOperation for rollback

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

* change Source to *ApplicationSource in ApplicationSpec

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

* fix env variable read logic for ValueFiles

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

* update multiple sources doc

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

generate mock reposerverclient

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* path resolution tests (#12)

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

do things in better ways

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

consolidate

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

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

* add regex check for value of source.ref

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

* add webhook tests

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

change Source to *ApplicationSource in ApplicationSpec

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

address PR comments

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

rebase with master

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

Retrigger CI pipeline

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

rebased with master

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

* fix env variable read logic for ValueFiles

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

Address PR comments

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

* add repository lock and checkout target revision

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

fix codegen

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

* checkout all sources before generating manifest

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

* generate mock reposerverclient

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

* address comments

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

* update logic for returning ManifestResponse to avoid nil pointer issues

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

* fix nil reference and key mismatch bugs; add more logs (#6)

* fix nil reference and key mismatch bugs; add more logs
* remove temporary comment
* addressed the lint failure and added chart to RefTargeRevisionMapping
* normalize git repo (#7)
* do not leak lock releases
* prevent deadlock
* allow spec update
* move settings fetch outside loop
* cache busing
* return err instead of logging it
* no caching in test
* fix cache key marshaling

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

Rebase with master

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

* update grpc field numbers

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

* add regex check for value of source.ref

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

* Rebase with master

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

* Added unit tests (#15)

* add unit tests 1
* fix lint

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

* fix application parameters tab, rebased UI changes, tests

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

* More tests (#16)

* more tests

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

fix lint error

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

Test get ref sources (#17)

* test GetRefSources

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

* fix lint

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

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

GenerateManifests test (#18)

* GenerateManifests test

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

Fix broken tests (#19)

* fix broken tests

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

Symlink test (#20)

* check referenced sources for out-of-bounds symlinks

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

* unlock the values file repo before doing a symlink check (#22)

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

* multi source docs (#21)

* multi source docs

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

* fix warning title

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

* clarify

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

* clarify

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

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

* add e2e tests for multiple sources and fix UI lint (#23)

* add e2e tests for multiple sources and fix UI lint

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

* add auto-sync and hard refresh to e2e tests

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

* change refresh type to RefreshTypeNormal for e2e

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

* update e2e testcase with helm data

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

* add TestMultiSourceAppWithSourceOverride

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

* add missing yaml file

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

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

* rebase with master

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

* rebase with master

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

* fix lint

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

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

* feat: AppSet Progressive Rollouts with RollingSync (#9437) (#10048)

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

Signed-off-by: Matt Groot <[email protected]>
Co-authored-by: Matt Groot <[email protected]>
Signed-off-by: schakrad <[email protected]>

* [Bot] Update Snyk reports (#11739)

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

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

* feat: support Knative Serving 1.5 Custom Health Checks (#9719)

* fix: Update account.proto annotaion for gen grpc gateway

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

* fix: Changes from codegen

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

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* fix: Changes from codegen

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

* Retrigger CI pipeline

Signed-off-by: wei.wan <[email protected]>

* Retrigger CI pipeline

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

* feat: support Knative Serving 1.5 Custom Health Checks

Signed-off-by: wei.wan <[email protected]>

* fix: fix test fail

Signed-off-by: wei.wan <[email protected]>

* feat: support Knative Serving 1.5 Custom Health Checks

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

Signed-off-by: wei840222 <[email protected]>
Signed-off-by: wei.wan <[email protected]>
Signed-off-by: wei <[email protected]>
Co-authored-by: wei.wan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: Extend Cluster.cluster.x-k8s.io health check (#11705)

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

Signed-off-by: Jellyfrog <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: add health checks for keptn resources (#11716)

* feat: add keptnappversion

Signed-off-by: Thomas Schuetz <[email protected]>

* feat: added keptn resource tests

Signed-off-by: Thomas Schuetz <[email protected]>

* feat: fix test for KeptnWorkloadInstance

Signed-off-by: Thomas Schuetz <[email protected]>

* fix: apiVersion and quotes

Signed-off-by: Thomas Schuetz <[email protected]>

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

* [Bot] Update Snyk reports (#11748)

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

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

* docs: add appset progressive rollout strategy proposal (#9979)

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

Signed-off-by: wmgroot <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: update cosign docs (#11749)

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

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

* chore: upgrade git-url-parse to avoid CVE-2022-2900 (#11744)

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

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

* chore: upgrade minimatch to avoid CVE-2022-3517 (#11745)

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

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

* chore: add kustomize project for testing param CMP locally (#11265)

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

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

* chore: upgrade go-oidc (#11579)

* chore: upgrade go-oidc

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

* take advantage of new error type

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

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

* fix: sign container images by digest (#11151)

* chore: sign container images by digest

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

* use sha hash

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

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

* doc: correct kustomize demo path (#11762)

Signed-off-by: Yixing Yan <[email protected]>

Signed-off-by: Yixing Yan <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump actions/setup-go from 3.4.0 to 3.5.0 (#11697)

Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3.4.0 to 3.5.0.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](https://github.com/actions/setup-go/compare/d0a58c1c4d2b25278816e339b944508c875f3613...6edd4406fa81c3da01a34fa6f6343087c207a568)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: add OpsMx to USERS.md (#11765)

adding our company name to Argo CD users.

Signed-off-by: Balaji Siva <[email protected]>

Signed-off-by: Balaji Siva <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: clarify that all labels must exist (#11693)

It's unclear if all or any of the labels need to exist. This clarifies that all of the labels must exist.

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: ssa e2e tests failing after updating to kubectl 1.26 (#11753)

* fix: ssa e2e test failing after updating to kubectl 1.26

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

* Remove pinned kubectl version

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

* Cleaner approach to fix e2e test

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

* Fix

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

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

* docs: Update example dockerfile (#11721)

The latest tag hasn't been updated in almost a year, and as a result, the ubuntu repositories are out of date and are throwing errors. This updates the example to use a fixed version, which are updated much more frequently.

Signed-off-by: Phil Wright- Christie <[email protected]>

Signed-off-by: Phil Wright- Christie <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: correct SSO configuration URL in example configmap (#11720)

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

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

* chore(deps): bump github.com/itchyny/gojq from 0.12.9 to 0.12.10 (#11677)

Bumps [github.com/itchyny/gojq](https://github.com/itchyny/gojq) from 0.12.9 to 0.12.10.
- [Release notes](https://github.com/itchyny/gojq/releases)
- [Changelog](https://github.com/itchyny/gojq/blob/main/CHANGELOG.md)
- [Commits](https://github.com/itchyny/gojq/compare/v0.12.9...v0.12.10)

---
updated-dependencies:
- dependency-name: github.com/itchyny/gojq
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* docs: bump elasticsearch version to 8.5.1 (#11771)

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

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

* chore: change logging level to Debug (#11773)

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

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

* fix: support for enabling progressive rollouts from `argocd-cmd-params-cm` (#11776)

* fix(applicationset): use consistent syntax for env vars

Signed-off-by: Nicholas Morey <[email protected]>

* fix(manifests): add new appset env var from configmap

Signed-off-by: Nicholas Morey <[email protected]>

Signed-off-by: Nicholas Morey <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: sources.ref allow hyphen and underscore (#11775)

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

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

* docs: clarify project destination possibilities (#11706)

Clarify that it's possible to reference clusters by `cluster` or by `name`.

Signed-off-by: Gaël Jourdan-Weil <[email protected]>

Signed-off-by: Gaël Jourdan-Weil <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* chore(deps): bump github.com/fsnotify/fsnotify from 1.5.1 to 1.6.0 (#11553)

Bumps [github.com/fsnotify/fsnotify](https://github.com/fsnotify/fsnotify) from 1.5.1 to 1.6.0.
- [Release notes](https://github.com/fsnotify/fsnotify/releases)
- [Changelog](https://github.com/fsnotify/fsnotify/blob/main/CHANGELOG.md)
- [Commits](https://github.com/fsnotify/fsnotify/compare/v1.5.1...v1.6.0)

---
updated-dependencies:
- dependency-name: github.com/fsnotify/fsnotify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore: get image digest in seperate step (#11778)

* chore: get image digest in seperate step

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

* Retrigger CI pipeline

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

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

* chore: fix lint error (#11788)

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

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

* chore(deps): bump golang.org/x/term from 0.1.0 to 0.3.0 (#11792)

Bumps [golang.org/x/term](https://github.com/golang/term) from 0.1.0 to 0.3.0.
- [Release notes](https://github.com/golang/term/releases)
- [Commits](https://github.com/golang/term/compare/v0.1.0...v0.3.0)

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

Signed-off-by: dependabot[bot] <[email protected]>

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

* chore(deps): bump github.com/aws/aws-sdk-go from 1.44.156 to 1.44.164 (#11791)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.156 to 1.44.164.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.44.156...v1.44.164)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* fix: set Path as empty if path is not specified for a source in multiple sources (#11756) (#11774)

* set Path as '' if path is not specified for a source in multiple sources

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

* update check for not setting value of path

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

* cleanup

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

* address comments

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

* fix lint

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

* fix lint

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

* Update ui/src/app/shared/components/revision.tsx

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

Signed-off-by: ishitasequeira <[email protected]>
Signed-off-by: Ishita Sequeira <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]>
Signed-off-by: schakrad <[email protected]>

* docs: Update bullet formatting on Progressive Rollouts.md (#11777)

The bullet list in the example format was rendering inline in the paragraph on the doc site rather than showing a bulleted list. This also makes the rest of the doc follow the same convention.

Signed-off-by: Chris Reilly <[email protected]>

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

* fix: disable rollback button for apps with multiple sources (#11785)

* disble rollback button for apps with multiple sources

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

* fix lint errors

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

* disble rollback button for apps with multiple sources

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

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

* ci: enforce semantic PR title (#11779)

* ci: enforce semantic PR title

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

* concurrency limit

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

* remove scopes

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

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

* fix: web terminal outside argocd namespace (#11166) (#11400)

* fix: web terminal outside argocd namespace (#11166)

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

* reorganize

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

* fix reference

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

* move things around, fix stuff maybe

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

* tests

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

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

* chore(deps): bump actions/cache from 3.0.11 to 3.2.0 (#11809)

Bumps [actions/cache](https://github.com/actions/cache) from 3.0.11 to 3.2.0.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/9b0c1fce7a93df8e3bb8926b0d6e9d89e92f20a7...c17f4bf4666a8001b1a45c09eb7a485c41aa64c3)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

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

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

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

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

* fix: Applications with suspended jobs now marked "Suspended" instead of "Progressing" (#11603) (#11626)

* fix: add suspended condition

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

* fix: add suspended condition

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

* Update go.sum

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

* fix: add suspended condition

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

* fix: add suspended condition

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

* Update go.sum

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

* upgrade notes for 2.6

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

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

* fix: Application's own link in UI (#11123) (#11124)

Signed-off-by: Alex Eftimie <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* fix: ui cluster server url overlaps (#11873)

Signed-off-by: Jiwon Kim <[email protected]>
Co-authored-by: Remington Breeze <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: set cluster command (#9996)

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

Signed-off-by: maheshbaliga <[email protected]>
Signed-off-by: schakrad <[email protected]>

* feat: inversion selection support for the reso…
@codechirag123
Copy link

codechirag123 commented Oct 10, 2023

Hi @vl-kp, I'm new to ArgoCD. Following your comment I updated my manifest application as I want to deploy public helm charts with my own custom-values managed in Git. Can you take a look at my application manifest and help me with what is wrong in it.

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: app
spec:
  destination:
    name: ''
    namespace: portainer
    server: 'https://kubernetes.default.svc'
  source:
    repoURL: 'https://portainer.github.io/k8s/'
    chart: portainer
    helm:
      valueFiles:
        - $values/helm/monitoring/portainer/prod-custom-values.yaml
  sources:
    - repoURL: 'https://portainer.github.io/k8s/'
      chart: portainer
      helm:
        valueFiles:
          - $values/helm/monitoring/portainer/prod-custom-values.yaml
    - repoURL: 'https://bitbucket.org/wishboxteam/duve-staging-k8s.git'
      targetRevision: HEAD
      ref: values
  project: portainer

I did not understand what to put in the source: section when I have provided everything in the sources: section. So I adde the helm chart repo for portainer.
The error that I receive is this:

Unable to create application: application spec for app is invalid: InvalidSpecError: Unable to generate manifests in : rpc error: code = Unknown desc = invalid revision '': improper constraint:

@dllegru do you have any opinion about this?

@alekseydemidov
Copy link

alekseydemidov commented Oct 10, 2023

Hello, @codechirag123,
first of all, you don't need to use source and sources at the same time, only sources will be enough.
secondly, if you use helm repo, you have to set up 'targetRevision'
for instance:

  sources:
    - repoURL: 'https://portainer.github.io/k8s/'
      chart: portainer
      targetRevision: portainer-1.0.46
      helm:
        valueFiles:
          - $values/helm/monitoring/portainer/prod-custom-values.yaml
    - repoURL: 'https://bitbucket.org/wishboxteam/duve-staging-k8s.git'
      targetRevision: HEAD
      ref: values
  project: portainer

targetRevision is usually equal repository tag (https://github.com/portainer/k8s/tree/portainer-1.0.46/charts/portainer)

@JordanYefet
Copy link

We're having the same issues as @dllegru has mentioned above.
Is there a workaround for fixing the syncing/refreshing issues? I'm clueless.

@MohammedNoureldin
Copy link

MohammedNoureldin commented Oct 24, 2023

Is the issue in point 2 mentioned by @dllegru still there? I.e., doesn't modifying the values.yaml file for a running application is recognized as out of sync?

@JordanYefet, I am wondering if you also mean the issue in point 2.

@JordanYefet
Copy link

@MohammedNoureldin
Updating the values.yaml file that is found in a different git repo doesn't trigger the argoCD application, I have to manually press the refresh twice - one for acknowledging the changes, the next one actually to apply them.

Argo version: v2.6.1+3f143c9
Build Date: 2023-02-08T18:51:05Z

Maybe is the version too old and has already been fixed in newer versions?

Current workaround:
Instead of using multiple sources, I use a single-source git repo that contains an umbrella chart. Works like a charm, but is harder to maintain.

@MohammedNoureldin
Copy link

MohammedNoureldin commented Oct 25, 2023

@JordanYefet I gave it a try yesterday, and it seems to be working. I use no Git hooks, just relying on Argo CD to pull the changes, after waiting a few minutes, it triggers a sync process and the changes are also deployed.

Probably it is your version, I am using the latest, 2.8.4

@zbloss
Copy link

zbloss commented Dec 20, 2023

Is it possible to point the source of my helm chart to a private github repository? I have two repos, one holds my helm charts and the other holds my values.yaml files both repos have been added w/ SSH Keys to ArgoCD.

@abdennour
Copy link

any update ?

@Keramblock
Copy link

any update ?

Issue was fixed more then 6 months ago. What update you are waiting for?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:core Syncing, diffing, cluster state cache enhancement New feature or request type:usability Enhancement of an existing feature workaround There's a workaround, might not be great, but exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.