Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Operator no longer includes annotations within the created secret #144

Open
tomjohnburton opened this issue Dec 22, 2022 · 5 comments
Open
Labels
bug Something isn't working

Comments

@tomjohnburton
Copy link

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 tomjohnburton added the bug Something isn't working label Dec 22, 2022
@tomjohnburton
Copy link
Author

I see the commit where it was changed
afa076d

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"
			

@edif2008
Copy link
Member

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

@vjm
Copy link

vjm commented Jul 28, 2023

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

@tomjohnburton
Copy link
Author

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

@amaldonadomat
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants