onepassword-operator icon indicating copy to clipboard operation
onepassword-operator copied to clipboard

Operator no longer includes annotations within the created secret

Open tomjohnburton opened this issue 3 years ago • 11 comments

Your environment

Operator Version: v1.6.0

Connect Server Version: 1.5.7

Kubernetes Version: 1.23.9

What happened?

In version 1.3.0 of the operator, any annotation I added to the operator would be propagated to the resulting secret. Now, none of my annotations are added.

What did you expect to happen?

I expected the resulting secret to have the annotations I set

Steps to reproduce

  1. Create a OnePasswordItem with annotations
  2. Deploy
  3. Validate the resulting secret doesn't have annotations
  4. Use v1.3.0 of the operator
  5. Do steps 1-3
  6. Validate annotations are there

Notes & Logs

kind: OnePasswordItem
metadata:
  annotations:
    argocd.argoproj.io/compare-options: IgnoreExtraneous
    kubed.appscode.com/sync: "true"
kind: Secret
metadata:
  annotations:
    operator.1password.io/item-path: vaults/***/items/***
    operator.1password.io/item-version: "3"

tomjohnburton avatar Dec 22 '22 13:12 tomjohnburton

I see the commit where it was changed https://github.com/1Password/onepassword-operator/commit/afa076d321c017a9f94acb17781ef64c0beb5dd6

IMO, we should allow users to add extra annotations.

One option could be to add a template field to the spec. A common practise among the k8s community

kind: OnePasswordItem
metadata:
	name: xyz
spec:
	template:
		  annotations:
		     argocd.argoproj.io/compare-options: IgnoreExtraneous
		     kubed.appscode.com/sync: "true"
			

tomjohnburton avatar Dec 23 '22 09:12 tomjohnburton

Thank you for raising this with us. We will further investigate this on our side and come back to you.

edif2008 avatar Mar 13 '23 16:03 edif2008

Hi, any progress on this, or a workaround perhaps using kustomize?

vjm avatar Jul 28 '23 14:07 vjm

I forked it and added the functionality back https://github.com/tomjohnburton/onepassword-operator

tomjohnburton avatar Aug 22 '23 15:08 tomjohnburton

Any updates on this @edif2008 ? It's still happening on versions: Operator Version: 1.8.0 Connect Server Version: 1.7.2

amaldonadomat avatar Sep 11 '23 08:09 amaldonadomat

Still not fixed? Come on....

Tipsmark avatar Aug 29 '24 14:08 Tipsmark