fix: validate destination request always fails#405
Merged
oleksandr-codefresh merged 1 commit intosync-3.0.2from Jun 19, 2025
Merged
fix: validate destination request always fails#405oleksandr-codefresh merged 1 commit intosync-3.0.2from
oleksandr-codefresh merged 1 commit intosync-3.0.2from
Conversation
ATGardner
approved these changes
Jun 19, 2025
…ster, we can simple rely on error returned from this request Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com>
oleksandr-codefresh
force-pushed
the
CR-28173-validate-destionation-fails
branch
from
8936ad6 to
dee4a5e
Compare
Codecov ReportAttention: Patch coverage is
📢 Thoughts on this report? Let us know! |
ppapapetrou76
pushed a commit
that referenced
this pull request
Sep 16, 2025
…ster, we can simply rely on error returned from this request (#405) Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com>
ppapapetrou76
added a commit
that referenced
this pull request
Sep 18, 2025
* chore: sync all codefresh code changes into v3.0.2 (#397) * chore: sync all codefresh code changes into v3.0.2 without event-reporter related changes Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * removed cf script Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * upgraded git-lfs to 3.6.1 in Dockerfile (#386) Signed-off-by: reggie-k <regina.voloshin@codefresh.io> Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * fixed webstorm go.mod issue Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * e2e: improved error logs Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * fixed changes on generated files Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * chore: replace heptio-images with argocd-e2e-container (argoproj#23040) Signed-off-by: nitishfy <justnitish06@gmail.com> Signed-off-by: Nitish Kumar <justnitish06@gmail.com> (cherry picked from commit 309acd1) Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * feat: upgraded github.com/expr-lang/expr from 0.16.9 to 0.17.0 Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * e2e [TestTrackAppStateAndSyncApp / TestNewStyleResourceActionMixedOk / TestNewStyleResourceActionPermitted / TestNamespacedPermissions]: added wait for sync operation Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> --------- Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> Signed-off-by: reggie-k <regina.voloshin@codefresh.io> Co-authored-by: Regina Voloshin <regina.voloshin@codefresh.io> Co-authored-by: Nitish Kumar <justnitish06@gmail.com> # Conflicts: # .github/workflows/ci-build.yaml # cmd/argocd/commands/app_test.go # go.mod # go.sum # manifests/base/kustomization.yaml # manifests/core-install-with-hydrator.yaml # manifests/core-install.yaml # manifests/core-install/kustomization.yaml # manifests/ha/base/kustomization.yaml # manifests/ha/install-with-hydrator.yaml # manifests/ha/install.yaml # manifests/ha/namespace-install-with-hydrator.yaml # manifests/ha/namespace-install.yaml # manifests/install-with-hydrator.yaml # manifests/install.yaml # manifests/namespace-install-with-hydrator.yaml # manifests/namespace-install.yaml # pkg/apiclient/application/application.pb.go # pkg/apiclient/application/application.pb.gw.go # pkg/apis/application/v1alpha1/generated.pb.go # reposerver/apiclient/mocks/RepoServerServiceClient.go # reposerver/apiclient/repository.pb.go # server/application/application.proto # util/git/mocks/Client.go * fix(validateDestination query): as we moved to argo.GetDestinationCluster, we can simply rely on error returned from this request (#405) Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> * removed curl from image (#406) Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * feat: Add GitHub API metrics (#404) * added github api metrics Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * fix(docs): fix applicationsetcontroller.enable.github.api.metrics to false in docs cm (argoproj#23516) Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * fix: Account for batch event processing in e2e tests (argoproj#22356) Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com> --------- Signed-off-by: reggie-k <regina.voloshin@codefresh.io> Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com> Co-authored-by: Andrii Korotkov <137232734+andrii-korotkov-verkada@users.noreply.github.com> * cherry-pick 1b48f36 Upgrade ubuntu base image to latest 25.04 digest (#407) Signed-off-by: reggie-k <regina.voloshin@codefresh.io> Co-authored-by: dudinea <eugene.doudine@octopus.com> * feat: CR-29912 manual cherry pick app set pr generator return 0 results if the repo does not exist (#409) * manually added the changes Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * pull request functionality Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * pull request functionality Signed-off-by: reggie-k <regina.voloshin@codefresh.io> --------- Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * feat: move pkg/exec in-tree and add fatal timeout upgrade with SIGKILL to ARGO_EXEC_TIMEOUT (#419) * chore: move pkg/exec in-tree (argoproj#22175) (argoproj#22460) Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * fix: add fatal timeout upgrade with SIGKILL to ARGO_EXEC_TIMEOUT (closes argoproj#20785, argoproj#18478) (argoproj#22713) Signed-off-by: Hazel Sudzilouski <dsudzilouski@olin.edu> Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com> Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com> --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Signed-off-by: Hazel Sudzilouski <dsudzilouski@olin.edu> Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Hazel Sudzilouski <t-danielsu@microsoft.com> Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com> * feat: CR-30512 stop using bitnami images (#420) * removed references from all the images except for astra healthcheck Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * removed reposerver/repository/testdata/helm-with-local-dependency/.argocd-helm-dep-up Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * reverted health check references since they are treated as text Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * reverted health check references since they are treated as text Signed-off-by: reggie-k <regina.voloshin@codefresh.io> --------- Signed-off-by: reggie-k <regina.voloshin@codefresh.io> * chore: bumps redis to 8.x (#422) * bumps Docker test container to redis 8 * bumps redis version to 8.2.1 Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> * use a previous version of go-redis Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> --------- Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> * upgrade sqlite in docker image to address CVE-2025-6965 (#425) * final changes after rebase Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> * final changes after rebase Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> * address new linter issues Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> --------- Signed-off-by: oleksandr-codefresh <oleksandr.saulyak@octopus.com> Signed-off-by: reggie-k <regina.voloshin@codefresh.io> Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Signed-off-by: Hazel Sudzilouski <dsudzilouski@olin.edu> Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com> Signed-off-by: Patroklos Papapetrou <ppapapetrou76@gmail.com> Co-authored-by: Oleksandr Saulyak <oleksandr.saulyak@octopus.com> Co-authored-by: Regina Voloshin <regina.voloshin@codefresh.io> Co-authored-by: Nitish Kumar <justnitish06@gmail.com> Co-authored-by: Andrii Korotkov <137232734+andrii-korotkov-verkada@users.noreply.github.com> Co-authored-by: dudinea <eugene.doudine@octopus.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Hazel Sudzilouski <t-danielsu@microsoft.com> Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We previously used validateDestination from OSS, which adds missing fields (server / name) to provided spec.destionation of an application. In 3.x.x validateDestination method was removed, and new one already performs all check and throws not found error in case if server of name are not correct.