Signed-off-by: Kim Brose <2803622+HarHarLinks@users.noreply.github.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: Ability to disable Server Side Apply on individual resource level
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* fix: Ability to disable Server Side Apply on individual resource level
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* fix: Ability to disable Server Side Apply on individual resource level
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* chore: improve doc
Signed-off-by: pashakostohrys <pavel@codefresh.io>
---------
Signed-off-by: pashakostohrys <pavel@codefresh.io>
* docs: Consistent proper case of Kubernetes
Signed-off-by: Adam Harvey <33203301+adamdmharvey@users.noreply.github.com>
* docs: Fix minor typo
Signed-off-by: Adam Harvey <33203301+adamdmharvey@users.noreply.github.com>
* fix: Proper case Kubernetes
Signed-off-by: Adam Harvey <33203301+adamdmharvey@users.noreply.github.com>
---------
Signed-off-by: Adam Harvey <33203301+adamdmharvey@users.noreply.github.com>
* chore(deps): upgrade k8s version and client-go
Signed-off-by: fengshunli <1171313930@qq.com>
* revert bad merge
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix codegen
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix codegen
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: check for double definition
As found in #13965 (and as a follow-up to #13999), we also need to
define what happens if _both_ managedNamespaceMetadata _and_ an
Application manifest are both defined for the same namespace.
The idea here is that if that happens, we emit an
`ApplicationConditionRepeatedResourceWarning`, and set the sync status
to `Unknown`, since it's unclear what is supposed to happen.
The user will then have the option of removing one of the two
definitions.
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* fix: check for double definition
A simpler fix - don't add a managed namespace to the targetObjs list if
a namespace already exists in the application source.
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* build: extra space in doc
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* build: extra space in doc, again
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* chore: bump gitops-engine
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
---------
Signed-off-by: fengshunli <1171313930@qq.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
Co-authored-by: fengshunli <1171313930@qq.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* namespace labels
Signed-off-by: pashavictorovich <pavel@codefresh.io>
* create namespace should support annotations
Signed-off-by: pashavictorovich <pavel@codefresh.io>
* handle also modification hook
Signed-off-by: pashavictorovich <pavel@codefresh.io>
* regenerate entity on modify hook
Signed-off-by: pashavictorovich <pavel@codefresh.io>
* manifests
Signed-off-by: pashavictorovich <pavel@codefresh.io>
* feat: enable metadata to be set on namespaces
This builds upon the work that @pasha-codefresh did in #10288.
The main differences between this PR and the previous one is that we use
SSA to diff between different versions of the namespace, as well as
having a slightly different API in gitops-engine for setting the
namespace modifier.
We now also set the ownership of the namespace in ArgoCD.
Closes#4628Closes#6215Closes#7799
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* fix: don't always track namespaces
For now, only allow namespaces managed with `managedNamespaceMetadata`
to have tracking set by Argo. Ideally we'd like new namespaces to also
be tracked by Argo, but there's currently an issue with a failing
integration test.
Also wrap error message if setting the app instance errors on the
namespace.
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* fix: always return true with `hasManagedMetadata`
If `hasManagedMetadata` is set, `true` should always be returned.
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* docs: add clarifying docs on resource tracking
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* style: pr tweaks
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* fix: re-add label unsetting
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
* Update gitops-engine to current master
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Signed-off-by: pashavictorovich <pavel@codefresh.io>
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Co-authored-by: pashavictorovich <pavel@codefresh.io>
Co-authored-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* docs: remove more version notes - rely on docs versioning
Signed-off-by: CI <michael@crenshaw.dev>
* missed some things
Signed-off-by: CI <michael@crenshaw.dev>
Signed-off-by: CI <michael@crenshaw.dev>
* docs: Add example about how to patch with SSA syncs
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
* docs: minor fixes
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>
This part of the documentation is not clear enough. It takes a lot of digging to actually find a clear explanation of what those policies are bringing to the end-user.
Source: https://github.com/argoproj/argo-cd/discussions/6239
Signed-off-by: Miletskyi Vladyslav <vladyslav.miletskyi@eu.equinix.com>
