* refactor: remove app destination inferrence logic
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* simplify
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more fixes
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more fixes
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* codegen
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix mocks
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* simplify
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* clusters all the way down
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* simplify
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* test
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* lint
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* tidy
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* be less radical
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
---------
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* feat: use log format config for klog (#5715)
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* use init, remove test line
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* sort imports
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix comment
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
---------
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: Graceful shutdown for the API server (#18642) (#20981)
* fix: Graceful shutdown for the API server (#18642)
Closes#18642
Implements a graceful shutdown the the API server. Without this, ArgoCD API server will eventually return 502 during rolling update. However, healthcheck would return 503 if the server is terminating.
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Init server only once, but keep re-initializing listeners
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Check error for SetParamInSettingConfigMap as needed after fresh master
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Prevent a data race
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Remove unused variable, don't pass lock when not necessary
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Try overriding URL instead of additional URLs
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Use a more specific url
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
---------
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Use a custom signal for graceful restart
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Re-run tests
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
---------
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: Graceful shutdown for the API server (#18642)
Closes#18642
Implements a graceful shutdown the the API server. Without this, ArgoCD API server will eventually return 502 during rolling update. However, healthcheck would return 503 if the server is terminating.
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Init server only once, but keep re-initializing listeners
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Check error for SetParamInSettingConfigMap as needed after fresh master
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Prevent a data race
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Remove unused variable, don't pass lock when not necessary
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Try overriding URL instead of additional URLs
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Use a more specific url
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
---------
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Adding skipSchemaValidation flag
Signed-off-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
* Adding specific test for skip schema validation
Signed-off-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
* Fix merge conflict
Signed-off-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
* Fixing index.yaml to support helm 3.16
Signed-off-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
---------
Signed-off-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
Co-authored-by: Daniel Mosesson (soandos) <danielmosesson@gmail.com>
* fix: api server is trying to list cluster wide secrets while generating apps
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
* Revert "fix: Fix argocd appset generate failure due to missing clusterrole (#20162)"
This reverts commit fad534bcfe.
---------
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
* chore: rename force-prompts-enabled to prompts-enabled flag name
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* chore: rename force-prompts-enabled to prompts-enabled flag name
Signed-off-by: pashakostohrys <pavel@codefresh.io>
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* chore: use testify instead of testing.Fatal or testing.Error in cmd
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
* chore: use testify instead of testing.Fatal or testing.Error in cmd
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
---------
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
* update azure/kubelogin to address CVE
Signed-off-by: Jason Hall <jason@chainguard.dev>
* actually emit token
Signed-off-by: Jason Hall <jason@chainguard.dev>
* update deps, go mod tidy
Signed-off-by: Jason Hall <jason@chainguard.dev>
* fix go.sum
Signed-off-by: Jason Hall <jason@chainguard.dev>
* bcho's suggestion
Signed-off-by: Jason Hall <jason@chainguard.dev>
---------
Signed-off-by: Jason Hall <jason@chainguard.dev>
* Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)"
This reverts commit 33ecbbc4a8.
* feat: add prompting to confirm project role deletion
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Revert "Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)""
This reverts commit cecf8f49fd.
* feat: add prompting to confirm project role deletion
Signed-off-by: pashakostohrys <pavel@codefresh.io>
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)"
This reverts commit 33ecbbc4a8.
* feat: add prompting to confirm project deletion
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Revert "Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)""
This reverts commit cecf8f49fd.
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)"
This reverts commit 33ecbbc4a8.
* feat: add prompting to confirm app resource deletion
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Revert "Revert "fix(ui): Prevent versionId 0 Error on New Applications with No History (#20574)""
This reverts commit cecf8f49fd.
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* Add `Prompt`, with `prompts.enabled` setting in `argocd-cm`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Make `SettingsOpts`, `SettingsOpts.ArgocdCMPath`, `SettingsOpts.CreateSettingsManager()` and `commandContext.CreateSettingsManager()` exportable
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add `prompt_test.go`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Refactor `NewPrompt()` and move into new package `utils`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Update `NewPrompt()` to use local config
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add `NewConfigurationCommand()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Move `prompt_test.go`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Remove `prompt_test.go` for now
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add back and update `prompt_test.go`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add `configuration_test.go`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix linting issues
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix linting issues
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Reverse early-termination logic in `Confirm()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Rename `ArgocdCMPath` to `argocdCMPath`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Rename `SettingsOpts` to `settingsOpts`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Rename `CreateSettingsManager()` to `createSettingsManager()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Rename `configuration` to `configure`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Further rename `configuration` to `configure`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Remove redundant Argo CD ConfigMap logic
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix terminal output spacing
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Make `argocd configure` use local config value as the default value for `--prompts-enabled`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add global CLI flag `--force-prompts-enabled`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Update existing `prompt_test.go` test cases
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add test case for `(p *Prompt).Confirm()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add test cases for `GetBoolFlagWithFallback()` to `env_test.go`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Format imports
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Further format imports and remove unused variable
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Again format imports
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add fallback to `GetPromptsEnabled()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix bug in `GetPromptsEnabled()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix missing import
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add auto-generated docs for `argocd configure`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add auto-generated docs for new global CLI flag `--force-prompts-enabled`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Update `NewPrompt()` to receive a `bool` rather than `*apiclient.ClientOptions` arg
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Remove arg `fallback` from `GetPromptsEnabled()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add arg `useCLIOpts bool` to `GetPromptsEnabled()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Make `config.LoadFlags()` exportable
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add tests for `GetPromptsEnabled()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix linting errors in tests
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Run `make codegen-local`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
---------
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
The argocd app diff command returns 1 if a difference is found. In related issues,
they want to return an error code that is distinguishable from common errors.
However, changing the existing behavior is likely to break user's automation code.
So we want to provide an explicit option(--diff-exit-code) to specify an error code.
Related: #3588
Signed-off-by: Eugene Kim <eugene70kim@gmail.com>
* fixed doc comments and added unit tests
Signed-off-by: anandf <anjoseph@redhat.com>
* Added comments for the newly added unit tests
Signed-off-by: anandf <anjoseph@redhat.com>
* Refactored method name to deriveServiceAccountToImpersonate
Signed-off-by: anandf <anjoseph@redhat.com>
* Using const name in return value
Signed-off-by: anandf <anjoseph@redhat.com>
* Added unit tests for argocd proj add-destination-service-accounts
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed failing e2e tests
Signed-off-by: anandf <anjoseph@redhat.com>
* Fix linting errors
Signed-off-by: anandf <anjoseph@redhat.com>
* Using require package instead of assert and fixed code generation
Signed-off-by: anandf <anjoseph@redhat.com>
* Removed parallel execution of tests for sync with impersonate
Signed-off-by: anandf <anjoseph@redhat.com>
* Added err checks for glob validations
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed e2e tests for sync impersonation
Signed-off-by: anandf <anjoseph@redhat.com>
* Using consistently based expects in E2E tests
Signed-off-by: anandf <anjoseph@redhat.com>
* Added more unit tests and fixed go generate
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed failed lint errors, unit and e2e test failures
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed goimports linter issue
Signed-off-by: anandf <anjoseph@redhat.com>
* Added code comments and added few missing unit tests
Signed-off-by: anandf <anjoseph@redhat.com>
* Added missing unit test for GetDestinationServiceAccounts method
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed goimports formatting with local for project_test.go
Signed-off-by: anandf <anjoseph@redhat.com>
* Corrected typo in a field name additionalObjs
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed failing unit tests
Signed-off-by: anandf <anjoseph@redhat.com>
---------
Signed-off-by: anandf <anjoseph@redhat.com>
* feat(appset): Add a cache layer for Argo Projects to speed-up application validation
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Use local client rather than custom cache
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Clean go.mod
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Merge master
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Fix after merging master
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Initialize appProject variable inside loop
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Remove unused ArgoAppClientset field
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* Fix linter issue
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
---------
Signed-off-by: Philippe Da Costa <pdacosta@gmail.com>
* chore: transmit manifest-generate-path resources to the cmp-server for plugin-based applications
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* use SecureJoin
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* make cmp manifests generation using manifest generate path annotation configurable by environment variable
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* fix missing doc running codegen-local
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* set reposerver.plugin.enable.manifests.generation.using.annotations false by default
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* define ARGOCD_REPO_SERVER_PLUGIN_ENABLE_GENERATE_MANIFESTS_USING_MANIFEST_GENERATE_PATHS_ANNOTATION properly
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* define ARGOCD_REPO_SERVER_PLUGIN_ENABLE_GENERATE_MANIFESTS_USING_MANIFEST_GENERATE_PATHS_ANNOTATION properly
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* Fix conflict
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* autogenerate install manifests
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* add note about common root path calculation for manifest paths annotation
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* log common root path calculated
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* app path must be the lower common path
Signed-off-by: Javier Solana <javier.solana@cabify.com>
* tweaks
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
---------
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Javier Solana <javier.solana@cabify.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* feat: option to disable writing k8s events
optioned to write logs for k8s events.
Each is passed as an environment variable and defaults to true,
disabling it requires explicitly setting the option to false.
Signed-off-by: Jack-R-lantern <tjdfkr2421@gmail.com>
* feat: option to disable writing k8s events
fix unit test
- application_test
- applicationset_test
- project_test
- appcontroller_tes
- audit_logger_test
Signed-off-by: Jack-R-lantern <tjdfkr2421@gmail.com>
* rebase
Signed-off-by: Jack-R-lantern <tjdfkr2421@gmail.com>
---------
Signed-off-by: Jack-R-lantern <tjdfkr2421@gmail.com>
* feat: allow auth token to be passed in via env
Allows the authentication token to come from environment if it's not found in cli args.
Signed-off-by: Nathan Shaaban <86252985+ctrlaltf24@users.noreply.github.com>
* chore: generate cli docs
Signed-off-by: Nathan Shaaban <86252985+ctrlaltf24@users.noreply.github.com>
---------
Signed-off-by: Nathan Shaaban <86252985+ctrlaltf24@users.noreply.github.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
* Add proxy url arg for repocreds command.
Co-authored-by: Li Wang <li.wang3@fmr.com>
Signed-off-by: Miao Haoda <Haoda.Miao@fmr.com>
* commit the results of clidocsgen
Signed-off-by: Miao Haoda <Haoda.Miao@fmr.com>
---------
Signed-off-by: Miao Haoda <Haoda.Miao@fmr.com>
Co-authored-by: Li Wang <li.wang3@fmr.com>
Closes#13096
Implement a new metric exposing Applications conditions.
This is particularly useful for SRE teams to be able
to setup alerts on issues that aren't displayed via
"health_status" and "sync_status" in the metric "argocd_app_info".
Signed-off-by: Foyer Unix <foyerunix@foyer.lu>
Co-authored-by: Foyer Unix <foyerunix@foyer.lu>
* Implementation of app sync with impersonation support
Signed-off-by: anandf <anjoseph@redhat.com>
* negation test
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* Update doc comments to remove server name as its not supported.
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
Signed-off-by: Anand Francis Joseph <anandfrancis.joseph@gmail.com>
* Update glob pattern check for matching destinations.
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
Signed-off-by: Anand Francis Joseph <anandfrancis.joseph@gmail.com>
* Corrected the code comments for namespace field and destination matching logic
Signed-off-by: anandf <anjoseph@redhat.com>
* Added missing generated files
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed golint errors caused due to to gofumpt validations
Signed-off-by: anandf <anjoseph@redhat.com>
* Fix golint errors with unit test code
Signed-off-by: anandf <anjoseph@redhat.com>
* Updated the go import ordering with local packages at the end
Signed-off-by: anandf <anjoseph@redhat.com>
* Addressed review comments
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed ES lint error caused due to missing class
Signed-off-by: anandf <anjoseph@redhat.com>
* Updated the documentation to address the review comments
Signed-off-by: anandf <anjoseph@redhat.com>
* Simplified the sync code and improved logs and error handling
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed E2E tests to fail when no sa is configured
Signed-off-by: anandf <anjoseph@redhat.com>
* Updated help message generated for CLI commands
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed failing tests due to default service account not used for sync operation
Signed-off-by: anandf <anjoseph@redhat.com>
* Fixed the error message when sync fails due to no matching sa
Signed-off-by: anandf <anjoseph@redhat.com>
* Removed repeating logs and added impersonation fields to logger
Signed-off-by: anandf <anjoseph@redhat.com>
* Made changes in the proposal to match the behaviour when no matching sa is found
Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>
---------
Signed-off-by: anandf <anjoseph@redhat.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Anand Francis Joseph <anandfrancis.joseph@gmail.com>
Signed-off-by: Anand Francis Joseph <anjoseph@redhat.com>
Co-authored-by: Mangaal <angommeeteimangaal@gmail.com>
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
This update refactors the `IncludeResource` logic to improve accuracy in resource filtering.
Also, it includes new test cases to ensure the correctness of the changes.
Fixes: #17397
Changes:
- Rewritten `IncludeResource` function for better logic flow.
- Added unit tests to validate the new behavior.
- Found that two existing test cases were incorrect and fixed them. A closer review of this part is needed.
Signed-off-by: Eugene Kim <eugene70kim@gmail.com>
* chore: remove unused params
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
---------
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* add support for all namespaces for apps and appsets in export and import
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* cleanup accidental merge issue
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* address comments and run lint
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* small fix
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* switch to make for slice
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* remove new line in command flags
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
---------
Signed-off-by: Ilia Medvedev <ilia.medvedev@codefresh.io>
* feat(helm): support `--kube-version`, `--api-versions`, and `--namespace` (#19325)
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* codegen
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* handle kube and api versions per-source
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* lint
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix nil ref error
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix nil ref error due to not handling multi-source in verify path
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* change casing to be more consistent with helm
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Kustomize too
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* simplify verify path
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* add e2e tests for kustomize
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* reorder for consistency, and add more tests
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* Update docs/operator-manual/application.yaml
Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
---------
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
* Fix: wrong application name passed into function
The application name which was passed into the function
was a namespace-qualified name for the application, but
actually a simple non-qualified name was needed.
This leads to issue on the kube-apiserver side
where qualified app name is used to retrieve
the resource from a lister, leading to a cache-miss
basically, since app was looked up by namespace/namespace/appname
key, instead of namespace/appname key.
Signed-off-by: TheCoolDrop <vanio.begic123@gmail.com>
* chore: Fix the name of the application used with tree=detailed output
Signed-off-by: TheCoolDrop <vanio.begic123@gmail.com>
---------
Signed-off-by: TheCoolDrop <vanio.begic123@gmail.com>
Closes#18923
There are some gaps in debugging information for long reconciliations. Fill in a lot of those gaps by adding more debug logs with timing information about different code execution steps.
Also, fix a flaky test in app_test.go.
Signed-off-by: Andrii Korotkov <andrii.korotkov@verkada.com>
* Add flag `--sso-launch-browser` to `login` and `relogin` commands
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Run `make codegen`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add `ssoBrowserFlow()` and refactor `oauth2Login()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Update references to "default browser" to "system default browser"
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Add test case for `ssoBrowserFlow()` with `ssoLaunchBrowser` as `false`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Run `make codegen`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Update `Test_ssoBrowserFlow_ssoLaunchBrowser_false()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix typo in unit test
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Fix linting issues
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
* Rename `ssoBrowserFlow()` to `ssoAuthFlow()`
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
---------
Signed-off-by: David Wu <155603967+david-wu-octopus@users.noreply.github.com>
Co-authored-by: pasha-codefresh <pavel@codefresh.io>
* feat: support log level in cmp plugin, and add possibility to trace generated manifest
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* feat: support log level in cmp plugin, and add possibility to trace generated manifest
Signed-off-by: pashakostohrys <pavel@codefresh.io>
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
* try putting profiles in different directories
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* does this upload the whole directory?
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* create directories and only run one test
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* shut everything down before uploading coverage
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* don't fail on goreman failure
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* sleep 30
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* always upload server output
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: implement graceful shutdown for api server
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* Shutdown servers in parallel
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* add noop shutdown function on api server constructor
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* fix waitgroup
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* more attempts to gracefully shut down
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more tests
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* cleanup
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* revert graceful shutdown work, needs more work
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more graceful shutdown cleanup
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* lint
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* feat: adding go coverage to e2e tests
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
---------
Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* Adds consistent hashing with bound loads sharding algorithm
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Make the assignement consistent accross all clusters
- The assignment or running of the algorithm has to be consistent across all the clusters. Changed the function to return a map where the consistent hash will be used to build the map
- Modifications to the createConsistentHashsingWithBoundLoads function. This will create the map for cluster to shard. Note that the list must be consistent across all shards so that is why the cluster list must be sorted before going through the consistent hash algorithm
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Extracting constant and simplifying boolean expression
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Update docs: consistent-hashing sharding algorithm
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
---------
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Update repository.go
Signed-off-by: BarAshkenazi <barashkenazi2@gmail.com>
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* add dir exclusion pattern
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
* support hidden directories in repo server
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
---------
Signed-off-by: BarAshkenazi <barashkenazi2@gmail.com>
Signed-off-by: Bar <bar.ashkenazi@appsflyer.com>
Co-authored-by: Bar <bar.ashkenazi@appsflyer.com>
So that log messages are propagated up, this caused issues for me when
trying to diagnose an issue linked to issue #17811. I've just copied the
approach used e.g. in `controller/appcontroller.go` when calling
`argodiff.NewDiffConfigBuilder`.
Sample output before this change:
$ argocd --loglevel debug --server argocd-system.dev.aws.uw.systems app diff dev-enablement/dev-enablement
===== apps/Deployment dev-enablement/go-docs ======
272c272
< cpu: "0"
---
> cpu: 0m
275c275
< cpu: "0"
---
> cpu: 0m
Output with this change:
$ argocd --loglevel debug --server argocd-system.dev.aws.uw.systems app diff dev-enablement/dev-enablement
INFO[0000] Could not unmarshal to object of type apps/v1, Kind=Deployment: json: unknown field "restartPolicy"
INFO[0000] Could not unmarshal to object of type apps/v1, Kind=Deployment: json: unknown field "restartPolicy"
===== apps/Deployment dev-enablement/go-docs ======
272c272
< cpu: "0"
---
> cpu: 0m
275c275
< cpu: "0"
---
> cpu: 0m
INFO[0000] Could not create new object of type argoproj.io/v1alpha1, Kind=Application: no kind "Application" is registered for version "argoproj.io/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
INFO[0000] Could not create new object of type argoproj.io/v1alpha1, Kind=Application: no kind "Application" is registered for version "argoproj.io/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
INFO[0000] Could not create new object of type traefik.containo.us/v1alpha1, Kind=IngressRoute: no kind "IngressRoute" is registered for version "traefik.containo.us/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
INFO[0000] Could not create new object of type traefik.containo.us/v1alpha1, Kind=IngressRoute: no kind "IngressRoute" is registered for version "traefik.containo.us/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
INFO[0000] Could not create new object of type traefik.containo.us/v1alpha1, Kind=IngressRoute: no kind "IngressRoute" is registered for version "traefik.containo.us/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
INFO[0000] Could not create new object of type traefik.containo.us/v1alpha1, Kind=IngressRoute: no kind "IngressRoute" is registered for version "traefik.containo.us/v1alpha1" in scheme "pkg/runtime/scheme.go:100"
Signed-off-by: Matt Hughes <mhughes@uw.co.uk>
* instead if ranging over the map varHistory to print the history, inroduced a string array varHistoryKeys
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* update unit test expectation, remove new line in the beginning
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
---------
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* update argocd app history command to print app history group by thier sources along with all the REVISIONS
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* upadte unit test to ahve both Source and Sources and update function to overlooked source if sources is persent
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* remove magic no 7 and introduc a variable MAX_ALLOWED_REVISIONS
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* remove extra unit test
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
* remove extra unit test TestPrintApplicationHistoryTableForWhenBothSourcesAndSourceFiledsExist()
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
---------
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
* Adding app list to sharding cache
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Add shard by apps test
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Fix lint
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Add coverage to test
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Fix lint
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Converted cluster/app accesors to private, add apps-in-any-namespace suport in shardingcache init, added read lock to GetAppDistribution
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* Fix tests
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
---------
Signed-off-by: Andrew Lee <andrewkl@enclavenet.com>
* feat: wait until resources are deleted
Signed-off-by: MichaelMorris <michael.morris@est.tech>
* Added unit and e2e test
Signed-off-by: MichaelMorris <michael.morris@est.tech>
---------
Signed-off-by: MichaelMorris <michael.morris@est.tech>
* fix: infer correct shard in statefulset setup
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix the case if only a single replica
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: resolving pointer on shard compare
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: add readlock for cluster accessor
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: use defer to protect access of 'shard'
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: revert locking in getclusteraccessor
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: handle nil shard case
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: handle any nil shard value as false
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: handle nil case and fix another missing pointer dereference
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* revert
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: added tests and fixed some behaviour bugs
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* test: add test to validate that Shard value is not overriden
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* fix: added tests and fixe the case when server is changed inside a secret
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
* tests: add test cases for infering the shard logic
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
---------
Signed-off-by: Lukas Wöhrl <lukas.woehrl@plentymarkets.com>
useful for argocd-servers which are not running in AWS and want to add multiple EKS clusters using
separate keys instead of assuming roles
#16766
Signed-off-by: Isaac Gaskin <isaac.gaskin@circle.com>
Co-authored-by: Soumya Ghosh Dastidar <44349253+gdsoumya@users.noreply.github.com>
* feat(sharding): use a cache
Signed-off-by: Alexandre Gaudreault <alexandre.gaudreault@logmein.com>
* cluster cmd
Signed-off-by: Alexandre Gaudreault <alexandre.gaudreault@logmein.com>
* - Assign shard 0 to in-cluster cluster and nil check updates
- Caching clusters while sharding: Fixing unit tests
- Update generated docs
- Debug e2e tests
- Default the shardNumber to the number of replicas if it is calculated to a higher value
- defered Unlock only when a lock is set
- Disabling temporarly other versions of k3s to check if e2e passes
- Do not fail if hostname format is not abc-n
- Fix unit test and skip some e2e
- Skip TestGitSubmoduleHTTPSSupport test
- Remove breaking defer c.lock.Unlock()
- Reverting testing all k3s version
- Default sharding fix
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* fixes related to code review: renaming structure param, moving db initialisation
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Code review
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Set default shard to 0
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Set different default value for Sts and Deployment mode
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Expose ClusterShardingCache
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Removing use of argoDB.db for DistributionFunction
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Update generated documentation
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
* Fix comment about NoShardingDistributionFunction and NoShardingAlgorithm
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
---------
Signed-off-by: Alexandre Gaudreault <alexandre.gaudreault@logmein.com>
Signed-off-by: Akram Ben Aissi <akram.benaissi@gmail.com>
Co-authored-by: Alexandre Gaudreault <alexandre.gaudreault@logmein.com>
* self service notification
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification engine
Signed-off-by: May Zhang <may_zhang@intuit.com>
* re-trigger build
Signed-off-by: May Zhang <may_zhang@intuit.com>
* self service notification
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification engine
Signed-off-by: May Zhang <may_zhang@intuit.com>
* re-trigger build
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification enginer version
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification enginer version
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* add back checkAppNotInAdditionalNamespaces
Signed-off-by: May Zhang <may_zhang@intuit.com>
* add cm and secret to clusterRole
Signed-off-by: May Zhang <may_zhang@intuit.com>
* if applicationNamespaces is not used, then use namespaced appClient
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix merge conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix doc and test based on review
Signed-off-by: May Zhang <may_zhang@intuit.com>
* self service notification
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification engine
Signed-off-by: May Zhang <may_zhang@intuit.com>
* re-trigger build
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* self service notification
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* revert back the changes for redis-ha
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification engine
Signed-off-by: May Zhang <may_zhang@intuit.com>
* re-trigger build
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix conflict
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification enginer version
Signed-off-by: May Zhang <may_zhang@intuit.com>
* update notification enginer version
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fixing go tidy
Signed-off-by: May Zhang <may_zhang@intuit.com>
* add back checkAppNotInAdditionalNamespaces
Signed-off-by: May Zhang <may_zhang@intuit.com>
* add cm and secret to clusterRole
Signed-off-by: May Zhang <may_zhang@intuit.com>
* if applicationNamespaces is not used, then use namespaced appClient
Signed-off-by: May Zhang <may_zhang@intuit.com>
* fix doc and test based on review
Signed-off-by: May Zhang <may_zhang@intuit.com>
* disable defining and using secrets within notification templates for self-service
Signed-off-by: May Zhang <may_zhang@intuit.com>
* tweaks
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix docs formatting
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* more docs and Procfile update for local run convenience
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
---------
Signed-off-by: May Zhang <may_zhang@intuit.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
