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

Reflector annotations disappearing on source secrets #138

Closed
gpsa opened this issue Jan 14, 2021 · 2 comments
Closed

Reflector annotations disappearing on source secrets #138

gpsa opened this issue Jan 14, 2021 · 2 comments
Labels

Comments

@gpsa
Copy link

gpsa commented Jan 14, 2021

I've followed these instructions:

  1. Annotate the source secret or configmap
    Add reflector.v1.k8s.emberstack.com/reflection-allowed: "true"
    Add reflector.v1.k8s.emberstack.com/reflection-allowed-namespaces: ""
  2. reflector.v1.k8s.emberstack.com/reflects: "/" to the destination secret.

It reflected the configurations to the destination secret, but the annotations above just disappeared. Probably because of that kubernetes-reflector can't keep them syncronized and now displays cannot be reflected to staging/ingress-cert-test.

Scenario:

  • kubernetes-reflector is installed on default namespace
  • cert-manager secrets are on cert-manager namespace
  • destination secret is on staging/ingress-cert-test

LOGS:
2021-01-14 07:19:42.673 +00:00 [INF] (ES.Kubernetes.Reflector.Secrets.SecretMirror) Reflected Secret cert-manager/ingress-cert-test (version: 31997466) to staging/ingress-cert-test 2021-01-14 07:19:53.464 +00:00 [INF] (ES.Kubernetes.Reflector.CertManager.SecretEtcher) Patched Secret cert-manager/ingress-cert-test to match Certificate cert-manager/ingress-cert-test reflector.v1.k8s.emberstack.com annotations 2021-01-14 07:19:53.512 +00:00 [WRN] (ES.Kubernetes.Reflector.Secrets.SecretMirror) Secret cert-manager/ingress-cert-test (version: 31997505) cannot be reflected to staging/ingress-cert-test. Reflection not allowed by source Secret 2021-01-14 07:21:28.476 +00:00 [INF] (ES.Kubernetes.Reflector.Secrets.SecretMirror) Reflected Secret cert-manager/ingress-cert-test (version: 31997839) to staging/ingress-cert-test 2021-01-14 07:21:53.476 +00:00 [INF] (ES.Kubernetes.Reflector.CertManager.SecretEtcher) Patched Secret cert-manager/ingress-cert-test to match Certificate cert-manager/ingress-cert-test reflector.v1.k8s.emberstack.com annotations reflector.v1.k8s.emberstack.com annotations 2021-01-14 07:21:53.517 +00:00 [WRN] (ES.Kubernetes.Reflector.Secrets.SecretMirror) Secret cert-manager/ingress-cert-test (version: 31997930) cannot be reflected to staging/ingress-cert-test. Reflection not allowed by source Secret 2021-01-14 07:23:52.233 +00:00 [WRN] (ES.Kubernetes.Reflector.Secrets.SecretMirror) Secret cert-manager/ingress-cert-test (version: 31997930) cannot be reflected to staging/ingress-cert-test.

@stale
Copy link

stale bot commented Jan 23, 2021

Automatically marked as stale due to no recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 23, 2021
@stale
Copy link

stale bot commented Jan 30, 2021

Automatically closed stale item.

@stale stale bot closed this as completed Jan 30, 2021
winromulus added a commit that referenced this issue Oct 16, 2021
- New multi-arch pipeline with proper tagging convention
- Removed cert-manager extension (deprecated due to new support from cert-manager) Fixes: #191
- Fixed healthchecks. Fixes: #208
- Removed Slack support links (GitHub issues only). Fixes: #199
- Simplified startup and improved performance. Fixes: #194
- Huge improvements in performance and stability. Fixes: #187 #182 #166 #150 #138 #121 #108
winromulus added a commit that referenced this issue Oct 16, 2021
- New multi-arch pipeline with proper tagging convention
- Removed cert-manager extension (deprecated due to new support from cert-manager) Fixes: #191
- Fixed healthchecks. Fixes: #208
- Removed Slack support links (GitHub issues only). Fixes: #199
- Simplified startup and improved performance. Fixes: #194
- Huge improvements in performance and stability. Fixes: #187 #182 #166 #150 #138 #121 #108
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

1 participant