Skip to content

Conversation

@hakuna-matatah
Copy link
Contributor

Moving tekton resources to another folder to fix sync issue caused by #488
When this file was created under assets, flux trying to apply this and failing to sync

{"level":"error","ts":"2025-03-10T20:48:46.930Z","logger":"controller.kustomization","msg":"Reconciliation failed after 68.634214ms, next try in 2m0s","reconciler group":"kustomize.toolkit.fluxcd.io","reconciler kind":"Kustomization","name":"kubernetes-iteration-toolkit","namespace":"tekton-pipelines","revision":"main/76b4d2e40becba185debc77c9cf5a815e3b22983","error":"failed to decode Kubernetes YAML from /tmp/kustomization-3721837646/tests/assets/eks_node_group_launch_template_al2023.yaml: missing Resource metadata"}

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Contributor

@mengqiy mengqiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hakuna-matatah hakuna-matatah merged commit 2180bca into awslabs:main Mar 10, 2025
7 checks passed
@mengqiy
Copy link
Contributor

mengqiy commented Mar 10, 2025

Providing some additional context: the flux controllers is configured to target all yaml files under tests directory. There's one yaml file that is not a k8s resource introduced in #488 under tests/assets.

Now we changed the flux controllers to only target tests/tekton-resources. As a result, it will skip tests/assets. The problem is confirmed to be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants