chore: create release 0.12.2

[ishitasequeira]

What type of PR is this?
/kind chore

What does this PR do / why we need it:
create release 0.12.2

[svghadi]

Are we planning a new upstream operator release? If it is just for downstream release, I think we can ignore this PR for now. Your previous changes (#1591) should be sufficient for downstream.
We can make an exception this time i.e not release a z-stream upstream. I am working on a release automation so in future we should be able to do quick z-stream release of upstream operator.

[svghadi]

Looks like image is built for arm, should be amd64

$ docker inspect quay.io/argoprojlabs/argocd-operator@sha256:f15781e2314d6baedcfc3bfa12181213d8d609803f73faecb2b8feddf3da4383 |grep Arch
          "Architecture": "arm64",

[anandf]

Can we build multi-arch images ?

[svghadi]

Yes. I feel we should support other archs aswell. After release automation, that is on my todo. #264 (comment)

[ishitasequeira]

Do we have a make command to build a multi-arch image? I did not configure any arch-type while building the image. I think it took my default laptop arch.

[svghadi]

Right now, no. You need to modify image build commands and add --platform linux/amd64 flag.

diff --git a/Makefile b/Makefile
index 45c02a62..634356bb 100644
--- a/Makefile
+++ b/Makefile
@@ -110,7 +110,7 @@ run: manifests generate fmt vet ## Run a controller from your host.
        REDIS_CONFIG_PATH="build/redis" go run -ldflags=$(LD_FLAGS) ./cmd/main.go
 
 docker-build: test ## Build docker image with the manager.
-       $(CONTAINER_RUNTIME) build --build-arg LD_FLAGS=$(LD_FLAGS) -t ${IMG} .
+       $(CONTAINER_RUNTIME) build --platform linux/amd64 --build-arg LD_FLAGS=$(LD_FLAGS) -t ${IMG} .
 
 docker-push: ## Push docker image with the manager.
        $(CONTAINER_RUNTIME) push ${IMG}

[ishitasequeira]

Thanks @svghadi !! Will build the image

[ishitasequeira]

The image is updated on quay to be of type amd64.

[anandf]

Please update the argocd-operator image either as multi-arch or amd64 supported. Other changes look good.

[ishitasequeira]

Are we planning a new upstream operator release?

I think the fix that was merged should have a specific argocd-operator release as well for the feature to work if used only via argocd-operator. Although I am okay to skip publishing the image if needed.

[anandf]

LGTM

[svghadi]

LGTM. Thanks